Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

fix .desktop support #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix .desktop support #1

wants to merge 1 commit into from

Conversation

noerw
Copy link

@noerw noerw commented Aug 20, 2019

Thanks for the docker image & the launch script!

Launching from gnome desktop was not working. Changes:

  • -t flag in docker run prevents from running in non-interactive shell.
    Calls from an interactive shell are still working according to my tests.
  • documented installation of qgis.desktop file
  • sudo docker is considered an antipattern

Comment=GIS 2.18 LTR in Docker
Exec=~/Documents/launch_scripts/docker_qgis.sh %U
Comment=QGIS 2.18 LTR in Docker
Exec=/home/norwin/bin/qgis2 %U
Copy link
Member

Choose a reason for hiding this comment

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

here the user name is hardcoded, could that be changed?

Copy link
Author

@noerw noerw Sep 2, 2019

Choose a reason for hiding this comment

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

That is what this PR is primarily about:
In my environment (Gnome 3.32.2), the .desktop file is not recognized when relative paths or enironment variables are in the path.

Terminal=false
Type=Application
StartupNotify=true
MimeType=text/shp;
Icon=/usr/local/qgis-icon-60x60.png
Icon=/home/norwin/src/docker-qgis2/qgis-icon-60x60.png
Copy link
Member

Choose a reason for hiding this comment

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

here the user name is hardcoded, could that be changed?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants