Skip to content

Add XDG path support (#476) #489

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add XDG path support (#476) #489

wants to merge 3 commits into from

Conversation

taep96
Copy link

@taep96 taep96 commented Mar 7, 2025

Fixes #476

Moved the .espup directory and the export file to $XDG_DATA_HOME

@SergioGasquez
Copy link
Member

Hi! Thanks for the PR, there are some failing test though, and we would also need the esp-idf-sys follow-up PR, otherwise this would leave it on a non-working state

@taep96
Copy link
Author

taep96 commented Mar 11, 2025

Hi! I planned to make a pr for esp-idf-sys after this one is done.

I started a review on the line causing the tests to fail. It seems to be checking whether the config location is a directory but it doesn’t exist yet; despite that it works properly on my system.

How should I deal with it?

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.

Install To Platform-Standard Locations | Reduce Home Directory Clutter
2 participants