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

hardcoded use lxterminal in the .desktop files. #3

Open
meuter opened this issue Oct 28, 2020 · 12 comments
Open

hardcoded use lxterminal in the .desktop files. #3

meuter opened this issue Oct 28, 2020 · 12 comments

Comments

@meuter
Copy link
Owner

meuter commented Oct 28, 2020

The .desktop files generated by the script hardcode Exec='lxterminal ...; which does not work on ubuntu-mate for instance.
Maybe use Terminal=true and run the script directly? to be tested...

(initially reported by Joel W.)

@meuter
Copy link
Owner Author

meuter commented Oct 28, 2020

Hi Joel,

Glad I could help. Thanks for the report. I actually did not know the script created .desktop files to integrate with the menu. I created an issue on github to keep track of it:

#3

I had a quick look at the section of the script that generates those desktop files:

https://github.com/meuter/argon-one-case-ubuntu-20.04/blob/master/argon1.sh#L530

And apparently, they run the script using lxterminal. Not sure why they picked that one in particular. But for portability, instead of using a specific terminal emulator (like mate-terminal in your case), I would suggest trying to use the default terminal provided by the desktop environment. I don't have my raspberry pie at hand at the moment (it's on loan at one of my colleagues) so I cannot test this, but I would try something like this: set the Terminal variable to true and change the Exec variable to execute the script directly. So for the config script, it would look like this:

echo "[Desktop Entry]" > $shortcutfile
echo "Name=Argon One Configuration" >> $shortcutfile
echo "Comment=Argon One Configuration" >> $shortcutfile
echo "Icon=/usr/share/pixmaps/ar1config.png" >> $shortcutfile
echo 'Exec=lxterminal -t "Argon One Configuration" --working-directory=/home/pi/ -e '$configscript >> $shortcutfile
echo "Type=Application" >> $shortcutfile
echo "Encoding=UTF-8" >> $shortcutfile
echo "Terminal=false" >> $shortcutfile
echo "Categories=None;" >> $shortcutfile

should be change to

echo "[Desktop Entry]" > $shortcutfile
echo "Name=Argon One Configuration" >> $shortcutfile
echo "Comment=Argon One Configuration" >> $shortcutfile
echo "Icon=/usr/share/pixmaps/ar1config.png" >> $shortcutfile
echo 'Exec='$configscript >> $shortcutfile
echo "Type=Application" >> $shortcutfile
echo "Encoding=UTF-8" >> $shortcutfile
echo "Terminal=true" >> $shortcutfile
echo "Categories=None;" >> $shortcutfile

And do a similar modification for the uninstall script. If you're willing to test this on your side, I can merge this change easily.

Thanks!

Kind regards

-Cédric

@joelw135
Copy link

joelw135 commented Oct 28, 2020 via email

@meuter
Copy link
Owner Author

meuter commented Oct 28, 2020

@joelw135 no worries, I'll take a look when I have my rpi4 back.

@joelw135
Copy link

joelw135 commented Oct 29, 2020 via email

@meuter
Copy link
Owner Author

meuter commented Nov 20, 2020

@joelw135 I just got my RPI4 back from my colleague. So I'll take care of this over the week-end.

@joelw135
Copy link

Thank you it is appreciated.

@meuter
Copy link
Owner Author

meuter commented Nov 22, 2020

Fixed in 3eda093

@meuter meuter closed this as completed Nov 22, 2020
@joelw135
Copy link

Thank you so very much. I just ordered the M.2 case from them. And I am hoping I will still be able to boot from the SSD. Thanks again.

@joelw135
Copy link

joelw135 commented Nov 22, 2020

I tried the script and it doesn't create the desktop icons. I have tried it using ssh and directly from terminal.

@meuter
Copy link
Owner Author

meuter commented Nov 25, 2020

That's rather strange. On my end, the two desktop shortcuts are created and they both work with the default terminal provided by the desktop environment as mentioned above. In order to investigate why it does not work on your side, could you try running the script using bash -ex argon1.sh and send me the output? I'll see if I can spot the error on your side.

Regarding the icons, they are downloaded from argon40's website and copied to /usr/share/pixmaps/ by the script itself. The icons are then referenced in the two .desktop file. Here's where one of the .destop file is generated in the script:

sudo wget http://download.argon40.com/ar1config.png -O /usr/share/pixmaps/ar1config.png
sudo wget http://download.argon40.com/ar1uninstall.png -O /usr/share/pixmaps/ar1uninstall.png
# Create Shortcuts
# NOTE(cme): don't assume /home/pi/
# shortcutfile="/home/pi/Desktop/argonone-config.desktop"
shortcutfile="$desktop/argonone-config.desktop"
echo "[Desktop Entry]" > $shortcutfile
echo "Name=Argon One Configuration" >> $shortcutfile
echo "Comment=Argon One Configuration" >> $shortcutfile
echo "Icon=/usr/share/pixmaps/ar1config.png" >> $shortcutfile
# NOTE(cme): don't assume lxterminal is installed
# echo 'Exec=lxterminal -t "Argon One Configuration" --working-directory=/home/pi/ -e '$configscript >> $shortcutfile
echo "Exec=$configscript" >> $shortcutfile
echo "Type=Application" >> $shortcutfile
echo "Encoding=UTF-8" >> $shortcutfile
# NOTE(cme): use builtin terminal instead
# echo "Terminal=false" >> $shortcutfile
echo "Terminal=true" >> $shortcutfile
echo "Categories=None;" >> $shortcutfile
chmod 755 $shortcutfile

These two lines download the icons:

sudo wget http://download.argon40.com/ar1config.png -O /usr/share/pixmaps/ar1config.png 
sudo wget http://download.argon40.com/ar1uninstall.png -O /usr/share/pixmaps/ar1uninstall.png 

In a .desktop file, the path to the icon is specified in Icon variable:

Icon=/usr/share/pixmaps/ar1config.png

@meuter meuter reopened this Nov 25, 2020
@joelw135
Copy link

joelw135 commented Nov 25, 2020 via email

@joelw135
Copy link

joelw135 commented Dec 15, 2020 via email

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