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

[WIP][Experimental] Use libSwiftPM #454

Closed
wants to merge 5 commits into from
Closed

Conversation

norio-nomura
Copy link
Collaborator

No description provided.

@norio-nomura norio-nomura changed the title [Experimental] Use libSwiftPM [WIP][Experimental] Use libSwiftPM Dec 1, 2017
@norio-nomura
Copy link
Collaborator Author

https://bugs.swift.org/browse/SR-6513 does not explain split() issue correctly.
I filed https://bugs.swift.org/browse/SR-6523 by myself.

@jpsim
Copy link
Owner

jpsim commented Dec 6, 2017

This is interesting to consider, but even when it's made to work, I'm not sure it's worth adding a large external dependency for something that's currently pretty simple (though somewhat fragile).

What do you think @norio-nomura?

- Remove clibc target
- Set `SWIFT_OBJC_BRIDGING_HEADER = "$(SRCROOT)/Sources/clibc/include/clibc.h"` in libc target
@norio-nomura
Copy link
Collaborator Author

I also reached the conclusion that using current libSwiftPM for Module.init(spmName:) is inefficient.
I wanted to make sure that I could actually build libSwiftPM as a depndency.

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.

None yet

2 participants