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: Fix `.desktop` file & force refresh #2479

Merged
merged 1 commit into from Feb 11, 2020

Conversation

@mrwulf
Copy link
Contributor

mrwulf commented Feb 9, 2020

Version in the desktop file caused parsing errors. Also, force the app file to be loaded/reloaded.

@tessus tessus added the linux label Feb 9, 2020
@laurent22

This comment has been minimized.

Copy link
Owner

laurent22 commented Feb 11, 2020

@tessus, @CalebJohn, any idea if we can merge this?

@tessus

This comment has been minimized.

Copy link
Collaborator

tessus commented Feb 11, 2020

@laurent22 yes, we can.

The previous desktop file was malformed. And the refresh only happens, if the file update-desktop-database exists. So no harm done. LGTM.

@laurent22

This comment has been minimized.

Copy link
Owner

laurent22 commented Feb 11, 2020

Perfect, thanks guys.

@laurent22 laurent22 merged commit ca5d6c5 into laurent22:master Feb 11, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.