Bug 160684 – webkit-gtk tarball fails to build due to missing files

• Component: WebKitGTK
• Status: RESOLVED FIXED

[Jeremy Huddleston Sequoia]

As mentioned on the mailing list, I decided to make the jump from git to tarballs to test out the 2.13.x series.  Unfortunately, it looks like tarball creation is dropping files on the floor.

CMake Error at Source/cmake/WebKitMacros.cmake:249 (add_library):
 Cannot find source file:

   spi/darwin/CommonCryptoSPI.h

 Tried extensions .c .C .c++ .cc .cpp .cxx .m .M .mm .h .hh .h++ .hm
.hpp
 .hxx .in .txx
Call Stack (most recent call first):
 Source/WTF/wtf/CMakeLists.txt:305 (WEBKIT_FRAMEWORK)


CMake Error: Cannot determine link language for target "WTF".
CMake Error: CMake can not determine linker language for target: WTF

I notice you're missing (at least) these paths:

Source/WebCore/platform/spi
Source/WebKit2/Platform/spi
Source/WTF/wtf/spi

I didn't do a deep comparison.  I just looked for spi in the tarball
and in the repository and came up with those hits.

Carlos responded with:

This is a regression introduced in r203620, see bug #160094. I guess Konstantin assumed that those spi files are only used by mac builds, but I guess they are also needed to build gtk on mac.

[Konstantin Tokarev]

My bad, indeed CommonCryptoSPI.h is needed e.g. for OSRandomSource.cpp

[Konstantin Tokarev]

Created attachment 285632 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 285632 [details]
Patch

Clearing flags on attachment: 285632

Committed r204283: <http://trac.webkit.org/changeset/204283>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.