migrate database paths to XDG user data directory #1

Merged

alisson merged 6 commits from fix/fix-data-location into master 2026-02-23 00:13:58 -03:00

Conversation 1 Commits 6 Files changed 6 +23 -29

andrielfr commented 2026-02-22 10:23:51 -03:00

Member

Copy link

summary:

• replace hardcoded build-time database paths (ZAP_DATABASE_PATH, PAPO_DATABASE_PATH) with a runtime DATA_DIR static that uses glib::user_data_dir(), following XDG Base Directory conventions
• remove PKGDATADIR-based path constants from config.rs.in and inline path construction at call sites

motivation:
database paths were previously baked in at compile time via PKGDATADIR, tying them to the install prefix. this meant dev builds wrote to /home/user/.local/share/papo/ regardless of actual XDG config, and the paths couldn't adapt to different environments at runtime. using glib::user_data_dir() resolves this by respecting the user's XDG data directory.

summary: - replace hardcoded build-time database paths (ZAP_DATABASE_PATH, PAPO_DATABASE_PATH) with a runtime DATA_DIR static that uses glib::user_data_dir(), following XDG Base Directory conventions - remove PKGDATADIR-based path constants from config.rs.in and inline path construction at call sites motivation: database paths were previously baked in at compile time via PKGDATADIR, tying them to the install prefix. this meant dev builds wrote to /home/user/.local/share/papo/ regardless of actual XDG config, and the paths couldn't adapt to different environments at runtime. using glib::user_data_dir() resolves this by respecting the user's XDG data directory.

andrielfr self-assigned this 2026-02-22 10:23:51 -03:00

alisson was assigned by andrielfr 2026-02-22 10:23:51 -03:00

andrielfr added 4 commits 2026-02-22 10:23:51 -03:00

build: add glib v0.22.0 dependency ... de544c7710

Adds glib crate to support user data directory queries

refactor: migrate database paths to glib user data directory ... 40ac24a653

Replaces hardcoded database paths with glib::user_data_dir() for better cross-platform support

- Removes hardcoded paths from config

- Updates session client to use dynamic path

- Updates database store to use dynamic path

refactor: tidy imports, dependency order, and static placement ... bb54fce337

- alphabetize glib in Cargo.toml
- import PathBuf directly
- group static declarations together near the top of main.rs.

refactor: inline database path helpers and import DATA_DIR directly ... 79013fb96a

removes zap_database_path() and papo_database_path() helper functions, inlining the path construction at call sites and importing DATA_DIR directly.

andrielfr 2026-02-22 10:24:03 -03:00

• added the
  enhancement
  label
• requested review from alisson

alisson added 1 commit 2026-02-22 12:56:14 -03:00

chore: restore cargo files e0c6b9d588

andrielfr added 1 commit 2026-02-23 00:03:59 -03:00

fix: ensure data directory is created on startup 7f76a4c893

alisson approved these changes 2026-02-23 00:09:21 -03:00

alisson force-pushed fix/fix-data-location from 7f76a4c893 to e8e72b9ebd 2026-02-23 00:12:42 -03:00 Compare

alisson merged commit c943a9ec5b into master 2026-02-23 00:13:58 -03:00

alisson referenced this pull request from a commit 2026-02-23 00:13:59 -03:00

chore: migrate database paths to XDG user data directory (#1)

Sign in to join this conversation.

Reviewers

No reviewers

alisson

Labels

Clear labels   

bug

Something is not working

  

duplicate

This issue or pull request already exists

  

enhancement

New feature

  

help wanted

Need some help

  

invalid

Something is wrong

  

question

More information is needed

  

wontfix

This won't be fixed

No labels

bug

duplicate

enhancement

help wanted

invalid

question

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

andrielfr

alisson

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

AmanoTeam/Papo!1

No description provided.