[Bug 672811] Review Request: perl-Perl-Critic-Compatibility - Perl::Critic policies for compatibility with Perl versions

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


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

Ruediger Landmann  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||r.landm...@redhat.com
 AssignedTo|nob...@fedoraproject.org|r.landm...@redhat.com
   Flag||fedora-review+

--- Comment #1 from Ruediger Landmann  2011-03-23 
02:38:15 EDT ---
Thanks Petr; all good here.

APPROVED

 - = N/A
 / = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [/] Rpmlint output is clean:
$ rpmlint SPECS/perl-Perl-Critic-Compatibility.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
$ rpmlint SRPMS/perl-Perl-Critic-Compatibility-1.001-1.fc14.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint RPMS/noarch/perl-Perl-Critic-Compatibility-1.001-1.fc14.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

 [/] Package is named according to the Package Naming Guidelines.
 [/] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [/] Package meets the Packaging Guidelines including the Language specific
items
 [/] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [/] License field in the package spec file matches the actual license.
 License type: GPL+ or Artistic
 [-] 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 is included in %doc.
 [/] Spec file is legible and written in American English.
 [/] Sources used to build the package matches the upstream source, as provided
in the spec URL.
$ md5sum SOURCES/Perl-Critic-Compatibility-1.001.tar.gz 
30d38d6106380333d12159604281cd62 
SOURCES/Perl-Critic-Compatibility-1.001.tar.gz
$ md5sum ~/Download/Perl-Critic-Compatibility-1.001.tar.gz 
30d38d6106380333d12159604281cd62 
/home/rlandmann/Download/Perl-Critic-Compatibility-1.001.tar.gz

 [/] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested: http://koji.fedoraproject.org/koji/taskinfo?taskID=2935396
 [/] Package is not known to require ExcludeArch
 [/] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly (with the %find_lang macro)
 [-] ldconfig called in %post and %postun if required.
 [/] Package does not bundle copies of system libraries
 [/] Package is not relocatable.
 [/] Package must own all directories that it creates.
 [/] Package does not contain duplicates in %files.
 [/] Permissions on files are set properly
 [/] %files section includes a %defattr(...) line
 [/] Package consistently uses macros.
 [-] Large documentation files are in a -doc subpackage, if required.
 [/] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -static subpackage, if present.
 [-] Development .so files in -devel subpackage, if present.
 [-] -devel packages require base package with full versioning.
 [/] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [/] Package does not own files or directories owned by other packages.
 [/] Filenames are valid UTF-8

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


[Bug 573917] Review Request: perl-NetPacket-SpanningTree - Assemble and disassemble IEEE 802.1D Spanning Tree protocol packets

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


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

--- Comment #11 from Ruediger Landmann  2011-03-23 
02:27:53 EDT ---
Actually, I take back comment #10 -- I've been doing some more exploring and
realise that I was asking for something beyond Fedora's normal requirements+.
Include the GPL text if you want to, but I'm not going to push the point. Sorry
for the drama.

One last thing I noticed however: GPL aside, the developer agreed to Artistic,
but in the License line, you put Artistic 2.0, which is something quite
different. Please drop the "2.0" and we're done here. 

Everything else checks out:

 - = N/A
 / = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [/] Rpmlint output is clean:
$ rpmlint SPECS/perl-NetPacket-SpanningTree.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
$ rpmlint SRPMS/perl-NetPacket-SpanningTree-0.01-2.fc14.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint RPMS/noarch/perl-NetPacket-SpanningTree-0.01-2.fc14.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

 [/] Package is named according to the Package Naming Guidelines.
 [/] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [/] Package meets the Packaging Guidelines including the Language specific
items
 [!] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [/] License field in the package spec file matches the actual license.
 License type: GPL+ or Artistic 2.0

 [/] 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 is included in %doc.
  Email from copyright holder included

 [/] Spec file is legible and written in American English.
 [/] Sources used to build the package matches the upstream source, as provided
in the spec URL.
$ md5sum SOURCES/NetPacket-SpanningTree-0.01.tar.gz 
bd657ce34022611b22f1882413a74184  SOURCES/NetPacket-SpanningTree-0.01.tar.gz
$md5sum ~/Download/NetPacket-SpanningTree-0.01.tar.gz 
bd657ce34022611b22f1882413a74184 
/home/rlandmann/Download/NetPacket-SpanningTree-0.01.tar.gz

 [/] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested: http://koji.fedoraproject.org/koji/taskinfo?taskID=2935377

 [/] Package is not known to require ExcludeArch
 [/] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly (with the %find_lang macro)
 [-] ldconfig called in %post and %postun if required.
 [/] Package does not bundle copies of system libraries
 [/] Package is not relocatable.
 [/] Package must own all directories that it creates.
 [/] Package does not contain duplicates in %files.
 [/] Permissions on files are set properly
 [/] %files section includes a %defattr(...) line
 [/] Package consistently uses macros.
 [-] Large documentation files are in a -doc subpackage, if required.
 [/] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -static subpackage, if present.
 [-] Development .so files in -devel subpackage, if present.
 [-] -devel packages require base package with full versioning.
 [/] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [/] Package does not own files or directories owned by other packages.
 [/] Filenames are valid UTF-8

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


[Bug 688867] Review Request: bharati-m17n - An Input Method based on InScript layout

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


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

sandeep shedmake  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from sandeep shedmake  2011-03-23 02:18:51 
EDT ---
Thanks for the review.

New Package SCM Request
===
Package Name: bharati-m17n
Short Description: An Input Method based on InScript layout
Owners: sandeeps
Branches: f14, f15, devel
InitialCC: i18n-team

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


[Bug 688786] Review Request: perl-NTLM - perl module for NTLM authentication

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


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

Ruediger Landmann  changed:

   What|Removed |Added

   Flag||needinfo?(nb@fedoraproject.
   ||org)

--- Comment #6 from Ruediger Landmann  2011-03-23 
01:56:10 EDT ---
Sorry Nick; I've done a bit more digging and conclude that I've been a bit too
paranoid here. 

It turns out, however, that although you don't need to explicitly Require
perl(Digest::HMAC_MD5), you *do* need to BR: it. 

If you fix that, I'm happy to approve the 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 690038] New: Review Request: ompl - The Open Motion Planning Library

2011-03-22 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: ompl - The Open Motion Planning Library

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

   Summary: Review Request: ompl - The Open Motion Planning
Library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: richmat...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://rmattes.fedorapeople.org/RPMS/ompl/ompl.spec
SRPM URL: http://rmattes.fedorapeople.org/RPMS/ompl/ompl-0.9.2-1.fc15.src.rpm
Description: 
The Open Motion Planning Library (OMPL) consists of many state-of-the-art
sampling-based motion planning algorithms. OMPL itself does not contain
any code related to, e.g., collision checking or visualization. This is
a deliberate design choice, so that OMPL is not tied to a particular
collision checker or visualization front end.

Scratch build: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=2935340

rpmlint:
$ rpmlint ompl.spec ../RPMS/x86_64/ompl*
ompl.x86_64: E: invalid-soname /usr/lib64/libompl.so libompl.so
3 packages and 1 specfiles checked; 1 errors, 0 warnings.

Upstream doesn't include a soversion.  It looks like they recently removed it
(going from 0.9.1 to 0.9.2,) I will ask why.

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


[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

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


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

--- Comment #30 from Rosen Diankov  2011-03-23 
01:19:21 EDT ---
hi tim,

Ok, i added the crlibm check back in (r2188).

You should also be able to set the path openravepy installs in by setting
OPENRAVEPY_INSTALL_DIR via the command line.


As for sympy, the biggest problem I reported is Issue 2081:

http://code.google.com/p/sympy/issues/detail?can=2&q=2081&colspec=ID%20Type%20Status%20Priority%20Milestone%20Owner%20Summary%20Stars&id=2081

Basically, it is possible for sympy to give you wrong equations, which would
make the openrave ikfast solver return wrong solutions.  They are still
wondering how to fix it so that everyone's happy.

We're looking at the problem from two different angles here: I need to make
openrave 100% reliable, you need to reduce unnecessary dependencies so
management is easier.

I committed a temporary compromise (r2189):

If you limit openrave to only sympy 0.6.7, I might have come up with a
workaround where you don't have to install a local sympy (ie executing ikfast
will replace the wrong function in memory).

We can probably support more sympy versions, but they will require IK to be
re-tested. There'a an openrave testing server now, so this might be possible in
the future:

http://www.openrave.org/testing/job/openrave/

If you ever decide to update the sympy version in fedora core, please email 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


[Bug 684112] Review Request: hindi-typing-booster - Hindi Typing Booster Table IMEngine for ibus

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


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version||hindi-typing-booster-0.0.1-
   ||1.fc15
 Resolution|RAWHIDE |ERRATA

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


[Bug 675478] Review Request: gprbuild - Ada project builder

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


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version||gprbuild-2010-7.fc15
 Resolution|NEXTRELEASE |ERRATA

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


[Bug 684112] Review Request: hindi-typing-booster - Hindi Typing Booster Table IMEngine for ibus

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


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

--- Comment #10 from Fedora Update System  
2011-03-23 01:06:09 EDT ---
hindi-typing-booster-0.0.1-1.fc15 has been pushed to the Fedora 15 stable
repository.

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


[Bug 675478] Review Request: gprbuild - Ada project builder

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


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

--- Comment #23 from Fedora Update System  
2011-03-23 01:06:23 EDT ---
gprbuild-2010-7.fc15 has been pushed to the Fedora 15 stable repository.

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


[Bug 683725] Review Request: marathi-typing-booster - Marathi Typing Booster Table IMEngine for ibus

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


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version||marathi-typing-booster-0.0.
   ||1-1.fc15
 Resolution|RAWHIDE |ERRATA

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


[Bug 688264] Review Request: perl-Test-CPAN-Meta-YAML - Validate a META.yml file within a CPAN distribution

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


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

--- Comment #9 from Fedora Update System  2011-03-23 
01:03:43 EDT ---
perl-Test-CPAN-Meta-YAML-0.17-2.fc15 has been pushed to the Fedora 15 stable
repository.

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


[Bug 683725] Review Request: marathi-typing-booster - Marathi Typing Booster Table IMEngine for ibus

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


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

--- Comment #11 from Fedora Update System  
2011-03-23 01:02:41 EDT ---
marathi-typing-booster-0.0.1-1.fc15 has been pushed to the Fedora 15 stable
repository.

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


[Bug 684915] Review Request: php-pecl-igbinary - Replacement for the standard PHP serializer

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||php-pecl-igbinary-1.1.1-1.f
   ||c15
 Resolution||ERRATA
Last Closed||2011-03-23 00:58:23

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


[Bug 684915] Review Request: php-pecl-igbinary - Replacement for the standard PHP serializer

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


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

--- Comment #7 from Fedora Update System  2011-03-23 
00:58:18 EDT ---
php-pecl-igbinary-1.1.1-1.fc15 has been pushed to the Fedora 15 stable
repository.

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


[Bug 676308] Review Request: rubygem-net-scp - A pure Ruby implementation of the SCP client protocol

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #14 from Fedora Update System  
2011-03-23 00:44:33 EDT ---
rubygem-net-scp-1.0.4-2.fc15 has been pushed to the Fedora 15 testing
repository.

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


[Bug 673030] Review Request: perl-Locale-US - Two letter codes for state identification in the United States and vice versa

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #11 from Fedora Update System  
2011-03-23 00:44:38 EDT ---
perl-Locale-US-1.2-2.fc15 has been pushed to the Fedora 15 testing repository.

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


[Bug 688659] Review Request: thunarx-python - Python bindings for the Thunar Extension Framework

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #12 from Fedora Update System  
2011-03-23 00:44:25 EDT ---
thunarx-python-0.2.3-3.fc15 has been pushed to the Fedora 15 testing
repository.

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


[Bug 682353] Review Request: wmtop - Mini graphical version of the CPU monitoring utility top

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #15 from Fedora Update System  
2011-03-23 00:44:54 EDT ---
wmtop-0.84-3.fc15 has been pushed to the Fedora 15 testing repository.

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


[Bug 669726] Review Request: rubygem-escape - HTML/URI/shell escaping utilities

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System  2011-03-23 
00:45:05 EDT ---
rubygem-escape-0.0.4-2.fc15 has been pushed to the Fedora 15 testing
repository.

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


[Bug 690026] New: Review Request: libfreenect - Open device driver for the Kinect

2011-03-22 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: libfreenect - Open device driver for the Kinect

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

   Summary: Review Request: libfreenect - Open device driver for
the Kinect
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: richmat...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://rmattes.fedorapeople.org/RPMS/libfreenect/libfreenect.spec
SRPM URL:
http://rmattes.fedorapeople.org/RPMS/libfreenect/libfreenect-0-0.1.687b2da5git.fc15.src.rpm
Description: 
libfreenect is a free and open source library that provides access to the
Kinect device.  Currently, the library supports the RGB webcam, the depth
image, the LED, and the tilt motor.

Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2935127

Rpmlint:
$ rpmlint libfreenect.spec ../RPMS/x86_64/libfreenect*
libfreenect.spec: W: invalid-url Source0: libfreenect-687b2da5.tar.bz2
 - OK, taken from a git snapshot
libfreenect.x86_64: W: spelling-error %description -l en_US webcam -> web cam,
web-cam, became
 - OK, "webcam" is a common spelling
libfreenect.x86_64: W: no-manual-page-for-binary freenect-glview
libfreenect.x86_64: W: no-manual-page-for-binary freenect-glpclview
libfreenect.x86_64: W: no-manual-page-for-binary freenect-tiltdemo
libfreenect.x86_64: W: no-manual-page-for-binary freenect-hiview
libfreenect.x86_64: W: no-manual-page-for-binary freenect-cppview
- OK, project does not provide manpages
libfreenect-python.x86_64: W: private-shared-object-provides
/usr/lib64/python2.7/site-packages/freenect.so freenect.so()(64bit)
- I think this is OK, it's part of the python bindings
libfreenect-python.x86_64: W: no-documentation
libfreenect-static.x86_64: W: no-documentation
- Both depend on packages with documentation
5 packages and 1 specfiles checked; 0 errors, 10 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 690025] New: Review Request: cminpack - Solver for nonlinear equations and nonlinear least squares problems

2011-03-22 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: cminpack - Solver for nonlinear equations and 
nonlinear least squares problems

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

   Summary: Review Request: cminpack - Solver for nonlinear
equations and nonlinear least squares problems
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: richmat...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://rmattes.fedorapeople.org/RPMS/cminpack/cminpack.spec
SRPM URL:
http://rmattes.fedorapeople.org/RPMS/cminpack/cminpack-1.1.3-1.fc15.src.rpm

Description: 
cminpack is an ISO C99 implementation of the FORTRAN Minpack solver package.
It is fully re-entrant and thread-safe.

Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2935119

Rpmlint:
$ rpmlint cminpack.spec ../RPMS/x86_64/cminpack*
3 packages and 1 specfiles checked; 0 errors, 0 warnings.

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


[Bug 635511] Review Request: assimp - Library to import various 3D model formats into applications

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


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

--- Comment #30 from Rich Mattes  2011-03-23 00:02:17 EDT 
---
I've emailed the assimp upstream about possibly changing the names of their
internal functions (applying spot's assimp patch) so that we can more easily
build against the external irrXML.  The thread should show up soon at
https://sourceforge.net/mailarchive/forum.php?forum_name=assimp-discussions

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


[Bug 670127] Review Request: the-board - A space for placing daily records in your GNOME desktop

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


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

Peter Robinson  changed:

   What|Removed |Added

 CC||dw...@infradead.org,
   ||pbrobin...@gmail.com
  Component|Package Review  |0x

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


[Bug 678855] Review Request: rpyc - A Transparent, Symmetrical Python Library for Distributed-Computing

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


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

--- Comment #3 from David Lutterkort  2011-03-22 20:50:47 
EDT ---
APPROVED

Please follow
http://fedoraproject.org/wiki/Package_SCM_admin_requests#New_Packages and
import the package. Close this bug as RAWHIDE once it's been successfully
imported and built.

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


[Bug 573917] Review Request: perl-NetPacket-SpanningTree - Assemble and disassemble IEEE 802.1D Spanning Tree protocol packets

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


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

--- Comment #10 from Ruediger Landmann  2011-03-22 
19:23:27 EDT ---
(In reply to comment #7)

> I was watching that discussion, however my understanding is that if upstream
> does not supply it, I will be liable if I will add license text and it will be
> not that one under which upstream released that package. 

The risk here is including the *wrong* license text.[0] Which is why it's so
important to show upstream the license text that you intend to include in the
package, obtain their consent to add that text, and include that consent (with
its headers) in the package itself. 

> From http://fedoraproject.org/wiki/Packaging:LicensingGuidelines (and similar
> text in reviewguidelines):
> 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. If the source package does not include the text of
> the license(s), the packager should contact upstream and encourage them to
> correct this mistake. 
> 
> Therefore I don't see any "must" requirement to include copy of GPL by myself
> if upstream decide that they will not bundle it with tarball.

There is no "must" here: if upstream does not want to (or cannot) include it in
the tarball, the advice on legal list suggests that we could probably ship the
package safely and rely on the copy of the GPL provided with Perl itself. 

However, best practice (as discussed in that thread) is to include a copy of
the GPL with GPL-licensed packages that we ship. In a best case, this is
included by upstream itself; less good is to include the license text with the
explicit consent of upstream; even less good is to ship without the license
text. 

Because it's not a "must", I will not block the review on this point, but I
would strongly prefer it if you asked upstream if you can include a copy of the
license if he cannot or will not. He seems to be responsive and friendly --
there's nothing to lose here.

If you respond here and tell me that you refuse to ask upstream about this, I
will note that in the same legal list thread (just for the purpose of any
future reference) and I will continue the review as normal.

Sorry to be a pain -- I just think we should always aim high! :)

Cheers
Rudi



[0] 'Now, the reason we simply do not have a policy that says "When a copy of
the license text is missing, you must add it" is because there is the
possibility that you, the Fedora packager, gets the license wrong, and
by including a copy of the incorrect license text, put yourself at
potential legal risk when the copyright holder claims you're
distributing their software under terms they never gave you permission
to use.'

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


[Bug 554101] Review Request: surf - Simple web browser

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


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

Christoph Wickert  changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|INSUFFICIENT_DATA   |
   Flag||needinfo?(cassmodiah@fedora
   ||project.org)

--- Comment #30 from Christoph Wickert  2011-03-22 
18:49:44 EDT ---
The package has not been renamed and re-reviewed yet. 

Simon, please rename the package and submit a new review. Post the bug number
here, clear the NEEDINFO flag and close this bug again. Thanks!

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


[Bug 625592] Review Request: PyCAM -Open Source CAM - Toolpath Generation for 3-Axis CNC machining

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


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

Dominic Hopf  changed:

   What|Removed |Added

   Flag||needinfo?(fabian.kanngiesse
   ||r...@yahoo.de)

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


[Bug 635511] Review Request: assimp - Library to import various 3D model formats into applications

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


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

--- Comment #29 from Tim Niemueller  2011-03-22 18:49:33 
EDT ---
I'd vote for speeding this up by including the patches in the Fedora packages
until they are upstreamed.

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


[Bug 554101] Review Request: surf - Simple web browser

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


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

Dominic Hopf  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||INSUFFICIENT_DATA
   Flag|needinfo?(cassmodiah@fedora |
   |project.org)|
Last Closed|2010-01-19 14:44:00 |2011-03-22 18:28:28

--- Comment #29 from Dominic Hopf  2011-03-22 
18:28:28 EDT ---
The bug is NEEDINFO since 2010-12-11 18:11:09 EST (see history of this bug). I
think it is valid to close the issue. Feel free to reopen in case of any
concerns.

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


[Bug 573917] Review Request: perl-NetPacket-SpanningTree - Assemble and disassemble IEEE 802.1D Spanning Tree protocol packets

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


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

--- Comment #9 from Ruediger Landmann  2011-03-22 
18:20:57 EDT ---
(In reply to comment #6)

> Legally, it's questionable, because a Fedora packager added license file isn't
> legally binding to upstream (only upstream can do this).

Right -- which is why we need to seek upstream's consent to add the file (or,
better, ask them to do it themselves for us). 

> > From the email, it sounds like it's unlikely that upstream will generate a 
> > new
> > tarball for us, so instead could you please contact them again and:
> > 
> > * attach a copy of the GPLv1 (not any other version) to the email
> Are you sure? This GPLv1 is dead for more than a decade.

I was initially surprised by this too; but Spot's rationale makes sense to me:

"(yes, the GPLv1, not a later version, because the perl licensing
they granted is GPLv1 or later. If upstream adds a copy of GPLv2 or
GPLv3, that is sufficient for us to distribute to meet this
technicality, but if we're going to do it, we're going to do it
right.)"[0]

[0] http://lists.fedoraproject.org/pipermail/legal/2011-March/001584.html

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


[Bug 539693] Review Request: plowshare - command-line downloader/uploader for some of the most popular file-sharing websites

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


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

--- Comment #37 from Fedora Update System  
2011-03-22 18:17:45 EDT ---
plowshare-0.9.4-0.5.svn1391.el6 has been submitted as an update for Fedora EPEL
6.
https://admin.fedoraproject.org/updates/plowshare-0.9.4-0.5.svn1391.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 539693] Review Request: plowshare - command-line downloader/uploader for some of the most popular file-sharing websites

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


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

--- Comment #36 from Fedora Update System  
2011-03-22 18:01:40 EDT ---
plowshare-0.9.4-0.5.svn1391.el5 has been submitted as an update for Fedora EPEL
5.
https://admin.fedoraproject.org/updates/plowshare-0.9.4-0.5.svn1391.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 539693] Review Request: plowshare - command-line downloader/uploader for some of the most popular file-sharing websites

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


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

--- Comment #35 from Fedora Update System  
2011-03-22 17:51:36 EDT ---
plowshare-0.9.4-0.5.svn1391.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/plowshare-0.9.4-0.5.svn1391.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 539693] Review Request: plowshare - command-line downloader/uploader for some of the most popular file-sharing websites

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 539693] Review Request: plowshare - command-line downloader/uploader for some of the most popular file-sharing websites

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


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

--- Comment #34 from Fedora Update System  
2011-03-22 17:46:06 EDT ---
plowshare-0.9.4-0.5.svn1391.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/plowshare-0.9.4-0.5.svn1391.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 689629] Review Request: gnome-tweak-tool - A tool for advanced GNOME 3 customization

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


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

--- Comment #11 from Jason Tibbitts  2011-03-22 17:43:45 EDT 
---
Git done (by process-git-requests).

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


[Bug 689629] Review Request: gnome-tweak-tool - A tool for advanced GNOME 3 customization

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


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

Michel Alexandre Salim  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #10 from Michel Alexandre Salim  
2011-03-22 17:38:44 EDT ---
Thanks to all the reviewers for their inputs.


New Package SCM Request
===
Package Name: gnome-tweak-tool
Short Description: A tool for advanced GNOME 3 customization
Owners: salimma
Branches: F-15
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 689961] New: Review Request: lego-udevrules - Provide access to LEGO robots and controller boards

2011-03-22 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: lego-udevrules - Provide access to LEGO robots and 
controller boards

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

   Summary: Review Request: lego-udevrules - Provide access to
LEGO robots and controller boards
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mar...@laptop.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://dev.laptop.org/git/packages/robots-udevrules/tree/
SRPM URL:
http://dev.laptop.org/~martin/public_rpms/lego-udevrules-1.0-1.fc14.src.rpm
Description: udev rules granting access to LEGO NXT and WeDo robots.

This trivial package will allow the existing nxt_python and nbc packages
coexist. OLPC's current efforts will probably bring about a few more packages
that control the same robots/boards.

I am working at the same time on revisions to nbc (with its maintainer) and I
am current maintainer of nxt_python.

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


[Bug 688408] Review Request: xfce4-cpufreq-plugin - CPU frequency scaling plugin for the Xfce4 panel

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


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

--- Comment #14 from Christoph Wickert  2011-03-22 
16:46:57 EDT ---
(In reply to comment #13)
> (In reply to comment #12)
> > * The plugin does not work with Fedora's default setup. Scaling is only 
> > allowed
> > for root or through PolicyKit, but the plugin doesn't support the latter. 
> > How
> > did you make the plugin work?
> 
> I missed that, I will work with upstream to add polkit support ASAP.

Thanks, but does the plugin work for you? I tried it and it didn't and as long
as it does not work, it shouldn't be in the repo.

> > * There usually is no need for changing frequencies or the governor, 
> > ondemand
> > will handle this best.
> 
> I don't agree, under certain conditions I want to force the cpu to use a
> certain frequency to keep my laptop cool.

I don't disagree, but we were told so by the kernel and power management
people.

> > Some notes on the package:
> > 1) The description is formatted a little strange
> 
> Define "strange".

The indented lines look strange, better use * as bullets or markdown (will give
you proper bullet points in packagekit)

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


[Bug 682353] Review Request: wmtop - Mini graphical version of the CPU monitoring utility top

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


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

--- Comment #13 from Fedora Update System  
2011-03-22 16:42:09 EDT ---
wmtop-0.84-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/wmtop-0.84-3.fc15

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


[Bug 682353] Review Request: wmtop - Mini graphical version of the CPU monitoring utility top

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 682353] Review Request: wmtop - Mini graphical version of the CPU monitoring utility top

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


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

