[Bug 593607] Review Request: plexus-active-collections - Plexus Container-Backed Active Collections

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


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

Yang Yong yy...@redhat.com changed:

   What|Removed |Added

 Blocks||593959

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


[Bug 593959] Review Request: maven-assembly-plugin - Maven Assembly Plugin

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


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

Yang Yong yy...@redhat.com changed:

   What|Removed |Added

 Depends on||593607

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


[Bug 593959] New: Review Request: maven-assembly-plugin - Maven Assembly Plugin

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

Summary: Review Request: maven-assembly-plugin - Maven Assembly Plugin

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

   Summary: Review Request: maven-assembly-plugin - Maven
Assembly Plugin
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: yy...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://yyang.fedorapeople.org/maven2/maven-assembly-plugin.spec
SRPM URL:
http://yyang.fedorapeople.org/maven2/maven-assembly-plugin-2.2-0.1.beta5.src.rpm
Description: A Maven 2 plugin to create archives of your project's sources,
classes, dependencies etc. from flexible assembly descriptors.

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


[Bug 593959] Review Request: maven-assembly-plugin - Maven Assembly Plugin

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


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

huwang huw...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

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


[Bug 593959] Review Request: maven-assembly-plugin - Maven Assembly Plugin

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


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

huwang huw...@redhat.com changed:

   What|Removed |Added

 CC||huw...@redhat.com
 AssignedTo|nob...@fedoraproject.org|huw...@redhat.com
   Flag||fedora-review?

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


[Bug 528288] Review Request: python-blist - A faster list implementation for Python

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


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

Michel Alexandre Salim michael.silva...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Michel Alexandre Salim michael.silva...@gmail.com 
2010-05-20 03:29:26 EDT ---
Thanks!

New Package CVS Request
===
Package Name: python-blist
Short Description: A faster list implementation for Python
Owners: salimma
Branches: F-12 F-13 EL-5 EL-6
InitialCC:

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


[Bug 460974] Review Request: xmobar - status bar for X

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


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

--- Comment #14 from Jens Petersen peter...@redhat.com 2010-05-20 04:14:44 
EDT ---
New upstream release:
http://hackage.haskell.org/package/xmobar-0.11.1

Anyone want to pick this up?

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


[Bug 593069] Review Request: pynie - Implementation of Python 3 on top of the Parrot virtual machine

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


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

--- Comment #2 from Gerd Pokorra g...@zimt.uni-siegen.de 2010-05-20 04:37:33 
EDT ---
Rakudo and Parrot RPMs will be updated every month. I ask every month for a
build root override to have no broken dependences with librarys between Rakudo
and Parrot. For having no broken dependences between Parrot and Pynie we need
pynie sources that build on the monthly releases of Parrot.

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


[Bug 590522] Review Request: plexus-cipher-1.4-1 - Plexus Cipher: encryption/decryption Component

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


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

--- Comment #12 from huwang huw...@redhat.com 2010-05-20 04:49:22 EDT ---
(In reply to comment #10)
 Excellent, please update your SPEC/SRPM and I will conduct a full review.

Sorry for the delay, as I updated it to the latest version, please review
again, thanks a lot.

spec URL:
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-cipher.speccan=2q=#makechanges

SRPM URL: 
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-cipher-1.5-1.src.rpmcan=2q=#makechanges

Note about this package:
 It built with dist-f14-maven221 target, koji building link is
http://koji.fedoraproject.org/koji/taskinfo?taskID=2198806

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


[Bug 593841] Review Request: wicd - Wireless and wired network connection manager

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||supercyp...@gmail.com
 Resolution||DUPLICATE

--- Comment #1 from Chen Lei supercyp...@gmail.com 2010-05-20 05:21:33 EDT ---


*** This bug has been marked as a duplicate of bug 511204 ***

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


[Bug 511204] Review Request: wicd - A wireless and wired network manager

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 CC||dcantr...@redhat.com

--- Comment #18 from Chen Lei supercyp...@gmail.com 2010-05-20 05:21:34 EDT 
---
*** Bug 593841 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 593839] Review Request: urwid - Console user interface library

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||supercyp...@gmail.com
 Resolution||DUPLICATE

--- Comment #1 from Chen Lei supercyp...@gmail.com 2010-05-20 05:25:30 EDT ---


*** This bug has been marked as a duplicate of bug 510097 ***

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


[Bug 510097] Review Request: python-urwid - console user interface for python

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 CC||dcantr...@redhat.com

--- Comment #31 from Chen Lei supercyp...@gmail.com 2010-05-20 05:25:30 EDT 
---
*** Bug 593839 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 593839] Review Request: urwid - Console user interface library

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


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

--- Comment #2 from Chen Lei supercyp...@gmail.com 2010-05-20 05:34:16 EDT ---
I suggest you to move to #510097, cwickert already take review for this
package. Since the reporter is non-responsive for a long time, you can take
that review, I think it'll be approved soon.

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


[Bug 511204] Review Request: wicd - A wireless and wired network manager

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


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

--- Comment #19 from leigh scott leigh123li...@googlemail.com 2010-05-20 
05:38:34 EDT ---
(In reply to comment #18)
 *** Bug 593841 has been marked as a duplicate of this bug. ***

Why close 593841 ?

I believe  Gareth John should submit a fresh review and this old review 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 492778] Review Request: xwine - Graphic Interface for Wine

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 CC||supercyp...@gmail.com
   Flag||needinfo?

--- Comment #5 from Chen Lei supercyp...@gmail.com 2010-05-20 05:52:27 EDT ---
How about to close this report? I think this package in not suitable for
fedora, e.g. Upstream is dead for long time, depend on very old gnome1.

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


[Bug 225764] Merge Review: fonts-ISO8859-2

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


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

Bug 225764 depends on bug 491976, which changed state.

Bug 491976 Summary: [fonts-ISO8859-2] Please rebuild for Fedora 11 to pick up 
font autodeps
https://bugzilla.redhat.com/show_bug.cgi?id=491976

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

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


[Bug 225764] Merge Review: fonts-ISO8859-2

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


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

Parag pnem...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||iso8859-2-fonts-1.0-24.fc14
 Resolution||NEXTRELEASE

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


[Bug 593999] New: Review Request: python-keyring - A python library for accessing the system keyring service

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

Summary: Review Request: python-keyring - A python library for accessing the 
system keyring service

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

   Summary: Review Request: python-keyring - A python library for
accessing the system keyring service
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: vcrho...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://vcrhonek.fedorapeople.org/python-keyring/python-keyring.spec
SRPM URL:
http://vcrhonek.fedorapeople.org/python-keyring/python-keyring-0.2-1.fc12.src.rpm
Description: The Python keyring lib provides a easy way to access the system
keyring service from python. It can be used in any application that needs safe
password storage.

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


[Bug 594002] Review Request: plexus-build-api - Plexus Build API

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


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

huwang huw...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?

--- Comment #1 from huwang huw...@redhat.com 2010-05-20 06:19:50 EDT ---
Note about this package:
 It build with dist-f14-maven221 target in koji, the link is
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-build-api-0.0.6-1.src.rpmcan=2q=#makechanges

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


[Bug 594002] New: Review Request: plexus-build-api - Plexus Build API

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

Summary: Review Request: plexus-build-api - Plexus Build API

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

   Summary: Review Request: plexus-build-api - Plexus Build API
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: huw...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-build-api.speccan=2q=#makechanges
SRPM URL:
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-build-api-0.0.6-1.src.rpmcan=2q=#makechanges
Description: Plexus Build 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 594002] Review Request: plexus-build-api - Plexus Build API

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


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

huwang huw...@redhat.com changed:

   What|Removed |Added

   Flag|needinfo?   |

--- Comment #2 from huwang huw...@redhat.com 2010-05-20 06:21:30 EDT ---
Please ignore comment 1.

Note about this package:
 It build with dist-f14-maven221 target in koji, the link is
http://koji.fedoraproject.org/koji/taskinfo?taskID=2198862

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


[Bug 594002] Review Request: plexus-build-api - Plexus Build API

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


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

--- Comment #4 from Yang Yong yy...@redhat.com 2010-05-20 06:29:21 EDT ---
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 594002] Review Request: plexus-build-api - Plexus Build API

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


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

Yang Yong yy...@redhat.com changed:

   What|Removed |Added

 CC||yy...@redhat.com

--- Comment #3 from Yang Yong yy...@redhat.com 2010-05-20 06:27:35 EDT ---
[yy...@localhost Desktop]$ rpmlint plexus-build-api-0.0.6-1.fc14.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

[yy...@localhost Desktop]$ rpmlint plexus-build-api-0.0.6-1.fc14.noarch.rpm 
plexus-build-api.noarch: W: no-documentation
plexus-build-api.noarch: W: non-conffile-in-etc
/etc/maven/fragments/plexus-build-api
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

[yy...@localhost Desktop]$ rpm -qpil plexus-build-api-0.0.6-1.fc14.src.rpm 
Name: plexus-build-api Relocations: (not relocatable)
Version : 0.0.6 Vendor: Fedora Project
Release : 1.fc14Build Date: Thu 20 May 2010
06:07:40 PM CST
Install Date: (not installed)   Build Host:
x86-03.phx2.fedoraproject.org
Group   : Development/Libraries Source RPM: (none)
Size: 8623 License: ASL 2.0
Signature   : (none)
Packager: Fedora Project
URL : http://svn.sonatype.org/spice/tags/plexus-build-api-0.0.6
Summary : Plexus Build API
Description :
Plexus Build API
plexus-build-api-0.0.6.tar.gz
plexus-build-api.spec

[yy...@localhost Desktop]$ rpm -qpil plexus-build-api-0.0.6-1.fc14.noarch.rpm 
Name: plexus-build-api Relocations: (not relocatable)
Version : 0.0.6 Vendor: Fedora Project
Release : 1.fc14Build Date: Thu 20 May 2010
06:09:53 PM CST
Install Date: (not installed)   Build Host:
x86-03.phx2.fedoraproject.org
Group   : Development/Libraries Source RPM:
plexus-build-api-0.0.6-1.fc14.src.rpm
Size: 11595License: ASL 2.0
Signature   : (none)
Packager: Fedora Project
URL : http://svn.sonatype.org/spice/tags/plexus-build-api-0.0.6
Summary : Plexus Build API
Description :
Plexus Build API
/etc/maven/fragments/plexus-build-api
/usr/share/java/plexus-build-api-0.0.6.jar
/usr/share/java/plexus-build-api.jar
/usr/share/maven2/poms/JPP-plexus-build-api.pom

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


[Bug 594002] Review Request: plexus-build-api - Plexus Build API

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


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

--- Comment #5 from Yang Yong yy...@redhat.com 2010-05-20 06:32:25 EDT ---
Please put the jars to /usr/share/java/plexus directory.

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


[Bug 593915] dconf - a configuration system

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


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

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

   What|Removed |Added

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

--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2010-05-20 06:32:07 EDT 
---

Review:
+ package builds in mock (rawhide i686).
koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=2198870
+ rpmlint output for SRPM and for RPM.
dconf.src: W: spelling-error %description -l en_US backend - backed, back end,
back-end
dconf.src: W: no-cleaning-of-buildroot %clean
dconf.src: W: no-buildroot-tag
dconf.src: W: no-%clean-section
dconf.x86_64: W: spelling-error %description -l en_US backend - backed, back
end, back-end
== false messages.

+ source files match upstream url (sha1sum)
49eafdb8a845e9d618a97a97313489259093e120  dconf-0.3.tar.bz2
49eafdb8a845e9d618a97a97313489259093e120  ../SOURCES/dconf-0.3.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ license is open source-compatible.
+ License text is included in package.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ ldconfig scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Package dconf-0.3-1.fc14.x86_64 =
Provides: libdconfsettings.so()(64bit)
Requires: libc.so.6()(64bit) libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.4)(64bit) libgio-2.0.so.0()(64bit) libglib-2.0.so.0()(64bit)
libgmodule-2.0.so.0()(64bit) libgobject-2.0.so.0()(64bit)
libgthread-2.0.so.0()(64bit) libpthread.so.0()(64bit) librt.so.1()(64bit)
rtld(GNU_HASH)
+ Not a GUI application

Suggestions:
1) It will be good to include some documentation say README file which can say
what this package provides. Please contact upstream and ask them to add README.

2) update summary as
Summary:A simple key-based configuration system

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 594002] Review Request: plexus-build-api - Plexus Build API

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


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

Yang Yong yy...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

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


[Bug 594002] Review Request: plexus-build-api - Plexus Build API

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


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

Yang Yong yy...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+

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


[Bug 594002] Review Request: plexus-build-api - Plexus Build API

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


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

--- Comment #6 from Yang Yong yy...@redhat.com 2010-05-20 07:01:02 EDT ---
[yy...@localhost Desktop]$ rpm -qpil plexus-build-api-0.0.6-1.fc14.noarch.rpm 
Name: plexus-build-api Relocations: (not relocatable)
Version : 0.0.6 Vendor: Fedora Project
Release : 1.fc14Build Date: Thu 20 May 2010
06:50:19 PM CST
Install Date: (not installed)   Build Host:
x86-03.phx2.fedoraproject.org
Group   : Development/Libraries Source RPM:
plexus-build-api-0.0.6-1.fc14.src.rpm
Size: 11602License: ASL 2.0
Signature   : (none)
Packager: Fedora Project
URL : http://svn.sonatype.org/spice/tags/plexus-build-api-0.0.6
Summary : Plexus Build API
Description :
Plexus Build API
/etc/maven/fragments/plexus-build-api
/usr/share/java/plexus/plexus-build-api-0.0.6.jar
/usr/share/java/plexus/plexus-build-api.jar
/usr/share/maven2/poms/JPP-plexus-build-api.pom

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


[Bug 594002] Review Request: plexus-build-api - Plexus Build API

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


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

--- Comment #7 from huwang huw...@redhat.com 2010-05-20 07:06:42 EDT ---
New Package CVS Request
===
Package Name: plexus-build-api
Short Description: Plexus Build API
Owners: huwang
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 593915] dconf - a configuration system

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


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

Matthias Clasen mcla...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Matthias Clasen mcla...@redhat.com 2010-05-20 07:06:48 
EDT ---
New Package CVS Request
===
Package Name: dconf
Short Description: A configuration system
Owners: mclasen
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 594002] Review Request: plexus-build-api - Plexus Build API

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


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

huwang huw...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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


[Bug 590522] Review Request: plexus-cipher-1.4-1 - Plexus Cipher: encryption/decryption Component

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


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

--- Comment #13 from huwang huw...@redhat.com 2010-05-20 07:26:38 EDT ---
Rebuilt in koji with dist-f14-maven221 target, the link is
http://koji.fedoraproject.org/koji/taskinfo?taskID=2198918

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


[Bug 511204] Review Request: wicd - A wireless and wired network manager

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


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

--- Comment #20 from Christoph Wickert cwick...@fedoraproject.org 2010-05-20 
07:38:45 EDT ---
When somebody starts a new approach to package this:
- Please take care of the noarch/arch problem with %{_libdir}/pm-utils
- Please CC me to the new review request.

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


[Bug 588142] Review Request: maven-release - Release a project updating the POM and tagging in the SCM

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


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

--- Comment #15 from Alexander Kurtakov akurt...@redhat.com 2010-05-20 
08:12:21 EDT ---
Hi Guido, 
Will you push this in rawhide soon ?

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


[Bug 591857] Review Request: maven-resources-plugin - Maven Resources Plugin

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


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

--- Comment #8 from Alexander Kurtakov akurt...@redhat.com 2010-05-20 
08:13:21 EDT ---
Koji build:
http://koji.fedoraproject.org/koji/buildinfo?buildID=174287

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


[Bug 593280] Review Request: servicelog - Servicelog Tools

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


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

--- Comment #6 from Roman Rakus rra...@redhat.com 2010-05-20 08:23:50 EDT ---
New Package CVS Request
===
Package Name: servicelog
Short Description: Servicelog Tools
Owners: rrakus
Branches: rawhide
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 594040] Review Request: apache-commons-fileupload - rename of jakarta-commons-fileupload

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


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

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

   What|Removed |Added

 Blocks||588764

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


[Bug 594040] New: Review Request: apache-commons-fileupload - rename of jakarta-commons-fileupload

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

Summary: Review Request: apache-commons-fileupload - rename of 
jakarta-commons-fileupload

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

   Summary: Review Request: apache-commons-fileupload - rename of
jakarta-commons-fileupload
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: socho...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://sochotni.fedorapeople.org/apache-commons-fileupload.spec
SRPM URL:
http://sochotni.fedorapeople.org/apache-commons-fileupload-1.2.1-2.fc12.src.rpm

Description: The javax.servlet package lacks support for rfc 1867, html file
upload.  This package provides a simple to use api for working with such data. 
The scope of this package is to create a utility classes to read
multipart/form-data within a javax.servlet.http.HttpServletRequest

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


[Bug 593999] Review Request: python-keyring - A python library for accessing the system keyring service

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


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

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

   What|Removed |Added

 Status|NEW |CLOSED
 CC||panem...@gmail.com
 Resolution||DUPLICATE

--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2010-05-20 08:37:01 EDT 
---
This package is already submitted in bug 593800. I have seen both the
submissions and your looks according to packaging guidelines but as other
review is already submitted this review should be considered as duplicate.
Also, you can help other review submission bug 593800 as its not following
packaging guidelines,review it and also can ask for co-maintainership.

*** This bug has been marked as a duplicate of bug 593800 ***

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

[Bug 532520] Review request: derby - An open source database implemented entirely in Java

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


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

--- Comment #3 from Alexander Kurtakov akurt...@redhat.com 2010-05-20 
08:35:33 EDT ---
ping?

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


[Bug 593800] Review Request: python-keyring - keyring module for python

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


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

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

   What|Removed |Added

 CC||vcrho...@redhat.com

--- Comment #2 from Parag AN(पराग) panem...@gmail.com 2010-05-20 08:37:01 EDT 
---
*** Bug 593999 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 593879] Review Request: libvpx - VP8 Video Codec SDK

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


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

--- Comment #3 from Tom spot Callaway tcall...@redhat.com 2010-05-20 
08:47:22 EDT ---
Permissions fixed.
Binaries renamed, as suggested.
Also fixed some missing weak symbols in the shared library.

New SRPM:
http://www.auroralinux.org/people/spot/review/new/libvpx-0.9.0-2.fc13.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/new/libvpx.spec

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


[Bug 593274] Review Request: rubygem-will_paginate - Most awesome pagination solution for Rails

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


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

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Michal Fojtik mfoj...@redhat.com 2010-05-20 08:48:05 EDT 
---
New Package CVS Request
===
Package Name: rubygem-xml-simple
Short Description: A simple API for XML processing
Owners: mfojtik
Branches: F-11 F-12 F-13 F-14 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 594064] Review Request: id3mtag - Command line mass ID3 tagging utility for audio files

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


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

--- Comment #1 from Avi Alkalay a...@unix.sh 2010-05-20 09:17:25 EDT ---
*** Bug 477958 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 593274] Review Request: rubygem-will_paginate - Most awesome pagination solution for Rails

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


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

--- Comment #9 from Michal Fojtik mfoj...@redhat.com 2010-05-20 09:19:35 EDT 
---
I'm apologize for wrong package name. Correct one:


New Package CVS Request
===
Package Name: rubygem-will_paginate
Short Description: Most awesome pagination solution for Rails
Owners: mfojtik
Branches: F-11 F-12 F-13 F-14 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 594064] New: Review Request: id3mtag - Command line mass ID3 tagging utility for audio files

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

Summary: Review Request: id3mtag - Command line mass ID3 tagging utility for 
audio files

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

   Summary: Review Request: id3mtag - Command line mass ID3
tagging utility for audio files
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: a...@unix.sh
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, herr...@owlriver.com,
cw...@alumni.drew.edu, fab...@bernewireless.net,
mtas...@ioa.s.u-tokyo.ac.jp,
fedora-package-rev...@redhat.com, fe...@fetzig.org
   Estimated Hours: 0.0
Classification: Fedora
  Clone Of: 477958


Spec URL: http://avi.alkalay.net/software/id3mtag/id3mtag.spec
SRPM URL: http://avi.alkalay.net/software/id3mtag/id3mtag-0.78-3.fc12.src.rpm
Other: http://avi.alkalay.net/software/id3mtag/

Description: ID3 mass tagger is a tool for manipulating id3 and id3v2 tags in
multiple files. It can generate tag fields from the filename and other
variables, and/or rename files, using an intuitive syntax.

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


[Bug 594064] Review Request: id3mtag - Command line mass ID3 tagging utility for audio files

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


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

--- Comment #2 from Avi Alkalay a...@unix.sh 2010-05-20 09:19:12 EDT ---
Sorry for the loong delay.

Everithing requested to be fixed on #477958 was fixed and I believe this is now
ready. Thanks in advance for reviewing.

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


[Bug 477958] Review Request: id3mtag - Command line mass ID3 tagging utility for audio files

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


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

Avi Alkalay a...@unix.sh changed:

   What|Removed |Added

 Resolution|NOTABUG |DUPLICATE

--- Comment #8 from Avi Alkalay a...@unix.sh 2010-05-20 09:17:25 EDT ---


*** This bug has been marked as a duplicate of bug 594064 ***

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


[Bug 594064] Review Request: id3mtag - Command line mass ID3 tagging utility for audio files

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


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

--- Comment #3 from Avi Alkalay a...@unix.sh 2010-05-20 09:21:29 EDT ---
Also check some notes about what this tool can do for you:
http://avi.alkalay.net/2009/06/mp3-mass-tagging-and-organization.html

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


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

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


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

Bastien Nocera bnoc...@redhat.com changed:

   What|Removed |Added

 CC||bnoc...@redhat.com

--- Comment #4 from Bastien Nocera bnoc...@redhat.com 2010-05-20 09:25:56 EDT 
---
Does this need to go through legal?

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


[Bug 561456] Review Request: jnr-x86asm - Pure-java port of asmjit

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


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

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

   What|Removed |Added

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

--- Comment #1 from Stanislav Ochotnicky socho...@redhat.com 2010-05-20 
09:29:15 EDT ---
I can do 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 594040] Review Request: apache-commons-fileupload - rename of jakarta-commons-fileupload

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


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

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

   What|Removed |Added

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

--- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2010-05-20 
09:34:34 EDT ---
I'll take this one.

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


[Bug 510097] Review Request: python-urwid - console user interface for python

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

   Flag|needinfo?   |

--- Comment #32 from David Cantrell dcantr...@redhat.com 2010-05-20 09:56:36 
EDT ---
I packaged this module yesterday.  Feel free to take anything from my spec
file:

http://dcantrel.fedorapeople.org/urwid/urwid.spec
http://dcantrel.fedorapeople.org/urwid/urwid-0.9.9.1-1.fc13.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 510097] Review Request: python-urwid - console user interface for python

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(cwick...@fedorapr
   ||oject.org)

--- Comment #33 from David Cantrell dcantr...@redhat.com 2010-05-20 10:00:23 
EDT ---
Christoph,

Chen Lei suggested I move my review request to here.  I've posted the links in
comment 32.  If you want to go with my urwid package for the review, let me
know.

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


[Bug 594040] Review Request: apache-commons-fileupload - rename of jakarta-commons-fileupload

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


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

--- Comment #2 from Alexander Kurtakov akurt...@redhat.com 2010-05-20 
10:05:28 EDT ---
Review:

OK: rpmlint must be run on every package. OUTPUT:

apache-commons-fileupload-javadoc.noarch: W: obsolete-not-provided
jakarta-commons-fileupload-javadoc
Just obsoleting documentation package is OK

apache-commons-fileupload.noarch: W: spelling-error Summary(en_US) api - pi,
ape, apt
apache-commons-fileupload.noarch: W: spelling-error Summary(en_US) html -
HTML, ht ml, ht-ml
apache-commons-fileupload.noarch: W: spelling-error %description -l en_US javax
- java, java x, Java
apache-commons-fileupload.noarch: W: spelling-error %description -l en_US
servlet - servile, serviette, servility
apache-commons-fileupload.noarch: W: spelling-error %description -l en_US rfc
- RFC, rec, Pfc
apache-commons-fileupload.noarch: W: spelling-error %description -l en_US html
- HTML, ht ml, ht-ml
apache-commons-fileupload.noarch: W: spelling-error %description -l en_US api
- pi, ape, apt
apache-commons-fileupload.noarch: W: spelling-error %description -l en_US
multipart - multiparty, multiplier, multiplexer
apache-commons-fileupload.noarch: W: spelling-error %description -l en_US http
- HTTP
apache-commons-fileupload.noarch: W: spelling-error %description -l en_US
HttpServletRequest 
apache-commons-fileupload.noarch: W: non-conffile-in-etc
/etc/maven/fragments/apache-commons-fileupload

Not a problem. 

OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. 
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK: The License field in the package spec file must match the actual license. 
OK: The spec file must be written in American English.
OK: The spec file for the package MUST be legible. 
OK: The sources used to build the package must match the upstream source, as
provided in the spec URL.
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
OK: All build dependencies must be listed in BuildRequires, except for any that
are listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense.
OK: Packages must NOT bundle copies of system libraries.
OK: A package must own all directories that it creates. 
OK: A Fedora package must not list a file more than once in the spec file's
%files listings. 
OK: Permissions on files must be set properly. 
OK: Each package must consistently use macros.
OK: The package must contain code, or permissable content. 
OK: Large documentation files must go in a -doc subpackage. Javadocs
subpackage.
OK: If a package includes something as %doc, it must not affect the runtime of
the application. 
OK Packages must not own files or directories already owned by other
packages. 
OK: All filenames in rpm packages must be valid UTF-8.
OK: Package is correctly Obsoleting/Providing the old jakarta package. 

Things to fix:
* javadoc package is missing Requires:  jpackage-utils

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


[Bug 562097] Rename request: libsocialweb - A social network data aggregator

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


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

--- Comment #9 from Peter Robinson pbrobin...@gmail.com 2010-05-20 10:16:51 
EDT ---
(In reply to comment #8)
 Everything looks fine. APPROVED.

Thanks

 As a side note, if libsocialweb is not source-compatible with mojito (as in
 mojito-dependent packages could be rebuilt to depend on it), then I don't see
 why a Provides/Obsoletes needs to be added at all.

It will primarily be enable clean upgrades due to packages that currently
depend on mojito will depend on l-s-w so we don't leave mojito around.

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


[Bug 561456] Review Request: jnr-x86asm - Pure-java port of asmjit

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


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

--- Comment #2 from Stanislav Ochotnicky socho...@redhat.com 2010-05-20 
10:18:17 EDT ---
NOTOK: rpmlint must be run on every package. The output should be posted in the
review.
NOTOK: The package must be named according to the Package Naming 
NOTOK: The sources used to build the package must match the upstream source, as
provided in the spec URL.

Output:
jnr-x86asm.src: W: spelling-error Summary(en_US) asmjit - Asmara, Asquith,
fajitas
jnr-x86asm.src: W: spelling-error %description -l en_US asmjit - Asmara,
Asquith, fajitas
jnr-x86asm.src: W: invalid-url Source0: jnr-x86asm-0.1.tgz
jnr-x86asm.noarch: W: spelling-error Summary(en_US) asmjit - Asmara, Asquith,
fajitas
jnr-x86asm.noarch: W: spelling-error %description -l en_US asmjit - Asmara,
Asquith, fajitas
jnr-x86asm-javadoc.noarch: W: spelling-error Summary(en_US) asm - as, am, mas
jnr-x86asm-javadoc.noarch: W: spelling-error %description -l en_US asm - as,
am, mas
3 packages and 0 specfiles checked; 0 errors, 7 warnings.

While spec file explains how to create Source0 it fails to provide
reliable way to verify no changes happened between Source0 was taken
and building of RPM. You need to specify git hash (part of it at
least) of commit that should be checked out after cloning. Once some
version is released by developer, it would be ideal if git tag hash was
used instead.

This also applies to naming of package. This is most certainly a
pre-relase so naming convetions are these:
https://fedoraproject.org/wiki/Packaging:NamingGuidelines#PreReleasePackages


OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.  .
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
NOTOK: The License field in the package spec file must match the actual
license. 

License of package is obviously LGPLv3 not MIT

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

License is included,  but it would be good to include COPYING* files
too especially since COPYING file has the text of license itself.

OK: The spec file must be written in American English. 
OK: The spec file for the package MUST be legible. 
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. 
OK: All build dependencies must be listed in BuildRequires, except for any that
are listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense.
OK: A package must own all directories that it creates. If it does not create a
directory that it uses, then it should require a package which does create that
directory. 
OK: A Fedora package must not list a file more than once in the spec file's
%files listings. 
OK: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line. 
NOTOK: Each package must consistently use macros.
you mix $RPM_BUILD_ROOT and %{buildroot}. Pick one and stick to it

OK: The package must contain code, or permissable content.
OK: Large documentation files must go in a -doc subpackage. (The definition of
large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity). 
OK: Packages must not own files or directories already owned by other packages.
The rule of thumb here is that the first package to be installed should own the
files or directories that other packages may rely upon. This means, for
example, that no package in Fedora should ever share ownership with any of the
files or directories owned by the filesystem or man package. If you feel that
you have a good reason to own a file or directory that another package owns,
then please present that at package review time. 
OK: All filenames in rpm packages must be valid UTF-8. 


Please fix those naming/Source0 URL/license/macro problems, so that I can
approve this package.

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


[Bug 561456] Review Request: jnr-x86asm - Pure-java port of asmjit

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


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

--- Comment #3 from Stanislav Ochotnicky socho...@redhat.com 2010-05-20 
10:23:36 EDT ---
Also javadoc subpackage should require jpackage-utils

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


[Bug 562097] Rename request: libsocialweb - A social network data aggregator

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #10 from Peter Robinson pbrobin...@gmail.com 2010-05-20 10:20:40 
EDT ---

New Package CVS Request
===
Package Name: libsocialweb
Short Description: A social network data aggregator
Owners: pbrobinson
Branches: F-13
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 594040] Review Request: apache-commons-fileupload - rename of jakarta-commons-fileupload

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


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

--- Comment #3 from Stanislav Ochotnicky socho...@redhat.com 2010-05-20 
10:29:05 EDT ---
Spec URL: http://sochotni.fedorapeople.org/apache-commons-fileupload.spec
SRPM URL:
http://sochotni.fedorapeople.org/apache-commons-fileupload-1.2.1-3.fc12.src.rpm


I am sorry but I also forgot to mention that this is a RE-REVIEW. So review
provides/obsoletes separately.

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


[Bug 594064] Review Request: id3mtag - Command line mass ID3 tagging utility for audio files

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 CC||supercyp...@gmail.com

--- Comment #4 from Chen Lei supercyp...@gmail.com 2010-05-20 10:25:29 EDT ---
I suggest you to persuade upstream to rename %{_bindir}/id3 to
%{_bindir}/id3mtag as well.

It will conflict with some other packages that haven't in fedora temporarily.

See http://fedoraproject.org/wiki/Packaging:Conflicts

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


[Bug 594040] Review Request: apache-commons-fileupload - rename of jakarta-commons-fileupload

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


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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Alexander Kurtakov akurt...@redhat.com 2010-05-20 
10:32:31 EDT ---
(In reply to comment #3)
 Spec URL: http://sochotni.fedorapeople.org/apache-commons-fileupload.spec
 SRPM URL:
 http://sochotni.fedorapeople.org/apache-commons-fileupload-1.2.1-3.fc12.src.rpm
 
 
 I am sorry but I also forgot to mention that this is a RE-REVIEW. So review
 provides/obsoletes separately.

I've noticed and checked that.

This package is APPROVED.

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


[Bug 593804] Review Request: plasma-runner-events - Add new events or todos to your calendar from KRunner

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


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

--- Comment #1 from Thomas Janssen thom...@fedoraproject.org 2010-05-20 
10:39:05 EDT ---
Added missing %post and %postun

Spec URL: http://thomasj.fedorapeople.org/reviews/plasma-runner-events.spec
SRPM URL:
http://thomasj.fedorapeople.org/reviews/plasma-runner-events-0.3.0-2.fc12.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 593879] Review Request: libvpx - VP8 Video Codec SDK

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


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

--- Comment #6 from Christian Schaller ura...@linuxrising.org 2010-05-20 
11:03:36 EDT ---
Sorry, I mean in include/vpx

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


[Bug 510097] Review Request: python-urwid - console user interface for python

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


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

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

   What|Removed |Added

   Flag|needinfo?(cwick...@fedorapr |
   |oject.org)  |

--- Comment #34 from Christoph Wickert cwick...@fedoraproject.org 2010-05-20 
10:59:08 EDT ---
Please open a review request because the reporter of the bug can not be
changed. Then mark this bug a duplicate of yours, so we all get cc'ed to the
new bug. Do the same for wicd.

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


[Bug 593839] Review Request: urwid - Console user interface library

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

   Keywords||Reopened
 Status|CLOSED  |ASSIGNED
 Resolution|DUPLICATE   |

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


[Bug 510097] Review Request: python-urwid - console user interface for python

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


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

--- Comment #35 from David Cantrell dcantr...@redhat.com 2010-05-20 11:21:43 
EDT ---
*** Bug 593839 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 593841] Review Request: wicd - Wireless and wired network connection manager

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

   Keywords||Reopened
 Status|CLOSED  |ASSIGNED
 Resolution|DUPLICATE   |

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


[Bug 511204] Review Request: wicd - A wireless and wired network manager

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


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

Bug 511204 depends on bug 510097, which changed state.

Bug 510097 Summary: Review Request: python-urwid - console user interface for 
python
https://bugzilla.redhat.com/show_bug.cgi?id=510097

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||DUPLICATE

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


[Bug 593839] Review Request: urwid - Console user interface library

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DUPLICATE

--- Comment #3 from David Cantrell dcantr...@redhat.com 2010-05-20 11:21:43 
EDT ---


*** This bug has been marked as a duplicate of bug 510097 ***

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


[Bug 511204] Review Request: wicd - A wireless and wired network manager

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||DUPLICATE

--- Comment #21 from David Cantrell dcantr...@redhat.com 2010-05-20 11:25:11 
EDT ---


*** This bug has been marked as a duplicate of bug 593841 ***

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


[Bug 593839] Review Request: urwid - Console user interface library

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|DUPLICATE   |

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


[Bug 593839] Review Request: urwid - Console user interface library

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

 CC||ultimatekonstanti...@gmail.
   ||com

--- Comment #4 from David Cantrell dcantr...@redhat.com 2010-05-20 11:22:37 
EDT ---
*** Bug 510097 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 593841] Review Request: wicd - Wireless and wired network connection manager

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


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

David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

 CC||sherry...@gmail.com

--- Comment #2 from David Cantrell dcantr...@redhat.com 2010-05-20 11:25:12 
EDT ---
*** Bug 511204 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 593839] Review Request: urwid - Console user interface library

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 CC||cwick...@fedoraproject.org

--- Comment #5 from Chen Lei supercyp...@gmail.com 2010-05-20 11:27:31 EDT ---
Sorry for the inconvience.

I think this package should be renamed to python-urwid

See
http://fedoraproject.org/wiki/PackageNamingGuidelines#Addon_Packages_.28python_modules.29

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


[Bug 593839] Review Request: urwid - Console user interface library

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW

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


[Bug 593839] Review Request: urwid - Console user interface library

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 Blocks||593841

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


[Bug 593841] Review Request: wicd - Wireless and wired network connection manager

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 CC||cwick...@fedoraproject.org

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


[Bug 510097] Review Request: python-urwid - console user interface for python

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


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

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

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |201449(FE-DEADREVIEW)
   Flag|fedora-review?  |

--- Comment #37 from Christoph Wickert cwick...@fedoraproject.org 2010-05-20 
11:34:43 EDT ---
Unblock NEEDSPONSOR, block FE-DEADREVIEW

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


[Bug 593841] Review Request: wicd - Wireless and wired network connection manager

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


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

Chen Lei supercyp...@gmail.com changed:

   What|Removed |Added

 CC||gareth.l.j...@googlemail.co
   ||m
 Depends on||593839

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


[Bug 511204] Review Request: wicd - A wireless and wired network manager

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


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

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

   What|Removed |Added

 Blocks||201449(FE-DEADREVIEW)

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


[Bug 591222] gtk3 - version 3 of GTK+

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


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

--- Comment #7 from Bastien Nocera bnoc...@redhat.com 2010-05-20 11:37:19 EDT 
---
The latest SRPM's build failed:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2199494

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


[Bug 480279] Review Request: gnome-globalmenu - centralized menu bar

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


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

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

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |
  Alias|gnome-globalmenu|

--- Comment #62 from Christoph Wickert cwick...@fedoraproject.org 2010-05-20 
11:37:16 EDT ---
Remove alias and unblock NEEDSPONSOR

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


[Bug 594414] New: Review Request: ezmorph - Object transformation library for Java

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

Summary: Review Request: ezmorph - Object transformation library for Java

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

   Summary: Review Request: ezmorph - Object transformation
library for Java
   Product: Fedora
   Version: 13
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lkund...@v3.sk
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


SPEC: http://v3.sk/~lkundrak/SPECS/ezmorph.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/ezmorph-1.0.6-1.fc13.src.rpm

Description:

EZMorph is simple java library for transforming an Object to another
Object. It supports transformations for primitives and Objects and
multidimensional arrays.

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


[Bug 515752] Review Request: python-soaplib - python library for creating SOAP services

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


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

--- Comment #14 from Jordan OMara jom...@redhat.com 2010-05-20 11:41:44 EDT 
---
I have finally found enough free time to respond to the comments, incorporate
them into the project, and re-submit it for evaluation!

1. I updated soaplib to version 0.8.1. They moved webhosts and incorporated my
patch upstream, so it eliminates the need for the patch

2. I did my best to incorporate all of the comments and suggestions people
left. RPMlint on the spec  RPMs is clean on my machine

3. This is tested on koji against F-11, F-12, and F-13

new spec: http://jomara.fedorapeople.org/python_soaplib/python-soaplib.spec
new srpm:
http://jomara.fedorapeople.org/python_soaplib/python-soaplib-0.8.1-1.fc12.src.rpm

Please let me know how this looks!

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


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

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


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

--- Comment #7 from Tom spot Callaway tcall...@redhat.com 2010-05-20 
11:48:15 EDT ---
(In reply to comment #4)
 Does this need to go through legal?

It has been through legal, and there are no blockers at this time.

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


[Bug 594414] Review Request: ezmorph - Object transformation library for Java

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


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

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

   What|Removed |Added

 Blocks||594416

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


[Bug 594416] New: Review Request: json-lib - JSON library for Java

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

Summary: Review Request: json-lib - JSON library for Java

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

   Summary: Review Request: json-lib - JSON library for Java
   Product: Fedora
   Version: 13
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lkund...@v3.sk
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
Depends on: 594414
   Estimated Hours: 0.0
Classification: Fedora


SPEC: http://v3.sk/~lkundrak/SPECS/json-lib.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/json-lib-2.3-1.fc13.src.rpm

Description:

JSON-lib is a java library for transforming beans, maps, collections, java
arrays and XML to JSON and back again to beans and DynaBeans.

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


[Bug 593841] Review Request: wicd - Wireless and wired network connection manager

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


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

--- Comment #3 from leigh scott leigh123li...@googlemail.com 2010-05-20 
11:47:33 EDT ---
I believe you should add the python macro to the spec file if you intend to
build for F12.

https://fedoraproject.org/wiki/Packaging/Python#Macros


You don't need the CFLAGS=%{optflags} as you aren't compiling anything

CFLAGS=%{optflags} %{__python} setup.py configure \
--pmutils %{_libdir}/pm-utils/sleep.d
CFLAGS=%{optflags} %{__python} setup.py build



Also the srpm doesn't build on F13

Checking for unpackaged file(s): /usr/lib/rpm/check-files
/home/leigh/rpmbuild/BUILDROOT/wicd-1.7.0-1.fc13.x86_64
error: Installed (but unpackaged) file(s) found:
   /usr/share/autostart/wicd-tray.desktop


RPM build errors:
user dcantrel does not exist - using root
group dcantrel does not exist - using root
user dcantrel does not exist - using root
group dcantrel does not exist - using root
user dcantrel does not exist - using root
group dcantrel does not exist - using root
File listed twice: /usr/lib/python2.6/site-packages/wicd
Installed (but unpackaged) file(s) found:
   /usr/share/autostart/wicd-tray.desktop



This should be changed from


%{python_sitelib}/wicd


to


%dir %{python_sitelib}/wicd
%{python_sitelib}/wicd/*

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


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

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


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

Christian Schaller ura...@linuxrising.org changed:

   What|Removed |Added

 CC||ura...@linuxrising.org

--- Comment #5 from Christian Schaller ura...@linuxrising.org 2010-05-20 
10:58:08 EDT ---
Maybe trivial point, but the GStreamer plugins for vp8 is currently looking for
the header files in include/vp8. Maybe that is a 'cleaner' way of storing them
than just dropping them in include.

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


[Bug 587198] Review Request: jgraphx - Java Graph Drawing Component

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


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

--- Comment #5 from Lubomir Rintel lkund...@v3.sk 2010-05-20 11:50:26 EDT ---
(In reply to comment #3)
 Can we make scilab available as a new feature for F14?

This must be the worst place you could find to ask this.
Sure, try to make a feature page, propose it to FESCO and persuade them why is
this worth being a feature.

(In reply to comment #4)
 Version 1.4.0.0 is available now. Please think about using this.

Would you mind doing a complete review?

SPEC: http://fedorapeople.org/~lkundrak/jgraphx.spec
SRPM: http://fedorapeople.org/~lkundrak/jgraphx-4.0.0.2-1.fc13.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 587198] Review Request: jgraphx - Java Graph Drawing Component

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


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

--- Comment #6 from Lubomir Rintel lkund...@v3.sk 2010-05-20 11:58:30 EDT ---
Whoops, bad links. Good ones (hopefully):

SPEC: http://fedorapeople.org/~lkundrak/jgraphx.spec
SRPM: http://fedorapeople.org/~lkundrak/jgraphx-1.4.0.2-1.fc13.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 587198] Review Request: jgraphx - Java Graph Drawing Component

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


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

Markus Mayer lotharl...@gmx.de changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|lotharl...@gmx.de
   Flag||fedora-review?

--- Comment #7 from Markus Mayer lotharl...@gmx.de 2010-05-20 12:12:14 EDT ---
taking this

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


[Bug 460974] Review Request: xmobar - status bar for X

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


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

--- Comment #15 from Ben Boeckel maths...@gmail.com 2010-05-20 12:29:00 EDT 
---
I may take this; if I get around to it over the break before graduation and no
one else has taken it, I'll pick it up.

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


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

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


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

Adel Gadllah adel.gadl...@gmail.com changed:

   What|Removed |Added

 CC||adel.gadl...@gmail.com

--- Comment #9 from Adel Gadllah adel.gadl...@gmail.com 2010-05-20 12:31:26 
EDT ---

(In reply to comment #8)
 New package:
 
 * Header files in include/vpx
 * pkgconfig file for libvpx
 * Optimized now (thanks to debian for figuring out how to glue the shared lib
 together)
 
 New SRPM:
 http://www.auroralinux.org/people/spot/review/new/libvpx-0.9.0-3.fc13.src.rpm
 New SPEC: http://www.auroralinux.org/people/spot/review/new/libvpx.spec

# If/when we build an optimized version that uses the asm, we'll need this.
# BuildRequires: yasm
[...]
%changelog
* Thu May 20 2010 Tom spot Callaway tcall...@redhat.com 0.9.0-3
- add pkg-config file
- move headers into include/vpx/
- enable optimization

Something does not look right here ;)

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


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

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


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

--- Comment #10 from Tom spot Callaway tcall...@redhat.com 2010-05-20 
12:39:29 EDT ---
Yep. That's definitely not right. Fixed in -4:

New SRPM:
http://www.auroralinux.org/people/spot/review/new/libvpx-0.9.0-4.fc13.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/new/libvpx.spec

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


[Bug 594414] Review Request: ezmorph - Object transformation library for Java

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


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

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

   What|Removed |Added

 CC||akurt...@redhat.com

--- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2010-05-20 
12:41:14 EDT ---
Is there a reason to not build with maven as upstream is doing?

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


[Bug 594416] Review Request: json-lib - JSON library for Java

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


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

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

   What|Removed |Added

 CC||akurt...@redhat.com

--- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2010-05-20 
12:43:44 EDT ---
Is there a reason to not build with maven or ant as upstream is doing?

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


[Bug 587198] Review Request: jgraphx - Java Graph Drawing Component

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


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

--- Comment #8 from Markus Mayer lotharl...@gmx.de 2010-05-20 12:42:28 EDT ---
Just a few things for now.

According to https://fedoraproject.org/wiki/Packaging:Java

At a minimum, Java packages MUST:

BuildRequires: java-devel [= specific_version] 
BuildRequires:  jpackage-utils

Requires:  java = specific_version
Requires:  jpackage-utils

Group for javadoc should be Development/Documentation
Think about adding Requires: Requires: %{name}-%{version}-%{release}
 to javadoc subpackage.


SOURCE0 is not a vaild URL. Try using:
http://www.jgraph.com/downloads/jgraphx/archive/%{name}-%(echo %{version} |sed
's/\./_/g').zip

Your BuildRoot seams a little odd. Think about using
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) instead.

find -name '*.jar' -delete and rm -rf docs/api should be moved to %prep.
It is also a good idea to add find -name '*.class' -delete too.


I will continue reviewing after these things have been fixed.

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


  1   2   >