Bug 1231863 - Review Request: bcm283x-firmware - Broadcom bcm283x firmware for the Raspberry Pi
Summary: Review Request: bcm283x-firmware - Broadcom bcm283x firmware for the Raspberr...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: armhfp
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-06-15 13:58 UTC by Peter Robinson
Modified: 2015-06-18 17:20 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-06-18 17:20:02 UTC
Type: Bug
Embargoed:
tcallawa: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Peter Robinson 2015-06-15 13:58:38 UTC
SPEC: https://pbrobinson.fedorapeople.org/bcm283x-firmware.spec
SRPM: https://pbrobinson.fedorapeople.org/bcm283x-firmware-20150615-1.37600d5.fc22.src.rpm
description:

Firmware for the Broadcom bcm283x SoC as shipped in devices such as the
Raspberry Pi

koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=10060140

Comment 1 Tom "spot" Callaway 2015-06-18 14:09:43 UTC
Minor items:

* the license tag must be:

"License: Redistributable, no modification permitted"

You've got a comment tacked on to the end, change it to look like this:

# see LICENSE.broadcom
License: Redistributable, no modification permitted

* Add a period at the end of the %description text.

* Change the URL to the github link, that's way more accurate.

Everything else looks clean here. There is no compilation, just file copying, and the source is correct. Make those changes, show me the fixed spec, and I'll approve.

Comment 3 Tom "spot" Callaway 2015-06-18 15:12:56 UTC
Look at that license tag again. Almost there. ;)

Comment 4 Peter Robinson 2015-06-18 15:18:58 UTC
Bah.

SRPM: https://pbrobinson.fedorapeople.org/bcm283x-firmware-20150615-3.37600d5.fc22.src.rpm

Spec url the same

Comment 5 Tom "spot" Callaway 2015-06-18 15:21:07 UTC
Nice work. APPROVED.

Comment 6 Peter Robinson 2015-06-18 15:40:35 UTC
spot: thanks a lot for the review

New Package GIT Request
=======================
Package Name: bcm283x-firmware
Short Description: Broadcom bcm283x firmware for the Raspberry Pi 
Owners: pbrobinson pwhalen
Branches: F-22
InitialCC:

Comment 7 Gwyn Ciesla 2015-06-18 16:46:43 UTC
Git done (by process-git-requests).


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