Add VS11 (VS2012) support for mozilla-build

RESOLVED FIXED

Status

mozilla.org
MozillaBuild
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: bbondy, Assigned: gps)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
I think we'll need new mozilla-build support for VS2011.  

In particular 2 new scripts:
start-msvc11.bat
start-msvc11-x64.bat
Out of curiosity, have you tried building with VS2011 yet?
(Reporter)

Comment 2

6 years ago
No not yet, I plan to once I get back to a desktop I can install on. I don't want to try building on my laptop VM due to disk space and time :)
(In reply to Brian R. Bondy [:bbondy] from comment #2)
> No not yet, I plan to once I get back to a desktop I can install on. I don't
> want to try building on my laptop VM due to disk space and time :)

Definitely, makes sense.  Please file bugs (I'm sure there will be some!) and CC me on them as well.
(Assignee)

Updated

6 years ago
Blocks: 686843
(Assignee)

Comment 4

6 years ago
I have Windows 8 installed in Virtualbox and am in the process of trying to wrangle mozilla-central into building. I've started https://developer.mozilla.org/en/Windows_8 and will be documenting my progress there. As things mature, we can move the content to other developer.m.o articles (I don't want to pollute the existing articles until things are more solidified).

Expect some patches and new bugs to follow.
(Assignee)

Updated

6 years ago
Blocks: 687213
(Assignee)

Comment 5

6 years ago
Created attachment 560674 [details] [diff] [review]
Detect MSVC2011 and add startup scripts

This adds support for MSVC 2011 and SDK 8.0. I followed the convention of the existing files.

On my Windows 8 Developer Preview install, I dropped the changes in a built mozilla-build directory and the 2 batch files executed fine.

I have tested with Visual Studio 2011 preview installed. But, the scripts do detect Visual Studio 2011 Express properly.

I have NOT tested with Windows SDK 8.0 because I do not yet have it installed. But, I'm optimistic it will work. I'll let others decide if we will take that chance and check this in as-is.
Assignee: nobody → gps
Status: NEW → ASSIGNED
Attachment #560674 - Flags: review?(ted.mielczarek)
(Reporter)

Comment 6

6 years ago
Comment on attachment 560674 [details] [diff] [review]
Detect MSVC2011 and add startup scripts

Review of attachment 560674 [details] [diff] [review]:
-----------------------------------------------------------------

::: start-msvc10.bat
@@ +1,4 @@
>  @echo off
>  
>  SETLOCAL
> +SET MOZ_MSVCVERSION=11

We do not want to break compatibility with people who want to build with VS2010.  So this should be in a new file start-msvc11.bat and ditto for the x64 variant.
Attachment #560674 - Flags: review?(ted.mielczarek)
(Reporter)

Comment 7

6 years ago
Re the Windows SDK 8.0 change as well, we should be testing it since there are code changes for it. 
Also have you tried building as x64 yet? That should be tried first as well before a positive review can be given.

Here's an example x64 .mozconfig file:
ac_add_options --enable-application=browser
ac_add_options --enable-debug
ac_add_options --disable-static
ac_add_options --disable-optimize
ac_add_options --enable-tests
ac_add_options --enable-logging
ac_add_options --target=x86_64-pc-mingw32
ac_add_options --host=x86_64-pc-mingw32
(Assignee)

Comment 8

6 years ago
(In reply to Brian R. Bondy [:bbondy] from comment #6)
> ::: start-msvc10.bat
> @@ +1,4 @@
> >  @echo off
> >  
> >  SETLOCAL
> > +SET MOZ_MSVCVERSION=11
> 
> We do not want to break compatibility with people who want to build with
> VS2010.  So this should be in a new file start-msvc11.bat and ditto for the
> x64 variant.

Yeah, Bugzilla's diff tool doesn't show that I used `hg cp` to copy start-msvc10.bat to start-msvc11.bat and then modified the copy. If you look at the raw diff, you'll see I actually created new files.

w.r.t. x64, I'll test that.

As for Windows SDK 8.0, it appears to only be available on MSDN to people with a premium subscription. Unfortunately, it appears I am not currently one of those people. I thought the ISO release notes mentioned an SDK, though. Maybe it is in a weird location or only the ARM version is available. I'll poke around.
(Reporter)

Comment 9

6 years ago
re splinter review fail: OK thanks for letting me know :)

re Windows SDK 8.0: If we can't access it I think that is fine for now.  I'm not a module peer for this though so I'll let your reviewer make the call.

Please feel free to re-request review after you try on x64.
(Reporter)

Comment 10

6 years ago
Any update on this bug?
(Assignee)

Comment 11

6 years ago
I haven't tried running in x64 mode yet. And, I won't have access to the machine where the Windows 8 VM is until Monday.

If it can't wait until then, anyone is free to set up Windows 8 (in Virtualbox) to test this patch. Or, if you want to conditionally r+ on x64 working, I'll update the flags with checkin-needed once I've verified on x64.
(Reporter)

Comment 12

6 years ago
Sounds good, no there's no rush, I just wanted to check in.
Once done please request a review again as I'm not a peer in this yet. 
Thanks for the update.
(Assignee)

Comment 13

6 years ago
Comment on attachment 560674 [details] [diff] [review]
Detect MSVC2011 and add startup scripts

I tried running configure in x64 mode (start-msvc11-x64.bat) and it seems to work! Confirmed that configure output says a 64 bit OS is detected. Compiling seems to work as well.
Attachment #560674 - Flags: review?(ted.mielczarek)
Comment on attachment 560674 [details] [diff] [review]
Detect MSVC2011 and add startup scripts

Review of attachment 560674 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks! khuey is still working on getting a new release of MozillaBuild out, we should get this in when that happens.
Attachment #560674 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 15

6 years ago
Thanks, Ted. Setting checkin-needed, as I don't have the privs.
Keywords: checkin-needed
(Assignee)

Updated

6 years ago
Blocks: 631910
landed
http://hg.mozilla.org/mozilla-build/rev/a94eedd605ef
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
When is the new release of the MozillaBuild package planed that will contain this change?
Blocks: 712841
No longer blocks: 712841
Depends on: 712841
(In reply to Honza Bambas (:mayhemer) from comment #17)
> When is the new release of the MozillaBuild package planed that will contain
> this change?

I think MozillaBuild 1.6 contains this change.
(In reply to Gary Kwong [:gkw, :nth10sd] from comment #18)
> (In reply to Honza Bambas (:mayhemer) from comment #17)
> > When is the new release of the MozillaBuild package planed that will contain
> > this change?
> 
> I think MozillaBuild 1.6 contains this change.

Since installer has bug, start-vc11.bat isn't included into this package. See bug 712841.
(Assignee)

Comment 20

6 years ago
FWIW you can grab the .bat files from https://hg.mozilla.org/mozilla-build/file/3ba6c690b036.

Also, we found at least one compiler bug in MSVC 2011 where flat-out wrong assembly was emitted, so you may want to stick with a released/stable version if you can.

Comment 21

5 years ago
Curious when these changes end up in the release version of MozillaBuild? Looking at the date of the current release, it's still back on 23-Nov-2011 04:13 79M. 

http://ftp.mozilla.org/pub/mozilla.org/mozilla/libraries/win32/
This was supposed to go out in the last release, but bug 712841 made it not work. We could probably spin a simple 1.6.1 that includes these files if it would be helpful.

Comment 23

5 years ago
(In reply to Ted Mielczarek [:ted] from comment #22)
> This was supposed to go out in the last release, but bug 712841 made it not
> work. We could probably spin a simple 1.6.1 that includes these files if it
> would be helpful.

We'll need the batch scripts for win8 metro work. We have access so no major rush but it'd be nice to get these our where devs can easily get at them.

Comment 24

5 years ago
Can we please spin a newer version of mozilla-build that I can use for the Win8 infra work? (bug 737994)
I only see the November version [1]

I would prefer not to have to apply patches to the 1.6 release as much as possible.

[1]
http://ftp.mozilla.org/pub/mozilla.org/mozilla/libraries/win32/
Blocks: 754810
As per comment from @armenzg is this landed on the latest build? The documents on from this link (https://developer.mozilla.org/en-US/docs/Windows_8) says otherwise.
Also, now that Visual Studio 2012 is out, is there any major changes? Or I can say this patch will work well if I have visual studio 2012 installed?
The latest MozillaBuild (1.6.1) contains this change, yes.
(Reporter)

Updated

5 years ago
Summary: Add VS2011 support for mozilla-build → Add VS11 (VS2012) support for mozilla-build
You need to log in before you can comment on or make changes to this bug.