[Bug 674673] Review Request: lovelock-backgrounds - Lovelock desktop backgrounds

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


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

--- Comment #4 from Martin Sourada martin.sour...@gmail.com 2011-02-03 
03:30:54 EST ---
Thanks for the quick 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 674673] Review Request: lovelock-backgrounds - Lovelock desktop backgrounds

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


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

Martin Sourada martin.sour...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Martin Sourada martin.sour...@gmail.com 2011-02-03 
03:33:50 EST ---
New Package SCM Request
===
Package Name: lovelock-backgrounds
Short Description: Lovelock desktop backgrounds
Owners: mso
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 668243] Review Request: libqb - An IPC library for high performance servers.

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


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

--- Comment #14 from Fabio Massimo Di Nitto fdini...@redhat.com 2011-02-03 
03:40:45 EST ---
Checked the last package and it addresses all the issues mentioned above.

Now we just need a sponsor to do the job.

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


[Bug 672779] Review Request: perl-Module-Metadata - Gather package and POD information from perl module files

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


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

Bug 672779 depends on bug 672839, which changed state.

Bug 672839 Summary: Upgrade perl(version) to 0.88
https://bugzilla.redhat.com/show_bug.cgi?id=672839

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

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


[Bug 674587] Review Request: rubygem-sqlite3 - Allows Ruby scripts to interface with a SQLite3 database

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


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

--- Comment #2 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2011-02-03 
04:04:32 EST ---
(In reply to comment #1)
 
  rubygem-sqlite3.src:22: W: unversioned-explicit-obsoletes 
  rubygem-sqlite3-ruby
 
 I believe we can ignore this as we should retire the rubygem-sqlite3-ruby
 package after this.
 

Well, the upstream may change his thought again and may change the name
(although unlikely), so adding version to Obsoletes like
Obsoletes: rubygem-sqlite3-ruby  1.3.3 is anyway safer (and adding the
version should cause no problem). Note that missing version on Obsoletes
has causes some nasty problems in the past on other package.

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


[Bug 658420] Review Request: zorba - General purpose XQuery processor

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


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

--- Comment #11 from Martin Gieseking martin.giesek...@uos.de 2011-02-03 
04:10:01 EST ---
Yes, of course. ;)
Here's the request: https://fedorahosted.org/fpc/ticket/52

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


[Bug 671095] Review Request: rubygem-pr_geohash - GeoHash encode/decode library for pure Ruby

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


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

Vít Ondruch vondr...@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 672779] Review Request: perl-Module-Metadata - Gather package and POD information from perl module files

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


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

--- Comment #2 from Paul Howarth p...@city-fan.org 2011-02-03 04:50:45 EST ---
Upstream update:

Spec URL:
http://subversion.city-fan.org/repos/cfo-repo/perl-Module-Metadata/branches/fedora/perl-Module-Metadata.spec
SRPM URL:
http://www.city-fan.org/~paul/extras/perl-Module-Metadata/perl-Module-Metadata-1.04-1.fc15.src.rpm

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


[Bug 672779] Review Request: perl-Module-Metadata - Gather package and POD information from perl module files

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


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

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

   What|Removed |Added

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

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


[Bug 484644] Review Request: screenlets - Fully themeable mini-apps

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


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

--- Comment #31 from Tareq Al Jurf talj...@fedoraproject.org 2011-02-03 
05:00:42 EST ---
Actually there is work ongoing here: https://launchpad.net/~screenlets-dev

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


[Bug 674587] Review Request: rubygem-sqlite3 - Allows Ruby scripts to interface with a SQLite3 database

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


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

--- Comment #3 from Vít Ondruch vondr...@redhat.com 2011-02-03 05:27:31 EST 
---
(In reply to comment #1)
 Will take this one. The package builds fine in koji. Most review guidelines
 pass except for the following
 
 * URL and Source0 are wrong, still referring to the sqlite3-ruby package ones,
 please update to new locations: http://rubygems.org/gems/sqlite3

I have updated the Source0 link, however I updated the URL just slightly, still
left it pointing to rubyforge as long as it is not clear what is homepage of
this project. I have risen upstream ticket for clarification:
https://github.com/luislavena/sqlite3-ruby/issues/issue/26


 * you can remove the BuildRoot tag and the rm -rf %{buildroot} at the
 beginning of %install

Done

 * please change %define ruby_sitearch to %global ruby_sitearch at the top
 of the spec file

Done

 * rpmlint looks good
 
  $ rpmlint rpmbuild/RPMS/x86_64/rubygem-sqlite3-* 
  rpmbuild/SRPMS/rubygem-sqlite3-1.3.3-1.fc14.src.rpm 
  rubygem-sqlite3.x86_64: I: enchant-dictionary-not-found en_US
 
 Can be ignored
 
  rubygem-sqlite3.x86_64: W: obsolete-not-provided rubygem-sqlite3-ruby
 
 Believe this is what we intend and can be ignored.
 
  rubygem-sqlite3-doc.x86_64: W: unexpanded-macro 
  /usr/lib/ruby/gems/1.8/doc/sqlite3-1.3.3/ri/SQLite3/Database/authorizer%3d-i.yaml
   %3d
 
 Can be ignored 
 
  rubygem-sqlite3.src:22: W: unversioned-explicit-obsoletes 
  rubygem-sqlite3-ruby
 
 I believe we can ignore this as we should retire the rubygem-sqlite3-ruby
 package after this.

Version added according to Mamoru's hint. However now, rpmlint complains:
rubygem-sqlite3.x86_64: W: obsolete-not-provided rubygem-sqlite3-ruby

This should be probably ignored as Mamoru stated in email:

- on rawhide (on rawhide only) rubygem-sqlite3 should obsolete 
  rubygem-sqlite3-ruby (Provides: rubygem(sqlite3-ruby) should _not_ be added
  to rubygem-sqlite3, just obsolete rubygem-sqlite3-ruby) 

and I am going to push this gem just for rawhide.

 Everything else looks good.

Spec URL: http://people.redhat.com/vondruch/rubygem-sqlite3.spec
SRPM URL:
http://people.redhat.com/vondruch/rubygem-sqlite3-1.3.3-2.fc14.src.rpm

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

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

[Bug 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #15 from Vít Ondruch vondr...@redhat.com 2011-02-03 05:30:31 EST 
---
Thank you for your review!



New Package SCM Request
===
Package Name: rubygem-arel
Short Description: Arel is a Relational Algebra for Ruby
Owners: vondruch
Branches: 
InitialCC:

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

[Bug 657520] Review Request: perl-Array-Utils - Small utils for array manipulation

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


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

--- Comment #7 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 
2011-02-03 05:34:27 EST ---
Lubomir, is there something blocking this bug?

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


[Bug 657520] Review Request: perl-Array-Utils - Small utils for array manipulation

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


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

Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Lubomir Rintel lkund...@v3.sk 2011-02-03 05:46:42 EST ---
No, sorry, I have lost track of this somehow. Thanks for the reminder.

New Package SCM Request
===
Package Name: perl-Array-Utils
Short Description: Small utils for array manipulation
Owners: lkundrak
Branches: f14 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 672779] Review Request: perl-Module-Metadata - Gather package and POD information from perl module files

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


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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

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


[Bug 598688] Review Request: archivemount - FUSE based filesystem for mounting compressed archives

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


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

--- Comment #24 from Niels de Vos nde...@redhat.com 2011-02-03 05:59:39 EST 
---
Package Change Request
==
Package Name: archivemount
New Branches: el5 el6
Owners: bar devos

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


[Bug 672779] Review Request: perl-Module-Metadata - Gather package and POD information from perl module files

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


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

--- Comment #3 from Petr Pisar ppi...@redhat.com 2011-02-03 06:01:05 EST ---
Source tar ball is original. Ok.
Summary verified from lib/Module/Metadata.pm. Ok.
License verified from lib/Module/Metadata.pm. Ok.
No XS. BuildArch ok.
Description copied from %summary as upstream does not provide any. Ok.
Note: You can use the macro to avoid duplication in spec file.
Note: You can remove the BuildRoot code if you are not willing to support
EPEL=5 .

TODO: BuildRequire perl(File::Spec) (lib/Module/Metadata.pm:17) as the package
can dual-live in the future (http://search.cpan.org/~smueller/PathTools/).
TODO: BuildRequire perl(IO::File) (lib/Module/Metadata.pm:18) as the package
can dual-live in the future (http://search.cpan.org/~gbarr/IO/).
TODO: BuildRequire perl(File::Temp) (t/lib/MBTest.pm:7) as the package can
dual-live in the future (http://search.cpan.org/~tjenness/File-Temp/).
TODO: BuildRequire perl(Exporter) (t/lib/MBTest.pm:66) as the package can
dual-live in the future (http://search.cpan.org/~ferreira/Exporter/).
TODO: BuildRequire perl(Cwd) (t/lib/MBTest.pm:69) as the package can dual-live
in the future (http://search.cpan.org/~smueller/PathTools/).
TODO: BuildRequire perl(File::Path) (t/lib/DistGen.pm:16) as the package can
dual-live in the future (http://search.cpan.org/~dland/File-Path/)
TODO: BuildRequie perl(Data::Dumper) (t/lib/DistGen.pm:20) as the package can
dual-live in the future (http://search.cpan.org/~smueller/Data-Dumper/)

perl(Module::Build) used at t/lib/DistGen.pm:528 is not loaded at test-time.
Ok.

All tests pass. Ok.

$ rpmlint perl-Module-Metadata.spec
../SRPMS/perl-Module-Metadata-1.04-1.fc15.src.rpm
../RPMS/noarch/perl-Module-Metadata-1.04-1.fc15.noarch.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
rpmlint Ok.

$ rpm -q -lv -p ../RPMS/noarch/perl-Module-Metadata-1.04-1.fc15.noarch.rpm
drwxr-xr-x2 rootroot0 Feb  3 11:27
/usr/share/doc/perl-Module-Metadata-1.04
-rw-r--r--1 rootroot  790 Feb  3 08:54
/usr/share/doc/perl-Module-Metadata-1.04/Changes
-rw-r--r--1 rootroot 3157 Feb  3 11:27
/usr/share/man/man3/Module::Metadata.3pm.gz
drwxr-xr-x2 rootroot0 Feb  3 11:27
/usr/share/perl5/Module
-rw-r--r--1 rootroot20759 Feb  3 08:53
/usr/share/perl5/Module/Metadata.pm
File layout and permissions Ok.

$ rpm -q --requires -p
../RPMS/noarch/perl-Module-Metadata-1.04-1.fc15.noarch.rpm | sort | uniq -c
  1 perl(File::Find)  
  1 perl(File::Spec)  
  1 perl(IO::File)  
  1 perl(:MODULE_COMPAT_5.12.3)  
  1 perl(strict)  
  1 perl(vars)  
  1 perl(version) = 0.87
  1 rpmlib(CompressedFileNames) = 3.0.4-1
  1 rpmlib(FileDigests) = 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) = 4.0-1
  1 rpmlib(PayloadIsXz) = 5.2-1
Binary requires Ok.

$ rpm -q --provides -p
../RPMS/noarch/perl-Module-Metadata-1.04-1.fc15.noarch.rpm | sort | uniq -c
  1 perl(Module::Metadata) = 1.04
  1 perl-Module-Metadata = 1.04-1.fc15
Binary provides Ok.

$ resolvedeps-f15 perl-Module-Metadata-1.04-1.fc15.noarch.rpm 
Binary dependencies resolvable. Ok.

Package builds in F15
(http://koji.fedoraproject.org/koji/taskinfo?taskID=2759587). Ok.

Otherwise package is in line with Fedora and perl packaging guide lines.


Resolution: Package APPROVED.

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


[Bug 664817] Review Request: perl-HTML-Selector-XPath - CSS Selector to XPath compiler

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


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

--- Comment #14 from Petr Pisar ppi...@redhat.com 2011-02-03 06:12:30 EST ---
Dear Ralf,

I reevaluated my opinion on the not yet dual-lived dependencies. As reviewed
package is perfect at time of review, it's valid to accept the package. I will
mark such speculative requires as TODO item in my reviews in the future if
there will be no dual-lived package at that time.

I apology for any inconveniences.

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


[Bug 598688] Review Request: archivemount - FUSE based filesystem for mounting compressed archives

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


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

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

   What|Removed |Added

 CC||p...@city-fan.org
   Flag|fedora-cvs+ |fedora-cvs?

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


[Bug 669311] Review Request: mupdf - A lightweight PDF viewer and toolkit written in portable C

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


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

--- Comment #23 from Mohamed El Morabity pikachu.2...@gmail.com 2011-02-03 
06:12:56 EST ---
Do you really want to give up this review? Static libraries are not completely
forbidden in Fedora, only discouraged. And I believe that the fact that
upstream doesn't want to support shared libraries is a good reason to provide
anyway static libs in this package.
There was a similar case some months ago:
   https://bugzilla.redhat.com/show_bug.cgi?id=602279

If you're willing to re-open this review, I suggest you to carefully consider
the guidelines relative to static libs:
  
http://fedoraproject.org/wiki/Packaging/Guidelines#Packaging_Static_Libraries

mupdf is interesting enough to be distributed through the Fedora repos, in
spite of the static lib.

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


[Bug 598688] Review Request: archivemount - FUSE based filesystem for mounting compressed archives

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


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

Niels de Vos nde...@redhat.com changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |

--- Comment #25 from Niels de Vos nde...@redhat.com 2011-02-03 07:01:41 EST 
---
Please disregard comment #24, user bar should not be co-owner :)



Package Change Request
==
Package Name: archivemount
New Branches: el5 el6
Owners: devos

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


[Bug 669311] Review Request: mupdf - A lightweight PDF viewer and toolkit written in portable C

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


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

Pavel Zhukov pa...@zhukoff.net changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|CANTFIX |
   Keywords||Reopened

--- Comment #24 from Pavel Zhukov pa...@zhukoff.net 2011-02-03 06:59:13 EST 
---
Mohamed, 
I am sorry for irascibility

Mupdf is good software and I'm using it but I Has been surprised by behaviour
of developers.

So, Do I rebuild mupdf to use static libraries and build mudpf-static package?

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


[Bug 672779] Review Request: perl-Module-Metadata - Gather package and POD information from perl module files

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


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

--- Comment #4 from Marcela Mašláňová mmasl...@redhat.com 2011-02-03 07:21:20 
EST ---
(In reply to comment #0)
 Spec URL:
 http://subversion.city-fan.org/repos/cfo-repo/perl-Module-Metadata/branches/fedora/perl-Module-Metadata.spec
 SRPM URL:
 http://www.city-fan.org/~paul/extras/perl-Module-Metadata/perl-Module-Metadata-1.03-2.fc15.src.rpm
 
 Description:
 Gather package and POD information from perl module files.

I've built perl-version-0.88 in rawhide.

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

[Bug 672779] Review Request: perl-Module-Metadata - Gather package and POD information from perl module files

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


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

--- Comment #5 from Paul Howarth p...@city-fan.org 2011-02-03 07:41:20 EST ---
(In reply to comment #4)
 (In reply to comment #0)
  Spec URL:
  http://subversion.city-fan.org/repos/cfo-repo/perl-Module-Metadata/branches/fedora/perl-Module-Metadata.spec
  SRPM URL:
  http://www.city-fan.org/~paul/extras/perl-Module-Metadata/perl-Module-Metadata-1.03-2.fc15.src.rpm
  
  Description:
  Gather package and POD information from perl module files.
 
 I've built perl-version-0.88 in rawhide.

I see you've also had perl-version branched for f14; are you planning to update
it to 0.88 there as well?

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


[Bug 651898] Review Request: rubygem-activemodel - A toolkit for building modeling frameworks

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


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-02-03 07:46:38

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

[Bug 674674] Review Request: python-zope-configuration - Zope Configuration Markup Language (ZCML)

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


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

Luke Macken lmac...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(robinlee.sysu@gma
   ||il.com)

--- Comment #4 from Luke Macken lmac...@redhat.com 2011-02-03 08:58:44 EST ---
(In reply to comment #3)
   [=] Buildroot, %clean section and the rm command in %install section are 
   not
   required.
   http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag
   http://fedoraproject.org/wiki/Packaging/Guidelines#.25clean
  
  I'd rather leave these in at the moment, since I plan on building these 
  down to
  EL-5
 Then manual definition of %{python_sitelib} is also required:
 %{!?python_sitelib: %global python_sitelib %(%{__python} -c from
 distutils.sysconfig import get_python_lib; print(get_python_lib()))}

Added.

 [-] python-setuptools is not required in runtime.

Removed.

Updated the SRPM  SPEC with the same URL as before.

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


[Bug 674676] Review Request: python-zope-deprecation - Zope 3 Deprecation Infrastructure

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


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

Luke Macken lmac...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(robinlee.sysu@gma
   ||il.com)

--- Comment #2 from Luke Macken lmac...@redhat.com 2011-02-03 09:12:01 EST ---
(In reply to comment #1)
 Others:
 [=] Consider renaming python-setuptools-devel to python-setuptools

Done.

 [-] Source0 URL:
 http://pypi.python.org/packages/source/z/%{modname}/%{modname}-%{version}.tar.gz

Updated.

 [=] Buildroot, %clean section and the rm command in %install section are not
 required.
 http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag
 http://fedoraproject.org/wiki/Packaging/Guidelines#.25clean

I'd like to keep these for now, until I push this all of the way down to EL-5. 
I also added the python_sitelib definition to the top.

 [=] /usr/lib/python2.7/site-packages/zope/deprecation/DEPENDENCIES.cfg
 /usr/lib/python2.7/site-packages/zope/deprecation/README.txt
 should be excluded.

Done.

 [=] The tests not actually run.
 'zope.testrunner' is required but not available in Fedora. 'zope.testrunner'
 used to be part of 'zope.testing'.
 I ran the tests locally and all went well:
 
 $ ./bin/test 
 Running zope.testrunner.layer.UnitTests tests:
   Set up zope.testrunner.layer.UnitTests in 0.000 seconds.
   Ran 19 tests with 0 failures and 0 errors in 0.110 seconds.
 Tearing down left over layers:
   Tear down zope.testrunner.layer.UnitTests in 0.000 seconds.

Ok, removed %check block for now.

 [=] Consider excluding the tests
 /usr/lib/python2.7/site-packages/zope/deprecation/tests.py*

Removed during %install.

Updated SPEC/SRPM at original URL.

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


[Bug 672845] Review Request: rubygem-net-ping - Net::Ping rubygem library

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


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

Lukáš Zapletal l...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |rubygem-net-ping-1.3.7-1.fc |rubygem-net-ping -
   |14.src.rpm - Net::Ping  |Net::Ping rubygem library
   |rubygem library |

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

[Bug 565232] Review Request: perl-Log-Any - Bringing loggers and listeners together

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


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

--- Comment #12 from Ralf Corsepius rc040...@freenet.de 2011-02-03 09:24:36 
EST ---
Apparently this package was never built for f13 and f14.

I am going to build and push them.

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


[Bug 565232] Review Request: perl-Log-Any - Bringing loggers and listeners together

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


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

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-02-03 09:31:03 EST ---
perl-Log-Any-0.11-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/perl-Log-Any-0.11-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 565232] Review Request: perl-Log-Any - Bringing loggers and listeners together

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


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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-02-03 09:31:11 EST ---
perl-Log-Any-0.11-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/perl-Log-Any-0.11-1.fc13

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


[Bug 672440] Review Request: flann - Fast Library for Approximate Nearest Neighbors

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


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

Tim Niemueller t...@niemueller.de changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #6 from Tim Niemueller t...@niemueller.de 2011-02-03 09:44:08 EST 
---
Looks good now.

APPROVED.

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


[Bug 177827] Review Request: python-ctypes - Advanced Foreign Function Interface for Python

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


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

Matt Domsch matt_dom...@dell.com changed:

   What|Removed |Added

 CC||matt_dom...@dell.com
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #18 from Matt Domsch matt_dom...@dell.com 2011-02-03 10:33:49 EST 
---
Package Change Request
==
Package Name: python-ctypes
New Branches: el4
Owners: mdomsch mebrown

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


[Bug 674674] Review Request: python-zope-configuration - Zope Configuration Markup Language (ZCML)

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


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

Robin Lee robinlee.s...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?, |fedora-review+
   |needinfo?(robinlee.sysu@gma |
   |il.com) |

--- Comment #5 from Robin Lee robinlee.s...@gmail.com 2011-02-03 10:32:45 EST 
---
Rawhide build succeeded:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2759960

This package is approved by 'cheeselee'.

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


[Bug 674676] Review Request: python-zope-deprecation - Zope 3 Deprecation Infrastructure

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


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

Robin Lee robinlee.s...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?, |fedora-review+
   |needinfo?(robinlee.sysu@gma |
   |il.com) |

--- Comment #3 from Robin Lee robinlee.s...@gmail.com 2011-02-03 10:35:07 EST 
---
Since tests are disabled by now, 'BuildRequires:  python-zope-testing' can be
commented out.

Rawhide build succeeded:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2759976

This package is approved by 'cheeselee'.

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


[Bug 672779] Review Request: perl-Module-Metadata - Gather package and POD information from perl module files

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #6 from Paul Howarth p...@city-fan.org 2011-02-03 11:03:30 EST ---
I'll assume that f14 is going to have perl(version) = 0.87 at some point.

Thanks for the review.

New Package SCM Request
===
Package Name: perl-Module-Metadata
Short Description: Gather package and POD information from perl module file
Owners: pghmcfc
Branches: f14
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 672440] Review Request: flann - Fast Library for Approximate Nearest Neighbors

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


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

Rich Mattes richmat...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Rich Mattes richmat...@gmail.com 2011-02-03 11:21:12 EST 
---
New Package SCM Request
===
Package Name: flann
Short Description: Fast Library for Approximate Nearest Neighbors
Owners: rmattes
Branches: f13 f14
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 672440] Review Request: flann - Fast Library for Approximate Nearest Neighbors

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


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

--- Comment #8 from Tim Niemueller t...@niemueller.de 2011-02-03 11:28:43 EST 
---
Would you mind maintaining it in el5/el6? Maybe we can have OpenRAVE there,
then. Some teams use CentOS or the like as a long-term stable robot platform.

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


[Bug 474549] Review Request: ca-cacert.org - CAcert.org CA root certificates

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


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

--- Comment #38 from Tom spot Callaway tcall...@redhat.com 2011-02-03 
11:25:29 EST ---
I agree with Matt here. If CAcert's position is that their intention behind the
specifically does not permit is that it is meant to be functionally
equivalent to the GPL warranty disclaimer, then I would suggest that they
reword the RDL to use that warranty disclaimer and drop the additional
restrictions.

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


[Bug 225670] Merge Review: cups

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


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

Bug 225670 depends on bug 643455, which changed state.

Bug 643455 Summary: PIE support not working correctly
https://bugzilla.redhat.com/show_bug.cgi?id=643455

   What|Old Value   |New Value

 Resolution||NOTABUG
 Status|NEW |CLOSED

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


[Bug 672440] Review Request: flann - Fast Library for Approximate Nearest Neighbors

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


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

--- Comment #9 from Rich Mattes richmat...@gmail.com 2011-02-03 11:52:18 EST 
---
Sure, I can maintain them there too.  I just ran some mockbuilds, hdf5 and
gtest aren't available in EPEL.  el6 is ok without them, but I have to track
down a build error for el5.

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


[Bug 672440] Review Request: flann - Fast Library for Approximate Nearest Neighbors

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


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

--- Comment #10 from Rich Mattes richmat...@gmail.com 2011-02-03 11:52:47 EST 
---
New Package SCM Request
===
Package Name: flann
Short Description: Fast Library for Approximate Nearest Neighbors
Owners: rmattes
Branches: f13 f14 el5 el6
InitialCC:

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


[Bug 670040] Review Request: mediawiki116-ConfirmEdit - Adds captchas when saving an edit

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


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

--- Comment #6 from Jon Stanley jonstan...@gmail.com 2011-02-03 11:55:33 EST 
---
I think the problem is that Ian doesn't want to do a subpackage *or* require a
dep on python-imaging.

I'm thinking that one or the other has to be done. If you don't do either,
there's a dependency problem in the package (the python script will give an
ImportError when you try and run 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 668090] Review Request: rubygem-railties - Rails internals: application bootup, plugins, generators, and rake tasks.

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


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

--- Comment #13 from Minnikhanov minnikha...@gmail.com 2011-02-03 12:57:46 
EST ---
New Package SCM Request
===
Package Name: rubygem-railties
Short Description: Rails internals: application bootup, plugins, generators,
and rake tasks.
Owners: minn
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 188378] Review Request: perl-Test-NoWarnings

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


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

--- Comment #4 from Fedora Update System upda...@fedoraproject.org 2011-02-03 
12:57:58 EST ---
perl-Test-NoWarnings-0.083-1.el4 has been pushed to the Fedora EPEL 4 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 the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 668823] Review Request: rubygem-text-format - Text::Format formats fixed-width text nicely

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


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

Minnikhanov minnikha...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

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


[Bug 668823] Review Request: rubygem-text-format - Text::Format formats fixed-width text nicely

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


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

--- Comment #7 from Minnikhanov minnikha...@gmail.com 2011-02-03 13:02:11 EST 
---
Looks good.

-
This package (rubygem-text-format) is APPROVED by
minn
-

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


[Bug 668823] Review Request: rubygem-text-format - Text::Format formats fixed-width text nicely

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


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

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

   What|Removed |Added

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

--- Comment #8 from Mohammed Morsi mmo...@redhat.com 2011-02-03 13:12:13 EST 
---
Thanks alot for the review

New Package SCM Request
===
Package Name: rubygem-text-format
Short Description: Text::Format formats fixed-width text nicely 
Owners: mmorsi
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 573910] Review Request: dcmtk - Offis DICOM Toolkit

2011-02-03 Thread bugzilla
Please do not reply directly 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 #21 from Mario Ceresa mrcer...@gmail.com 2011-02-03 13:16:48 EST 
---
Hello all!
finally we got it. Packaged code is in:

https://github.com/mrceresa/dcmtk-offis

which follows the official dcmtk git up to 3.6.1 and adds:
 - proper shared lib soname generation.
 - reorganize files to conform with LFS

I still have to remove charls from the bundled libs as we did for gdcm. There
should be no more but I'd rather check with care.

SPEC and SRPMS are as follows:

http://mrceresa.fedorapeople.org/dcmtk.spec
http://mrceresa.fedorapeople.org/dcmtk-3.6.1-1.20110203git.fc14.src.rpm

Best,
Mario

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


[Bug 188380] Review Request: perl-Test-Deep

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


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

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-02-03 
13:19:09 EST ---
perl-Test-Deep-0.096-1.el4 has been submitted as an update for Fedora EPEL 4.
https://admin.fedoraproject.org/updates/perl-Test-Deep-0.096-1.el4

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


[Bug 668090] Review Request: rubygem-railties - Rails internals: application bootup, plugins, generators, and rake tasks.

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


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

--- Comment #14 from Minnikhanov minnikha...@gmail.com 2011-02-03 13:30:28 
EST ---
New Package SCM Request
===
Package Name: rubygem-railties
Short Description: Rails internals: application bootup, plugins, generators,
and rake tasks.
Owners: minn
Branches: f14
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 672955] Review Request: iwl100-firmware - Firmware for Intel(R) Wireless WiFi Link 100 Series Adapters

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


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

John W. Linville linvi...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-02-03 13:34:45

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


[Bug 672955] Review Request: iwl100-firmware - Firmware for Intel(R) Wireless WiFi Link 100 Series Adapters

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


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

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-02-03 
13:41:19 EST ---
iwl100-firmware-39.31.5.1-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/iwl100-firmware-39.31.5.1-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 674930] New: Review Request: sh-elf-gcc - Cross Compiling GNU GCC targeted at sh-elf

2011-02-03 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: sh-elf-gcc - Cross Compiling GNU GCC targeted at sh-elf

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

   Summary: Review Request: sh-elf-gcc - Cross Compiling GNU GCC
targeted at sh-elf
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: linvi...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://linville.fedorapeople.org/sh-elf-gcc.spec
SRPM URL: http://linville.fedorapeople.org/sh-elf-gcc-4.4.5-1.fc14.src.rpm
Description:
This is a Cross Compiling version of GNU GCC, which can be used to
compile programs for the sh-elf platform, instead of for the
native %{_arch} platform.

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


[Bug 674930] Review Request: sh-elf-gcc - Cross Compiling GNU GCC targeted at sh-elf

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


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

John W. Linville linvi...@redhat.com changed:

   What|Removed |Added

 Blocks||674931

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


[Bug 674930] Review Request: sh-elf-gcc - Cross Compiling GNU GCC targeted at sh-elf

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


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

John W. Linville linvi...@redhat.com changed:

   What|Removed |Added

 Depends on||674929

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


[Bug 674931] New: Review Request: sh-elf-newlib - Cross Compiled Red Hat Newlib targeted at sh-elf

2011-02-03 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: sh-elf-newlib -  Cross Compiled Red Hat Newlib 
targeted at sh-elf

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

   Summary: Review Request: sh-elf-newlib -  Cross Compiled Red
Hat Newlib targeted at sh-elf
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: linvi...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://linville.fedorapeople.org/sh-elf-newlib.spec
SRPM URL: http://linville.fedorapeople.org/sh-elf-newlib-1.19.0-1.fc14.src.rpm
Description:
This is a Cross Compiled version of Red Hat Newlib, which can be used to
compile programs for the sh-elf platform, instead of for the
native platform.

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


[Bug 674929] Review Request: sh-elf-binutils - Cross Compiling GNU binutils targeted at sh-elf

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


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

--- Comment #1 from John W. Linville linvi...@redhat.com 2011-02-03 13:54:30 
EST ---
The sh-elf cross toolchain is required for building the open-source firmware
for the carl9170 wireless driver.

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


[Bug 674930] Review Request: sh-elf-gcc - Cross Compiling GNU GCC targeted at sh-elf

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


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

--- Comment #1 from John W. Linville linvi...@redhat.com 2011-02-03 13:54:36 
EST ---
The sh-elf cross toolchain is required for building the open-source firmware
for the carl9170 wireless driver.

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


[Bug 668090] Review Request: rubygem-railties - Rails internals: application bootup, plugins, generators, and rake tasks.

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


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

--- Comment #15 from Minnikhanov minnikha...@gmail.com 2011-02-03 14:21:44 
EST ---
New Package SCM Request
===
Package Name: rubygem-railties
Short Description: Rails internals: application bootup, plugins, generators,
and rake tasks.
Owners: minn
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 646612] Rename review: drupal-date - drupal6-date

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


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

Paul W. Frields pfrie...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|sticks...@gmail.com

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


[Bug 668822] Review Request: rubygem-memcache-client - A Ruby library for accessing memcached

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


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

Minnikhanov minnikha...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

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


[Bug 668822] Review Request: rubygem-memcache-client - A Ruby library for accessing memcached

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


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

--- Comment #6 from Minnikhanov minnikha...@gmail.com 2011-02-03 14:32:49 EST 
---
Looks good.

-
This package (rubygem-memcache-client) is APPROVED by
minn
-

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


[Bug 673630] Review Request: habari - Lite blogging software

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


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

--- Comment #4 from Raghu Udiyar raghusidda...@gmail.com 2011-02-03 14:32:22 
EST ---
Hi Jakub,

Again, an Unofficial review, but now I'll go through all the review guidelines
:

-
 MUST Items :
-

- MUST: rpmlint must be run on every package. The output should be posted in
the review.

Good.

- MUST: The package must be named according to the Package Naming Guidelines.

Good.

- MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption on Package Naming Guidelines

Good.

- MUST: The package must meet the Packaging Guidelines .

== Can you consider moving the 'sed' commands to the %build section? -- just a
suggestion

== The latest change log has a typo, remove Initial build :

* Tue Feb 02 2011 Jakub Jedelsky jakub.jedel...@gmail.com - 0.6.6-2
- Initial build
- Add empty build section
- Change Summary and description
[..]

- MUST: The package must be licensed with a Fedora approved license and 
meet the Licensing Guidelines .

Good.

- MUST: The License field in the package spec file must match the actual
license.

== I think the license field earlier was correct. Why did you change it? 

From what I see, different components of the code are licensed under : ASL 2.0,
MIT and BSD. In addition you can put comments in the spec file to indicate the
license break down. Please See :
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Multiple_Licensing_Scenarios


- MUST: If (and only if) the source package includes the text of the 
license(s) in its own file, then that file, containing the text of the 
license(s) for the package must be included in %doc.

Good.

- MUST: The spec file must be written in American English.

Good.

- MUST: The spec file for the package MUST be legible.

Good.

- MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.

Good.

- MUST: The package must successfully compile and build into binary rpms on at
least one supported architecture.

Good.

- MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch.

Good.

- MUST: All build dependencies must be listed in BuildRequires, except for 
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.

Good.

- MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.


== Locale information is available in the Source : habari-0.6.6/system/locale

You have to use %find_lang to include these locales. Check this link :
http://fedoraproject.org/wiki/Packaging/Guidelines#Handling_Locale_Files


- MUST: Every binary RPM package which stores shared library files (not just
symlinks) in any of the dynamic linker's default paths, must call ldconfig in
%post and %postun.

NA.

- MUST: Packages must NOT bundle copies of system libraries.

Good.

- MUST: If the package is designed to be relocatable, the packager must state 
this fact in the request for review, along with the rationalization for 
relocation of that specific package. Without this, use of Prefix: /usr is 
considered a blocker.

NA.

- MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. Refer to the Guidelines for examples.

Good.

- MUST: A package must not contain any duplicate files in the %files listing.

Good.

- MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line.

Good.

- MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} ( or $RPM_BUILD_ROOT ).

Good.

- MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines .

Good.

- MUST: The package must contain code, or permissable content. This is
described in detail in the code vs. content section of Packaging Guidelines .

Good.

- MUST: Large documentation files should go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity)

== Some comments :

%files
%defattr(-,root,root,-)
%config(noreplace) %{_sysconfdir}/httpd/conf.d/habari.conf
%config(noreplace) %{_sysconfdir}/habari/config.php
%dir %{_datadir}/habari
%{_datadir}/habari/doc == Change this to %doc
%{_datadir}/habari/scripts
%{_datadir}/habari/system
%{_datadir}/habari/user
%{_datadir}/habari/3rdparty
%{_datadir}/habari/index.php
%{_datadir}/habari/config.php
%doc 

[Bug 668822] Review Request: rubygem-memcache-client - A Ruby library for accessing memcached

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


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

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

   What|Removed |Added

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

--- Comment #7 from Mohammed Morsi mmo...@redhat.com 2011-02-03 14:35:31 EST 
---
Cool stuff, thanks for the review.

New Package SCM Request
===
Package Name: rubygem-memcache-client
Short Description: A Ruby library for accessing memcached
Owners: mmorsi
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 643140] Review Request: autokey - A desktop automation utility

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


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

--- Comment #7 from Raghu Udiyar raghusidda...@gmail.com 2011-02-03 14:40:39 
EST ---

Unofficial review on : https://bugzilla.redhat.com/show_bug.cgi?id=673630


Some more changes to package :

SPEC : http://dl.dropbox.com/u/7899621/autokey/autokey.spec
SRPM : http://dl.dropbox.com/u/7899621/autokey/autokey-0.71.2-2.fc14.src.rpm


$ rpmlint SPECS/autokey.spec SRPMS/autokey-0.71.2-2.fc14.src.rpm
RPMS/noarch/autokey-*
SPECS/autokey.spec: W: invalid-url Source0:
http://autokey.googlecode.com/files/autokey_0.71.2.tar.gz HTTP Error 404: Not
Found
autokey.src: W: invalid-url Source0:
http://autokey.googlecode.com/files/autokey_0.71.2.tar.gz HTTP Error 404: Not
Found
autokey.noarch: W: no-documentation
autokey-common.noarch: E: explicit-lib-dependency python-xlib
autokey-common.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/daemon.py 0644L /usr/bin/env
autokey-common.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/common.py 0644L /usr/bin/env
autokey-common.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/evdev.py 0644L /usr/bin/env
autokey-common.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/service.py 0644L /usr/bin/env
autokey-common.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/nogui.py 0644L /usr/bin/env
autokey-common.noarch: W: service-default-enabled /etc/rc.d/init.d/autokey
autokey-common.noarch: W: service-default-enabled /etc/rc.d/init.d/autokey
autokey-common.noarch: E: subsys-not-used /etc/rc.d/init.d/autokey
autokey-common.noarch: W: incoherent-init-script-name autokey
('autokey-common', 'autokey-commond')
autokey-gtk.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/gtkapp.py 0644L /usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/windowfiltersettings.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/hotkeysettings.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/interfacesettings.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/generalsettings.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/centralwidget.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/folderpage.py 0644L /usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/settingswidget.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/dialogs.py 0644L /usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/scriptpage.py 0644L /usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/specialhotkeysettings.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtapp.py 0644L /usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/phrasepage.py 0644L /usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/enginesettings.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/abbrsettings.py 0644L
/usr/bin/env
autokey-qt.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/autokey/qtui/recorddialog.py 0644L
/usr/bin/env
5 packages and 1 specfiles checked; 23 errors, 6 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 177827] Review Request: python-ctypes - Advanced Foreign Function Interface for Python

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


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

--- Comment #19 from Kevin Fenzi ke...@tummy.com 2011-02-03 14:49:48 EST ---
Git done (by process-git-requests).

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


[Bug 668090] Review Request: rubygem-railties - Rails internals: application bootup, plugins, generators, and rake tasks.

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


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

--- Comment #16 from Minnikhanov minnikha...@gmail.com 2011-02-03 14:55:49 
EST ---
Spec URL: http://dl.dropbox.com/u/14118661/rubygem-railties.spec
SRPM URL:
http://dl.dropbox.com/u/14118661/rubygem-railties-3.0.3-7.fc14.src.rpm 

koji scratch build: green
http://koji.fedoraproject.org/koji/taskinfo?taskID=2760659 

(In reply to comment #11)
 Looks good. The only thing I'd add is to take the license out of the test
 tarball downloaded from the git repo and include it as its own source, or at
 least have a comment about it. nbd though.
 

+ Fixed.

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


[Bug 189678] Review Request: perl-List-MoreUtils - Provide the stuff missing in List::Util

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


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

--- Comment #9 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:00:37 EST ---
Right. This is in rhel6 optional. It cannot be in EPEL6.

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


[Bug 598688] Review Request: archivemount - FUSE based filesystem for mounting compressed archives

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


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

--- Comment #26 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:07:43 EST ---
Something seems to be going on with your account: 

AppError(PackageDBError, Unable to save all information for archivemount: Email
address ni...@nixpanic.net is not a valid bugzilla email address.  Either make
a bugzilla account with that email address or change your email address in the
Fedora Account System https://admin.fedoraproject.org/accounts/ to a valid
bugzilla email address and try again., extras=None)

Can you change your account to match whichever one you use in bugzilla please?

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


[Bug 646610] Rename review: drupal-views - drupal6-views

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #19 from Jon Ciesla l...@jcomserv.net 2011-02-03 15:10:28 EST ---
New Package SCM Request
===
Package Name: drupal6-views
Short Description: Provides a method for site designers to control content
presentation
Owners: limb
Branches: EL-5 EL-6
InitialCC:

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


[Bug 646611] Rename review: drupal-cck - drupal6-cck

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #20 from Jon Ciesla l...@jcomserv.net 2011-02-03 15:11:21 EST ---


New Package SCM Request
===
Package Name: drupal6-cck
Short Description: Allows you to add custom fields to nodes using a web browser
Owners: limb
Branches: EL-5 EL-6
InitialCC:

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


[Bug 657520] Review Request: perl-Array-Utils - Small utils for array manipulation

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


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

--- Comment #9 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:17:59 EST ---
Git done (by process-git-requests).

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


[Bug 611048] Review Request: python-amqplib - Client library for AMQP

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


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

--- Comment #15 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:14:26 EST ---
Is user 'bar' really one you want here? 

Also, does the fedora maintainer not wish to maintain in 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 666726] Review Request: Amide - A Medical Image Data Examiner:

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


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

--- Comment #11 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:22:34 EST ---
Also, is the package amide or Amide? The summary and the package name don't
match. ;(

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


[Bug 670701] Review Request: ghc-ansi-terminal - Haskell ansi-terminal library

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-02-03 
15:19:56 EST ---
ghc-ansi-terminal-0.5.5-1.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 664817] Review Request: perl-HTML-Selector-XPath - CSS Selector to XPath compiler

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #15 from Fedora Update System upda...@fedoraproject.org 
2011-02-03 15:22:04 EST ---
perl-HTML-Selector-XPath-0.04-1.fc14 has been pushed to the Fedora 14 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update perl-HTML-Selector-XPath'.  You
can provide feedback for this update here:
https://admin.fedoraproject.org/updates/perl-HTML-Selector-XPath-0.04-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 664911] Review Request: perl-Test-WWW-Mechanize-PSGI - Test PSGI programs using WWW::Mechanize

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-02-03 
15:19:51 EST ---
perl-Test-WWW-Mechanize-PSGI-0.35-1.fc13 has been pushed to the Fedora 13
testing repository.  If problems still persist, please make note of it in this
bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update perl-Test-WWW-Mechanize-PSGI'. 
You can provide feedback for this update here:
https://admin.fedoraproject.org/updates/perl-Test-WWW-Mechanize-PSGI-0.35-1.fc13

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


[Bug 665995] Review Request: fmit - Free Music Instrument Tuner

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


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

--- Comment #30 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:20:27 EST ---
Try now?

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


[Bug 670701] Review Request: ghc-ansi-terminal - Haskell ansi-terminal library

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||ghc-ansi-terminal-0.5.5-1.f
   ||c14
 Resolution||ERRATA
Last Closed||2011-02-03 15:20:01

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


[Bug 673485] Review Request: libldb - A schema-less, ldap like, API and database

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


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

--- Comment #2 from Matthew Barnes mbar...@redhat.com 2011-02-03 15:20:12 EST 
---
rpmlint output:

libldb.i686: W: spelling-error Summary(en_US) ldap - lap, dap, leap
libldb.i686: W: spelling-error %description -l en_US tdb - db, tab, tub

Obviously bogus.

libldb.i686: W: shared-lib-calls-exit /usr/lib/libldb.so.0.9.22 exit@GLIBC_2.0

I assume this is okay?  Not sure what exit() call it's complaining about.

pyldb.i686: W: private-shared-object-provides
/usr/lib/python2.7/site-packages/ldb.so ldb.so

Python C modules = bogus.

libldb.i686: W: no-documentation
pyldb.i686: W: no-documentation
pyldb-devel.i686: W: no-documentation

Didn't see any relevant documentation files in the tarball.  Are there any
license or README files from Samba that would be appropriate to include?

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


[Bug 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

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


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

--- Comment #84 from Fedora Update System upda...@fedoraproject.org 
2011-02-03 15:27:24 EST ---
kde-plasma-translatoid-1.30-2.svn01092011.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 670558] Review Request: ape - A tool for generating atomic pseudopotentials within a DFT framework

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-02-03 
15:27:06 EST ---
ape-1.1.0-2.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 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

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


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

--- Comment #83 from Fedora Update System upda...@fedoraproject.org 
2011-02-03 15:24:17 EST ---
kde-plasma-translatoid-1.30-2.svn01092011.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 670558] Review Request: ape - A tool for generating atomic pseudopotentials within a DFT framework

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


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

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

   What|Removed |Added

   Fixed In Version|ape-1.1.0-2.fc13|ape-1.1.0-2.fc14

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


[Bug 670558] Review Request: ape - A tool for generating atomic pseudopotentials within a DFT framework

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


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

Bug 670558 depends on bug 670541, which changed state.

Bug 670541 Summary: Review Request: libxc - Library of exchange and correlation 
functionals to be used in DFT codes
https://bugzilla.redhat.com/show_bug.cgi?id=670541

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-02-03 
15:26:51 EST ---
ape-1.1.0-2.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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby

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


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

--- Comment #16 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:28:44 EST ---
Git done (by process-git-requests).

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


[Bug 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

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


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

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

   What|Removed |Added

   Fixed In Version|kde-plasma-translatoid-1.30 |kde-plasma-translatoid-1.30
   |-2.svn01092011.fc14 |-2.svn01092011.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 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

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


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

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

   What|Removed |Added

   Fixed In Version|kde-plasma-translatoid-1.30 |kde-plasma-translatoid-1.30
   |-1.fc14 |-2.svn01092011.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 670558] Review Request: ape - A tool for generating atomic pseudopotentials within a DFT framework

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||ape-1.1.0-2.fc13
 Resolution||ERRATA
Last Closed||2011-02-03 15:26:56

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


[Bug 668090] Review Request: rubygem-railties - Rails internals: application bootup, plugins, generators, and rake tasks.

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


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

--- Comment #17 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:36:16 EST ---
Git done (by process-git-requests).

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


[Bug 666726] Review Request: amide - A Medical Image Data Examiner:

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


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

--- Comment #13 from Susmit thinklinux@gmail.com 2011-02-03 15:37:58 EST 
---
New Package SCM Request
===
Package Name: amide
Short Description: A Medical Image Data Examiner
Owners: susmit
Branches: f14
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 666726] Review Request: amide - A Medical Image Data Examiner:

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


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

--- Comment #12 from Susmit thinklinux@gmail.com 2011-02-03 15:37:45 EST 
---
I have updated the wiki with the information. It was no where mentioned in the
wiki that these fields are case sensitive.

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


[Bug 611048] Review Request: python-amqplib - Client library for AMQP

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


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

--- Comment #16 from Brian Pepple bdpep...@gmail.com 2011-02-03 15:36:17 EST 
---
(In reply to comment #15)
 Is user 'bar' really one you want here? 
 
 Also, does the fedora maintainer not wish to maintain in epel?

oops, bar was copy/paste error.

This is a dep for python-carrot, and the maintainer has no interest/time for
the epel branch.

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


[Bug 666726] Review Request: amide - A Medical Image Data Examiner:

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


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

Susmit thinklinux@gmail.com changed:

   What|Removed |Added

Summary|Review Request: Amide - A   |Review Request: amide - A
   |Medical Image Data  |Medical Image Data
   |Examiner:   |Examiner:

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


[Bug 668822] Review Request: rubygem-memcache-client - A Ruby library for accessing memcached

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


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

--- Comment #8 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:39:48 EST ---
Git done (by process-git-requests).

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


[Bug 668823] Review Request: rubygem-text-format - Text::Format formats fixed-width text nicely

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


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

--- Comment #9 from Kevin Fenzi ke...@tummy.com 2011-02-03 15:43:38 EST ---
Git done (by process-git-requests).

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


[Bug 622904] Review Request: rubygem-state_machine - Adds support for creating state machines for attributes on any Ruby class

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


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

--- Comment #11 from Guillermo Gómez guillermo.go...@gmail.com 2011-02-03 
15:42:32 EST ---
Package Change Request
==
Package Name: rubygem-state_machine
New Branches: el5 el6
Owners: gomix

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

[Bug 474549] Review Request: ca-cacert.org - CAcert.org CA root certificates

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


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

--- Comment #39 from Rod Montgomery r...@thecomplex.com 2011-02-03 15:40:44 
EST ---
(In reply to comment #37)
 A priori, I don't need CAcert's permission to rely on (= take on risk or
 liability based on) their certificates.  There is no law, and I have agreed to
 no contract, that would forbid me to do this.  A simple statement that they
 withhold permission is meaningless; there is no permission to withhold.  It
 seems more likely, especially in view of the policy discussion you linked, 
 that
 the intended interpretation is to make non-reliance a condition of the
 copyright license.  (If this is not the case, CAcert should issue an official
 clarification.)

Yes, in further reading at
https://lists.cacert.org/wws/arc/cacert-policy/2010-06/msg00062.html, I found
Ian Grigg's opinion that non-related persons should be banned from relying on
the certificates. His comment was made in the context of the NRP-DaL (since
removed in favor of the RDL), but the intention carries on in the language of
the RDL.

 Companies rely (= take on risk and liability based on) the correct
 operation of GPL software every day.  They just can't sue the copyright 
 holders
 if it breaks.

Companies choose to take those risks and liabilities of their own volition, not
as a result of any assurance, warranty, or claim in the GPL. IANAL, but it
seems that the first portion of the RDL Disclaimer would sever any CACert
liability from relying inappropriately. CACert's concern over possible
litigation seems to be the driving concern behind the reliance language in the
second portion, but I do not see why the warranty disclaimer is insufficient
for CACert's concern.

I cannot speak for CACert or Fedora, but I am an interested user/participant of
both organizations. Thanks for the quick and thoughtful reply.

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


[Bug 673485] Review Request: libldb - A schema-less, ldap like, API and database

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


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

--- Comment #3 from Matthew Barnes mbar...@redhat.com 2011-02-03 15:44:35 EST 
---
* MUST: The License field in the package spec file must match the actual
license.

Seems to be the case from the random sample of source files I looked at, but a
LICENSE or COPYING file in the top-level directory would be more helpful.

* MUST: Packages must NOT bundle copies of system libraries.

The /lib directory contains several libraries which are packaged separately in
Fedora, but they don't appear to be used during building nor are they
installed. Just wanted to confirm that this is true.


Other:

* Package builds fine under mock in a Rawhide environment.

* The BuildRoot tag isn't needed in current Fedora releases, but I see
  conditionals for older Fedora / RHEL releases.  Not sure which releases
  you're targeting.  Might be needed for older releases, just wanted to
  mention it.

* In the tools subpackage I see:

Requires: libldb = %{version}-%{release}

  Should that be '=' instead of '='?  It's '=' in all other subpackages.


Other than these nitpicky things, it looks good to me.

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