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

Install the AppStream file to the canonical location #2195

Merged
merged 1 commit into from Apr 20, 2020
Merged

Install the AppStream file to the canonical location #2195

merged 1 commit into from Apr 20, 2020

Conversation

pinotree
Copy link
Contributor

The canonical location for AppStream XML files has been changed to /usr/share/metainfo four years ago at least, with /usr/share/appdata left as legacy location. It is time to switch to the right location.

The canonical location for AppStream XML files has been changed to
/usr/share/metainfo four years ago at least, with /usr/share/appdata
left as legacy location. It is time to switch to the right location.
@justinclift
Copy link
Member

Awesome, thanks @pinotree. 😄

@justinclift justinclift merged commit ad39882 into sqlitebrowser:master Apr 20, 2020
@justinclift justinclift added this to the 3.12.0 - Next release milestone Apr 20, 2020
@pinotree pinotree deleted the appdata-location branch April 20, 2020 06:38
@pinotree
Copy link
Contributor Author

Why the osx label?

@justinclift
Copy link
Member

Ahhh. Should it be Linux instead of osx?

@pinotree
Copy link
Contributor Author

The appdata file is installed on any UNIX system actually (just like the application .desktop file).
It is used on Linux mostly though.

@justinclift justinclift removed the osx label Apr 20, 2020
@justinclift
Copy link
Member

Thanks. Just killed the osx label, as it's really more just my habit of adding them rather than something I actively use. Not sure if other people use them though. 😉

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

Successfully merging this pull request may close these issues.

None yet

2 participants