Open
Conversation
HashBangAU
commented
Jan 25, 2026
HashBangAU
commented
- Recommend manual install to a dir accessible to all users, to prevent issues with TryExec check before login
- Do the find+replace for path fixing in .desktop files after moving them, rather than inside the repo/install before
- Mention that the session technique only works with X11 and not Wayland
* Recommend manual install to a dir accessible to all users, to prevent issues with TryExec check before login * Do the find+replace for path fixing in .desktop files after moving them, rather than inside the repo/install before
HashBangAU
force-pushed
the
readme-desktop-file-instructions
branch
from
634902f to
e00fc29
Compare
porridge
reviewed
Feb 7, 2026
Owner
porridge
left a comment
porridge
left a comment
There was a problem hiding this comment.
Thanks, this looks good, but the bit about Debian is a bit confusing - AFAICT the directory in /etc is valid for Debian, but not on PopOS, so perhaps we should make the note a bit more generic and let the user find out on their own?
| sudo sed -i -e "s,/usr/games/bambam,`pwd`/bambam.py," /etc/X11/sessions/bambam-session.desktop | ||
| ``` | ||
|
|
||
| The path shown above as `/etc/X11/sessions/` may vary, depending on your Linux distribution. For example, it may be `/usr/share/xsessions/` on Debian. |
Owner
There was a problem hiding this comment.
Suggested change
| The path shown above as `/etc/X11/sessions/` may vary, depending on your Linux distribution. For example, it may be `/usr/share/xsessions/` on Debian. | |
| The path shown above as `/etc/X11/sessions/` may vary, depending on your Linux distribution. For example, it may be `/usr/share/xsessions/` on some distributions. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.