Bug 807375 - Review Request: kwooty - A friendly nzb usenet binary download application
Summary: Review Request: kwooty - A friendly nzb usenet binary download application
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-03-27 15:49 UTC by Gregor Tätzner
Modified: 2014-11-07 20:44 UTC (History)
3 users (show)

Fixed In Version: kwooty-0.8.3-2.fc17
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-04-08 03:27:54 UTC
Type: ---
rdieter: fedora-review+


Attachments (Terms of Use)

Description Gregor Tätzner 2012-03-27 15:49:15 UTC
Spec URL: http://brummbq.fedorapeople.org/kwooty.spec
SRPM URL: http://brummbq.fedorapeople.org/kwooty-0.8.3-1.fc16.src.rpm
Description:
Kwooty is a NZB usenet binary downloader for KDE 4. Features : automatic file verification - repairing with par2 files, automatic archive extraction (Rar, Zip and 7z archive formats supported), built-in YEnc and UUEncode file decoders, ...

atm I'm using version 0.8.3 - a non released version (contains various fixes) upstream send directly to me. I will upgrade as soon as possible.

Comment 1 Rex Dieter 2012-03-27 16:16:37 UTC
Quick initial comment:

1.  no need for a -devel pkg for a single lib symlink, with no headers or exported API (that I can tell).  I'd recommend omitting that from packaging (via whatever method you prefer, like simply deleting it in %install or using %exclude).

Comment 2 Kevin Kofler 2012-03-27 16:21:27 UTC
> Requires:       unrar
is not a valid Requires, packages in Fedora must not depend on packages which are not in Fedora (and unrar is not, because it's non-Free).

Comment 3 Gregor Tätzner 2012-03-27 16:38:27 UTC
(In reply to comment #2)
> > Requires:       unrar
> is not a valid Requires, packages in Fedora must not depend on packages which
> are not in Fedora (and unrar is not, because it's non-Free).

yeah forgot to remove that. if the user want the automatic rar archive extraction he has to install unrar manually. the app recognizes on runtime if unrar is installed or not. The same applies to 7z and par2 except they are in fedora. I think kwooty should require at least par2 (par2cmdline)

Comment 4 Rex Dieter 2012-03-27 19:35:39 UTC
So, in addition to stuff in comment #1 and comment #2 (lets consider that item 2), also:

3.  MUST:  Source0 not a full URL.  (maybe because it's not referring to an official release... yet?).  If so, please say so in a comment in the .spec


naming: ok

license: ok

sources: NOT ok, cannot verify (see 3)

macros: ok

scriptlets: ok


4.  MUST %files
change
%{_kde4_appsdir}/%{name}/*
to
%{_kde4_appsdir}/%{name}/
else the parent dir isn't properly owned


So, fixup items 1-4, and I think we're good to go.

Comment 5 Gregor Tätzner 2012-03-29 15:51:28 UTC
a new day a new kwooty. I think I addressed all your issues.

Spec URL: http://brummbq.fedorapeople.org/kwooty.spec
SRPM URL: http://brummbq.fedorapeople.org/kwooty-0.8.3-2.fc16.src.rpm

(In reply to comment #4)
> 3.  MUST:  Source0 not a full URL.  (maybe because it's not referring to an
> official release... yet?).  If so, please say so in a comment in the .spec
yep, it wasn't released...until today

Comment 6 Rex Dieter 2012-03-29 19:03:08 UTC
looks good to me, all issues fixed.  APPROVED.

Comment 7 Gregor Tätzner 2012-03-30 18:17:31 UTC
New Package SCM Request
=======================
Package Name: kwooty
Short Description: A friendly nzb usenet binary download application
Owners: brummbq
Branches: f15 f16 f17
InitialCC:

Comment 8 Gwyn Ciesla 2012-03-30 18:26:47 UTC
Git done (by process-git-requests).

Comment 9 Fedora Update System 2012-03-30 19:33:20 UTC
kwooty-0.8.3-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/kwooty-0.8.3-2.fc15

Comment 10 Fedora Update System 2012-03-30 19:34:33 UTC
kwooty-0.8.3-2.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/kwooty-0.8.3-2.fc16

Comment 11 Fedora Update System 2012-03-30 19:36:03 UTC
kwooty-0.8.3-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/kwooty-0.8.3-2.fc17

Comment 12 Fedora Update System 2012-03-31 03:17:32 UTC
kwooty-0.8.3-2.fc16 has been pushed to the Fedora 16 testing repository.

Comment 13 Fedora Update System 2012-04-08 03:27:54 UTC
kwooty-0.8.3-2.fc16 has been pushed to the Fedora 16 stable repository.

Comment 14 Fedora Update System 2012-04-08 03:30:40 UTC
kwooty-0.8.3-2.fc15 has been pushed to the Fedora 15 stable repository.

Comment 15 Fedora Update System 2012-04-12 02:40:17 UTC
kwooty-0.8.3-2.fc17 has been pushed to the Fedora 17 stable repository.

Comment 16 Rex Dieter 2014-11-07 20:27:46 UTC
Package Change Request
======================
Package Name: kwooty
New Branches: f21
Owners: rdieter

Comment 17 Gwyn Ciesla 2014-11-07 20:44:48 UTC
Complete.


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