Bug 420931 - Review Request: libertas-usb8388-firmware - firmware for libertas usb8388 wireless adapter
Summary: Review Request: libertas-usb8388-firmware - firmware for libertas usb8388 wir...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jima
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-12-12 04:18 UTC by Dennis Gilmore
Modified: 2007-12-13 02:19 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-12-13 02:19:22 UTC
Type: ---
Embargoed:
jima: fedora-review+
dennis: fedora-cvs+


Attachments (Terms of Use)

Description Dennis Gilmore 2007-12-12 04:18:29 UTC
Spec URL: http://ausil.us/packages/libertas-usb8388-firmware.spec
SRPM URL: http://ausil.us/packages/libertas-usb8388-firmware-5.110.20.p47-1.olpc2.src.rpm
Description: 
Firmware for Marvell Libertas USB 8388 Network Adapter

Comment 1 Jima 2007-12-12 19:09:50 UTC
Picking up for review.

Comment 3 Jima 2007-12-12 19:38:28 UTC
This will be brief, since there's not really that much to review. :-)

- MD5SUM matches upstream, as well as the self-contained .md5 file.
- LICENSE looks reasonable enough.
- Driver is in kernel RPM, so binary-only firmware is permitted.
- Package builds in mock.
- rpmlint has no complaints.

I don't have hardware to test, but I'm sure you've verified that side of
things.

By the power vested in me by my sponsor, I now pronounce this package   
APPROVED.  You may now request CVS.

Comment 4 Dennis Gilmore 2007-12-12 20:27:16 UTC
New Package CVS Request
=======================
Package Name: libertas-usb8388-firmware
Short Description: Firmware for Marvell Libertas USB 8388 Network Adapter
Owners: ausil
Branches: F-7 F-8 
InitialCC: 
Cvsextras Commits:


cvs done

Comment 5 Dennis Gilmore 2007-12-12 20:27:43 UTC
thanks for the review


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