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.

Sign up for GitHub

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

Jump to bottom

[SR-3413] swift build tries to build Tests in obj-c packages #5149

Closed
masters3d opened this issue Dec 13, 2016 · 6 comments
Closed

[SR-3413] swift build tries to build Tests in obj-c packages #5149

masters3d opened this issue Dec 13, 2016 · 6 comments
Labels
bug

Comments

@masters3d
Copy link
Contributor

Previous ID SR-3413
Radar None
Original Reporter @masters3d
Type Bug
Status Closed
Resolution Done
Environment

xcode 8.1, 3.0.1

Additional Detail from JIRA
Votes 0
Component/s Package Manager
Labels Bug
Assignee None
Priority Medium

md5: d2a5f4c3d134af34e1c3b3d820894e76

Issue Description:

related to SR-3408
the default behavior of swift build is to not build tests yet in obj-c packages with Tests, swift build will fail because it says it doesnt support tests.
It would be great to leave the test code in place in the package and only fail when running swift test.
@ankitspd
Copy link
Member

You can add Test path in excludes

@masters3d
Copy link
Contributor Author

Doesn't seem to work even if Tests is excluded. I don't really want to exclude it because I am currently creating working xcode projects out of these packages where I am able to run XCTest.

@masters3d
Copy link
Contributor Author

In my mind I should be able to build these with out an error with out having to exclude so I can still keep the functionality of exporting as an xcode project to run tests.

@ankitspd
Copy link
Member

can you attach or point to a minimal sample package?

PS: https://github.com/apple/swift-package-manager/blob/master/Documentation/Resources.md#reporting-a-good-swiftpm-bug

@masters3d
Copy link
Contributor Author

https://github.com/masters3d/SwiftPackageManagerSamples/tree/master/ObjC-PackageAcronym

@masters3d
Copy link
Contributor Author

#850

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
@shahmishal shahmishal transferred this issue from apple/swift May 4, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ankitspd @masters3d