-
-
Notifications
You must be signed in to change notification settings - Fork 310
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: Use project for G_OPT_M_LOCATION #3130
lib/parser: Use project for G_OPT_M_LOCATION #3130
Conversation
Standard option G_OPT_M_LOCATION which creates option named location now creates option named location. Each module using G_OPT_M_LOCATION without further modifications has its own entry in in the renamed options file for backwards compatibility.
…OCATION from g.mapset documentation.
@wenzeslaus Do you consider adding new |
Yes. That fits with the general plan of adding "project" to the API and then gradually removing usage of "location" internally. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note the inconsistencies in how the terms Mapset and Location are written in this and other places: mapset, MAPSET, and Mapset. Since we are changing terminology, this is a good opportunity to standardize how these important terms are written.
As suggested in #2993 and other places, I suggest that we capitalize the first letter and use lower case for the rest of the letters in all descriptions, labels, and documentation--i.e., Mapset and Project--wherever possible. This reduces confusion when people read about these important terms.
For command-line arguments, I suggest keeping everything in lower case letters of course.
Standard option G_OPT_M_LOCATION which creates option named location now creates option named project. Each module using G_OPT_M_LOCATION without further modifications has its own entry in in the renamed options file for backwards compatibility.
New usage of g.mapset:
Still works:
The latter produces a warning:
g.mapset --h
:This applies to: