Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

Wrong config files placement when XDG_CONFIG_HOME is set #289

Closed
1 of 5 tasks
ariqfadlan opened this issue Jun 25, 2020 · 1 comment · Fixed by #290
Closed
1 of 5 tasks

Wrong config files placement when XDG_CONFIG_HOME is set #289

ariqfadlan opened this issue Jun 25, 2020 · 1 comment · Fixed by #290

Comments

@ariqfadlan
Copy link
Contributor

How have you installed cordless

  • Arch User Repository
  • Snap
  • scoop
  • brew
  • go get

What happened

When XDG_CONFIG_HOME is set, all config files is placed on that directory instead of $XDG_CONFIG_HOME/cordless

How do you reproduce this bug

$ export XDG_CONFIG_HOME=~/.config
$ cordless

The login screen will output: Config lies at: /home/user/.config. Whereas after executing

$ unset XDG_CONFIG_HOME
$ cordless

login screen reports Config lies at: /home/user/.config/cordless.

How to fix

On my machine, unset XDG_CONFIG_HOME is enough since it points on ~/.config anyway.
In the meantime, forcing config directory with -config-dir command line flag along with desired path is also works.

System information

Key Value
OS Debian Stable
Architechture x86_64
Go version go1.13.1 linux/amd64
ariqfadlan added a commit to ariqfadlan/cordless that referenced this issue Jun 25, 2020
@ariqfadlan
fix Bios-Marcel#289
f6139c1
@ariqfadlan ariqfadlan mentioned this issue Jun 25, 2020
Merged
Bios-Marcel pushed a commit that referenced this issue Jun 25, 2020
@ariqfadlan
fix #289 (#290)
5391a14 
Config directory was the system-configurated directory directly instead of a app-named subdirectory.
@Bios-Marcel
Copy link
Owner

ty

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix config directory ariqfadlan/cordless
2 participants
@Bios-Marcel @ariqfadlan