--- Comment #14 from Fedora Update System  
2011-03-22 16:42:18 EDT ---
wmtop-0.84-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/wmtop-0.84-3.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 627637] Review Request: qroneko - A front end of crontab application

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


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|qroneko-0.5.4-4.fc15|qroneko-0.5.4-4.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 627637] Review Request: qroneko - A front end of crontab application

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


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

--- Comment #21 from Fedora Update System  
2011-03-22 14:55:22 EDT ---
qroneko-0.5.4-4.fc13 has been pushed to the Fedora 13 stable repository.

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


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

2011-03-22 Thread bugzilla
Please do not reply directly 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 #29 from Per Inge Mathisen  2011-03-22 
14:33:34 EDT ---
Now it works fine here, as well. Great work.

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


[Bug 674188] Review Request: libzeitgeist - Library to access zeitgeist; needed by synapse

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


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

--- Comment #24 from Mamoru Tasaka  2011-03-22 
13:54:19 EDT ---
By the way for 0.3.6-1:

* BR
  - Why is ruby-gio2-devel listed as BR?

(In reply to comment #21)
> (In reply to comment #20)
> >   - And it is very likely that "Requires: zeitgeist" should be
> > %{_isa} specific.
> 
> I think not; since zeitgeist is .noarch ;)

Ah, thank you.

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


[Bug 597307] Review Request: fastx_toolkit - Tools to process short-reads FASTA/FASTQ files

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


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

--- Comment #18 from Jason Tibbitts  2011-03-22 13:51:58 EDT 
---
Git done (by process-git-requests).

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


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

2011-03-22 Thread bugzilla
Please do not reply directly 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 #28 from Mario Ceresa  2011-03-22 13:47:09 EDT 
---
Ok let's try again:

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

Added cmake br. I don't understand why it doesn't find the man pages:
http://koji.fedoraproject.org/koji/getfile?taskID=2933305&name=build.log

In my workstation compiles and installs just fine... *puzzled*

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 597307] Review Request: fastx_toolkit - Tools to process short-reads FASTA/FASTQ files

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


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

Adam Huffman  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #17 from Adam Huffman  2011-03-22 13:30:00 EDT 
---
Many thanks for the review - I've made that last change in -4

New Package SCM Request
===
Package Name: fastx_toolkit
Short Description:  Tools to process short-reads FASTA/FASTQ files
Owners: verdurin
Branches: f14 f15 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 635511] Review Request: assimp - Library to import various 3D model formats into applications

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


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

--- Comment #28 from Tom "spot" Callaway  2011-03-22 
13:25:11 EDT ---
I'm working with the irrlicht upstream to merge the fast_atof improvements that
assimp needs. They're on board in principle, but they didn't like some of the
function naming (the original patch changed the existing function strtol10 from
returning a signed int to an unsigned int), so I reworked the code so that
strtol10 returns a signed int, and there are new "u" functions (e.g. strtoul10)
that return the unsigned ints.

I had to patch assimp for the new names, here's that patch:
http://spot.fedorapeople.org/assimp-2.0.863-new-fastatof-typenames.patch

For reference, the irrlicht patch is here:
http://spot.fedorapeople.org/irrlicht-1.7.2-fastatof-improvements-typefixes.patch

I've done test compiles with both patches in play, and things look good. Now,
we just need more feedback from irrlicht upstream as to whether they will take
this change or not. I'm happy to carry the irrlicht patch in the short term. It
would probably be a good idea to let assimp know about this, and get their
feedback on the name changes.

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


[Bug 689870] New: Review Request: tabix - Generic indexer for TAB-delimited genome position files

2011-03-22 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: tabix - Generic indexer for TAB-delimited genome 
position files

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

   Summary: Review Request: tabix - Generic indexer for
TAB-delimited genome position files
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: bl...@verdurin.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://verdurin.fedorapeople.org/reviews/tabix/tabix.spec
SRPM URL:
http://verdurin.fedorapeople.org/reviews/tabix/tabix-0.2.3-1.fc16.src.rpm
Description: 
Tabix indexes a TAB-delimited genome position file in.tab.bgz and
creates an index file in.tab.bgz.tbi when region is absent from the
command-line. The input data file must be position sorted and
compressed by bgzip which has a gzip(1) like interface. After
indexing, tabix is able to quickly retrieve data lines overlapping
regions specified in the format "chr:beginPos-endPos". Fast data
retrieval also works over network if URI is given as a file name and
in this case the index file will be downloaded if it is not present
locally.

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


[Bug 688659] Review Request: thunarx-python - Python bindings for the Thunar Extension Framework

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


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

--- Comment #11 from Fedora Update System  
2011-03-22 13:11:14 EDT ---
thunarx-python-0.2.3-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/thunarx-python-0.2.3-3.fc15

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


[Bug 597307] Review Request: fastx_toolkit - Tools to process short-reads FASTA/FASTQ files

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


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

Jussi Lehtola  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #16 from Jussi Lehtola  2011-03-22 13:07:59 
EDT ---
You're missing ownership of
 %{_datadir}/%{name}
It should be owned by -galaxy. Otherwise things should be fine.

Please fix the ownership before git import; what you need to do is change
 %{_datadir}/%{name}/galaxy/
to
 %{_datadir}/%{name}/
in -galaxy.

This package has been

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 673030] Review Request: perl-Locale-US - Two letter codes for state identification in the United States and vice versa

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


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

--- Comment #9 from Fedora Update System  2011-03-22 
12:57:19 EDT ---
perl-Locale-US-1.2-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/perl-Locale-US-1.2-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 673030] Review Request: perl-Locale-US - Two letter codes for state identification in the United States and vice versa

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


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

--- Comment #8 from Fedora Update System  2011-03-22 
12:57:11 EDT ---
perl-Locale-US-1.2-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/perl-Locale-US-1.2-2.fc15

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


[Bug 673030] Review Request: perl-Locale-US - Two letter codes for state identification in the United States and vice versa

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


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

--- Comment #10 from Fedora Update System  
2011-03-22 12:57:26 EDT ---
perl-Locale-US-1.2-2.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/perl-Locale-US-1.2-2.fc13

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


[Bug 673030] Review Request: perl-Locale-US - Two letter codes for state identification in the United States and vice versa

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 674188] Review Request: libzeitgeist - Library to access zeitgeist; needed by synapse

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


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

--- Comment #22 from Mamoru Tasaka  2011-03-22 
12:42:32 EDT ---
Created attachment 486845
  --> https://bugzilla.redhat.com/attachment.cgi?id=486845
gdb log for test-log failure

I appreciate if you would post the correct srpm URL :)

By the way for test failure on test-log, gdb log is attached

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


[Bug 674188] Review Request: libzeitgeist - Library to access zeitgeist; needed by synapse

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


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

--- Comment #23 from Mamoru Tasaka  2011-03-22 
12:44:00 EDT ---
Created attachment 486846
  --> https://bugzilla.redhat.com/attachment.cgi?id=486846
Workaround patch

And the workaround patch

Build result on koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2933142

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


[Bug 688659] Review Request: thunarx-python - Python bindings for the Thunar Extension Framework

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


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

--- Comment #10 from Fedora Update System  
2011-03-22 12:42:29 EDT ---
thunarx-python-0.2.3-3.fc14.1 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/thunarx-python-0.2.3-3.fc14.1

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


[Bug 683463] Review Request: trafficserver - Apache Traffic Server

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


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

--- Comment #6 from Zhao Yongming  2011-03-22 12:42:54 EDT 
---
Bump to v2.1.7, uploaded in http://yum.zymlinux.net/trafficserver/rpm/2.1.7-1/

http://yum.zymlinux.net/trafficserver/rpm/2.1.7-1/trafficserver.spec
http://yum.zymlinux.net/trafficserver/rpm/2.1.7-1/trafficserver-2.1.7-1.fc16.src.rpm
http://yum.zymlinux.net/trafficserver/rpm/2.1.7-1/trafficserver-2.1.7-1.fc16.x86_64.rpm

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


[Bug 688659] Review Request: thunarx-python - Python bindings for the Thunar Extension Framework

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 674207] Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API

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


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

--- Comment #22 from Fedora Update System  
2011-03-22 12:30:55 EDT ---
python-cloudfiles-1.7.8-1.el5 has been pushed to the Fedora EPEL 5 stable
repository.

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


[Bug 220217] Review Request: perl-Test-Spelling - Check for spelling errors in POD files

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


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

--- Comment #6 from Fedora Update System  2011-03-22 
12:31:12 EDT ---
perl-Test-Spelling-0.11-1.el4 has been pushed to the Fedora EPEL 4 stable
repository.

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


[Bug 674207] Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API

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


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|python-cloudfiles-1.7.8-1.e |python-cloudfiles-1.7.8-1.e
   |l5  |l6

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


[Bug 674207] Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API

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


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|python-cloudfiles-1.7.8-1.f |python-cloudfiles-1.7.8-1.e
   |c14 |l5

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


[Bug 674207] Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API

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


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

--- Comment #23 from Fedora Update System  
2011-03-22 12:32:12 EDT ---
python-cloudfiles-1.7.8-1.el6 has been pushed to the Fedora EPEL 6 stable
repository.

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


[Bug 166065] Review Request: perl-Exception-Class

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


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

--- Comment #5 from Fedora Update System  2011-03-22 
12:31:23 EDT ---
perl-Exception-Class-1.23-2.el4 has been pushed to the Fedora EPEL 4 stable
repository.

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


[Bug 688408] Review Request: xfce4-cpufreq-plugin - CPU frequency scaling plugin for the Xfce4 panel

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


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

--- Comment #13 from Hicham HAOUARI  2011-03-22 
12:14:26 EDT ---
(In reply to comment #12)
> Too bad I wasn't aware of this review. Next time you package something for
> Xfce, please ping the Xfce SIG.

Right, I talked to nirik before submitting it.

> 
> I packaged this plugin earlier, but we agreed to not include it in Fedora for
> several reasons:
> * Usually the power managers take care of scaling the CPU frequency.

Right
> * The plugin does not work with Fedora's default setup. Scaling is only 
> allowed
> for root or through PolicyKit, but the plugin doesn't support the latter. How
> did you make the plugin work?

I missed that, I will work with upstream to add polkit support ASAP.

> * There usually is no need for changing frequencies or the governor, ondemand
> will handle this best.

I don't agree, under certain conditions I want to force the cpu to use a
certain frequency to keep my laptop cool.

> 
> Some notes on the package:
> 1) The description is formatted a little strange

Define "strange".
> 2) "Requires: hicolor-icon-theme" is not needed because the plugin requires
> gtk2 which already has a dependency to hicolor-icon-theme.

Right

> 3) The Requires for xfce4-panel should be versioned and the version should be
> adjusted for the different Fedora releases. A plugin built for Xfce4-panel
> xfce4-panel 4.8 will not work with 4.6 or 4.4, even though the srpm is the
> same.

I will work on that



Thanks Christoph for helping with this 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 688408] Review Request: xfce4-cpufreq-plugin - CPU frequency scaling plugin for the Xfce4 panel

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


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

Christoph Wickert  changed:

   What|Removed |Added

 CC||cwick...@fedoraproject.org

--- Comment #12 from Christoph Wickert  2011-03-22 
11:59:10 EDT ---
Too bad I wasn't aware of this review. Next time you package something for
Xfce, please ping the Xfce SIG.

I packaged this plugin earlier, but we agreed to not include it in Fedora for
several reasons:
* Usually the power managers take care of scaling the CPU frequency.
* The plugin does not work with Fedora's default setup. Scaling is only allowed
for root or through PolicyKit, but the plugin doesn't support the latter. How
did you make the plugin work?
* There usually is no need for changing frequencies or the governor, ondemand
will handle this best.

Some notes on the package:
1) The description is formatted a little strange
2) "Requires: hicolor-icon-theme" is not needed because the plugin requires
gtk2 which already has a dependency to hicolor-icon-theme.
3) The Requires for xfce4-panel should be versioned and the version should be
adjusted for the different Fedora releases. A plugin built for Xfce4-panel
xfce4-panel 4.8 will not work with 4.6 or 4.4, even though the srpm is the
same.

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


[Bug 537640] Review Request: pdfmod - A simple application for modifying PDF documents

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


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

--- Comment #20 from Claudio Rodrigo Pereyra DIaz  
2011-03-22 11:33:31 EDT ---
Update the .spec and .srpm files for the latest upstream version

Spec URL: http://elsupergomez.fedorapeople.org/SPECS/pdfmod.spec
SRPM URL:
http://elsupergomez.fedorapeople.org/SRPMS/pdfmod-0.9.1-1.fc14.src.rpm

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


[Bug 673116] Review Request: perl-Test-Perl-Critic-Progressive - Gradually enforce coding standards

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


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

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-Test-Perl-Critic-Progr
   ||essive-0.03-1.fc16
 Resolution||RAWHIDE
Last Closed||2011-03-22 10:52:52

--- Comment #6 from Petr Pisar  2011-03-22 10:52:52 EDT ---
Thank you for the review and the repository.

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


[Bug 689815] New: Review Request: perl-MooseX-Role-TraitConstructor - Wrapper for new that can accept a traits parameter

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

Summary: Review Request: perl-MooseX-Role-TraitConstructor - Wrapper for new 
that can accept a traits parameter

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

   Summary: Review Request: perl-MooseX-Role-TraitConstructor -
Wrapper for new that can accept a traits parameter
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


SRPM:
http://mmaslano.fedorapeople.org/review/perl-MooseX-Role-TraitConstructor-0.01-1.fc14.src.rpm
SPEC:
http://mmaslano.fedorapeople.org/review/perl-MooseX-Role-TraitConstructor.spec
Description:
This role allows you to easily accept a traits argument (or another name)
into your constructor, which will easily mix roles into an anonymous class
before construction, much like Moose::Meta::Attribute does.

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


[Bug 689629] Review Request: gnome-tweak-tool - A tool for advanced GNOME 3 customization

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


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

Cristian Ciupitu  changed:

   What|Removed |Added

   Hours Worked||1
   Flag|fedora-review?  |fedora-review+

--- Comment #9 from Cristian Ciupitu  2011-03-22 
10:15:15 EDT ---
I declare this 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 689629] Review Request: gnome-tweak-tool - A tool for advanced GNOME 3 customization

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


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

Mohamed El Morabity  changed:

   What|Removed |Added

 CC||pikachu.2...@gmail.com

--- Comment #8 from Mohamed El Morabity  2011-03-22 
10:12:27 EDT ---
Works very fine here too :)

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


[Bug 689629] Review Request: gnome-tweak-tool - A tool for advanced GNOME 3 customization

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


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

Tomasz Torcz  changed:

   What|Removed |Added

 CC||zdzi...@irc.pl

--- Comment #7 from Tomasz Torcz  2011-03-22 10:11:42 EDT ---
gnome-tweak-tool-2.91.92-3.fc15.noarch.rpm works fine. After installation, it
is available in GNOME-Shell application list, can be run, settings can be
changed and take effect.

Great work!

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


[Bug 678924] Review Request: perl-Math-Random-ISAAC-XS - C implementation of the ISAAC PRNG algorithm

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


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

--- Comment #7 from Jason Tibbitts  2011-03-22 09:56:08 EDT 
---
Git done (by process-git-requests).

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


[Bug 607405] Review Request: poppler-sharp - C Sharp Bindings for Poppler

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


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

--- Comment #5 from Claudio Rodrigo Pereyra DIaz  
2011-03-22 09:57:35 EDT ---
Translate summary and description to spanish

Spec URL: http://elsupergomez.fedorapeople.org/SPECS/poppler-sharp.spec
SRPM URL:
http://elsupergomez.fedorapeople.org/SRPMS/poppler-sharp-0.0.1-3.fc14.src.rpm

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


[Bug 689629] Review Request: gnome-tweak-tool - A tool for advanced GNOME 3 customization

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


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

--- Comment #6 from Cristian Ciupitu  2011-03-22 
10:09:05 EDT ---
Review for the latest release:
 -
http://salimma.fedorapeople.org/specs/gnome/gnome-tweak-tool-2.91.92-3.fc15.src.rpm
 - http://koji.fedoraproject.org/koji/taskinfo?taskID=2932379

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

· MUST: The License field in the package spec file must match the actual
license. ^[3]
OK

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

· MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section. If you feel that your packaged GUI application does not need
a .desktop file, you must put a comment in the spec file with your explanation.
^[22]
OK

===

I still can not test the package because I can not run GNOME3 on my hardware,
but as soon as someone else tests it, I will approve this package.

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

[Bug 673030] Review Request: perl-Locale-US - Two letter codes for state identification in the United States and vice versa

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


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

--- Comment #7 from Jason Tibbitts  2011-03-22 09:45:38 EDT 
---
Git done (by process-git-requests).

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


[Bug 673116] Review Request: perl-Test-Perl-Critic-Progressive - Gradually enforce coding standards

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


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

--- Comment #5 from Jason Tibbitts  2011-03-22 09:46:06 EDT 
---
Git done (by process-git-requests).

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


[Bug 689797] New: Review Request: perl-Task-Kensho-Toolchain - Task::Kensho::Toolchain Perl module

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

Summary: Review Request: perl-Task-Kensho-Toolchain - Task::Kensho::Toolchain 
Perl module

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

   Summary: Review Request: perl-Task-Kensho-Toolchain -
Task::Kensho::Toolchain Perl module
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


SRPM:
http://mmaslano.fedorapeople.org/review/perl-Task-Kensho-Toolchain-0.27-1.fc14.src.rpm
SPEC: http://mmaslano.fedorapeople.org/review/perl-Task-Kensho-Toolchain.spec
Description:
Kenshō (見性) (C. Wu) is a Japanese term for enlightenment
experiences—most commonly used within the confines of Zen
Buddhism—literally meaning "seeing one's nature"[1] or "true self."[2] It
generally "refers to the realization of nonduality of subject and
object."[3]

Task::Kensho is a first cut at building a list of recommended modules for
Enlightened Perl development. CPAN is wonderful, but there are too many
wheels and you have to pick and choose amongst the various competing
technologies.

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

[Bug 607405] Review Request: poppler-sharp - C Sharp Bindings for Poppler

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


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

--- Comment #4 from Claudio Rodrigo Pereyra DIaz  
2011-03-22 09:31:08 EDT ---
Translate summary and description to spanish

Spec URL: http://elsupergomez.fedorapeople.org/SPECS/poppler-sharp.spec
SRPM URL:
http://elsupergomez.fedorapeople.org/SRPMS/poppler-sharp-0.0.1-3.fc14.src.rpm

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


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

2011-03-22 Thread bugzilla
Please do not reply directly 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 #27 from Peter Lemenkov  2011-03-22 08:59:34 
EDT ---
(In reply to comment #26)
> It is great to see activity on this again! I tried building the src rpm on
> Fedora 13, and that worked, but on install I got this:
> 
> bash-4.1# rpm -Uvh rpmbuild/RPMS/x86_64/dcmtk-3.6.0-2.fc13.x86_64.rpm 
> error: Failed dependencies:
>  libdcmdsig.so()(64bit) is needed by dcmtk-3.6.0-2.fc13.x86_64
>  libijg12.so()(64bit) is needed by dcmtk-3.6.0-2.fc13.x86_64
>  libijg16.so()(64bit) is needed by dcmtk-3.6.0-2.fc13.x86_64
>  libijg8.so()(64bit) is needed by dcmtk-3.6.0-2.fc13.x86_64
>  liblibi2d.so()(64bit) is needed by dcmtk-3.6.0-2.fc13.x86_64
> 
> Not sure why.

They were moved by mistake to -devel subpackage because they don't have a
versioned suffix (as other libraries). This is an issue and must be fixed.

Mario, you forgot to add cmake to the buildrequires and I commented out
man-pages from %files in order to build it. See koji build-log:

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

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


[Bug 673116] Review Request: perl-Test-Perl-Critic-Progressive - Gradually enforce coding standards

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


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

Petr Pisar  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Petr Pisar  2011-03-22 08:48:13 EDT ---
New Package SCM Request
===
Package Name: perl-Test-Perl-Critic-Progressive
Short Description: Gradually enforce coding standards
Owners: ppisar mmaslano psabata
Branches: 
InitialCC: perl-sig

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


[Bug 495563] Review Request: febootstrap - Bootstrap a new Fedora system (like debootstrap)

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


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

Richard W.M. Jones  changed:

   What|Removed |Added

  Alias|febootstrap |

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


[Bug 598511] Review Request: libgtextutils - Assaf Gordon text utilities

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


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

--- Comment #17 from Fedora Update System  
2011-03-22 08:23:29 EDT ---
libgtextutils-0.6-5.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/libgtextutils-0.6-5.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 598511] Review Request: libgtextutils - Assaf Gordon text utilities

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


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

--- Comment #18 from Fedora Update System  
2011-03-22 08:24:16 EDT ---
libgtextutils-0.6-5.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/libgtextutils-0.6-5.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 676308] Review Request: rubygem-net-scp - A pure Ruby implementation of the SCP client protocol

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


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

--- Comment #13 from Fedora Update System  
2011-03-22 08:16:39 EDT ---
rubygem-net-scp-1.0.4-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/rubygem-net-scp-1.0.4-2.fc15

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


[Bug 676308] Review Request: rubygem-net-scp - A pure Ruby implementation of the SCP client protocol

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


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 598511] Review Request: libgtextutils - Assaf Gordon text utilities

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


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

--- Comment #16 from Fedora Update System  
2011-03-22 08:07:12 EDT ---
libgtextutils-0.6-5.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/libgtextutils-0.6-5.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 598511] Review Request: libgtextutils - Assaf Gordon text utilities

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


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

Adam Huffman  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-03-22 08:03:27

--- Comment #15 from Adam Huffman  2011-03-22 08:03:27 EDT 
---
Forgot to build this before now - closing.  Thanks for the review.

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


[Bug 597307] Review Request: fastx_toolkit - Tools to process short-reads FASTA/FASTQ files

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


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

Bug 597307 depends on bug 598511, which changed state.

Bug 598511 Summary: Review Request: libgtextutils - Assaf Gordon text utilities
https://bugzilla.redhat.com/show_bug.cgi?id=598511

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED

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


[Bug 689432] Review Request: collada-dom - COLLADA Document Object Model Library

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


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

Peter Lemenkov  changed:

   What|Removed |Added

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

--- Comment #2 from Peter Lemenkov  2011-03-22 07:54:04 EDT 
---
I'll review it

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


[Bug 597307] Review Request: fastx_toolkit - Tools to process short-reads FASTA/FASTQ files

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


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

--- Comment #15 from Jussi Lehtola  2011-03-22 07:45:27 
EDT ---
Crap, I had forgotten all about this review. Taking a look shortly.

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


[Bug 597307] Review Request: fastx_toolkit - Tools to process short-reads FASTA/FASTQ files

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


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

--- Comment #14 from Adam Huffman  2011-03-22 07:32:49 EDT 
---
Jussi,

Have you had a change to look at the latest version?

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


[Bug 637402] Review Request: sqlninja - A tool for SQL server injection and takeover

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


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

--- Comment #17 from Hicham HAOUARI  2011-03-22 
07:27:23 EDT ---
(In reply to comment #16)
> (In reply to comment #15)
> > @Arun,
> > 
> > The software would be still useful without the binaries ?
> 
> Yes! The software supports multiple modes, one of the mode is upload mode.
> Binaries are required for only upload mode. If the user wants to use upload
> mode, he can manually download these binaries.
> 
> Also please look into the scripts directory, the files under the scripts
> directory qualify as a binary?

Of course not.

I am not sure if the tarball needs to be cleaned up from the binaries though, I
will look more into that.

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


[Bug 637402] Review Request: sqlninja - A tool for SQL server injection and takeover

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


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

Hicham HAOUARI  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

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


[Bug 669726] Review Request: rubygem-escape - HTML/URI/shell escaping utilities

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


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

--- Comment #7 from Fedora Update System  2011-03-22 
07:22:14 EDT ---
rubygem-escape-0.0.4-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/rubygem-escape-0.0.4-2.fc15

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


  1   2   >