Bug 160684 - webkit-gtk tarball fails to build due to missing files
Summary: webkit-gtk tarball fails to build due to missing files
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Konstantin Tokarev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-08 23:52 PDT by Jeremy Huddleston Sequoia
Modified: 2016-08-09 09:59 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.21 KB, patch)
2016-08-09 00:02 PDT, Konstantin Tokarev
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Huddleston Sequoia 2016-08-08 23:52:41 PDT
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.
Comment 1 Konstantin Tokarev 2016-08-08 23:58:54 PDT
My bad, indeed CommonCryptoSPI.h is needed e.g. for OSRandomSource.cpp
Comment 2 Konstantin Tokarev 2016-08-09 00:02:46 PDT
Created attachment 285632 [details]
Patch
Comment 3 WebKit Commit Bot 2016-08-09 09:59:17 PDT
Comment on attachment 285632 [details]
Patch

Clearing flags on attachment: 285632

Committed r204283: <http://trac.webkit.org/changeset/204283>
Comment 4 WebKit Commit Bot 2016-08-09 09:59:21 PDT
All reviewed patches have been landed.  Closing bug.