fix PREFIX in .desktop file #390
Closed
Conversation
This file contains 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
> By default Tiled will install to /usr/local. ([README](https://github.com/bjorn/tiled/commits/master/README.md))
|
Right, this is because the .desktop file (and the install step) was expected to be only used by distributions. For running Tiled from the source directory, no install step is needed. However, it should still be fixed. I guess it's better to have the file match the default prefix, but the real fix would be to generate the .desktop file and fill in the prefix that is being used. For reference, this is about issue #383. |
|
Closing this request since the plan is to go with a proper solution of either automatically putting the right path in there or to stop using absolute paths entirely. Going with this intermediate patch would only cause more work for current packagers. @Ablu is currently looking into it. |
Ablu
pushed a commit
to Ablu/tiled
that referenced
this pull request
Mar 28, 2013
The application icons now install into the system icon theme. This also results in the system always picking the best available icon to display. To make installing simpler (and to make the names more consistent) the icon files were moved into the folders of their resolution. Fixes: mapeditor#383 Obsolotes: mapeditor#390
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
(README)