Bug 237129 – [XCBuild] WTF's SPI dependencies are copied via a folder reference and do not emit task outputs

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Elliott Williams]

[XCBuild] WTF's SPI dependencies are copied via a folder reference and do not emit task outputs

[Elliott Williams]

Created attachment 453074 [details]
Patch

[Elliott Williams]

rdar://board/58950

[Elliott Williams]

Created attachment 453075 [details]
Patch

[Alexey Proskuryakov]

Comment on attachment 453075 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=453075&action=review

> Source/WTF/WTF.xcodeproj/project.pbxproj:822
> +				"$(HEADER_OUTPUT_DIR)/$(INPUT_FILE_DIR:dir:standardizepath:base)/$(INPUT_FILE_DIR:base)/$(INPUT_FILE_NAME)",

Nice.

> Source/WTF/WTF.xcodeproj/project.pbxproj:825
> +			script = "cp -f \"${INPUT_FILE_PATH}\" \"${SCRIPT_OUTPUT_FILE_0}\"\n";

Not sure if this matters at all, but do we want to preserve timestamps?

> Source/WTF/WTF.xcodeproj/project.pbxproj:872
> -			dstSubFolderSpec = 0;
> +			dstSubfolderSpec = 0;

:-O

[Elliott Williams]

(In reply to Alexey Proskuryakov from comment #4)
> > Source/WTF/WTF.xcodeproj/project.pbxproj:825
> > +			script = "cp -f \"${INPUT_FILE_PATH}\" \"${SCRIPT_OUTPUT_FILE_0}\"\n";
> 
> Not sure if this matters at all, but do we want to preserve timestamps?

Xcode's native copy (PbxCp) only copies file contents, not metadata, so I am matching that behavior here. AFAIK, build systems typically don't preserve timestamps, because it would break use cases like two different source checkouts building to the same directory.
 
> > Source/WTF/WTF.xcodeproj/project.pbxproj:872
> > -			dstSubFolderSpec = 0;
> > +			dstSubfolderSpec = 0;
> 
> :-O

Oh no, I thought I had caught all of these! Some recent Xcode has changed the capitalization here, unfortunately.

[Elliott Williams]

(In reply to Elliott Williams from comment #5)
> (In reply to Alexey Proskuryakov from comment #4)
> > > Source/WTF/WTF.xcodeproj/project.pbxproj:872
> > > -			dstSubFolderSpec = 0;
> > > +			dstSubfolderSpec = 0;
> > 
> > :-O
> 
> Oh no, I thought I had caught all of these! Some recent Xcode has changed
> the capitalization here, unfortunately.

Looks like this is actually the other way around: the only project which used camel-cased "SubFolder" is WTF. I'll leave the renaming as-is to match our other pbxprojs.

[EWS]

Committed r290452 (247753@main): <https://commits.webkit.org/247753@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453075 [details].

[Ryan Haddad]

Reverted r290452 for reason:

Caused failues with clean builds

Committed r290471 (247770@trunk): <https://commits.webkit.org/247770@trunk>

[Elliott Williams]

Created attachment 453147 [details]
Patch for relanding

The original patch was missing a build rule for subdirectories of Scripts/, so Scripts/Preferences/* was being copied to /usr/local/include/Scripts.

Tested with a clean build locally.

[EWS]

Committed r290523 (247806@main): <https://commits.webkit.org/247806@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453147 [details].