Skip to content

Support Swift Package Manager#147

Open
mtfum wants to merge 4 commits into
delba:masterfrom
mtfum:support-SPM
Open

Support Swift Package Manager#147
mtfum wants to merge 4 commits into
delba:masterfrom
mtfum:support-SPM

Conversation

@mtfum

@mtfum mtfum commented Apr 6, 2020

Copy link
Copy Markdown

Thank you for your great OSS!

This PR is about SPM, Although as I mentioned at #146!
I tried to create Package.swift.

@mtfum mtfum changed the title Support Swift PackageManager Support Swift Package Manager Apr 7, 2020
@SixFiveSoftware

Copy link
Copy Markdown

+1 for this! Might be worth adding a sentence or two in the README about the added SPM support in this PR. Came here wanting to make my own PR and saw there already was one 😄

@mtfum

mtfum commented May 11, 2020

Copy link
Copy Markdown
Author

Thanks for the response! @SixFiveSoftware

But this PR doesn't work sadly.
This is because Permission must provide custom build flags before building (#configuration).

To deal with the problem, in case of Cocoapods, use subspecs, and in case of Carthage, use xcconfig

I don't have the idea to do this correspondence in SPM.

If anyone has any advice, please let me know.

@SixFiveSoftware

Copy link
Copy Markdown

@mtfum , correct, I had actually tried to pull from your branch using SPM the other day and found that out too. I experimented with a few things, but you're right, the subspec behavior is difficult to mimic via SPM (at least with my limited knowledge of it). I reverted back to CocoaPods after a bit of trying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants