<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK][W32] WTF tries to include non-existent WTFHeaderDetection.h"
href="https://bugs.webkit.org/show_bug.cgi?id=143753#c6">Comment # 6</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK][W32] WTF tries to include non-existent WTFHeaderDetection.h"
href="https://bugs.webkit.org/show_bug.cgi?id=143753">bug 143753</a>
from <span class="vcard"><a class="email" href="mailto:lrn1986@gmail.com" title="LRN <lrn1986@gmail.com>"> <span class="fn">LRN</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=143753#c5">comment #5</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=250796&action=diff" name="attach_250796" title="Don't include non-existing WTF/WTFHeaderDetection.h in 2.4.x">attachment 250796</a> <a href="attachment.cgi?id=250796&action=edit" title="Don't include non-existing WTF/WTFHeaderDetection.h in 2.4.x">[details]</a></span>
> Don't include non-existing WTF/WTFHeaderDetection.h in 2.4.x
>
> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=250796&action=review">https://bugs.webkit.org/attachment.cgi?id=250796&action=review</a>
>
> I don't think this is quite right. I think it would be better to have the
> GTK Win32 build simply generate a "false" case WTFHeaderDetection.h file.
> That will work with all compiler combinations. I know there are some people
> using MingW32 to build the WinCairo port, and others working with the Intel
> compiler. Now that Clang is somewhat usable on Windows, I expect that to be
> an issue as well.</span >
I went for the simplest fix possible that does not immediately break anything else. Considering the fact that building webkit takes a few hours, i'm not very well-motivated to try anything fancier, unless i absolutely have to.
<span class="quote">>
> > Source/WTF/ChangeLog:1
> > +2015-04-15 Ð ÑÑлан ÐжбÑлаÑов <<a href="mailto:lrn1986@gmail.com">lrn1986@gmail.com</a>>
>
> The string encoding here seems wrong!</span >
Poor UTF-8 support in the review module. The text is in UTF-8 in the attachment itself.
<span class="quote">>
> > Source/WTF/wtf/Platform.h:614
> > +#ifdef _MSC_VER
>
> This change breaks the build for people using MingW32 (or other compilers)
> to build WebKit on Windows.</span >
What about #ifdef HAVE_WTFHEADERDETECTION_H and modifying these mysterious other-ports-with-buildsystems-that-use-gnu-toolchain-but-refuse-to-use-gnu-autotools-which-do-not-generate-this-header? Well, i guess they'd tell me to fix the autotools-based buildsystem used to build webkitgtk instead, which would be fair enough.
Thing is, 2.4.x is the last version to have autotools, newer webkitgtk uses cmake, and we'll go back to square one.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>