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

export: Actually fix XDG_DATA_DIRS and XDG_DATA_HOME handling #1582

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

TigerGorilla2
Copy link
Contributor

This fixes the use of XDG_DATA_DIRS and XDG_DATA_HOME in accordance to the specification.
As per the specification

There is a set of preference ordered base directories relative to which data files should be searched. This set of directories is defined by the environment variable $XDG_DATA_DIRS.

and

There is a single base directory relative to which user-specific data files should be written. This directory is defined by the environment variable $XDG_DATA_HOME.

This change now uses the correct fallbacks for each variable (was swapped) and does not try to iterate over XDG_DATA_HOME as it should only contain a single directory.

@89luca89
Copy link
Owner

Thanks @TigerGorilla2 LGTM!

@89luca89 89luca89 merged commit 2166140 into 89luca89:main Oct 10, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants