Skip to content

Conversation

@c-bata
Copy link

@c-bata c-bata commented Jun 16, 2018

First, I need to say this changes are still not perfect because daltoniam/zlib-spm and daltoniam/Starscream didn't support Swift PM v4 yet. So I fork these and add changes to support it. You can check my changes via following pages.

There is a problem when sending a Pull Request to Starscream because C header files are moved inside include directory to support Swift PM v4 cause of constraints to support C language targets (See https://github.com/apple/swift-package-manager/blob/master/Documentation/Usage.md#c-language-targets for more details).
So maybe Starscream builds in XCode will be failed cause of changing directory of header files.

@c-bata
Copy link
Author

c-bata commented Jun 16, 2018

I bumped into a problem when building on Linux(dockerhub) 😭 CommonCrypt is only available at iOS/Mac.
daltoniam/Starscream#454

@harryjburr
Copy link

Yeah. That is true

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants