[Bug 697829] Review Request: libpipeline - A pipeline manipulation library

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Ivana Varekova varek...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Ivana Varekova varek...@redhat.com 2011-04-20 02:11:03 
EDT ---
New Package SCM Request
===
Package Name: libpipeline
Short Description: A pipeline manipulation library
Owners: varekova
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 689477] Review Request: perl-Task-Kensho-Testing - Task::Kensho::Testing Perl module

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-04-20 02:19:23

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

[Bug 697778] Review Request: rubygem-formtastic - A Rails form builder plugin/gem with semantically rich and accessible markup

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Parag AN(पराग) panem...@gmail.com 2011-04-20 02:23:01 EDT 
---
Review:
1) koji build-http://koji.fedoraproject.org/koji/taskinfo?taskID=3012790
2) rpmlint output is
rubygem-formtastic.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/formtastic-1.2.3/ri/Formtastic/SemanticFormBuilder/options_require_validation%3f-i.yaml
%3f
rubygem-formtastic.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/formtastic-1.2.3/ri/Formtastic/SemanticFormBuilder/detect_label_and_value_method%21-i.yaml
%21
rubygem-formtastic.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/formtastic-1.2.3/ri/Formtastic/Util/rails3%3f-i.yaml
%3f
rubygem-formtastic.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/formtastic-1.2.3/ri/Formtastic/SemanticFormBuilder/has_errors%3f-i.yaml
%3f
rubygem-formtastic.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/formtastic-1.2.3/ri/Formtastic/SemanticFormBuilder/is_file%3f-i.yaml
%3f
rubygem-formtastic.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/formtastic-1.2.3/ri/Formtastic/SemanticFormBuilder/render_inline_errors%3f-i.yaml
%3f

== But as upstream has introduced naming for above files we can ignore this.
3) upstream source verified as
b1e67ba4cd4574a307a868af585a9243a3a8df54  formtastic-1.2.3.gem
b1e67ba4cd4574a307a868af585a9243a3a8df54  ../SOURCES/formtastic-1.2.3.gem

suggestions
1) I think you can drop the gem word from summary so it will be 
A Rails form builder plugin with semantically rich and accessible markup

Description can be added as
Formtastic is a Rails FormBuilder DSL (with some other goodies) to make it far
easier to create beautiful, semantically rich, syntactically awesome, readily
stylable and wonderfully accessible HTML forms in your Rails applications.

2)Generally we don't want to add versions to Requires also so that yum should
always pull latest dependencies. Please drop the versions.

3) fix the duplicate lines 
Requires: rubygem(i18n) = 0.4
Requires: rubygem(i18n)  1


4) I think you can also add -doc subpackage

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

[Bug 690728] Review Request: Nitrate - A test case management system written in Django

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Yuguang Wang yuw...@redhat.com 2011-04-20 02:29:59 EDT 
---
hi, any updates on this?
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 693425] Review Request: openerp - OpenERP business application

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #32 from Panos Christeas x...@hellug.gr 2011-04-20 02:54:26 EDT 
---
pyftpdlib: agreed, although theoretically our changes could be applied upstream
w/o breaking the others (I tried to code them carefully). However, they are
against previous ver., so we'd need to work merging them.

trml2pdf: an exception should be granted. Saw the same link as you did, cannot
locate where these forks keep repositories (tried some combinations of their
main project in the url). Ideally, we should be able to publish a revised
version of /our/ lib[1] that will cover the satchmo or whatever other project.

the OEP License: I think we're not really using it. We are at AGPLv3 and thus I
could ask to remove that text file from upstream to clear the situation.


[1] one thing I know for sure, is that we have a strict replacement of python's
'eval()' for security reasons. This binds the trml2pdf to our code so far,
non-trivial to break.

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


[Bug 697778] Review Request: rubygem-formtastic - A Rails form builder plugin/gem with semantically rich and accessible markup

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Daiki Ueno du...@redhat.com 2011-04-20 03:33:18 EDT ---
Thanks, updated.

Spec URL:
http://ueno.fedorapeople.org/rubygem-formtastic/rubygem-formtastic.spec
SRPM URL:
http://ueno.fedorapeople.org/rubygem-formtastic/rubygem-formtastic-1.2.3-3.fc15.src.rpm

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

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


[Bug 693425] Review Request: openerp - OpenERP business application

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #33 from Alec Leamas leamas.a...@gmail.com 2011-04-20 03:57:42 
EDT ---
OEP License: It is used, 'grep -r  OpenERP Public License .' reveals some
files which needs to be edited/relicensed(?)

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


[Bug 694099] Review Request: perl-Pod-MinimumVersion - Perl version for POD directives used

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #3 from Petr Sabata psab...@redhat.com 2011-04-20 04:39:00 EDT ---
Seems you're right.
Approving.

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


[Bug 697778] Review Request: rubygem-formtastic - A Rails form builder plugin/gem with semantically rich and accessible markup

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Daiki Ueno du...@redhat.com 2011-04-20 04:45:12 EDT ---
Fixed some mistakes.

Spec URL:
http://ueno.fedorapeople.org/rubygem-formtastic/rubygem-formtastic.spec
SRPM URL:
http://ueno.fedorapeople.org/rubygem-formtastic/rubygem-formtastic-1.2.3-4.fc15.src.rpm

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

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


[Bug 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
 AssignedTo|nob...@fedoraproject.org|psab...@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 693425] Review Request: openerp - OpenERP business application

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #34 from Panos Christeas x...@hellug.gr 2011-04-20 05:01:56 EDT 
---
My grep reveals these 2 places:
addons/thunderbird/plugin/...
addons/wiki/web/...

For the second, I have not yet asked you to package the web-interface, which
means we can surely leave it out. For the first one, we can discuss.

You see, the point is that both these parts have originally be developed by
partners of OpenERP and have been relicensed to our open-source terms. I will
need to check with them if GPL can be applied.

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


[Bug 697778] Review Request: rubygem-formtastic - A Rails form builder plugin/gem with semantically rich and accessible markup

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Parag AN(पराग) panem...@gmail.com 2011-04-20 05:08:19 EDT 
---
Thanks for your updates.

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 697779] Review Request: rubygem-file-tail - File::Tail for Ruby

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Peng Wu p...@redhat.com 2011-04-20 05:08:54 EDT ---
Here are new spec and rpm:
Spec URL: http://pwu.fedorapeople.org/rubygem-file-tail/rubygem-file-tail.spec
SRPM URL:
http://pwu.fedorapeople.org/rubygem-file-tail/rubygem-file-tail-1.0.5-2.fc14.src.rpm

Please review it.

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


[Bug 697139] Review Request: apache-commons-email - Apache Commons Email Package

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Spike spikefed...@gmail.com 2011-04-20 05:12:06 EDT ---
Thanks for reviewing. Glad to be back in the team :)

(In reply to comment #2)
 [x]  Rpmlint output:
 apache-commons-email.noarch: W: non-conffile-in-etc
 /etc/maven/fragments/apache-commons-email
 apache-commons-email.src: W: strange-permission commons-email-1.2-src.tar.gz
 0400L
 3 packages and 0 specfiles checked; 0 errors, 2 warnings.
 
 Those permissions are indeed a bit weird, just make sure you fix them up 
 before
 final upload

No idea, where that came from. Fixed in srpm.


New Package SCM Request
===
Package Name: apache-commons-email
Short Description: Apache Commons Email Package
Owners: spike
Branches: F-15
InitialCC: java-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 697778] Review Request: rubygem-formtastic - A Rails form builder plugin with semantically rich and accessible markup

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Daiki Ueno du...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |rubygem-formtastic - A  |rubygem-formtastic - A
   |Rails form builder  |Rails form builder plugin
   |plugin/gem with |with semantically rich and
   |semantically rich and   |accessible markup
   |accessible markup   |
   Flag||fedora-cvs?

--- Comment #6 from Daiki Ueno du...@redhat.com 2011-04-20 05:15:47 EDT ---
Thanks for the review, Parag.

New Package SCM Request
===
Package Name: rubygem-formtastic
Short Description: A Rails form builder plugin with semantically rich and
accessible markup
Owners: ueno
Branches: f15 f14 f13
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 697401] Review Request: perl-Ouch - Exceptions that don't hurt

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Petr Sabata psab...@redhat.com 2011-04-20 05:22:24 EDT ---
Thanks for the review and the repo!

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


[Bug 697401] Review Request: perl-Ouch - Exceptions that don't hurt

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-04-20 
05:25:30 EDT ---
perl-Ouch-0.0300-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/perl-Ouch-0.0300-1.fc14

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


[Bug 697401] Review Request: perl-Ouch - Exceptions that don't hurt

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 640455] Review Request: python-pyro - Pyro is short for PYthon Remote Objects.

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

David Hannequin david.hanneq...@gmail.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 697401] Review Request: perl-Ouch - Exceptions that don't hurt

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-04-20 
05:26:01 EDT ---
perl-Ouch-0.0300-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/perl-Ouch-0.0300-1.fc15

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


[Bug 697780] Review Request: rubygem-daemon_controller - A library for implementing daemon management capabilities

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Peng Wu p...@redhat.com 2011-04-20 05:34:22 EDT ---
Here are new spec and rpm:
Spec URL:
http://pwu.fedorapeople.org/rubygem-daemon_controller/rubygem-daemon_controller.spec
SRPM URL:
http://pwu.fedorapeople.org/rubygem-daemon_controller/rubygem-daemon_controller-0.2.5-2.fc14.src.rpm

Please review it.

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


[Bug 640455] Review Request: python-pyro - Pyro is short for PYthon Remote Objects.

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #21 from David Hannequin david.hanneq...@gmail.com 2011-04-20 
05:58:39 EDT ---
Sorry for my mistake.

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


[Bug 640455] Review Request: python-pyro - Pyro is short for PYthon Remote Objects.

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #20 from David Hannequin david.hanneq...@gmail.com 2011-04-20 
05:58:01 EDT ---
New Package SCM Request
===
Package Name: python-pyro
Short Description: Pyro stands for PYthon Remote Objects.
Owners: hvad
Branches: f14 f15 
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 698139] New: Review Request: ioblazer - multi-platform storage stack micro-benchmark

2011-04-20 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: ioblazer - multi-platform storage stack micro-benchmark

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

   Summary: Review Request: ioblazer - multi-platform storage
stack micro-benchmark
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rubi...@frameos.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL: http://rbel.frameos.org/review/ioblazer.spec
SRPM URL: http://rbel.frameos.org/review/ioblazer-1.01-2.el6.src.rpm
Description: 

From VMWare Labs website:

IOBlazer is a multi-platform storage stack micro-benchmark. IOBlazer runs on
Linux, Windows and OSX and it is capable of generating a highly customizable
workload. Parameters like IO size and pattern, burstiness (number of
outstanding IOs), burst interarrival time, read vs. write mix, buffered vs.
direct IO, etc., can be configured independently. IOBlazer is also capable of
playing back VSCSI traces captured using vscsiStats. The performance metrics
reported are throughput (in terms of both IOPS and bytes/s) and IO latency.

Please review.

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 697779] Review Request: rubygem-file-tail - File::Tail for Ruby

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 CC||vondr...@redhat.com

--- Comment #5 from Vít Ondruch vondr...@redhat.com 2011-04-20 06:18:50 EDT 
---
Could you please execute the test suite during build?

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

[Bug 693425] Review Request: openerp - OpenERP business application

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #35 from Alec Leamas leamas.a...@gmail.com 2011-04-20 06:17:08 
EDT ---
So lets skip addons/wiki/web/..., my bad.

Relicensing the thunderbird plugin is the way to go, it's hopefully easier than
trying to get the obsolete OEP license approved (if it's at all possible).

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


[Bug 697778] Review Request: rubygem-formtastic - A Rails form builder plugin with semantically rich and accessible markup

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 CC||vondr...@redhat.com

--- Comment #7 from Vít Ondruch vondr...@redhat.com 2011-04-20 06:17:17 EDT 
---
It would be nice if you could execute test suite during build.

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

[Bug 697801] Review Request: rubygem-recaptcha - Helpers for reCAPTCHA API

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 CC||vondr...@redhat.com

--- Comment #3 from Vít Ondruch vondr...@redhat.com 2011-04-20 06:23:24 EDT 
---
Could you please execute test suite during build?

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

[Bug 697780] Review Request: rubygem-daemon_controller - A library for implementing daemon management capabilities

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 CC||vondr...@redhat.com

--- Comment #4 from Vít Ondruch vondr...@redhat.com 2011-04-20 06:20:27 EDT 
---
Could you please execute test suite during build?

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

[Bug 693425] Review Request: openerp - OpenERP business application

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #36 from Alec Leamas leamas.a...@gmail.com 2011-04-20 06:26:49 
EDT ---
With all issues in comment #31 resolved, we are approaching another other big
issue: how to handle the server addons: their deps and in some cases also
licenses.

To decouple the thunderbird addon license, one obvious option is to package it
in in a sub-package. This seems reasonable from many aspects for me.

However, we also have all other addons and their deps. I know from experience
that using certain addons will break due to missing deps. So, we need to
somehow fix this. One could of course try to add all deps for all addons, but
this would pull in just to much for simple installations(?)

OTOH, creating a sub-package for each addon (150+) seems just not feasible. Or
is it? Seems that tryton (an OpenERP fork) is packaged this way in rawhide, 60+
packages.

Would it be possible to somehow group the addons based on functionality, deps
etc., into a more manageable set of sub-packages?

Basically, I have no idea of the best way to handle 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 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Remi Collet fed...@famillecollet.com 2011-04-20 06:33:58 
EDT ---
Update to 1.0.2 (in fact, no change in code, just to have latest version)

https://github.com/remicollet/remirepo/commit/322b10d5c7e833506c14fcbdb0c9ab31bd0012f0

SRPM URL: http://rpms.famillecollet.com/SRPMS/php-pecl-rrd-1.0.2-1.remi.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 644711] Review Request: diaser - disk based backup volume accumulator, replication and management system

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Damian d...@interlinux.co.uk changed:

   What|Removed |Added

   Flag|needinfo?   |

--- Comment #6 from Damian d...@interlinux.co.uk 2011-04-20 06:43:38 EDT ---
I see - I'll find another package to 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 693425] Review Request: openerp - OpenERP business application

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #37 from Panos Christeas x...@hellug.gr 2011-04-20 07:03:47 EDT 
---
http://members.hellug.gr/xrg/Mandriva/2010.1/RPMS/noarch/

If you see there, I had been packaging one rpm per addon, for those non-trivial
addons.
The way to do that is to have a /generated/ spec file for the addons (one, big
spec for all of them), but I guess that's way off the practices of Fedora. I
can also tell you that my colleagues at OpenERP want all of the addons packaged
into the server rpm.
.. which means we need to discuss more.

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


[Bug 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Petr Sabata psab...@redhat.com 2011-04-20 07:24:06 EDT ---
Package doesn't build on F-14 due to systemtap. I haven't looked into this yet.
I suppose this package is intented for F-15 and newer.

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


[Bug 694099] Review Request: perl-Pod-MinimumVersion - Perl version for POD directives used

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Petr Pisar ppi...@redhat.com 2011-04-20 07:25:32 EDT ---
New Package SCM Request
===
Package Name: perl-Pod-MinimumVersion 
Short Description: Perl version for POD directives used
Owners: ppisar mmaslano psabata
Branches: 
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 694099] Review Request: perl-Pod-MinimumVersion - Perl version for POD directives used

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Pisar ppi...@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 698139] Review Request: ioblazer - multi-platform storage stack micro-benchmark

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Sergio Rubio rubi...@frameos.org 2011-04-20 07:32:54 EDT 
---
First Fedora Package, I need 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 697801] Review Request: rubygem-recaptcha - Helpers for reCAPTCHA API

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Akira TAGOH ta...@redhat.com 2011-04-20 07:31:37 EDT ---
Updated. no bumped version.

(In reply to comment #1)
 4) I think following can be skipped from -doc subpackage as its already
 installed by main package
 %doc %{geminstdir}/[A-Z]*

just concerned that rpmlint complains no doc for -doc. guess it's not a big
deal.

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


[Bug 697801] Review Request: rubygem-recaptcha - Helpers for reCAPTCHA API

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Akira TAGOH ta...@redhat.com 2011-04-20 07:35:37 EDT ---
(In reply to comment #3)
 Could you please execute test suite during build?

I'm not sure if unit test code in every gem packages is well maintained though,
is it really worth doing with even adding rake and other deps to BR, which
isn't necessary for real build?

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


[Bug 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Marcela Mašláňová mmasl...@redhat.com 2011-04-20 07:49:58 
EDT ---
I plan it only for F-15+, because we do not have systemtap enabled in F-14.

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

[Bug 693425] Review Request: openerp - OpenERP business application

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #38 from Alec Leamas leamas.a...@gmail.com 2011-04-20 07:50:07 
EDT ---
...which means we need a second opinion, at least. Anyone, out there?

My thoughts:

One spec file w any number of subpackages is fine w Fedora from a formal point
of view.

Since both tryton/Fedora and OpenERP/Mandriva is walking the one
addon/subpackage path, why not stick to this approach (even if it means 150+
packages)?

Did you have any support for finding out subpackage/addons deps in the script
generating the spec file?

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


[Bug 640455] Review Request: python-pyro - Pyro is short for PYthon Remote Objects.

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #22 from David Hannequin david.hanneq...@gmail.com 2011-04-20 
07:51:00 EDT ---
Sorry for my mistake.

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


[Bug 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Petr Sabata psab...@redhat.com 2011-04-20 07:55:46 EDT ---
Package: perl-Data-Alias
Version: 1.12
Release: 1.fc14
Sources: Data-Alias-1.12.tar.gz
Patches: Data-Alias-systemtap.patch
--
Package failed to build locally!
Package successfully built in mock, fedora-rawhide-x86_64.
Package successfully built in mock, fedora-rawhide-i386.
Package successfully built in mock, fedora-15-x86_64.
Package successfully built in mock, fedora-15-i386.
Package failed to build in mock, fedora-14-x86_64.
Package failed to build in mock, fedora-14-i386.
Package successfully built in koji, dist-rawhide.

MUST items:
[  OK  ] Package does NOT include pre-built binaries or libraries
[  OK  ] Spec file is legible and written in American english
[  OK  ] Package successfully builds on at least one supported primary
architecture
[  --  ] All ExcludeArch tags valid, referencing proper bug reports
[  OK  ] Package obeys FHS (with _libexecdir and /srv exceptions)
[  OK  ] No errors reported by rpmlint
[ NOTE ] Changelog present and properly formatted
[  OK  ] Package does NOT include Packager, Vendor, Copyright or PreReq tags
[  OK  ] Source tags are working URLs and sources match upstream or justified
otherwise
[ NOTE ] Requires correct or justified otherwise
[ FAIL ] BuildRequires correct or justified otherwise
[  OK  ] All file names are in proper UTF-8 encoding
[  OK  ] All plain text failes are in proper UTF-8 encoding
[  --  ] Large documentation files are located in doc subpackage
[  OK  ] All documentation prefixed with %doc
[  OK  ] Documentation is NOT executable
[  OK  ] No files in %doc are needed at run-time
[  OK  ] Compiler flags honor Fedora defaults or are justified
[  OK  ] Package generates useful debuginfo packages
[  --  ] Header files are placed in devel subpackage
[  --  ] Unversioned shared libraries are placed in devel subpackage
[  --  ] Pkgconfig files are placed in devel subpackage
[  --  ] Full-versioned Requires of the base package in subpackages
[  --  ] Package calls ldconfig in post and postun sections for all
subpackages, if applicable
[  --  ] Static libraries are provided by static subpackage
[  OK  ] Package contains no static executables unless approved by FESCo
[  OK  ] Package does NOT bundle any system libraries
[  OK  ] RPath not used for anything besides internal libraries
[  --  ] All config files are marked noreplace or justified otherwise
[  OK  ] No config files are located under /usr
[  --  ] Package contains a SystemV-compatible initscript
[  --  ] A GUI application installs a proper desktop file
[  --  ] All desktop files are installed by desktop-file-install or justified
otherwise
[  OK  ] Package consistently uses macros
[  --  ] makeinstall macro is used only if make install DESTDIR=%{buildroot}
does NOT work
[  --  ] Macros in Summary and description are expandable at build-time
[  --  ] globals used in place of defines
[  --  ] Locales handled correctly -- package requires gettext and uses
find_lang, if applicable
[  --  ] Scriptlets are sane
[  OK  ] Package is not relocatable unless justified
[  OK  ] Package contains only acceptable code or content
[  OK  ] Package owns all the files and directories it creates, installs and/or
uses unless those are already owned by another package
[  OK  ] files sections do NOT contain duplicate files except for licenses
[  OK  ] All files sections use defattr or justify otherwise
[  OK  ] Package does NOT cause any conflicts
[  OK  ] Package does NOT contain kernel modules
[  OK  ] Package does NOT bundle fonts or other general purpose data
[ NOTE ] Final Requires and Provides are sane

SHOULD items:
[  OK  ] The Summary does NOT end with a period
[  OK  ] Package does NOT include BuildRoot tag, clean section or buildroot
removal in install section
[  OK  ] Package should preserve files timestamps
[  OK  ] Package does NOT explicitly BuildRequire bash, bzip2, coreutils, cpio,
diffutils, fedora-release, findutils, gawk, gcc, gcc-c++, grep, gzip, info,
make, patch, redhat-rpm-config, rpm-build, sed, shadow-utils, tar, unzip,
util-linux-ng, which or xz
[  OK  ] Description does NOT consist of lines longer than 80 characters
[  OK  ] Package uses parallel make
[  --  ] In case of a web application, package installs date into /usr/share
instead of /var/www
[  OK  ] All patches have a comment or an upstream bug link
[  --  ] Package installs manpages for all executables
[  OK  ] Package contains check section and all tests pass
[  ??  ] Package works as expected

NOTES:
--
1. A cosmetic issue: The second line of the last changelog entry should use two
spaces indentation.
2. Duplicate Requires: perl(Exporter), perl(DynaLoader) -- rpm finds them
automatically; remove them from Requires.
3. ...on the other hand, they should be BuildRequired.

-- 

[Bug 690854] Review Request: wimax - WiMAX Network Service for the Intel 2400m

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jiri Popelka jpope...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #3 from Jiri Popelka jpope...@redhat.com 2011-04-20 08:01:03 EDT 
---
OK. I see no other problems. 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 680657] Review Request: mpdas - An MPD audioscrobbling client

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-04-20 08:19:55 EDT ---
mpdas-0.3.0-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/mpdas-0.3.0-2.fc15

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


[Bug 697801] Review Request: rubygem-recaptcha - Helpers for reCAPTCHA API

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Vít Ondruch vondr...@redhat.com 2011-04-20 08:17:16 EDT 
---
(In reply to comment #4)
 Updated. no bumped version.
 
 (In reply to comment #1)
  4) I think following can be skipped from -doc subpackage as its already
  installed by main package
  %doc %{geminstdir}/[A-Z]*
 
 just concerned that rpmlint complains no doc for -doc. guess it's not a big
 deal.

There is LICENSE file which should be IMO in both packages. Otherwise there are
small discrepancy in opinions if files in -doc subpackage should or should not
be marked as doc.

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

[Bug 680657] Review Request: mpdas - An MPD audioscrobbling client

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 697976] Review Request: pyrenamer - A mass file renamer written in PyGTK

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 CC||rdie...@math.unl.edu

--- Comment #1 from Rex Dieter rdie...@math.unl.edu 2011-04-20 08:31:17 EDT 
---
one small pet-peave of mine, in Summary, I'd suggest dropping written in
PyGTK

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


[Bug 690282] Review Request: jogl - Java bindings for OpenGL

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #27 from Alexander Kurtakov akurt...@redhat.com 2011-04-20 
08:55:05 EDT ---
Jon, would you please provide a srpm with Jerry's patches?

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


[Bug 697976] Review Request: pyrenamer - A mass file renamer written in PyGTK

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Jon Ciesla l...@jcomserv.net 2011-04-20 08:51:57 EDT ---
Fixed.

SPEC: http://zanoni.jcomserv.net/fedora/pyrenamer/pyrenamer.spec
SRPM:
http://zanoni.jcomserv.net/fedora/pyrenamer/pyrenamer-0.6.0.1-6.fc14.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 561484] Review Request: jruby - Pure Java implementation of the Ruby interpreter

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Alexander Kurtakov akurt...@redhat.com 2011-04-20 
08:55:41 EDT ---
FWIW, JRuby 1.6.1 is out

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


[Bug 690854] Review Request: wimax - WiMAX Network Service for the Intel 2400m

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bill Nottingham nott...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Bill Nottingham nott...@redhat.com 2011-04-20 08:59:11 
EDT ---
New Package SCM Request
===
Package Name: wimax
Short Description: WiMAX Network Service for the Intel 2400m
Owners: notting
Branches: F-15
InitialCC: dcbw

Thanks for the 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Marcela Mašláňová mmasl...@redhat.com 2011-04-20 08:56:14 
EDT ---
(In reply to comment #5)
 NOTES:
 --
 1. A cosmetic issue: The second line of the last changelog entry should use 
 two
 spaces indentation.
Nitpicking but ok.
 2. Duplicate Requires: perl(Exporter), perl(DynaLoader) -- rpm finds them
 automatically; remove them from Requires.
Ok, will fix.
 3. ...on the other hand, they should be BuildRequired.
Ok.

Updated SRPM:
http://mmaslano.fedorapeople.org/review/perl-Data-Alias-1.12-1.fc14.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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Petr Sabata psab...@redhat.com 2011-04-20 09:13:09 EDT ---
--- Data-Alias/perl-Data-Alias.spec 2011-04-14 10:05:19.0 +0200
+++ Data-Alias2/perl-Data-Alias.spec2011-04-20 15:11:17.0 +0200
@@ -9,6 +9,8 @@
 # http://rt.cpan.org/Public/Bug/Display.html?id=67430
 Patch0: Data-Alias-systemtap.patch

+BuildRequires:  perl(DynaLoader)
+BuildRequires:  perl(Exporter)
 BuildRequires:  perl(ExtUtils::MakeMaker)
 BuildRequires:  perl(File::Spec)
 BuildRequires:  perl(Filter::Util::Call)
@@ -17,7 +19,6 @@
 BuildRequires:  perl(strict)
 BuildRequires:  perl(Test::More)
 BuildRequires:  perl(warnings)
-Requires:   perl(Exporter), perl(DynaLoader)
 Requires:   perl(:MODULE_COMPAT_%(eval `%{__perl} -V:version`; echo
$version))

 %{?perl_default_filter}
@@ -49,7 +50,7 @@

 %files
 %defattr(-,root,root,-)
-%doc Changes README
+%doc README Changes
 %{perl_vendorarch}/auto/*
 %{perl_vendorarch}/Data*
 %{_mandir}/man3/*
@@ -57,7 +58,7 @@
 %changelog
 * Thu Apr 14 2011 Marcela Mašláňová mmasl...@redhat.com 1.12-1
 - Specfile autogenerated by cpanspec 1.79 for new release. This package
- was resurected for 5.12.x
+  was resurected for 5.12.x

 * Mon Dec  7 2009 Stepan Kasal ska...@redhat.com - 1.07-6
 - rebuild against perl 5.10.1


--
Approving.

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

[Bug 495330] Review Request: perl-CSS-Minifier - Remove unnecessary whitespace from CSS files

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

 CC||xav...@bachelot.org

--- Comment #5 from Xavier Bachelot xav...@bachelot.org 2011-04-20 09:52:45 
EDT ---
Hi,
Would you be ok to branch this package for EPEL ?

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


[Bug 498188] Review Request: perl-JavaScript-Minifier - Perl extension for minifying JavaScript code

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

 CC||xav...@bachelot.org

--- Comment #4 from Xavier Bachelot xav...@bachelot.org 2011-04-20 09:52:32 
EDT ---
Hi,
Would you be ok to branch this package for EPEL ?

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


[Bug 498188] Review Request: perl-JavaScript-Minifier - Perl extension for minifying JavaScript code

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Marcela Mašláňová mmasl...@redhat.com 2011-04-20 09:59:20 
EDT ---
(In reply to comment #4)
 Hi,
 Would you be ok to branch this package for EPEL ?

Fine with me. Could you maintain it for EPEL?

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

[Bug 573910] Review Request: dcmtk - Offis DICOM Toolkit

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #30 from Mario Ceresa mrcer...@gmail.com 2011-04-20 10:14:01 EDT 
---
Peter: this solves the koji build error. man pages were missing because doxygen
was not listed in the BRs.

http://mrceresa.fedorapeople.org/dcmtk-3.6.0-4.fc14.src.rpm
http://mrceresa.fedorapeople.org/dcmtk.spec

Now it builds correctly:

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

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


[Bug 698251] New: Review Request: perl-B-Hooks-OP-PPAddr - Hook into opcode execution

2011-04-20 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-B-Hooks-OP-PPAddr - Hook into opcode execution

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

   Summary: Review Request: perl-B-Hooks-OP-PPAddr - Hook into
opcode execution
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


SPEC: http://mmaslano.fedorapeople.org/review/perl-B-Hooks-OP-PPAddr.spec
SRPM:
http://mmaslano.fedorapeople.org/review/perl-B-Hooks-OP-PPAddr-0.03-1.fc14.src.rpm
Description:
This module provides a c api for XS modules to hook into the execution of perl
opcodes.

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

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


[Bug 498188] Review Request: perl-JavaScript-Minifier - Perl extension for minifying JavaScript code

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Xavier Bachelot xav...@bachelot.org 2011-04-20 10:25:48 
EDT ---
Sure. Do you want to co-maintain ?

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


[Bug 698251] Review Request: perl-B-Hooks-OP-PPAddr - Hook into opcode execution

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Blocks||698264

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

[Bug 698264] Review Request: perl-TryCatch - First class try catch semantics for Perl, without source filters

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Depends on||698251

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

[Bug 698264] New: Review Request: perl-TryCatch - First class try catch semantics for Perl, without source filters

2011-04-20 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-TryCatch - First class try catch semantics for 
Perl, without source filters

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

   Summary: Review Request: perl-TryCatch - First class try catch
semantics for Perl, without source filters
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


SPEC: http://mmaslano.fedorapeople.org/review/perl-TryCatch.spec
SRPM:
http://mmaslano.fedorapeople.org/review/perl-TryCatch-1.003000-1.fc14.src.rpm
Description:
This module aims to provide a nicer syntax and method to catch errors in Perl,
similar to what is found in other languages (such as Java, Python or C++). The
standard method of using eval {}; if ($@) {} is often prone to subtle bugs,
primarily that its far too easy to stomp on the error in error handlers. And
also eval/if isn't the nicest idiom.

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


[Bug 498188] Review Request: perl-JavaScript-Minifier - Perl extension for minifying JavaScript code

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Marcela Mašláňová mmasl...@redhat.com 2011-04-20 10:31:13 
EDT ---
(In reply to comment #6)
 Sure. Do you want to co-maintain ?

It's not necessary.

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

[Bug 698139] Review Request: ioblazer - multi-platform storage stack micro-benchmark

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Sergio Rubio rubi...@frameos.org 2011-04-20 10:37:58 EDT 
---
SRPM URL updated: http://rbel.frameos.org/review/ioblazer-1.01-3.el6.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 498188] Review Request: perl-JavaScript-Minifier - Perl extension for minifying JavaScript code

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #8 from Xavier Bachelot xav...@bachelot.org 2011-04-20 10:38:18 
EDT ---
Package Change Request
==
Package Name: perl-JavaScript-Minifier
New Branches: el5 el6
Owners: xavierb

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


[Bug 573910] Review Request: dcmtk - Offis DICOM Toolkit

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #31 from Peter Lemenkov lemen...@gmail.com 2011-04-20 10:43:28 
EDT ---
Ok, good.

REVIEW:

Legend: + = PASSED, - = FAILED, 0 = Not Applicable

+ rpmlint is not silent however all its messages can be safely omitted:


work ~: rpmlint ~/Desktop/dcmtk-*
dcmtk.src: W: spelling-error Summary(en_US) Offis - Offs, Offish, Off is
dcmtk.src: W: spelling-error %description -l en_US worklist - work list,
work-list, workstation

^^^ false positive 

dcmtk.src:92: W: macro-in-comment %{name}
dcmtk.src:93: W: macro-in-comment %{name}
dcmtk.src:94: W: macro-in-comment %{_lib}
dcmtk.src:94: W: macro-in-comment %{name}
dcmtk.src:112: W: macro-in-comment %{buildroot}
dcmtk.src:112: W: macro-in-comment %{_lib}
dcmtk.src:112: W: macro-in-comment %{name}
dcmtk.src:113: W: macro-in-comment %{buildroot}
dcmtk.src:113: W: macro-in-comment %{_lib}
dcmtk.src:113: W: macro-in-comment %{buildroot}
dcmtk.src:113: W: macro-in-comment %{_lib}
dcmtk.src:113: W: macro-in-comment %{name}
dcmtk.src:135: W: macro-in-comment %{_sysconfdir}
dcmtk.src:135: W: macro-in-comment %{name}

^^^ that's ok - just a commented out strings. You should just remove them later 

dcmtk.src:69: W: mixed-use-of-spaces-and-tabs (spaces: line 68, tab: line 69)

^^ cosmetic.

dcmtk.x86_64: W: spelling-error Summary(en_US) Offis - Offs, Offish, Off is
dcmtk.x86_64: W: spelling-error %description -l en_US worklist - work list,
work-list, workstation

^^^ false positives

dcmtk.x86_64: W: conffile-without-noreplace-flag /etc/ld.so.conf.d/dcmtk.conf

^^^ That's by design. This file isn't designed to be replaced by user.

dcmtk.x86_64: W: no-manual-page-for-binary dcmsign-3.6.0

^^^ that's ok.

dcmtk-devel.x86_64: W: only-non-binary-in-usr-lib

^^^ false positive.

dcmtk-devel.x86_64: W: no-documentation

^^^ by design - we just don't provide any docs with devel sub-package.

4 packages and 0 specfiles checked; 0 errors, 23 warnings.
work ~: 

+ The package is named according to the  Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format
%{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines.
+ The License field in the package spec file matches the actual license.
+ The file, containing the text of the license(s) for the package, is included
in %doc.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package, match the upstream source, as provided
in the spec URL.

sulaco ~/rpmbuild/SOURCES: sha256sum dcmtk-3.6.0.tar.gz*
cfc509701122adfa359f1ee160e943c1548c7696b607dbb646c5a06f015ed33a 
dcmtk-3.6.0.tar.gz
cfc509701122adfa359f1ee160e943c1548c7696b607dbb646c5a06f015ed33a 
dcmtk-3.6.0.tar.gz.1
sulaco ~/rpmbuild/SOURCES:

+ The package successfully compiles and builds into binary rpms on at least one
primary architecture. See koji link above.
+ All build dependencies are listed in BuildRequires.
0 No need to handle locales.
+ The package stores shared library files in some of the dynamic linker's
default paths, and it calls ldconfig in %post and %postun.
+ The package does NOT bundle copies of system libraries.
0 The package is not designed to be relocatable.

- The package MUST  own all directories that it creates. The following
directories are left unowned:

/etc/dcmtk
/usr/lib64/dcmtk
/usr/share/dcmtk
/usr/share/doc/dcmtk-3.6.0


+ The package does not list a file more than once in the spec file's %files
listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissible content.
0 No extremely large documentation files.
+ Anything, the package includes as %doc, does not affect the runtime of the
application.
+ Header files are stored in a -devel package.
0 No static libraries.
0 No pkgconfig(.pc) files.
+ The library file(s) that end in .so (without suffix) is(are) stored in a
-devel package.
+ The -devel package requires the base package using a fully versioned
dependency: Requires: %{name} = %{version}-%{release}
+ The package does NOT contain any .la libtool archives.
0 Not a GUI application.
+ The package does not own files or directories already owned by other
packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.

Ok, please claim ownership on the directories mentioned above, and I'll
continue.

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

[Bug 697127] Review Request: eclipse-mylyn-tasks - Eclipse Mylyn task connectors

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Severin Gehwolf sgehw...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-04-20 11:03: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 573910] Review Request: dcmtk - Offis DICOM Toolkit

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #32 from Mario Ceresa mrcer...@gmail.com 2011-04-20 11:08:08 EDT 
---
done! sorry for that...

http://mrceresa.fedorapeople.org/dcmtk.spec
http://mrceresa.fedorapeople.org/dcmtk-3.6.0-5.fc14.x86_64.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 573910] Review Request: dcmtk - Offis DICOM Toolkit

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #33 from Mario Ceresa mrcer...@gmail.com 2011-04-20 11:12:42 EDT 
---
The link to the srpms is:
http://mrceresa.fedorapeople.org/dcmtk-3.6.0-5.fc14.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 698301] New: Review Request: perl-CPAN-Changes - Read and write Changes files

2011-04-20 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-CPAN-Changes - Read and write Changes files

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

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


Spec URL:
http://subversion.city-fan.org/repos/cfo-repo/perl-CPAN-Changes/branches/fedora/perl-CPAN-Changes.spec
SRPM URL:
http://www.city-fan.org/~paul/extras/perl-CPAN-Changes/perl-CPAN-Changes-0.16-1.fc16.src.rpm

Description
It is standard practice to include a Changes file in your distribution. The
purpose of the Changes file is to help a user figure out what has changed
since the last release.

People have devised many ways to write the Changes file. A preliminary
specification has been created (CPAN::Changes::Spec) to encourage module
authors to write clear and concise Changes.

This module will help users programmatically read and write Changes files
that conform to the specification.

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


[Bug 573910] Review Request: dcmtk - Offis DICOM Toolkit

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #34 from Peter Lemenkov lemen...@gmail.com 2011-04-20 11:32:25 
EDT ---
Ok, good. 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 573910] Review Request: dcmtk - Offis DICOM Toolkit

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mario Ceresa mrcer...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #35 from Mario Ceresa mrcer...@gmail.com 2011-04-20 12:01:03 EDT 
---
New Package SCM Request
===
Package Name: dcmtk
Short Description: Offis DICOM Toolkit
Owners: mrceresa
Branches: f14 f15 el6
InitialCC: peter

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


[Bug 698333] New: Review Request: cowdancer - Tool for limited copy-on-write directory tree access

2011-04-20 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: cowdancer - Tool for limited copy-on-write directory 
tree access

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

   Summary: Review Request: cowdancer - Tool for limited
copy-on-write directory tree access
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jvce...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL: http://jvcelak.fedorapeople.org/cowdancer/cowdancer.spec
SRPM URL:
http://jvcelak.fedorapeople.org/cowdancer/src/cowdancer-0.16-1.fc15.src.rpm
x86_64 build: http://jvcelak.fedorapeople.org/cowdancer/x86_64/
i686 build: http://jvcelak.fedorapeople.org/cowdancer/i686/

Description:

Cowdancer provides limited copy-on-write file access on hard-link copied
directory trees ('cp -al'). The cowdancer session is started with 'cow-shell'
command. Within that session, cowdancer will create a new file when existing
inode is opened for a write operation. The list of protected inodes is
generated on session start from directory tree where 'cow-shell' was invoked.
Tool is completely implemented in userspace and uses LD_PRELOAD technique.

Limitations:
- support for chmod/chown is not reliable
- requires filesystem supporting hard-links
- operation is not semantically correct when files are originally hard-linked
- if chroot is needed, has to be started from inside of it
- is silent and does nothing when LD_PRELOAD breaks


rpmlint reports only spelling-error which are fine

$ rpmlint ./cowdancer.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint ./cowdancer-0.16-1.fc15.src.rpm 
cowdancer.src: W: spelling-error %description -l en_US inode - node, anode, i
node
cowdancer.src: W: spelling-error %description -l en_US inodes - nodes, anodes,
i nodes
cowdancer.src: W: spelling-error %description -l en_US userspace - user space,
user-space, users pace
cowdancer.src: W: spelling-error %description -l en_US chmod - ch mod, ch-mod,
chm od
cowdancer.src: W: spelling-error %description -l en_US chown - sown, chow,
chon
cowdancer.src: W: spelling-error %description -l en_US filesystem - file
system, file-system, systematic
cowdancer.src: W: spelling-error %description -l en_US chroot - cheroot, ch
root, ch-root
1 packages and 0 specfiles checked; 0 errors, 7 warnings.

$ rpmlint ./cowdancer-0.16-1.fc15.i686.rpm 
cowdancer.i686: W: spelling-error %description -l en_US inode - node, anode, i
node
cowdancer.i686: W: spelling-error %description -l en_US inodes - nodes,
anodes, i nodes
cowdancer.i686: W: spelling-error %description -l en_US userspace - user
space, user-space, users pace
cowdancer.i686: W: spelling-error %description -l en_US chmod - ch mod,
ch-mod, chm od
cowdancer.i686: W: spelling-error %description -l en_US chown - sown, chow,
chon
cowdancer.i686: W: spelling-error %description -l en_US filesystem - file
system, file-system, systematic
cowdancer.i686: W: spelling-error %description -l en_US chroot - cheroot, ch
root, ch-root
1 packages and 0 specfiles checked; 0 errors, 7 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 690282] Review Request: jogl - Java bindings for OpenGL

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #28 from Jon Ciesla l...@jcomserv.net 2011-04-20 13:20:35 EDT ---
Jerry's spec, fixed X11R6 patch.

SPEC: http://zanoni.jcomserv.net/fedora/jogl/jogl.spec
SRPM: http://zanoni.jcomserv.net/fedora/jogl/jogl-1.1.1a-3.fc14.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 560894] Review Request: perl-Pod-Coverage-TrustPod - Allow a module's pod to contain Pod::Coverage hints

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #27 from Fedora Update System upda...@fedoraproject.org 
2011-04-20 13:26:53 EDT ---
perl-Pod-Coverage-TrustPod-0.092830-4.el4 has been pushed to the Fedora EPEL 4
stable repository.

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


[Bug 560894] Review Request: perl-Pod-Coverage-TrustPod - Allow a module's pod to contain Pod::Coverage hints

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Fixed In Version|perl-Pod-Coverage-TrustPod- |perl-Pod-Coverage-TrustPod-
   |0.092830-4.el5  |0.092830-4.el4

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


[Bug 698362] Review Request: writetype - Light word processor

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

PRABIN KUMAR DATTA linux.n@gmail.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 698362] New: Review Request: writetype - Light word processor

2011-04-20 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: writetype - Light word processor

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

   Summary: Review Request: writetype - Light word processor
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: linux.n@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL: http://prabindatta.fedorapeople.org/SPECS/writetype.spec
SRPM URL:
http://prabindatta.fedorapeople.org/SRPMS/writetype-1.2.130-1.fc14.src.rpm

Description: 
WriteType is a free  (and open source) program that helps 
younger students experience success in writing. It is designed 
especially for schools to transform technology from a barrier 
into an opportunity for success.

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

Rpmlint Output:-
For RPM file:
$ rpmlint -i RPMS/noarch/writetype-1.2.130-1.fc14.noarch.rpm 
writetype.noarch: W: no-manual-page-for-binary writetype
Each executable in standard binary directories should have a man page.
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

For SRPM file:
$ rpmlint -i SRPMS/writetype-1.2.130-1.fc14.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

For SPEC file:
$ rpmlint -i SPECS/writetype.spec 
0 packages and 1 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 675557] Review Request: matreshka - set of Ada libraries to help to develop information systems

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Pavel Zhukov pa...@zhukoff.net 2011-04-20 14:15:48 EDT ---
(In reply to comment #8)
 · When you package a snapshot from Subversion, there should still be a normal
 release number before the so-called alphatag, for example Release:
 1.20110326svn%{?dist}.
fixed
 · If you think it's better to have http://Adaforge.qtAda.com; than
 http://Adaforge.qtAda.com/cgi-bin/tracker.fcgi/matreshka; in the URL field,
 then please explain why you think so. I think the latter is better because it
 points directly to a page about the Matreshka project.
fixed
 · All the -devel subpackages should be in the group Development/Libraries, not
 System Environment/Libraries.
fixed
 · Many of the dependencies between the subpackages lack %{?_isa}. As far as 
 I
 can see all of these dependencies are architecture-specific, so %{?_isa}
 should be used.
fixed
 · The -devel subpackages should have Requires: fedora-gnat-project-common =
 2, not BuildRequires. Only the base package should have BuildRequires:
 fedora-gnat-project-common = 2. There's no need to repeat build-time
 dependencies in subpackages as they're all built together in one go.
fixed
 · matreshka-sql-sqlite gets an automatic dependency on libsqlite3.so.0, so the
 explicit dependency on sqlite is unnecessary and should be removed. Quoting 
 the
 guidelines: Packages must not contain explicit Requires on libraries except
 when absolutely necessary. When explicit library Requires are necessary, there
 should be a spec file comment justifying it.
 https://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires
removed
 · Does matreshka-sql-postgresql really depend on the PostgreSQL client 
 programs
 in the package postgresql? If it only depends on the libraries in
 postgresql-libs, then the automatic dependency on libpq.so.5 should be
 sufficient and the explicit dependency should be removed.
removed
 · You pass GNATFLAGS to Make but there is no such variable in the makefiles.
 Did you mean GPRBUILD_FLAGS?
 
 · I think the make check command should probably be in a %check section
 rather than in the %build section.
moved
 · No package owns the directory /usr/include/matreshka.
fixed
 
 Some tips, just in case you didn't know:
 
 · You can add translations of the summary and description fields as
 Summary(ru) and %description -l ru. Look at the PragmARC spec as an
 example:
There is no Russian translation of matreshka. I'll add it 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 668863] Review Request: dolphin-connector - Simple MySQL C API wrapper for C++

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #28 from Fedora Update System upda...@fedoraproject.org 
2011-04-20 14:28:45 EDT ---
dolphin-connector-1.2-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/dolphin-connector-1.2-1.el6

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


[Bug 675557] Review Request: matreshka - set of Ada libraries to help to develop information systems

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Pavel Zhukov pa...@zhukoff.net 2011-04-20 14:27:48 EDT 
---
http://landgraf.fedorapeople.org/packages/requested/matreshka/matreshka-0.1.0-1.20110326svn.fc14.src.rpm

koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=3013858

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


[Bug 698380] New: Review Request: eclipse-mylyn-versions - Eclipse Mylyn Versions

2011-04-20 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: eclipse-mylyn-versions - Eclipse Mylyn Versions

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

   Summary: Review Request: eclipse-mylyn-versions - Eclipse Mylyn
Versions
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: sgehw...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL:
http://fedorapeople.org/~jerboaa/rpm/mylyn/eclipse-mylyn-versions.spec
SRPM URL:
http://fedorapeople.org/~jerboaa/rpm/mylyn/eclipse-mylyn-versions-3.5.0-1.fc15.src.rpm
Description: Provides a framework for accessing team providers for Eclipse
Mylyn.

Dependencies should be already available in rawhide. Thanks for the 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 698362] Review Request: writetype - Light word processor

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from PRABIN KUMAR DATTA linux.n@gmail.com 2011-04-20 
14:59:18 EDT ---
NEED A SPONSOR.

List of My Packages waiting for review:
https://bugzilla.redhat.com/show_bug.cgi?id=687875
https://bugzilla.redhat.com/show_bug.cgi?id=688421
https://bugzilla.redhat.com/show_bug.cgi?id=692131
https://bugzilla.redhat.com/show_bug.cgi?id=698362

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


[Bug 688421] Review Request: pyicu - Python extension wrapping IBM's ICU C++ libraries

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from PRABIN KUMAR DATTA linux.n@gmail.com 2011-04-20 
15:00:57 EDT ---
NEED A SPONSOR.

This package is a dependency package for aardtool which I will submit for
review request once this package will be submitted

List of My Packages waiting for review:
https://bugzilla.redhat.com/show_bug.cgi?id=687875
https://bugzilla.redhat.com/show_bug.cgi?id=688421
https://bugzilla.redhat.com/show_bug.cgi?id=692131
https://bugzilla.redhat.com/show_bug.cgi?id=698362

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


[Bug 687875] Review Request: aarddict - Multi-platform dictionary and offline Wikipedia reader

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from PRABIN KUMAR DATTA linux.n@gmail.com 2011-04-20 
14:59:45 EDT ---
NEED A SPONSOR.

List of My Packages waiting for review:
https://bugzilla.redhat.com/show_bug.cgi?id=687875
https://bugzilla.redhat.com/show_bug.cgi?id=688421
https://bugzilla.redhat.com/show_bug.cgi?id=692131
https://bugzilla.redhat.com/show_bug.cgi?id=698362

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


[Bug 698362] Review Request: writetype - Light word processor

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 CC||sanjay.an...@gmail.com

--- Comment #2 from Ankur Sinha sanjay.an...@gmail.com 2011-04-20 15:00:52 
EDT ---
Some notes:

1. The patch needs a comment describing its requirement.
2. You no longer need the buildroot tag, and the clean section.
3. It's shebang, not sheband ;)
4. In the files section, at the egginfo part, use py?.?.egg-info, it'll make
updates easier. 
5. Do check if the python-setuptools is also a runtime requirement once. 
6. I don't see a LICENSE file in the files section. Please request upstream to
include one.
7. You haven't handled the desktop file. Please refer to
http://fedoraproject.org/wiki/PackagingGuidelines#Desktop_files for details
8. Isn't there an icon (Since there is a desktop file?)

The spec looks okay at a glance, but a detailed review might reveal more
issues.

Regards,
Ankur

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


[Bug 694287] Review Request: openCOLLADA - 3D import and export libraries

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #21 from Richard Shaw hobbes1...@gmail.com 2011-04-20 15:13:16 
EDT ---
(In reply to comment #20)
 (In reply to comment #19)
  (In reply to comment #17)
   2 issues:
   
   * The tarball contains a lot of bundled libraries (cf, Externals/).
  
  cf?
 Compare for the contents of the directory called Externals/ inside of the
 tarball. It contains zlib, Cg,, libxml, lib3ds and other libraries.
 
 A more detailed look into the package tells that openCOLLADA currently only
 uses MathMLSolver/ and UTF/.
 
 = Make sure the other directories are not being used when building for 
 Fedora.
 Brute-force way to do so would be to remove them in %prep (This is what a
 recent change to the FPG recommends).

I have done this and it still builds.


   This is problematic twice:
   - In general, the Fedora package should not not use them. 
   
   - These packages' licenses need to be checked for whether they are 
   properly
   licensed and whether these package's licenses are compatible to 
   openCOLLADA's
   license. 
   
   From a coarse glance into tarball, I'd suspect Externals/MathMLSolver not 
   to be
   properly licensed (I can't find any licence). Googling however directed 
   me to 
   http://mathmlsolver.svn.sourceforge.net/viewvc/mathmlsolver/trunk, but I
   haven't checked details, yet.
  
  Drilled down to the actual SF page at it says it is MIT licensed.
 Yes. I meanwhile also found some copyright notices in Externals/MathMLSolver's
 headers and found openCOLLADA/Externals/MathMLSolver to be a hacked up version
 of the code on sourceforge.
 
 I don't know why openCOLLADA is doing so - Could be they are just hacking 
 and
 don't care about proper integration/packaging, could be the sourceforge 
 project
 is dead. AFAICT, googling indicates openCOLLADA is the only user of
 MathMLSolver while the sourceforge project might be dead.

Yes, I haven't seen much activity on their SF page so I would assume it's dead.

As far as UTF goes. It looks to be a copy of ConvertUTF which also appears to
be defunct. I found a ftp link for the code on unicode.org through:

http://gears.googlecode.com/svn/trunk/third_party/convert_utf/README.google

If there is nothing happening upstream for these two libraries, are they OK
being bundled in openCOLLADA?

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


[Bug 680593] Review Request: perl-Browser-Open - Open a browser in a given URL

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-04-20 15:18:07 EDT ---
perl-Browser-Open-0.03-2.fc13 has been pushed to the Fedora 13 stable
repository.

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


[Bug 691153] Review Request: pam_shield - pam module to block brute force attacks

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #26 from Fedora Update System upda...@fedoraproject.org 
2011-04-20 15:18:52 EDT ---
pam_shield-0.9.5-7.fc13 has been pushed to the Fedora 13 stable repository.  If
problems still persist, please make note of it in this bug report.

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


[Bug 691153] Review Request: pam_shield - pam module to block brute force attacks

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Fixed In Version|pam_shield-0.9.5-7.fc15 |pam_shield-0.9.5-7.fc13

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


[Bug 680593] Review Request: perl-Browser-Open - Open a browser in a given URL

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Fixed In Version|perl-Browser-Open-0.03-2.fc |perl-Browser-Open-0.03-2.fc
   |15  |13

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


[Bug 680593] Review Request: perl-Browser-Open - Open a browser in a given URL

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-04-20 15:18:24 EDT ---
perl-Browser-Open-0.03-2.fc14 has been pushed to the Fedora 14 stable
repository.

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


[Bug 680593] Review Request: perl-Browser-Open - Open a browser in a given URL

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Fixed In Version|perl-Browser-Open-0.03-2.fc |perl-Browser-Open-0.03-2.fc
   |13  |14

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


[Bug 691153] Review Request: pam_shield - pam module to block brute force attacks

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Fixed In Version|pam_shield-0.9.5-7.fc13 |pam_shield-0.9.5-7.fc14

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


[Bug 691153] Review Request: pam_shield - pam module to block brute force attacks

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #27 from Fedora Update System upda...@fedoraproject.org 
2011-04-20 15:21:17 EDT ---
pam_shield-0.9.5-7.fc14 has been pushed to the Fedora 14 stable repository.  If
problems still persist, please make note of it in this bug report.

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


[Bug 697401] Review Request: perl-Ouch - Exceptions that don't hurt

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-04-20 
15:22:30 EDT ---
perl-Ouch-0.0300-1.fc14 has been pushed to the Fedora 14 testing repository.

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


[Bug 698380] Review Request: eclipse-mylyn-versions - Eclipse Mylyn Versions

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Severin Gehwolf sgehw...@redhat.com 2011-04-20 15:39:23 
EDT ---
Updated Spec file. New SRPM is:
http://fedorapeople.org/~jerboaa/rpm/mylyn/eclipse-mylyn-versions-3.5.0-2.fc15.src.rpm

Sorry about that. I only just discovered that myself (due to filing an upstream
bug). 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 692541] Review Request: rep-gtk - GTK+ binding for librep Lisp environment

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-04-20 15:38:24 EDT ---
rep-gtk-0.90.5-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/rep-gtk-0.90.5-4.fc15

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


[Bug 649495] Review Request: openstack-nova - OpenStack Compute (nova)

2011-04-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Fabian Deutsch fabian.deut...@gmx.de 2011-04-20 15:41:37 
EDT ---
I've updated the spec file created by Silas, so it builds fine for the latest
stable openstack-nova release 2011.2

SRPM:
http://fedorapeople.org/~fabiand/openstack/openstack-nova-2011.2-1.fc15.src.rpm

SPEC:
https://gitorious.org/rpms/rpms/blobs/master/openstack-nova/openstack-nova.spec

It is still a bit rough around the edges, my intention is to get it into
fedora, so people can start testing. This might also be interesting for the
Cloud SIG.

opestack-glance will be the next step.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are 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   >