Bug 365811 - Review Request: gnome-applet-tvn24 - Scrolled RSS aggregator for the polish TVN24 channel
Summary: Review Request: gnome-applet-tvn24 - Scrolled RSS aggregator for the polish T...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Michał Bentkowski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-11-04 14:51 UTC by Jakub 'Livio' Rusinek
Modified: 2007-11-30 22:12 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-11-05 20:21:51 UTC
Type: ---
Embargoed:
mr.ecik: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Jakub 'Livio' Rusinek 2007-11-04 14:51:30 UTC
Spec URL: http://liviopl.jogger.pl/files/gnome-applet-tvn24.spec
SRPM URL: http://liviopl.jogger.pl/files/gnome-applet-tvn24-0.2.8-1.fc7.src.rpm
Description: Scrolled RSS aggregator for the polish TVN24 channel

Comment 1 Michał Bentkowski 2007-11-04 15:12:02 UTC
Fails to build on x86_64:

error: Installed (but unpackaged) file(s) found:
   /usr/lib/bonobo/servers/TVN24.server
   /usr/lib/tvn24/tvn24
   /usr/lib/tvn24/tvn24.png

RPM build errors:
    user livio does not exist - using root
    group livio does not exist - using root
    user livio does not exist - using root
    group livio does not exist - using root
    user livio does not exist - using root
    group livio does not exist - using root
    File not found: /var/tmp/gnome-applet-tvn24-0.2.8-1.fc7-root-build/usr/
lib64/tvn24
    File not found: /var/tmp/gnome-applet-tvn24-0.2.8-1.fc7-root-build/usr/
lib64/bonobo/servers/TVN24.server
    Installed (but unpackaged) file(s) found:
   /usr/lib/bonobo/servers/TVN24.server
   /usr/lib/tvn24/tvn24
   /usr/lib/tvn24/tvn24.png

It seems you gotta make some fixes in Makefile.am. 
You can easily use sed to this task, e.g:
sed -i "s#tvn24dir = .*#tvn24dir = %{_libdir}/tvn24#" Makefile.am
sed -i "s#data = .*#data = %{_datadir}/tvn24#" Makefile.am
etc.
(there aren't many files to install so it may be easier to do that by yourself,
instead of fixing Makefile.am)
Also there are some hard-coded paths in TVN24.server file,
use sed once again to fix them.


Comment 3 Jakub 'Livio' Rusinek 2007-11-04 16:07:43 UTC
PS: It is not my first review request, but I'm waiting for a sponsor.

Comment 5 Michał Bentkowski 2007-11-05 12:46:36 UTC
REVIEW:

 * %{?dist} tag present
 * package licensed under GPLv3, license text included
 * rpmlint is quiet
 * source files match upstream
 * md5sum fine: 384b022405a673ff75307905735904ea
 * BRs seem well (not checked by mock though)
 * final provides and requires sane
 * package doesn't create any new directory and there's nothing wrong with 
owning
 * %clean section present
 * buildroot fine
 * %files section good
 * no check suite
 * no scriptlets required
 * no subpackages
 * no .la files
 * no GUI

I'm going to sponsor you.
Go to http://fedoraproject.org/wiki/PackageMaintainers/Join now,
read point 4 of "get a Fedora Account" and I'll approve your membership.
Information on all the things you need to do next you'll find on the site
I mentioned above. If you have any problems, questions, feel free to contact me.
Good luck!

Comment 6 Jakub 'Livio' Rusinek 2007-11-05 13:48:40 UTC
New Package CVS Request
=======================
Package Name: gnome-applet-tvn24
Short Description: Scrolled RSS aggregator for the polish TVN24 channel
Owners: liviopl
Branches: F-8
InitialCC: liviopl
Cvsextras Commits:yes

Comment 7 Kevin Fenzi 2007-11-05 16:44:40 UTC
cvs done.

Comment 8 Jakub 'Livio' Rusinek 2007-11-05 16:45:42 UTC
Package Change Request
======================
Package Name: gnome-applet-tvn24
New Branches: F-7

Comment 9 Jason Tibbitts 2007-11-05 20:12:07 UTC
CVS done.

Comment 10 Jakub 'Livio' Rusinek 2007-11-05 20:21:51 UTC
Thanks for help, Fedora users and developers :) .
Package built successfully do F-7, F-8 and devel.

Ready for users. Closing as NEXTRELASE.


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