Bug 136286 - [WinCairo] Link error.
Summary: [WinCairo] Link error.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-27 06:07 PDT by peavo
Modified: 2014-08-27 12:37 PDT (History)
4 users (show)

See Also:


Attachments
Patch (1.66 KB, patch)
2014-08-27 06:16 PDT, peavo
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description peavo 2014-08-27 06:07:22 PDT
Linking WebKit.dll fails with the following error:

1>MSVCRT.lib(MSVCR120.dll) : error LNK2005: _longjmp already defined in LIBCMT.lib(longjmp.obj) 1>MSVCRT.lib(MSVCR120.dll) : error LNK2005: _modf already defined in LIBCMT.lib(modf_pentium4.obj) Creating library WebKit.lib and object WebKit.exp 1>LINK : warning LNK4098: defaultlib 'MSVCRT' conflicts with use of other libs; use /NODEFAULTLIB:library 1>WebKit.dll : fatal error LNK1169: one or more multiply defined symbols found 1>Done Building Project "WebKit.vcxproj" (Build target(s)) -- FAILED. Build FAILED. Time Elapsed 00:02:01.75

I believe this happens because the WebKit libraries are set up to link statically with the runtime libraries, while the WinCairo libraries are using the dynamic runtime libraries.
Comment 1 peavo 2014-08-27 06:16:08 PDT
Created attachment 237222 [details]
Patch
Comment 2 peavo 2014-08-27 06:26:54 PDT
(In reply to comment #1)
> Created an attachment (id=237222) [details]
> Patch

I'm not sure that this is the right fix; maybe it's better to recompile the WinCairo libraries with static runtime libraries, to avoid differences between AppleWin and WinCairo ...
Comment 3 Brent Fulgham 2014-08-27 12:15:31 PDT
(In reply to comment #2)
> (In reply to comment #1)
> > Created an attachment (id=237222) [details] [details]
> > Patch
> 
> I'm not sure that this is the right fix; maybe it's better to recompile the WinCairo libraries with static runtime libraries, to avoid differences between AppleWin and WinCairo ...

No, your change is correct. I'm intending to switch the Apple build back to DLL's at some point in the near future. Sorry for the inconvenience!
Comment 4 peavo 2014-08-27 12:34:58 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > (In reply to comment #1)
> > > Created an attachment (id=237222) [details] [details] [details]
> > > Patch
> > 
> > I'm not sure that this is the right fix; maybe it's better to recompile the WinCairo libraries with static runtime libraries, to avoid differences between AppleWin and WinCairo ...
> 
> No, your change is correct. I'm intending to switch the Apple build back to DLL's at some point in the near future. Sorry for the inconvenience!

No worries :)
Comment 5 WebKit Commit Bot 2014-08-27 12:37:33 PDT
Comment on attachment 237222 [details]
Patch

Clearing flags on attachment: 237222

Committed r173013: <http://trac.webkit.org/changeset/173013>
Comment 6 WebKit Commit Bot 2014-08-27 12:37:36 PDT
All reviewed patches have been landed.  Closing bug.