Skip to content

[SR-1605] [SwiftPM] Add options to sub commands that directly take values #5370

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

Closed
swift-ci opened this issue May 24, 2016 · 4 comments
Closed
Assignees
Labels

Comments

@swift-ci
Copy link
Contributor

Previous ID SR-1605
Radar None
Original Reporter bhargavg (JIRA User)
Type Bug
Status Resolved
Resolution Done
Additional Detail from JIRA
Votes 0
Component/s Package Manager
Labels Bug
Assignee @abertelrud
Priority Medium

md5: 07b00d8a2386e5e58b08aadca7eb2ee1

Issue Description:

After moving to swift package sub command, the syntax for invoking init mode looks odd:

$ swift-package init executable
$ #or
$ swift-package init=executable

IMHO, this could be improved by adding a --type=executable option to init mode.

$ # Suggested
$ swift-package init --type=executable

/cc @abertelrud

@abertelrud
Copy link
Contributor

I definitely agree — the mechanical renaming of the options left this really confusing. I will fix this forthwith.

@abertelrud
Copy link
Contributor

I do have to figure out how to add another level of option here... I do have a refactoring of the option definition infrastructure in mind, but I don't want to block this fix on that work, so this warrants a more targeted fix.

And the same thing will probably be needed for the other commands, e.g. generate-xcodeproj etc.

@abertelrud
Copy link
Contributor

I have gone through and cleaned this up. Since it will be a visible change, I've sent out the new syntax to [email protected] to increase the likelihood that interested parties get a chance to comment. I'll stage the PR in a moment and so there's some time for discussion before landing the change.

@abertelrud
Copy link
Contributor

#370

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
@shahmishal shahmishal transferred this issue from swiftlang/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
Labels
Projects
None yet
Development

No branches or pull requests

2 participants