lib/init: add GRASS_CONFIG_DIR environment variable #3899
Conversation
imincik
force-pushed
the
config-dir-env-variable
branch
from
eb22446
to
e15d5da
Compare
GRASS_CONFIG_DIR variable allows to set alternative root path for `.grass<VERSION>` configuration directory. If not set, $HOME directory is used.
imincik
force-pushed
the
config-dir-env-variable
branch
from
e15d5da
to
8781e89
Compare
|
Related #3153 |
|
Would it be possible to, instead of introducing another GRASS env var, adopt our code to respect (if set) $XDG_CONFIG_HOME (see https://specifications.freedesktop.org/basedir-spec/basedir-spec-0.6.html)? |
The main use case of this PR is to be able to run multiple isolated instances of grass. Using $HOME or $XDG_CONFIG_HOME variables to force grass to store configuration in alternative directory is not a good idea, because this will also have an impact on other tools relying on those variables. That's why I introduced a new grass specific environment variable. |
But I agree, that usage of XDG_CONFIG_HOME instead of HOME is a good fix for #3153 . |
I see. |
Co-authored-by: Nicklas Larsson <n_larsson@yahoo.com>
github-actions
bot
added
Python
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
- create grass.utils.config to avoid code duplication in grass.py and core.utils - change grass.py in order to use grass.utils.config.get_grass_config_path()
|
I will not be able to look at and test this this week, but please hold on, I see there are Mac specific changes now that need hands-on testing/review. |
Sure. This PR shouldn't change behaviour compared to #3694. |
There was a problem hiding this comment.
This seems like a reasonable feature, although I'm wondering whether other (all?) software allows the same, i.e., having its own variable to set config dir just for that software.
As for the implementation, there is couple of other places which may need fixing. g.extension needs to do something, but I didn't examine what. lib/gis/home.c worries me a little bit as it seems to figure out the path as well, at this point duplicating what is in Python which is more complex code with this PR and the anticipated XDG change.
- replace IsADirectoryError by NotADirectoryError
GRASS_CONFIG_DIR variable allows to set alternative root path for
.grass<VERSION>configuration directory. If not set, $HOME directoryis used.
The main use case of this PR is to be able to run multiple isolated instances of grass. Using $HOME or $XDG_CONFIG_HOME variables to force grass to store configuration in alternative directory is not a good idea, because this will also have an impact on other tools relying on those variables. That's why I introduced a new grass specific environment variable.
/cc @landam