Skip to content
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

Proper Linux support and testing (ish). #20

Closed
wants to merge 8 commits into from

Conversation

swizzlr
Copy link

@swizzlr swizzlr commented Dec 23, 2015

This commit adds a Dockerfile to quickly check that Curry and the generator compiles successfully.

Furthermore, it moves the curry generator into its own target, to support more recent changes to SPM that failed a build when any source files were present in the root of the directory.

Run docker build . to compile the package inside a Linux container.

gfontenot and others added 8 commits December 3, 2015 15:29
This follows the Swift Package Manager conventions
The presence of this folder (or maybe the presence of these non-swift
files) causes SPM to not compile the library properly. Moving these to
the top-level of the repo fixes the issue.
This actually _does_ work by default. I was wrong.
This commit adds a Dockerfile to quickly check that Curry and the generator compile successfully.

Furthermore, it moves the curry generator into its own target, to support more recent changes to SPM that failed a build when any source files were present in the root of the directory.
@swizzlr
Copy link
Author

swizzlr commented Dec 23, 2015

After merging master, the generator is a non issue due to its implicit dependence on a Mac environment.

@gfontenot
Copy link
Collaborator

Oops, this closed because I deleted my spm branch. @swizzlr mind re-opening this against master and I'll get it merged?

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