[Bug 589167] Review Request: perl-ParseTemplate - ParseTemplate Perl module

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589167

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #3 from Marcela Mašláňová mmasl...@redhat.com 2010-05-06 03:15:48 
EDT ---
Thank, I'll remove this.

New Package CVS Request
===
Package Name: perl-ParseTemplate
Short Description: ParseTemplate Perl module
Owners: mmaslano
Branches: devel
InitialCC: perl-sig

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 574977] Review Request: rubygem-amqp - AMQP client implementation in Ruby/EventMachine

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=574977

Matthew Kent mk...@magoazul.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Matthew Kent mk...@magoazul.com 2010-05-06 03:22:11 EDT 
---
New Package CVS Request
===
Package Name: rubygem-amqp
Short Description: AMQP client implementation in Ruby/EventMachine
Owners: mkent
Branches: F-11 F-12 F-13
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588473] Review Request: rubygem-rr - double techniques for ruby

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588473

Matthew Kent mk...@magoazul.com changed:

   What|Removed |Added

 CC||mk...@magoazul.com

--- Comment #1 from Matthew Kent mk...@magoazul.com 2010-05-06 04:11:01 EDT 
---
Quick notes, not a full review:

* MIT licensed
* Summary is way too long

See general notes in Bug #588474

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588474] Review Request: rubygem-rubyzip - zipfile support in Ruby

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588474

Matthew Kent mk...@magoazul.com changed:

   What|Removed |Added

 CC||mk...@magoazul.com

--- Comment #4 from Matthew Kent mk...@magoazul.com 2010-05-06 04:09:16 EDT 
---
Wow, lots of rubygem submissions. This applies to all of yours:

This isn't a full review but have you read

https://fedoraproject.org/wiki/Packaging/Ruby

you'll need at least the rubyabi added, see 

http://magoazul.com/wip/SPECS/rubygem-amqp.spec

for a reference (accepted yesterday).

Also give
http://fedoraproject.org/wiki/PackagingGuidelines#.25global_preferred_over_.25define
a look.

The ruby_sitelib macro is unnecessary for anything without a compiled
extension.

Finally have you verified the License for each? Bug #588473 for example is MIT
licensed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 582234] Review Request: python-eventlet - Highly concurrent networking library

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=582234

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2010-05-06 04:36:18 EDT ---
python-eventlet-0.9.7-1.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/python-eventlet-0.9.7-1.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 225834] Merge Review: gnome-python2

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=225834

Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org
Summary|Merge Review: gnome-python2 |Merge Review: gnome-python2
   |(stalled)   |
   Flag||fedora-review?

--- Comment #5 from Thomas Spura toms...@fedoraproject.org 2010-05-06 
04:57:45 EDT ---
ping

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588482] Review Request: libktorrent - Library providing torrent downloading code

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588482

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review+

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 545045] Review Request: CQRlib - ANSI C API for quaternion arithmetic and rotation

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=545045

--- Comment #11 from Takanori MATSUURA t.mat...@gmail.com 2010-05-06 05:21:37 
EDT ---
Updated
Spec URL: http://t-matsuu.sakura.ne.jp/mock/CQRlib/CQRlib.spec
SRPM URL:
http://t-matsuu.sakura.ne.jp/mock/CQRlib/CQRlib-1.0.5-1.fc12.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589479] New: Review Request: perl-Compress-Raw-Lzma - Low-level interface to lzma compression library

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-Compress-Raw-Lzma - Low-level interface to lzma 
compression library

https://bugzilla.redhat.com/show_bug.cgi?id=589479

   Summary: Review Request: perl-Compress-Raw-Lzma - Low-level
interface to lzma compression library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: p...@city-fan.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://subversion.city-fan.org/repos/cfo-repo/perl-Compress-Raw-Lzma/branches/fedora/perl-Compress-Raw-Lzma.spec
SRPM URL:
http://www.city-fan.org/~paul/extras/perl-Compress-Raw-Lzma/perl-Compress-Raw-Lzma-2.028-1.fc14.src.rpm
Description:

This module provides a Perl interface to the lzma compression library.
It is used by IO::Compress::Lzma.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589482] Review Request: perl-IO-Compress-Lzma - Read and write lzma compressed data

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589482

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

 Depends on||589479,
   ||573932(perl-IO-Compress)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589482] New: Review Request: perl-IO-Compress-Lzma - Read and write lzma compressed data

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-IO-Compress-Lzma - Read and write lzma compressed 
data

https://bugzilla.redhat.com/show_bug.cgi?id=589482

   Summary: Review Request: perl-IO-Compress-Lzma - Read and write
lzma compressed data
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: p...@city-fan.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://subversion.city-fan.org/repos/cfo-repo/perl-IO-Compress-Lzma/branches/fedora/perl-IO-Compress-Lzma.spec
SRPM URL:
http://www.city-fan.org/~paul/extras/perl-IO-Compress-Lzma/perl-IO-Compress-Lzma-2.027-1.fc12.src.rpm
Description:

This distribution provides a Perl interface to allow reading and writing of
compressed data created with the lzma library.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 573932] Review Request: perl-IO-Compress - IO::Compress Perl module

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=573932

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

 Blocks||589482

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589479] Review Request: perl-Compress-Raw-Lzma - Low-level interface to lzma compression library

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589479

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

 Blocks||589482

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588692] Review Request: rubygem-rerun - Restarts your app when file changes

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588692

--- Comment #4 from Michal Fojtik mfoj...@redhat.com 2010-05-06 06:02:36 EDT 
---
(In reply to comment #3)
 Everything looks good save for one more blocker. The changelog cannot contain
 any macros as the one you just added does.  I'd change the entry to something
 along the lines of:
 
 Fixed wrong path in check

Oh damm, thanks I just forget about this ;-)

Spec URL: http://mifo.sk/RPMS/rubygem-rerun.spec
SRPM URL: http://mifo.sk/RPMS/rubygem-rerun-0.5.2-2.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 520278] Review Request: notify-osd - On-screen-display notification agent

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=520278

--- Comment #26 from Gareth John gareth.l.j...@googlemail.com 2010-05-06 
06:51:32 EDT ---
ping Heyde??

Cwickert, do you know the rules on hijacking someone else's review request?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589504] New: Review Request: apache-commons-compress - Java API for working with tar, zip and bzip2 files

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: apache-commons-compress - Java API for working with 
tar, zip and bzip2 files

https://bugzilla.redhat.com/show_bug.cgi?id=589504

   Summary: Review Request: apache-commons-compress - Java API for
working with tar, zip and bzip2 files
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: s...@sandro-mathys.ch
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com,
akurt...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 CC||akurt...@redhat.com

Spec URL: http://red.fedorapeople.org/SRPMS/apache-commons-compress.spec
SRPM URL:
http://red.fedorapeople.org/SRPMS/apache-commons-compress-1.0-2.fc12.src.rpm
Description:
The code in this component came from Avalon's Excalibur, but originally
from Ant, as far as life in Apache goes. The tar package is originally
Tim Endres' public domain package. The bzip2 package is based on the
work done by Keiron Liddle. It has migrated via:
Ant - Avalon-Excalibur - Commons-IO - Commons-Compress.

Note: this is a re-review request for a package rename as described here:
http://fedoraproject.org/wiki/Package_Renaming_Process

The old package name was jakarta-commons-compress but was changed upstream to
apache-commons-compress. To see the difference between this and the old spec
file more easily I created a diff:
http://red.fedorapeople.org/SRPMS/apache-commons-compress.diff

As you can see nothing changed that's not necessary for the rename. Therefore
this review should be easy.

Let me also provide the rpmlint output:
$ rpmlint {SPECS,RPMS/noarch,SRPMS}/apache-commons-compress*
apache-commons-compress.noarch: W: spelling-error Summary(en_US) bzip - zip,
blip, b zip
apache-commons-compress.noarch: W: spelling-error %description -l en_US bzip -
zip, blip, b zip
apache-commons-compress-javadoc.noarch: W: spelling-error Summary(en_US)
Javadocs - Java docs, Java-docs, Javanese
apache-commons-compress.src: W: spelling-error Summary(en_US) bzip - zip,
blip, b zip
apache-commons-compress.src: W: spelling-error %description -l en_US bzip -
zip, blip, b zip
3 packages and 1 specfiles checked; 0 errors, 5 warnings.

I think all those warnings can be ignored as they're about technical terms
only.

--- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
07:05:38 EDT ---
Hmm, why are there 2 bugs for this?
See https://bugzilla.redhat.com/show_bug.cgi?id=589502

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589504] Review Request: apache-commons-compress - Java API for working with tar, zip and bzip2 files

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589504

--- Comment #2 from Sandro Mathys s...@sandro-mathys.ch 2010-05-06 07:11:00 
EDT ---
*** Bug 589502 has been marked as a duplicate of this bug. ***

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589502] New: Review Request: apache-commons-compress - Java API for working with tar, zip and bzip2 files

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: apache-commons-compress - Java API for working with 
tar, zip and bzip2 files

https://bugzilla.redhat.com/show_bug.cgi?id=589502

   Summary: Review Request: apache-commons-compress - Java API for
working with tar, zip and bzip2 files
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: CLOSED
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: s...@sandro-mathys.ch
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Sandro Mathys s...@sandro-mathys.ch changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||DUPLICATE

Spec URL: http://red.fedorapeople.org/SRPMS/apache-commons-compress.spec
SRPM URL:
http://red.fedorapeople.org/SRPMS/apache-commons-compress-1.0-2.fc12.src.rpm
Description:
The code in this component came from Avalon's Excalibur, but originally
from Ant, as far as life in Apache goes. The tar package is originally
Tim Endres' public domain package. The bzip2 package is based on the
work done by Keiron Liddle. It has migrated via:
Ant - Avalon-Excalibur - Commons-IO - Commons-Compress.

Note: this is a re-review request for a package rename as described here:
http://fedoraproject.org/wiki/Package_Renaming_Process

The old package name was jakarta-commons-compress but was changed upstream to
apache-commons-compress. To see the difference between this and the old spec
file more easily I created a diff:
http://red.fedorapeople.org/SRPMS/apache-commons-compress.diff

As you can see nothing changed that's not necessary for the rename. Therefore
this review should be easy.

Let me also provide the rpmlint output:
$ rpmlint {SPECS,RPMS/noarch,SRPMS}/apache-commons-compress*
apache-commons-compress.noarch: W: spelling-error Summary(en_US) bzip - zip,
blip, b zip
apache-commons-compress.noarch: W: spelling-error %description -l en_US bzip -
zip, blip, b zip
apache-commons-compress-javadoc.noarch: W: spelling-error Summary(en_US)
Javadocs - Java docs, Java-docs, Javanese
apache-commons-compress.src: W: spelling-error Summary(en_US) bzip - zip,
blip, b zip
apache-commons-compress.src: W: spelling-error %description -l en_US bzip -
zip, blip, b zip
3 packages and 1 specfiles checked; 0 errors, 5 warnings.

I think all those warnings can be ignored as they're about technical terms
only.

--- Comment #1 from Sandro Mathys s...@sandro-mathys.ch 2010-05-06 07:11:00 
EDT ---


*** This bug has been marked as a duplicate of bug 589504 ***

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

Sandro Mathys s...@sandro-mathys.ch changed:

   What|Removed |Added

 Depends on|589504  |

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

Sandro Mathys s...@sandro-mathys.ch changed:

   What|Removed |Added

 Depends on||589504

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589504] Review Request: apache-commons-compress - Java API for working with tar, zip and bzip2 files

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589504

Sandro Mathys s...@sandro-mathys.ch changed:

   What|Removed |Added

 Blocks|589168  |588764

--- Comment #4 from Sandro Mathys s...@sandro-mathys.ch 2010-05-06 07:31:00 
EDT ---
Awww, sorry...blocked the wrong bug :/ Bugzilla's errors are confusing me a
little :/

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 573932] Review Request: perl-IO-Compress - IO::Compress Perl module

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=573932

--- Comment #16 from Marcela Mašláňová mmasl...@redhat.com 2010-05-06 
07:30:32 EDT ---
(In reply to comment #15)
 Can we update the IO::Compress stack to 2.027? I need that for
 IO::Compress::Lzma and Compress::Raw::Lzma (IO::Compress interface to xz) that
 I'm ready to submit.

I've already updated BR. 
SRPM of this:
http://mmaslano.fedorapeople.org/review/perl-IO-Compress-2.027-1.fc14.src.rpm
RPM: http://mmaslano.fedorapeople.org/review/perl-IO-Compress.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 569038] Review Request: pinta - Simple painting for Gtk

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=569038

Sebastian Dziallas sebast...@when.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX

--- Comment #9 from Sebastian Dziallas sebast...@when.com 2010-05-06 07:38:58 
EDT ---
I've been rotating around here while working out some parts of 'life' as marked
here: http://fedoraproject.org/wiki/Vacation :)

While I still think that Pinta is a cool application, I might not be able to
continue to give it the attention it deserves. If somebody wants to package it,
I'm willing to act as co-maintainer, though.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 563318] Review Request: ceph - User space components of the CEPH file system

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=563318

--- Comment #21 from Fabian Deutsch fabian.deut...@gmx.de 2010-05-06 07:44:27 
EDT ---
@Josef , is the spec landing in CVS at some point? :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

--- Comment #4 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
07:44:53 EDT ---
Unversioned javadoc directory should be created as a symlink in %install and
there should be no %post/%postun for javadoc subpackage.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 520278] Review Request: notify-osd - On-screen-display notification agent

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=520278

--- Comment #27 from Christoph Wickert cwick...@fedoraproject.org 2010-05-06 
08:05:27 EDT ---
Don't hijack it but open a new one instead. Then close this one a duplicate of
your request.

Details: https://fedoraproject.org/wiki/Policy_for_stalled_package_review

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

--- Comment #5 from Stanislav Ochotnicky socho...@redhat.com 2010-05-06 
08:24:42 EDT ---
New SRPM URL:
http://sochotni.fedorapeople.org/apache-commons-logging-1.1.1-2.fc12.src.rpm

Spec URL: http://sochotni.fedorapeople.org/apache-commons-logging.spec


Few other QA improvement changes noted in changelog as well

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589504] Review Request: apache-commons-compress - Java API for working with tar, zip and bzip2 files

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589504

--- Comment #5 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
08:46:57 EDT ---
I would really like to see this package build with maven2 as it is supposed to
be done by upstream developers. 
All the additional maven integration work should be done too. 
We need this so any maven project depending on commons-compress can find it
without further work from the packager. Additionally building with maven should
make the jar a valid OSGi bundle suitable to be used in Eclipse and other OSGi
containers.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

--- Comment #6 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
09:11:27 EDT ---
Review:

OK: rpmlint must be run on every package. Output:

apache-commons-logging.noarch: W: spelling-error %description -l en_US runtime
- run time, run-time, untimely
apache-commons-logging.noarch: W: self-obsoletion jakarta-commons-logging =
0:1.0.4 obsoletes jakarta-commons-logging = 0:1.0.4
apache-commons-logging.noarch: W: non-conffile-in-etc
/etc/maven/fragments/apache-commons-logging

False positives. 

OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. 
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK: The License field in the package spec file must match the actual license.
OK: If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.
OK: The spec file must be written in American English. 
OK: The spec file for the package MUST be legible. 
OK: The sources used to build the package must match the upstream source, as
provided in the spec URL. 
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. 
OK: All build dependencies must be listed in BuildRequires, except for any that
are listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense.
OK: Packages must NOT bundle copies of system libraries.
OK: A package must own all directories that it creates. If it does not create a
directory that it uses, then it should require a package which does create that
directory. 
OK: A Fedora package must not list a file more than once in the spec file's
%files listings. 
NOTOK: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line. 

I don't see a reason why not using default %defattr(-,root,root,-)
From the guidelines:
Unless you have a very good reason to deviate from that, you should use
%defattr(-,root,root,-) for all %files sections in your package. 


OK: Each package must consistently use macros. 
OK: The package must contain code, or permissable content. 
OK: Large documentation files must go in a -doc subpackage. 
OK: If a package includes something as %doc, it must not affect the runtime of
the application. 
OK: Packages must not own files or directories already owned by other packages. 
OK: All filenames in rpm packages must be valid UTF-8. 


Please drop Package from the Summary.
Also javadoc subpackage will be better tohave a description like API
documentation for %{name}.
Please also use the default the permissions. I know that they will match in
this case but we should use the defaults.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 563318] Review Request: ceph - User space components of the CEPH file system

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=563318

--- Comment #22 from Josef Bacik jba...@redhat.com 2010-05-06 09:11:52 EDT ---
Yeah, I've had to fiddle with things to get it working nice for 0.20.  I plan
to add it to CVS tomorrow (since my fedora box is at home and i'm at work atm).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 587646] Review Request: not-yet-commons-ssl - Library to make SSL and Java Easier

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=587646

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

--- Comment #6 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
09:16:03 EDT ---
Built in rawhide.
http://koji.fedoraproject.org/koji/buildinfo?buildID=171870

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588654] Review Request: plexus-component-api - Plexus Component API

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588654

--- Comment #4 from Yang Yong yy...@redhat.com 2010-05-06 09:14:54 EDT ---
Fixed: 1.incoherent-version-in-changelog;2.drop the blank line immediately
following %changelog;3.typo %{prject_version}; 4. jars to plexus directory


Please review again.

Spec URL: http://yyang.fedorapeople.org/plexus/plexus-component-api.spec
SRPM URL:
http://yyang.fedorapeople.org/plexus/plexus-component-api-1.0-0.1.alpha15.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561452] Review Request: yydebug - Supports tracing and animation for a Java-based parser generated by jay

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561452

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #10 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
09:22:46 EDT ---
%changelog entries are missing your name. Otherwise the package looks good.
Please make sure that you fix the %changelog before committing.

This package is APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 563318] Review Request: ceph - User space components of the CEPH file system

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=563318

--- Comment #23 from Fabian Deutsch fabian.deut...@gmx.de 2010-05-06 09:23:36 
EDT ---
Ah okay, nice. Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 433398] Review Request: synbak - Synbak Universal Backup System (first package, need sponsor)

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=433398

Jon Ciesla l...@jcomserv.net changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG

--- Comment #15 from Jon Ciesla l...@jcomserv.net 2010-05-06 09:22:06 EDT ---
Closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579369] Review Request: kanjistrokeorders-fonts - Font to view stroke order diagrams for Kanji, Kana and etc...

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=579369

--- Comment #2 from Parag AN(पराग) panem...@gmail.com 2010-05-06 09:26:40 EDT 
---
sorry I am busy with some other work. Will review this tomorrow.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

--- Comment #7 from Stanislav Ochotnicky socho...@redhat.com 2010-05-06 
09:32:55 EDT ---
New SRPM URL:
http://sochotni.fedorapeople.org/apache-commons-logging-1.1.1-3.fc12.src.rpm

Spec URL: http://sochotni.fedorapeople.org/apache-commons-logging.spec


Thanks

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 541207] Review Request: liborigin2 - library for reading OriginLab Origin 7.5 project files

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=541207

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

  Alias|liborigin2  |

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588194] Review Request: quickly - Command line tools to ease the creation of a new project

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588194

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

 CC||sanjay.an...@gmail.com
 AssignedTo|nob...@fedoraproject.org|sanjay.an...@gmail.com
   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589585] New: Review Request: maven-javadoc-plugin - Maven Javadoc plugin

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: maven-javadoc-plugin - Maven Javadoc plugin

https://bugzilla.redhat.com/show_bug.cgi?id=589585

   Summary: Review Request: maven-javadoc-plugin - Maven Javadoc
plugin
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mefos...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.macs.hw.ac.uk/~mef3/maven221/maven-javadoc-plugin.spec
SRPM URL:
http://www.macs.hw.ac.uk/~mef3/maven221/maven-javadoc-plugin-2.4-1.fc12.src.rpm

Description:
The Maven Javadoc Plugin is a plugin that uses the javadoc tool for
generating javadocs for the specified project.

Notes about this package:
- It uses version 2.4, which is not the latest, so that it can be built with
the current maven2-2.0.8; it will be updated to the newest version once the
newer maven is built
- Javadoc generation for the plugin is disabled for the initial version because
that currently also won't work until the rest of maven is updated -- this is
the role of the bootstrap flag

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589585] Review Request: maven-javadoc-plugin - Maven Javadoc plugin

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589585

--- Comment #1 from Mary Ellen Foster mefos...@gmail.com 2010-05-06 09:41:38 
EDT ---
Additional note: this builds successfully in koji against the dist-f14-maven221
target:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2167910

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 581509] Review Request: yacas - easy to use, general purpose Computer Algebra System

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=581509

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

 CC||sanjay.an...@gmail.com
 AssignedTo|nob...@fedoraproject.org|sanjay.an...@gmail.com
   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588654] Review Request: plexus-component-api - Plexus Component API

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588654

--- Comment #5 from Andrew Overholt overh...@redhat.com 2010-05-06 09:51:34 
EDT ---
I still get incoherent-version-in-changelog.  Are we running the same rpmlint?

$ rpmlint -V
rpmlint version 0.95 Copyright (C) 1999-2007 Frederic Lepied, Mandriva

Other than the incorrect version in %changelog, things are fine.  Correct that
and this will be approved.  Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #8 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
09:51:12 EDT ---
Thanks

This package is APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589504] Review Request: apache-commons-compress - Java API for working with tar, zip and bzip2 files

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589504

--- Comment #6 from Sandro Mathys s...@sandro-mathys.ch 2010-05-06 09:51:08 
EDT ---
Spec URL: http://red.fedorapeople.org/SRPMS/apache-commons-compress.spec
SRPM URL:
http://red.fedorapeople.org/SRPMS/apache-commons-compress-1.0-3.fc12.src.rpm

I never used maven2 in a package before but by looking at another spec file I
was able to come up with something that seems to work. Not sure if it's normal
that pretty some checksums fail, tho.

Also, rpmlint now shows a new warning but I think that can be ignored:
apache-commons-compress.noarch: W: non-conffile-in-etc
/etc/maven/fragments/apache-commons-compress

Oh, that OSGi stuff is done automatically, isn't it? Or do I need to cp/install
something?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 511204] Review Request: wicd - A wireless and wired network manager

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=511204

--- Comment #17 from Gareth John gareth.l.j...@googlemail.com 2010-05-06 
10:12:46 EDT ---
(In reply to comment #11)
  
  I'm going to review this package when Rangeen gives us an updated spec. I
  really like to see this in Fedora ASAP, but first we need to fix
  selinux-policy-targeted.  
 
 Please take up the this package as I am not able to dedicate enough time
 towards packaging and Fedora in general ( Little busy with my job). You can 
 put
 me as a co maintainer. If you want I can also review the package at some point
 of time. By the way, if this package can wait for some time then I will take 
 up
 the task after some time, once I have sufficient time.

http://gljohn.fedorapeople.org/wicd/wicd.spec
http://gljohn.fedorapeople.org/wicd/wicd-1.7.0-4.fc12.src.rpm

I have updated to version 1.7.0 and modified the files as required. I have also
used glob and macros to remove excessive file lines. I have never package a
python package before and i have just picked up and modified the current one in
this review. Having looked at the python packaging guide I'm not sure if the
spec file used is acceptable.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589504] Review Request: apache-commons-compress - Java API for working with tar, zip and bzip2 files

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589504

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review?

--- Comment #7 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
10:12:03 EDT ---
Thanks,
I'm taking this one.

Maven2 part looks good.
Yes the OSGi stuff are just additional headers added automatically to the
MANIFEST.MF file when you built with maven.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #9 from Stanislav Ochotnicky socho...@redhat.com 2010-05-06 
10:23:31 EDT ---
Thanks. Requesting CVS:

New Package CVS Request
===
Package Name: apache-commons-logging
Short Description: Apache Commons Logging
Owners: sochotni mbooth
Branches: 
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588202] Review Request: perl-Class-ISA - Report the search path for a class's ISA tree

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588202

Huzaifa S. Sidhpurwala huzai...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588202] Review Request: perl-Class-ISA - Report the search path for a class's ISA tree

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588202

--- Comment #3 from Huzaifa S. Sidhpurwala huzai...@redhat.com 2010-05-06 
10:27:00 EDT ---
cvs done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 587315] Review Request: pmars - Portable corewar system with ICWS'94 extensions

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=587315

Jon Ciesla l...@jcomserv.net changed:

   What|Removed |Added

   Flag|needinfo?   |

--- Comment #6 from Jon Ciesla l...@jcomserv.net 2010-05-06 10:32:37 EDT ---
SRPM: http://zanoni.jcomserv.net/fedora/pmars/pmars-0.9.2-3.fc12.src.rpm
SPEC: http://zanoni.jcomserv.net/fedora/pmars/pmars.spec

Addressed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 581509] Review Request: yacas - easy to use, general purpose Computer Algebra System

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=581509

--- Comment #5 from Henrique LonelySpooky Junior henrique...@gmail.com 
2010-05-06 10:33:11 EDT ---
No one is interested in reviewing? =)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561455] Review Request: yecht - A YAML processor based on Syck

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561455

--- Comment #10 from Alexander Kurtakov akurt...@redhat.com 2010-05-06 
10:39:31 EDT ---
Review:

OK: rpmlint must be run on every package. OUTPUT:

yecht.noarch: W: no-documentation
yecht-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs - Java docs,
Java-docs, Javanese

Not a problem. I really haven't found anything suitable for documentation and
javadocs is a common word for developers.

OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. 
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK: The License field in the package spec file must match the actual license. 
OK: The spec file must be written in American English.
OK: The spec file for the package MUST be legible. 

NOTOK: The sources used to build the package must match the upstream source, as
provided in the spec URL. SOURCES are different.
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
OK: All build dependencies must be listed in BuildRequires, except for any that
are listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense.
OK: Packages must NOT bundle copies of system libraries.
OK: A package must own all directories that it creates. 
OK: A Fedora package must not list a file more than once in the spec file's
%files listings. 
OK: Permissions on files must be set properly. 
OK: Each package must consistently use macros.
OK: The package must contain code, or permissable content. 
OK: Large documentation files must go in a -doc subpackage. Javadocs
subpackage.
OK: If a package includes something as %doc, it must not affect the runtime of
the application. 
OK: Packages must not own files or directories already owned by other packages. 
OK: All filenames in rpm packages must be valid UTF-8. 


Please fix the sources

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589162] Review Request: stress - A tool to put given subsystems under a specified load

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589162

--- Comment #2 from Jon Ciesla l...@jcomserv.net 2010-05-06 10:43:43 EDT ---
SRPM: http://zanoni.jcomserv.net/fedora/stress/stress-1.0.4-2.fc12.src.rpm
SPEC: http://zanoni.jcomserv.net/fedora/stress/stress.spec

I've addressed the permissions and doc issues, but I'm not sure I agree on the
licensing issue.  The license file is GPL3, the code says GPLv2+.  I see no
reason I can't then use GPLv3+.  I agree that upstream should probably change
their website.  If you like, you can block FE-LEGAL and we can get a formal
decision on this, because I very well may be wrong.  I probably should have
commented in the spec my reasoning on the license tag, and I can certainly do
that if you like.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 581509] Review Request: yacas - easy to use, general purpose Computer Algebra System

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=581509

--- Comment #6 from Ankur Sinha sanjay.an...@gmail.com 2010-05-06 10:52:35 
EDT ---
on my TODO list

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 586777] Review Request: glyphtracer - Program for creating fonts from images

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=586777

--- Comment #7 from Parag pnem...@redhat.com 2010-05-06 10:49:15 EDT ---
Spec URL: http://paragn.fedorapeople.org/fedora-work/SPECS/glyphtracer.spec
SRPM URL:
http://paragn.fedorapeople.org/fedora-work/SRPMS/glyphtracer-1.0-1.r73bzr.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589617] Review Request: apache-commons-discovery - rename of jakarta-commons-discovery

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589617

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

 Blocks||588764

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589617] New: Review Request: apache-commons-discovery - rename of jakarta-commons-discovery

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: apache-commons-discovery - rename of 
jakarta-commons-discovery

https://bugzilla.redhat.com/show_bug.cgi?id=589617

   Summary: Review Request: apache-commons-discovery - rename of
jakarta-commons-discovery
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: socho...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://sochotni.fedorapeople.org/apache-commons-discovery.spec
SRPM URL:
http://sochotni.fedorapeople.org/apache-commons-discovery-0.4-1.fc12.src.rpm

Description: Package was renamed from jakarta-commons-discovery. Discovery
provides facilities for finding and instantiating classes, and for lifecycle
management of singleton (factory) classes.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

 Blocks||589617

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589617] Review Request: apache-commons-discovery - rename of jakarta-commons-discovery

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589617

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

 Depends on||589168

--- Comment #1 from Stanislav Ochotnicky socho...@redhat.com 2010-05-06 
11:07:09 EDT ---
Adding bug #589168 as blocker because this package contains Requires on
apache-commons-logging.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 505360] Review Request: JSCookMenu - Javascript GUI-like web menus

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=505360

--- Comment #17 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:10:44 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 530374] Review Request: nitrogen - A background browser and setter for X windows

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=530374

--- Comment #15 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:21:47 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 538303] Review Request: rubygem-minitest - Small and fast replacement for ruby's huge and slow test/unit

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=538303

Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #18 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:23:28 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 542292] Review Request: mintmenu - gnome-panel menu from LinuxMint

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=542292

--- Comment #20 from Valent Turkovic valent.turko...@gmail.com 2010-05-06 
11:26:53 EDT ---
Bummer :(
I was really hoping this package makes it into fedora repos.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 485652] Review Request: navit - Car navigation system with routing engine

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485652

--- Comment #63 from Valent Turkovic valent.turko...@gmail.com 2010-05-06 
11:23:48 EDT ---
Isn't there a reply in upstream ticket?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 574977] Review Request: rubygem-amqp - AMQP client implementation in Ruby/EventMachine

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=574977

--- Comment #9 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:25:18 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561451] Review Request: nailgun - Framework for running Java from the cli without the JVM startup overhead

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561451

--- Comment #8 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:23:56 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 527549] Review Request: osm2go - A simple openstreetmap editor

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=527549

--- Comment #14 from Valent Turkovic valent.turko...@gmail.com 2010-05-06 
11:25:52 EDT ---
Cool, seams this packages will soon be in Fedora repos, jay!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 575496] Review Request: perl-CGI-Compile - Compile .cgi scripts to a code reference like ModPerl::Registry

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=575496

--- Comment #4 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:27:29 EDT ---
CVS done (by process-cvs-requests.py).

(and could you answer my email query about the zim package? :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 581509] Review Request: yacas - easy to use, general purpose Computer Algebra System

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=581509

--- Comment #7 from Henrique LonelySpooky Junior henrique...@gmail.com 
2010-05-06 11:37:33 EDT ---
(In reply to comment #6)
 on my TODO list

Thanks, Ankur.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579227] Review Request: byobu - Light-weight, configurable window manager built upon GNU screen

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=579227

Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #13 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:39:16 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 584728] Review Request: rktime - Multi-zone time display utility

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=584728

--- Comment #7 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:44:50 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583141] Review Request: nautilus-pastebin - Nautilus extension to send files to a pastebin

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=583141

--- Comment #16 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:44:29 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585902] Review Request: cardpeek - Tool to read the contents of smart cards

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585902

--- Comment #5 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:47:47 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 582974] Review Request: dnsenum - DNSenum is a penetration testing tool created to enumerate DNS info about domains

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=582974

--- Comment #18 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:43:38 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #14 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:46:52 EDT ---
CVS done (by process-cvs-requests.py).

(We are not doing F14 branches yet, so left that out)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585467] Review Request: mozilla-googlesharing - Anonymizing proxy service for google sharing system

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585467

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag||needinfo?(toms...@fedorapro
   ||ject.org)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 589167] Review Request: perl-ParseTemplate - ParseTemplate Perl module

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589167

--- Comment #4 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:49:16 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588482] Review Request: libktorrent - Library providing torrent downloading code

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588482

--- Comment #4 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:48:33 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579682] Review Request: perl-version - Perl extension for Version Objects

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=579682

Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #8 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:50:00 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589168] Review Request: apache-commons-logging - rename of jakarta-commons-logging

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589168

--- Comment #10 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:49:39 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 587315] Review Request: pmars - Portable corewar system with ICWS'94 extensions

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=587315

Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Jussi Lehtola jussi.leht...@iki.fi 2010-05-06 11:51:32 
EDT ---
rpmlint output:

$ rpmlint pmars-*
pmars.src: W: spelling-error Summary(en_US) corewar - core war, core-war,
forewarn
pmars.src: W: spelling-error %description -l en_US corewar - core war,
core-war, forewarn
pmars.src: W: spelling-error %description -l en_US redcode - red code,
red-code, redcoat
pmars.src: W: spelling-error %description -l en_US multi - mulch, mufti
pmars.x86_64: W: spelling-error Summary(en_US) corewar - core war, core-war,
forewarn
pmars.x86_64: W: spelling-error %description -l en_US corewar - core war,
core-war, forewarn
pmars.x86_64: W: spelling-error %description -l en_US redcode - red code,
red-code, redcoat
pmars.x86_64: W: spelling-error %description -l en_US multi - mulch, mufti
3 packages and 0 specfiles checked; 0 errors, 8 warnings.

Please fix the spelling error.

Also, please place the temporary doc dir stuff in %prep, after the application
of the patches.


MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. OK
MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the 
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual license.
OK
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. N/A
MUST: Optflags are used and time stamps preserved. OK
MUST: Packages containing shared library files must call ldconfig. N/A
MUST: A package must own all directories that it creates or require the package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. OK
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. N/A
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. OK
MUST: Header files must be in a -devel package. N/A
MUST: Static libraries must be in a -static package. N/A
MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. N/A
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. N/A
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. N/A
MUST: Packages does not contain any .la libtool archives. N/A

MUST: Desktop files are installed properly. ~OK
- Although this is an X application, it is operated purely from the command
prompt (it needs a warrior file as input).

MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The package builds in mock. OK


APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 527549] Review Request: osm2go - A simple openstreetmap editor

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=527549

--- Comment #15 from Martin Gieseking martin.giesek...@uos.de 2010-05-06 
11:56:51 EDT ---
Fabian, did you already get any information about the license from upstream? If
so, please let me know.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #15 from Shakthi Kannan shakthim...@gmail.com 2010-05-06 12:01:41 
EDT ---
What about EL-5?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 589585] Review Request: maven-javadoc-plugin - Maven Javadoc plugin

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=589585

Jeff Johnston jjohn...@redhat.com changed:

   What|Removed |Added

 CC||jjohn...@redhat.com
 AssignedTo|nob...@fedoraproject.org|jjohn...@redhat.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585467] Review Request: mozilla-googlesharing - Anonymizing proxy service for google sharing system

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585467

Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

   Flag|needinfo?(toms...@fedorapro |
   |ject.org)   |

--- Comment #17 from Thomas Spura toms...@fedoraproject.org 2010-05-06 
12:11:14 EDT ---
(In reply to comment #16)
 I tried above patch on my F-13 machine and see its showing in installed
 extensions but also saw showing Not compatible with SeaMonkey 2.0.4. So If
 its not going to work in SeaMonkey then better not to include it in SPEC.

Oh, right...
(As noted above not tested, just it's showing up^^)

Maybe it's worth to suround a conditional around the seamonkey part, so it can
easily enabled/disabled. Don't know if it will ever be working with seamonkey
(I don't use seamonkey anyway, just wanted to note, that some user want to have
that part...)

--- mozilla-googlesharing.spec 2010-05-06 18:09:50.058954346 +0200
+++ orig 2010-05-04 13:17:56.714392098 +0200
@@ -4,11 +4,9 @@
 %global src_ext_id googleshar...@extension.thoughtcrime.org
 %global inst_dir %{moz_extensions}/%{firefox_app_id}/%{src_ext_id}

-%global seamonkey_app_dir \{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}
+%global seamonkey_app_dir \{googlesharing}
 %global sym_link_dir %{moz_extensions}/%{seamonkey_app_dir} 

-%global with_seamonkey 0
-
 Name:   mozilla-googlesharing
 Version:0.18
 Release:5%{?dist}
@@ -60,10 +58,8 @@
 popd

 # symlink from seamonkey extension to firefox extension
-%if 0%{?with_seamonkey}
 mkdir -p %{buildroot}%{sym_link_dir}
 ln -s %{inst_dir} %{buildroot}%{sym_link_dir}
-%endif

 %clean
 %{__rm} -rf $RPM_BUILD_ROOT
@@ -72,9 +68,7 @@
 %defattr(-,root,root,-)
 %doc COPYING
 %{inst_dir}
-%if 0%{?with_seamonkey}
 %{sym_link_dir}
-%endif


 %changelog

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 582352] Review Request: sentinella - Monitors your system activity and takes chosen action

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=582352

--- Comment #15 from Carlos Olmedo Escobar carlos.olmed...@gmail.com 
2010-05-06 12:17:48 EDT ---

Hi, version 0.6.1 is just released as promised. Here is the changelog so you
don't miss what have changed:

 * Replaced audio files released under CC sampling plus by three self-made
files released under CC-BY-SA 3.0
 * Added six new translations.
 * The final binary name will be sentinella (with lower s).
 * Added .desktop file.
 * Now the condition runs in a high-prioritized thread.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 584090] Review Request: mashpodde - mashpodder is a podcatching client based on BashPodder

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=584090

--- Comment #6 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-05-06 
12:37:43 EDT ---
Mohammed, when you do pre-review using some template,
please also write the summary of your (pre-)review so that
everyone (including the submitter) can easily find what
needs fixing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561452] Review Request: yydebug - Supports tracing and animation for a Java-based parser generated by jay

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561452

Mohammed Morsi mmo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from Mohammed Morsi mmo...@redhat.com 2010-05-06 12:41:18 EDT 
---
Ah yes, my mistake. New version of the spec / srpm, with my name added to the
changelog (the only change):

New Spec/SRPM w/ javadocs
Spec URL: http://mo.morsi.org/files/jruby/yydebug.spec
SRPM URL: http://mo.morsi.org/files/jruby/yydebug-1.1.0-5.fc11.src.rpm

New Package CVS Request
===
Package Name: yydebug
Short Description: Supports tracing and animation for a Java-based parser
generated by jay.
Owners: mmorsi
Branches: F-12 F-13
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588122

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2010-05-06 
12:45:10 EDT ---
perl-Symbol-Util-0.0202-3.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/perl-Symbol-Util-0.0202-3.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588122

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2010-05-06 
12:45:05 EDT ---
perl-Symbol-Util-0.0202-3.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/perl-Symbol-Util-0.0202-3.fc12

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588122

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2010-05-06 
12:45:14 EDT ---
perl-Symbol-Util-0.0202-3.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/perl-Symbol-Util-0.0202-3.fc11

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588474] Review Request: rubygem-rubyzip - zipfile support in Ruby

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588474

--- Comment #5 from Adam Young ayo...@redhat.com 2010-05-06 12:52:35 EDT ---
Mathew,
THanks.  I've read those, and I'm, going through and scrubbing now.  Got it on
review from a nother package.  All of these are required to get buildr, which
is the real goal.  

Mohammad, so

Source0: http://downloads.sourceforge.net/%{name}/%{name}-%{version}.gem should
be OK.  Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588474] Review Request: rubygem-rubyzip - zipfile support in Ruby

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=588474

--- Comment #6 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-05-06 
13:03:04 EDT ---
Usually we use
http://gems.rubyforge.org/gems/%{gemname}-%{version}.gem
for sourceURL. Also the latest seems 0.9.4.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 587315] Review Request: pmars - Portable corewar system with ICWS'94 extensions

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=587315

Jon Ciesla l...@jcomserv.net changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Jon Ciesla l...@jcomserv.net 2010-05-06 12:59:23 EDT ---
Thanks for the review!

I moved the doc manipulation to after the patches, but I left the prep header
before setup, because otherwise it tries to do it's thing before the tarball is
unpacked, which fails.

New Package CVS Request
===
Package Name: pmars
Short Description: Portable corewar system with ICWS'94 extensions
Owners: limb
Branches: F-13 F-12
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585902] Review Request: cardpeek - Tool to read the contents of smart cards

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585902

Kalev Lember ka...@smartlink.ee changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

--- Comment #6 from Kalev Lember ka...@smartlink.ee 2010-05-06 13:00:29 EDT 
---
Package imported and built.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561455] Review Request: yecht - A YAML processor based on Syck

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561455

--- Comment #11 from Mohammed Morsi mmo...@redhat.com 2010-05-06 13:06:10 EDT 
---
Sync'd srpm source w/ upstream (md5sums now match). New spec/srpm:

Spec URL: http://mo.morsi.org/files/jruby/yecht.spec
SRPM URL: http://mo.morsi.org/files/jruby/yecht-0.0.2-4.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 582352] Review Request: sentinella - Monitors your system activity and takes chosen action

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=582352

--- Comment #16 from Thomas Janssen thom...@fedoraproject.org 2010-05-06 
13:35:48 EDT ---
Spec URL: http://thomasj.fedorapeople.org/reviews/sentinella.spec
SRPM URL:
http://thomasj.fedorapeople.org/reviews/sentinella-0.6.1-1.fc12.src.rpm

Upstreamed patch to fix the .desktop file:
http://sourceforge.net/tracker/?func=detailatid=1179456aid=2997765group_id=28

rpmlint sentinella.spec ../SRPMS/sentinella-0.6.1-1.fc12.src.rpm
../RPMS/x86_64/sentinella-*0.6.1*
sentinella-debuginfo.x86_64: E: world-writable
/usr/src/debug/Sentinella-0.6.1/x86_64-redhat-linux-gnu/src/ui_MainWindow.h
0666
3 packages and 1 specfiles checked; 1 errors, 0 warnings.

The world-writable hurts me. The ui_MainWindow.h gets created trough compile
time. I have no clue how to fix/patch this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585151] Review Request: R-sciplot - Scientific Graphing Functions for Factorial Designs

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585151

--- Comment #10 from Pierre-YvesChibon pin...@pingoured.fr 2010-05-06 
13:52:50 EDT ---
arf I am seen this too late, you are missing this:
BuildRequires:R-devel, tex(latex)
see http://fedoraproject.org/wiki/Packaging/R

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 587438] Review Request: rubygem-snmp - A Ruby implementation of SNMP (the Simple Network Management Protocol)

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=587438

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 CC||mtas...@ioa.s.u-tokyo.ac.jp

--- Comment #1 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-05-06 
13:51:27 EDT ---
Some notes:

* Unused macro
  - %ruby_sitelib macro doesn't seem to be used anywhere

* License
  - When the license is the same as Ruby, it means
GPLv2 or Ruby for license tag.

* %doc
  - Please mark document files as %doc properly
* README file should be marked as %doc
* examples/ test/ directory should also be marked
  as %doc

* Enabling test
  - As this gem contains test/ directory, please add %check
section and execute some tests (like $ rake test) there.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


  1   2   >