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

SPM error: NoSources #11

Closed
damuellen opened this issue Feb 21, 2016 · 7 comments
Closed

SPM error: NoSources #11

damuellen opened this issue Feb 21, 2016 · 7 comments

Comments

@damuellen
Copy link

I tried to build your Package with the newest version of the Swift Package Manager. I failed with: NoSources(".../Packages/PathKit-0.6.1/Tests/Fixtures").

@kylef
Copy link
Owner

kylef commented Feb 22, 2016

Given the documentation for SPM says the following:

  • Directories named Tests are ignored
  • Sub directories of a directory named Sources, Source, srcs or src become modules

I'd say this is an SPM bug and not a problem with PathKit.

@damuellen
Copy link
Author

The newest Version of SPM has support for testing, so the folder isn't ignored anymore. The documentation is out of date. Take a look at this blog post: http://ankit.im/swift/2016/02/17/swift-package-manager-testing-preview/ Some of your other packages have the same issue. SPM don't accept subfolders in Tests which have no .swift files in it.

@sauliusgrigaitis
Copy link

I've got same issue. Testing feature comes with latest DEVELOPMENT-SNAPSHOT-2016-02-25-a release, so would be great to get released fix soon.

@shnhrrsn
Copy link

Running into the same problem with the 2-25 snapshot. Temporary workaround seems to be deleting the Tests directory entirely.

@kylef
Copy link
Owner

kylef commented Feb 26, 2016

Yeah, can't solve this at the moment. There is a bug in the swift package manager's exclude [https://twitter.com/mxcl/status/703073165254070275].

@kylef
Copy link
Owner

kylef commented Feb 26, 2016

I think swiftlang/swift-package-manager@02cf82d will ultimately solve this problem. Tests shouldn't even be built when using PathKit as a dependency.

@kylef
Copy link
Owner

kylef commented Feb 27, 2016

This can stay open, will need to add the Fixtures directory to excludes once swiftlang/swift-package-manager#152 is merged and released.

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

No branches or pull requests

4 participants