Skip to content

desktop: Normalize launcher file modes - #245

Merged
BenWestgate merged 2 commits into
masterfrom
codex/desktop-launcher-modes
Sep 27, 2026
Merged

BenWestgate merged 2 commits into
masterfrom
codex/desktop-launcher-modes

Conversation

@BenWestgate

Copy link
Copy Markdown
Owner

Make the tracked desktop launchers regular 0644 data files instead of executable 0755 files.

This is intentionally a separate, mode-only change from the #202/#230 wallet work so it is easy to review. No launcher contents change.

@BenWestgate BenWestgate left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

AI-generated review (Claude), posted at the maintainer's request.

ACK b5c62e7

  • Menu entries in ~/.local/share/applications don't need +x.
  • bails/.config/autostart/bitcoin.desktop is still 0755. Include it for consistency?
  • Conflicts with #202 (modify/delete on decrypt-vault.desktop, which #202 removes) and with #230. Drop that file from this PR, or land it after #202.

@BenWestgate
BenWestgate enabled auto-merge (rebase) September 27, 2026 23:57
@BenWestgate
BenWestgate merged commit 462c5f1 into master Sep 27, 2026
4 of 5 checks passed
@BenWestgate
BenWestgate deleted the codex/desktop-launcher-modes branch September 27, 2026 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant