The default bug view has changed. See this FAQ.
Bug 368316 (vc9-configure)

Configure fails with "Unsupported compiler" when building with VC9 (Codename "Orcas")

VERIFIED FIXED

Status

()

Core
Build Config
--
critical
VERIFIED FIXED
10 years ago
9 years ago

People

(Reporter: Gábor Stefanik, Assigned: Gábor Stefanik)

Tracking

Trunk
x86
Windows Vista
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

10 years ago
Created attachment 252916 [details] [diff] [review]
Add VC9/CL15 support to Configure.in

Configure.in is currently set to accept the compiler versions 6, 7, 7.1 and 8. As a result, trying to build with the new "Orcas" pre-release (VC9) fails with an "Unsupported Compiler" error. Before any progress can be made into supporting "Orcas", we need to extend the version check to accept the new compiler.
Flags: blocking1.9?
Attachment #252916 - Flags: superreview?(netrolller.3d)
Attachment #252916 - Flags: review?(netrolller.3d)
(Assignee)

Updated

10 years ago
Summary: Add VC9 (Visual Studio Codename "Orcas") support to configure.in → Configure fails with "Unsupported compiler" when building with VC9 (Codename "Orcas")
Flags: blocking1.9? → blocking1.9-
Comment on attachment 252916 [details] [diff] [review]
Add VC9/CL15 support to Configure.in

netroller, you should request review from the build-config owner or peers (i.e. me) to get a patch reviewed
Attachment #252916 - Flags: superreview?(netrolller.3d)
Attachment #252916 - Flags: review?(netrolller.3d)
Attachment #252916 - Flags: review?(benjamin)
(Assignee)

Comment 2

10 years ago
Take this comment as a review request. (My build with this patch is still going on, it will finish later in the evening.)
Flags: blocking1.9- → blocking1.9?
(Assignee)

Updated

10 years ago
Flags: blocking1.9?
(Assignee)

Comment 3

10 years ago
This patch seems to be worthy for checkin, by applying this patch I could build Firefox 3.0a2pre without any further changes. No regressions (except for maybe the ones caused by compiler changes). I now only need an r+sr for the patch to continue.
(Assignee)

Comment 4

10 years ago
Just tested, there are some other problems with "Orcas": Firefox works correctly on systems with "Orcas" installed, but other computers require copying of the C/C++ Runtime DLLs to the Firefox install directory. (This is due to switching to a new runtime, version 9. Microsoft's fault? Probably... But let's not forget that it's still not final. Also note that copying the runtime DLLs to system32 doesn't work, you have to copy them to the Firefox directory - and to all other programs' directories that are built with "Orcas".) Otherwise I'm accepting this formally, because I'm working on this bug.
Status: NEW → ASSIGNED
Comment on attachment 252916 [details] [diff] [review]
Add VC9/CL15 support to Configure.in

You removed the CRT_SECURE_NO_DEPRECATE and CRT_NOSTDC_NO_DEPRECATE flags from VC8.

The CRT issue is nothing new, we have the same issues with VC8. There is code in build/win32/Makefile.in to do CRT copying for release builds when WIN32_REDIST_DIR is set.
Attachment #252916 - Flags: review?(benjamin) → review-
(Assignee)

Comment 6

10 years ago
Oops... my mistake. _CRT_{SECURE|NOSTDC}_NO_DEPRECATE has been renamed to _CRT_{SECURE|NOSTDC}_NO_WARNINGS in VC9, and I've done a bit of, well, error when trying to apply that... new patch coming soon!
(Assignee)

Comment 7

10 years ago
Created attachment 253096 [details] [diff] [review]
Add VC9/CL15 support to Configure.in, v1.1

Handle deprecation warnings correctly this time. (Finally I know why I saw lots of deprecation warnings while building! I thought it's a bug in "Orcas".)

Other news: The DLL issue can be fixed by installing .NetFX 3.5 (GreenBits) from the "Orcas" installer package (that you can download from Microsoft).
Attachment #252916 - Attachment is obsolete: true
(Assignee)

Updated

10 years ago
Attachment #253096 - Flags: superreview?(netrolller.3d)
Attachment #253096 - Flags: review?(netrolller.3d)
(Assignee)

Updated

10 years ago
Assignee: nobody → netrolller.3d
Status: ASSIGNED → NEW
(Assignee)

Updated

10 years ago
Status: NEW → ASSIGNED
Stefanik: you need to put the requestee's email address in the review? field (benjamin), not your own.
(Assignee)

Comment 9

10 years ago
Comment on attachment 253096 [details] [diff] [review]
Add VC9/CL15 support to Configure.in, v1.1

Oops... I misread "Requestee" as "Requester", so I entered my email.

On a side note, why are we openly giving out all e-mail addresses for spambots to harvest? Definitely this isn't what we should do! (Maybe we should do something like Bonsai, where @'s are substituted by %'s. Or we should captcha the e-mails.) Is there a bug on this behavior? If no, I'll open one.
Attachment #253096 - Flags: superreview?(netrolller.3d)
Attachment #253096 - Flags: superreview?(benjamin)
Attachment #253096 - Flags: review?(netrolller.3d)
Attachment #253096 - Flags: review?(benjamin)
(Assignee)

Comment 10

10 years ago
Comment on attachment 253096 [details] [diff] [review]
Add VC9/CL15 support to Configure.in, v1.1

Undoing and re-doing review request so that Benjamin gets a mail.
Attachment #253096 - Flags: superreview?(benjamin)
Attachment #253096 - Flags: review?(benjamin)
(Assignee)

Comment 11

10 years ago
Comment on attachment 253096 [details] [diff] [review]
Add VC9/CL15 support to Configure.in, v1.1

Undo and redo, part 2.
Attachment #253096 - Flags: superreview?(benjamin)
Attachment #253096 - Flags: review?(benjamin)
Comment on attachment 253096 [details] [diff] [review]
Add VC9/CL15 support to Configure.in, v1.1

There are already several bugs on the issue, like bug 120030, bug 219021, bug 218917, and bug 215439. I don't think it's really as big of an issue as people think it is.

You don't need to "undo and redo" the review request for Benjamin to get mail about it, and you don't need to request super-review for build patches.
Attachment #253096 - Flags: superreview?(benjamin)
(Assignee)

Comment 13

10 years ago
When I set the request (but even after the undo/redo!) I got "excluding benjamin@smedbergs.us"! Does this mean he didn't get a mail?
No, request mail is separate from bugmail (plus, I think his bsmedberg@gmail account watches this entire component).
Attachment #253096 - Flags: review?(benjamin) → review+
Fixed on trunk, thanks Stefanik
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Assignee)

Comment 16

10 years ago
I'm going to update the Build Documentation now as a patch isn't req'd anymore. Also, a tracking bug for all current, previous or future "Orcas" compatibility work will be opened, as well as a bug about DLL copying (because the DLL names have changed in "Orcas").
(Assignee)

Updated

10 years ago
Blocks: 368708
(Assignee)

Comment 17

10 years ago
Oh, and I almost forgot: VERIFIED FIXED
Status: RESOLVED → VERIFIED
That means that Firefox can be officially build with VC9 now? MDC still mentioning it as experimental.

http://developer.mozilla.org/en/docs/Windows_Build_Prerequisites
Keywords: dev-doc-needed
It works, but there are probably still remaining issues, such as bug 419665.
If there are issues, should this be documented as working at this time?  I'm inclined to say "no."
I think no. Just remove the flag until it is definitely working.
Keywords: dev-doc-needed
I think the current notation of "Experimental" is just fine. I don't think this needs extra dev documentation even if we work out the remaining bugs. Sheppy isn't the only one who can edit MDC, seriously.
Ted, you just made me cry with joy a little bit. :)
You need to log in before you can comment on or make changes to this bug.