[Bug 623425] Review Request: python-pyside - Python bindings for Qt4

2010-09-12 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=623425

--- Comment #25 from Chen Lei supercyp...@gmail.com 2010-09-12 02:11:00 EDT 
---
Kalev,

Do you have a idea for the naming scheme of qt-mobility and libmeegotouch
python bindings? They will be included in F15 because of meego 1.2 spin, so we
need a consistent name for those three python bindings for the same upstream.

-- 
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 539983] Review Request: qjson - qt-based library that maps JSON data to QVariant object

2010-09-12 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=539983

--- Comment #29 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 02:29:56 EDT ---
qjson-0.7.1-2.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/qjson-0.7.1-2.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 539983] Review Request: qjson - qt-based library that maps JSON data to QVariant object

2010-09-12 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=539983

--- Comment #30 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 02:30:09 EDT ---
qjson-0.7.1-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/qjson-0.7.1-2.fc14

-- 
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 539983] Review Request: qjson - qt-based library that maps JSON data to QVariant object

2010-09-12 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=539983

--- Comment #31 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 02:30:18 EDT ---
qjson-0.7.1-2.fc12 has been submitted as an update for Fedora 12.
https://admin.fedoraproject.org/updates/qjson-0.7.1-2.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 632919] Review Request: rubygem-anemone - Anemone web-spider framework

2010-09-12 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=632919

Bug 632919 depends on bug 632917, which changed state.

Bug 632917 Summary: Review Request: rubygem-shoulda - Making ruby tests easy on 
the fingers and eyes
https://bugzilla.redhat.com/show_bug.cgi?id=632917

   What|Old Value   |New Value

 Resolution||DUPLICATE
 Status|NEW |CLOSED

-- 
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 632912] Review Request: rubygem-robots - Simple robots.txt parser

2010-09-12 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=632912

Bug 632912 depends on bug 632917, which changed state.

Bug 632917 Summary: Review Request: rubygem-shoulda - Making ruby tests easy on 
the fingers and eyes
https://bugzilla.redhat.com/show_bug.cgi?id=632917

   What|Old Value   |New Value

 Resolution||DUPLICATE
 Status|NEW |CLOSED

-- 
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 553908] Review Request: rubygem-shoulda - Making tests easy on the fingers and eyes

2010-09-12 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=553908

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

   What|Removed |Added

 CC||re...@seznam.cz

--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-09-12 
06:34:09 EDT ---
*** Bug 632917 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 587978] Review Request: whatweb - Web scanner to identify what websites are running

2010-09-12 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=587978

Bug 587978 depends on bug 632917, which changed state.

Bug 632917 Summary: Review Request: rubygem-shoulda - Making ruby tests easy on 
the fingers and eyes
https://bugzilla.redhat.com/show_bug.cgi?id=632917

   What|Old Value   |New Value

 Resolution||DUPLICATE
 Status|NEW |CLOSED

-- 
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 632917] Review Request: rubygem-shoulda - Making ruby tests easy on the fingers and eyes

2010-09-12 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=632917

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

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||DUPLICATE
Last Closed||2010-09-12 06:34:09

--- Comment #1 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-09-12 
06:34:09 EDT ---
rubygem-shoulda is already in Fedora.

https://admin.fedoraproject.org/pkgdb/acls/name/rubygem-shoulda
http://koji.fedoraproject.org/koji/packageinfo?packageID=9810

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

-- 
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 632986] New: Review Request: python-zope-contenttype - Content-Type Handling Utility Module

2010-09-12 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: python-zope-contenttype - Content-Type Handling 
Utility Module

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

   Summary: Review Request: python-zope-contenttype - Content-Type
Handling Utility Module
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: robinlee.s...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://cheeselee.fedorapeople.org/python-zope-contenttype.spec
SRPM URL:
http://cheeselee.fedorapeople.org/python-zope-contenttype-3.5.1-2.fc13.src.rpm
Description:
This package provides a utility module for content-type handling.

rpmlint results:
$ rpmlint ./python-zope-contenttype.spec 
./python-zope-contenttype.spec: W: no-cleaning-of-buildroot %install
./python-zope-contenttype.spec: W: no-cleaning-of-buildroot %clean
./python-zope-contenttype.spec: W: no-buildroot-tag
./python-zope-contenttype.spec: W: no-%clean-section
0 packages and 1 specfiles checked; 0 errors, 4 warnings.

$ rpmlint ./python-zope-contenttype-3.5.1-2.fc13.src.rpm 
python-zope-contenttype.src: W: no-cleaning-of-buildroot %install
python-zope-contenttype.src: W: no-cleaning-of-buildroot %clean
python-zope-contenttype.src: W: no-buildroot-tag
python-zope-contenttype.src: W: no-%clean-section
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

$ rpmlint ./python-zope-contenttype-3.5.1-2.fc13.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
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 632986] Review Request: python-zope-contenttype - Content-Type Handling Utility Module

2010-09-12 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=632986

--- Comment #1 from Robin Lee robinlee.s...@gmail.com 2010-09-12 06:58:40 EDT 
---
Created attachment 446738
  -- https://bugzilla.redhat.com/attachment.cgi?id=446738
The litte patch to register the test suite

-- 
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 609638] Review Request: kpassgen - Random password creater

2010-09-12 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=609638

Siddharth Sharma siddharth@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?, needinfo-,  |
   |needinfo-, 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 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop

2010-09-12 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=632970

--- Comment #2 from Cédric OLIVIER cedric.oliv...@free.fr 2010-09-12 07:23:57 
EDT ---
Thanks a lot for yours comments.

You can find now package updated here :

SPEC URL :
http://cedric.olivier.free.fr/rpms/rec-applet-0.2.20100821/rec-applet.spec
SRPMS URL :
http://cedric.olivier.free.fr/rpms/rec-applet-0.2.20100821/rec-applet-0-0.2.20100821bzr.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 571993] Review Request: non-sequencer - A powerful, real-time, pattern-based MIDI sequencer

2010-09-12 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=571993

Adam Huffman bl...@verdurin.com changed:

   What|Removed |Added

   Flag|needinfo?(bl...@verdurin.co |
   |m)  |

--- Comment #9 from Adam Huffman bl...@verdurin.com 2010-09-12 07:40:03 EDT 
---
I am interested in packaging it.  However, the upstream activity seems rather
erratic - no commits since January and there are bugs that have remained
unfixed for quite a while.

If you think it's worthwhile proceeding with all that in mind, I'll do so.

-- 
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 478613] Review Request: ledger - A powerful command-line double-entry accounting system

2010-09-12 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=478613

Michel Alexandre Salim fed...@michelsylvain.info changed:

   What|Removed |Added

   Flag|fedora-review?, |fedora-review+
   |needinfo?(fed...@michelsylv |
   |ain.info)   |

--- Comment #22 from Michel Alexandre Salim fed...@michelsylvain.info 
2010-09-12 08:20:38 EDT ---
Apologies for the delay. The last update fixes all outstanding issue -- 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 603481] Review Request: freerdp - remote desktop protocol client

2010-09-12 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=603481

Stefan Becker chemob...@gmail.com changed:

   What|Removed |Added

 CC||chemob...@gmail.com

--- Comment #17 from Stefan Becker chemob...@gmail.com 2010-09-12 09:54:09 
EDT ---
Good to see that this already in progress. freerdp worked out-of-the-box with
Win7, whereas rdesktop-1.6.0 does not (only the latest SVN code does).

FYI: I have filed an enhancement request to KDEs krdc to support freerdp:

  https://bugs.kde.org/show_bug.cgi?id=250979

-- 
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 593559] Review Request: protobuf-c - C bindings for Google's Protocol Buffers

2010-09-12 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=593559

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

   Flag|needinfo?   |

--- Comment #10 from Martin Gieseking martin.giesek...@uos.de 2010-09-12 
10:01:58 EDT ---
(In reply to comment #9)
 Martin, if you are going to review this package formally,
 would you assign this bug to yourself?

Yes, sure. My previous comment was just intended as a general note. But if
David is still interested in joining the packager group and has no sponsor yet,
I'm willing to take this 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 620385] Review Request: BEDTools - A flexible suite of utilities for comparing genomic features

2010-09-12 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=620385

--- Comment #12 from Martin Gieseking martin.giesek...@uos.de 2010-09-12 
10:12:31 EDT ---
Adam, what's the status of this 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 620385] Review Request: BEDTools - A flexible suite of utilities for comparing genomic features

2010-09-12 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=620385

--- Comment #13 from Adam Huffman bl...@verdurin.com 2010-09-12 11:07:00 EDT 
---
Martin, I was waiting for a reply from upstream about the licensing of the PDF.
 In case you're happy to proceed without that, I've made a new version with
GPLv2 added to -docs.

http://verdurin.fedorapeople.org/reviews/BEDTools/BEDTools.spec

http://verdurin.fedorapeople.org/reviews/BEDTools/BEDTools-2.9.0-5.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 573848] Review Request: perl-NetPacket - Modules to assemble/disassemble network packets at the protocol level

2010-09-12 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=573848

Jose Pedro Oliveira j...@di.uminho.pt changed:

   What|Removed |Added

 CC||j...@di.uminho.pt

--- Comment #5 from Jose Pedro Oliveira j...@di.uminho.pt 2010-09-12 11:04:53 
EDT ---
Jan,

Could you update NetPacket to the latest upstream version? 

Regards,
jpo

-- 
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 453422] Review Request: songbird - Mozilla based multimedia player

2010-09-12 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=453422

--- Comment #153 from Hicham HAOUARI hicham.haou...@gmail.com 2010-09-12 
11:11:30 EDT ---
(In reply to comment #152)
 1.  This has already been compiled as an RPM for Fedora (see above contributed
 builds).  This bug was closed since Songbird announced they were no longer
 going to do Linux.  They have since reversed course.  The Generic Linux on
 that page is cited as being contributed by Songbird.  The project has in fact
 not ceased for Linux, and they are still providing the source.
 

Everything can be packaged in rpm format, that is not an issue. What matters is
if the package follows fedora packaging guidelines or not. This one bundles its
own xulrunner, which is really ugly and creates security problems ( unless the
packager backports the patches to the bundled xulrunner ).

 2.  Who cares?  What does it matter if it uses GTK, Qt, or Tk for the GUI? 
 That has never been a package requirement.

It is not a package requirement, but I think ( IMHO ) that applications which
respect the DE theme fit better.

 
 If David Halik wishes to package this for Fedora, I suggest we re-open this
 Review.

If he can fix the issues, it is up to him.

-- 
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 478613] Review Request: ledger - A powerful command-line double-entry accounting system

2010-09-12 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=478613

Jim Radford radf...@blackbean.org changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #23 from Jim Radford radf...@blackbean.org 2010-09-12 11:36:34 
EDT ---
New Package SCM Request
===
Package Name: ledger
Short Description: A powerful command-line double-entry accounting system
Owners: radford
Branches: f12 f13 f14 el5 el6
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 573848] Review Request: perl-NetPacket - Modules to assemble/disassemble network packets at the protocol level

2010-09-12 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=573848

--- Comment #6 from Jose Pedro Oliveira j...@di.uminho.pt 2010-09-12 11:43:57 
EDT ---
Created attachment 446770
  -- https://bugzilla.redhat.com/attachment.cgi?id=446770
Patch for EPEL5

Downgrades the Module::Build version requirement so that it the module can be
packaged for EPEL5.


Error messages without the patch:
--
...
+ /usr/bin/perl Build.PL installdirs=vendor
Module::Build version 0.360100 required--this is only version 0.280700 at
Build.PL line 5
BEGIN failed--compilation aborted at Build.PL line 5, DATA line 537.
error: Bad exit status from /var/tmp/rpm-tmp.54415 (%build)
...
--

-- 
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 620385] Review Request: BEDTools - A flexible suite of utilities for comparing genomic features

2010-09-12 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=620385

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #14 from Martin Gieseking martin.giesek...@uos.de 2010-09-12 
11:48:35 EDT ---
Ah OK, thanks for the update. Sorry, I didn't intend to urge you. 

To me, the current status of the package is fine since the License field
reflects the latest feedback from upstream. If you get additional information,
you can update the spec file later on.


Package 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 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-12 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=631558

--- Comment #4 from Peter Oliver fedoraproject@mavit.org.uk 2010-09-12 
11:50:20 EDT ---
Sorry, I forgot to re-run mock after the previous changes.  The BuildRequires
should all be present, now.  I've also removed the file-based Requires.

Updated SRPM at
http://www.triv.org.uk/~mavit/rpm/fedora-13/SRPMS/arduino-0019-5.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 633058] New: Review Request: mingw32-gdk-pixbuf - MinGW Windows GDK Pixbuf library

2010-09-12 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: mingw32-gdk-pixbuf - MinGW Windows GDK Pixbuf library

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

   Summary: Review Request: mingw32-gdk-pixbuf - MinGW Windows GDK
Pixbuf library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: erik-fed...@vanpienbroek.nl
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ftd4linux.nl/contrib/mingw32-gdk-pixbuf.spec
SRPM URL: http://ftd4linux.nl/contrib/mingw32-gdk-pixbuf-2.21.7-1.fc14.src.rpm
Description: MinGW Windows GDK Pixbuf library

This package is split off from the mingw32-gtk2 package as upstream has decided
to separate the gdk-pixbuf library from the gtk2 package. This package is
needed to build mingw32-gtk2-2.21.7 (or higher)

-- 
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 633063] New: Review Request: python-txamqp - A Python library for communicating with AMQP peers and brokers using Twisted

2010-09-12 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: python-txamqp - A Python library for communicating 
with AMQP peers and brokers using Twisted

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

   Summary: Review Request: python-txamqp - A Python library for
communicating with AMQP peers and brokers using
Twisted
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: si...@sewell.ch
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec:
http://github.com/silas/rpms/raw/master/python-txamqp/python-txamqp.spec

SRPM:
http://github.com/downloads/silas/rpms/python-txamqp-0.3-1.fc15.src.rpm

Description:
txAMQP is a Python library for communicating with AMQP peers and brokers using
Twisted.

This project contains all the necessary code to connect, send and receive
messages to/from an AMQP-compliant peer or broker (Qpid, OpenAMQ, RabbitMQ)
using Twisted.

rpmlint

python-txamqp.noarch: I: enchant-dictionary-not-found en_US
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
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 571993] Review Request: non-sequencer - A powerful, real-time, pattern-based MIDI sequencer

2010-09-12 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=571993

--- Comment #10 from Adam Huffman bl...@verdurin.com 2010-09-12 13:44:05 EDT 
---
Put a new version addressing the compilation problems at:

http://verdurin.fedorapeople.org/reviews/non-sequencer/non-sequencer.spec

http://verdurin.fedorapeople.org/reviews/non-sequencer/non-sequencer-1.9.3-4.20100131gitba94d2c354145.fc13.src.rpm

I tried your suggestion regarding CXXFLAGS and that didn't seem to work as then
the various include paths were not picked up.

The icon is a resized version of the logo file included in the documentation,
for want of something better.

-- 
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 613993] Review Request: mingw32-celt051 - An audio codec for use in low-delay speech and audio communication

2010-09-12 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=613993

Erik van Pienbroek erik-fed...@vanpienbroek.nl changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #6 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2010-09-12 
13:44:28 EDT ---
$ rpmlint mingw32-celt051.spec 
mingw32-celt051.spec: W: no-cleaning-of-buildroot %clean
mingw32-celt051.spec: W: no-buildroot-tag
mingw32-celt051.spec: W: no-%clean-section
0 packages and 1 specfiles checked; 0 errors, 3 warnings.

$ rpmlint mingw32-celt051-0.5.1.3-3.fc14.src.rpm 
mingw32-celt051.src: W: spelling-error Summary(en_US) codec - cosec, codex,
code
mingw32-celt051.src: W: spelling-error %description -l en_US codec - cosec,
codex, code
mingw32-celt051.src: W: spelling-error %description -l en_US realtime -
mealtime, real time, real-time
mingw32-celt051.src: W: spelling-error %description -l en_US codecs - codes,
coders, code's
mingw32-celt051.src: W: no-cleaning-of-buildroot %clean
mingw32-celt051.src: W: no-buildroot-tag
mingw32-celt051.src: W: no-%clean-section
1 packages and 0 specfiles checked; 0 errors, 7 warnings.

$ rpmlint mingw32-celt051-0.5.1.3-3.fc14.noarch.rpm 
mingw32-celt051.noarch: W: spelling-error Summary(en_US) codec - cosec, codex,
code
mingw32-celt051.noarch: W: spelling-error %description -l en_US codec - cosec,
codex, code
mingw32-celt051.noarch: W: spelling-error %description -l en_US realtime -
mealtime, real time, real-time
mingw32-celt051.noarch: W: spelling-error %description -l en_US codecs -
codes, coders, code's
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

All these rpmlint warnings can be ignored

$ rpmquery --requires mingw32-celt051
pkgconfig  
mingw32-libogg  
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(CompressedFileNames) = 3.0.4-1
mingw32-filesystem = 57
mingw32-runtime  
mingw32(kernel32.dll)  
mingw32(libcelt051-0.dll)  
mingw32(libogg-0.dll)  
mingw32(msvcrt.dll)  
mingw32(user32.dll)  
rpmlib(PayloadIsXz) = 5.2-1

$ rpmquery --provides mingw32-celt051
mingw32(libcelt051-0.dll)  
mingw32-celt051 = 0.5.1.3-3.fc14

$ rpmquery --fileprovide mingw32-celt051
/usr/i686-pc-mingw32/sys-root/mingw/bin/celtdec051.exe 
/usr/i686-pc-mingw32/sys-root/mingw/bin/celtenc051.exe 
/usr/i686-pc-mingw32/sys-root/mingw/bin/libcelt051-0.dll 
/usr/i686-pc-mingw32/sys-root/mingw/include/celt051 
/usr/i686-pc-mingw32/sys-root/mingw/include/celt051/celt.h 
/usr/i686-pc-mingw32/sys-root/mingw/include/celt051/celt_header.h 
/usr/i686-pc-mingw32/sys-root/mingw/include/celt051/celt_types.h 
/usr/i686-pc-mingw32/sys-root/mingw/lib/libcelt051.dll.a 
/usr/i686-pc-mingw32/sys-root/mingw/lib/libcelt051.la 
/usr/i686-pc-mingw32/sys-root/mingw/lib/pkgconfig/celt051.pc 
/usr/share/doc/mingw32-celt051-0.5.1.3 
/usr/share/doc/mingw32-celt051-0.5.1.3/COPYING 
/usr/share/doc/mingw32-celt051-0.5.1.3/README 
/usr/share/doc/mingw32-celt051-0.5.1.3/TODO 


$ curl http://downloads.us.xiph.org/releases/celt/celt-0.5.1.3.tar.gz | md5sum
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
100  430k  100  430k0 0   176k  0  0:00:02  0:00:02 --:--:--  308k
67e7b5e45db57a6f1f0a6962f5ecb190  -
$ md5sum celt-0.5.1.3.tar.gz 
67e7b5e45db57a6f1f0a6962f5ecb190  celt-0.5.1.3.tar.gz


Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2463137


+ OK
! Needs to be looked into
/ Not applicable
* Overridden by MinGW guidelines

[+] Files are installed in /usr/i686-pc-mingw32/sys-root/mingw
[+] BuildRequires: mingw32-filesystem = xx is in the .spec file
[+] Requires are OK
[+] BuildArch: noarch
[+] No man pages or info files
[+] default strip and objdump commands are overridden with mingw32 specific
ones


[+] rpmlint must be run on every package. The output should be posted in the
review
[+] MUST: The package must be named according to the Package Naming Guidelines
[+] MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption on Package Naming Guidelines
[+] MUST: The package must meet the Packaging Guidelines
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: 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.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source,
as 

[Bug 632912] Review Request: rubygem-robots - Simple robots.txt parser

2010-09-12 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=632912

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-09-12 
14:15:47 EDT ---
Some notes:

* %define - %global
  - Now we prefer to use %global instead of %define:
   
https://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define

* BuildRoot
  - BuildRoot tag is no longer used on Fedora and EPEL6:
https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

* ruby(abi) Requires
  - Writing R: ruby(abi) = 1.8 is a must
https://fedoraproject.org/wiki/Packaging/Ruby#Ruby_Packaging_Guidelines

* Requires
  - R: rubygem(shoulda) is not needed
- thoughtbot-shoulda (not shoulda) dependency appeas in installed gemspec,
  however this is for development dependency and is not needed on runtime.

* Duplicate %files entry

58  warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/robots-0.10.0/CHANGELOG
59  warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/robots-0.10.0/README

  - Please make it sure that all files are listed only once in %files
entry.
  ! Note
The %files entry

%files
/some_dir/

contains the directory /some_dir/ itself and all files/directories/etc
under /some_dir/ , while

%files
%dir /some_dir/

contains the directory /some_dir/ only.

* rpmlint issue

rubygem-robots.noarch: E: non-executable-script
/usr/lib/ruby/gems/1.8/gems/robots-0.10.0/test/test_robots.rb 0644L
/usr/bin/env

  - This script need not have shebang and the shebang should be removed.

* %check
  - As this gem contains test/ directory, please add %check section and execute
some test program there
! Note
  To make $ rake test succeed, some modification against Rakefile is
  needed.

-- 
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 632912] Review Request: rubygem-robots - Simple robots.txt parser

2010-09-12 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=632912

--- Comment #2 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-09-12 
14:17:43 EDT ---
One more comment
* Macros
  - Please use the defined %{geminstdir} macro consistently also
in %files

-- 
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 632919] Review Request: rubygem-anemone - Anemone web-spider framework

2010-09-12 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=632919

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-09-12 
14:23:17 EDT ---
For some general issues, please also check my commentes
on bug 632912.
! Note
  Usually when gem contains spec/ directory, $ rake spec should be
  executed in %check.

-- 
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 632912] Review Request: rubygem-robots - Simple robots.txt parser

2010-09-12 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=632912

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

   What|Removed |Added

 Blocks|587978  |

-- 
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 587978] Review Request: whatweb - Web scanner to identify what websites are running

2010-09-12 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=587978

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

   What|Removed |Added

 Depends on|632912  |

-- 
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 632912] Review Request: rubygem-robots - Simple robots.txt parser

2010-09-12 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=632912

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

   What|Removed |Added

 Blocks||632919

-- 
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 632917] Review Request: rubygem-shoulda - Making ruby tests easy on the fingers and eyes

2010-09-12 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=632917

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

   What|Removed |Added

 Blocks|632919  |

-- 
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 632919] Review Request: rubygem-anemone - Anemone web-spider framework

2010-09-12 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=632919

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

   What|Removed |Added

 Depends on|632917  |632912

-- 
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 589867] Review Request: logcheck - analyzes logfiles and sends email

2010-09-12 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=589867

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #25 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 14:24:49 EDT ---
logcheck-1.3.13-2.el5 has been pushed to the Fedora EPEL 5 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update logcheck'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/logcheck-1.3.13-2.el5

-- 
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 626605] Review Request: perl-MIME-Base32 - Encode data similar way like MIME::Base64 does

2010-09-12 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=626605

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #21 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 14:24:22 EDT ---
perl-MIME-Base32-1.02a-1.el5 has been pushed to the Fedora EPEL 5 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update perl-MIME-Base32'.  You can
provide feedback for this update here:
https://admin.fedoraproject.org/updates/perl-MIME-Base32-1.02a-1.el5

-- 
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 625242] Review Request: clustershell - Efficient cluster administration

2010-09-12 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=625242

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 625242] Review Request: clustershell - Efficient cluster administration

2010-09-12 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=625242

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 14:43:54 EDT ---
clustershell-1.3.2-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/clustershell-1.3.2-1.fc14

-- 
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 632874] Review Request: freedup - Links substantially identical, duplicate files to save file system space

2010-09-12 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=632874

Randall Randy Berry randyn3...@gmail.com changed:

   What|Removed |Added

 CC||randyn3...@gmail.com
 AssignedTo|nob...@fedoraproject.org|randyn3...@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 625242] Review Request: clustershell - Efficient cluster administration

2010-09-12 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=625242

--- Comment #16 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 14:54:13 EDT ---
clustershell-1.3.2-1.fc12 has been submitted as an update for Fedora 12.
https://admin.fedoraproject.org/updates/clustershell-1.3.2-1.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 625242] Review Request: clustershell - Efficient cluster administration

2010-09-12 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=625242

--- Comment #15 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 14:52:53 EDT ---
clustershell-1.3.2-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/clustershell-1.3.2-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 625242] Review Request: clustershell - Efficient cluster administration

2010-09-12 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=625242

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2010-09-12 14:56:24 EDT ---
clustershell-1.3.2-1.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/clustershell-1.3.2-1.el5

-- 
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 483846] Review Request: gsim85 - An 8085 microprocessor simulator

2010-09-12 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=483846

Chitlesh GOORAH chitl...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #10 from Chitlesh GOORAH chitl...@gmail.com 2010-09-12 14:57:40 
EDT ---
Fabian and I agreed that as from now, I'll maintain gsim85 for EL branches
while he will continue to maintain the Fedora branches.

Package CVS Change Request
===
Package Name: gsim85
Short Description: An 8085 microprocessor simulator
Owners: fab chitlesh
Branches: EL-5 EL-6

-- 
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 631190] Review Request: libnxt - An utility for flashing LEGO Mindstorms NXT firmware

2010-09-12 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=631190

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

 CC||martin.giesek...@uos.de

--- Comment #1 from Martin Gieseking martin.giesek...@uos.de 2010-09-12 
15:01:36 EDT ---
Hi Damian,

here are some initial comments on your package:

- Remove the article An and the final dot from the Summary. It's just a 
  convention to keep the summary concise. 

- add short comments to the spec file telling what the patches do

- drop Requires: libusb as rpm automatically adds it as a dependency

- You must ensure that the binaries are built with CCFLAGS='%{optflags}' (the
  cross-compiled flash binary is an exception). Unfortunately, scons doesn't 
  seem to accept variable definitions given as command-line arguments. Thus, 
  you have to patch them into SConstruct.

-- 
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 328411] Review Request: archimedes - 2D Quantum Monte Carlo simulator for semiconductor devices

2010-09-12 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=328411

Chitlesh GOORAH chitl...@gmail.com changed:

   What|Removed |Added

 CC||chitl...@gmail.com
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #6 from Chitlesh GOORAH chitl...@gmail.com 2010-09-12 15:04:53 
EDT ---
Package Change Request
===
Package Name: archimedes
Short Description: 2D Quantum Monte Carlo simulator for semiconductor devices.
Owners: rishi chitlesh
Branches: EL-5, EL-6
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 633081] New: Review Request: Stephen Smoogen - mediawiki114 package for EPEL

2010-09-12 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: Stephen Smoogen - mediawiki114 package for EPEL

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

   Summary: Review Request: Stephen Smoogen - mediawiki114 package
for EPEL
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: smo...@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://smooge.fedorapeople.org/mediawiki/mediawiki114.spec
SRPM URL:
http://smooge.fedorapeople.org/mediawiki/rhel-5-i386/mediawiki114-1.14.1-5.el5.src.rpm

Description: 
MediaWiki is the software used for Wikipedia and the other Wikimedia
Foundation websites. Compared to other wikis, it has an excellent
range of features and support for high-traffic websites using multiple
servers

This bundles the 1.14.1 version of mediawiki for replacement of
mediawiki-1.14.0 in RHEL-4/5 EPEL.

-- 
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 576757] Review Request: moovida-plugins-good - Media Center

2010-09-12 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=576757

--- Comment #4 from Hans de Goede hdego...@redhat.com 2010-09-12 15:16:58 EDT 
---
Hi,

Sorry for the long delay. I recently switched teams with in Red Hat and as such
have been overwhelmed with work a bit. Here is a full review of this package:

Full review:

Good:
- rpmlint checks return:
moovida-plugins-good.noarch: W: no-documentation
2 packages and 0 specfiles checked; 0 errors, 1 warnings.
These can all be ignored
- package meets naming guidelines
- package meets packaging guidelines
- license (GPLv3) OK, text not %doc, but also not in upstream tarbal, so no
  problem,  matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime

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 633085] New: Review Request: mediawiki116 - Mediawiki package for EPEL

2010-09-12 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: mediawiki116 - Mediawiki package for EPEL

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

   Summary: Review Request: mediawiki116 - Mediawiki package for
EPEL
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: smo...@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://smooge.fedorapeople.org/mediawiki/mediawiki116.spec
SRPM URL:
http://smooge.fedorapeople.org/mediawiki/rhel-5-i386/mediawiki116-1.16.0-5.el5.src.rpm

Description: 
MediaWiki is the software used for Wikipedia and the other Wikimedia
Foundation websites. Compared to other wikis, it has an excellent
range of features and support for high-traffic websites using multiple
servers

This bundles the 1.16.x version of mediawiki for replacement for
RHEL-4/5/6 usage. This version does not contain patches that current
Fedora mediawiki contains due to their conflict with upstreams patch
sets.

-- 
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 633082] New: Review Request: mediawiki115 - EPEL mediawiki package

2010-09-12 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: mediawiki115 - EPEL mediawiki package

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

   Summary: Review Request: mediawiki115 - EPEL mediawiki package
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: smo...@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://smooge.fedorapeople.org/mediawiki/mediawiki115.spec  
SRPM URL:  
http://smooge.fedorapeople.org/mediawiki/rhel-5-i386/mediawiki115-1.15.5-5.el5.src.rpm
 

Description:
MediaWiki is the software used for Wikipedia and the other wikimedia 
Foundation websites. Compared to other wikis, it has an excellent   
range of features and support for high-traffic websites using multiple 
servers 
This bundles the 1.15.x version of mediawiki for replacement for 
RHEL-4/5/6 usage. This version does not contain patches that current
Fedora mediawiki contains due to their conflict with upstreams patch
sets.

-- 
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 576758] Review Request: moovida-plugins-bad - Media Center

2010-09-12 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=576758

--- Comment #4 from Hans de Goede hdego...@redhat.com 2010-09-12 15:24:56 EDT 
---
Hi,

Sorry for the long delay. I recently switched teams with in Red Hat and as such
have been overwhelmed with work a bit. Here is a full review of this package:

Full review:

Good:
- rpmlint checks return:
moovida-plugins-bad.src: W: invalid-url Source0:
http://www.moovida.com/media/public/moovida-plugins-bad-1.0.9.tar.gz HTTP Error
403: Forbidden
moovida-plugins-bad.noarch: W: no-documentation
2 packages and 0 specfiles checked; 0 errors, 2 warnings.
Well the one of the 403 error is not so good, so below
- package meets naming guidelines
- package meets packaging guidelines
- license (GPLv3) OK, text not %doc, but also not in upstream tarbal, so no
  problem, matches source
- spec file legible, in am. english
- source matches upstream
  Could not check, as the upstream url is giving 403 errors? I double
  checked the url and it is the one from upstreams download page.
  So I wilol just assume this is a temporary glitch and believe the tarbal in
  the srpm is the correct one.
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime

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 576758] Review Request: moovida-plugins-bad - Media Center

2010-09-12 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=576758

Hans de Goede hdego...@redhat.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 576757] Review Request: moovida-plugins-good - Media Center

2010-09-12 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=576757

Hans de Goede hdego...@redhat.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 576758] Review Request: moovida-plugins-bad - Media Center

2010-09-12 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=576758

--- Comment #5 from Hans de Goede hdego...@redhat.com 2010-09-12 15:47:31 EDT 
---
Hi Graeme,

What is your FAS account name? Once I have that I'll add you to the packagers
group and sponsor you, then you can do the request for creating a git repo for
the moovida package, import, build, etc.

Regards.

Hans

-- 
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 630754] Review Request: mscgen - Message Sequence Chart Rendering tool

2010-09-12 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=630754

--- Comment #7 from Michael McTernan michael.mcternan.2...@cs.bris.ac.uk 
2010-09-12 16:28:31 EDT ---
There's a short delay here while I fix upstream.  Then I'll make a new package
(likely 0.19-1) with matching MD5s and retry.

-- 
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 530747] Review Request: iodine - Solution to tunnel IPv4 data through a DNS server

2010-09-12 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=530747

Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||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 530747] Review Request: iodine - Solution to tunnel IPv4 data through a DNS server

2010-09-12 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=530747

--- Comment #13 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 
2010-09-12 17:09:51 EDT ---
As Linus did not answer about license, I request branch:

New Package CVS Request
=== 
Package Name: iodine
Short Description: Solution to tunnel IPv4 data through a DNS server
Owners: hubbitus,lystor
Branches: F-12 F-13 EL-5 EL-6
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 614520] Review Request: gnustep-examples - The GNUstep examples

2010-09-12 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=614520

Michel Alexandre Salim fed...@michelsylvain.info changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||fed...@michelsylvain.info
   Flag||fedora-review?,
   ||needinfo?(joc...@herr-schmi
   ||tt.de)

--- Comment #1 from Michel Alexandre Salim fed...@michelsylvain.info 
2010-09-12 17:30:59 EDT ---
Doing the review now, however, currently it does not compile on F-13:

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

-- 
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 633081] Review Request: Stephen Smoogen - mediawiki114 package for EPEL

2010-09-12 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=633081

--- Comment #2 from Stephen John Smoogen smo...@redhat.com 2010-09-12 
18:20:19 EDT ---
https://bugzilla.redhat.com/show_bug.cgi?id=484855

https://fedorahosted.org/fesco/ticket/225

The issue trying to solve are the following:

1) mediawiki has a short life time per releases and major updates require
schema changes to the database. There is no way to automate this in a way in
%post due to local changes and such and assuming system has database up etc.
etc.

2) some sites want to stay at a major version of mediawiki without moving to
the next one. The idea is to parallel install them and 'obsolete' old versions
when they are no longer supported upstream. THis allows an enterprise site to
do upgrades when they want to versus on a 'yum update'

Does that help explain the reasoning?

-- 
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 633104] New: Review Request: monkeysphere - Use the OpenPGP web of trust to verify ssh connections

2010-09-12 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: monkeysphere - Use the OpenPGP web of trust to verify 
ssh connections

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

   Summary: Review Request: monkeysphere - Use the OpenPGP web of
trust to verify ssh connections
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ber...@codewiz.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://people.sugarlabs.org/bernie/fedora/specs/monkeysphere.spec
SRPM URL:
http://people.sugarlabs.org/bernie/fedora/source/monkeysphere-0.31-1.src.rpm
Description:
SSH key-based authentication is tried-and-true, but it lacks a true
Public Key Infrastructure for key certification, revocation and
expiration.  Monkeysphere is a framework that uses the OpenPGP web of
trust for these PKI functions.  It can be used in both directions: for
users to get validated host keys, and for hosts to authenticate users.

-- 
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 328411] Review Request: archimedes - 2D Quantum Monte Carlo simulator for semiconductor devices

2010-09-12 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=328411

--- Comment #7 from Kevin Fenzi ke...@tummy.com 2010-09-12 18:51:01 EDT ---
Git done (by process-git-requests).

-- 
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 478613] Review Request: ledger - A powerful command-line double-entry accounting system

2010-09-12 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=478613

--- Comment #24 from Kevin Fenzi ke...@tummy.com 2010-09-12 18:52:03 EDT ---
Git done (by process-git-requests).

-- 
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 549286] Review Request: rubygem-merb-core - Lightweight Ruby-based MVC framework for web development

2010-09-12 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=549286

--- Comment #16 from Kevin Fenzi ke...@tummy.com 2010-09-12 18:55:23 EDT ---
Git done (by process-git-requests).

-- 
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 490940] Review Request: pubcookie - Open-source software for intra-institutional web authentication

2010-09-12 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=490940

--- Comment #8 from Gabriel Somlo so...@cmu.edu 2010-09-12 19:47:13 EDT ---
spec URL: http://www.contrib.andrew.cmu.edu/~somlo/mod_pubcookie.spec
srpm URL:
http://www.contrib.andrew.cmu.edu/~somlo/mod_pubcookie-3.3.4a-2.fc12.src.rpm

I believe this addresses all but one of your concerns:

mod_pubcookie-server.i686: E: arch-dependent-file-in-usr-share
/usr/share/pubcookie/cgi-bin/index.cgi

I did that on purpose, per the packgaging guidelines, specifically:

http://fedoraproject.org/wiki/Packaging/Guidelines#Web_Applications

If you know of another, more recent document that overrides the above (or know
of a better location to put a binary index.cgi file), please let me know and
I'll change this.

Thanks,
--Gabriel

-- 
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 576758] Review Request: moovida-plugins-bad - Media Center

2010-09-12 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=576758

--- Comment #6 from Graeme Gillies ggill...@redhat.com 2010-09-12 19:57:31 
EDT ---
Hi Hans,

Thanks for finishing the reviews!

I had a look at the url giving 403 errors and it seems that only the
moovida-plugins-bad-1.0.9.tar.gz link isn't working. The page with them all is
at http://www.moovida.com/download/moovidaclassic/

My FAS account name is ggillies

Regards,

Graeme

-- 
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 632874] Review Request: freedup - Links substantially identical, duplicate files to save file system space

2010-09-12 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=632874

--- Comment #2 from David Riches davi...@ultracar.co.uk 2010-09-12 20:02:36 
EDT ---
Thanks very much for the review.

I've made the relevant changes - rpmlint doesn't report anything for me now.

Dave

-- 
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 631874] Review Request: liboauth - OAuth library functions

2010-09-12 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=631874

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|supercyp...@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 633127] New: Review Request: python-zope-dottedname - Resolver for Python dotted names

2010-09-12 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: python-zope-dottedname - Resolver for Python dotted 
names

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

   Summary: Review Request: python-zope-dottedname - Resolver for
Python dotted names
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: robinlee.s...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://cheeselee.fedorapeople.org/python-zope-dottedname.spec
SRPM URL:
http://cheeselee.fedorapeople.org/python-zope-dottedname-3.4.6-2.fc13.src.rpm
Description:
The zope.dottedname module provides one function, 'resolve' that
resolves strings containing dotted names into the appropriate Python
object.

rpmlint results:
$ rpmlint ./python-zope-dottedname.spec 
./python-zope-dottedname.spec: W: no-cleaning-of-buildroot %install
./python-zope-dottedname.spec: W: no-cleaning-of-buildroot %clean
./python-zope-dottedname.spec: W: no-buildroot-tag
./python-zope-dottedname.spec: W: no-%clean-section
0 packages and 1 specfiles checked; 0 errors, 4 warnings.

$ rpmlint ./python-zope-dottedname-3.4.6-2.fc13.src.rpm 
python-zope-dottedname.src: W: no-cleaning-of-buildroot %install
python-zope-dottedname.src: W: no-cleaning-of-buildroot %clean
python-zope-dottedname.src: W: no-buildroot-tag
python-zope-dottedname.src: W: no-%clean-section
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

$ rpmlint ./python-zope-dottedname-3.4.6-2.fc13.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
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 631874] Review Request: liboauth - OAuth library functions

2010-09-12 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=631874

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #3 from Chen Lei supercyp...@gmail.com 2010-09-12 22:50:35 EDT ---
formal review here:
+:ok, =:needs attention, -:needs fixing

MUST Items:
[+] MUST: rpmlint must be run on every package.
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}
[+] MUST: The package must meet the Packaging Guidelines. [FIXME?: covers this
list and more]
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: 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.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
md5sum checksum353e5dfbc0e6f4277ff04a7eebd96bec 
[+] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.
[+] MUST: All build dependencies must be listed in BuildRequires
[+] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro.
[+] MUST: Every binary RPM package which stores shared library files (not just
symlinks) in any of the dynamic linker's default paths, must call ldconfig in
%post and %postun.
[+] MUST: 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.
[+] MUST: A package must not contain any duplicate files in the %files listing.
[+] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line.
[+] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[+] MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines.
[+] MUST: The package must contain code, or permissible content. This is
described in detail in the code vs. content section of Packaging Guidelines.
[+] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.
[+] MUST: Packages must NOT contain any .la libtool archives, these should be
removed in the spec.
[+] MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: All filenames in rpm packages must be valid UTF-8.

SHOULD Items:
[+] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
[=] SHOULD: The description and summary sections in the package spec file
should contain translations for supported Non-English languages, if available.
[+] SHOULD: The reviewer should test that the package builds in mock.
[+] SHOULD: The package should compile and build into binary rpms on all
supported architectures.
[+] SHOULD: The reviewer should test that the package functions as described.
[+] SHOULD: If scriptlets are used, those scriptlets must be sane.
[+] SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin,
/usr/bin, or /usr/sbin consider requiring the package which provides the file
instead of the file itself.


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 619991] Review Request: ghc-text - Haskell efficient packed Unicode text type

2010-09-12 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=619991

--- Comment #6 from Jens Petersen peter...@redhat.com 2010-09-12 23:23:55 EDT 
---
Well the tests need http://hackage.haskell.org/package/criterion
to run which is not yet packaged for Fedora, I left it there as
a comment for now.

-- 
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 632808] Review Request: python-zope-schema - Zope 3 schemas

2010-09-12 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=632808

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

   What|Removed |Added

 Blocks||633138

-- 
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 633127] Review Request: python-zope-dottedname - Resolver for Python dotted names

2010-09-12 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=633127

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

   What|Removed |Added

 Blocks||633138

-- 
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 632986] Review Request: python-zope-contenttype - Content-Type Handling Utility Module

2010-09-12 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=632986

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

   What|Removed |Added

 Blocks||633138

-- 
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 571993] Review Request: non-sequencer - A powerful, real-time, pattern-based MIDI sequencer

2010-09-12 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=571993

--- Comment #11 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-09-13 
00:52:14 EDT ---
Thanks for the update. I will hopefully find time this week to have a look.

(In reply to comment #10)
 I tried your suggestion regarding CXXFLAGS and that didn't seem to work as 
 then
 the various include paths were not picked up.
 

My suggestion was a guide for you to start-up. It wasn't meant to be _the_
solution. I assume that you fixed the issue with the patch.

 The icon is a resized version of the logo file included in the documentation,
 for want of something better.

BuildRequires: ImageMagick and do the conversion in %build? Or at least some
reference would be good that says where you got the image from (so that when
someone looks into the specfile, he knows there are no license issues with the
image).

-- 
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 461007] Review Request: libftdi - Library to program and control the FTDI USB controller

2010-09-12 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=461007

Lucian Langa co...@gnome.eu.org changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #15 from Lucian Langa co...@gnome.eu.org 2010-09-13 01:03:31 EDT 
---
Package Change Request
===
Package Name: libftdi
Short Description: Library to program and control the FTDI USB controller
Owners: lucilanga
Branches: EL-5 EL-6
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 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop

2010-09-12 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=632970

--- Comment #3 from Cédric OLIVIER cedric.oliv...@free.fr 2010-09-13 01:36:51 
EDT ---
I had forgotten to add gtk-update-icon-cache :

Spec URL :
http://cedric.olivier.free.fr/rpms/rec-applet-0.3.20100821/rec-applet.spec
SRPMS URL :
http://cedric.olivier.free.fr/rpms/rec-applet-0.3.20100821/rec-applet-0-0.3.20100821bzr.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