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

docs: document media_baseurl property of components #371

Merged
merged 1 commit into from Dec 29, 2021

Conversation

pabloyoyoista
Copy link
Contributor

This property is inserted[1] into XML collection metadata by
appstream-generator, but is undocumented.

[1]https://github.com/ximion/appstream-generator/blob/master/src/asgen/engine.d#L318

This property is inserted[1] into XML collection metadata by
appstream-generator, but is undocumented.

[1]https://github.com/ximion/appstream-generator/blob/master/src/asgen/engine.d#L318
@ximion
Copy link
Owner

ximion commented Dec 29, 2021

Jup, this was an experimental feature for a while, that kind of became the norm for collection data.
Thanks for the patch!

@ximion ximion merged commit e245588 into ximion:master Dec 29, 2021
@pabloyoyoista pabloyoyoista deleted the xml-media_baseurl branch December 29, 2021 10:29
gnomesysadmins pushed a commit to GNOME/gnome-software that referenced this pull request Jan 18, 2022
This property was not documented for XML data until quite
recently: ximion/appstream#371

However, remote icons and screenshots generated with appstream-generator
only contain urls relative to media_baseurl property. Fix all relative uris
prepending the corresponding media_baseurl for those objects to be downloadable.
gnomesysadmins pushed a commit to GNOME/gnome-software that referenced this pull request Mar 14, 2022
This property was not documented for XML data until quite
recently: ximion/appstream#371

However, remote icons and screenshots generated with appstream-generator
only contain urls relative to media_baseurl property. Fix all relative uris
prepending the corresponding media_baseurl for those objects to be downloadable.
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