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

Add location of Xresources for XDG Base Dir Spec #251

Merged
merged 1 commit into from
Jun 14, 2023
Merged

Add location of Xresources for XDG Base Dir Spec #251

merged 1 commit into from
Jun 14, 2023

Conversation

kinjalik
Copy link
Contributor

Some of users want to keep their configs in unified folder. XDG Base Directory allows to do so, but there is no support of than in ly

@nullgemm nullgemm added ok This issue or pull request was confirmed/reviewed needs review This issue or pull request must be confirmed/reviewed labels Apr 24, 2022
@AnErrupTion
Copy link
Collaborator

@kinjalik I don't understand. The line above is for ~/.Xresources, isn't that the correct path?

@UtkarshVerma
Copy link
Contributor

While this could be merged. Based on the code we have, we could already achieve the desired functionality.

We can specify xnitrc in /etc/ly/config.ini:

xinitrc = ~/.config/X11/xinitrc

This will tell ly to source xinitrc from this custom path. I merge xrdb in my xinitrc which makes everything work out.

@AnErrupTion
Copy link
Collaborator

This will tell ly to source xinitrc from this custom path. I merge xrdb in my xinitrc which makes everything work out.

The problem about this is that not everybody launches their DE/WM from an xinitrc file. By directly including it in the xsetup.sh, it ensures everything can be merged even if they select another entry than xinitrc.

@AnErrupTion AnErrupTion merged commit 93cb839 into fairyglade:master Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review This issue or pull request must be confirmed/reviewed ok This issue or pull request was confirmed/reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants