[Bug 591545] Review Request: apache-commons-net - rename of jakarta-commons-net

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


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

--- Comment #6 from Stanislav Ochotnicky socho...@redhat.com 2010-05-17 
03:15:47 EDT ---
(In reply to comment #5)
 When you say Chen is right, does that mean you intended to drop the use of
 epoch in your provides/obsoletes? Is there any point in specifying an epoch of
 zero?

I meant that packaging guidelines state that we should avoid using Epoch if
possible. This was possible with this rename because it was enough to specify
epoch in provides/obosletes, no need to actually specify one for this package.
It was there from way back, probably when no guidelines on Epoch use existed.

Current guidelines state (2nd link from Chen):
If the provided package had an Epoch set, it must be preserved in both the
Provides and Obsoletes. It may and should be removed from the actual new
package. 


 On a side note for future reference, did you know that your two calls to
 install on lines 71/72 and again on lines 83/84 can be combined into a single
 call? For example, these two lines:
 
 install -d -m 755 foo_dir
 install -p -m 644 bar_file foo_dir/bar_file
 
 Are equivalent to this one line:
 
 install -pD -T -m 644 bar_file foo_dir/bar_file

Thanks, I'll keep it in mind :-) I suppose 2 commands were originally mkdir 
cp and I just converted them to install.


New Package CVS Request
===
Package Name: apache-commons-net
Short Description: Internet protocol suite Java library
Owners: sochotni
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 591545] Review Request: apache-commons-net - rename of jakarta-commons-net

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


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

Stanislav Ochotnicky socho...@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 592137] Review Request: koffice-kivio - A flowcharting application

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


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

Thomas Janssen thom...@fedoraproject.org changed:

   What|Removed |Added

 CC||thom...@fedoraproject.org
 AssignedTo|nob...@fedoraproject.org|thom...@fedoraproject.org
   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 592137] Review Request: koffice-kivio - A flowcharting application

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


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

--- Comment #5 from Thomas Janssen thom...@fedoraproject.org 2010-05-17 
03:40:18 EDT ---
Starting the review now.

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


[Bug 592387] Review Request: maven-install-plugin - Maven Install Plugin

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


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

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 #3 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
03:43:19 EDT ---
I'm taking 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 578759] Review request: django-mako - Mako Templates Plugin for Django

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


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

--- Comment #6 from Rakesh Pandit rpan...@redhat.com 2010-05-17 03:46:00 EDT 
---
http://rakesh.fedorapeople.org/spec/django-mako.spec
http://rakesh.fedorapeople.org/srpm/django-mako-0.1.4-0.1.pre.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 549223] Review Request: ignuit - A memorization aiding tool

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


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

--- Comment #2 from Rakesh Pandit rpan...@redhat.com 2010-05-17 03:52:44 EDT 
---
Fabian, Any updates about Comment #1 ?

Regards,
Rakesh Pandit

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


[Bug 509619] Review Request: srtp - Secure Real-Time Transport Protocol (SRTP) Library

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


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

--- Comment #6 from Rakesh Pandit rpan...@redhat.com 2010-05-17 03:51:14 EDT 
---
Hi Itamar,

Any updates here about Comment#4 and Comment #5 ?

Regards,
Rakesh Pandit

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


[Bug 537366] Review Request: cgal-python - Python bindings for the CGAL library

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


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

--- Comment #19 from Rakesh Pandit rpan...@redhat.com 2010-05-17 03:55:25 EDT 
---
Paulo,
 Any updates in here? It is nearing a month day after tomorrow ? :)

--
Regards,
Rakesh Pandit

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


[Bug 591305] Review Request: apache-commons-digester - XML to Java object mapping module

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


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

--- Comment #8 from Stanislav Ochotnicky socho...@redhat.com 2010-05-17 
04:00:40 EDT ---
Spec file looks good, I believe there will be no problem with this review once
beanutils is done. Do you still plan to do the spec file for beanutils?

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


[Bug 518892] Review Request: gedit-valencia - A gedit plugin providing lightweight Vala IDE

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


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

--- Comment #2 from Rakesh Pandit rpan...@redhat.com 2010-05-17 04:00:41 EDT 
---
Michel,

It has been three months 

In case you did addressed above issue in a week I will mark this request as
deferred.

--
Regards,
Rakesh Pandit

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


[Bug 524257] Review Request: Sympa - An electronic mailing list manager

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


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

Rakesh Pandit rpan...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DEFERRED

--- Comment #9 from Rakesh Pandit rpan...@redhat.com 2010-05-17 04:03:01 EDT 
---
Deferring it for time being. In case you find some time for it later on, pleas
reopen the bug and update.

Regards,
Rakesh Pandit

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on 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-17 Thread bugzilla
Please do not reply directly 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

Akira TAGOH ta...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #18 from Akira TAGOH ta...@redhat.com 2010-05-17 04:15:26 EDT ---
Review results:
+ rpmlint result:
  ISO8859-2-100dpi-fonts.noarch: W: spelling-error Summary(en_US) dpi - dip,
pi, dpt
  ISO8859-2-100dpi-fonts.noarch: W: spelling-error %description -l en_US dpi -
dip, pi, dpt
  ISO8859-2-100dpi-fonts.noarch: W: no-url-tag
  ISO8859-2-100dpi-fonts.noarch: W: no-documentation
  ISO8859-2-75dpi-fonts.noarch: W: spelling-error Summary(en_US) dpi - dip,
pi, dpt
  ISO8859-2-75dpi-fonts.noarch: W: spelling-error %description -l en_US dpi -
dip, pi, dpt
  ISO8859-2-75dpi-fonts.noarch: W: no-url-tag
  ISO8859-2-75dpi-fonts.noarch: W: no-documentation
  ISO8859-2-fonts.src: W: no-url-tag
  ISO8859-2-fonts.src: W: invalid-url Source0: ISO8859-2-bdf.tar.gz
  ISO8859-2-fonts-common.noarch: W: no-url-tag
  ISO8859-2-misc-fonts.noarch: W: no-url-tag
  ISO8859-2-misc-fonts.noarch: W: no-documentation
  5 packages and 0 specfiles checked; 0 errors, 13 warnings.

+ the package met the Packaging Naming Guidelines.
+ the spec file name matches the base package %{name}.
+ the package met the Packaging Guidelines.
+ the license is OSS-compatible.
+ License tag matches the actual license.
_ no license file provided by upstream.
+ the spec file is written in American English.
+ the spec file is legible.
_ according to the comment in the spec file, the upstream link is dead.
+ the package is buildable on mock.
+ the package provides noarch only.
+ the package has the certain BuildRequires.
_ no locales files provided.
_ no shared libraries provided.
+ the package owns all directories in the package without duplicates.
+ no duplicates in %files.
+ files in the package has certain permissions.
+ the package consistently use macros.
+ the package contains permissable content.
_ no large documentation files provided.
+ %doc contains the release notes only.
_ no header files provided.
_ no static libraries provided.
_ no desktop files provided.
+ all filenames in rpm is valid UTF-8.

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 592864] New: Review Request: kmyfirewall - IPTables Based Firewall

2010-05-17 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: kmyfirewall - IPTables Based Firewall

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

   Summary: Review Request: kmyfirewall - IPTables Based Firewall
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: adima...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://adimania.fedorapeople.org/specs/kmyfirewall.spec
SRPM URL:
http://adimania.fedorapeople.org/src.rpms/kmyfirewall-1.1.1-1.fc12.src.rpm
Description: KMyFirewall attempts to make it easier to setup IPTables based
firewalls on Linux systems.
Koji Scratch Build URL:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2191747

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


[Bug 592864] Review Request: kmyfirewall - IPTables Based Firewall

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


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

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

   What|Removed |Added

 CC||panem...@gmail.com
 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2010-05-17 05:35:46 EDT 
---
As you are not sponsored yet so its always recommended to add FE-NEEDSPONSOR to
all your review requests.

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

[Bug 568315] Review Request: xapply - Parallel Execution tool ala xargs/apply

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


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

--- Comment #7 from Terje Røsten terje...@phys.ntnu.no 2010-05-17 05:59:09 
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 578135] Review Request: patcher - Quick creation of patches against a project source tree

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


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

--- Comment #5 from Terje Røsten terje...@phys.ntnu.no 2010-05-17 06:00:29 
EDT ---
Any progress?

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

[Bug 569368] Review Request: ipa-ex-mincho-fonts - Japanese Mincho-typeface OpenType font by IPA

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


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

--- Comment #5 from Akira TAGOH ta...@redhat.com 2010-05-17 06:01:30 EDT ---
Sorry for not getting back to you earlier. updated spec file and srpm is
available here:

Spec URL:
http://tagoh.fedorapeople.org/ipa-ex-mincho-fonts/ipa-ex-mincho-fonts.spec
SRPM URL:
http://tagoh.fedorapeople.org/ipa-ex-mincho-fonts/ipa-ex-mincho-fonts-001.01-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 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager

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


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

--- Comment #98 from Michal Nowak mno...@redhat.com 2010-05-17 06:29:54 EDT 
---
3.4.5 in fp.o.
http://mnowak.fedorapeople.org/awesome/awesome-3.4.5-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 592614] Review Request: apache-commons-launcher - A cross platform Java application launcher

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


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

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-17 
06:29:23 EDT ---
I can take 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 591454] Review Request: spice-parent - Sonatype Spice Components

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


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

--- Comment #7 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-05-17 
06:38:13 EDT ---
Thanks,

However, during reviews (and also after the reviews) when we make changes in
the package, we bump the Release tag and list our changes in the %changelog
with the latest change date. Please include these changes.

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


[Bug 575529] Review Request: dwm - Dynamic window manager

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


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

--- Comment #11 from Damien Durand splinu...@gmail.com 2010-05-17 07:08:27 
EDT ---
Hello

Did you have some news about the desktop file ?

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


[Bug 592137] Review Request: koffice-kivio - A flowcharting application

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


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

Aditya Patawari adima...@gmail.com changed:

   What|Removed |Added

 CC||adima...@gmail.com

--- Comment #6 from Aditya Patawari adima...@gmail.com 2010-05-17 07:14:54 
EDT ---
Informal Review

rpmlint output:
koffice-kivio.src: W: spelling-error Summary(en_US) flowcharting - flow
charting, flow-charting, flowchart
koffice-kivio.src: W: spelling-error %description -l en_US flowcharting - flow
charting, flow-charting, flowchart
Error checking signature of
../../Downloads/koffice-kivio-1.6.3-31.trinity.20100511svn.fc13.src.rpm:
../../Downloads/koffice-kivio-1.6.3-31.trinity.20100511svn.fc13.src.rpm: (SHA1)
DSA sha1 md5 (GPG) NOT OK (MISSING KEYS:GPG#1634f842)
koffice-kivio.src: W: invalid-url Source1: kivio-translations-20100511.tar.bz2
koffice-kivio.src: W: invalid-url Source0: koffice-trinity-20100511.tar.bz2
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

+ Package meets naming and packaging guidelines
+ Spec file matches base package name.
+ Spec has consistant macro usage.
+ Meets Packaging Guidelines.
+ License
+ License field in spec matches
+ License file included in package
+ Spec in American English
+ Spec is legible.
+ BuildRequires correct
+ defattr present and correct
+ %doc is fine
+ BuildRoot is correct
{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+ desktop files are properly validated.
+ Package is code and has permissible content.
+ Package owns all the directories it creates.
- md5sum not verified as the source was tarred manually from the svn snapshot
- There is no step-by-step guide on how to get the source tarball.

The package looks fine, has to reviewed by a packager.

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


[Bug 569366] Review Request: ipa-ex-gothic-fonts - Japanese Gothic-typeface OpenType font by IPA

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


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

--- Comment #7 from Akira TAGOH ta...@redhat.com 2010-05-17 07:16:00 EDT ---
Okay, another try:

Spec URL:
http://tagoh.fedorapeople.org/ipa-ex-gothic-fonts/ipa-ex-gothic-fonts.spec
SRPM URL:
http://tagoh.fedorapeople.org/ipa-ex-gothic-fonts/ipa-ex-gothic-fonts-001.01-3.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 591011] Review Request: kfilefactory - An easy to use filefactory.com uploader

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


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

--- Comment #2 from Aditya Patawari adima...@gmail.com 2010-05-17 07:19:35 
EDT ---
For potential sponsors:
Check out my other request (kmyfirewall) at
https://bugzilla.redhat.com/show_bug.cgi?id=592864
I have dine some informal reviews at:
1. thomasj's vor : https://bugzilla.redhat.com/show_bug.cgi?id=592388#c4
2. Kevin_Kofler's koffice-kivio :
https://bugzilla.redhat.com/show_bug.cgi?id=592137#c6

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


[Bug 592387] Review Request: maven-install-plugin - Maven Install Plugin

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


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

--- Comment #4 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
07:22:53 EDT ---
Review:

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

maven-install-plugin.noarch: W: no-documentation
maven-install-plugin.noarch: W: non-conffile-in-etc
/etc/maven/fragments/maven-install-plugin
maven-install-plugin.src: W: invalid-url Source0:
maven-install-plugin-2.3.tar.gz

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.

One small problem only:
install -pm 644 pom.xml \
%{buildroot}%{_mavenpomdir}/JPP.%{name}.pom 
should be
install -pm 644 pom.xml \
%{buildroot}%{_mavenpomdir}/JPP-%{name}.pom

This is needed for proper finding of the 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 591857] Review Request: maven-resources-plugin - Maven Resources Plugin

2010-05-17 Thread bugzilla
Please do not reply directly 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|NEW |ASSIGNED
 CC||akurt...@redhat.com
 AssignedTo|nob...@fedoraproject.org|akurt...@redhat.com
   Flag||fedora-review?

--- Comment #2 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
07:52:29 EDT ---
I'm taking 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 588142] Review Request: maven-release - Release a project updating the POM and tagging in the SCM

2010-05-17 Thread bugzilla
Please do not reply directly 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

Andrew Overholt overh...@redhat.com changed:

   What|Removed |Added

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

--- Comment #7 from Andrew Overholt overh...@redhat.com 2010-05-17 08:32:05 
EDT ---
I'm sorry, but I won't be able to review this in a timely manner.

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


[Bug 588654] Review Request: plexus-component-api - Plexus Component API

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


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

Andrew Overholt overh...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #10 from Andrew Overholt overh...@redhat.com 2010-05-17 08:55:29 
EDT ---
Thanks, Yong.  Everything is now okay so this package is APPROVED.  Please
follow the next steps here:

https://fedoraproject.org/wiki/Package_Review_Process#Review_Process

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


[Bug 570731] Review Request: ibus-handwrite - IBus handwrite project

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


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

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

   What|Removed |Added

 Depends on|575005  |

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


[Bug 575005] Review Request: zinnia-tomoe - Online hand recognition system with machine learning

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


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

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

   What|Removed |Added

 Status|ASSIGNED|NEW
 Blocks|570731  |
   Flag|fedora-review?  |

--- Comment #50 from Chen Lei supercyp...@gmail.com 2010-05-17 09:23:10 EDT 
---
I find that this package is not need now.

We already have handwriting-ja.xml and handwriting-zh_CN.xml in fedora and can
be used to generate model files.

See http://koji.fedoraproject.org/koji/rpminfo?rpmID=1442023

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


[Bug 588654] Review Request: plexus-component-api - Plexus Component API

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from Yang Yong yy...@redhat.com 2010-05-17 09:26:43 EDT ---
New Package CVS Request
===
Package Name: plexus-component-api
Short Description: Plexus Component API
Owners: yyang
Branches: devel
InitialCC: overholt huwang liweinan

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


[Bug 570731] Review Request: ibus-handwrite - IBus handwrite project

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


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

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

   What|Removed |Added

   Flag||fedora-review?

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


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

2010-05-17 Thread bugzilla
Please do not reply directly 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

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 #8 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
09:47:00 EDT ---
OK, I'll do that

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


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

2010-05-17 Thread bugzilla
Please do not reply directly 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

--- Comment #3 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
09:45:27 EDT ---
Review:

OK: rpmlint must be run on every package. OUTPUT:
maven-resources-plugin.noarch: W: no-documentation
maven-resources-plugin.noarch: W: non-conffile-in-etc
/etc/maven/fragments/maven-resources-plugin
maven-resources-plugin.src: W: invalid-url Source0:
maven-resources-plugin-2.2.tar.gz

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.

One small problem only:
install -pm 644 pom.xml \
%{buildroot}%{_mavenpomdir}/JPP.%{name}.pom 
should be
install -pm 644 pom.xml \
%{buildroot}%{_mavenpomdir}/JPP-%{name}.pom

This is needed for proper finding of the 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 591011] Review Request: kfilefactory - An easy to use filefactory.com uploader

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


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

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

   What|Removed |Added

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

--- Comment #3 from Rex Dieter rdie...@math.unl.edu 2010-05-17 10:14:36 EDT 
---
I'll take a look.

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


[Bug 592670] Review Request: mongoose - An easy-to-use self-sufficient web server

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


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

--- Comment #2 from Rafael Aquini aqu...@linux.com 2010-05-17 10:15:32 EDT ---
Spec URL: http://aquini.tchesoft.com/RPMS/mongoose/mongoose.spec
SRPM URL: http://aquini.tchesoft.com/RPMS/mongoose/mongoose-2.8-2.fc12.src.rpm

MD5:
fa58e9038709d389b2fc7c6cffc9c068  mongoose-2.8-2.fc12.src.rpm
3ffb7344a8f8cb8056cfa2c8e220649c  mongoose.spec


Chen,
Thanks for reviewing the spec.

I did make some of the modifications you suggested, as follows:
[2] -devel package was stripped off the spec file;
[3] The misusing of %doc was fixed;
[4] Group changed to Applications/System -- IMHO, I do not think it could fit
in anyone else in System Environment class;
[5] Now the man page compression is done by rpmbuild itself.

Unfortunately, there was some suggestions of yours I could not deliver as
promptly as you wish: 
[1] I do not see a real necessity to ship a SysV initscrip, since mongoose is a
regular user tool not intended to work as a daemon. The purpose of mongoose is
to give flexibility to an ordinary user (not administrator) when he/she needs
to set up quickly an web server to share files in a specific directory, or just
do some tests. Please, let me know if you observation [1] is a real blocker to
this package. 

[6] As you may notice, there is no install section on Makefile delivered by
upstream, so I have the real need of being using %{__install} macros. IMHO, it 
would be preferable using this macros than the commands itself, wouldn' be?
Please, let me know if you observation [6] is also a real blocker to this
package.  

Once again, thanks a lot for your attention on this matter.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on 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-17 Thread bugzilla
Please do not reply directly 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 #10 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
10:25:02 EDT ---
OK, answer to self - it requires newer maven than what we have

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


[Bug 588474] Review Request: rubygem-rubyzip - zipfile support in Ruby

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


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

--- Comment #15 from Adam Young ayo...@redhat.com 2010-05-17 10:23:25 EDT ---
Much cleaned up from previous reviews:  


http://github.com/admiyo/MySpecs/blob/c1f91849426aed4747472ebe7291e2209293c786/rubygem-rubyzip.spec

http://admiyo.fedorapeople.org/buildr-repo/rubygem-rubyzip-0.9.4-2.young.src.rpm


I went back to the 0.9.4 version.  It was the latest, it had a lot cleaned up
from 0.9.1, and the issue that caused the breakage could be handled by a patch
to buildr.

The lint errors are fixed.  The lint warnings about template strings in the
installed files are still there, but that is an artifact of the ruby info
build, which has not yet been addressed by the developers.

License is back to GPL V2 or Ruby.
Permissions are fixed

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


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

2010-05-17 Thread bugzilla
Please do not reply directly 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 #9 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
10:23:17 EDT ---
Why do we need to package a trunk snapshot instead of using what's released
http://svn.apache.org/repos/asf/maven/release/tags/maven-release-2.0/ ?

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


[Bug 592208] Review Request: apache-commons-daemon - rename of jakarta-commons-daemon

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


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

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 588142] Review Request: maven-release - Release a project updating the POM and tagging in the SCM

2010-05-17 Thread bugzilla
Please do not reply directly 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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #11 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
10:57:23 EDT ---
Review:

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

maven-release-manager-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs
- Java docs, Java-docs, Javanese
maven-release-plugin-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs
- Java docs, Java-docs, Javanese
maven-release-manager.noarch: W: spelling-error %description -l en_US plugin -
plug in, plug-in, plugging
maven-release-manager.noarch: W: no-documentation
maven-release.noarch: W: spelling-error %description -l en_US plugin - plug
in, plug-in, plugging
maven-release.noarch: W: non-conffile-in-etc /etc/maven/fragments/maven-release
maven-release-plugin.noarch: W: no-documentation
maven-release.src: W: spelling-error %description -l en_US plugin - plug in,
plug-in, plugging
maven-release.src: W: invalid-url Source0: maven-release-2.0.tar.xz

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.


Btw, What is the reason for putting BuildArch:noarch when the main package is
noarch already. Plese remove it.

Other than that the package is good.

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 192311] Review Request: cobbler

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


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

Scott J Henson shen...@redhat.com changed:

   What|Removed |Added

 CC||shen...@redhat.com

--- Comment #25 from Scott J Henson shen...@redhat.com 2010-05-17 10:59:46 
EDT ---
Package Change Request
==
Package Name: cobbler
New Branches: EL-6
Owners: shenson jeckersb awood

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


[Bug 192311] Review Request: cobbler

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


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

Scott J Henson shen...@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 the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588406] Review Request: rubygem-buildr - Ruby Based build system that uses Maven style repositories

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


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

--- Comment #4 from Adam Young ayo...@redhat.com 2010-05-17 11:06:31 EDT ---
http://admiyo.fedorapeople.org/buildr-repo/rubygem-buildr-1.3.5-6.young.src.rpm
http://github.com/admiyo/MySpecs/blob/b3f15623e82c3b322b7a7a082f193cd2ef07a1d8/rubygem-buildr.spec


Added a patch for a bug introduced by a change in the rubyzip API.  Now it uses
named parameters, which will work with either 1.9.1 or 1.9.4 rubyzip.

Cleaned up rpmlint errors.

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


[Bug 192311] Review Request: cobbler

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


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

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

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #26 from Jason Tibbitts ti...@math.uh.edu 2010-05-17 11:11:53 EDT 
---
This package already seems to have an EL-6 branch.

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


[Bug 592387] Review Request: maven-install-plugin - Maven Install Plugin

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


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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #6 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
11:17:55 EDT ---
Thanks,

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 591857] Review Request: maven-resources-plugin - Maven Resources Plugin

2010-05-17 Thread bugzilla
Please do not reply directly 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

--- Comment #4 from huwang huw...@redhat.com 2010-05-17 11:21:46 EDT ---
(In reply to comment #3)
 Review:
 
 OK: rpmlint must be run on every package. OUTPUT:
 maven-resources-plugin.noarch: W: no-documentation
 maven-resources-plugin.noarch: W: non-conffile-in-etc
 /etc/maven/fragments/maven-resources-plugin
 maven-resources-plugin.src: W: invalid-url Source0:
 maven-resources-plugin-2.2.tar.gz
 
 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.
 
 One small problem only:
 install -pm 644 pom.xml \
 %{buildroot}%{_mavenpomdir}/JPP.%{name}.pom 
 should be
 install -pm 644 pom.xml \
 %{buildroot}%{_mavenpomdir}/JPP-%{name}.pom
 
 This is needed for proper finding of the pom.
Fixed.
Spec URL:
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=maven-resources-plugin.speccan=2q=#makechanges
SPRM URL:
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=maven-resources-plugin-2.2-2.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 591857] Review Request: maven-resources-plugin - Maven Resources Plugin

2010-05-17 Thread bugzilla
Please do not reply directly 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

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

--- Comment #5 from Alexander Kurtakov akurt...@redhat.com 2010-05-17 
11:33:28 EDT ---
Thanks,

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 592445] Review Request: maven-pmd-plugin - Maven PMD Plugin

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


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

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

   What|Removed |Added

 Status|MODIFIED|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 591947] Review Request: maven-jar-plugin - Maven JAR Plugin

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


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

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

   What|Removed |Added

 Status|MODIFIED|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 591222] gtk3 - version 3 of GTK+

2010-05-17 Thread bugzilla
Please do not reply directly 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 #6 from Matthias Clasen mcla...@redhat.com 2010-05-17 12:08:46 
EDT ---
These fixes the wrapper script file permissions and the source url:

srpm: http://mclasen.fedorapeople.org/gtk3-2.90.0-3.fc13.src.rpm
spec: http://mclasen.fedorapeople.org/gtk3.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 588474] Review Request: rubygem-rubyzip - zipfile support in Ruby

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


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

--- Comment #16 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-05-17 
13:45:52 EDT ---
(From next time, please change the release number when you modify your
 spec file to avoid confusion).

For (second) -2:
* rpmlint
  - non-standard-executable-perm rpmlint errors still exist
-
rubygem-rubyzip-doc.noarch: E: non-executable-script
/usr/lib/ruby/gems/1.8/gems/rubyzip-0.9.4/test/data/file2.txt 0664L
/usr/bin/env
rubygem-rubyzip-doc.noarch: E: non-standard-executable-perm
/usr/lib/ruby/gems/1.8/gems/rubyzip-0.9.4/test/gentestfiles.rb 0775L
rubygem-rubyzip-doc.noarch: E: non-standard-executable-perm
/usr/lib/ruby/gems/1.8/gems/rubyzip-0.9.4/test/ziprequiretest.rb 0775L
rubygem-rubyzip-doc.noarch: E: non-standard-executable-perm
/usr/lib/ruby/gems/1.8/gems/rubyzip-0.9.4/test/stdrubyexttest.rb 0775L
rubygem-rubyzip-doc.noarch: E: non-standard-executable-perm
/usr/lib/ruby/gems/1.8/gems/rubyzip-0.9.4/test/alltests.rb 0775L
rubygem-rubyzip-doc.noarch: E: non-standard-executable-perm
/usr/lib/ruby/gems/1.8/gems/rubyzip-0.9.4/test/ioextrastest.rb 0775L
rubygem-rubyzip-doc.noarch: E: non-standard-executable-perm
/usr/lib/ruby/gems/1.8/gems/rubyzip-0.9.4/test/data/notzippedruby.rb 0775L
-
   Again, permissions should usually be 0644 or 0755.

* license tag
  - is still GPLv2+ or Ruby.

* Consistent macro usage
  - Please use %geminstdir when possible (please check %check section)

* Test program
  - Please enable test program (also please check my previous comment)
  ! Note
require 'net/sftp' line in Rakefile doesn't seem to be needed.

* Document files
  - I guess sample/ directory should be marked as %doc (and are there
any reason you want to put sample/ file in main binary 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 592579] Review Request: Frama-c - Framework for source code analysis of C software

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


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

--- Comment #2 from David A. Wheeler dwhee...@dwheeler.com 2010-05-17 
14:17:43 EDT ---
Thanks for doing this!!  I'll start reviewing, but I cannot serve as a sponsor.

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


[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

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


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

David A. Wheeler dwhee...@dwheeler.com changed:

   What|Removed |Added

 CC||dwhee...@dwheeler.com
 AssignedTo|nob...@fedoraproject.org|dwhee...@dwheeler.com

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


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

2010-05-17 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: pynie - Implementation of Python 3 on top of the 
Parrot virtual machine

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

   Summary: Review Request: pynie - Implementation of Python 3 on
top of the Parrot virtual machine
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dmalc...@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://dmalcolm.fedorapeople.org/python-packaging/pynie.spec
SRPM URL:
http://dmalcolm.fedorapeople.org/python-packaging/pynie-0.0.r114-1.fc13.src.rpm

Description:
Pynie is an implementation of Python 3 on top of the Parrot virtual machine.

Successful scratch build in Koji here:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2192819

rpmlint output on a local build on my machine:
pynie.x86_64: W: incoherent-version-in-changelog 0.1-1 ['0.0.r114-1.fc13',
'0.0.r114-1']
pynie-debuginfo.x86_64: E: debuginfo-without-sources
2 packages and 0 specfiles checked; 1 errors, 1 warnings.

The former is inconsequential (dist tag)
Re the latter: I'm not sure why it isn't picking up the sources.  Possibly a
missing -g flag in the invocation of gcc by pbc_to_exe

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


[Bug 513896] Review Request: pcp - performance monitoring and collection service

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


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

--- Comment #29 from Eric Sandeen esand...@redhat.com 2010-05-17 14:40:49 EDT 
---
Hm, what's next?  An awful lot of effort on this bug to not yet have it built
;)

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


[Bug 584666] Review Request: lv2-invada-plugins - A collection of LV2 plugins from Invada Records

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


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

--- Comment #16 from Fedora Update System upda...@fedoraproject.org 
2010-05-17 14:51:08 EDT ---
lv2-invada-plugins-1.2.0-3.fc12 has been pushed to the Fedora 12 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 499987] Review Request: mingw32-curl - MinGW Windows port of curl and libcurl

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


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||mingw32-curl-7.20.1-1.fc12
 Resolution||ERRATA

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


[Bug 584666] Review Request: lv2-invada-plugins - A collection of LV2 plugins from Invada Records

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||lv2-invada-plugins-1.2.0-3.
   ||fc12
 Resolution||ERRATA

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


[Bug 499987] Review Request: mingw32-curl - MinGW Windows port of curl and libcurl

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


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

--- Comment #16 from Fedora Update System upda...@fedoraproject.org 
2010-05-17 14:54:03 EDT ---
mingw32-curl-7.20.1-1.fc12 has been pushed to the Fedora 12 stable repository. 
If problems still persist, please make note of it in this bug report.

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


[Bug 584666] Review Request: lv2-invada-plugins - A collection of LV2 plugins from Invada Records

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


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

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

   What|Removed |Added

   Fixed In Version|lv2-invada-plugins-1.2.0-3. |lv2-invada-plugins-1.2.0-3.
   |fc12|fc13

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


[Bug 584666] Review Request: lv2-invada-plugins - A collection of LV2 plugins from Invada Records

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


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

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2010-05-17 15:02:06 EDT ---
lv2-invada-plugins-1.2.0-3.fc13 has been pushed to the Fedora 13 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 584666] Review Request: lv2-invada-plugins - A collection of LV2 plugins from Invada Records

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


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

--- Comment #18 from Fedora Update System upda...@fedoraproject.org 
2010-05-17 15:06:01 EDT ---
lv2-invada-plugins-1.2.0-3.fc11 has been pushed to the Fedora 11 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 584666] Review Request: lv2-invada-plugins - A collection of LV2 plugins from Invada Records

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


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

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

   What|Removed |Added

   Fixed In Version|lv2-invada-plugins-1.2.0-3. |lv2-invada-plugins-1.2.0-3.
   |fc13|fc11

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


[Bug 499987] Review Request: mingw32-curl - MinGW Windows port of curl and libcurl

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


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

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2010-05-17 15:08:46 EDT ---
mingw32-curl-7.20.1-1.fc13 has been pushed to the Fedora 13 stable repository. 
If problems still persist, please make note of it in this bug report.

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


[Bug 499987] Review Request: mingw32-curl - MinGW Windows port of curl and libcurl

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


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

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

   What|Removed |Added

   Fixed In Version|mingw32-curl-7.20.1-1.fc12  |mingw32-curl-7.20.1-1.fc13

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


[Bug 542292] Review Request: mintmenu - gnome-panel menu from LinuxMint

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


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

--- Comment #21 from Valent Turkovic valent.turko...@gmail.com 2010-05-17 
15:34:23 EDT ---
I made a new spec file for 4.9.9 and I have lots of issues, most of patches
from last package don't work, I guess some of them are obsolete and already
incorporated into mainstream code tree.

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


[Bug 542292] Review Request: mintmenu - gnome-panel menu from LinuxMint

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


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

--- Comment #22 from Valent Turkovic valent.turko...@gmail.com 2010-05-17 
15:37:28 EDT ---
Created an attachment (id=414643)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=414643)
mintmenu.spec file for mintmenu 4.9.9 version, package builds but doesn't run.
It needs files in /usb/lib/linuxmint but package installs files into
/usr/share/linuxmint/ any idea how to fix 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 593125] New: Review Request: gridsite - Grid Security for the Web, Web platforms for Grids

2010-05-17 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: gridsite -  Grid Security for the Web, Web platforms 
for Grids

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

   Summary: Review Request: gridsite -  Grid Security for the Web,
Web platforms for Grids
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: steve.tray...@cern.ch
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://cern.ch/straylen/rpms/gridsite/gridsite.spec
SRPM URL: http://cern.ch/straylen/rpms/gridsite/gridsite-1.5.18-1.fc13.src.rpm
Description:
GridSite was originally a web application developed for managing and formatting 
the content of the http://www.gridpp.ac.uk/ website. Over the past years it 
has grown into a set of extensions to the Apache web server and a toolkit for 
Grid credentials, GACL access control lists and HTTP(S) protocol operations.

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


[Bug 513896] Review Request: pcp - performance monitoring and collection service

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


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

Mark Goodwin mgood...@redhat.com changed:

   What|Removed |Added

 CC||overh...@redhat.com

--- Comment #30 from Mark Goodwin mgood...@redhat.com 2010-05-17 18:25:27 EDT 
---
(In reply to comment #29)
 Hm, what's next?  An awful lot of effort on this bug to not yet have it built
 ;)

Hi Eric, yes activity has resumed - Bill Cohen is talking with Andrew Overholt
in the tools group as a possible sponsor. Alternatively, we could get you
promoted to become sponsor-capable - you maintain enough important packages
in Fedora that maybe this would be a good idea regardless (?).

Cheers
-- Mark

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


[Bug 565251] Review Request: coan - A commandline tool for simplifying the preprocessor conditionals in source code

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


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

--- Comment #7 from Eric Smith e...@brouhaha.com 2010-05-17 18:58:59 EDT ---
I just emailed the author to get a clarification with regard to the license and
that COPYING file.

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


[Bug 591305] Review Request: apache-commons-digester - XML to Java object mapping module

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


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

--- Comment #9 from Mat Booth fed...@matbooth.co.uk 2010-05-17 19:05:34 EDT 
---
(In reply to comment #8)
 Spec file looks good, I believe there will be no problem with this review once
 beanutils is done. Do you still plan to do the spec file for beanutils?

At some point during the next week. I will check and update the wiki page
before I start on it.

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


[Bug 513896] Review Request: pcp - performance monitoring and collection service

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


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

--- Comment #32 from Andrew Overholt overh...@redhat.com 2010-05-17 20:23:46 
EDT ---
This is the first I've heard of this :)  However, I'm not really well-versed
enough in the packaging guidelines for non-Java stuff so I can't be a sponsor,
sorry.

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


[Bug 542292] Review Request: mintmenu - gnome-panel menu from LinuxMint

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


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

--- Comment #23 from William Witt will...@witt-family.net 2010-05-17 21:44:19 
EDT ---
When I moved to SUSE, much of the work I did to get mintmenu to work on Fedora
also worked on SUSE.  You can find a SUSE src rpm here: 
http://download.opensuse.org/repositories/home:/unamanic/openSUSE_11.2/src/mintmenu-4.9.4-2.1.src.rpm
 you'll want to to tweak the branding patch and you may also need to tweak the
datadir patch since it's applied after the branding, but it should get you
going the right direction.

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


[Bug 575005] Review Request: zinnia-tomoe - Online hand recognition system with machine learning

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


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

--- Comment #51 from Peng Huang phu...@redhat.com 2010-05-17 21:56:04 EDT ---
Probably we still need a package to include zinnia format data files converted
from tomoe. It could be a new package zinnia-tomoe or in zinnia self.

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


[Bug 575005] Review Request: zinnia-tomoe - Online hand recognition system with machine learning

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


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

--- Comment #52 from Chen Lei supercyp...@gmail.com 2010-05-17 22:16:40 EDT 
---
(In reply to comment #51)
 Probably we still need a package to include zinnia format data files converted
 from tomoe. It could be a new package zinnia-tomoe or in zinnia self.

I think we can merge those data to zinnia or ibus-handwrite as a subpackage.

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


[Bug 591812] Review Request: perlbrew - Manage perl installations in your $HOME

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


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

Chris Weyl cw...@alumni.drew.edu 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 591812] Review Request: perlbrew - Manage perl installations in your $HOME

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


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

--- Comment #1 from Chris Weyl cw...@alumni.drew.edu 2010-05-17 22:28:04 EDT 
---
The licensing here is a little confused, as perlbrew contains HTTP::Lite...
or did?  Makefile.PL says same as Perl, as does the Changelog entries;
LICENSE in the root says MIT, as does lib/App/perlbrew.pm.

If it actually includes HTTP::Lite, then it will have to be Perl (or GPL+ at
least).  If not, then it can be MIT.  Everything else here looks good; I hate
to say it, but I think we need to get some sort of clarification from the
author; the statement of license in perlbrew.pm controls here.

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

Spec looks sane, clean and consistent; make test passes cleanly.

Source tarballs match upstream (sha1sum):
aef837763f49c7c8f4bca88a1878669aa734684f App-perlbrew-0.06.tar.gz
aef837763f49c7c8f4bca88a1878669aa734684f App-perlbrew-0.06.tar.gz.srpm

Final provides / requires are sane:

 rpmlint perlbrew.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

== perlbrew-0.06-1.fc14.noarch.rpm ==
 rpmlint
perlbrew.noarch: W: spelling-error Summary(en_US) perl - Perl, peel, perk
perlbrew.noarch: W: spelling-error %description -l en_US perl - Perl, peel,
perk
perlbrew.noarch: W: spelling-error %description -l en_US bashrc - Basho,
bashful, bashing
perlbrew.noarch: W: spelling-error %description -l en_US cshrc - cashier,
crush, crash
perlbrew.noarch: W: spelling-error %description -l en_US sudo - suds, ludo,
sumo
perlbrew.noarch: W: spelling-error %description -l en_US cpan - can, pan, span
1 packages and 0 specfiles checked; 0 errors, 6 warnings.
 provides
perl(App::perlbrew) = 0.06
perl-App-perlbrew = 0.06-1.fc14
perlbrew = 0.06-1.fc14
 requires
/usr/bin/perl  
perl = 1:5.8.0
perl(:MODULE_COMPAT_5.10.1)  
perl(App::perlbrew)  
perl(Getopt::Long)  
perl(HTTP::Lite) = 2.2
perl(Pod::Usage)  
perl(strict)  
 obsoletes
 conflicts

== perlbrew-0.06-1.fc14.src.rpm ==
 rpmlint
perlbrew.src: W: spelling-error Summary(en_US) perl - Perl, peel, perk
perlbrew.src: W: spelling-error %description -l en_US perl - Perl, peel, perk
perlbrew.src: W: spelling-error %description -l en_US bashrc - Basho, bashful,
bashing
perlbrew.src: W: spelling-error %description -l en_US cshrc - cashier, crush,
crash
perlbrew.src: W: spelling-error %description -l en_US sudo - suds, ludo, sumo
perlbrew.src: W: spelling-error %description -l en_US cpan - can, pan, span
1 packages and 0 specfiles checked; 0 errors, 6 warnings.
 provides
 requires
perl = 1:5.8.0
perl(ExtUtils::MakeMaker)  
perl(HTTP::Lite) = 2.2
perl(Test::More)  
 obsoletes
 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 549980] Review Request: jama - C++ matrix templates

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


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

--- Comment #8 from matt chan talc...@gmail.com 2010-05-17 23:49:36 EDT ---
New Package CVS Request
===
Package Name: jama
Short Description: A collection of C++ matrix templates
Owners: talcite
Branches: F-12 F-13 EL-5
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 550234] Review Request: tnt - C++ templates for scientific computing

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


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

matt chan talc...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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


[Bug 549980] Review Request: jama - C++ matrix templates

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


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

matt chan talc...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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


[Bug 550234] Review Request: tnt - C++ templates for scientific computing

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


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

--- Comment #17 from matt chan talc...@gmail.com 2010-05-17 23:50:21 EDT ---
New Package CVS Request
===
Package Name: tnt
Short Description: C++ templates for scientific computing
Owners: talcite
Branches: F-12 F-13 EL-5
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 225764] Merge Review: fonts-ISO8859-2

2010-05-17 Thread bugzilla
Please do not reply directly 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

--- Comment #19 from Parag pnem...@redhat.com 2010-05-18 01:20:32 EDT ---
Thanks for the review!

here is updated package.
SPEC:http://paragn.fedorapeople.org/fedora-work/SPECS/iso8859-2-fonts.spec
SRPM:http://paragn.fedorapeople.org/fedora-work/SRPMS/iso8859-2-fonts-1.0-24.fc13.src.rpm

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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on 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-17 Thread bugzilla
Please do not reply directly 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

   Flag||fedora-cvs?

--- Comment #20 from Parag pnem...@redhat.com 2010-05-18 01:32:22 EDT ---
New Package CVS Request
===
Package Name: iso8859-2-fonts
Short Description: Central European language fonts for the X Window System
Owners: pnemade
Branches: F-13
InitialCC: i18n-team, fonts-sig

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


[Bug 591812] Review Request: perlbrew - Manage perl installations in your $HOME

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


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

Chris Weyl cw...@alumni.drew.edu changed:

   What|Removed |Added

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

--- Comment #3 from Chris Weyl cw...@alumni.drew.edu 2010-05-18 01:30:30 EDT 
---
Ok, that makes sense to me :)

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