Bug 219870 – "make clean" fails on EmptyDSTROOT target

• Component: WebKit Misc.
• Status: RESOLVED FIXED

[Ryan Hostetler]

rdar://72309233

After the fix for rdar://55716322 in https://trac.webkit.org/changeset/270536/webkit, issuing "make clean" from the command line fails when it hits the EmptyDSTROOT target that was added in that revision.

Steps To Reproduce:
1. From the root of OpenSource, issue "make clean" in Terminal.

Results:
After cleaning a bunch of targets successfully, it fails when it hits the EmptyDSTROOT target with a "Build setting PRODUCT_NAME undefined" error.

Output:

=== CLEAN AGGREGATE TARGET EmptyDSTROOT OF PROJECT WebKit WITH CONFIGURATION Debug ===

Check dependencies
Build setting PRODUCT_NAME undefined

** CLEAN FAILED **


The following build commands failed:
	Check dependencies
(1 failure)
make[2]: *** [clean] Error 65
make[1]: *** [clean] Error 2
make: *** [clean] Error 2

[Ryan Hostetler]

Created attachment 416188 [details]
Patch

[Alexey Proskuryakov]

Comment on attachment 416188 [details]
Patch

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

> Source/WebKit/WebKit.xcodeproj/project.pbxproj:14427
> +				PRODUCT_NAME = "$(TARGET_NAME)";

How do we decide when to put PRODUCT_NAME in xcconfig, and when do we put it in pbxproj? I see precedent for both.

[Ryan Hostetler]

(In reply to Alexey Proskuryakov from comment #2)
> Comment on attachment 416188 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=416188&action=review
> 
> > Source/WebKit/WebKit.xcodeproj/project.pbxproj:14427
> > +				PRODUCT_NAME = "$(TARGET_NAME)";
> 
> How do we decide when to put PRODUCT_NAME in xcconfig, and when do we put it
> in pbxproj? I see precedent for both.

There are a few similar script-only aggregate targets in bmalloc, JavaScriptCore, libwebrtc, WTF that keep this setting in pbxproj.
I opted for the pbxproj approach as to not introduce a new file for a target that is designed to work around a bug.

[EWS]

Committed r270843: <https://trac.webkit.org/changeset/270843>

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