Bug 426098 - Review Request: libwiimote - library for the Nintendo Wii Remote
Summary: Review Request: libwiimote - library for the Nintendo Wii Remote
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Marek Mahut
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-12-18 15:11 UTC by Adam Jackson
Modified: 2008-01-07 22:06 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-12-21 16:31:12 UTC
Type: ---
Embargoed:
mmahut: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Adam Jackson 2007-12-18 15:11:01 UTC
Spec URL: http://ajax.fedorapeople.org/libwiimote/libwiimote.spec
SRPM URL: http://ajax.fedorapeople.org/libwiimote/libwiimote-0.4-1.fc9.src.rpm

Description:
Libwiimote is a C-library that provides a simple API for communicating with the Nintendo Wii Remote (aka. wiimote) on a Linux system. The goal of this library is to be a complete and easy to use framework for interfacing applications with the wiimote.

Comment 1 Adam Jackson 2007-12-18 15:56:00 UTC
Broken, doesn't build in koji due to -fpic hilarity, and the headers go to
someplace completely insane.  New stuff:

Spec URL: http://ajax.fedorapeople.org/libwiimote/libwiimote.spec
SRPM URL: http://ajax.fedorapeople.org/libwiimote/libwiimote-0.4-2.fc9.src.rpm

Builds in koji, so I think this one is solid.

Comment 2 Marek Mahut 2007-12-18 19:39:32 UTC
Hello Adam,

The package looks good, only cosmetics issue.

W: mixed-use-of-spaces-and-tabs

Comment 3 Tom "spot" Callaway 2007-12-18 21:32:12 UTC
License tag is incorrect, should be GPLv2+.

Comment 4 Adam Jackson 2007-12-18 22:15:23 UTC
New Package CVS Request
=======================
Package Name: libwiimote
Short Description: Library for the Nintendo Wii Remote
Owners: ajax
Branches: F-7
InitialCC: 
Cvsextras Commits: yes

Comment 5 Kevin Fenzi 2007-12-19 02:30:31 UTC
Do you really want only F-7 and devel (assumed) branches? Not also F-8?

Also, There appears to be a stray "#Requires:' in the spec. ;) 


Comment 6 Adam Jackson 2007-12-19 19:52:43 UTC
erk, no, meant F-8 not F-7.

New Package CVS Request
=======================
Package Name: libwiimote
Short Description: Library for the Nintendo Wii Remote
Owners: ajax
Branches: F-8
InitialCC: 
Cvsextras Commits: yes

Comment 7 Jason Tibbitts 2007-12-19 20:55:10 UTC
CVS done.

Comment 8 Adam Jackson 2007-12-21 16:31:12 UTC
Imported.  Thanks!


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