-
Notifications
You must be signed in to change notification settings - Fork 24
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
Move away from the xdg
package
#239
Comments
Hi @stdedos thank you for opening the issue and pointing out the solution. Will fix it as soon as I can. Maybe if you make a PR the fix will be implemented sooner (although I think the CI is broken right now :( ) |
Working on it right now |
I'm afraid that we can't migrate to I'm adding a comment on the code to migrate away once support for 3.9 is dropped. We use the library literally for one line, I don't think we'd need the features of the new package nor think it's probable that we hit a critical vulnerability running Therefore I'm closing the issue, but if you think of a better solution I'm fine reopening it again |
Can you try to use https://pypi.org/project/pyxdg/ for the <3.10 versions? My concern is not "vulnerabilities"; my real issue is
Ofc I could've used a venv for https://github.com/lyz-code/autoimport/ to make this problem go away. However, I have a logical gap:
|
For these kind of tools I use Adding PyXDG for Python <3.10 and xdg-base-dirs for Python >3.10 looks an unnecessary complex increase, and I'm not quite sure how that will work with Even if I don't like the idea, if you feel strong about it and want to make a PR I'll check it, and if it's not that complex I'm fine with merging it. Despite if you use it to solve this issue I encourage you to give |
Description
https://pypi.org/project/xdg/:
tl;dr: you'd need to change this
autoimport/pyproject.toml
Line 34 in f0ac03d
(and that
autoimport/pyproject.toml
Line 382 in f0ac03d
)
and then this
autoimport/src/autoimport/entrypoints/cli.py
Line 9 in f0ac03d
to
xdg_base_dirs
(and the one reference ofc 😅)The text was updated successfully, but these errors were encountered: