Bug 187625 - Review Request: ices IceS is a source client for a streaming server
Summary: Review Request: ices IceS is a source client for a streaming server
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Ignacio Vazquez-Abrams
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-04-02 00:52 UTC by Richard June
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-04-27 21:09:29 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Richard June 2006-04-02 00:52:16 UTC
I do not have a sponsor for this. It passes RPMLint, and I believe it adheres to the guidelines.
ices.spec: http://home.bravegnuworld.com/~rjune/rpm/SPECS/ices.spec

ices-2.0.1-0.1.src.rpm: http://home.bravegnuworld.com/~rjune/rpm/SRPMS/ices-2.0.1-0.1.src.rpm

Description: 
IceS is a source client for a streaming server. The purpose of this client is
to provide an audio stream to a streaming server such that one or more
listeners can access the stream. With this layout, this source client can be
situated remotely from the icecast server

Comment 1 Ignacio Vazquez-Abrams 2006-04-02 02:08:12 UTC
- Release should start at 1
- Summary should not contain the app name
- Drop / before %{_localstatedir} and %{_sysconfdir}
- Need to own %{_datadir}/%{name}
- Don't be afraid to use %{name} and %{version} where appropriate

Comment 2 Richard June 2006-04-02 02:43:43 UTC
Fixed all of those. 
ices-2.0.1-2.src.rpm: 
http://home.bravegnuworld.com/~rjune/rpm/SRPMS/ices-2.0.1-2.src.rpm 

Comment 3 Ignacio Vazquez-Abrams 2006-04-05 04:05:13 UTC
- First word of summary should be capitalized
- Drop %{_datadir}/ices/* from %files as the previous line covers it
- Put one newline between chnagelog entries

Comment 4 Richard June 2006-04-05 05:20:48 UTC
Fixed all of those.  
ices-2.0.1-3.src.rpm:  
http://home.bravegnuworld.com/~rjune/rpm/SRPMS/ices-2.0.1-3.src.rpm  

Comment 5 Ignacio Vazquez-Abrams 2006-04-09 18:53:26 UTC
Almost there.

W: ices conffile-without-noreplace-flag /etc/logrotate.d/ices

Also a number of build warnings I don't like, but I wont't consider them
blockers. Kick them upstream.

Also, I can't help but think that most of the items in %{_datadir}/ices might be
better off all being in %doc instead.

Comment 6 Richard June 2006-04-09 20:02:34 UTC
Fixed the conffile without noreplace flag     
ices-2.0.1-4.src.rpm:     
http://home.bravegnuworld.com/~rjune/rpm/SRPMS/ices-2.0.1-4.src.rpm  
  

Comment 7 Richard June 2006-04-15 19:53:20 UTC
Moved HTML files to %doc  
ices-2.0.1-5.src.rpm:       
http://home.bravegnuworld.com/~rjune/rpm/SRPMS/ices-2.0.1-5.src.rpm    

Comment 8 Ignacio Vazquez-Abrams 2006-04-20 02:38:56 UTC
The HTML and CSS files have been moved, but the XML files are now MIA. Also, the
XML files should probably be marked %config(noreplace) and relocated to
%{_sysconfdir}/%{name}.

Comment 9 Ignacio Vazquez-Abrams 2006-04-27 21:09:29 UTC
So it turns out that someone had already submitted ices for review. Silly me for
not noticing.


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