Closed Bug 401140 (vc9-zlib) Opened 16 years ago Closed 16 years ago

VC9 issues building zlib


(Firefox Build System :: General, defect)

Windows XP
Not set


(Not tracked)



(Reporter: aaronlev, Assigned: sciguyryan)




(3 files)

C:\run\vstudio9\VC\INCLUDE\stdio.h(358) : error C3163: '_vsnprintf': attributes inconsistent with previous declaration
        C:\run\vstudio9\VC\INCLUDE\stdio.h(350) : see declaration of '_vsnprintf'

Once you fix that, you still get:
LINK : fatal error LNK1117: syntax error in option 'MAPINFO:LINES'
I'm not sure what the actual correct fixes are, but this shows what I have to do to build.
We have just fixed the _vsnprintf issue for the zlib in NSS.
You can use the patch for zutil.h in bug 406258.
Alias: vc9
The "vc9" alias belongs to the tracking bug for VC9. Changing to "vc9-zlib".
Alias: vc9 → vc9-zlib
Bug 406258 is marked fixed, but after repulling I still had an issue with vsnprintf. Perhaps it wasn't checked in everywhere necessary. The zlib.h exists in mozilla/security and mozilla/modules/zlib

Also the MAPINFO issue still exists.
I meant zutil.h
Not sure who should review this, nor what the fix to the MAPINFO issue is.
Comment on attachment 291151 [details] [diff] [review]
Fixes the vsnprintf part of the problem


I'm sorry that I can't help you with the MAPINFO
issue because NSS doesn't use that linker flag.
Attachment #291151 - Flags: review+
Attachment #291151 - Flags: approval1.9?
Attachment #291151 - Flags: approval1.9? → approval1.9+
Checked in the _vsnprintf fix.

The MAPINFO issue still remains.
Does anyone have any news on getting this fixed? Not that VS 2008 has been released for a while its possible more people will be using it and finding this little problem.
Per, MAPINFO:LINES actually went away in VC8, but the compiler must not have errored on it.  I guess we should just remove that flag completely.  Aaron: can you submit a patch that remove -MAPINFO:LINES from that line, and r? me?  It would be nice to make sure that the trunk compiles in VC9 before we go into release mode, otherwise it's going to become painful to support the branch.  (VC6 on the 1.8 branch anyone?)
Ted, I don't feel qualified to be on the hook for this. Can you work with someone on the build team?
Aaron: that's the whole point of review, isn't it?  I'm quite happy to review that patch, but I don't have VC9, so I'm not in a situation to test it.
I'm willing to test a patch, if someone who understands the build system and all of our references to MAPINFO and MOZ_MAPINFO wants to make that. I'm sure it's not that hard, but I don't have cycles for it.
Attached patch Patch v1.0Splinter Review
Patch v1.0

Removed the flag and compiled under Visual Studio 2008 and ran the build with no problems. Not sure if this will need a sr too so I'll leave that to luser.
Assignee: nobody → sciguyryan
Attachment #301954 - Flags: review?(ted.mielczarek)
Comment on attachment 301954 [details] [diff] [review]
Patch v1.0

Works for me.  Since this flag doesn't work in VC8, anything that's broken without it is already broken.
Attachment #301954 - Flags: review?(ted.mielczarek) → review+
Attachment #301954 - Flags: approval1.9?
Comment on attachment 301954 [details] [diff] [review]
Patch v1.0

Attachment #301954 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
Checking in config/;
/cvsroot/mozilla/config/,v  <--
new revision: 3.585; previous revision: 3.584
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta4
Version: unspecified → Trunk
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.