[Bug 801439] Review Request: rubygem-netrc - Library to read and write netrc files

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


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

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

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #7 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-03-14 
02:31:11 EDT ---
Package Change Request
==
Package Name: rubygem-netrc
New Branches: f17
Owners: vondruch bkabrda

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

[Bug 737293] Review Request: python-django - A high-level Python Web framework

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


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

--- Comment #17 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-03-14 
02:47:25 EDT ---
(In reply to comment #16)
 Weird, those are different from the build errors I get from a local mock 
 setup.
 
 My suggestion is adding an optional --with build option for running the test
 suite, until we resolve these test problems; until then, maintainers can make
 sure that a local rpmbuild rebuild (which currently works for 1.3.1-7) at 
 least
 passes. When the tests are fixed we can flip the switch to make building with
 tests the default (and maybe make it non-default for unbranched Rawhide so we
 can quickly test new versions).
 
 Bohuslav, any other change we need to make for the review, apart from the test
 suite as already discussed? Thanks.

Hi guys, I've been ill for a few days, so if you let me catch up a bit, I'll
try to have a look at this as soon as I have some time. 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 652987] Review Request: go - The Go programming language

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


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

--- Comment #51 from Albert Strasheim full...@gmail.com 2012-03-14 02:58:49 
EDT ---
The Go 1 release is approaching. Any updates here?

I am thinking of putting together a spec file so that multiple versions of the
Go SDK can be installed in parallel. It will allow users to switch between them
using alternatives like you can when multiple Java SDKs are installed.

It would make it easy to test out weeklies in between stable releases of Go 1.

We could then also package extra stuff like go.crypto, goprotobuf, etc. to work
with the releases they are compatible with.

Potentially this could also allow switching between a /usr/bin/go that uses
gccgo instead of [568]g by default.

Another thought: it would be useful to make an ARM package for the Fedora ARM
on Raspberry Pi effort.

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

[Bug 750898] Review Request: icaro - A robot automation language

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


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

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

   What|Removed |Added

 CC||rc040...@freenet.de

--- Comment #7 from Ralf Corsepius rc040...@freenet.de 2012-03-14 03:02:42 
EDT ---
2 nits:

* %release is poorly chosen. It needs to contain a fedora specific release
number.
You are trying to reuse some upstream release number, which doesn't leave the
fedora package maintainer an option to increment it.

* Please move the chmod -x from %build into %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 798248] Review Request: rubygem-ruby-dbus - Ruby module for interaction with D-Bus

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


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

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

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #11 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-03-14 
04:01:13 EDT ---
Package Change Request
==
Package Name: rubygem-ruby-dbus
New Branches: f16
Owners: bkabrda

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

[Bug 737293] Review Request: python-django - A high-level Python Web framework

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


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

--- Comment #18 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-03-14 
04:22:16 EDT ---
Ok, looking at the spec and srpm from Comment 11:
- I think the definitions of python_sitelib and pyver don't have to be there
(I'm not familiar with the EPEL's, but I think that these variables are always
defined in python-devel.
- As for the tests, I have run into similar issues with one package and the
problem was, that when the output was redirected into a file (as mock does,
when run from outside), the tests failed, but when ran from inside, the tests
ran ok. So I'm gonna try to dig into this and find out a solution so that we
can run the tests - if I'm not able to find it in a reasonable time, then we
should probably comment the tests out, until it is sorted out. Is this ok for
you all?

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

[Bug 718395] Review Request: libmusicbrainz4-4.0.0 - Library for accessing MusicBrainz servers

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


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

--- Comment #16 from Christophe Fergeau cferg...@redhat.com 2012-03-14 
05:05:04 EDT ---
*** Bug 802962 has been marked as a duplicate of this bug. ***

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

[Bug 770615] Review Request: baobab - A graphical directory tree analyzer

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


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

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 770615] Review Request: baobab - A graphical directory tree analyzer

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


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

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2012-03-14 05:48:06 EDT ---
baobab-3.3.3-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/baobab-3.3.3-1.fc17

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

[Bug 737293] Review Request: python-django - A high-level Python Web framework

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


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

--- Comment #19 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-03-14 
05:47:37 EDT ---
I think I got it. The rpmbuilds (both mock and non-mock) set the environment
LANG=C, which effectively forces everything to work in us-ascii. When you put a
statement export LANG=en_US.utf8 just before running tests, all tests run
fine. I'm however not sure, what is the real cause of this issue. My guess is,
that the environment values somehow get into the Python mechanism and cause
troubles - maybe some of you can come up with an explanation.

Anyway, using export LANG=en_US.utf8 is the way to run all tests and then I
see no problem with approving this package.

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

[Bug 611372] Review Request: perl-Net-Twitter - Perl interface to Twitter

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


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

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

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

--- Comment #11 from Petr Šabata psab...@redhat.com 2012-03-14 05:52:57 EDT 
---
I'll do the review and possibly sponsor you once I'm happy with the package :)

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

[Bug 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

Juan Hernández juan.hernan...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |commons-ognl - Object Graph |apache-commons-ognl -
   |Navigation Library  |Object Graph Navigation
   ||Library

--- Comment #9 from Juan Hernández juan.hernan...@redhat.com 2012-03-14 
05:55:02 EDT ---
The package doesn't build in Koji. I think this is due to a bug in JNA.

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

[Bug 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

Juan Hernández juan.hernan...@redhat.com changed:

   What|Removed |Added

 Depends on||803241

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

[Bug 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

--- Comment #10 from Juan Hernández juan.hernan...@redhat.com 2012-03-14 
06:07:47 EDT ---
I was wrong regarding the jna dependency. It is not required as it is a
transitive dependency. The problem is that there is currently an issue with the
dependencies map in jna. See bug #803241.

This package looks nice now. I will approve once bug #803241 is fixed and I can
build in Koji.

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

[Bug 799896] Review Request: python-django-tracking - Django site visitor tracking, including basic blacklisting

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


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

--- Comment #3 from Matthias Runge mru...@matthias-runge.de 2012-03-14 
06:20:37 EDT ---
Cédric, thank you for your hint! 

Fixed SPEC: http://www.matthias-runge.de/fedora/python-django-tracking.spec
SRPM:
http://www.matthias-runge.de/fedora/python-django-tracking-0.3.7-7.fc16.src.rpm

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

[Bug 786093] Review Request: python-windmill - A web application testing framework

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


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

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

   What|Removed |Added

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

--- Comment #8 from Karel Klíč kk...@redhat.com 2012-03-14 06:44:32 EDT ---
Thank you for the review.

New Package SCM Request
===
Package Name: python-windmill
Short Description: A web application testing framework
Owners: kklic
Branches: f16 f17 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 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

Bug 794715 depends on bug 803241, which changed state.

Bug 803241 Summary: jna artifact duplicated and pointing to platform.jar
https://bugzilla.redhat.com/show_bug.cgi?id=803241

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|NEW |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 615047] Review Request: scim-panel-vkb-gtk - A GTK Virtual Keyboard for SCIM

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


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

Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX
Last Closed||2012-03-14 07:21:17

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

[Bug 800720] Review Request: resteasy - Framework for RESTful Web services and Java applications

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


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

--- Comment #4 from Juan Hernández juan.hernan...@redhat.com 2012-03-14 
07:39:18 EDT ---

=== REQUIRED ITEMS ===
[!]  Rpmlint output:

Output of rpmlint of the source package:

$ rpmlint resteasy-2.3.2-1.fc17.src.rpm
resteasy.src: W: name-repeated-in-summary C RESTEasy
resteasy.src: W: invalid-url URL: http://www.jboss.org/resteasy HTTP Error 403:
Forbidden
resteasy.src:217: W: macro-in-comment %{namedversion}
resteasy.src:218: W: macro-in-comment %{buildroot}
resteasy.src:218: W: macro-in-comment %{_javadir}
resteasy.src:218: W: macro-in-comment %{name}
...

The rpmlint command complais about macros in the comments, but it has been
discussed and agreed in #fedora-java that these comments are acceptable in this
case.

Output of rpmlint of the binary packages:

$ rpmlint resteasy-2.3.2-1.fc18.noarch.rpm
resteasy-javadoc-2.3.2-1.fc18.noarch.rpm 
resteasy.noarch: E: explicit-lib-dependency cglib
resteasy.noarch: W: name-repeated-in-summary C RESTEasy
resteasy.noarch: W: invalid-url URL: http://www.jboss.org/resteasy HTTP Error
403: Forbidden
resteasy-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs - Java
docs, Java-docs, Avocados
resteasy-javadoc.noarch: W: invalid-url URL: http://www.jboss.org/resteasy HTTP
Error 403: Forbidden
2 packages and 0 specfiles checked; 1 errors, 4 warnings.

The cglib dependency is not a library, but a legitimate package name.

[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.

The License.html file states that the license used is ASL 2.0, but them some
files contain different license statements:

* The files under tjws use a BDS style license. They are used mostly for
tests and we are building and packaging them.

* The files under the eagledns use a BSD style license. The latest upstream
version of this software uses LGPLv3. The resteasy upstream developers forked
it, but it is used only for some examples and tests and we are not building or
packaging it at the moment.

* These files tate that the license is CDDL:

jaxrs-api/src/main/java/javax/ws/rs/PathParam.java
jaxrs-api/src/main/java/javax/ws/rs/Consumes.java
jaxrs-api/src/main/java/javax/ws/rs/PUT.java
jaxrs-api/src/main/java/javax/ws/rs/DefaultValue.java
jaxrs-api/src/main/java/javax/ws/rs/ext/MessageBodyWriter.java
jaxrs-api/src/main/java/javax/ws/rs/ext/Providers.java
jaxrs-api/src/main/java/javax/ws/rs/ext/RuntimeDelegate.java
jaxrs-api/src/main/java/javax/ws/rs/ext/Provider.java
jaxrs-api/src/main/java/javax/ws/rs/ext/ContextResolver.java
jaxrs-api/src/main/java/javax/ws/rs/ext/ExceptionMapper.java
jaxrs-api/src/main/java/javax/ws/rs/ext/MessageBodyReader.java
jaxrs-api/src/main/java/javax/ws/rs/ext/FactoryFinder.java
jaxrs-api/src/main/java/javax/ws/rs/DELETE.java
jaxrs-api/src/main/java/javax/ws/rs/HEAD.java
jaxrs-api/src/main/java/javax/ws/rs/POST.java
jaxrs-api/src/main/java/javax/ws/rs/ApplicationPath.java
jaxrs-api/src/main/java/javax/ws/rs/WebApplicationException.java
jaxrs-api/src/main/java/javax/ws/rs/MatrixParam.java
jaxrs-api/src/main/java/javax/ws/rs/Produces.java
jaxrs-api/src/main/java/javax/ws/rs/GET.java
jaxrs-api/src/main/java/javax/ws/rs/core/SecurityContext.java
jaxrs-api/src/main/java/javax/ws/rs/core/UriBuilder.java
jaxrs-api/src/main/java/javax/ws/rs/core/Variant.java
jaxrs-api/src/main/java/javax/ws/rs/core/CacheControl.java
jaxrs-api/src/main/java/javax/ws/rs/core/Context.java
jaxrs-api/src/main/java/javax/ws/rs/core/Response.java
jaxrs-api/src/main/java/javax/ws/rs/core/HttpHeaders.java
jaxrs-api/src/main/java/javax/ws/rs/core/UriBuilderException.java
jaxrs-api/src/main/java/javax/ws/rs/core/MediaType.java
jaxrs-api/src/main/java/javax/ws/rs/core/StreamingOutput.java
jaxrs-api/src/main/java/javax/ws/rs/core/NewCookie.java
jaxrs-api/src/main/java/javax/ws/rs/core/GenericEntity.java
jaxrs-api/src/main/java/javax/ws/rs/core/MultivaluedMap.java
jaxrs-api/src/main/java/javax/ws/rs/core/Request.java
jaxrs-api/src/main/java/javax/ws/rs/core/EntityTag.java
jaxrs-api/src/main/java/javax/ws/rs/core/PathSegment.java
jaxrs-api/src/main/java/javax/ws/rs/core/Application.java
jaxrs-api/src/main/java/javax/ws/rs/core/Cookie.java
jaxrs-api/src/main/java/javax/ws/rs/core/UriInfo.java
jaxrs-api/src/main/java/javax/ws/rs/Path.java
jaxrs-api/src/main/java/javax/ws/rs/HttpMethod.java
jaxrs-api/src/main/java/javax/ws/rs/OPTIONS.java

[Bug 463771] Package review: perl-Probe-Perl

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


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

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

   What|Removed |Added

 CC||p...@city-fan.org

--- Comment #5 from Paul Howarth p...@city-fan.org 2012-03-14 07:43:16 EDT ---
Marcela, could I please have an EPEL-5 branch for this package?

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

[Bug 800720] Review Request: resteasy - Framework for RESTful Web services and Java applications

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


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

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 CC||mgold...@redhat.com

--- Comment #5 from Marek Goldmann mgold...@redhat.com 2012-03-14 07:49:04 
EDT ---
Regarding licensing issues - legal and upstream were notified.

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

[Bug 251128] Review Request: perl-Test-Script - Cross-platform basic tests for scripts

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


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

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

   What|Removed |Added

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

--- Comment #9 from Paul Howarth p...@city-fan.org 2012-03-14 07:52:41 EDT ---
Package Change Request
==
Package Name: perl-Test-Script
New Branches: EL-5
Owners: pghmcfc
InitialCC: perl-sig

Ralf has previously declared his non-interest in EPEL, which is why I am
requesting this branch myself.

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

[Bug 803018] Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format

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


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

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #3 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:04:12 EDT 
---
That's an excellent question I included it from the lzip spec, and I've seen
several packages include them, but I've never seen anything done with them.  I
suppose it would ease verification of the SRPM vs. upstream for end users.

Anyway, thanks for the review!

New Package SCM Request
===
Package Name: lziprecover
Short Description: Data recovery tool and decompressor for files in the lzip
compressed format
Owners: limb
Branches: 
InitialCC:

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

[Bug 773011] Review Request: api-sanity-checker - An automatic generator of basic unit tests for a shared C/C++ library.

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


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

--- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:15:58 EDT 
---
Build fails on help2man:

+ help2man -h --info -o
/home/limb/rpmbuild/BUILDROOT/api-sanity-checker-1.12.9-2.fc16.x86_64/usr/share/man/man1/api-sanity-checker.1
-N api-sanity-checker
help2man: can't get `--info' info from api-sanity-checker
Try `--no-discard-stderr' if option outputs to stderr


Change help2man line to:

help2man -o %{buildroot}%{_mandir}/man1/api-sanity-checker.1 -N 'perl
api-sanity-checker.pl'

License tag needs updating to GPLv2.

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

[Bug 392261] Review Request: perl-File-Find-Rule-Perl - Common rules for searching for Perl things

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


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

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

   What|Removed |Added

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

--- Comment #5 from Paul Howarth p...@city-fan.org 2012-03-14 08:16:57 EDT ---
Package Change Request
==
Package Name: perl-File-Find-Rule-Perl
New Branches: EL-5
Owners: pghmcfc
InitialCC: perl-sig

Ralf has previously declared his non-interest in EPEL, which is why I am
requesting this branch myself.

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

[Bug 392271] Review Request: perl-Perl-MinimumVersion - Find a minimum required version of perl

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


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

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

   What|Removed |Added

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

--- Comment #5 from Paul Howarth p...@city-fan.org 2012-03-14 08:20:15 EDT ---
Package Change Request
==
Package Name: perl-Perl-MinimumVersion
New Branches: EL-5
Owners: pghmcfc
InitialCC: perl-sig

Ralf has previously declared his non-interest in EPEL, which is why I am
requesting this branch myself.

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

[Bug 801092] Review Request: sumwars - a hack and slash role playing game

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


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

--- Comment #8 from Martin Preisler mprei...@redhat.com 2012-03-14 08:18:38 
EDT ---
I tried to address all issues, hopefully I haven't missed anything. Thanks
everyone for comments!

Spec URL: http://mpreisle.fedorapeople.org/pkgs/sumwars/sumwars.spec
SRPM URL:
http://mpreisle.fedorapeople.org/pkgs/sumwars/sumwars-0.5.6-2.fc16.src.rpm

Changes:
* added Amusements/Games group to the base package
* cmake flags -DSUMWARS_NO_TINYXML:BOOL=ON and -DSUMWARS_NO_ENET:BOOL=ON added
(use system wide tinyxml and enet), added -devel packages of these to
BuildRequires
* snippet from Brendan Jones that deals with encoding of AUTHORS in %prep

Regarding CMAKE_BUILD_TYPE, I am using the upstream default RelWithDebInfo to
get a release build with debug info, debug info is stripped to -debug
subpackage.

Copyright:
* /tools/{meshtest,graphicengine}/Example* - this sample code is public domain,
I have asked if it's even used in sumwars, could be just leftover
* ./share/resources/gui/imagesets/TaharezLook.tga is a derivate of CEGUI stock
datafile, it's CC-BY-SA only in sumwars
* ./src/core/nlfg.h - I listed MIT in Licenses
* I am not sure what to do about the Find*.cmake modules, they are used all
over the place, does everyone list BSD just because of this?

Mutual dependency is intentional.

Fixed the desktop file with suggestions given above, I will suggest upstream to
adopt this.

I am removing OgreCore.zip after install, it is not used according to
resources.cfg

Instead of using the bundled DejaVu fonts I am removing them and installing
symbolic links (also added dejavu-{sans,serif}-fonts to Requires).

The errorchecker.py script would be great but it checks for everything in the
same folder as the executable is, wouldn't work for this scenario.

rpmlint output changed for sumwars-data:
$ rpmlint ~/rpmbuild/RPMS/noarch/sumwars-data-0.5.6-2.fc16.noarch.rpm 
sumwars-data.noarch: W: no-documentation
sumwars-data.noarch: W: dangling-symlink
/usr/share/sumwars/resources/gui/fonts/DejaVuSerif.ttf
/usr/share/fonts/dejavu/DejaVuSerif.ttf
sumwars-data.noarch: W: dangling-symlink
/usr/share/sumwars/resources/gui/fonts/DejaVuSans.ttf
/usr/share/fonts/dejavu/DejaVuSans.ttf

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

[Bug 251128] Review Request: perl-Test-Script - Cross-platform basic tests for scripts

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


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

--- Comment #10 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:23:55 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 392261] Review Request: perl-File-Find-Rule-Perl - Common rules for searching for Perl things

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


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

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:24:19 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 772432] Review Request: gnome-applet-sensors - GNOME panel applet for hardware sensors

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


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

--- Comment #11 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:25:20 EDT 
---
Must specify owners.

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

[Bug 392271] Review Request: perl-Perl-MinimumVersion - Find a minimum required version of perl

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


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

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:24:35 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 251128] Review Request: perl-Test-Script - Cross-platform basic tests for scripts

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


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

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

   What|Removed |Added

 Blocks||729601

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

[Bug 392271] Review Request: perl-Perl-MinimumVersion - Find a minimum required version of perl

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


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

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

   What|Removed |Added

 Blocks||729601

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

[Bug 392261] Review Request: perl-File-Find-Rule-Perl - Common rules for searching for Perl things

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


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

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

   What|Removed |Added

 Blocks||729601

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

[Bug 786093] Review Request: python-windmill - A web application testing framework

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


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

--- Comment #9 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:26:37 EDT 
---
Git done (by process-git-requests).

Jiri, please take ownership of review BZs.  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 771252] Review Request: cinnamon - Window management and application launching for GNOME

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


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

--- Comment #36 from Christoph Wickert cwick...@fedoraproject.org 2012-03-14 
08:21:50 EDT ---
(In reply to comment #35)
 We don't
 do code review as part of the review process clearly and there is no real
 history of even checking for functionality.  

I agree that a code review is not mandatory part of a package review,
nevertheless I consider it very useful. I recall a review that revealed serious
bugs and even a security issue in one of my packages. Me and the reviewer
worked on patches and I upstreamed them before the package was released in
Fedora. This is how successful collaboration between developers and package
maintainers should look like.

Besides that, checking for basic functionality *is* definitely part of the
review checklist: The reviewer should test that the package functions as
described. A package should not segfault instead of running, for example.

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

[Bug 798248] Review Request: rubygem-ruby-dbus - Ruby module for interaction with D-Bus

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


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

--- Comment #12 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:27: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 801439] Review Request: rubygem-netrc - Library to read and write netrc files

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


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

--- Comment #8 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:28:31 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 803018] Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format

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


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

--- Comment #4 from Jon Ciesla limburg...@gmail.com 2012-03-14 08:29:10 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 803317] New: Review Request: trac-themeengine-plugin - Simple API and GUI for packaging and configuring Trac themes

2012-03-14 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: trac-themeengine-plugin - Simple API and GUI for 
packaging and configuring Trac themes

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

   Summary: Review Request: trac-themeengine-plugin - Simple API
and GUI for packaging and configuring Trac themes
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: andrea.v...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL:
http://averi.fedorapeople.org/RPMs/trac-themeengine-plugin/trac-themeengine-plugin.spec
SRPM URL:
http://averi.fedorapeople.org/RPMs/trac-themeengine-plugin/trac-themeengine-plugin-2.0.1-20120314svn11382.fc16.src.rpm
Description: A trac plugin that adds a simple API and GUI for packaging and
configuring Trac themes.

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

[Bug 803018] Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format

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


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

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-03-14 09:01:09

--- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-03-14 09:01:09 EDT 
---
Imported and built.

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

[Bug 463771] Package review: perl-Probe-Perl

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


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

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

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #6 from Marcela Mašláňová mmasl...@redhat.com 2012-03-14 09:10:58 
EDT ---
New Package CVS Request
===
Package Name: perl-Probe-Perl
Short Description: obtain information about perl interpreter
Owners: pghmcfc
Branches: EL-5
InitialCC: perl-devel

[Paul asked me for a new branch for EL-5.]

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

[Bug 463771] Package review: perl-Probe-Perl

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


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

--- Comment #7 from Jon Ciesla limburg...@gmail.com 2012-03-14 09:15:17 EDT 
---
Already exists, use a Package Change request to add a branch.  And use
perl-sig, not perl-devel.

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

[Bug 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

Juan Hernández juan.hernan...@redhat.com changed:

   What|Removed |Added

 Blocks|652183(FE-JAVASIG)  |
   Flag|fedora-review?  |fedora-review+

--- Comment #11 from Juan Hernández juan.hernan...@redhat.com 2012-03-14 
09:17:56 EDT ---
Now that the issue in jna has been addressed the package builds correctly in
Koji:

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

The binary packages are fine as well:

$ rpmlint apache-commons-ognl-3.0.2-1.20120313svn1102435.fc18.noarch.rpm
apache-commons-ognl-javadoc-3.0.2-1.20120313svn1102435.fc18.noarch.rpm
2 packages and 0 specfiles checked; 0 errors, 0
warnings.http://koji.fedoraproject.org/koji/taskinfo?taskID=3893599

All the issues have been addressed, and #4 was in fact not needed. You may want
to double check that and remove it before committing.


*** 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 801439] Review Request: rubygem-netrc - Library to read and write netrc files

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2012-03-14 
09:26:07 EDT ---
rubygem-netrc-0.7.1-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/rubygem-netrc-0.7.1-1.fc17

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

[Bug 801439] Review Request: rubygem-netrc - Library to read and write netrc files

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


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

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 737293] Review Request: python-django - A high-level Python Web framework

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


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

--- Comment #20 from Matthias Runge mru...@matthias-runge.de 2012-03-14 
09:30:55 EDT ---
Great catch! export LANG=en_US.utf8 did it for me for rpmbuild; mock --rebuild
is fine, too.

koji still reports errors (I didn't increment the release number, took the last
srpm from salimma). The error messages lead me to some networking problems. 
(Tests try to connect to google.com, afaik, the builders are not allowed to do
that).

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


..s..s..EE...s.x.ss..!
 
s..ss...xs...s..!
 s.!
 
s...s..s.....x...!
 
...E.EEss..s...
==
ERROR: test_correct_url_value_passes
(modeltests.validation.tests.BaseModelValidationTests)
--
Traceback (most recent call last):
  File
/builddir/build/BUILD/Django-1.3.1/tests/modeltests/validation/tests.py, line
63, in test_correct_url_value_passes
self.assertEqual(None, mtv.full_clean()) # This will fail if there's no
Internet connection
  File /builddir/build/BUILD/Django-1.3.1/django/db/models/base.py, line 828,
in full_clean
raise ValidationError(errors)
ValidationError: {'url_verify': [u'This URL appears to be a broken link.']}

[Bug 463771] Package review: perl-Probe-Perl

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Paul Howarth p...@city-fan.org 2012-03-14 09:36:08 EDT ---
Package Change Request
==
Package Name: perl-Probe-Perl
New Branches: EL-5
Owners: pghmcfc
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 737293] Review Request: python-django - A high-level Python Web framework

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


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

--- Comment #21 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-03-14 
09:36:08 EDT ---
Yes, koji builds are in no-network environment. So perhaps you could create a
patch that would comment these out - or just run the tests in BUILD rather than
BUILDROOT - in BUILD, you can modify the files all you want (so possibly delete
the tests entirely) and the changes won't actually get to BUILDROOT (and
therefore they won't get into the resulting srpm/rpms).

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

[Bug 463771] Package review: perl-Probe-Perl

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


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

--- Comment #9 from Jon Ciesla limburg...@gmail.com 2012-03-14 09:40:00 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 802862] Review Request: drupal6-votingapi - Voting API module for Drupal6

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


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

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

   What|Removed |Added

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

--- Comment #1 from Rex Dieter rdie...@math.unl.edu 2012-03-14 09:39:50 EDT 
---
I'll take a gander.

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

[Bug 784605] Review Request: lv2-instance-access: An LV2 audio plug-in extension which enables plugin UIs access to an LV2 plugin

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


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

Martin Preisler mprei...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Martin Preisler mprei...@redhat.com 2012-03-14 09:44:02 
EDT ---
You are probably right regarding noarch. Also the binary blob I was talking
about is just the infamous waf self extracting part and while it's horrible,
it's also acceptable with current policy.

regarding rpmlint messages:
* no-binary is safe to ignore in this case
* the dangling symlink as well

OK matches upstream tarball
OK naming guidelines
OK Fedora approved license
OK license matches upstream
OK package builds
OK permissions
OK spec file clean and legible,
OK no desktop file
OK owns all directories it creates

The package is very similar to lv2-ui which has already been approved. Looks
like hdegoede did most of the work for me and you applied all that to this
package :-D

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 803350] New: Review Request: python-django-simple-captcha - Django application to add captcha images to any Django form

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

Summary: Review Request: python-django-simple-captcha - Django application to 
add captcha images to any Django form

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

   Summary: Review Request: python-django-simple-captcha - Django
application to add captcha images to any Django form
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mru...@matthias-runge.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://www.matthias-runge.de/fedora/python-django-simple-captcha.spec
SRPM URL:
http://www.matthias-runge.de/fedora/python-django-simple-captcha-0.3.0-2.fc16.src.rpm
Description: Django Simple Captcha is an extremely simple, yet highly
customizable
Django application to add captcha images to any Django form


koji scratchbuild http://koji.fedoraproject.org/koji/taskinfo?taskID=3893730

[mrunge@mrungexp SPECS]$ rpmlint ./python-django-simple-captcha.spec
../SRPMS/python-django-simple-captcha-0.3.0-2.fc16.src.rpm
../RPMS/noarch/python-django-simple-captcha-0.3.0-2.fc16.noarch.rpm 
python-django-simple-captcha.src: W: spelling-error %description -l en_US
customizable - customization
python-django-simple-captcha.noarch: W: spelling-error %description -l en_US
customizable - customization
2 packages and 1 specfiles checked; 0 errors, 2 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 803350] Review Request: python-django-simple-captcha - Django application to add captcha images to any Django form

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


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

Matthias Runge mru...@matthias-runge.de changed:

   What|Removed |Added

 Blocks||736776

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

[Bug 801092] Review Request: sumwars - a hack and slash role playing game

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


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

--- Comment #9 from Martin Erik Werner martinerikwer...@gmail.com 2012-03-14 
09:59:43 EDT ---
(In reply to comment #8)
 I tried to address all issues, hopefully I haven't missed anything. Thanks
 everyone for comments!
 
 Spec URL: http://mpreisle.fedorapeople.org/pkgs/sumwars/sumwars.spec
 SRPM URL:
 http://mpreisle.fedorapeople.org/pkgs/sumwars/sumwars-0.5.6-2.fc16.src.rpm

(...)

 Copyright:
 * /tools/{meshtest,graphicengine}/Example* - this sample code is public 
 domain,
 I have asked if it's even used in sumwars, could be just leftover
 * ./share/resources/gui/imagesets/TaharezLook.tga is a derivate of CEGUI stock
 datafile, it's CC-BY-SA only in sumwars
 * ./src/core/nlfg.h - I listed MIT in Licenses
 * I am not sure what to do about the Find*.cmake modules, they are used all
 over the place, does everyone list BSD just because of this?

I inquired on #fedora-devel and the word there was that since it's not linked
into the final binaries it is not required to be listed in the spec file.

Also, as per:
https://fedoraproject.org/wiki/Licensing/FAQ#What_is_.22effective_license.22_and_do_I_need_to_know_that_for_the_License:_tag.3F
...you might not be required to list 'MIT' at all, since the effective license
would be GPLv3+ there. So in fact my copyright comments seems to have a bit
unnecessary (I'm used to detailing the whole source tarball Debian-style :)

I also asked about subpackage licensing and it might be reasonable to only
state 'GPLv3+' for the main package and add a License: tag with 'CC-BY-SA' to
the -data %package section instead. Provided the license separation is as clear
as this, of course.


Overall great stuff! I don't really see anything that's awry at the moment,
IANAReviewer though, so Brendan has the final word there ;)

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

[Bug 463771] Package review: perl-Probe-Perl

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


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

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2012-03-14 10:05:41 EDT ---
perl-Probe-Perl-0.01-4.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/perl-Probe-Perl-0.01-4.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 800294] Review Request: python-django-piston - A mini-framework for Django for creating RESTful APIs

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


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

Matthias Runge mru...@matthias-runge.de changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-03-14 10:11:07

--- Comment #7 from Matthias Runge mru...@matthias-runge.de 2012-03-14 
10:11:07 EDT ---
build, orphaned, blocked

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

[Bug 799650] Review Request: python-django-tagging - A generic tagging application for Django projects

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


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

Matthias Runge mru...@matthias-runge.de changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||RAWHIDE
Last Closed||2012-03-14 10:08:03

--- Comment #6 from Matthias Runge mru...@matthias-runge.de 2012-03-14 
10:08:03 EDT ---
built, old package blocked, orphaned for f18+

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

[Bug 802862] Review Request: drupal6-votingapi - Voting API module for Drupal6

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


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

--- Comment #2 from Wesley Hearn whe...@redhat.com 2012-03-14 10:38:59 EDT ---
Found that the wrong LICENSE.txt was included I have updated it to include the
correct one.
SPEC: http://jknife.fedorapeople.org/SPECS/drupal6-votingapi.spec
SRPM: http://jknife.fedorapeople.org/SRPMS/drupal6-votingapi-2.3-4.el6.src.rpm

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

[Bug 687987] Review Request: plymouth-theme-hot-dog - Plymouth Happy Hot Dog Theme

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


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

John Brier jbr...@redhat.com changed:

   What|Removed |Added

 CC||jbr...@redhat.com

--- Comment #18 from John Brier jbr...@redhat.com 2012-03-14 10:50:34 EDT ---
What is the status of this? I don't see the package in F16

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

[Bug 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

Andy Grimm agr...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #12 from Andy Grimm agr...@gmail.com 2012-03-14 10:49:34 EDT ---
New Package SCM Request
===
Package Name: apache-commons-ognl
Short Description: Object Graph Navigation Library
Owners: arg
Branches: f17
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 251128] Review Request: perl-Test-Script - Cross-platform basic tests for scripts

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


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

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2012-03-14 10:53:48 EDT ---
perl-Test-Script-1.06-1.el5.1 has been submitted as an update for Fedora EPEL
5.
https://admin.fedoraproject.org/updates/perl-Test-Script-1.06-1.el5.1

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

[Bug 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

--- Comment #13 from Jon Ciesla limburg...@gmail.com 2012-03-14 11:05:35 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 803376] Review Request: weld-parent - Parent POM for Weld

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


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

Marek Goldmann mgold...@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 803376] New: Review Request: weld-parent - Parent POM for Weld

2012-03-14 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: weld-parent - Parent POM for Weld

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

   Summary: Review Request: weld-parent - Parent POM for Weld
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mgold...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL:
http://goldmann.fedorapeople.org/package_review/weld-parent/1/weld-parent.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/weld-parent/1/weld-parent-17-1.fc17.src.rpm
Description: Parent POM for Weld

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

[Bug 803381] New: Review Request: weld-api - Weld API

2012-03-14 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: weld-api - Weld API

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

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


Spec URL:
http://goldmann.fedorapeople.org/package_review/weld-api/1/weld-api.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/weld-api/1/weld-api-1.1-1.fc17.src.rpm
Description: Weld specifc extensions to the CDI API

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

[Bug 803376] Review Request: weld-parent - Parent POM for Weld

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


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

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks||803381

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

[Bug 730234] Review Request: jboss-ejb-3.1-api - EJB 3.1 API

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


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

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks||803381

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

[Bug 730306] Review Request: jboss-interceptors-1.1-api - Interceptors 1.1 API

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


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

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks||803381

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

[Bug 803381] Review Request: weld-api - Weld API

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


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

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends on||730306, 730234, 803376
 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 784605] Review Request: lv2-instance-access: An LV2 audio plug-in extension which enables plugin UIs access to an LV2 plugin

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


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

Brendan Jones brendan.jones...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Brendan Jones brendan.jones...@gmail.com 2012-03-14 
11:39:42 EDT ---
Thanks for the review!

New Package SCM Request
===
Package Name: lv2-instance-access
Short Description: An LV2 audio plug-in UI extension 
Owners: bsjones
Branches: f15 f16 f17
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 652987] Review Request: go - The Go programming language

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


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

--- Comment #52 from W. Michael Petullo m...@flyn.org 2012-03-14 11:47:27 EDT 
---
Go for it. I have not had the time to update this proposed package in a while.
It sounds like you have some good ideas. I'd be happy to transition into a role
of reviewer.

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

[Bug 801092] Review Request: sumwars - a hack and slash role playing game

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


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

--- Comment #10 from Martin Preisler mprei...@redhat.com 2012-03-14 11:47:06 
EDT ---
Thanks Martin!

I have very slightly updated the spec:
http://mpreisle.fedorapeople.org/pkgs/sumwars/sumwars.spec

Separate license for -data subpackage, moved dejavu font Requires to data
(shouldn't hurt anything since the packages depend on each other but still,
this is more pragmatic).

Also removed MIT license since GPLv3+ is the effective license according to
the document refered.

I am not uploading a SRPM this time since that would be quite wasteful ;-)

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

[Bug 772432] Review Request: gnome-applet-sensors - GNOME panel applet for hardware sensors

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


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

Huzaifa S. Sidhpurwala huzai...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #12 from Huzaifa S. Sidhpurwala huzai...@redhat.com 2012-03-14 
11:52:12 EDT ---
Package Change Request
==
Package Name: gnome-applet-sensors
New Branches: f17
Owners: huzaifas

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

[Bug 784605] Review Request: lv2-instance-access: An LV2 audio plug-in extension which enables plugin UIs access to an LV2 plugin

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


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

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2012-03-14 11:57:10 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 772432] Review Request: gnome-applet-sensors - GNOME panel applet for hardware sensors

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


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

--- Comment #13 from Jon Ciesla limburg...@gmail.com 2012-03-14 12:14:49 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 786093] Review Request: python-windmill - A web application testing framework

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


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

Jiri Moskovcak jmosk...@redhat.com changed:

   What|Removed |Added

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

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

[Bug 803423] Review Request: perl-Net-IP-Minimal - Minimal functions from Net::IP

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


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

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

 Blocks||802998

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

[Bug 803423] New: Review Request: perl-Net-IP-Minimal - Minimal functions from Net::IP

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

Summary: Review Request: perl-Net-IP-Minimal - Minimal functions from Net::IP

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

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


Spec URL:
http://psabata.fedorapeople.org/pkgs/perl-Net-IP-Minimal/perl-Net-IP-Minimal.spec
SRPM URL:
http://psabata.fedorapeople.org/pkgs/perl-Net-IP-Minimal/perl-Net-IP-Minimal-0.02-1.fc16.src.rpm
Description:
Net::IP is very feature complete, but I found I was only using three of its
functions and it uses quite a bit of memory.


NOTE: This is a dependency of the latest POE::Component::Client::Keepalive.  I
plan this for Rawhide only.

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

[Bug 785923] Review Request: xgap - GUI for GAP

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


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

--- Comment #7 from Volker Fröhlich volke...@gmx.at 2012-03-14 13:14:28 EDT 
---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated



 C/C++ 
[-]: MUST Header files in -devel subpackage, if present.
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contain kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.


 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[-]: MUST Each %files section contains %defattr if rpm  4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[-]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package contains a properly installed %{name}.desktop using desktop-
 file-install file if it is a GUI application.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[!]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[-]: MUST If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[!]: MUST Rpmlint output is silent.

rpmlint xgap-debuginfo-4.21-2.fc18.i686.rpm

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


rpmlint xgap-4.21-2.fc18.i686.rpm

xgap.i686: E: non-standard-executable-perm /usr/bin/xgap.bin 0775L
xgap.i686: W: no-manual-page-for-binary xgap
xgap.i686: W: no-manual-page-for-binary xgap.bin
1 packages and 0 specfiles checked; 1 errors, 2 warnings.


rpmlint xgap-4.21-2.fc18.src.rpm

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


[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
/home/makerpm/785923/xgap4r21.zoo :
  MD5SUM this package : 160d5e8a4ddc872526ce57026cc92eb2
  MD5SUM upstream package : 160d5e8a4ddc872526ce57026cc92eb2

[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
 separate file from upstream, the packager SHOULD query upstream to
 include it.

I suppose there is no active upstream anymore.

[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[?]: SHOULD Package functions as described.

It starts, but I haven't tried further.

[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD Patches link to upstream bugs/comments/lists or are otherwise
 justified.
[x]: SHOULD 

[Bug 802862] Review Request: drupal6-votingapi - Voting API module for Drupal6

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


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

--- Comment #3 from Rex Dieter rdie...@math.unl.edu 2012-03-14 13:14:35 EDT 
---
$ rpmlint *.rpm noarch/*.rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

Sources: ok
$ md5sum *.gz
d448a23d95204b05a927067d0aa4a704  votingapi-6.x-2.3.tar.gz

Naming: ok

macros: ok

scriptlets: n/a

Licenses: NOT ok
seems the tarball includes a copy of LICENSE.txt which is standard GPLv2, and I
can't find mention of any or later clause to justify GPLv2+


SHOULD  consider using a simpler approach for handling
Source1: %{name}-fedora-README.txt

and drop the relevant existing %build and %files %exclude snippets, and just
use:
%files
%doc %{SOURCE1}



The rest of it is an *very* simple noarch package, though there seems to be
some unused drupal macro boilerplate, which you could trim too.


Otherwise, remaining issue is licensing per my comment above.

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

[Bug 783655] Review Request: aimage - Advanced Disk Imager

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


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

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

   What|Removed |Added

   Flag||needinfo?(kwiz...@gmail.com
   ||)

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

[Bug 802862] Review Request: drupal6-votingapi - Voting API module for Drupal6

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


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

--- Comment #5 from Rex Dieter rdie...@math.unl.edu 2012-03-14 13:41:41 EDT 
---
the links you provide seem to be the same as from comment #2 still. ??

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

[Bug 802862] Review Request: drupal6-votingapi - Voting API module for Drupal6

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


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

--- Comment #4 from Wesley Hearn whe...@redhat.com 2012-03-14 13:37:17 EDT ---
Ok, I removed the added LICENSE.txt from the SRPM and I am using the one inside
the tarball. I have also updated License from GPLv2+ to GPLv2.

SPEC: http://jknife.fedorapeople.org/SPECS/drupal6-votingapi.spec
SRPM: http://jknife.fedorapeople.org/SRPMS/drupal6-votingapi-2.3-4.el6.src.rpm

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

[Bug 802862] Review Request: drupal6-votingapi - Voting API module for Drupal6

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


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

--- Comment #6 from Wesley Hearn whe...@redhat.com 2012-03-14 13:47:19 EDT ---
Sorry about that

SPEC: http://jknife.fedorapeople.org/SPECS/drupal6-votingapi.spec
SRPM: http://jknife.fedorapeople.org/SRPMS/drupal6-votingapi-2.3-5.el6.src.rpm

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

[Bug 802862] Review Request: drupal6-votingapi - Voting API module for Drupal6

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


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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Rex Dieter rdie...@math.unl.edu 2012-03-14 13:57:03 EDT 
---
win, 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 797165] Review Request: jboss-el-2.2-api - Expression Language 2.2 API

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


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

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2012-03-14 14:11:24 EDT ---
jboss-el-2.2-api-1.0.1-0.2.20120212git2fabd8.fc17 has been submitted as an
update for Fedora 17.
https://admin.fedoraproject.org/updates/jboss-el-2.2-api-1.0.1-0.2.20120212git2fabd8.fc17

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

[Bug 783655] Review Request: aimage - Advanced Disk Imager

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


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

Nicolas Chauvet (kwizart) kwiz...@gmail.com changed:

   What|Removed |Added

   Flag|needinfo?(kwiz...@gmail.com |
   |)   |

--- Comment #5 from Nicolas Chauvet (kwizart) kwiz...@gmail.com 2012-03-14 
14:26:54 EDT ---
Hi,
I think the spec file is obvious. aimage requires at least a specific version
of afflib, period. 
That's exactly what BuildRequires afflib = 3.6.15 means.
aimage/afflib are both from afflib.org, so when one is updated another comes
along.

A previous aimage used to be in earlier fedora. I have orphaned it because it
was orphaned upstream over another tool. But as it has been resurrected, that's
need a new review.

Which format-warning are you talking about ?
There is no documentation
The description was picked from the website:
http://afflib.org/software/aimage-the-advanced-disk-imager

Thx for the review

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

[Bug 518318] Review Request: vanessa_socket - Simplify TCP/IP socket operations

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


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

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

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|or...@cora.nwra.com
  Status Whiteboard|NotReady|
   Flag||fedora-review?

--- Comment #16 from Orion Poplawski or...@cora.nwra.com 2012-03-14 14:27:19 
EDT ---
rpmlint:

vanessa_socket.i686: W: unused-direct-shlib-dependency
/usr/lib/libvanessa_socket.so.2.1.0 /lib/libdl.so.2

Looks like libvanessa_socket.so.2.1.0 does not need to be linked against -ldl.

vanessa_socket.i686: E: incorrect-fsf-address
/usr/share/doc/vanessa_socket-0.0.12/README
vanessa_socket-pipe.i686: E: incorrect-fsf-address
/usr/share/doc/vanessa_socket-pipe-0.0.12/COPYING

FSF now just uses their url:

You should have received a copy of the GNU General Public License
along with this program.  If not, see http://www.gnu.org/licenses/.

Please notify upstream.

More to come

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

[Bug 802758] Review Request: drupal6-drush - cli for drupal management

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


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

--- Comment #2 from Anderson Silva ansi...@redhat.com 2012-03-14 14:34:09 EDT 
---
update SRPM and SPEC:

Spec URL: http://r-bean2.rhcloud.com/drupal6-drush.spec
SRPM URL: http://r-bean2.rhcloud.com/drupal6-drush-4.1-3.el6_2.src.rpm
Description: 
Drush is a command line shell and scripting interface for Drupal,
a veritable Swiss Army knife designed to make life easier for
those of us who spend some of our working hours hacking away
at the command prompt.

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

[Bug 802758] Review Request: drupal6-drush - cli for drupal management

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


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

--- Comment #3 from Anderson Silva ansi...@redhat.com 2012-03-14 14:35:13 EDT 
---
apolgies, trying again: 

Spec URL: http://r-bean2.rhcloud.com/drupal6-drush.spec
SRPM URL: http://r-bean2.rhcloud.com/drupal6-drush-4.1-3.fc16.src.rpm
Description: 
Drush is a command line shell and scripting interface for Drupal,
a veritable Swiss Army knife designed to make life easier for
those of us who spend some of our working hours hacking away
at the command prompt.

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

[Bug 802761] Review Request: drupal6-geshifilter - geshifilter module for drupal6

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


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

--- Comment #2 from Anderson Silva ansi...@redhat.com 2012-03-14 14:36:02 EDT 
---
Spec URL: http://r-bean2.rhcloud.com/drupal6-geshifilter.spec
SRPM URL: http://r-bean2.rhcloud.com/drupal6-geshifilter-1.4-5.fc16.src.rpm
Description: 
The GeShi Filter module provides a filter for source code syntax highlighting
for a wide range of languages.

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

[Bug 791323] Review Request: jboss-remoting-jmx - JMX via JBoss Remoting

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


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

--- Comment #2 from Marek Goldmann mgold...@redhat.com 2012-03-14 14:34:02 
EDT ---
Spec URL:
http://goldmann.fedorapeople.org/package_review/jboss-remoting-jmx/2/jboss-remoting-jmx.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/jboss-remoting-jmx/2/jboss-remoting-jmx-1.0.2-1.fc17.src.rpm

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

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

[Bug 753262] Review Request: qxmpp-dev - Qt XMPP Library

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


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

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2012-03-14 14:46:31 EDT ---
qxmpp-dev-0.3.45.2-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/qxmpp-dev-0.3.45.2-1.fc17

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

[Bug 798014] Review Request: trac-advancedticketworkflow-plugin - Advanced workflow operations Trac plugin

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


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

Christoph Wickert cwick...@fedoraproject.org changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #3 from Christoph Wickert cwick...@fedoraproject.org 2012-03-14 
14:57:20 EDT ---
New Package SCM Request
===
Package Name: trac-advancedticketworkflow-plugin
Short Description: Advanced workflow operations Trac plugin
Owners: spot cwickert
Branches: f15 f16 f17 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 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2012-03-14 14:59:26 EDT ---
apache-commons-ognl-3.0.2-1.20120313svn1102435.fc17 has been submitted as an
update for Fedora 17.
https://admin.fedoraproject.org/updates/apache-commons-ognl-3.0.2-1.20120313svn1102435.fc17

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

[Bug 794715] Review Request: apache-commons-ognl - Object Graph Navigation Library

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


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

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 798014] Review Request: trac-advancedticketworkflow-plugin - Advanced workflow operations Trac plugin

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


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

--- Comment #4 from Jon Ciesla limburg...@gmail.com 2012-03-14 15:06:45 EDT 
---
Git done (by process-git-requests).

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

[Bug 711313] Review Request: wicd-kde - a Wicd client built on the KDE Development Platform

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


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

--- Comment #55 from Fedora Update System upda...@fedoraproject.org 
2012-03-14 15:12:01 EDT ---
wicd-kde-0.3.0-3.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/wicd-kde-0.3.0-3.fc17

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

[Bug 781260] Review Request: leechcraft - A Free Open Source Cross-Platform Modular Internet-Client

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


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

--- Comment #24 from Fedora Update System upda...@fedoraproject.org 
2012-03-14 15:46:11 EDT ---
leechcraft-0.5.0-4.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/leechcraft-0.5.0-4.fc17

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

  1   2   >