Skip to content

Conversation

keith
Copy link
Member

@keith keith commented Mar 17, 2022

This matches the behavior implemented in swiftlang/swift-package-manager#3410 for single file modules.

Fixes #913

@keith
Copy link
Member Author

keith commented Mar 17, 2022

I'm not sure if we should do this or not. But folks having to know to pass -parse-as-library isn't ideal either.

@lyft-lint-bot
Copy link

Lyft integration job started: https://buildkite.com/lyft/rules-swift/builds/258 (must be Lyft employee to view)

Copy link
Contributor

@jpsim jpsim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks correct to me!

@keith keith force-pushed the ks/add-special-parse-as-library-handling-for-swift_binary branch from d1157c8 to 5776d5b Compare October 3, 2022 17:31
@keith keith marked this pull request as ready for review October 3, 2022 17:33
@keith keith merged commit 75a5430 into master Oct 3, 2022
@keith keith deleted the ks/add-special-parse-as-library-handling-for-swift_binary branch October 3, 2022 17:34
tymurmustafaiev pushed a commit to tymurmustafaiev/rules_swift that referenced this pull request Jul 19, 2023
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.

Cannot use @main with swift_binary srcs
5 participants