Teach JavaScriptCore GYP build about DEPTH
Created attachment 84150 [details] Patch
Comment on attachment 84150 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=84150&action=review > Source/JavaScriptCore/gyp/JavaScriptCore.gyp:27 > + '<(DEPTH)', # Some paths in API include JavaScriptCore/ > + '<(DEPTH)/JavaScriptCore', > + '<(DEPTH)/JavaScriptCore/ForwardingHeaders', > + '<(DEPTH)/JavaScriptCore/API', > + '<(DEPTH)/JavaScriptCore/assembler', > + '<(DEPTH)/JavaScriptCore/collector/handles', > + '<(DEPTH)/JavaScriptCore/bytecode', > + '<(DEPTH)/JavaScriptCore/bytecompiler', > + '<(DEPTH)/JavaScriptCore/debugger', > + '<(DEPTH)/JavaScriptCore/icu', > + '<(DEPTH)/JavaScriptCore/interpreter', > + '<(DEPTH)/JavaScriptCore/jit', > + '<(DEPTH)/JavaScriptCore/parser', > + '<(DEPTH)/JavaScriptCore/profiler', > + '<(DEPTH)/JavaScriptCore/runtime', > + '<(DEPTH)/JavaScriptCore/wtf', > + '<(DEPTH)/JavaScriptCore/wtf/unicode', This seems annoyingly verbose. Would be nice to have an INCLUDE_ROOT feature for gyp. :)
We can create an INCLUDE_ROOT variable pretty easily.
I mean so that we don't have to use DEPTH for every include line.
Comment on attachment 84150 [details] Patch Clearing flags on attachment: 84150 Committed r79974: <http://trac.webkit.org/changeset/79974>
All reviewed patches have been landed. Closing bug.