Skip to content

macOS: use default config dir consistently - #7307

Merged
nilason merged 1 commit into
OSGeo:mainfrom
nilason:test_mac_config
Apr 13, 2026
Merged

nilason merged 1 commit into
OSGeo:mainfrom
nilason:test_mac_config

Conversation

@nilason

@nilason nilason commented Apr 12, 2026 •

Copy link
Copy Markdown
Contributor

Default configuration directory on mac: ~/Library/GRASS/GRASS{major version}. For 8.4, and probably a few releases before, *nix default ~/.grass{major version} was used. This restores the "old" behaviour for mac, with a few moderations. Addons directory name is now versioned, based on major and minor version, but separated from general configurations.

For convenient upgrade experience from 8.4, old configuration files are copied to the new location, on first use.

Closes #7305.

cc @cmbarton

Default configuration directory on mac: `~/Library/GRASS/GRASS{major version}`.
For 8.4, and probably a few before releases before, *nix default,
`~/.grass{major version}` was used. This restores the "old" behaviour for mac,
with a few moderations. Addons directory name is now versioned, based on major
and minor version, but separated from general configurations.

For convenient upgrade experience from 8.4, old configuration files are copied
to the new location, on first use.
@nilason nilason added this to the 8.5.0 milestone Apr 12, 2026
@nilason nilason added the backport to 8.5 PR needs to be backported to release branch 8.5 label Apr 12, 2026
@github-actions github-actions Bot added macOS macOS specific Python Related code is in Python C Related code is in C libraries labels Apr 12, 2026
@nilason

nilason commented Apr 12, 2026

Copy link
Copy Markdown
Contributor Author

Hmm, win runner fails all over since yesterday...

@echoix

echoix commented Apr 12, 2026

Copy link
Copy Markdown
Member

The first errors are for the test of the raster3d lib.

It fails at documentation. Which it shouldn't have anyways. Something is wired up weirdly.

image

@wenzeslaus wenzeslaus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know where the origin of the issue is, but the new code looks good.

@nilason
nilason merged commit 37ba485 into OSGeo:main Apr 13, 2026
26 of 29 checks passed
neteler pushed a commit to neteler/grass that referenced this pull request Apr 13, 2026
Default configuration directory on mac: `~/Library/GRASS/GRASS{major version}`.
For 8.4, and probably a few releases before, *nix default,
`~/.grass{major version}` was used. This restores the "old" behaviour for mac,
with a few moderations. Addons directory name is now versioned, based on major
and minor version, but separated from general configurations.

For convenient upgrade experience from 8.4, old configuration files are copied
to the new location, on first use.
nilason added a commit that referenced this pull request Apr 13, 2026
Default configuration directory on mac: `~/Library/GRASS/GRASS{major version}`.
For 8.4, and probably a few releases before, *nix default,
`~/.grass{major version}` was used. This restores the "old" behaviour for mac,
with a few moderations. Addons directory name is now versioned, based on major
and minor version, but separated from general configurations.

For convenient upgrade experience from 8.4, old configuration files are copied
to the new location, on first use.
@nilason nilason removed the backport to 8.5 PR needs to be backported to release branch 8.5 label Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C Related code is in C libraries macOS macOS specific Python Related code is in Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]

3 participants