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

Accept any XDG_ environment variable to determine desktop #2037

Merged
merged 4 commits into from
May 23, 2024

Conversation

mlsmaycon
Copy link
Collaborator

Describe your changes

Issue ticket number and link

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

@mlsmaycon mlsmaycon requested a review from bcmmbaga May 23, 2024 09:46
@mlsmaycon mlsmaycon merged commit 67e2185 into main May 23, 2024
23 checks passed
@mlsmaycon mlsmaycon deleted the check-for-xdg-prefixed-env-vars branch May 23, 2024 10:34
@CertainLach
Copy link

CertainLach commented May 23, 2024

XDG_ variables exist not only on desktop, default debian server installation also sets those variables using pam & systemd integration

@mlsmaycon
Copy link
Collaborator Author

@CertainLach thanks for pointing that up. The change will be reverted.

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.

3 participants