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

Add basic AppStream metainfo data #16

Merged

Conversation

Conan-Kudo
Copy link
Contributor

For packaging the Red Hat fonts in Linux distributions (such as Fedora),
AppStream metadata is required to support discoverability and installability
through modern software centers like GNOME Software and Plasma Discover.

This adds the most basic data for supporting that, which can be fleshed
out more over time as needed.

For packaging the Red Hat fonts in Linux distributions (such as Fedora),
AppStream metadata is required to support discoverability and installability
through modern software centers like GNOME Software and Plasma Discover.

This adds the most basic data for supporting that, which can be fleshed
out more over time as needed.
@Conan-Kudo
Copy link
Contributor Author

This is being used for the Fedora and EPEL packaging for the Red Hat fonts, by the way.

@Conan-Kudo
Copy link
Contributor Author

ping?

1 similar comment
@Conan-Kudo
Copy link
Contributor Author

ping?

@starryeyez024
Copy link
Member

Hi @Conan-Kudo - thank you for submitting this MR. I've notified the appropriate folks who need to review it.

@starryeyez024
Copy link
Member

Adding information from the email thread:

I'd now be inclined not to use a special metadata license and to assume that OFL 1.1 covers font metadata well enough. This is mainly because of my lingering concerns around the use of "Red Hat" as the name of these fonts (and even though the CC licenses are clear that they do not license trademark rights).

@richardfontana please add further details here as necessary, thanks!

@Conan-Kudo
Copy link
Contributor Author

There's a limited number of metadata licenses accepted by the AppStream spec today, mostly CC licenses. I've typically used CC-BY-SA for my metadata licenses for metainfo files, because it's a good default for that and does handle trademarks appropriately.

@funnelfiasco
Copy link

@richardfontana is CC BY-SA acceptable for the metadata given @Conan-Kudo's comment?

@richardfontana
Copy link
Contributor

@funnelfiasco @Conan-Kudo @starryeyez024 Yes this is fine. Thanks!

@Conan-Kudo
Copy link
Contributor Author

@starryeyez024 Can we get it merged now?

@starryeyez024 starryeyez024 merged commit 65d7792 into RedHatOfficial:master Jul 29, 2019
@Conan-Kudo Conan-Kudo deleted the add-redhat-font-metainfo branch July 29, 2019 00:52
@Conan-Kudo
Copy link
Contributor Author

@starryeyez024 Thanks for merging it!

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.

4 participants