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.

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

Ags4 modern command line #1749

Draft
wants to merge 4 commits into
base: ags4
Choose a base branch
from

Conversation

jeancallisti
Copy link

No description provided.

@ivan-mogilko ivan-mogilko added ags 4 related to the ags4 development context: code fixing/improving existing code: refactor, optimise, tidy... labels Aug 6, 2022
@ivan-mogilko
Copy link
Contributor

ivan-mogilko commented Aug 8, 2022

Since this is related to the larger engine update, I may suggest to first of all clarify command line syntax rules. In the past the commands were added without much thinking about this, except we moved to using double dashes ("--") before multicharacter option names following the unix standard. But some things may be still inconsistent, both among ags commands, or compared to the widely used conventions. Thus, I think it may be a good idea to clarify that, especially for commands which have arguments, and multiple arguments.

@ericoporto
Copy link
Member

I have been thinking about this, and still don't have a good idea for the interface of passing the number of expected parameters for a flag, in theory I think nice flags should have just one parameter but unfortunately we do have some flags that take multiple parameters in ags. It could either use something like ^3 in the string or use a set of pair string,int.

Since this is for ags4, we could alternatively redesign the command line interface where it takes multiple parameters for one --key? Like either separate them without using spaces or use multiple flags or have a comma.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ags 4 related to the ags4 development context: code fixing/improving existing code: refactor, optimise, tidy...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants