[Bug 705363] Review Request: spacewalk-web - Spacewalk Web site packages

2011-08-18 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=705363

Petr Pisar  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|ppi...@redhat.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 728826] Review Request: wadofstuff-django-serializers - Extended Django Serializer Module

2011-08-18 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=728826

--- Comment #2 from Jens Petersen  2011-08-19 02:38:49 EDT 
---
Could you please post an srpm?

You need to include the release number in changelog entries:

wadofstuff-django-serializers.noarch: W: incoherent-version-in-changelog 1.1.0
['1.1.0-1.fc15', '1.1.0-1']

-- 
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 690728] Review Request: Nitrate - A test case management system written in Django

2011-08-18 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=690728

--- Comment #55 from Jens Petersen  2011-08-19 02:39:40 
EDT ---
Ok that's cool.  I think all 3 now need input from you. :)

-- 
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 731219] Review Request: pipebench - Measures the speed of stdin/stdout communication

2011-08-18 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=731219

--- Comment #2 from Nathan Owe  2011-08-18 23:39:18 EDT ---
Also in the files section why is it %{_mandir}/man?/ instead of
%{_mandir}/man1/

-- 
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 731219] Review Request: pipebench - Measures the speed of stdin/stdout communication

2011-08-18 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=731219

Nathan Owe  changed:

   What|Removed |Added

 CC||ndowen...@gmail.com

--- Comment #1 from Nathan Owe  2011-08-18 23:16:27 EDT ---
I have created a patch to use in place of the sed. As I have seen some
reviewers prefer the patch over sed. Here is the patch if you would like to use
it:

https://gitorious.org/ndowens-packager/master/blobs/master/patches/pipebench-0.40-Makefile-CFLAGS.patch

-- 
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 730815] Review Request: pamtester - Utility to test Pluggable Authentication Modules (PAM)

2011-08-18 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=730815

Nathan Owe  changed:

   What|Removed |Added

 CC||ndowen...@gmail.com

--- Comment #1 from Nathan Owe  2011-08-18 22:39:17 EDT ---
I am wanting to make sure, you know the upstream source of this program hasn't
been updated since 2005 nor in CVS? You are probably aware of this, just
wanting to point out, just in case you didn't see it.

-- 
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 731906] Review Request: logsurfer+ - Real Time Log Monitoring and Alerting

2011-08-18 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=731906

Tomonori Takekawa  changed:

   What|Removed |Added

 CC||fedora-package-review@redha
   ||t.com
  Component|Package Review  |Package Review
Version|rawhide |el5
Product|Fedora  |Fedora 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 729512] Review Request: graphite2 - Font rendering capabilities for complex non-Roman writing systems

2011-08-18 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=729512

--- Comment #7 from Nicholas van Oudtshoorn  2011-08-18 
22:27:18 EDT ---
Thanks Martin (and Martin). Have updated the SPEC according to your suggestions
above.

As for the FSF address, I'd already notified upstream - and am pleased to see
that it's already been fixed in a new release (to which I have re-based the
package).


Spec URL: http://dl.dropbox.com/u/8648526/graphite2.spec
SRPM URL: http://dl.dropbox.com/u/8648526/graphite2-1.0.2-1.fc15.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 731906] Review Request: logsurfer+ - Real Time Log Monitoring and Alerting

2011-08-18 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=731906

Tomonori Takekawa  changed:

   What|Removed |Added

 CC||fedora-package-review@redha
   ||t.com
  Component|Package Review  |Package Review
Version|rawhide |el5
Product|Fedora  |Fedora 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 731906] Review Request: logsurfer+ - Real Time Log Monitoring and Alerting

2011-08-18 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=731906

Nathan Owe  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from Nathan Owe  2011-08-18 22:11:53 EDT ---
Setting FE-NEEDSPONSOR since you need a sponsor.

-- 
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 731906] New: Review Request: logsurfer+ - Real Time Log Monitoring and Alerting

2011-08-18 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: logsurfer+ - Real Time Log Monitoring and Alerting

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

   Summary: Review Request: logsurfer+ - Real Time Log Monitoring
and Alerting
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: t...@takekawa.tk
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://www.takekawa.tk/packages/rhel5/SRPMS/logsurfer+.spec
SRPM URL:
http://www.takekawa.tk/packages/rhel5/SRPMS/logsurfer+-1.7-1.el5.src.rpm

Description:
Logsurfer is a program for monitoring system logs in real-time, and reporting
on the occurrence of events. It is similar to the well-known swatch program on
which it is based, but offers a number of advanced features which swatch does
not support.

Hello.
This is my first package here.
So I'm very happy if I find a package sponsor.
Thank you for your time.

-- 
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 676335] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes

2011-08-18 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=676335

--- Comment #22 from Kapil Arya  2011-08-18 20:45:06 EDT ---
Hi All,

Here are the links to updated SPRM and SPEC which address the concerns raised
by Neal and others. Please let us know the feedback.

Spec URL: http://www.ccs.neu.edu/home/kapil/fedora_rpms/dmtcp.spec
SRPM URL:
http://www.ccs.neu.edu/home/kapil/fedora_rpms/dmtcp-1.2.3-1.svn1264.fc15.src.rpm

Thanks,
-Kapil

-- 
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 632056] Review Request: libsocialweb-qt - Qt4 API for libsocialweb client DBUS interface

2011-08-18 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=632056

Rex Dieter  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG
   Flag|needinfo?(supercyper1@gmail |
   |.com)   |
Last Closed||2011-08-18 20:22:48

--- Comment #6 from Rex Dieter  2011-08-18 20:22:48 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 731898] Review Request: opendkim - DKIM library and milter

2011-08-18 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=731898

Steve Jenkins  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
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 632056] Review Request: libsocialweb-qt - Qt4 API for libsocialweb client DBUS interface

2011-08-18 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=632056

Rex Dieter  changed:

   What|Removed |Added

 Blocks|656997(kde-reviews) |
 AssignedTo|rdie...@math.unl.edu|nob...@fedoraproject.org
   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 731898] New: Review Request: opendkim - DKIM library and milter

2011-08-18 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: opendkim - DKIM library and milter

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

   Summary: Review Request: opendkim - DKIM library and milter
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: st...@stevejenkins.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


This is my first package and I am seeking a sponsor.

Spec URL: http://packages.stevejenkins.com/opendkim/2.4.2/sources/opendkim.spec

SRPM URL:
http://packages.stevejenkins.com/opendkim/2.4.2/sources/opendkim-2.4.2-2.src.rpm

Description: OpenDKIM provides an open source library that implements the DKIM
service, plus a milter-based filter application that can plug in to any
milter-aware MTA, including sendmail, Postfix, or any other MTA that supports
the milter protocol.

-- 
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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System  
2011-08-18 18:03:59 EDT ---
edg-gridftp-client-1.2.9.2-2.fc16 has been pushed to the Fedora 16 testing
repository.

-- 
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 731191] Review Request: libspnav - Open source alternative to 3DConnextion drivers

2011-08-18 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=731191

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #12 from Fedora Update System  
2011-08-18 18:04:10 EDT ---
libspnav-0.2.2-2.fc16 has been pushed to the Fedora 16 testing repository.

-- 
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 731524] Review Request: gnatcoll - Suite of reusable software components and utilities

2011-08-18 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=731524

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System  2011-08-18 
18:04:21 EDT ---
gnatcoll-2011-5.fc16 has been pushed to the Fedora 16 testing repository.

-- 
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 717864] Review Request: ghc-case-insensitive - Haskell case-insensitive string comparison

2011-08-18 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=717864

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System  
2011-08-18 18:03:41 EDT ---
Package ghc-case-insensitive-0.2.0.2-1.fc16:
* should fix your issue,
* was pushed to the Fedora 16 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing
ghc-case-insensitive-0.2.0.2-1.fc16'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/ghc-case-insensitive-0.2.0.2-1.fc16
then log in and leave karma (feedback).

-- 
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 731891] New: Review Request: python-cairo - Python 2 cairo bindings

2011-08-18 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-cairo - Python 2 cairo bindings

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

   Summary: Review Request: python-cairo - Python 2 cairo bindings
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jo...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://johnp.fedorapeople.org/downloads/python-cairo/python-cairo.spec
SRPM URL:
http://johnp.fedorapeople.org/downloads/python-cairo/python-cairo-1.10.0-1.fc15.src.rpm
Description: The pycairo package changed its name internally to python-cairo to
be consistent with python3-cairo.  This unfortunately causes issues with
uploading new sources.  Since upstream has changed the build procedure and we
need to get the new package name approved and listed as a existing package, it
makes sense to put the package through another 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 731851] Review Request: pygobject3 - Parallel installable python module for GObject Introspection

2011-08-18 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=731851

--- Comment #1 from John (J5) Palmieri  2011-08-18 16:42:22 
EDT ---
You will need the latest version of pygobject2 from rawhide if you wish to
install this otherwise you will get conflicts.

-- 
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 715180] Review Request: FreeMat - A free environment for rapid engineering, scientific prototyping and data processing

2011-08-18 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=715180

--- Comment #19 from Richard Shaw  2011-08-18 16:43:12 
EDT ---
Ok, I got what looks like a good build. 

I've got two questions for now:

1. Shouldn't the desktop file go in the main package and not the data package?

2. Does blas.ini belong in the main package or data 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 731851] New: Review Request: pygobject3 - Parallel installable python module for GObject Introspection

2011-08-18 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: pygobject3 - Parallel installable python module for 
GObject Introspection

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

   Summary: Review Request: pygobject3 - Parallel installable
python module for GObject Introspection
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jo...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://johnp.fedorapeople.org/downloads/pygobject3/pygobject3.spec
SRPM URL:
http://johnp.fedorapeople.org/downloads/pygobject3/pygobject3-2.90.2-1.fc15.src.rpm
Description: This packages installs in parallel with pygobject2 to provide
GObject Introspection support.  pygobject2 sticks around to provide legacy
support for the static python gobject bindings.

-- 
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 698405] Review Request: mcollective - A framework to build server orchestration or parallel job execution systems

2011-08-18 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=698405

--- Comment #2 from Steve Traylen  2011-08-18 15:45:00 
EDT ---
1) You have a few mentions of '/etc/init.d' which should be /etc/rc.d/init.d
   and in fact should be %{_initrddir}. You have to set this if you 
   are targeting EPEL5 as well (yes please).

2) Given that mcollective and mcollective-client requires mcollective-common
   I see no need to duplicate the 

Requires: ruby
Requires: rubygems
Requires: rubygem-stomp

   on these packages.

3) Version 1.3.0 looks to be the newest version.

4) You have 
Requires: rubygem(stomp)
   and 
Requires: rubygem-stomp
   in some places. 
   See ruby guidelines:

5) The ruby(abi) requires should be defined.
   http://fedoraproject.org/wiki/Packaging:Ruby

6) Notice how ruby_sitelib is defined here.
   http://fedoraproject.org/wiki/Packaging:Ruby

-- 
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 698405] Review Request: mcollective - A framework to build server orchestration or parallel job execution systems

2011-08-18 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=698405

--- Comment #1 from Steve Traylen  2011-08-18 15:18:28 
EDT ---
rpmlint errors as below, with the exception of "no-manual-page-for-binary" I
expect the rest can 
possibly be fixed.

I'll look some more now.


SPECS/mcollective.spec:7: W: non-standard-group System Tools
SPECS/mcollective.spec:36: W: non-standard-group System Tools
SPECS/mcollective.spec:50: W: non-standard-group System Tools
SPECS/mcollective.spec:121: W: mixed-use-of-spaces-and-tabs (spaces: line 73,
tab: line 121)

mcollective.noarch: W: non-standard-group System Tools
mcollective.noarch: W: invalid-license Apache License, Version 2
mcollective.noarch: E: non-readable /etc/mcollective/server.cfg 0440L
mcollective.noarch: W: no-manual-page-for-binary mcollectived
mcollective-client.noarch: W: non-standard-group System Tools
mcollective-client.noarch: W: invalid-license Apache License, Version 2
mcollective-client.noarch: W: conffile-without-noreplace-flag
/etc/mcollective/rpc-help.erb
mcollective-client.noarch: W: no-manual-page-for-binary mc-controller
mcollective-client.noarch: W: no-manual-page-for-binary mc-rpc
mcollective-client.noarch: W: no-manual-page-for-binary mc-find-hosts
mcollective-client.noarch: W: no-manual-page-for-binary mc-inventory
mcollective-client.noarch: W: no-manual-page-for-binary mc-facts
mcollective-client.noarch: W: no-manual-page-for-binary mc-call-agent
mcollective-client.noarch: W: no-manual-page-for-binary mc-ping
mcollective-common.noarch: W: non-standard-group System Tools
mcollective-common.noarch: W: invalid-license Apache License, Version 2
mcollective.src: W: non-standard-group System Tools
mcollective.src: W: invalid-license Apache License, Version 2
mcollective.src:121: W: mixed-use-of-spaces-and-tabs (spaces: line 73, tab:
line

-- 
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 731820] Review Request: kcm-qt-graphicssystem - Easily configure the standard Qt graphics system

2011-08-18 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=731820

Rex Dieter  changed:

   What|Removed |Added

 Blocks||656997(kde-reviews)

-- 
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 731820] New: Review Request: kcm-qt-graphicssystem - Easily configure the standard Qt graphics system

2011-08-18 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: kcm-qt-graphicssystem - Easily configure the standard 
Qt graphics system

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

   Summary: Review Request: kcm-qt-graphicssystem - Easily
configure the standard Qt graphics system
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rdie...@math.unl.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://rdieter.fedorapeople.org/rpms/kcm-qt-graphicssystem/kcm-qt-graphicssystem.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/kcm-qt-graphicssystem/kcm-qt-graphicssystem-1.2-1.fc15.src.rpm
Description: Easily configure the standard Qt graphics system

-- 
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 698405] Review Request: mcollective - A framework to build server orchestration or parallel job execution systems

2011-08-18 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=698405

Steve Traylen  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||steve.tray...@cern.ch
  QAContact|extras...@fedoraproject.org |steve.tray...@cern.ch
   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 691894] Review Request: pyrit - A GPGPU-driven WPA/WPA2-PSK key cracker

2011-08-18 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=691894

--- Comment #7 from Garrett Holmstrom  2011-08-18 
14:29:32 EDT ---
Most everything looks good.  Just fix the permissions on _cpyrit_cpu.so and
that should do it.

Review of pyrit-0.4.0-2.fc15:

Mandatory review guidelines:
NO - rpmlint output
 pyrit.src: W: spelling-error %description -l en_US pre -> per, ore, pee
 pyrit.x86_64: W: spelling-error %description -l en_US pre -> per, ore, pee
 pyrit.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/cpyrit/_cpyrit_cpu.so 0775L
 pyrit.x86_64: W: no-manual-page-for-binary pyrit
 --
 The spelling-error warnings are bogus.  Man pages are optional.
ok - Package meets naming guidelines
ok - Spec file name matches base package name
ok - License is acceptable (GPLv3+ with exceptions)
ok - License field in spec is correct
 The file "pyrit" has no license header; assuming GPLv3+
ok - License files included in package %docs or not included in upstream source
ok - License files installed when any subpackage combination is installed
ok - Spec written in American English
ok - Spec is legible
ok - Sources match upstream unless altered to fix permissibility issues
 Upstream MD5:  7258b6f3dacfb09736ddeed2a379df2d  pyrit-0.4.0.tar.gz
 Your MD5:  7258b6f3dacfb09736ddeed2a379df2d  pyrit-0.4.0.tar.gz
ok - Build succeeds on at least one supported platform
ok - Build succeeds on all supported platforms or has ExcludeArch + bugs filed
ok - BuildRequires correct
-- - Package handles locales with %find_lang
-- - %post, %postun call ldconfig if package contains shared .so files
ok - No bundled system libs
-- - Relocatability is justified
ok - Package owns all directories it creates
ok - Package requires other packages for directories it uses but does not own
ok - No duplicate files in %files unless necessary for license files
NO - File permissions are sane
 -rwxrwxr-x /usr/lib64/python2.7/site-packages/cpyrit/_cpyrit_cpu.so
ok - Each %files section contains %defattr on EL4
ok - Consistent use of macros
ok - Sources contain only permissible code or content
-- - Large documentation files go in -doc package
ok - Missing %doc files do not affect runtime
-- - Headers go in -devel package
-- - Static libs go in -static package
-- - Unversioned .so files go in -devel package
-- - Devel packages require base with fully-versioned dependency
ok - Package contains no .la files
-- - GUI app installs .desktop file w/desktop-file-install or has justification
ok - Package's files and directories don't conflict with others' or justified
ok - File names are valid UTF-8

Optional review guidelines:
ok - Query upstream about including license files
no - Translations of description, Summary
ok - Builds in mock
ok - Builds on all supported platforms
ok - Functions as described
-- - Scriptlets are sane
-- - Non-devel subpackage Requires are sane
-- - .pc files go in -devel unless main package is a development tool
ok - No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin
no - Man pages included for all executables
ok - Package with test-suite executes it in %check section

Packaging guidelines:
ok - Has dist tag
ok - Useful without external bits
ok - Package obeys FHS, except libexecdir, /usr/target, /run
-- - Programs launched before FS mounting use /run instead of /var/run
-- - Binaries in /bin, /sbin do not depend on files in /usr
ok - Changelog in prescribed format
ok - Spec file lacks Packager, Vendor, PreReq tags
no - Correct BuildRoot tag on < F10/EL6
 This prevents building for epel-5
no - Correct %clean section on < F13/EL6
 This prevents building for epel-5
ok - Requires correct, justified where necessary
ok - Summary, description do not use trademarks incorrectly
ok - All relevant documentation is packaged, tagged appropriately
ok - Documentation files do not have executable permissions
ok - %build honors applicable compiler flags or justifies otherwise
-- - Package with .pc files Requires pkgconfig on < EL6
ok - Useful -debuginfo package or disabled and justified
ok - No static executables
ok - Rpath absent or only used for internal libs
-- - Config files marked with %config
-- - %config files marked noreplace or justified
-- - No %config files under /usr
-- - SysV-style init script
ok - Spec uses macros instead of hard-coded directory names where appropriate
ok - Spec uses macros for executables only when configurability is needed
-- - %makeinstall used only when ``make install DESTDIR=...'' doesn't work
ok - Macros in Summary, %description expandable at SRPM build time
-- - Spec uses %{SOURCE#} instead of $RPM_SOURCE_DIR or %{sourcedir}
ok - %global instead of %define where appropriate
-- - Package containing translations BuildRequires gettext
-- - File timestamps preserved by file ops
-- - Parallel make
ok - Spec does not use Requires(pr

[Bug 715180] Review Request: FreeMat - A free environment for rapid engineering, scientific prototyping and data processing

2011-08-18 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=715180

--- Comment #18 from Richard Shaw  2011-08-18 13:52:29 
EDT ---
Ok, that will work. I tried doing some simple searching but didn't find that
bug report.

-- 
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 731524] Review Request: gnatcoll - Suite of reusable software components and utilities

2011-08-18 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=731524

--- Comment #7 from Fedora Update System  2011-08-18 
13:43:34 EDT ---
gnatcoll-2011-5.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/gnatcoll-2011-5.fc16

-- 
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 697049] Review Request: sawfish-pager - A pager for the Sawfish window manager

2011-08-18 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=697049

Fedora Update System  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 731524] Review Request: gnatcoll - Suite of reusable software components and utilities

2011-08-18 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=731524

--- Comment #6 from Fedora Update System  2011-08-18 
13:20:20 EDT ---
gnatcoll-2011-5.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/gnatcoll-2011-5.fc15

-- 
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 697049] Review Request: sawfish-pager - A pager for the Sawfish window manager

2011-08-18 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=697049

--- Comment #21 from Fedora Update System  
2011-08-18 13:20:35 EDT ---
sawfish-pager-0.90.2-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/sawfish-pager-0.90.2-3.fc15

-- 
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 731524] Review Request: gnatcoll - Suite of reusable software components and utilities

2011-08-18 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=731524

Fedora Update System  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 731683] Review Request: pyppd - PPD file compressor and generator for CUPS

2011-08-18 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=731683

--- Comment #3 from Jiri Popelka  2011-08-18 13:22:35 EDT 
---
[OK] MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
pyppd.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/pyppd/pyppd-ppdfile.in 0644L /usr/bin/python
There's a comment in spec file saying that pyppd-ppdfile.in is only a template
so I think this is acceptable.

[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}, in the
format %{name}.spec unless your package has an exemption.
[OK] MUST: The package must meet the Packaging Guidelines.
[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: 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] MUST: The spec file must be written in American English.
[OK] MUST: The spec file for the package MUST be legible.
[OK] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task.
7161916c91af10b0493fa6b31ff53a0d  pyppd-0.4.9.tar.gz
7161916c91af10b0493fa6b31ff53a0d  pyppd-0.4.9.tar.gz

[OK] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[N/A] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch.
[OK] MUST: All build dependencies must be listed in BuildRequires.
[N/A] MUST: The spec file MUST handle locales properly. This is done by using
the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[N/A] MUST: Every binary RPM package (or subpackage) which stores shared
library files (not just symlinks) in any of the dynamic linker's default paths,
must call ldconfig in %post and %postun.
[OK] MUST: Packages must NOT bundle copies of system libraries.
[N/A] MUST: If the package is designed to be relocatable, the packager must
state this fact in the request for review, along with the rationalization for
relocation of that specific package.
[OK] 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.
[OK] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)
[OK] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example.
[OK] MUST: Each package must consistently use macros.
[OK] MUST: The package must contain code, or permissable content.
[OK] MUST: Large documentation files must go in a -doc subpackage.
Do we need the -doc subpackage for 4 txt files having 32KiB in sum ?

[OK] MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program must
run properly if it is not present.
[N/A] MUST: Header files must be in a -devel package.
[N/A] MUST: Static libraries must be in a -static package.
[N/A] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) 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: Requires: %{name}%{?_isa} =
%{version}-%{release}
[N/A] MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built.
[N/A] 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.
[N/A] MUST: Packages must not own files or directories already owned by other
packages.
[OK] MUST: All filenames in rpm packages must be valid UTF-8.

[N/A] 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.
[N/A] SHOULD: The description and summary sections in the package spec file
should contain translations for supported Non-English languages, if available.
[OK] SHOULD: The reviewer should test that the package builds in mock.
[OK] SHOULD: The package should compile and build into binary rpms on all
supported architectures.
[OK] SHOULD: The reviewer should test that the package functions as described.
A package should not segfault instead of running, for example.
[N/A

[Bug 728649] Review Request: airrac - C++ Simulated Revenue Accounting (RAC) System Library

2011-08-18 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=728649

Martin Gieseking  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||martin.giesek...@uos.de
 AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de
   Flag||fedora-review?

--- Comment #2 from Martin Gieseking  2011-08-18 
13:16:46 EDT ---
I'm going to sponsor Martin, so taking over the review. Since I'm busy at the
moment, I'll continue later tonight or tomorrow morning.

-- 
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 731683] Review Request: pyppd - PPD file compressor and generator for CUPS

2011-08-18 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=731683

Jiri Popelka  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|jpope...@redhat.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 727780] Review Request: ruby-qt - Ruby bindings for Qt 4

2011-08-18 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=727780

Ngo Than  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-08-18 13:10:31

--- Comment #18 from Ngo Than  2011-08-18 13:10:31 EDT ---
ruby-qt was commited and built. 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 731455] Review Request: django-authopenid - Django application to integrate Django authentication system with OpenID

2011-08-18 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=731455

--- Comment #9 from Jon Ciesla  2011-08-18 13:08:12 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 731794] New: Review Request: Qt Quick 3D - Qt 3D for QML scripts

2011-08-18 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: Qt Quick 3D - Qt 3D for QML scripts

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

   Summary: Review Request: Qt Quick 3D - Qt 3D for QML scripts
   Product: Fedora
   Version: 15
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: tomasmiljeno...@hotmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


SPEC file:
http://live.tomasm.tk/fedorarepo/TMM/qt-quick3d.spec

SRPM:
http://live.tomasm.tk/fedorarepo/TMM/SRPMS/qt-quick3d-0.tp1.1.TMM.fc15.srpm


Despite what I said on #fedora-kde, this can also be built separately from Qt
(just needs qt4 sources); specfile attached

-- 
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 715180] Review Request: FreeMat - A free environment for rapid engineering, scientific prototyping and data processing

2011-08-18 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=715180

--- Comment #17 from Ankur Sinha  2011-08-18 12:41:01 
EDT ---
(In reply to comment #16)
> Ankur,
> 
> It looks like FreeMat requires llvm 2.9 and only 2.8 is available for Fedora.
> Have you contacted the llvm maintainer to see if there is a plan to go to 2.9?

Hi Richard,

There was already a bug filed. llvm 2.9 will not hit f15. It'll only hit f16
onwards. Here's Michael's comment to the bug:

https://bugzilla.redhat.com/show_bug.cgi?id=694785#c6

Therefore, freemat builds for F16+ :)
I won't request an f15 branch. 

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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

--- Comment #5 from Fedora Update System  2011-08-18 
12:41:26 EDT ---
edg-gridftp-client-1.2.9.2-2.el5 has been submitted as an update for Fedora
EPEL 5.
https://admin.fedoraproject.org/updates/edg-gridftp-client-1.2.9.2-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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

--- Comment #6 from Fedora Update System  2011-08-18 
12:41:33 EDT ---
edg-gridftp-client-1.2.9.2-2.fc15 has been submitted as an update for Fedora
15.
https://admin.fedoraproject.org/updates/edg-gridftp-client-1.2.9.2-2.fc15

-- 
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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

--- Comment #7 from Fedora Update System  2011-08-18 
12:41:40 EDT ---
edg-gridftp-client-1.2.9.2-2.el4 has been submitted as an update for Fedora
EPEL 4.
https://admin.fedoraproject.org/updates/edg-gridftp-client-1.2.9.2-2.el4

-- 
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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

Fedora Update System  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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

--- Comment #9 from Fedora Update System  2011-08-18 
12:41:56 EDT ---
edg-gridftp-client-1.2.9.2-2.el6 has been submitted as an update for Fedora
EPEL 6.
https://admin.fedoraproject.org/updates/edg-gridftp-client-1.2.9.2-2.el6

-- 
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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

--- Comment #8 from Fedora Update System  2011-08-18 
12:41:48 EDT ---
edg-gridftp-client-1.2.9.2-2.fc16 has been submitted as an update for Fedora
16.
https://admin.fedoraproject.org/updates/edg-gridftp-client-1.2.9.2-2.fc16

-- 
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 730657] Review Request: perl-DateTime-TimeZone - Time zone object base class and factory

2011-08-18 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=730657

Iain Arnell  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Blocks||730658
 Depends on|730658  |
 Resolution||RAWHIDE
Last Closed||2011-08-18 12:35:34

--- Comment #8 from Iain Arnell  2011-08-18 12:35:34 EDT ---
Closing this. Track progress in bug 730658.

-- 
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 731455] Review Request: django-authopenid - Django application to integrate Django authentication system with OpenID

2011-08-18 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=731455

Ankur Sinha  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Ankur Sinha  2011-08-18 12:33:59 
EDT ---
Package Change Request
==
Package Name: django-authopenid
New Branches: f15 f16
Owners: ankursinha sundaram
InitialCC: 

Creating branches for f15 f16.

-- 
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 730656] Review Request: perl-DateTime-Locale - Localization support for DateTime.pm

2011-08-18 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=730656

Iain Arnell  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Blocks||730658
 Depends on|730658  |
 Resolution||RAWHIDE
Last Closed||2011-08-18 12:34:48

--- Comment #7 from Iain Arnell  2011-08-18 12:34:48 EDT ---
Closing this. Track progress in bug 730658.

-- 
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 715180] Review Request: FreeMat - A free environment for rapid engineering, scientific prototyping and data processing

2011-08-18 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=715180

--- Comment #16 from Richard Shaw  2011-08-18 12:35:13 
EDT ---
Ankur,

It looks like FreeMat requires llvm 2.9 and only 2.8 is available for Fedora.
Have you contacted the llvm maintainer to see if there is a plan to go to 2.9?

-- 
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 730657] Review Request: perl-DateTime-TimeZone - Time zone object base class and factory

2011-08-18 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=730657

Iain Arnell  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

--- Comment #7 from Iain Arnell  2011-08-18 12:10:05 EDT ---
Imported and built for rawhide.

-- 
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 730656] Review Request: perl-DateTime-Locale - Localization support for DateTime.pm

2011-08-18 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=730656

Iain Arnell  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

--- Comment #6 from Iain Arnell  2011-08-18 12:09:44 EDT ---
Imported and built for rawhide.

-- 
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 725614] Review Request: libcryptui - Library for prompting for GPG keys

2011-08-18 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=725614

--- Comment #9 from Jon Ciesla  2011-08-18 12:03:20 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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

--- Comment #4 from Jon Ciesla  2011-08-18 12:02:28 EDT ---
Git done (by process-git-requests).

Edited request for syntax, let me know if something isn't as expected.

-- 
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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

Steve Traylen  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+, fedora-cvs?

--- Comment #3 from Steve Traylen  2011-08-18 11:43:55 
EDT ---

Mattias,

  I switched the wrong flag, could your reassign the flag.



Steve

-- 
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 708765] Review Request: Frogr - Flickr Remote Organizer for GNOME

2011-08-18 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=708765

--- Comment #75 from Mario Sanchez Prada  2011-08-18 
11:44:53 EDT ---
As you wish, but as I said, I think it's very likely that I release the 0.6.1
during today or tomorrow (I found some kind beta testers who will help me
trying it :-)), so keep that in mind in case you finally prefer to wait a day
more or 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 727030] Review Request: ufw - uncomplicated firewall

2011-08-18 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=727030

Nathan Owe  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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

Steve Traylen  changed:

   What|Removed |Added

   Flag|fedora-review+  |fedora-review?

--- Comment #2 from Steve Traylen  2011-08-18 11:40:27 
EDT ---
New Package SCM Request
===
Package Name: edg-gridftp-client
Short Description: Command line clients to GridFTP libraries
Owners: stevetraylen
Branches: f15 f16 el4 el5 el6
InitialCC

Thanks for the review, I'll glob the man page endings before import.

Steve.

-- 
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 725614] Review Request: libcryptui - Library for prompting for GPG keys

2011-08-18 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=725614

Matthew Barnes  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Matthew Barnes  2011-08-18 11:43:07 EDT 
---
New Package SCM Request
===
Package Name: libcryptui
Short Description: Interface components for OpenPGP
Owners: tbzatek
Branches: f16
InitialCC: mbarnes

Tomas already owns seahorse.  He agreed to take libcryptui.

-- 
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 708765] Review Request: Frogr - Flickr Remote Organizer for GNOME

2011-08-18 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=708765

--- Comment #74 from Gianluca Sforna  2011-08-18 11:20:23 EDT 
---
Once in the repositories, the package can be updated a number of times during
the lifetime of each Fedora release so no, the memory leak is not a problem for
the review. I'll try to finalize it as soon as possible.

-- 
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 731066] Review Request: django-annoying - Eliminate annoying things in the Django framework

2011-08-18 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=731066

--- Comment #3 from Jon Ciesla  2011-08-18 11:18:55 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 725614] Review Request: libcryptui - Library for prompting for GPG keys

2011-08-18 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=725614

Matthias Clasen  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Matthias Clasen  2011-08-18 11:13:33 
EDT ---
Looks good to me now; 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 728691] Review Request: django-registration - A user-registration application for Django

2011-08-18 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=728691

--- Comment #11 from Jon Ciesla  2011-08-18 11:18:18 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 731683] Review Request: pyppd - PPD file compressor and generator for CUPS

2011-08-18 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=731683

--- Comment #2 from Tim Waugh  2011-08-18 11:02:59 EDT ---
Thanks.  Updated URLs:

Spec URL: http://twaugh.fedorapeople.org/pyppd/pyppd.spec
SRPM URL: http://twaugh.fedorapeople.org/pyppd/pyppd-0.4.9-1.fc16.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 719456] Review Request: edg-gridftp-client - Command line clients to GridFTP libraries

2011-08-18 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=719456

Mattias Ellert  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mattias.ell...@fysast.uu.se
 AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se
   Flag||fedora-review+

--- Comment #1 from Mattias Ellert  2011-08-18 
11:03:08 EDT ---
Fedora review edg-gridftp-client 2011-08-18

+ rpmlint output:

edg-gridftp-client.x86_64: W: spelling-error %description -l en_US mkdir -> mk
dir, mk-dir, midiron
edg-gridftp-client.x86_64: W: spelling-error %description -l en_US rmdir -> rm
dir, rm-dir, midiron
edg-gridftp-client.src: W: spelling-error %description -l en_US mkdir -> mk
dir, mk-dir, midiron
edg-gridftp-client.src: W: spelling-error %description -l en_US rmdir -> rm
dir, rm-dir, midiron
edg-gridftp-client.src: W: invalid-url Source0:
edg-gridftp-client-1.2.9.2.tar.gz
3 packages and 0 specfiles checked; 0 errors, 5 warnings.

These warnings can be ignored.

+ The package is named according to the guidelines
+ The specfile is named after the package
+ The package's License tag (EU Datagrid) is a Fedora approved license
+ The License tag matches the actual license statements in the sources
+ Package installs LICENSE file as %doc
+ Specfile is written in legible English

+ Source matches upstream (ignoring CVS directories)

diff -ur --exclude CVS checkout/edg-gridftp-client-1.2.9.2
edg-gridftp-client-1.2.9.2
(no output)

+ The package builds in mock (F15 x96_64)
+ BuildRequires are sane
+ no locales
+ no shared libraries
+ no bundled sources
+ no created directories
+ no duplicate %files

Comment: The manpages are normally listed as xxx.1* and not xxx.1.* in the
%files section in order to also allow non-compressed files.

+ File permissions are sane
+ Macros are used consistently
+ Package contains code
+ No large documentation files
+ %doc is not runtime essential
+ no headers
+ no static libraries
+ no sub-packages
+ no libtool archives
+ no GUIs
+ Package does not own other's directories
+ Filenames are UTF8


Apart from the comment about the manpages in %files there are no issues.

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 716299] Review Request: clipit - lightweight, fully featured GTK+ clipboard manager

2011-08-18 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=716299

--- Comment #14 from Nikos Roussos  2011-08-18 11:01:28 
EDT ---
Hi Christoph. Sorry for the delay, but we are kind in a vacation mode here in
Greece :) I'm getting back to packaging right away. I'll soon have 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 731066] Review Request: django-annoying - Eliminate annoying things in the Django framework

2011-08-18 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=731066

Praveen Kumar  changed:

   What|Removed |Added

   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 728691] Review Request: django-registration - A user-registration application for Django

2011-08-18 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=728691

Praveen Kumar  changed:

   What|Removed |Added

   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 621559] Review Request: perl-Data-Properties - Perl equivalent of java.util.Properties

2011-08-18 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=621559

--- Comment #7 from Jessica Jones  2011-08-18 10:03:48 EDT 
---
As this was being added as a required dependency of Gold Allocation Manager,
I've asked the Gold maintainer if a different Perl module might be used
instead, but understandably he does not wish to change.

The maintainer of Data::Properties does not wish to maintain his package and
has refused to update it as 'a superior version exists'.

I attempted to build perl-ExtUtils-MakeMaker, but at present it does not
compile on Fedora 16.

-- 
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 648093] Review Request: ghc-attoparsec-enumerator - Convert an Attoparsec parser into an iteratee

2011-08-18 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=648093

Lakshmi Narasimhan  changed:

   What|Removed |Added

 CC||lakshminaras2...@gmail.com
   Flag||needinfo?(maths...@gmail.co
   ||m)

--- Comment #2 from Lakshmi Narasimhan  2011-08-18 
10:03:02 EDT ---
I can take up this for review.

Ben,
Can you update the spec file to cabal2spec-0.24?

-- 
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 717867] Review Request: ghc-http-types - Generic HTTP types for Haskell

2011-08-18 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=717867

Lakshmi Narasimhan  changed:

   What|Removed |Added

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

--- Comment #1 from Lakshmi Narasimhan  2011-08-18 
10:00:08 EDT ---
Will review 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 621559] Review Request: perl-Data-Properties - Perl equivalent of java.util.Properties

2011-08-18 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=621559

--- Comment #6 from Jessica Jones  2011-08-18 09:56:35 EDT 
---
Upstream does not want to update the module in CPAN as it is no longer
maintained.  I am only really adding it as it is a dependency of something
else.

Attempting to rebuild has highlighted a dependency on perl-ExtUtils-MakeMaker,
which is not part of Fedora at present.

-- 
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 231861] Merge Review: cyrus-imapd - high-performance mail server (IMAP, POP3, ...)

2011-08-18 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=231861

Michal Sekletar  changed:

   What|Removed |Added

   Flag||fedora-review?

--- Comment #20 from Michal Sekletar  2011-08-18 09:39:11 
EDT ---
Git commit
6fe53495447ec5cf6bb6bf697335c732f839f81e
YES source file match upstream
YES package meets naming and versioning guidelines
YES spec file is properly named, is cleanly written and uses macros
consistently
YES dist tag is present
YES clean section and buildroot present
YES license field matches the actual license
YES license is included in package
YES latest version is being packaged
YES build BuildRequires are proper
YES compiler flags are appropriate
YES package builds in mock
YES debuginfo package seems ok
NO  rpmlint is silent
 - some spelling-error warnings could be ignored
 - %patch1 and %patch2 are commented out but still in preamble of spec file
 - invalid patch url (Patch3)
 - some warnings about missing manpages for utils - ignored
 - warnings about directory permissions - ignored
 - warnings about hardcoded library paths - might be solved by replacing 
   hard-coded paths with macros
 - private-shared-objects-provides warnings
 - rpmlint output was consulted with maintainer, added as attachment

YES final provides and requires look sane
N/A %check is present and all tests pass
YES no shared libraries are added to regular linker search paths
YES it owns directories it creates
YES it doesn't own directories it shouldn't
YES no duplicates in %files
YES scriptlets looks sane
YES code, not content
YES large documentation must go in a -doc subpackage
 - documentation is about 1MB in size, not big enough that it is worthed
   to make standalone doc package 
YES %docs are not necessary for the proper functioning of the package
YES no headers
YES no pkgconfig files
YES no libtool .la droppings
YES not a GUI app

-- 
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 231861] Merge Review: cyrus-imapd - high-performance mail server (IMAP, POP3, ...)

2011-08-18 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=231861

--- Comment #19 from Michal Sekletar  2011-08-18 09:37:39 
EDT ---
Created attachment 518874
  --> https://bugzilla.redhat.com/attachment.cgi?id=518874
cyrus-imapd fedpkg lint log

-- 
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 728649] Review Request: airrac - C++ Simulated Revenue Accounting (RAC) System Library

2011-08-18 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=728649

Martin Krizek  changed:

   What|Removed |Added

 CC||mkri...@redhat.com

Bug 728649 depends on bug 702987, which changed state.

Bug 702987 Summary: Review Request: stdair - C++ Standard Airline IT Library
https://bugzilla.redhat.com/show_bug.cgi?id=702987

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #1 from Martin Krizek  2011-08-18 09:24:08 EDT 
---
Hi Denis,
*informal* review follows:

[ OK ] MUST: rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.
$ rpmlint rpmbuild/SPECS/airrac.spec Downloads/airrac-0.1.0-1.fc15.src.rpm 
1 packages and 1 specfiles checked; 0 errors, 0 warnings.
[ OK ] MUST: The package must be named according to the Package Naming
Guidelines.
[ BAD ] MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption. ===> Needs to be
renamed to airrac.spec
[ OK ] MUST: The package must meet the Packaging Guidelines.
[ 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: 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 ] MUST: The spec file must be written in American English.
[ OK ] MUST: The spec file for the package MUST be legible.
[ OK ] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task.
If no upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
[ BAD ] MUST: The package MUST successfully compile and build into binary rpms
on at least one primary architecture.
[ BAD ] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
===> I could not build rpm from provided srpm using 'mock -r fedora-15-x86_64
--rebuild airrac-0.1.0-1.fc15.src.rpm' -- seems like problem with tex, sorry if
it was my bad due to unapproriate usage of mock.
[ OK ] MUST: 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.
[ NA ] MUST: The spec file MUST handle locales properly. This is done by using
the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[ OK ] MUST: Every binary RPM package (or subpackage) which stores shared
library files (not just symlinks) in any of the dynamic linker's default paths,
must call ldconfig in %post and %postun.
[ OK ] MUST: Packages must NOT bundle copies of system libraries.
[ OK ] MUST: If the package is designed to be relocatable, the packager must
state this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.
[ OK ] 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.
[ OK ] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)
[ OK ] MUST: Permissions on files must be set properly. Executables should be
set with executable permissions, for example.
[ OK ] MUST: Each package must consistently use macros.
[ OK ] MUST: The package must contain code, or permissable content.
[ OK ] MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity).
[ OK ] MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program must
run properly if it is not present.
[ OK ] MUST: Header files must be in a -devel package.
[ OK ] MUST: St

[Bug 729263] Review Request: rubygem-virt-p2v - Send a machine's storage and metadata to virt-p2v-server

2011-08-18 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=729263

Richard W.M. Jones  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|rjo...@redhat.com

--- Comment #7 from Richard W.M. Jones  2011-08-18 09:12:24 
EDT ---
New Package SCM Request
===
Package Name: rubygem-virt-p2v
Short Description: Send a machine's storage and metadata to virt-p2v-server
Owners: mdbooth rjones
Branches: f14 f15 f16
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 727780] Review Request: ruby-qt - Ruby bindings for Qt 4

2011-08-18 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=727780

--- Comment #17 from Jon Ciesla  2011-08-18 09:02:40 EDT ---
Complete, but there were hiccups, let me know if anything isn't right.  It
should be fine.

-- 
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 729263] Review Request: rubygem-virt-p2v - Send a machine's storage and metadata to virt-p2v-server

2011-08-18 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=729263

--- Comment #6 from Jon Ciesla  2011-08-18 09:01:23 EDT ---
Richard, please take ownership of the bug, and correct your FAS username
in the request.  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 729263] Review Request: rubygem-virt-p2v - Send a machine's storage and metadata to virt-p2v-server

2011-08-18 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=729263

Matthew Booth  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Matthew Booth  2011-08-18 08:56:39 EDT 
---
New Package SCM Request
===
Package Name: rubygem-virt-p2v
Short Description: Send a machine's storage and metadata to virt-p2v-server
Owners: mdbooth rwmjones
Branches: f14 f15 f16
InitialCC: mdbooth

-- 
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 730657] Review Request: perl-DateTime-TimeZone - Time zone object base class and factory

2011-08-18 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=730657

--- Comment #6 from Jon Ciesla  2011-08-18 08:57:42 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 730656] Review Request: perl-DateTime-Locale - Localization support for DateTime.pm

2011-08-18 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=730656

--- Comment #5 from Jon Ciesla  2011-08-18 08:56:43 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 708765] Review Request: Frogr - Flickr Remote Organizer for GNOME

2011-08-18 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=708765

--- Comment #73 from Mario Sanchez Prada  2011-08-18 
08:44:31 EDT ---
(In reply to comment #72)
> [...]
> If »soon« means within a week or two, I would wait with packaging. The "new
> package" policy in Fedora allows me to add a package during the whole
> distribution forming process. However, the package could remain in the testing
> repository somewhat longer, but it would be available anyway. In my mind, it 
> is
> better to have as less patches as possible. Let's wait for 0.6.1.

In this specific case, I'm afraid soon means "really soon", like in "this
week", which was my original idea when I wrote that comment, but even perhaps
during today or tomorrow, if I manage to get some time.

Since it's a bugfixing release, and the number of patches to include is fairly
small, not much additional testing will be needed, nor the usual additional
steps that use to take more time when doing a normal release, so let's be
optimist :-)

Thanks for understanding,
Mario

-- 
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 730657] Review Request: perl-DateTime-TimeZone - Time zone object base class and factory

2011-08-18 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=730657

Iain Arnell  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Iain Arnell  2011-08-18 08:37:30 EDT ---
New Package SCM Request
===
Package Name: perl-DateTime-TimeZone
Short Description: Time zone object base class and factory
Owners: iarnell
Branches: f14 f15 f16
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 730656] Review Request: perl-DateTime-Locale - Localization support for DateTime.pm

2011-08-18 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=730656

Iain Arnell  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Iain Arnell  2011-08-18 08:36:14 EDT ---
New Package SCM Request
===
Package Name: perl-DateTime-Locale
Short Description: Localization support for DateTime.pm
Owners: iarnell
Branches: f14 f15 f16
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 726131] Review Request: yourls - your own url shortening service

2011-08-18 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=726131

Martin Gieseking  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de
   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 708765] Review Request: Frogr - Flickr Remote Organizer for GNOME

2011-08-18 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=708765

--- Comment #72 from Mario Blättermann  2011-08-18 08:18:01 
EDT ---
(In reply to comment #71)
> So, I'll probably release 0.6.1 soon, including that patch and some others 
> less
> important (but important anyway) bugs detected after the release. In summary,
> the patches I'd be applying, unless some other sever bugs show up, would be:
> 
>   - fix a HUGE memory leak loading pixbufs in details window
>   - more mem leaks fixed by Christophe Fergeau
>   - a segfault in systems using GTK < 2.24
>   - some problems in the Mac version (can't authorize)
> 
> Not sure whether this impacts or not packaging frogr for fedora, but just
> telling you in case you prefer to wait for the new bugfixing release.

If »soon« means within a week or two, I would wait with packaging. The "new
package" policy in Fedora allows me to add a package during the whole
distribution forming process. However, the package could remain in the testing
repository somewhat longer, but it would be available anyway. In my mind, it is
better to have as less patches as possible. Let's wait for 0.6.1.

-- 
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 225658] Merge Review: cpuspeed

2011-08-18 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=225658

Ondrej Vasik  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||ova...@redhat.com
 Resolution||WONTFIX
Last Closed||2011-08-18 08:10:53

--- Comment #5 from Ondrej Vasik  2011-08-18 08:10:53 EDT ---
as cpuspeed is dead package now, closing WONTFIX...

-- 
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 731110] Review Request: maven-filtering - Shared component providing resource filtering

2011-08-18 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=731110

Tomas Radej  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-08-18 08:11:45

-- 
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 731683] Review Request: pyppd - PPD file compressor and generator for CUPS

2011-08-18 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=731683

Jiri Popelka  changed:

   What|Removed |Added

 CC||jpope...@redhat.com

--- Comment #1 from Jiri Popelka  2011-08-18 08:07:25 EDT 
---
Just informally:
You can remove the %defattr clauses, see
https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions

-- 
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 731110] Review Request: maven-filtering - Shared component providing resource filtering

2011-08-18 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=731110

Fedora Update System  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 731110] Review Request: maven-filtering - Shared component providing resource filtering

2011-08-18 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=731110

--- Comment #6 from Fedora Update System  2011-08-18 
08:01:56 EDT ---
maven-filtering-1.0-2.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/maven-filtering-1.0-2.fc16

-- 
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 729263] Review Request: rubygem-virt-p2v - Send a machine's storage and metadata to virt-p2v-server

2011-08-18 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=729263

Richard W.M. Jones  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Richard W.M. Jones  2011-08-18 07:36:38 
EDT ---
Updated spec file includes a changelog section:
http://oirase.annexia.org/reviews/rubygem-virt-p2v.spec

I have pushed a patch upstream which updates the FSF
addresses in all comments and documentation:
http://git.fedorahosted.org/git/?p=virt-v2v.git;a=commitdiff;h=2b4d755b3e1f249f2a7631e1450f5b225c31761e

-

Therefore this package is APPROVED by rjones.

-- 
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 729263] Review Request: rubygem-virt-p2v - Send a machine's storage and metadata to virt-p2v-server

2011-08-18 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=729263

--- Comment #3 from Richard W.M. Jones  2011-08-18 07:22:25 
EDT ---
This package is ExclusiveArch: i686 (for perfectly good reasons).
However this means I could not build it on my local machine.  Therefore
all testing was done using the packages built in Koji.

- rpmlint output

No %changelog section in the spec file.

Multiple instances of 'incorrect-fsf-address' in the source.  I will
submit a patch upstream to fix this.

+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines

%defattr lines are not strictly required, but can be left in for
EPEL compatibility.

BuildRoot is no longer needed, but can be left in for EPEL compat.

+ license meets guidelines and is acceptable to Fedora
+ license matches the actual package license
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
+ package successfully builds on at least one architecture
  (only on i686)
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
  (because of Koji scratch build)
n/a %find_lang instead of %{_datadir}/locale/*
n/a binary RPM with shared library files must call ldconfig in %post and
%postun
+ does not use Prefix: /usr
+ package owns all directories it creates
+ no duplicate files in %files
+ consistent use of macros
+ package must contain code or permissible content
+ large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
n/a header files should be in -devel
n/a static libraries should be in -static
n/a packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
n/a libfoo.so must go in -devel
n/a -devel must require the fully versioned base
n/a packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file

Although this is a "GUI app" it is not a desktop app.  It boots
from an ISO on a separate machine, so no desktop file is required.

+ packages must not own files or directories owned by other packages
+ filenames must be valid UTF-8

Optional:

n/a if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if
available
+ reviewer should build the package in mock
+ the package should build into binary RPMs on all supported architectures
+ review should test the package functions as described
n/a scriptlets should be sane
n/a pkgconfig files should go in -devel
n/a shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
/usr/sbin

-

The package has a couple of errors which are very simple to fix, so I'll
just go ahead and submit fixes on behalf of the packager.

-- 
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 729263] Review Request: rubygem-virt-p2v - Send a machine's storage and metadata to virt-p2v-server

2011-08-18 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=729263

Richard W.M. Jones  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 726210] Review Request: freewrl - X3D / VRML visualization program

2011-08-18 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=726210

--- Comment #20 from Fedora Update System  
2011-08-18 07:10:59 EDT ---
freewrl-1.22.12-0.3.pre2.el5.2 has been submitted as an update for Fedora EPEL
5.
https://admin.fedoraproject.org/updates/freewrl-1.22.12-0.3.pre2.el5.2

-- 
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 708765] Review Request: Frogr - Flickr Remote Organizer for GNOME

2011-08-18 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=708765

--- Comment #71 from Mario Sanchez Prada  2011-08-18 
07:03:30 EDT ---
(In reply to comment #70)
> Sorry for the noise, the bug was in my own system, see
> https://bugzilla.gnome.org/show_bug.cgi?id=656688#c2. All is OK now, please go
> ahead with reviewing.

That's great. However, and in order to perpetuate the bad luck I'm having with
this tiny app :(, I think I will need to rollout a 0.6.1 bugfix release anyway,
since I found a HUGE memory leak that is causing frogr to waste quite a lot of
memory whenever the user opens the details dialog (a pretty common operation).

So, I'll probably release 0.6.1 soon, including that patch and some others less
important (but important anyway) bugs detected after the release. In summary,
the patches I'd be applying, unless some other sever bugs show up, would be:

  - fix a HUGE memory leak loading pixbufs in details window
  - more mem leaks fixed by Christophe Fergeau
  - a segfault in systems using GTK < 2.24
  - some problems in the Mac version (can't authorize)

Not sure whether this impacts or not packaging frogr for fedora, but just
telling you in case you prefer to wait for the new bugfixing release.

Thanks and sorry for the mess,
Mario

-- 
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 731683] New: Review Request: pyppd - PPD file compressor and generator for CUPS

2011-08-18 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: pyppd - PPD file compressor and generator for CUPS

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

   Summary: Review Request: pyppd - PPD file compressor and
generator for CUPS
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: twa...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://twaugh.fedorapeople.org/pyppd/pyppd.spec
SRPM URL: http://twaugh.fedorapeople.org/pyppd/pyppd-0.4.9-1.fc16.src.rpm
Description:
This program holds a compressed archive of PostScript Printer
Description files.  It can generate the PPD files on the fly for CUPS.

The point of this program is to be run at build time by printer driver packages
such as hplip or foomatic.

For packages that ship lots of static PPDs, pyppd can save huge amounts of disk
space.

For foomatic, which generates PPDs on the fly from XML files, pyppd can greatly
increase performance.

-- 
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   >