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

Desktop path string get cut when it is too long #67

Open
grapemix opened this issue Dec 2, 2024 · 2 comments
Open

Desktop path string get cut when it is too long #67

grapemix opened this issue Dec 2, 2024 · 2 comments

Comments

@grapemix
Copy link

grapemix commented Dec 2, 2024

In short, when the entry's image path or the .desktop path are too long, the path as string will be cut silently and make the launching action fails. My setup is basically identical with the https://complexlogic.github.io/flex-launcher/setup_linux#kiosk-mode-setup. My path is so long because I want to use Jellyfin media player and their recommended installation is thru flatpak.

# Menu configurations
[Main]
Entry1=JF;/home/USERNAME/.config/flex-launcher/assets/icons/kodi.png;~/.local/share/jellyfin-media-player.desktop;TVF
Entry2=term;/home/USERNAME/.config/flex-launcher/assets/icons/kodi.png;/usr/share/applications/Alacritty.desktop
Entry3=Steam;/home/USERNAME/.config/flex-launcher/assets/icons/steam.png;/usr/share/applications/steam.desktop;BigPicture
Entry4=System;/home/USERNAME/.config/flex-launcher/assets/icons/system.png;:submenu System

The work around is create a sym link which makes the path shorter. I spent quite a while on this bug until I use the debug mode and examine the log. Anyone should be able to reproduce pretty quick by using a very long path.

@complexlogic
Copy link
Owner

Thanks for the report. This is actually a compile time configuration variable in the inih library, which Flex Launcher uses to read the ini file. Since I don't control the library or the packaging of it, changing it will not be easy. However, I will consider adding a note to the documentation to inform users of the limitation.

@grapemix
Copy link
Author

In that way, I am cool to close this issue. Thanks for the response. I will leave it open in case you want to use this ticket to track the doc update.

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

No branches or pull requests

2 participants