Bug 1418520 - linux-firmware missing %license macro
Summary: linux-firmware missing %license macro
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: linux-firmware
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: David Woodhouse
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: BaseRuntimeDocs, BaseRuntimeLicense
TreeView+ depends on / blocked
 
Reported: 2017-02-02 01:39 UTC by Stephen Gallagher
Modified: 2017-02-03 12:39 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-02-03 12:39:37 UTC
Type: Bug


Attachments (Terms of Use)
Patch to add %license (1.27 KB, patch)
2017-02-02 01:39 UTC, Stephen Gallagher
jwboyer: review+
Details | Diff

Description Stephen Gallagher 2017-02-02 01:39:43 UTC
Created attachment 1246951 [details]
Patch to add %license

Description of problem:
This package contains one or more files containing the package license in the source tarball, but it is not installed on the system with the %license macro. This is important, because the license files will not be installed if the RPM is installed with docs disabled (such as when generating a container base image), which may be a compliance violation.

A provenpackager will apply the attached patch in two weeks if we receive no negative response from the maintainer.

Comment 1 Josh Boyer 2017-02-02 02:22:03 UTC
Comment on attachment 1246951 [details]
Patch to add %license

Patch seems fine.  There's no reason to keep this limited to rawhide from a package standpoint, correct?  If not, I'll apply it tomorrow on all the branches.

Comment 2 Stephen Gallagher 2017-02-02 11:34:36 UTC
No, it's fine to backport it. Thanks!

Comment 3 Josh Boyer 2017-02-03 12:39:37 UTC
Patch applied on all branches and builds started.  Thanks!


Note You need to log in before you can comment on or make changes to this bug.