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

feat: Respect XDG Base Directory Specification on Linux #3310

Closed
1 task done
imtuyethan opened this issue Aug 8, 2024 · 1 comment
Closed
1 task done

feat: Respect XDG Base Directory Specification on Linux #3310

imtuyethan opened this issue Aug 8, 2024 · 1 comment
Labels
category: app shell Installation, updater, global application issues P1: important Important feature / fix type: feature request A new feature

Comments

@imtuyethan
Copy link
Contributor

  • I have searched the existing issues

Is your feature request related to a problem? Please describe it

I help created request for this user:
Screenshot 2024-08-08 at 10 50 42 PM

Jan currently doesn't follow the XDG Base Directory Specification on Linux systems. This goes against established conventions and best practices for Linux application development.

Describe the solution

Update Jan to respect the XDG Base Directory Specification by moving the Jan folder to the XDG_DATA_HOME directory (usually ~/.local/share/jan).

Teachability, documentation, adoption, migration strategy

What is the motivation / use case for changing the behavior?

  • Improve compatibility with Linux systems
  • Follow established conventions, making it easier for users to manage app data
  • Align with best practices for Linux application development
  • Potentially improve system cleanliness by storing data in the expected location
@imtuyethan imtuyethan added the type: feature request A new feature label Aug 8, 2024
@imtuyethan imtuyethan assigned namchuai and marknguyen1302 and unassigned namchuai Aug 8, 2024
@imtuyethan imtuyethan moved this to Planned in Menlo Aug 8, 2024
@imtuyethan imtuyethan added this to the v0.6.1 milestone Aug 8, 2024
@imtuyethan imtuyethan removed this from the v0.7.1 milestone Aug 28, 2024
@freelerobot freelerobot added the P1: important Important feature / fix label Sep 5, 2024
@freelerobot freelerobot added the category: app shell Installation, updater, global application issues label Oct 14, 2024
@imtuyethan
Copy link
Contributor Author

imtuyethan commented Jan 20, 2025

After internal discussion, we've decided to keep Jan's current directory structure since:

  1. Jan is designed to self-manage its data folder including Cortex data
  2. The current implementation allows Jan to properly handle its data isolation
  3. This approach enables Jan to maintain proper separation between Jan and Cortex components
  4. The current Jan app location in ~/.config/Jan actually follows common app formatting conventions. Since Jan manages its own data folder structure and this is working as intended.

@github-project-automation github-project-automation bot moved this from Icebox to QA in Menlo Jan 20, 2025
@imtuyethan imtuyethan moved this from QA to Completed in Menlo Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: app shell Installation, updater, global application issues P1: important Important feature / fix type: feature request A new feature
Projects
Status: Completed
Development

No branches or pull requests

4 participants