fix: broken spm package if added to multiple targets#3
Open
farfromrefug wants to merge 2 commits intorigor789:feat/spm-package-supportfrom
Open
fix: broken spm package if added to multiple targets#3farfromrefug wants to merge 2 commits intorigor789:feat/spm-package-supportfrom
farfromrefug wants to merge 2 commits intorigor789:feat/spm-package-supportfrom
Conversation
rigor789
requested changes
Feb 2, 2026
Owner
rigor789
left a comment
There was a problem hiding this comment.
I'm not sure why this change is required, this will result in duplicate entries if executed multiple times, ie. rebuilding without clearing/re-creating the xcode project.
Author
|
@rigor789 thanks and sorry for the typo. FIxed |
Author
you need one entry per target. It has to be duplicated |
Author
|
@rigor789 could we try to finish this one ? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is needed for an upcoming PR on the cli for more complex appwatch projects