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

Prefer RS_THEME_LOCAL_HOME if set #5260

Merged
merged 1 commit into from Aug 19, 2019

Conversation

@jmcphers
Copy link
Member

commented Aug 19, 2019

The variable RS_THEME_LOCAL_HOME specifies the path to local (per user) themes. In 1.3, we prefer the XDG paths, but this isn't strictly correct since the environment variable is more specific.

This change causes us to prefer the path set by RS_THEME_LOCAL_HOME over the XDG paths, if it's set.

Fixes #5259, and some unit tests.

@jmcphers jmcphers requested a review from MariaSemple Aug 19, 2019
Copy link
Contributor

left a comment

LGTM!

@jmcphers jmcphers merged commit bd9cc31 into master Aug 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.