Closed Bug 365952 Opened 17 years ago Closed 17 years ago

Make VC8 SP1 the compiler for Win32 ref platform

Categories

(Release Engineering :: General, defect, P2)

x86
All
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ted, Assigned: preed)

References

()

Details

bug 331404 is a NSS crasher that appears to be a bug in the Visual C++ 2005 CRT.  There's a fix on NSS trunk, but that doesn't help us on trunk builds of Firefox.  The bug appears to be fixed in VC8 SP1, see bug 331404 comment 58 and onward.  We should make SP1 the compiler for the Win32 ref platform to get this CRT bugfix among other things.
This change is needed for Tinderbox, and also for nightlies.
I upgraded VC8 on my tpol tinderbox (suiterunner build on SeaMonkey-Ports) to SP1 yesterday, not without problems though. Even though the box had enough free diskspace and has 1 GB of RAM, it once or twice stopped in the middle of the installation (which takes quite long anyways) telling it has too little disk space.
Note that the downloaded .exe for that SP1 already is ~440 MB and it probably needs quite a lot when extracting during installation.
Here it only did work after freeing up even more disk space, rebooting and launching the installation on the freshly rebooted system.

So you probably should take a freshly booted new win32 ref platform install with plenty (2-3 GB should work, I think) of free diskspace, install SP1 there and create a new win32 ref platform image from the result.
We're getting reports on the Mozillazine forums now from people crashing due to bug 331404 using Gran Paradiso alpha 2. It would be really nice if this could be done so those crashes go away. For what it's worth, I've been building with SP1 since it was released two months ago and haven't had any issues with it thus far.
Component: Tinderbox Configuration → Build & Release
QA Contact: ccooper → preed
As a workaround, there are two tiny Windows programs attached to bug 331404.

Attachment 224295 [details] (findold.exe, 29 KB) will list all the files with dates
older than Jan 1, 1970, or with dates in the future, that exist in the directories that NSS searches.  It takes no arguments.  

Attachment 224177 [details] (touch.exe, 32 KB) will set the date/time on a 
user-specified file to the user-specified date/time or current date/time.

Usage: touch [ccyymmdd[hhmm[ss]]] filename

Date/time is UTC, not local time zone.  
Like Unix touch, if the date/time is omitted, touch will use the current 
date and time, and if the named file does not exist, touch will create it.

example:   touch 200701171200 C:/WINDOWS/system32/REALLY.OLD.FILE

Admin privileges may be required for files in %windir%.  
Use at your own risk.
Blocks: 384624
We're either going to need to take bug 382297 or get the hotfix mentioned there in our ref platform.
Depends on: 382297
Probably worth taking both -- even with one PDB file per library, it's still pokey.
Assignee: build → preed
Priority: -- → P1
(In reply to comment #5)
> We're either going to need to take bug 382297 or get the hotfix mentioned there
> in our ref platform.

http://fs/public/IT/Microsoft/VS%20Hotfix/
Priority: P1 → P2
The hotfix was pulled from the MS KB URL/issue above.
These are both included and apparently working in the latest Win32 ref platform, deployed on fxnewref-win32-tbox. Closing.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Thanks to all who helped accomplish this!

If I may request one point of information:
What ftp URL(s) can use I to ensure that I am downloading a nightly trunk
build *built with VC8*  of FF3 or SM2 ?
(In reply to comment #10)

> If I may request one point of information:
> What ftp URL(s) can use I to ensure that I am downloading a nightly trunk
> build *built with VC8*  of FF3 or SM2 ?

Since VC8 is the default, you can get VC8 builds of FF3 at the regular nightly location (http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/); SM2 builds will get it when bug 392438 gets fixed.
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.