Bug 655496 - Review Request: cambozola - A viewer for multipart jpeg streams
Summary: Review Request: cambozola - A viewer for multipart jpeg streams
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
Assignee: Mat Booth
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 654052
TreeView+ depends on / blocked
 
Reported: 2010-11-21 11:46 UTC by Cédric OLIVIER
Modified: 2012-03-19 11:49 UTC (History)
2 users (show)

Fixed In Version: cambozola-0.92-2.fc14
Clone Of:
Environment:
Last Closed: 2011-01-29 17:25:21 UTC
Type: ---
Embargoed:
mat.booth: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Cédric OLIVIER 2010-11-21 11:46:06 UTC
SPEC URL : http://cedric.olivier.free.fr/rpms/cambozola-0.80-1/cambozola.spec
SRPM URL : http://cedric.olivier.free.fr/rpms/cambozola-0.80-1/cambozola-0.80-1.fc13.src.rpm

Description:
Cambozola is a very simple (cheesey!) viewer for multipart jpeg streams
that are often pumped out by a streaming webcam server,
sending over multiple images per second.

rpmlint output :
cambozola.src: W: spelling-error Summary(en_US) multipart -> multiparty, multipara, multipartite
cambozola.src: W: spelling-error Summary(en_US) jpeg -> peg, j peg, MPEG
cambozola.src: W: spelling-error %description -l en_US cheesey -> cheese, cheesy, cheeses
cambozola.src: W: spelling-error %description -l en_US multipart -> multiparty, multipara, multipartite
cambozola.src: W: spelling-error %description -l en_US jpeg -> peg, j peg, MPEG
cambozola.src: W: spelling-error %description -l en_US webcam -> web cam, web-cam, webcast
1 packages and 0 specfiles checked; 0 errors, 6 warnings.

mock build :
SRPM builds fine in mock.

Comment 1 Hicham HAOUARI 2010-12-21 13:05:13 UTC
the links are unreachable

Comment 2 Cédric OLIVIER 2010-12-22 17:07:50 UTC
I don't understand, links are reachable for me.

I have done an update with last release 0.92.

SPEC URL : http://cedric.olivier.free.fr/rpms/cambozola-0.92-1/cambozola.spec
SRPM URL :
http://cedric.olivier.free.fr/rpms/cambozola-0.92-1/cambozola-0.92-1.fc13.src.rpm

Comment 3 Hicham HAOUARI 2010-12-22 17:50:24 UTC
Thanks, the links are working now.

First remark : did you test if this package works with gcj ?

Comment 4 Cédric OLIVIER 2010-12-22 19:44:49 UTC
I have tested it with gcj, it works ... but frame rate is very low.

Comment 5 Mat Booth 2011-01-13 15:16:44 UTC
I can take this review.

Comment 6 Mat Booth 2011-01-13 16:34:12 UTC
Ok, a quick look shows the following needs to be corrected:

* Licence is incorrect, it should be GPLv2+ according the distributed licence and source file headers.

* Unless you plan on participating in EPEL5, the BuildRoot tag is obsolete. EPEL6+ and Fedora 10+ ignore it entirely and it may be removed.

* There is no require for "Requires: %{name}" in the javadoc sub-package. This should be removed.

* In the recently approved new Java guidelines, we no longer should be shipping versioned jars or javadocs, so please drop the symlinks and install everything without the %{version} suffix.

* A %clean section containing only "rm -rf $RPM_BUILD_ROOT" is no longer needed (except perhaps if you intend on participating in EPEL5) and should be removed

* Please reduce the spelling errors that rpmlint complains about where you can. Change "javadocs" to "javadoc" and change "cheesey" to "cheesy". The rest really should be in the dictionary, so I'm willing to overlook them.

Also, if you wish you may put the accent aigu "é" in your name in the change log -- you are allowed to submit UTF-8 encoded spec files. :-)

Comment 7 Mat Booth 2011-01-13 16:54:40 UTC
BTW, if you remove the BuildRoot tag, you may ignore the rpmlint warning "no-buildroot-tag."

Comment 8 Cédric OLIVIER 2011-01-16 07:52:47 UTC
SPEC URL : http://cedric.olivier.free.fr/rpms/cambozola-0.92-2/cambozola.spec
SRPM URL :
http://cedric.olivier.free.fr/rpms/cambozola-0.92-2/cambozola-0.92-2.fc13.src.rpm

Licence is now corrected.

For EPEL5 participation, I don't know. actually cambozola is only used in zoneminder package which is a camera monitoring and analysis tool, it should be a good idea to make zoneminder available for EPEL. Is it possible to leave the BuildRoot tag ?

Removed unneeded requires in javadoc section.

According to Java packaging guidelines version removed from jar file name.

Remove clean section.

Reduce the spelling errors in description.

Comment 9 Mat Booth 2011-01-16 14:07:30 UTC
(In reply to comment #8)
> For EPEL5 participation, I don't know. actually cambozola is only used in
> zoneminder package which is a camera monitoring and analysis tool, it should be
> a good idea to make zoneminder available for EPEL. Is it possible to leave the
> BuildRoot tag ?
> 


Yes, that is fine. Also leave in the %clean section, as EPEL5 also needs that, so make sure that gets added back before you request an EPEL5 branch. :-)

I don't see anything else wrong with this package, so it is:

APPROVED!

Comment 10 Cédric OLIVIER 2011-01-18 19:21:56 UTC
New Package SCM Request
=======================
Package Name: cambozola
Short Description: A viewer for multipart jpeg streams
Owners: cquad
Branches: F-13 F-14
InitialCC: cquad

Comment 11 Jason Tibbitts 2011-01-19 21:58:53 UTC
Git done (by process-git-requests).

Comment 12 Fedora Update System 2011-01-20 19:48:44 UTC
cambozola-0.92-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/cambozola-0.92-2.fc14

Comment 13 Fedora Update System 2011-01-20 19:50:08 UTC
cambozola-0.92-2.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/cambozola-0.92-2.fc13

Comment 14 Fedora Update System 2011-01-21 22:58:37 UTC
cambozola-0.92-2.fc14 has been pushed to the Fedora 14 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update cambozola'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/cambozola-0.92-2.fc14

Comment 15 Fedora Update System 2011-01-29 17:25:16 UTC
cambozola-0.92-2.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 16 Fedora Update System 2011-01-29 17:26:07 UTC
cambozola-0.92-2.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 17 Cédric OLIVIER 2012-03-18 10:36:49 UTC
Package Change Request
======================
Package Name: cambozola
New Branches: el6
Owners: cquad

Comment 18 Gwyn Ciesla 2012-03-19 11:49:13 UTC
Git done (by process-git-requests).


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