Add a swift-argument-parser dependency and use it in swift-help

[owenv]

swift-argument-parser needs some CMake fixes before this can land

[owenv]

Tested a cmake build with the change in apple/swift-argument-parser#138 and it seemed to work

[DougGregor]

Good idea! To do this, we'll need to update (or add on to) Swift's list of repos to check out (see swiftlang/swift#31664) and the SwiftPM bootstrap script in swiftlang/swift-package-manager#2736.

[DougGregor]

I'm pulling in swift-argument-parser as a dependency for CI purposes in swiftlang/swift#31664.

[owenv]

@DougGregor Cool, I hadn't seen that SwiftPM PR! I opened apple/swift-argument-parser#139 to update swift-argument-parser cmake and swiftlang/swift-package-manager#2739 in SwiftPM based on your PR. They probably need a little additional work, but should be mostly on the right track.

[natecook1000]

@owenv This is great! Merged your PR in the argument parser.

We'll want to follow up later to add support for querying help for the other swift tools using swift-help, so that writing e.g. swift help package init gives you the same help as swift package init --help.

[DougGregor]

I think this is great! Now that the integrated driver is merged into SwiftPM, there are a few steps needed to make this work out:

• swift-argument-parser will need to tag (0.0.6 or whatever) that includes @owenv 's change
• update-checkout will need to switch to that tag
• SwiftPM's bootstrap script will need to build ArgumentParser and pass its location in when building the SwiftDriver

... and then we can merge this

[natecook1000]

@DougGregor I've tagged an 0.0.6 release for swift-argument-parser that includes @owenv's CMake fixes.

[owenv]

Thanks @natecook1000, I'll switch update-checkout to start using the new tag and then we should be all set to adopt ArgumentParser!

[DougGregor]

@owenv apparently the source compatibility suite has another copy of the update-checkout information that will also need updating (ugh)

[owenv]

@DougGregor Thanks for the heads up, I had no idea that had its own config. I just saw this commit: swiftlang/swift-source-compat-suite@8d478f0 , I'll copy that

[owenv]

This is up-and-running with swift-argument-parser 0.3.0 now. The current plan is

1. Merge [update-checkout] Update swift-argument-parser checkout to 0.3.0 swift#33593 and update swift-argument-parser to 0.3.0 swift-source-compat-suite#454 to update update-checkout with the new version tag
2. Merge Adopt the ArgumentParser library for the command-line tools swift-package-manager#2653 which adopts ArgumentParser in SPM
3. Merge https://github.com/apple/swift-package-manager/pull/2739/files and this PR to adopt it in the driver

[owenv]

@swift-ci test

[owenv]

Once the tests at swiftlang/swift-package-manager#2739 pass, we should be able to merge this 🎉

[DougGregor]

@swift-ci test