Bug 917191 - Review Request: awake - A command to 'wake on LAN' a remote host
Summary: Review Request: awake - A command to 'wake on LAN' a remote host
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Björn Esser (besser82)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-03-01 22:56 UTC by Fabian Affolter
Modified: 2013-08-15 03:05 UTC (History)
3 users (show)

Fixed In Version: awake-1.0-1.fc19
Clone Of:
Environment:
Last Closed: 2013-08-15 03:03:38 UTC
Type: ---
Embargoed:
besser82: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Fabian Affolter 2013-03-01 22:56:55 UTC
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/awake.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/awake-1.0-1.fc18.src.rpm

Project URL: https://github.com/cyraxjoe/awake

Description:
A command to 'wake on LAN' a remote host.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=5067854

rpmlint output:
[fab@localhost SRPMS]$ rpmlint awake-1.0-1.fc18.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

[fab@localhost noarch]$ rpmlint *awake*
awake.noarch: W: no-manual-page-for-binary awake
3 packages and 0 specfiles checked; 0 errors, 1 warnings.

Fedora Account System Username: fab

Comment 1 Björn Esser (besser82) 2013-05-16 20:24:11 UTC
Hi Fabian!

fedora-review and manual inspection shows me all is fine, but...

there are three rpms:

awake-1.0-1.fc20.noarch.rpm 
python-awake-1.0-1.fc20.noarch.rpm
python3-awake-1.0-1.fc20.noarch.rpm

awake-1.0-1 seems to be the main package having /usr/bin/awake executing the main-method (located in python{,3}-awake} on system's default python-interpreter. But there's no dependency on python{,3}-awake in the rpm...

I think you should resolve this using a virtual package (e.g. Provides: pyawake = %{version}) in python{,3}-awake and depending on this virt-pkg in awake-1.0-1 (Requires: pyawake = %{version}). This way yum will decide itself how to resolve this on the "cheapest" way, when a user invokes `yum install awake`.

Adding a man-page using helptoman, pod or asciidoc would be nice, too.

Since you are in packager-group since five years or so, I suppose it is ok when you fix this on SCM.

APPROVED!

Comment 2 Fabian Affolter 2013-06-02 12:19:47 UTC
Thanks for the review, Björn.

Comment 3 Fabian Affolter 2013-06-02 12:21:22 UTC
New Package SCM Request
=======================
Package Name: awake
Short Description: A command to 'wake on LAN' a remote host
Owners: fab
Branches: F18 F19
InitialCC:

Comment 4 Gwyn Ciesla 2013-06-03 10:23:45 UTC
Git done (by process-git-requests).

Comment 5 Fedora Update System 2013-07-27 19:12:22 UTC
awake-1.0-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/awake-1.0-1.fc19

Comment 6 Fedora Update System 2013-07-27 19:13:04 UTC
awake-1.0-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/awake-1.0-1.fc18

Comment 7 Fedora Update System 2013-07-29 00:29:23 UTC
awake-1.0-1.fc18 has been pushed to the Fedora 18 testing repository.

Comment 8 Fedora Update System 2013-08-15 03:03:38 UTC
awake-1.0-1.fc18 has been pushed to the Fedora 18 stable repository.

Comment 9 Fedora Update System 2013-08-15 03:05:28 UTC
awake-1.0-1.fc19 has been pushed to the Fedora 19 stable repository.


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