-
-
Notifications
You must be signed in to change notification settings - Fork 326
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
aravis-tools: Correct appstream metainfo description and file name #932
Conversation
Co-authored-by: Petter Reinholdtsen <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Chiara,
Thanks for the pull request.
The appdata filename change should keep the api version, and the component ID should be fixed to include it.
viewer/data/meson.build
Outdated
@@ -4,7 +4,7 @@ appdata_conf = configuration_data() | |||
appdata_conf.set ('ARAVIS_API_VERSION', aravis_api_version) | |||
|
|||
configure_file (input: files('org.aravis.viewer.appdata.xml.in'), | |||
output: 'org.aravis.viewer-@[email protected].xml'.format (aravis_api_version), | |||
output: 'org.aravis.viewer.metainfo.xml'.format (aravis_api_version), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Different aravis series are supposed to be installable in parallel. This change breaks this possibility.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Understood!
The last commit should have fixed this issue, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nope... I have to modify the id tag too... :) coming soon...
Thanks Chiara. |
Dear aravis maintainers,
this pull request fixes a bug reported during debian packaging, applying the patch offered by @petterreinholdtsen.
It corrects appstream metainfo validation errors, adding missing
<p>
to description and making sure file name match ID.Also it uses the new
metainfo.xml
ending replacing the obsoleteappdata.xml
ending.Thanks for considering it.