[Bug 782220] Review Request: dlm - cluster infrastructure for dlm (distributed lock manager)

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fabio Massimo Di Nitto fdini...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #17 from Fabio Massimo Di Nitto fdini...@redhat.com 2012-01-30 
03:02:57 EST ---
New Package SCM Request
===
Package Name: dlm
Short Description: dlm - cluster infrastructure for dlm (distributed lock
manager)
Owners: teigland fabbione
Branches: 
InitialCC:

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

[Bug 719908] Review Request: rubygem-multi_json - A gem to provide swappable JSON backends

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Vít Ondruch vondr...@redhat.com 2012-01-30 03:34:58 EST 
---
(In reply to comment #10)
 Did this make it into F16?

No. I am not pushing new packages into older releases without reasons. Do you
like it in 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 785639] Review Request: rubygem-multi_xml - A generic swappable back-end for XML parsing

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bohuslav Kabrda bkab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bkab...@redhat.com
 AssignedTo|nob...@fedoraproject.org|bkab...@redhat.com
   Flag||fedora-review?

--- Comment #1 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 03:31:10 
EST ---
I'll take this for a 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 772994] Review Request: globus-simple-ca - Globus Toolkit - Simple CA Utility

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2012-01-30 
03:53:49 EST ---
Updated package:

http://www.grid.tsl.uu.se/review/globus-simple-ca.spec
http://www.grid.tsl.uu.se/review/globus-simple-ca-3.0-2.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 772995] Review Request: globus-xioperf - XIO Performance Tool

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2012-01-30 
03:53:54 EST ---
Updated package:

http://www.grid.tsl.uu.se/review/globus-xioperf.spec
http://www.grid.tsl.uu.se/review/globus-xioperf-3.0-2.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 772987] Review Request: globus-gram-job-manager-condor - Globus Toolkit - Condor Job Manager Support

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2012-01-30 
03:53:36 EST ---
Updated package:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager-condor.spec
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-condor-1.0-2.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 772993] Review Request: globus-gram-audit - Globus Toolkit - GRAM Jobmanager Auditing

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2012-01-30 
03:53:46 EST ---
Updated package:

http://www.grid.tsl.uu.se/review/globus-gram-audit.spec
http://www.grid.tsl.uu.se/review/globus-gram-audit-3.1-2.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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2012-01-30 
03:53:40 EST ---
Updated package:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager-pbs.spec
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-pbs-1.1-2.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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2012-01-30 
03:53:33 EST ---
Updated package:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager-fork.spec
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-fork-1.0-2.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 785639] Review Request: rubygem-multi_xml - A generic swappable back-end for XML parsing

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 03:55:24 
EST ---
- Run the specs
-- Add BR: rubygem(rspec-core) and BR: rubygem(nokogiri)
-- Comment out/delete the first two lines in spec/helper.rb to avoid the
unnecessary simplecov dependency.
-- Run the tests like rspec spec in %geminstdir - note that you will probably
need to add some load paths later with Ruby 1.9.3.
- Other than that, the package looks fine. I would maybe consider using
%exclude in %files section rather than removing files with rm in %install,
but this is definitely not a blocker. So please run the rspecs and then this
package can be approved. (When building for rawhide, please save yourself some
work and build it right into the f17-ruby target, it will be merged into
rawhide anyway.)

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

[Bug 785663] New: Review Request: rubygem-journey - Journey is a router

2012-01-30 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: rubygem-journey - Journey is a router

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

   Summary: Review Request: rubygem-journey - Journey is a router
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: vondr...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://people.redhat.com/vondruch/rubygem-journey.spec
SRPM URL:
http://people.redhat.com/vondruch/rubygem-journey-1.0.1-1.fc17.src.rpm
Description: Journey is a router. It routes requests.

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3745455

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

[Bug 785663] Review Request: rubygem-journey - Journey is a router

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bohuslav Kabrda bkab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bkab...@redhat.com
 AssignedTo|nob...@fedoraproject.org|bkab...@redhat.com
   Flag||fedora-review?

--- Comment #1 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 04:15:53 
EST ---
I'll take this for a 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 717920] Review Request: rubygem-hike - Find files in a set of paths

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Vít Ondruch vondr...@redhat.com 2012-01-30 04:34:18 EST 
---
(In reply to comment #2)
 - Please update to the latest version of gem (currently 1.2.1).

Done

 - The specfile you provided differs from the specfile in SRPM (just the two
 comments in %check, but please fix this).

Done

 - License is MIT, not GPLv2+ or Ruby.

Done

 - Please consider updating to the new standard of Rubygems packaging for
 Rawhide with Ruby 1.9.3 [1] (and please build the package with new Ruby, not
 the old one - use f17-ruby target in Koji).

Done

 - Otherwise the package looks good, but I'd like to see it once more after it
 gets updated to the latest version, then I'll gladly approve it.
 
 [1] https://fedoraproject.org/wiki/PackagingDrafts/Ruby

Spec URL: http://people.redhat.com/vondruch/rubygem-hike.spec
SRPM URL: http://people.redhat.com/vondruch/rubygem-hike-1.2.1-1.fc17.src.rpm

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3745471

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

[Bug 739367] Review Request: php-pecl-mysqlnd-qc - A query cache plugin for mysqlnd

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Remi Collet fed...@famillecollet.com 2012-01-30 04:35:55 
EST ---
Ok, I also have a working spec for fedora 17
https://raw.github.com/remicollet/remirepo/master/php-pecl-mysqlnd-qc/php-pecl-mysqlnd-qc-dev.spec

And
http://rpms.famillecollet.com/SRPMS/php-pecl-mysqlnd-qc-1.1.0-0.1.alpha.svn322926.remi.src.rpm

Version 1.0.1 still the target for fedora 16
version 1.1.0 will be the target for fedora 17

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

[Bug 785663] Review Request: rubygem-journey - Journey is a router

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bohuslav Kabrda bkab...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 04:41:13 
EST ---
- README.md should be present in the main package, as it contains licensing
information.
- Otherwise the package is OK (although, as FPC [1], I have a general feel of
distaste about the bundled javascript libraries - but since they are currently
allowed, it is not a blocker).

[1] https://fedorahosted.org/fpc/ticket/113


So after you fix the issue, this package is APPROVED.

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

[Bug 756780] Review Request: xosd - On-screen display library for X

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
   Fixed In Version||xosd-2.2.14-15.fc17
 Resolution||RAWHIDE
Last Closed||2012-01-30 04:49:21

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

[Bug 717920] Review Request: rubygem-hike - Find files in a set of paths

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bohuslav Kabrda bkab...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 04:49:03 
EST ---
The package looks good 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 784589] Review Request: ii - IRC IT, simple FIFO based IRC client

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Petr Šabata psab...@redhat.com 2012-01-30 04:54:37 EST ---
(In reply to comment #1)
 Please build verbosely by removing the @-sign from the compiler invocation.

Ok, I updated the package with
sed -i 's!@${CC}!${CC}!' Makefile

 Public domain is for query.sh?

Yes.

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

[Bug 785663] Review Request: rubygem-journey - Journey is a router

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #3 from Vít Ondruch vondr...@redhat.com 2012-01-30 04:55:47 EST 
---
(In reply to comment #2)
 - README.md should be present in the main package, as it contains licensing
 information.

Done

 - Otherwise the package is OK (although, as FPC [1], I have a general feel of
 distaste about the bundled javascript libraries - but since they are currently
 allowed, it is not a blocker).
 
 [1] https://fedorahosted.org/fpc/ticket/113
 
 
 So after you fix the issue, this package is APPROVED.


Thank you for your review!

New Package SCM Request
===
Package Name: rubygem-journey
Short Description: Journey is a router
Owners: vondruch
Branches: 
InitialCC:

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

[Bug 785681] New: Review Request: python-functest - Functional test framework

2012-01-30 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-functest - Functional test framework

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

   Summary: Review Request: python-functest - Functional test
framework
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kk...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://kklic.fedorapeople.org/python-functest.spec
SRPM URL: http://kklic.fedorapeople.org/python-functest-0.8.8-1.fc16.src.rpm
Description:
Functional test framework with no boiler plate code, setup/teardown
module hierarchies, and distributed result reporting.

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

[Bug 717920] Review Request: rubygem-hike - Find files in a set of paths

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Vít Ondruch vondr...@redhat.com 2012-01-30 05:36:25 EST 
---
Thank you for your review!



New Package SCM Request
===
Package Name: rubygem-hike
Short Description: Find files in a set of paths
Owners: vondruch
Branches: 
InitialCC:

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

[Bug 785681] Review Request: python-functest - Functional test framework

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Karel Klíč kk...@redhat.com 2012-01-30 05:44:58 EST ---
rpmlint python-functest-0.8.8-1.fc17.noarch.rpm

python-functest.noarch: W: spelling-error %description -l en_US teardown -
tear down, tear-down, downhearted
python-functest.noarch: W: no-documentation
python-functest.noarch: W: no-manual-page-for-binary functest
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

- http://en.wiktionary.org/wiki/teardown#English
- no-documentation: the upstream archive doesn't contain any documentation
- no-manual-page-for-binary: we should deal with it later


rpmlint python-functest-0.8.8-1.fc17.src.rpm

python-functest.src: W: spelling-error %description -l en_US teardown - tear
down, tear-down, downhearted
python-functest.src:7: W: macro-in-comment %{version}
python-functest.src:8: W: macro-in-comment %{version}
python-functest.src:8: W: macro-in-comment %{version}
python-functest.src: W: invalid-url Source0: python-functest-0.8.8.tar.xz
1 packages and 0 specfiles checked; 0 errors, 5 warnings.

- The %{version} macro is used intentionally in the comments.
- The project does not release tarballs.

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

[Bug 784596] Review Request: jabref - Graphical frontend to manage BibTeX bibliographical databases

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Mary Ellen Foster mefos...@gmail.com 2012-01-30 06:35:26 
EST ---
- Update after looking at Debian package:
  - Use separately packaged JPFCodeGen library
  - Add man page
  - Add file argument to Exec line in desktop file

http://mef.fedorapeople.org/packages/jabref/jabref-2.8-0.3.b.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 785694] New: Review Request: jpfcodegen - A tool for generating classes from JPF plug-ins

2012-01-30 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: jpfcodegen - A tool for generating classes from JPF 
plug-ins

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

   Summary: Review Request: jpfcodegen - A tool for generating
classes from JPF plug-ins
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mefos...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://mef.fedorapeople.org/packages/jabref/jpfcodegen.spec
SRPM URL:
http://mef.fedorapeople.org/packages/jabref/jpfcodegen-0.4-1.fc16.src.rpm

Description:
JPF Code Generator is a handy little tool that generates classes for
accessing the attributes and extensions of JPF plug-ins from plugin.xml
files. This has the advantage of providing a strongly typed access to the
plug-in and simplyfies working with plug-ins.

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

[Bug 785694] Review Request: jpfcodegen - A tool for generating classes from JPF plug-ins

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mary Ellen Foster mefos...@gmail.com changed:

   What|Removed |Added

 Blocks||784596

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

[Bug 784596] Review Request: jabref - Graphical frontend to manage BibTeX bibliographical databases

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mary Ellen Foster mefos...@gmail.com changed:

   What|Removed |Added

 Depends on||785694

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

[Bug 785694] Review Request: jpfcodegen - A tool for generating classes from JPF plug-ins

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mary Ellen Foster mefos...@gmail.com changed:

   What|Removed |Added

 Depends on||784592

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

[Bug 784592] Review Request: jpf - Java Plug-in Framework

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mary Ellen Foster mefos...@gmail.com changed:

   What|Removed |Added

 Blocks||785694

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

[Bug 784847] Review Request: aplpy - The Astronomical Plotting Library in Python

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Germán Racca gra...@gmail.com 2012-01-30 06:44:01 EST ---
Hi Golo:

Yes, I tried to build the package without the explicit requirement of
matplotlib, but it didn't work:

$ rpm -qp --requires aplpy-0.9.6-1.fc16.noarch.rpm 
numpy  
pyfits  
python(abi) = 2.7
pywcs  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(FileDigests) = 4.6.0-1
rpmlib(PartialHardlinkSets) = 4.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(PayloadIsXz) = 5.2-1

so there is no matplotlib in the dependencies of the package, that's why I had
to make an explicit requirement of matplotlib.

Thanks for taking 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 785702] Review Request: relaxngcc - RELAX NG Compiler Compiler

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

gil cattaneo punto...@libero.it 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 785702] New: Review Request: relaxngcc - RELAX NG Compiler Compiler

2012-01-30 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: relaxngcc - RELAX NG Compiler Compiler

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

   Summary: Review Request: relaxngcc - RELAX NG Compiler Compiler
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: punto...@libero.it
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://www.mediafire.com/file/aee8lw9blfag6h8/relaxngcc.spec
SRPM URL:
http://www.mediafire.com/file/sotqhr2fofd8a67/relaxngcc-1.12-1.fc16.src.rpm
Description: RelaxNGCC is a tool for generating Java source code from a
given RELAX NG grammar.

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

[Bug 785700] New: Review Request: relaxngcc - RELAX NG Compiler Compiler

2012-01-30 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: relaxngcc - RELAX NG Compiler Compiler

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

   Summary: Review Request: relaxngcc - RELAX NG Compiler Compiler
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: punto...@libero.it
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://www.mediafire.com/file/aee8lw9blfag6h8/relaxngcc.spec
SRPM URL:
http://www.mediafire.com/file/sotqhr2fofd8a67/relaxngcc-1.12-1.fc16.src.rpm
Description: RelaxNGCC is a tool for generating Java source code from a
given RELAX NG grammar.

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

[Bug 785701] New: Review Request: relaxngcc - RELAX NG Compiler Compiler

2012-01-30 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: relaxngcc - RELAX NG Compiler Compiler

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

   Summary: Review Request: relaxngcc - RELAX NG Compiler Compiler
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: punto...@libero.it
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://www.mediafire.com/file/aee8lw9blfag6h8/relaxngcc.spec
SRPM URL:
http://www.mediafire.com/file/sotqhr2fofd8a67/relaxngcc-1.12-1.fc16.src.rpm
Description: RelaxNGCC is a tool for generating Java source code from a
given RELAX NG grammar.

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

[Bug 736262] Review Request: proxyfuzz - man-in-the-middle non-deterministic network fuzzer

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Sklenar pskle...@redhat.com 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 736262] Review Request: proxyfuzz - man-in-the-middle non-deterministic network fuzzer

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Petr Sklenar pskle...@redhat.com 2012-01-30 07:15:31 EST 
---
New Package SCM Request
===
Package Name: proxyfuzz
Short Description: Man-in-the-middle non-deterministic network fuzzer
Owners: psklenar
Branches: f15 f16 el6
InitialCC:

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

[Bug 433679] Review Request: clustermon - cluster monitor component of conga

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Jan Pokorny jpoko...@redhat.com 2012-01-30 07:23:42 EST 
---
Ok, there's no reason to keep this bug opened (being irrelevant for a few
years already).  If nobody closes it, I will in 14 days.

BTW. clustermon package will be obsoleted shortly.

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

[Bug 785700] Review Request: relaxngcc - RELAX NG Compiler Compiler

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2012-01-30 07:30:31 
EST ---
*** Bug 785702 has been marked as a duplicate of this bug. ***

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

[Bug 785702] Review Request: relaxngcc - RELAX NG Compiler Compiler

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|NEW |CLOSED
 CC||jussi.leht...@iki.fi
 Resolution||DUPLICATE
Last Closed||2012-01-30 07:30:31

--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2012-01-30 07:30:31 
EST ---


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

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

[Bug 785700] Review Request: relaxngcc - RELAX NG Compiler Compiler

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2012-01-30 07:30:31 
EST ---
*** Bug 785702 has been marked as a duplicate of this bug. ***

--- Comment #2 from Jussi Lehtola jussi.leht...@iki.fi 2012-01-30 07:30:34 
EST ---
*** Bug 785701 has been marked as a duplicate of this bug. ***

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

[Bug 785701] Review Request: relaxngcc - RELAX NG Compiler Compiler

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|NEW |CLOSED
 CC||jussi.leht...@iki.fi
 Resolution||DUPLICATE
Last Closed||2012-01-30 07:30:33

--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2012-01-30 07:30:33 
EST ---


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

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

[Bug 738744] Review Request: rubygem-execjs - A package for the execjs Ruby gem

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Vít Ondruch vondr...@redhat.com 2012-01-30 07:35:13 EST 
---
I'll review it, however I am not 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 784657] Review Request: python-mozbase - the Mozilla suite of Python utilities

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Karel Klíč kk...@redhat.com 2012-01-30 07:37:45 EST ---
rpmlint python-mozprocess-0-0.1.gitb077641.fc17.noarch.rpm

python-mozprocess.noarch: W: spelling-error %description -l en_US subprocess -
sub process, sub-process, processors
python-mozprocess.noarch: W: spelling-error %description -l en_US Popen -
Pope, Open, P open
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


rpmlint python-mozinfo-0-0.1.gitb077641.fc17.noarch.rpm

python-mozinfo.noarch: W: spelling-error %description -l en_US os - OS, och,
so
python-mozinfo.noarch: W: no-manual-page-for-binary mozinfo
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


rpmlint python-mozbase-0-0.1.gitb077641.fc17.src.rpm

python-mozbase.src: W: invalid-url Source0:
mozilla-mozbase-mozilla-central-25-gb077641.zip
1 packages and 0 specfiles checked; 0 errors, 1 warnings.


rpmlint python-mozdevice-0-0.1.gitb077641.fc17.noarch.rpm

python-mozdevice.noarch: W: spelling-error %description -l en_US adb - dab,
adv, ad
1 packages and 0 specfiles checked; 0 errors, 1 warnings.


rpmlint python-mozprofile-0-0.1.gitb077641.fc17.noarch.rpm

python-mozprofile.noarch: W: spelling-error %description -l en_US addons - ad
dons, ad-dons, adorns
python-mozprofile.noarch: W: no-manual-page-for-binary mozprofile
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


rpmlint python-mozinstall-0-0.1.gitb077641.fc17.noarch.rpm

python-mozinstall.noarch: W: spelling-error %description -l en_US exe - ex,
exes, exec
python-mozinstall.noarch: W: spelling-error %description -l en_US dmg - mg,
deg, dig
python-mozinstall.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/mozinstall/mozinstall.py 0644L /usr/bin/env
python-mozinstall.noarch: W: no-manual-page-for-binary mozinstall
1 packages and 0 specfiles checked; 1 errors, 3 warnings.


rpmlint python-mozhttpd-0-0.1.gitb077641.fc17.noarch.rpm

python-mozhttpd.noarch: W: spelling-error Summary(en_US) webserver - web
server, web-server, observer
python-mozhttpd.noarch: W: spelling-error %description -l en_US webserver -
web server, web-server, observer
python-mozhttpd.noarch: W: no-manual-page-for-binary mozhttpd
1 packages and 0 specfiles checked; 0 errors, 3 warnings.


rpmlint python-mozbase-0-0.1.gitb077641.fc17.noarch.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.


rpmlint python-manifestdestiny-0-0.1.gitb077641.fc17.noarch.rpm

python-manifestdestiny.noarch: W: no-manual-page-for-binary manifestparser
1 packages and 0 specfiles checked; 0 errors, 1 warnings.



Spelling errors are false positives.
The missing manual pages should be solved later.

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

[Bug 785711] New: Review Request: rubygem-map - String/symbol indifferent ordered hash

2012-01-30 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: rubygem-map - String/symbol indifferent ordered hash

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

   Summary: Review Request: rubygem-map - String/symbol
indifferent ordered hash
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: bkab...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://bkabrda.fedorapeople.org/pkgs/map/rubygem-map.spec
SRPM URL:
http://bkabrda.fedorapeople.org/pkgs/map/rubygem-map-5.2.0-1.fc16.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3745841

Description: The awesome ruby container you've always wanted: a string/symbol
indifferent ordered hash that works in all rubies.

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

[Bug 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bohuslav Kabrda bkab...@redhat.com changed:

   What|Removed |Added

 CC||bkab...@redhat.com

--- Comment #1 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 07:42:52 
EST ---
I'll take this one.

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

[Bug 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bohuslav Kabrda bkab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|bkab...@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 738744] Review Request: rubygem-execjs - A package for the execjs Ruby gem

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Vít Ondruch vondr...@redhat.com 2012-01-30 07:52:18 EST 
---
* Please update the library to upstream version
  - It seems that there is available 1.3.0 version already.

* Duplicated files
  - README.md is duplicated in main package and -doc subpackage. This should
be avoided. Instead of including whole %{geminstdir} in the main package,
you should include just the LICENSE file there.

* Please consider excluding the cached .gem file
  - The file has no meaning in Fedora, it is just waste-load.

* Test suite
  - Although the test suite is not embedded in the gem, it is available
upstream. It would be nice if you could run it during build time following
the guidelines [2]

* Please consider updating the package for Ruby 1.9.3
  - Since we are in the process of rebuild for Ruby 1.9.3, it would be nice
if you can prepare this package according to the new guidelines [1]




[1] https://fedoraproject.org/wiki/PackagingDrafts/Ruby
[2]
https://fedoraproject.org/wiki/PackagingDrafts/Ruby#Test_suite_not_included_in_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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 07:56:36 
EST ---
- Please update to the latest upstream version (currently 1.2.0).
- Consider excluding cached gem. This is only used with gem pristine command,
which is irrelevant for RPM gems, as they should be restored by yum
reinstall.

* You probably haven't noticed, but we are moving to Ruby 1.9.3 in rawhide,
which involves some changes to the gem packaging - see [1] for new guidelines
draft. Some information can be found on Ruby-sig fedora mailing list [2] (you
should subscribe to that one, if you haven't already).
* Currently, we only build new packages for Ruby 1.9.3 in its special target
f17-ruby (again, all the info is is [2]).
* I suggest you update to the latest version, then we can sort out any
leftovers and I can also help you with the new-style packaging. The package
looks good, so I think it shouldn't be a problem.


[1] https://fedoraproject.org/wiki/PackagingDrafts/Ruby
[2] http://lists.fedoraproject.org/pipermail/ruby-sig/2012-January/000805.html

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

[Bug 785719] New: Review Request: python-wsgi-jsonrpc - Expose Python classes via JSON using WSGI

2012-01-30 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-wsgi-jsonrpc - Expose Python classes via JSON 
using WSGI

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

   Summary: Review Request: python-wsgi-jsonrpc - Expose Python
classes via JSON using WSGI
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kk...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://kklic.fedorapeople.org/python-wsgi-jsonrpc.spec
SRPM URL:
http://kklic.fedorapeople.org/python-wsgi-jsonrpc-0.2.9-1.fc16.src.rpm
Description:
A library for exposing methods of Python classes via JSON RPC using
WSGI.


rpmlint python-wsgi-jsonrpc-0.2.9-1.fc17.src.rpm

python-wsgi-jsonrpc.src:7: W: macro-in-comment %{version}
python-wsgi-jsonrpc.src:8: W: macro-in-comment %{version}
python-wsgi-jsonrpc.src:8: W: macro-in-comment %{version}
python-wsgi-jsonrpc.src: W: invalid-url Source0:
python-wsgi-jsonrpc-0.2.9.tar.xz
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

- The %{version} macro is used intentionally in the comments.
- The project does not release tarballs.

rpmlint python-wsgi-jsonrpc-0.2.9-1.fc17.noarch.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.

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

[Bug 717920] Review Request: rubygem-hike - Find files in a set of paths

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2012-01-30 08:24:25 EST 
---
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 782220] Review Request: dlm - cluster infrastructure for dlm (distributed lock manager)

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #18 from Jon Ciesla limburg...@gmail.com 2012-01-30 08:29:27 EST 
---
teigland is not a valid packager member.

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

[Bug 736262] Review Request: proxyfuzz - man-in-the-middle non-deterministic network fuzzer

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Jon Ciesla limburg...@gmail.com 2012-01-30 08:27:11 EST 
---
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 785016] Review Request: python-quantumclient - API and CLI for Openstack Quantum

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Cole Robinson crobi...@redhat.com 2012-01-30 08:27:31 EST 
---
Okay, package updated:

Spec URL: http://fedorapeople.org/~crobinso/reviews/python-quantumclient.spec
SRPM URL:
http://fedorapeople.org/~crobinso/reviews/python-quantumclient-2012.1-0.1.e3.fc16.src.rpm

Changes:
- Add python2-devel to BuildRequires (required for python packages)
- Drop rm tests (wasn't necc, they aren't installed anyways)
- Add quantum/__init__.py from upstream quantum tar.gz to source list and
install it, reference launchpad bug
https://bugs.launchpad.net/quantum/+bug/922440

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

[Bug 785663] Review Request: rubygem-journey - Journey is a router

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Jon Ciesla limburg...@gmail.com 2012-01-30 08:31:24 EST 
---
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 784910] Review Request: python-keystoneclient - library for accessing Openstack Keystones API

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Cole Robinson crobi...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #6 from Cole Robinson crobi...@redhat.com 2012-01-30 08:34:12 EST 
---
New Package SCM Request
===
Package Name: python-keystoneclient
Short Description: Python API and CLI for OpenStack Keystone
Owners: crobinso apevec
Branches:
InitialCC:

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

[Bug 785727] New: Review Request: ocaml-camlimages - OCaml image processing library

2012-01-30 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: ocaml-camlimages  - OCaml image processing library

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

   Summary: Review Request: ocaml-camlimages  - OCaml image
processing library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: br...@wolff.to
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://people.fedoraproject.org/~bruno/ocaml-camlimages.spec
SRPM URL:
http://people.fedoraproject.org/~bruno/ocaml-camlimages-4.0.1-1.fc17.src.rpm
Description: 
This is an image processing library, which provides some basic
functions of image processing and loading/saving various image file
formats. In addition the library can handle huge images that cannot be
(or can hardly be) stored into the memory (the library automatically
creates swap files and escapes them to reduce the memory usage).

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

[Bug 784910] Review Request: python-keystoneclient - library for accessing Openstack Keystones API

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Jon Ciesla limburg...@gmail.com 2012-01-30 08:37:09 EST 
---
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 782220] Review Request: dlm - cluster infrastructure for dlm (distributed lock manager)

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Dake sd...@redhat.com 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 785727] Review Request: ocaml-camlimages - OCaml image processing library

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bruno Wolff III br...@wolff.to changed:

   What|Removed |Added

 CC||br...@wolff.to

--- Comment #1 from Bruno Wolff III br...@wolff.to 2012-01-30 08:37:53 EST ---
Note that ocaml-camlimages was previously packaged for Fedora. The last
successful build was for F13. There have been significant changes in the build
process between the current and previous versions of the package since then.

I have done a scratch build at:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3745888

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

[Bug 782220] Review Request: dlm - cluster infrastructure for dlm (distributed lock manager)

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #19 from Steven Dake sd...@redhat.com 2012-01-30 08:46:56 EST ---
Jon,

I am sponsoring teigland through this bugzilla.  I had failed to add him to the
appropriate group.  This should be resolved now.  Apologies for the error.

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

[Bug 720857] Review Request: datalog - A Lightweight Deductive Database using Datalog

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #33 from John D. Ramsdell ramsd...@mitre.org 2012-01-30 08:51:04 
EST ---
I still cannot get koji working.  I reran fedora-packager-setup, but when I
asked Firefox to import the certificate, it ignored the file I gave it.  I
tried to run koji from the command line, and got a new error.

$ koji build --arch-override=i386 --scratch dist-f17 datalog-1.8-1.fc16.src.rpm 
Error: []
$ 

This error occurred immediately -- there was no timeout or wait involved.

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

[Bug 782220] Review Request: dlm - cluster infrastructure for dlm (distributed lock manager)

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #21 from Fabio Massimo Di Nitto fdini...@redhat.com 2012-01-30 
08:53:19 EST ---
resubmitting the SCM request...

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

[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Dake sd...@redhat.com changed:

   What|Removed |Added

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

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

[Bug 785711] Review Request: rubygem-map - String/symbol indifferent ordered hash

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Vít Ondruch vondr...@redhat.com 2012-01-30 08:51:46 EST 
---
* Spec file differences
  - The spec file you provides differs from the spec file in SRPM. The SRPM
version looks better IMO. Please make sure which one you are using.

* License
  - The license file says same as ruby's. However, the license between
Ruby 1.8 and Ruby 1.9, differs. Would you mind to clarify the license with
upstream?

Otherwise I have no other concerns = 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 784156] Review Request: uwsgi - Fast, self-healing, application container server

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Dake sd...@redhat.com changed:

   What|Removed |Added

 CC||sd...@redhat.com
 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 782220] Review Request: dlm - cluster infrastructure for dlm (distributed lock manager)

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fabio Massimo Di Nitto fdini...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #20 from Fabio Massimo Di Nitto fdini...@redhat.com 2012-01-30 
08:53:00 EST ---
New Package SCM Request
===
Package Name: dlm
Short Description: dlm - cluster infrastructure for dlm (distributed lock
manager)
Owners: teigland fabbione
Branches: 
InitialCC:

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

[Bug 785711] Review Request: rubygem-map - String/symbol indifferent ordered hash

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||vondr...@redhat.com
 AssignedTo|nob...@fedoraproject.org|vondr...@redhat.com
   Flag||fedora-review?

--- Comment #1 from Vít Ondruch vondr...@redhat.com 2012-01-30 08:51:27 EST 
---
I take it for a 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 772608] Review Request: ovirt-guest-agent - oVirt Guest Agent

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Dake sd...@redhat.com 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 772608] Review Request: ovirt-guest-agent - oVirt Guest Agent

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Dake sd...@redhat.com changed:

   What|Removed |Added

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

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

[Bug 717920] Review Request: rubygem-hike - Find files in a set of paths

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-30 09:08:50

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

[Bug 772608] Review Request: ovirt-guest-agent - oVirt Guest Agent

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Dake sd...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #10 from Steven Dake sd...@redhat.com 2012-01-30 09:06:17 EST ---
Gal,

I will sponsor you as a packager.  The first step is to review a Fedora
contributor's package.  Please execute the review process on this bugzilla:

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

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

[Bug 784156] Review Request: uwsgi - Fast, self-healing, application container server

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Dake sd...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

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

[Bug 785663] Review Request: rubygem-journey - Journey is a router

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-30 09:14: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 784156] Review Request: uwsgi - Fast, self-healing, application container server

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Steven Dake sd...@redhat.com 2012-01-30 09:11:05 EST ---
Jorge,

I will sponsor you as a packager after meeting certain criteria.  First you
must demonstrate that you are able to submit packages for review (this package
looks pretty good on first glance) and also that you are able to execute the
review process.

Please execute the review process on:
https://bugzilla.redhat.com/show_bug.cgi?id=772608

The checklist is here:
http://fedoraproject.org/wiki/Packaging:ReviewGuidelines

An example review is here:
https://bugzilla.redhat.com/show_bug.cgi?id=781831

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

[Bug 709328] Review Request: psi-plus - Jabber client based on Qt

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #56 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 
2012-01-30 09:22:15 EST ---
Yes, and why did not send patches back to upstream?

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

[Bug 709328] Review Request: psi-plus - Jabber client based on Qt

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #57 from Rex Dieter rdie...@math.unl.edu 2012-01-30 09:32:20 EST 
---
Those questions (and answers) will hopefully get answered in the fpc iris
bundling exception request,
https://fedorahosted.org/fpc/ticket/137

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

[Bug 785016] Review Request: python-quantumclient - API and CLI for Openstack Quantum

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bob Kukura rkuk...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #3 from Bob Kukura rkuk...@redhat.com 2012-01-30 09:50:18 EST ---
That fixed it! The package is ready.

-Bob

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

[Bug 785438] Review Request: php-horde-Horde-Url - Horde Url class

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Remi Collet fed...@famillecollet.com 2012-01-30 09:53:15 
EST ---
Same comment as others...

If you don't set Buildroot, you can also remove %clean

--- php-horde-Horde-Url.spec.old 2012-01-30 15:46:30.0 +0100
+++ php-horde-Horde-Url.spec 2012-01-30 15:51:21.0 +0100
@@ -12,13 +12,16 @@
 Source0:http://pear.horde.org/get/%{pear_name}-%{version}.tgz

 BuildArch:  noarch
-BuildRequires:  php-pear = 1:1.4.9-1.2
+BuildRequires:  php-pear(PEAR) = 1.7.0
+BuildRequires:  php-channel(pear.horde.org)
+
 Requires(post): %{__pear}
 Requires(postun): %{__pear}
-Requires:   php-pear(pear.horde.org/Horde_Exception) = 2.0.0,
php-pear(PEAR) = 1.7.0
-Conflicts:  php-pear(pear.horde.org/Horde_Exception) = 2.0.0
-Provides:   php-pear(pear.horde.org/Horde_Url) = %{version}
-BuildRequires:  php-channel(pear.horde.org)
+Requires:   php-pear(pear.horde.org/Horde_Exception) = 1.0.0
+Requires:   php-pear(pear.horde.org/Horde_Exception)   2.0.0
+Requires: php-pear(PEAR) = 1.7.0
+Requires: php-common = 5.2.0
+Provides:   php-pear(pear.horde.org/%{pear_name}) = %{version}
 Requires:   php-channel(pear.horde.org)

 %description
@@ -51,10 +54,6 @@
 install -pm 644 %{name}.xml $RPM_BUILD_ROOT%{pear_xmldir}


-%clean
-rm -rf $RPM_BUILD_ROOT
-
-
 %post
 %{__pear} install --nodeps --soft --force --register-only \
 %{pear_xmldir}/%{name}.xml /dev/null || :
@@ -67,13 +66,9 @@


 %files
-
 %{pear_xmldir}/%{name}.xml
-# Expand this as needed to avoid owning dirs owned by our dependencies
-# and to avoid unowned dirs
-%{pear_phpdir}/Horde/Url/Exception.php
+%{pear_phpdir}/Horde/Url
 %{pear_phpdir}/Horde/Url.php
-
 %{pear_testdir}/Horde_Url

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

[Bug 784847] Review Request: aplpy - The Astronomical Plotting Library in Python

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Sergio Pascual sergio.pa...@gmail.com changed:

   What|Removed |Added

 CC||sergio.pa...@gmail.com

--- Comment #3 from Sergio Pascual sergio.pa...@gmail.com 2012-01-30 09:56:05 
EST ---
Hi, good to see you have packaged this, it was in my queue also.

Just a question about the name of the package. I happen to have packaged ATpy,
(astronomical tables in python), another package by the same developers. 

In that case I chose to use the project name instead of the tarball name. Would
you consider doing the same here for consistency? I remark this is only a
suggestion, as the guidelines allow you to use the tarball name or the project
name.


In any case, please add me as a co maintainer of package once it's reviewed

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

[Bug 785439] Review Request: php-horde-Horde-Util - Horde Utility Libraries

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 CC||fed...@famillecollet.com

--- Comment #1 from Remi Collet fed...@famillecollet.com 2012-01-30 10:00:43 
EST ---
--- php-horde-Horde-Util.spec.old 2012-01-30 15:54:02.0 +0100
+++ php-horde-Horde-Util.spec 2012-01-30 15:59:27.0 +0100
@@ -7,18 +7,22 @@
 Summary:Horde Utility Libraries

 Group:  Development/Libraries
-License:LGPLv2
+License:LGPLv2+
 URL:http://pear.horde.org
 Source0:http://pear.horde.org/get/%{pear_name}-%{version}.tgz

 BuildArch:  noarch
-BuildRequires:  php-pear = 1:1.4.9-1.2
+BuildRequires:  php-pear(PEAR) = 1.7.0
+BuildRequires:  php-channel(pear.horde.org)
+
 Requires(post): %{__pear}
 Requires(postun): %{__pear}
-Requires:   php-pear(pear.horde.org/Horde_Url) = 2.0.0, php-pear(PEAR) =
1.7.0
-Conflicts:  php-pear(pear.horde.org/Horde_Url) = 2.0.0
-Provides:   php-pear(pear.horde.org/Horde_Util) = %{version}
-BuildRequires:  php-channel(pear.horde.org)
+Requires:   php-pear(pear.horde.org/Horde_Url) = 1.0.0
+Requires:   php-pear(pear.horde.org/Horde_Url)   2.0.0
+Requires:   php-pear(PEAR) = 1.7.0
+Requires:   php-xml = 5.2.0
+Requires:   php-mbstring = 5.2.0
+Provides:   php-pear(pear.horde.org/%{pear_name}) = %{version}
 Requires:   php-channel(pear.horde.org)

 %description
@@ -67,12 +71,8 @@

 %files
 %doc %{pear_docdir}/%{pear_name}
-
-
 %{pear_xmldir}/%{name}.xml
-# Expand this as needed to avoid owning dirs owned by our dependencies
-# and to avoid unowned dirs
-%{pear_phpdir}/Horde/Array/Sort/Helper.php
+%{pear_phpdir}/Horde/Array
 %{pear_phpdir}/Horde/Array.php
 %{pear_phpdir}/Horde/Domhtml.php
 %{pear_phpdir}/Horde/String.php

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

[Bug 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Fotios Lindiakos fot...@redhat.com 2012-01-30 10:08:30 
EST ---
Thanks for picking this up! I'll look over those suggestions and update the
package to the newest version.

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

[Bug 738744] Review Request: rubygem-execjs - A package for the execjs Ruby gem

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Fotios Lindiakos fot...@redhat.com 2012-01-30 10:11:38 
EST ---
Thanks for the feedback, this was one of my first attempts at creating an RPM
from a gem. I'll update the library and in the process perform your other
suggestions as well. I'll update it to 1.9.3 as well.

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

[Bug 785767] New: Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl module

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl module

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

   Summary: Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl
module
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: berra...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec
URL:http://berrange.fedorapeople.org/review/perl-Gnome2-Vte/perl-Gnome2-Vte.spec
SRPM
URL:http://berrange.fedorapeople.org/review/perl-Gnome2-Vte/perl-Gnome2-Vte-0.09-1.fc16.src.rpm
Description:
Gnome2::Vte exposes the GNOME Virtual Terminal Emulator APIs to
Perl applications

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

[Bug 785767] Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl module

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Daniel Berrange berra...@redhat.com 2012-01-30 10:19:36 
EST ---
I did a F17 scratch build on x86_64

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


There are the following rpmlint errors to be expected

$ rpmlint perl-Gnome2-Vte-0.09-1.fc16.x86_64.rpm 
perl-Gnome2-Vte.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/perl5/vendor_perl/Gnome2/Vte/Install/vte2perl-version.h
perl-Gnome2-Vte.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/perl5/vendor_perl/Gnome2/Vte/Install/vte2perl.h
perl-Gnome2-Vte.x86_64: E: incorrect-fsf-address
/usr/share/doc/perl-Gnome2-Vte-0.09/LICENSE
perl-Gnome2-Vte.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/perl5/vendor_perl/Gnome2/Vte/Install/vte2perl-autogen.h
1 packages and 0 specfiles checked; 1 errors, 3 warnings.

For the FSF address problem I filed upstream bug

   https://rt.cpan.org/Ticket/Display.html?id=74533

As per the Perl packaging guidelines the devel file warnings should be ignored

  https://fedoraproject.org/wiki/Packaging:Perl#.h_files_in_module_packages


$ rpmlint  perl-Gnome2-Vte-0.09-1.fc16.src.rpm
perl-Gnome2-Vte.src:24: W: comparison-operator-in-deptoken font(:lang=en)
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

This warning is to be expected, when following the Perl packaging guidelines

   https://fedoraproject.org/wiki/Perl/Tips#Tests_require_X11_server


The Patch1 file has an upstream bug report too

  https://rt.cpan.org/Ticket/Display.html?id=74532

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

[Bug 785767] Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl module

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

 CC||rjo...@redhat.com
 AssignedTo|nob...@fedoraproject.org|rjo...@redhat.com
   Flag||fedora-review?

--- Comment #2 from Richard W.M. Jones rjo...@redhat.com 2012-01-30 10:20:55 
EST ---
Taking this for review since I use this package.

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

[Bug 783716] Review Request:gmail-notify-ext - Gmail checker Gnome Online Accounts based

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Stephen Gordon sgor...@redhat.com 2012-01-30 10:23:29 EST 
---
The license string issue has been corrected :). I can also see you also added
an expanded description which is great, unfortunately though the lines in it
are  80 characters long, please ensure you put line breaks in the description
field at or before 80 characters.

Otherwise the issues I raised in c#1 still stand:

- Use the %find_lang macro for the locale files:
  http://fedoraproject.org/wiki/Packaging/Guidelines#Handling_Locale_Files

- This looks like a noarch package but is not marked as such:
  http://fedoraproject.org/wiki/Common_Rpmlint_issues#no-binary

The following additional issues appear to have been introduced because upstream
have added a Greek translation but it is using the wrong locale (el instead of
gr):

gmail-notify-ext.x86_64: E: incorrect-locale-el
/usr/share/locale/gr/LC_MESSAGES
gmail-notify-ext.x86_64: E: invalid-lc-messages-dir
/usr/share/locale/gr/LC_MESSAGES/gmail_notify.mo

See http://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-locale-el

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

[Bug 785767] Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl module

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Richard W.M. Jones rjo...@redhat.com 2012-01-30 10:22:45 
EST ---
Koji scratch build:

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

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

[Bug 722204] Review Request: calligra - An integrated office suite

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Rex Dieter rdie...@math.unl.edu 2012-01-30 10:31:47 EST 
---
http://rdieter.fedorapeople.org/rpms/calligra/calligra.spec
http://rdieter.fedorapeople.org/rpms/calligra/calligra-2.3.87-1.fc16.src.rpm

%changelog
* Sat Jan 28 2012 Rex Dieter rdie...@fedoraproject.org 2.3.87-1
- 2.3.87

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

[Bug 782220] Review Request: dlm - cluster infrastructure for dlm (distributed lock manager)

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #22 from Jon Ciesla limburg...@gmail.com 2012-01-30 10:37:31 EST 
---
Git done (by process-git-requests).

Excellent, 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 785767] Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl module

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |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 785767] Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl module

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Richard W.M. Jones rjo...@redhat.com 2012-01-30 10:38:31 
EST ---
+ rpmlint output

rpmlint output:

perl-Gnome2-Vte.src:24: W: comparison-operator-in-deptoken font(:lang=en)

Apparently this syntax is correct, according to the packaging
guidelines.  I've not seen it before.

perl-Gnome2-Vte.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/perl5/vendor_perl/Gnome2/Vte/Install/vte2perl-version.h
perl-Gnome2-Vte.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/perl5/vendor_perl/Gnome2/Vte/Install/vte2perl.h
perl-Gnome2-Vte.x86_64: E: incorrect-fsf-address
/usr/share/doc/perl-Gnome2-Vte-0.09/LICENSE
perl-Gnome2-Vte.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/perl5/vendor_perl/Gnome2/Vte/Install/vte2perl-autogen.h

These ones have been addressed in comment 1.

+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
+ license matches the actual package license

The license file included in the package is the LGPLv2+, which
is what the spec says.  The source files don't reference any
license at all.

+ %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
+ BuildRequires list all build dependencies

Yes, built in Koji.

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
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
n/a header files should be in -devel

This is OK, see comment 1.

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
n/a packages must not own files or directories owned by other packages
+ filenames must be valid UTF-8
+ use %global instead of %define

Optional:

+ 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

I have tested this with Tech Talk PSE.

n/a scriptlets should be sane
n/a pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
/usr/sbin

===
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 722205] Review Request: calligra-l10n - Language files for calligra

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Rex Dieter rdie...@math.unl.edu 2012-01-30 10:43:23 EST 
---
http://rdieter.fedorapeople.org/rpms/calligra/calligra-l10n.spec
http://rdieter.fedorapeople.org/rpms/calligra/calligra-l10n-2.3.87-1.fc16.src.rpm

%changelog
* Mon Jan 30 2012 Rex Dieter rdie...@fedoraproject.org 2.3.87-1
- 2.3.87

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

[Bug 785785] New: Review Request: pkgdiff - A tool for analyzing changes in Linux software packages

2012-01-30 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: pkgdiff - A tool for analyzing changes in Linux 
software packages

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

   Summary: Review Request: pkgdiff - A tool for analyzing changes
in Linux software packages
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: hobbes1...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://hobbes1069.fedorapeople.org/pkgdiff/pkgdiff.spec
SRPM URL: http://hobbes1069.fedorapeople.org/pkgdiff/pkgdiff-1.0-1.fc16.src.rpm
Description:
Package Changes Analyzer (pkgdiff) is a tool for analyzing changes
in Linux software packages (RPM, DEB, TAR.GZ, etc). The tool is
intended for Linux maintainers who are interested in ensuring
compatibility of old and new versions of packages.

$ rpmlint SRPMS/pkgdiff-1.0-1.fc16.src.rpm
RPMS/noarch/pkgdiff-1.0-1.fc16.noarch.rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

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

[Bug 709328] Review Request: psi-plus - Jabber client based on Qt

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #58 from Ivan Romanov dr...@land.ru 2012-01-30 10:57:43 EST ---
(In reply to comment #56)
 Yes, and why did not send patches back to upstream?

Because ask Rion why he didn't send patches to himself. ;)

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

[Bug 785767] Review Request: perl-Gnome2-Vte - Gnome2::Vte Perl module

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Daniel Berrange berra...@redhat.com changed:

   What|Removed |Added

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

--- Comment #5 from Daniel Berrange berra...@redhat.com 2012-01-30 10:56:12 
EST ---
New Package SCM Request
===
Package Name: perl-Gnome2-Vte
Short Description: Perl interface to Gnome2 VTE widget
Owners: berrange
Branches: 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 785436] Review Request: php-horde-Horde-Exception - Horde Exception Handler

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Nick Bebout n...@fedoraproject.org 2012-01-30 11:04:33 
EST ---
Updated

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

[Bug 784847] Review Request: aplpy - The Astronomical Plotting Library in Python

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Germán Racca gra...@gmail.com 2012-01-30 11:01:40 EST ---
(In reply to comment #3)
 Hi, good to see you have packaged this, it was in my queue also.
 
 Just a question about the name of the package. I happen to have packaged ATpy,
 (astronomical tables in python), another package by the same developers. 
 
 In that case I chose to use the project name instead of the tarball name. 
 Would
 you consider doing the same here for consistency? I remark this is only a
 suggestion, as the guidelines allow you to use the tarball name or the project
 name.
 
 
 In any case, please add me as a co maintainer of package once it's reviewed

Hi Sergio:

You mean APLpy instead of aplpy? Any problem for me, just waiting for the
opinion of the reviewer. If it's all right, I can make the change right now.

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

[Bug 785439] Review Request: php-horde-Horde-Util - Horde Utility Libraries

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Nick Bebout n...@fedoraproject.org 2012-01-30 11:32:16 
EST ---
Updated

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

[Bug 770283] Review Request: uptimed - A daemon to record and keep track of system up times

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #23 from Ivan Romanov dr...@land.ru 2012-01-30 11:35:17 EST ---
Okey. Now I can start a formal 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 785432] Review Request: php-horde-Horde-Translation - Horde translation library

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Nick Bebout n...@fedoraproject.org 2012-01-30 11:37:11 
EST ---
Updated.

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

[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

2012-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #30 from Fedora Update System upda...@fedoraproject.org 
2012-01-30 11:46:59 EST ---
srm-ifce-1.12.2-6.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/srm-ifce-1.12.2-6.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

  1   2   >