RedHatOfficial / RedHatFont

Red Hat's Open Source Fonts - Red Hat Display and Red Hat Text
SIL Open Font License 1.1
479 stars 55 forks source link

Add basic AppStream metainfo data #16

Closed Conan-Kudo closed 5 years ago

Conan-Kudo commented 5 years ago

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 commented 5 years ago

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

Conan-Kudo commented 5 years ago

ping?

Conan-Kudo commented 5 years ago

ping?

starryeyez024 commented 5 years ago

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

starryeyez024 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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

richardfontana commented 5 years ago

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

Conan-Kudo commented 5 years ago

@starryeyez024 Can we get it merged now?

Conan-Kudo commented 5 years ago

@starryeyez024 Thanks for merging it!