WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
88879
Fix compiler warnings due to WebKitSystemInterface.lib on Windows for Release config
https://bugs.webkit.org/show_bug.cgi?id=88879
Summary
Fix compiler warnings due to WebKitSystemInterface.lib on Windows for Release...
Vivek Galatage
Reported
2012-06-12 09:08:01 PDT
Fix compiler warnings due to WebKitSystemInterface.lib on Windows for Release config
Attachments
Patch
(2.14 KB, patch)
2012-06-12 09:13 PDT
,
Vivek Galatage
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Vivek Galatage
Comment 1
2012-06-12 09:13:19 PDT
Created
attachment 147095
[details]
Patch
Vivek Galatage
Comment 2
2012-06-12 09:15:12 PDT
For release configuration, the GenerateDebugInformation flag for VC project file results in having the following warnings: 7>Finished generating code 7>WebKitSystemInterface.lib(MediaUI.obj) : warning LNK4099: PDB 'webkitsysteminterface.vc80.pdb' was not found with 'C:\cygwin\home\buildbot\slave\win-release\build\WebKitLibraries\win\lib\WebKitSystemInterface.lib' or at 'c:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\bin\webkitsysteminterface.vc80.pdb'; linking object as if no debug info 7>WebKitSystemInterface.lib(LegacyMediaUI.obj) : warning LNK4099: PDB 'webkitsysteminterface.vc80.pdb' was not found with 'C:\cygwin\home\buildbot\slave\win-release\build\WebKitLibraries\win\lib\WebKitSystemInterface.lib' or at 'c:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\bin\webkitsysteminterface.vc80.pdb'; linking object as if no debug info 7>WebKitSystemInterface.lib(WebKitSystemInterface.obj) : warning LNK4099: PDB 'webkitsysteminterface.vc80.pdb' was not found with 'C:\cygwin\home\buildbot\slave\win-release\build\WebKitLibraries\win\lib\WebKitSystemInterface.lib' or at 'c:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\bin\webkitsysteminterface.vc80.pdb'; linking object as if no debug info 7>WebKitSystemInterface.lib(SharedMediaUI.obj) : warning LNK4099: PDB 'webkitsysteminterface.vc80.pdb' was not found with 'C:\cygwin\home\buildbot\slave\win-release\build\WebKitLibraries\win\lib\WebKitSystemInterface.lib' or at 'c:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\bin\webkitsysteminterface.vc80.pdb'; linking object as if no debug info These warnings can be removed if we set the GenerateDebugInformation flag to false for the release configuration.
Vivek Galatage
Comment 3
2012-06-12 09:20:27 PDT
Fixing this in two projects which are affected by this library: QTMovieWin and WebKit2
Adam Roben (:aroben)
Comment 4
2012-06-12 10:04:16 PDT
Comment on
attachment 147095
[details]
Patch Will this cause us not to generate PDBs for these projects at all in Release builds? That would be bad, because it would make them very hard to debug.
Vivek Galatage
Comment 5
2012-06-12 10:19:46 PDT
(In reply to
comment #4
)
> (From update of
attachment 147095
[details]
) > Will this cause us not to generate PDBs for these projects at all in Release builds? That would be bad, because it would make them very hard to debug.
My apologies as I didn't notice about this perspective. Yes you are right, it will not generate any pdb for these projects which is bad.
Vivek Galatage
Comment 6
2012-06-12 11:34:16 PDT
Closing the bug.
Adam Roben (:aroben)
Comment 7
2012-06-12 21:43:03 PDT
Maybe we can tell the linker to ignore these warnings?
Vivek Galatage
Comment 8
2012-06-13 01:13:47 PDT
(In reply to
comment #7
)
> Maybe we can tell the linker to ignore these warnings?
Yeah that would have been a desirable feature but unfortunately according to
http://msdn.microsoft.com/en-us/library/b7whw3f3(v=vs.80).aspx
this is not supported at least in Visual Studio 2005.
Adam Barth
Comment 9
2012-07-27 01:12:16 PDT
Comment on
attachment 147095
[details]
Patch Cleared review? from
attachment 147095
[details]
so that this bug does not appear in
http://webkit.org/pending-review
. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug