miurahr / aqtinstall

aqt: Another (unofficial) Qt CLI Installer on multi-platforms
https://aqtinstall.readthedocs.io/en/latest/
MIT License
915 stars 84 forks source link

Fixed command to check tools_mingw90 version #680

Closed bjorn closed 1 year ago

bjorn commented 1 year ago

Document command gives:

aqt: error: unrecognized arguments: --long-modules tools_mingw90
ddalcino commented 1 year ago

Thanks for the PR, it’s a nice catch!

This seems to be a quirk of argparse; it allows you to write flags and options either before all the positional arguments or after them. I don’t think I (we?) have been particularly consistent with the way we write these in the docs; it might be worthwhile to audit all the example commands for problems like this. I bet there are more!

bjorn commented 1 year ago

Please use the long form of the -l flag.

Makes sense. Feel free to squash when merging. :-)