Skip to content

Conversation

@hustlerone
Copy link

NixOS is non FHS compliant. Phrog will not be able to find graphical sessions there without this.

This patch makes it so Phrog reads off of XDG_DATA_DIRS first. In case the envvar is unavailable, it'll fall back to well-known FHS compliant directory.

@samcday samcday added the ci-ok PR looks safe / author is trusted, run pull_request_target build workflow label Jul 23, 2025
@hustlerone hustlerone force-pushed the main branch 2 times, most recently from 32fa44d to 1ef7f26 Compare July 25, 2025 09:40
@github-actions

This comment has been minimized.

@hustlerone hustlerone force-pushed the main branch 2 times, most recently from 31b6169 to fc0da83 Compare July 27, 2025 14:41
@github-actions
Copy link

The CI build recorded and generated some videos

Demo video (shown on README and release notes)

accent-colours

emergency-calls

first-run

simple-flow

trivial-flow

@samcday
Copy link
Owner

samcday commented Nov 26, 2025

Hey @hustlerone thanks for this contribution. Sorry it took so long for me to get back to you. Gonna run a little experiment here, hope that's alright.

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@samcday
Copy link
Owner

samcday commented Nov 26, 2025

@codex review: please expand on #119 (comment) with a link to Rust std::env dox, and maybe also the (canonical) spec containing XDG_DATA_DIRS verbiage?

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@hustlerone hustlerone force-pushed the main branch 2 times, most recently from 84cf2dc to 0beb5c9 Compare January 2, 2026 14:22
@hustlerone
Copy link
Author

hustlerone commented Jan 2, 2026

once everything's satisfied i'll squash into one single commit to be merged

@hustlerone hustlerone force-pushed the main branch 4 times, most recently from 4b7a298 to 0539754 Compare January 2, 2026 14:52
@hustlerone
Copy link
Author

my computer is currently rebuilding 60gb of code sorry

@hustlerone
Copy link
Author

hustlerone commented Jan 2, 2026

i'll test this later on my machine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-ok PR looks safe / author is trusted, run pull_request_target build workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants