On 19 May 2017 at 16:16, Fujii Hironori <fujii.hiron...@gmail.com> wrote:
> On Fri, May 19, 2017 at 8:18 AM, Isaac Devine <is...@devinesystems.co.nz>
> > Hi,
> > I'm looking at linking against WebKit in my application, which is linked
> > against the MultiThreadedDLL CRT (i.e. dynamically linked); however
> > is linked against the MultiThreaded CRT (i.e. statically linked). I've
> > linked against previous builds with this configuration "without
> What problem do you have. It should not cause any problem by using any
> type of CRT in WebKit.dll except this problem
I have a linker warning because it was no longer linked to the
MultiThreadedDLL, and was statically linked instead. The behavior described
matched some issues I was having. I'm basically updating to the latest
trunk of WebKit for the application and the /nodefaultlib changes have
arrived after my previous snapshot.
> > I've found bug https://bugs.webkit.org/show_bug.cgi?id=157067 which
> > removing the /nodefaultlib for the libcmt (the MultiThreaded static
> > but in the comments they mention an eventual transition to the
> > CRT.
> > Is that time now? :-)
> Bug 170594 – [CMake][Windows] Use DLL CRT instead of static CRT
> Thanks I've attached my WIP patch for comparison, notably I modify some
props files as well, but from your patch I'm guessing they aren't used
> > I've been making local changes to do this and have noticed in the cmake
> > files that a few of them will swap out the Debug versions of the CRT for
> > release ones, is there a specific reason for that?
> The debug CRT makes WebKit unbearably slow.
> Makes sense.
> > Additionally, relating to the /nodefaultlib directives that are present
> > several build files; my understanding is that if all of the projects were
> > compiled against the same CRT these can be removed?
> I think so, too.
> > From my reading of bug
> > 157067, I believe that the nodefaultlib directives have hidden bugs in
> > past, is that correct?
> Fujii Hironori
> Sony Interactive Entertainment Inc.
webkit-dev mailing list