Bug 2089366

Summary: Generated RPM tag License is missing MIT
Product: [Fedora] Fedora Reporter: Petr Pisar <ppisar>
Component: fonts-rpm-macrosAssignee: Parag Nemade <pnemade>
Status: NEW --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 39CC: pnemade, tagoh
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Petr Pisar 2022-05-23 14:05:36 UTC
When porting sil-mingzat-fonts package the the current packaging standard which uses macros from fonts-rpm-macros-2.0.5-7.fc36.noarch, I found that the generated RPM License is not sufficient.

The font binary RPM package has:

  License     : OFL

because of in the spec file,

  %global fontlicense     OFL

but the generated file /usr/share/metainfo/org.fedoraproject.sil-mingzat-fonts.metainfo.xml reads:

  <?xml version="1.0" encoding="UTF-8"?>
  <!-- SPDX-License-Identifier: MIT -->
  <component type="font">
    <id>org.fedoraproject.sil-mingzat-fonts</id>
    <metadata_license>MIT</metadata_license>
    <project_license>OFL</project_license>

Therefore an expected License tag is "OFL and MIT".

I cannot work around with "%global fontlicense OFL and MIT" because that would store a wrong license into project_license element. I think fonts-rpm-macros needs a correction.

Comment 1 Parag Nemade 2022-05-25 04:44:55 UTC
There is no correct or wrong way decided. During the RHEL9 development I discussed this similar query to internal legal team as well but i did not get final answer. There are many scenarios to look at this same issue of listing licenses from source code in package license tag. I think its upto package maintainer to decide whether to add metadata license in package license tag or not.

Now i suppose here that you want to keep license for sil-mingzat-fonts package as "OFL and MIT" and not just "OFL".

Okay then either I or someone will check the macros implementation and will see what need to be fixed. I will also look forward to packaging-committee to decide on this as it will be fonts guidelines change or better say all types of packaging guidelines license tag issue. I am sure there can be many packages in Fedora which do not include metadata license to its package license tag.

Also, remember this new fonts guidelines were written and implemented by Nicolas Mailhot only and I guess he is not around or not contributing to Fedora project currently.

Comment 2 Akira TAGOH 2022-05-25 05:46:32 UTC
Given that all the licenses for all the files in the package should be added to License tag, this issue looks like a bug in fonts-rpm-macros because License tag and metainfo file is genereated by it and metadata_license tag is hardcoded in their macro. we should fix fonts-rpm-macros to add correct license to License tag.

Comment 3 Petr Pisar 2022-05-25 11:33:46 UTC
Thanks for looking into it. That's what I want.

Comment 4 Parag Nemade 2022-05-31 08:43:50 UTC
Reported https://pagure.io/packaging-committee/issue/1179
now let's wait till FPC decides on this.

Comment 5 Ben Cotton 2022-08-09 13:16:47 UTC
This bug appears to have been reported against 'rawhide' during the Fedora Linux 37 development cycle.
Changing version to 37.

Comment 6 Fedora Release Engineering 2023-08-16 07:05:38 UTC
This bug appears to have been reported against 'rawhide' during the Fedora Linux 39 development cycle.
Changing version to 39.