[Bug 705798] Review Request: perl-Data-Hexify - Perl extension for hexdumping arbitrary data

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


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

Ralf Corsepius rc040...@freenet.de changed:

   What|Removed |Added

 CC||rc040...@freenet.de

--- Comment #3 from Ralf Corsepius rc040...@freenet.de 2011-05-19 02:11:02 
EDT ---
One remark:

perl(Test::More) is a build-time-only requirement. It is not required at
run-time.

= R: perl(Test::More) should be removed.

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


[Bug 705133] Review Request: btparser - Parser and analyzer for backtraces produced by GDB

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


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

Karel Klíč kk...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|jmosk...@redhat.com
   Flag||fedora-cvs?

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

[Bug 704544] Review Request: perl-Dancer - Lightweight yet powerful web application framework

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Marcela Mašláňová mmasl...@redhat.com 2011-05-19 02:30:08 
EDT ---
New Package SCM Request
===
Package Name: perl-Dancer
Short Description: Lightweight yet powerful web application framework
Owners: mmaslano psabata ppisar
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 653435] Review Request: perl-Catalyst-Engine-PSGI - PSGI engine for Catalyst

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


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-Catalyst-Engine-PSGI-0
   ||.12-1.fc16
 Resolution||RAWHIDE
Last Closed||2011-05-19 02:43:34

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

[Bug 705848] Review Request: maven-jarsigner-plugin - Signs or verifies a project artifact and attachments using jarsigner

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


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

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

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

--- Comment #1 from Stanislav Ochotnicky socho...@redhat.com 2011-05-19 
03:07:34 EDT ---
I'll review this

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


[Bug 703461] Review Request: perl-threads-tbb - Interface to the Threading Building Blocks (TBB) API

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


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-threads-tbb-0.02-1.fc1
   ||6
 Resolution||RAWHIDE
Last Closed||2011-05-19 03:18:22

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


[Bug 705798] Review Request: perl-Data-Hexify - Perl extension for hexdumping arbitrary data

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


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

--- Comment #4 from Richard W.M. Jones rjo...@redhat.com 2011-05-19 03:49:21 
EDT ---
(In reply to comment #3)
 One remark:
 
 perl(Test::More) is a build-time-only requirement. It is not required at
 run-time.
 
 = R: perl(Test::More) should be removed.

Agreed.  I removed that line from the spec file, although
I didn't bother to rebuild the SRPM.

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


[Bug 705376] Review Request: perl-MIME-Lite-HTML - Provide routine to transform a HTML page in a MIME-Lite mail

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


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

--- Comment #3 from Xavier Bachelot xav...@bachelot.org 2011-05-19 04:15:31 
EDT ---
(In reply to comment #2)

  [!] Package meets the Packaging Guidelines including the Perl specific items
 
 Requires:   perl(HTML::LinkExtor)
 Requires:   perl(LWP::UserAgent)
 Requires:   perl(MIME::Lite) = 1
 Requires:   perl(Test::More)
 Requires:   perl(URI::URL)
 
 perl(Test::More) is a BuildRequires, not a Requires as far as I can see.
 All of the others are detected by auto-Requires. There is no need to include
 them by hand. There is no need to version the requirement perl(MIME::Lite).
 Please fix these.


Fixed. cpanspec was overzealous and I was too lazy when checking the spec :-(.

 Please consider removing the obsolete BuildRoot cleaning in %install section.
 Please consider removing the already implicit %defattr in %files section.
 
Needed for EL 5 and I'd rather keep the spec branches sync'ed.

  [x] Rpmlint output:
 perl-MIME-Lite-HTML.noarch: E: incorrect-fsf-address
 /usr/share/doc/perl-MIME-Lite-HTML-1.23/COPYING
 2 packages and 1 specfiles checked; 1 errors, 0 warnings.
 
 This again? Xavier, please consider filing a bug upstream to correct this.
 

Done : https://rt.cpan.org/Public/Bug/Display.html?id=68303

  [x] Buildroot is correct
 %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
 While this is correct, it's only used in EPEL5.
 In any other branch, this will be ignored.
 
See above, I'm targeting EL5 too.

  [x] 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.
 
 Again, please file a bug with upstream asking them to update their COPYING
 file.


See above.

Spec URL: http://www.bachelot.org/fedora/SPECS/perl-MIME-Lite-HTML.spec
SRPM URL:
http://www.bachelot.org/fedora/SRPMS/perl-MIME-Lite-HTML-1.23-2.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 705848] Review Request: maven-jarsigner-plugin - Signs or verifies a project artifact and attachments using jarsigner

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


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

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

 Blocks|652183(FE-JAVASIG)  |

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


[Bug 705848] Review Request: maven-jarsigner-plugin - Signs or verifies a project artifact and attachments using jarsigner

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


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

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Stanislav Ochotnicky socho...@redhat.com 2011-05-19 
04:31:26 EDT ---
Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[!]  Rpmlint output:
maven-jarsigner-plugin.noarch: W: spelling-error %description -l en_US xml -
XML, cml, ml
maven-jarsigner-plugin.noarch: W: non-conffile-in-etc
/etc/maven/fragments/maven-jarsigner-plugin
maven-jarsigner-plugin.src: W: spelling-error %description -l en_US xml - XML,
cml, ml
maven-jarsigner-plugin.src: W: invalid-url Source0:
http://search.maven.org/remotecontent?filepath=org/apache/maven/plugins/maven-jarsigner-plugin/1.2/maven-jarsigner-plugin-1.2-source-release.zip
HTTP Error 500: Sitebricks received a null reply from the resource.
3 packages and 0 specfiles checked; 0 errors, 4 warnings.

The url is maven repo problem, so OK.

[x]  Package is named according to the Package Naming Guidelines[1].
[x]  Spec file name must match the base package name, in the format
%{name}.spec.
[x]  Package meets the Packaging Guidelines[2].
[x]  Package successfully compiles and builds into binary rpms.
[x]  Buildroot definition is not present
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x]  License field in the package spec file matches the actual license.
License type: ASL 2.0
[x]  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.
[x]  All independent sub-packages have license of their own
[x]  Spec file is legible and written in American English.
[x]  Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package: aedf1c26b13bd55dd3b78d12fd88e2d4
[x]  All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x]  Package must own all directories that it creates.
[x]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[x]  Permissions on files are set properly.
[x]  Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x]  Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x]  Package contains code, or permissable content.
[x]  Fully versioned dependency in subpackages, if present.
[-]  Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x]  Package does not own files or directories owned by other packages.
[x]  Javadoc documentation files are generated and included in -javadoc
subpackage
[x]  Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x]  Packages have proper BuildRequires/Requires on jpackage-utils
[x]  Javadoc subpackages have Require: jpackage-utils
[x]  Package uses %global not %define
[-]  If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[x]  If source tarball includes bundled jar/class files these need to be
removed prior to building
[x]  All filenames in rpm packages must be valid UTF-8.
[x]  Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[x]  If package contains pom.xml files install it (including depmaps) even when
building with ant
[x]  pom files has correct add_to_maven_depmap call which resolves to the pom
file (use JPP. and JPP- correctly)

=== Maven ===
[x]  Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[-]  If package uses -Dmaven.test.skip=true explain why it was needed in a
comment
[-]  If package uses custom depmap -Dmaven2.jpp.depmap.file=* explain why
it's needed in a comment
[x]  Package uses %update_maven_depmap in %post/%postun
[x]  Packages have Requires(post) and Requires(postun) on jpackage-utils (for
%update_maven_depmap macro)

=== Other suggestions ===
[x]  If possible use upstream build method (maven/ant/javac)
[x]  Avoid having BuildRequires on exact NVR unless necessary
[x]  Package has BuildArch: noarch (if possible)
[x]  Latest version is packaged.
[x]  Reviewer should test that the package builds in mock.
Tested on: fedora-rawhide-x86_64


Package is nice and shiny.



*** APPROVED ***


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You 

[Bug 592645] Review Request: mkgmap - Convert OpenStreetMap data for Garmin GPSes

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


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

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR), |201449(FE-DEADREVIEW)
   |652183(FE-JAVASIG)  |

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


[Bug 592645] Review Request: mkgmap - Convert OpenStreetMap data for Garmin GPSes

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


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

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DEFERRED
Last Closed||2011-05-19 04:47:17

--- Comment #27 from Alexander Kurtakov akurt...@redhat.com 2011-05-19 
04:47:17 EDT ---
Bruce, 
I'm closing this bug now.
Please reopen once you gain interest again.

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


[Bug 705848] Review Request: maven-jarsigner-plugin - Signs or verifies a project artifact and attachments using jarsigner

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


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

Jaromír Cápík jca...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #3 from Jaromír Cápík jca...@redhat.com 2011-05-19 05:08:40 EDT 
---
New Package SCM Request
===
Package Name: maven-jarsigner-plugin
Short Description: Signs or verifies a project artifact and attachments using
jarsigner
Owners: jcapik
Branches: f15
InitialCC: java-sig

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

[Bug 655954] Review Request: drupal6-token - Tokens are small bits of text that can be placed into larger documents

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


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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 06:46:17 EDT ---
drupal6-token-1.16-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/drupal6-token-1.16-1.fc14

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


[Bug 705773] Review Request: erlang-meck - A mocking library for Erlang

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


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

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

   What|Removed |Added

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

--- Comment #1 from Peter Lemenkov lemen...@gmail.com 2011-05-19 07:05:54 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 706070] New: Review Request: keytool-maven-plugin - A plugin that wraps the keytool program and allows to manipulate keystores

2011-05-19 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: keytool-maven-plugin - A plugin that wraps the keytool 
program and allows to manipulate keystores

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

   Summary: Review Request: keytool-maven-plugin - A plugin that
wraps the keytool program and allows to manipulate
keystores
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jca...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL:
http://jcapik.fedorapeople.org/files/keytool-maven-plugin/1/keytool-maven-plugin.spec
SRPM URL:
http://jcapik.fedorapeople.org/files/keytool-maven-plugin/1/keytool-maven-plugin-1.0-1.fc14.src.rpm
Description:
Hi. I would appreciate a review for keytool-maven-plugin.

A plugin that wraps the keytool program bundled with Sun's Java SDK.
It provides the capability to manipulate keys and keystores
with the goals keytool:genkey and keytool:clean.

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


[Bug 706070] Review Request: keytool-maven-plugin - A plugin that wraps the keytool program and allows to manipulate keystores

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


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

Jaromír Cápík jca...@redhat.com changed:

   What|Removed |Added

 Blocks||652183(FE-JAVASIG)

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

[Bug 651521] Review Request: qtwebkit - Qt WebKit bindings

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Rex Dieter rdie...@math.unl.edu 2011-05-19 09:07:54 EDT 
---
New Package SCM Request
===
Package Name: qtwebkit
Short Description: Qt WebKit bindings
Owners: rdieter,jreznik,than
Branches: 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 703464] Review Request: google-gson - Java lib for conversion of Java objects into JSON representation

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-05-19 
09:33:28 EDT ---
google-gson-1.7.1-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/google-gson-1.7.1-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 703464] Review Request: google-gson - Java lib for conversion of Java objects into JSON representation

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


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 546871] Review Request: qpixman - Modified version of pixman for spice

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


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

Christophe Fergeau cferg...@redhat.com changed:

   What|Removed |Added

 CC||cferg...@redhat.com

--- Comment #4 from Christophe Fergeau cferg...@redhat.com 2011-05-19 
09:45:27 EDT ---
spice no longer needs a modified pixman version, I think this bug can be
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 666233] Review Request: gnome-paint - Easy to use paint program for GNOME

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


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

--- Comment #13 from Tareq Al Jurf talj...@fedoraproject.org 2011-05-19 
09:55:47 EDT ---
Sorry, almost forgot about it.

Last build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3081156

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


[Bug 666233] Review Request: gnome-paint - Easy to use paint program for GNOME

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


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

--- Comment #14 from Tareq Al Jurf talj...@fedoraproject.org 2011-05-19 
09:56:40 EDT ---
New Package CVS Request
===
Package Name: gnome-paint
Short Description: Easy to use paint program
Owners: taljurf
Branches: F-13 F-15
InitialCC: taljurf

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


[Bug 666233] Review Request: gnome-paint - Easy to use paint program for GNOME

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


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

Tareq Al Jurf talj...@fedoraproject.org changed:

   What|Removed |Added

   Flag||fedora-cvs?

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


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

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


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

Jan Vcelak jvce...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX
   Flag|needinfo?   |
Last Closed||2011-05-19 10:22:10

--- Comment #3 from Jan Vcelak jvce...@redhat.com 2011-05-19 10:22:10 EDT ---
I agree. I needed this for one project, but I would rather take a look at
something else. Thank you, guys.

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


[Bug 674738] Review Request: kamoso - Application for taking pictures and videos from a webcam

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


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

Jaroslav Reznik jrez...@redhat.com changed:

   What|Removed |Added

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

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


[Bug 674738] Review Request: kamoso - Application for taking pictures and videos from a webcam

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


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

--- Comment #11 from Jaroslav Reznik jrez...@redhat.com 2011-05-19 11:00:12 
EDT ---
Name: ok
Version and release tag: ok, pre-release beta1
Summary: ok
Group: ok
License: ok
URL: not ok - kamoso moved from playground to kdereview, not a blocker, at it's
going to move again probably soon, please update it when it hits final place
Sources: ok, md5sum 6aa9b7f24454eee844ceba35ecaa36fc
BRs: ok
Description: ok

Macros usage: ok
Scriptlets: ok, desktop-file-validation, icons, no ldconfig needed

Build: ok, scratch build included, build locally on x86_64

rpmlint kamoso-2.0-0.4.beta1.fc16.src.rpm 
kamoso.src: W: spelling-error Summary(en_US) webcam - web cam, web-cam,
webcast
kamoso.src: W: spelling-error %description -l en_US webcam - web cam, web-cam,
webcast
kamoso.src: W: invalid-url URL:
https://projects.kde.org/projects/playground/multimedia/kamoso/ HTTP Error 404:
Not Found

 Please check my comment above.

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

rpmlint kamoso-2.0-0.4.beta1.fc14.x86_64.rpm 
kamoso.x86_64: W: spelling-error Summary(en_US) webcam - web cam, web-cam,
webcast
kamoso.x86_64: W: spelling-error %description -l en_US webcam - web cam,
web-cam, webcast
kamoso.x86_64: W: invalid-url URL:
https://projects.kde.org/projects/playground/multimedia/kamoso/ HTTP Error 404:
Not Found
kamoso.x86_64: W: no-manual-page-for-binary kamoso
kamoso.x86_64: W: no-manual-page-for-binary kamosoPluginTester
1 packages and 0 specfiles checked; 0 errors, 5 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 674738] Review Request: kamoso - Application for taking pictures and videos from a webcam

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


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

--- Comment #12 from Jaroslav Reznik jrez...@redhat.com 2011-05-19 11:05:44 
EDT ---
Back to licenses - I think the GPLv3 in youtube code is just a mistake - looks
like it's the default license for some IDE - even the author is not filled in.
Could you ask upstream?

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


[Bug 700587] Review Request: surfer - visualization of algebraic geometry

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


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

--- Comment #4 from Jerry James loganje...@gmail.com 2011-05-19 11:45:36 EDT 
---
(In reply to comment #1)
 %{_datadir}/surfer
 %exclude %{_datadir}/surfer/gallery
 
 These directories and their contents are not properly owned by your package.
 Please put a / after the name to do so.

That is incorrect.  Those directories and their contents ARE properly owned by
the package, as I already pointed out when you made the same comment about the
surf-geometry package.  Please stop making this false claim.

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


[Bug 651521] Review Request: qtwebkit - Qt WebKit bindings

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


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

--- Comment #8 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 11:56:01 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 666233] Review Request: gnome-paint - Easy to use paint program for GNOME

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


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

--- Comment #15 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 11:56:20 EDT 
---
Git done (by process-git-requests).

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


[Bug 679401] Review Request: qtsoap - The Simple Object Access Protocol Qt-based client side library

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


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

--- Comment #2 from Jaroslav Reznik jrez...@redhat.com 2011-05-19 11:56:59 
EDT ---
spec URL: http://rezza.hofyland.cz/fedora/packages/qtsoap/qtsoap.spec
SRPM URL:
http://rezza.hofyland.cz/fedora/packages/qtsoap/qtsoap-2.7-1.fc14.src.rpm

Fixed 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 673589] Review Request: UpTools - C++ library for hpc, networking, db, memory, etc.

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


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

--- Comment #55 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 11:56: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 701845] Review Request: shunit2 - A xUnit based unit testing for Unix shell scripts

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


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

--- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 11:56:59 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 679401] Review Request: qtsoap - The Simple Object Access Protocol Qt-based client side library

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


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

Trever Fischer wm...@wm161.net changed:

   What|Removed |Added

 CC||wm...@wm161.net

--- Comment #3 from Trever Fischer wm...@wm161.net 2011-05-19 12:05:42 EDT ---
Not a review, but a brief comment: the library should be installed as
'libqtsoap', not 'libQtSolutions_SOAP', for consistency with the package name.
Or vice-versa, calling the package 'qtsolutions-soap'.

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


[Bug 700346] Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes

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


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

Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review?

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


[Bug 700346] Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes

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


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

Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|iarn...@gmail.com

--- Comment #3 from Iain Arnell iarn...@gmail.com 2011-05-19 12:13:02 EDT ---
(In reply to comment #1)
 Not an official review since I'm not a sponsor

I went and got myself promoted, so I *can* start the official review for you
now.

One minor niggle before I start, though. When making changes to a package,
please always increment the release number, update the changelog, rebuild the
srpm, upload the new srpm and spec, and post the new link.

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


[Bug 700346] Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes

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


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

--- Comment #4 from Iain Arnell iarn...@gmail.com 2011-05-19 12:20:19 EDT ---
Koji (FAILURE) http://koji.fedoraproject.org/koji/taskinfo?taskID=3081440

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


[Bug 704544] Review Request: perl-Dancer - Lightweight yet powerful web application framework

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


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

--- Comment #6 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 12:24:44 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 700346] Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes

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


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

--- Comment #5 from Iain Arnell iarn...@gmail.com 2011-05-19 12:25:35 EDT ---
Aha. It seems you also need to BuildRequire perl(GD). 

And as it's technically optional, rpm won't automatically detect this as a
runtime dependency either. But since the module seems pretty useless without
being able to produce images, I would suggest adding an explicit Requires:
perl(GD) to the spec file 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 705133] Review Request: btparser - Parser and analyzer for backtraces produced by GDB

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


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

--- Comment #6 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 12:25:27 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 705848] Review Request: maven-jarsigner-plugin - Signs or verifies a project artifact and attachments using jarsigner

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


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

--- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 12:25:42 EDT 
---
Git done (by process-git-requests).

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


[Bug 674738] Review Request: kamoso - Application for taking pictures and videos from a webcam

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


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

--- Comment #14 from nucleo alekc...@googlemail.com 2011-05-19 13:06:16 EDT 
---
License fix:
http://quickgit.kde.org/?p=kamoso.gita=commitdiffh=15d1e050ad24ec71b2cf541f47caba389dfec0c7hp=547755fd34c9e4f3de43d1c359f367542c2c9fbb

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


[Bug 674738] Review Request: kamoso - Application for taking pictures and videos from a webcam

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


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

--- Comment #13 from nucleo alekc...@googlemail.com 2011-05-19 13:05:27 EDT 
---
Stable release available
ftp://ftp.kde.org/pub/kde/stable/kamoso/2.0.1/src/kamoso-2.0.1.tar.bz2
Also afiestas going to fix license, so then I can add this pacth to 2.0.1 and
change license to GPLv2+.
So URL for now will be https://projects.kde.org/projects/kdereview/kamoso/?

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


[Bug 674738] Review Request: kamoso - Application for taking pictures and videos from a webcam

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


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

--- Comment #15 from nucleo alekc...@googlemail.com 2011-05-19 13:41:15 EDT 
---
Or maybe URL should be
http://kde-apps.org/content/show.php/Kamoso?content=111750 ?

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


[Bug 679401] Review Request: qtsoap - The Simple Object Access Protocol Qt-based client side library

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


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

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

   What|Removed |Added

 CC||rdie...@math.unl.edu
 AssignedTo|nob...@fedoraproject.org|rdie...@math.unl.edu
   Flag||fedora-review?

--- Comment #4 from Rex Dieter rdie...@math.unl.edu 2011-05-19 13:54:09 EDT 
---
I can review this.

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


[Bug 679401] Review Request: qtsoap - The Simple Object Access Protocol Qt-based client side library

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


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

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

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Rex Dieter rdie...@math.unl.edu 2011-05-19 14:04:20 EDT 
---

licensing: OK

scriptlets: OK

naming: OK

URL provided doesn't work for me, please find something better.  Not a blocker.

rpmlint: clean, OK

builds/installs OK

In a perfect world, I'd like to see comment #3 addressed too, wrt keeping
library naming more consistent, using something from trever's .spec he showed
us earlier today:
%build
echo 'yes' | \
./configure -library

echo QTSOAP_LIBNAME = \$\$qtLibraryTarget(qtsoap)  common.pri
echo VERSION=%{version}  common.pri

qmake-qt4

make %{?_smp_mflags}


But, I'll leave it up to you on how best to resolve that (with some potential
upstream poking).  not a blocker.



Otherwise, looks good, 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 554603] Review Request: garcon - Implementation of the freedesktop.org menu specification

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


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

Orion Poplawski or...@cora.nwra.com changed:

   What|Removed |Added

 CC||or...@cora.nwra.com
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #8 from Orion Poplawski or...@cora.nwra.com 2011-05-19 14:08:13 
EDT ---
Package Change Request
==
Package Name: garcon
New Branches: el6
Owners: cwickert kevin orion
InitialCC: 

Kevin/Chris - please ACK or update Owners.  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 554599] Review Request: libxfce4ui - Commonly used Xfce widgets

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


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

Orion Poplawski or...@cora.nwra.com changed:

   What|Removed |Added

 CC||or...@cora.nwra.com
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #10 from Orion Poplawski or...@cora.nwra.com 2011-05-19 14:17:33 
EDT ---
Package Change Request
==
Package Name: libxfce4ui
New Branches: el6
Owners: cwickert kevin orion
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 554599] Review Request: libxfce4ui - Commonly used Xfce widgets

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


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

--- Comment #11 from Kevin Fenzi ke...@scrye.com 2011-05-19 14:18:24 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 554603] Review Request: garcon - Implementation of the freedesktop.org menu specification

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


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

--- Comment #9 from Kevin Fenzi ke...@scrye.com 2011-05-19 14:18:40 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 706199] New: Review Request: gnupg-pkcs11-scd - GnuPG-compatible smart-card daemon with PKCS#11 support

2011-05-19 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: gnupg-pkcs11-scd - GnuPG-compatible smart-card daemon 
with PKCS#11 support

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

   Summary: Review Request: gnupg-pkcs11-scd - GnuPG-compatible
smart-card daemon with PKCS#11 support
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: m...@flyn.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL: http://www.flyn.org/SRPMS/gnupg-pkcs11-scd.spec
SRPM URL: http://www.flyn.org/SRPMS/gnupg-pkcs11-scd-0.7.2-1.fc15.src.rpm
Description:
gnupg-pkcs11-scd is a drop-in replacement for the smart-card daemon (scd)
shipped with the next-generation GnuPG (gnupg2). The daemon interfaces
with smart-cards by using RSA Security Inc.'s PKCS#11 Cryptographic Token
Interface.

This package allows the use of US DoD smart cards (CAC) with GnuPG.

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


[Bug 706205] New: Review Request: ssh-contact - Establish SSH connections to your IM contacts using Telepathy

2011-05-19 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: ssh-contact - Establish SSH connections to your IM 
contacts using Telepathy

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

   Summary: Review Request: ssh-contact - Establish SSH
connections to your IM contacts using Telepathy
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: bdpep...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL: http://bpepple.fedorapeople.org/rpms/ssh-contact.spec
SRPM URL: http://bpepple.fedorapeople.org/rpms/ssh-contact-0.6-1.fc15.src.rpm
Description: ssh-contact is a client/service tool that makes it easy to connect
to
your telepathy IM contacts via SSH. No need to care about dynamic IP, NAT, port
forwarding, or firewalls anymore; if you can chat witha friend, you can also
SSH to their machine.

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


[Bug 666233] Review Request: gnome-paint - Easy to use paint program for GNOME

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


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

Tareq Al Jurf talj...@fedoraproject.org changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #16 from Tareq Al Jurf talj...@fedoraproject.org 2011-05-19 
15:13:22 EDT ---
Sorry, forgot to add F-14:

Package Change Request
==
Package Name: gnome-paint
New Branches: F-14
Owners: taljurf
InitialCC: taljurf

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


[Bug 679060] Review Request: mingw32-antlr - MinGW Windows ANTLR C++ run-time library

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


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

Thomas Sailer t.sai...@alumni.ethz.ch changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #9 from Thomas Sailer t.sai...@alumni.ethz.ch 2011-05-19 17:09:26 
EDT ---
New Package SCM Request
===
Package Name: mingw-antlr
Short Description: MinGW Windows ANTLR C++ run-time library
Owners: sailer
Branches: f14 f15

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


[Bug 701112] Review Request: mingw32-wxWidgets - C++ cross-platform GUI library

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


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

Thomas Sailer t.sai...@alumni.ethz.ch changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #9 from Thomas Sailer t.sai...@alumni.ethz.ch 2011-05-19 17:08:20 
EDT ---
New Package SCM Request
===
Package Name: mingw-wxWidgets
Short Description: MinGW port of the wxWidgets GUI library
Owners: sailer kalev
Branches: f14 f15

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


[Bug 705108] Review Request: shinken - python monitoring tool

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


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

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at

--- Comment #1 from Volker Fröhlich volke...@gmx.at 2011-05-19 17:11:28 EDT 
---
You can use the name macro in the tarball name.

The license is AGPLv3+. Rpmlint still complains, because the license was
forgotten in the list of acceptable licenses. It is included by now.

Requiring a Python newer or equal to 2.6 is not necessary, as all Fedora
versions have it. If you consider maintaining it in EPEL 5 as well, you must
add a clean section and rm -rf the buildroot in the install section. In that
case please also pick a BuildRoot syntax from
http://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#BuildRoot_tag

If you don't want to maintain in EPEL, drop the buildroot definition. Besides
that, FOR_PACKAGERS points out Python 2.4 would also work.

Rather use %global than %define.

defattr seems unnecessary now: https://fedorahosted.org/fpc/ticket/77
But if you use it, add the minus in the end: %defattr(-,root,root,-)

Drop the -n %{name}-%{version} from the setup macro. It's not necessary. You
also don't have to define the __python macro.

It's good practice to backup patched files like: %patch1 -p1 -b
.something_that_identifies_the_patch

Don't use the install macro. Just use the normal command. Probably put a few
comments to make everything clear. Especially when you decide to do other than
obvious.

Why do you need to have executable __init__.py (chmod)?

The unitdir macro is not defined. Thus a rebuild fails.

The paths inside FROM_NAGIOS_TO_SHINKEN don't reflect the situation for Fedora.
Other files may be affected as well. Please check the paths in the config
files, whether the targets exist and are sane. I can see /usr/local/...

README is only a symlink pointing towards README.rst -- no need to include it.

Any chance to run the unit tests?

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

[Bug 674738] Review Request: kamoso - Application for taking pictures and videos from a webcam

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


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

Scott Williams vwfoxg...@gmail.com changed:

   What|Removed |Added

 CC||vwfoxg...@gmail.com

--- Comment #16 from Scott Williams vwfoxg...@gmail.com 2011-05-19 16:55:59 
EDT ---
I recompiled your last source RPM post for x86_64, f15, and looks like it's
working just fine here so far.  Installed with no problem and displays web cam
input, so it's working on a least a basic level here.

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


[Bug 700199] Review Request: tomcat - Apache Servlet/JSP Engine, RI for Servlet 3.0/JSP 2.2 API

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


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

--- Comment #11 from Alexander Kurtakov akurt...@redhat.com 2011-05-19 
17:33:58 EDT ---
Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[!]  Rpmlint output:
tomcat-lib.noarch: W: no-documentation
tomcat-lib.noarch: W: dangling-relative-symlink
/usr/share/java/tomcat/tomcat-jsp-2.2-api.jar ../tomcat-jsp-2.2-api.jar
tomcat-lib.noarch: W: dangling-relative-symlink
/usr/share/java/tomcat/tomcat-servlet-3.0-api.jar ../tomcat-servlet-3.0-api.jar
tomcat-lib.noarch: W: dangling-symlink /usr/share/java/tomcat/commons-dbcp.jar
/usr/share/java/apache-commons-dbcp.jar
tomcat-lib.noarch: W: dangling-symlink
/usr/share/java/tomcat/commons-collections.jar
/usr/share/java/apache-commons-collections.jar
tomcat-lib.noarch: W: dangling-symlink /usr/share/java/tomcat/log4j.jar
/usr/share/java/log4j.jar
tomcat-lib.noarch: W: dangling-symlink /usr/share/java/tomcat/jasper-jdt.jar
/usr/share/java/ecj.jar
tomcat.noarch: W: spelling-error %description -l en_US servlet - settler
tomcat.noarch: W: non-conffile-in-etc /etc/maven/fragments/tomcat
These are OK.
tomcat.noarch: E: non-standard-dir-perm /etc/tomcat 0775L
This looks wrong. Do we really want /etc/tomcat that open?

tomcat.noarch: E: non-standard-dir-perm /var/log/tomcat 0775L
tomcat.noarch: E: non-standard-dir-perm /usr/share/tomcat 0775L
tomcat.noarch: E: non-standard-dir-perm /etc/tomcat/Catalina/localhost 0775L
Same for previous three.

tomcat.noarch: W: dangling-symlink /usr/share/tomcat/lib /usr/share/java/tomcat
tomcat.noarch: E: non-readable /etc/tomcat/tomcat-users.xml 0660L
OK
tomcat.noarch: E: non-standard-dir-perm /var/cache/tomcat 0775L
tomcat.noarch: E: non-standard-dir-perm /etc/tomcat/Catalina 0775L
tomcat.noarch: E: non-standard-dir-perm /var/cache/tomcat/work 0775L
tomcat.noarch: E: non-standard-dir-perm /var/cache/tomcat/temp 0775L
tomcat.noarch: E: non-standard-dir-perm /var/lib/tomcat/webapps 0775L
Should these be that open?
tomcat.noarch: W: no-manual-page-for-binary tomcat-digest
tomcat.noarch: W: no-manual-page-for-binary dtomcat
tomcat.noarch: W: no-manual-page-for-binary tomcat-tool-wrapper
tomcat.noarch: W: no-manual-page-for-binary tomcat
tomcat.noarch: W: dangerous-command-in-%preun rm
OK. The preun is simply cleaning the work and temp directories.
tomcat.noarch: W: incoherent-subsys /etc/init.d/tomcat ${NAME}
tomcat.noarch: W: incoherent-subsys /etc/init.d/tomcat ${NAME}
tomcat.noarch: W: incoherent-subsys /etc/init.d/tomcat ${NAME}
tomcat.noarch: W: incoherent-subsys /etc/init.d/tomcat ${NAME}
tomcat.noarch: W: incoherent-subsys /etc/init.d/tomcat ${NAME}
tomcat.noarch: W: incoherent-subsys /etc/init.d/tomcat ${NAME}
Can you ellaborate on this?
tomcat-docs-webapp.noarch: W: no-documentation
tomcat-admin-webapps.noarch: W: no-documentation
OK.
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/host-manager/META-INF 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/host-manager 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/host-manager/WEB-INF 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/host-manager/WEB-INF/jsp 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/manager/WEB-INF 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/manager/images 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/manager/META-INF 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/manager 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/host-manager/images 0775L
tomcat-admin-webapps.noarch: E: non-standard-dir-perm
/var/lib/tomcat/webapps/manager/WEB-INF/jsp 0775L
Is tomcat really writing into these folders? I don't think so and they might
use standard permissions.
tomcat-webapps.noarch: W: no-documentation
tomcat-jsp-2.2-api.noarch: W: no-documentation
tomcat-servlet-3.0-api.noarch: W: no-documentation
tomcat-servlet-3.0-api.noarch: W: non-conffile-in-etc
/etc/maven/fragments/tomcat-servlet-api
tomcat-el-2.2-api.noarch: W: no-documentation
OK.
[x]  Package is named according to the Package Naming Guidelines[1].
[x]  Spec file name must match the base package name, in the format
%{name}.spec.
[x]  Package meets the Packaging Guidelines[2].
[x]  Package successfully compiles and builds into binary rpms.
[x]  Buildroot definition is not present
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x]  License field in the package spec file matches the actual license.
License type: ASL 2.0
[x]  If (and only if) the source package includes 

[Bug 704705] Review Request: perl-Parallel-Iterator - Simple parallel execution

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-05-19 
17:55:54 EDT ---
perl-Parallel-Iterator-1.00-1.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 698067] Review Request: hiredis - A C client library for redis

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 17:59:17 EDT ---
hiredis-0.10.0-3.fc14 has been pushed to the Fedora 14 testing repository.

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


[Bug 684511] Review Request: perl-DBIx-Class-Cursor-Cached - Cursor class with built-in caching support

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 17:57:34 EDT ---
perl-DBIx-Class-Cursor-Cached-1.001001-1.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 701183] Review Request: perl-CGI-Application-Plugin-RateLimit - Limits runmode call rate per user

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-05-19 
17:57:15 EDT ---
perl-CGI-Application-Plugin-RateLimit-1.0-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 593559] Review Request: protobuf-c - C bindings for Google's Protocol Buffers

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


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

--- Comment #34 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 18:03:14 EDT ---
protobuf-c-0.15-2.fc14 has been pushed to the Fedora 14 stable repository.

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


[Bug 703464] Review Request: google-gson - Java lib for conversion of Java objects into JSON representation

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 18:00:11 EDT ---
google-gson-1.7.1-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 694568] Review Request: python-testtools - Extensions to the Python unit testing framework

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-05-19 
18:02:26 EDT ---
python-testtools-0.9.8-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 593559] Review Request: protobuf-c - C bindings for Google's Protocol Buffers

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


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

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

   What|Removed |Added

   Fixed In Version|protobuf-c-0.15-2.fc15  |protobuf-c-0.15-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 694568] Review Request: python-testtools - Extensions to the Python unit testing framework

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


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

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

   What|Removed |Added

   Fixed In Version|python-testtools-0.9.8-2.fc |python-testtools-0.9.8-2.fc
   |14  |15

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


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

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


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

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review?

--- Comment #29 from Alexander Kurtakov akurt...@redhat.com 2011-05-19 
18:26:45 EDT ---
Seems that it finally compiles.

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


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

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


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

--- Comment #30 from Alexander Kurtakov akurt...@redhat.com 2011-05-19 
18:45:00 EDT ---
Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[x]  Rpmlint output:
[x]  Package is named according to the Package Naming Guidelines[1].
[x]  Spec file name must match the base package name, in the format
%{name}.spec.
[x]  Package meets the Packaging Guidelines[2].
[x]  Package successfully compiles and builds into binary rpms.
[x]  Buildroot definition is not present
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[!]  License field in the package spec file matches the actual license.
License type: According to the License file should be MIT and BSD
[x]  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.
[!]  All independent sub-packages have license of their own
[x]  Spec file is legible and written in American English.
[x]  Sources used to build the package matches the upstream source, as provided
in the spec URL.
[x]  All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x]  Package must own all directories that it creates.
[x]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[x]  Permissions on files are set properly.
[x]  Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x]  Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x]  Package contains code, or permissable content.
[x]  Fully versioned dependency in subpackages, if present.
[-]  Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x]  Package does not own files or directories owned by other packages.
[x]  Javadoc documentation files are generated and included in -javadoc
subpackage
[x]  Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x]  Packages have proper BuildRequires/Requires on jpackage-utils
[x]  Javadoc subpackages have Require: jpackage-utils
[-]  Package uses %global not %define
[-]  If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[!]  If source tarball includes bundled jar/class files these need to be
removed prior to building
[x]  All filenames in rpm packages must be valid UTF-8.
[x]  Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[-]  If package contains pom.xml files install it (including depmaps) even when
building with ant
[-]  pom files has correct add_to_maven_depmap call which resolves to the pom
file (use JPP. and JPP- correctly)

=== Other suggestions ===
[x]  If possible use upstream build method (maven/ant/javac)
[x]  Avoid having BuildRequires on exact NVR unless necessary
[x]  Package has BuildArch: noarch (if possible)
[x]  Latest version is packaged.
[x]  Reviewer should test that the package builds in mock.



=== Issues ===
1. License should be MIT and BSD.
2. Javadoc and manual subpackages are missing license
3. Please look for any jar or class files and remove them in prep

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


[Bug 666233] Review Request: gnome-paint - Easy to use paint program for GNOME

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


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

--- Comment #17 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 19:19:22 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 679060] Review Request: mingw-antlr - MinGW Windows ANTLR C++ run-time library

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


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

--- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 19:21:04 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 679060] Review Request: mingw-antlr - MinGW Windows ANTLR C++ run-time library

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


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

Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

Summary|Review Request: |Review Request: mingw-antlr
   |mingw32-antlr - MinGW   |- MinGW Windows ANTLR C++
   |Windows ANTLR C++ run-time  |run-time library
   |library |

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


[Bug 701112] Review Request: mingw-wxWidgets - C++ cross-platform GUI library

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


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

--- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2011-05-19 19:21:29 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 701112] Review Request: mingw-wxWidgets - C++ cross-platform GUI library

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


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

Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |mingw32-wxWidgets - C++ |mingw-wxWidgets - C++
   |cross-platform GUI library  |cross-platform GUI library

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


[Bug 706299] New: Review Request: python3-cvxopt - A Python Package for Convex Optimization

2011-05-19 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: python3-cvxopt - A Python Package for Convex 
Optimization

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

   Summary: Review Request: python3-cvxopt - A Python Package for
Convex Optimization
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: loganje...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL: http://jjames.fedorapeople.org/python3-cvxopt/python3-cvxopt.spec
SRPM URL:
http://jjames.fedorapeople.org/python3-cvxopt/python3-cvxopt-1.1.3-1.fc14.src.rpm
Description: 
CVXOPT is a free software package for convex optimization based on
the Python programming language. Its main purpose is to make the
development of software for convex optimization applications
straightforward by building on Python's extensive standard library and
on the strengths of Python as a high-level programming language.

This package is the Python 3 counterpart to the existing python-cvxopt package.
 The spec file has been designed to be as close as possible to
python-cvxopt.spec to ease future maintenance of both packages.

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


[Bug 706299] Review Request: python3-cvxopt - A Python Package for Convex Optimization

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


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

Jerry James loganje...@gmail.com changed:

   What|Removed |Added

 Blocks||700288

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


[Bug 701845] Review Request: shunit2 - A xUnit based unit testing for Unix shell scripts

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


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 701845] Review Request: shunit2 - A xUnit based unit testing for Unix shell scripts

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


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

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 22:46:53 EDT ---
shunit2-2.1.6-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/shunit2-2.1.6-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 701845] Review Request: shunit2 - A xUnit based unit testing for Unix shell scripts

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


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

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 23:01:10 EDT ---
shunit2-2.1.6-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/shunit2-2.1.6-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 701845] Review Request: shunit2 - A xUnit based unit testing for Unix shell scripts

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


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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 23:07:35 EDT ---
shunit2-2.1.6-3.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/shunit2-2.1.6-3.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 706310] Need EPEL-6 branches of ltsp ltspfs ldm mkdst nbd

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


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

Warren Togami wtog...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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


[Bug 706310] New: Need EPEL-6 branches of ltsp ltspfs ldm mkdst nbd

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

Summary: Need EPEL-6 branches of ltsp ltspfs ldm mkdst nbd

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

   Summary: Need EPEL-6 branches of ltsp ltspfs ldm mkdst nbd
   Product: Fedora EPEL
   Version: el6
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: wtog...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


ltsp ltspfs ldm mkdst are EPEL-6 only packages.  Fedora 15+ is incompatible
with LTSP.  nbd and xorg-x11-drv-geode are no longer shipped as of RHEL-6 so I
need a branch in EPEL-6.

Package Change Request
==
Package Name: ltsp ltspfs ldm mkdst nbd xorg-x11-drv-geode
New Branches: el6
Owners: wtogami

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


[Bug 701845] Review Request: shunit2 - A xUnit based unit testing for Unix shell scripts

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


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

--- Comment #15 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 23:14:29 EDT ---
shunit2-2.1.6-3.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/shunit2-2.1.6-3.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 701845] Review Request: shunit2 - A xUnit based unit testing for Unix shell scripts

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


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

--- Comment #16 from Hushan Jia h...@redhat.com 2011-05-19 23:21:57 EDT ---
Thanks Daiki and Jason, updates have been submitted.

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


[Bug 706310] Need EPEL-6 branches of ltsp ltspfs ldm mkdst nbd

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


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

Kevin Fenzi ke...@scrye.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #1 from Kevin Fenzi ke...@scrye.com 2011-05-19 23:26:05 EDT ---
done.

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


[Bug 701845] Review Request: shunit2 - A xUnit based unit testing for Unix shell scripts

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


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

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-05-19 22:59:45 EDT ---
shunit2-2.1.6-3.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/shunit2-2.1.6-3.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 675542] Review Request: perl-Hash-MoreUtils - Provide the stuff missing in Hash::Util

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


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

Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-05-19 23:47:04

--- Comment #6 from Iain Arnell iarn...@gmail.com 2011-05-19 23:47:04 EDT ---
This was imported and built for all branches back in March.

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


[Bug 633104] Review Request: monkeysphere - Use the OpenPGP web of trust to verify ssh connections

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


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

Bernie Innocenti ber...@codewiz.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CURRENTRELEASE
Last Closed||2011-05-20 00:01:25

--- Comment #19 from Bernie Innocenti ber...@codewiz.org 2011-05-20 00:01:25 
EDT ---
Package imported and built for f14, f15 and rawhide. Thanks to everyone who
contributed to 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 699208] Review Request: perl-ORLite-Statistics - Statistics enhancement package for ORLite

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-05-20 
01:18:25 EDT ---
perl-ORLite-Statistics-0.03-1.fc14 has been submitted as an update for Fedora
14.
https://admin.fedoraproject.org/updates/perl-ORLite-Statistics-0.03-1.fc14

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