WebCore GYP build should build without help from normal WebCore build
Created attachment 86498 [details] Patch
Comment on attachment 86498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=86498&action=review OK. > Source/WebCore/ChangeLog:15 > + target, called "WebCore Support", which is less that aesthetically "less that"
Comment on attachment 86498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=86498&action=review > Source/WebCore/gyp/WebCore.gyp:282 > + 'mac_bundle': 1, This loks wrong.
Comment on attachment 86498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=86498&action=review > Source/WebCore/gyp/copy-forwarding-and-icu-headers.sh:4 > +rsync -aq --exclude ".svn" --exclude ".DS_Store" "$SRCROOT/../ForwardingHeaders" "$BUILT_PRODUCTS_DIR/$PRIVATE_HEADERS_FOLDER_PATH" > +rsync -aq --exclude ".svn" --exclude ".DS_Store" "$SRCROOT/../icu" "$BUILT_PRODUCTS_DIR/$PRIVATE_HEADERS_FOLDER_PATH" You might want to make the rsync command a variable if you add more of these...
Created attachment 86500 [details] Patch
Comment on attachment 86498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=86498&action=review > Source/WebCore/gyp/WebCore.gyp:150 > + 'inputs': [], > + 'outputs': [], Isn't [] the default here?
> Isn't [] the default here? Nope. The property is required. :(
Comment on attachment 86500 [details] Patch Clearing flags on attachment: 86500 Committed r81695: <http://trac.webkit.org/changeset/81695>
All reviewed patches have been landed. Closing bug.