Bug 200667 - [JSCOnly] build broken on macOS due to missing include file
Summary: [JSCOnly] build broken on macOS due to missing include file
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified macOS 10.14
: P2 Minor
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-08-13 02:49 PDT by Samuel Groß
Modified: 2019-08-14 09:40 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.44 KB, patch)
2019-08-14 00:50 PDT, Samuel Groß
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Samuel Groß 2019-08-13 02:49:54 PDT
The JSCOnly build on macOS appears to be broken (probably) after https://trac.webkit.org/changeset/248441/webkit:

 > ./Tools/Scripts/build-jsc --jsc-only --debug
 ...
 ../../Source/JavaScriptCore/jsc.cpp:141:10: fatal error: 'wtf/spi/darwin/ProcessMemoryFootprint.h' file not found
 #include <wtf/spi/darwin/ProcessMemoryFootprint.h>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 1 error generated.
 ninja: build stopped: subcommand failed.
Comment 1 Alexey Proskuryakov 2019-08-13 15:33:00 PDT
Would you be willing to post a patch for this?

https://webkit.org/contributing-code/
Comment 2 Samuel Groß 2019-08-14 00:50:04 PDT
Created attachment 376245 [details]
Patch
Comment 3 Samuel Groß 2019-08-14 00:55:41 PDT
I attempted to create a patch for this. I'm not sure if that's the correct place to add the include file, but `./Tools/Scripts/build-jsc --jsc-only --debug` does seem to work now.
Comment 4 WebKit Commit Bot 2019-08-14 09:39:21 PDT
Comment on attachment 376245 [details]
Patch

Clearing flags on attachment: 376245

Committed r248672: <https://trac.webkit.org/changeset/248672>
Comment 5 WebKit Commit Bot 2019-08-14 09:39:23 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2019-08-14 09:40:18 PDT
<rdar://problem/54305336>