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

lib/parser: Support renamed flags (flags/options to options/flags as well) #3256

Merged
merged 7 commits into from
Dec 7, 2023

Conversation

HuidaeCho
Copy link
Member

@HuidaeCho HuidaeCho commented Nov 21, 2023

This PR supports flag to flag, flag to option (fatal error), option to flag (fatal error) renaming.

t.info|-s|-d
# t.rast.list
t.rast.list|-s|-u
# t.rast.univar
t.rast.univar|-s|-u
# t.vect.list
t.vect.list|-s|-u
# t.vect.univar
t.vect.univar|-s|-u

Above lines need a fix (2nd | => :). Fixed in #3260.

@HuidaeCho
Copy link
Member Author

Related to #3214

@neteler neteler added this to the 8.4.0 milestone Nov 21, 2023
@neteler neteler added the enhancement New feature or request label Nov 21, 2023
@neteler neteler changed the title parser: Suppoert renamed flags (flags/options to options/flags as well) parser: Support renamed flags (flags/options to options/flags as well) Nov 21, 2023
@HuidaeCho HuidaeCho marked this pull request as draft November 22, 2023 03:32
@HuidaeCho HuidaeCho merged commit 823b5f4 into OSGeo:main Dec 7, 2023
18 checks passed
@HuidaeCho HuidaeCho deleted the renamed_flags branch December 7, 2023 15:37
HuidaeCho added a commit to HuidaeCho/grass that referenced this pull request Jan 9, 2024
OSGeo#3256)

* parser: Suppoert renamed flags (flags/options to options/flags as well)

* return after fatal error

* Fix short flag and --overwrite renaming

* Single return for long flag renaming

* Add more specific comments about option value for flag to/from option renaming

* Remove "in GRASS 7"

* Declare and assign renamed_key in one line
@wenzeslaus wenzeslaus changed the title parser: Support renamed flags (flags/options to options/flags as well) lib/parser: Support renamed flags (flags/options to options/flags as well) Jun 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants