298260 - Support midl.exe with version == 6.00.0366 (included in latest MS Platform SDK)

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Jeff Walden [:Waldo]]

The version of midl.exe included with the latest MS Platform SDK (the Windows
Server 2003 SP1 Platform SDK) is 6.00.0366, which isn't supported in
configure.in.    For those of us too poor to actually buy Microsoft's
development environment, this is the only version of midl.exe we have to make
builds using Microsoft's free C++ toolkit.  configure.in needs to be updated to
support midl version 6.00.0366.

Comment 1

[Christian :Biesinger (don't email me, ping me on IRC)]

note bug 249782 comment 30, and also note that the midl version from that patch
is the 3066 one

Comment 2

[Jeff Walden [:Waldo]]

Attachment: Patch

Comment 3

[Jeff Walden [:Waldo]]

(In reply to comment #1)

I didn't read the comment before posting the patch.

That said, the URL is from a comment I got from a person who said he'd made this
change and compiled successfully, so if you're willing to believe him it does
work.  I'm just interested in keeping the number of hacks I have to make to my
tree to a minimum.

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

I don't doubt that this patch works. I just mentioned that bug because it
suggests that the check can maybe be removed.

Comment 5

[cls]

Attachment: Remove midl check

Just remove the midl version check altogether since it predated the no-robust
check.

Comment 6

[cls]

The removal patch has been checked in.