-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Disambiguate PIF target names #3716
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@swift-ci please smoke test |
We may need to transform argument values for cases where the user uses |
The tests need adjustments for this change:
|
6ea42fa
to
a72dcb8
Compare
@swift-ci please smoke test |
A couple of the integration tests are still failing. |
This reverts commit 376fd15.
The odd thing is that locally, these tests seem to be passing 🤔 |
Oh, that's because I have been running them incorrectly. What I did was use the inferior |
Looks like the problem here are the tests themselves? They try to refer to products using There's also a potentially interesting concern of us not generating a PIF target for executable targets, but only for their products, so |
A product can have the same name as a target in SwiftPM, but since we create a PIF target for each, we need to come up with a unique name for products. rdar://82744792
a72dcb8
to
f1b8083
Compare
@swift-ci please smoke test |
A product can have the same name as a target in SwiftPM, but since we create a PIF target for each, we need to come up with a unique name for products.
rdar://82744792