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

test: ignore the real XDG_CONFIG_HOME during tests #173

Merged
merged 1 commit into from
Jul 27, 2020

Conversation

whot
Copy link
Contributor

@whot whot commented Jul 27, 2020

Let's not have our tests fail if the user has an incompatible $XDG_CONFIG_HOME/xkb directory.

libxkbcommon has fallbacks when XDG_CONFIG_HOME isn't set so we need to override this with a real directory instead of just unsetting it.

But how can this happen? Well, for example if you're trying to run the test suite on git master when you already have a xkb directory for #162 :)

Copy link
Member

@bluetech bluetech left a comment

Choose a reason for hiding this comment

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

libxkbcommon has fallbacks when XDG_CONFIG_HOME isn't set so we need to override this with a real directory instead of just unsetting it.

A comment saying this would be nice, otherwise it seems odd.

Otherwise LGTM (feel free to merge).

Let's not have our tests fail if the user has an incompatible
$XDG_CONFIG_HOME/xkb directory.

libxkbcommon has fallbacks when XDG_CONFIG_HOME isn't set so we need to override
this with a real directory instead of just unsetting it.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
@whot whot force-pushed the wip/test-ignore-home branch from bd75b92 to f943539 Compare July 27, 2020 09:20
@whot
Copy link
Contributor Author

whot commented Jul 27, 2020

comment added, merging. thanks for the review

@whot whot merged commit 03ecaa6 into xkbcommon:master Jul 27, 2020
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