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.

Sign up for GitHub

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

Jump to bottom

USE XDG_CACHE instead of ~/.config/wslu #290

Closed
1 task done
woutervh opened this issue Oct 11, 2023 · 1 comment
Closed
1 task done

USE XDG_CACHE instead of ~/.config/wslu #290

woutervh opened this issue Oct 11, 2023 · 1 comment
Assignees
@patrick330602
Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Patched This is fixed and is part of a released version. Target: 4.x This is targeted to the 4.x release.

Comments

@woutervh
Copy link

Windows Version

Windows 11

Windows Build Number

latest

WSL Version

WSL 2

Distro Version

Almalinux

WSL Utilities Version

wslu v4.1.1-1

Describe the bug

files are being created in a hardcoded ~/.config/wslu:

  • baseexec
  • oemcp
  • triggered_time

but these are not config-files.
rather cache-files or state

To avoid hardcoding: better use:
XDG_CONFIG_HOME (default to ~/.config)

or better:
XDG_STATE_HOME (default to~/.local/state)
XDG_CACHE_HOME (default to~/.cache)

Steps to Reproduce

ls ~/.config/wlu

  • baseexec
  • oemcp
  • triggered_time

Expected behavior

ls ~/.cache/wlu

  • baseexec
  • oemcp
  • triggered_time

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@woutervh woutervh added the Bug This is either an issue that describes a bug or a PR that resolves a bug label Oct 11, 2023
@patrick330602 patrick330602 added Status: In Progress This is being worked on and not ready. Target: 4.x This is targeted to the 4.x release. labels Nov 12, 2023
@patrick330602
Copy link
Member

Thanks for the report and currently working on it.

patrick330602 added a commit that referenced this issue Nov 12, 2023
@patrick330602
fix: [#290] use proper folder for the states
1e450d8
@patrick330602 patrick330602 added Status: Pending Release This is fixed but not yet released. and removed Status: In Progress This is being worked on and not ready. labels Nov 12, 2023
@patrick330602 patrick330602 mentioned this issue Dec 31, 2023
Merged
@patrick330602 patrick330602 added Status: Patched This is fixed and is part of a released version. and removed Status: Pending Release This is fixed but not yet released. labels Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@patrick330602 patrick330602

Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Patched This is fixed and is part of a released version. Target: 4.x This is targeted to the 4.x release.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@woutervh @patrick330602