[Bug 461139] Review Request: arabeyes-core-fonts - Core Arabic fonts from Arabeyes.org

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


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


Jens Petersen [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED],
   ||[EMAIL PROTECTED]
   ||at.com, [EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459073] Review Request: iok - Indic onscreen virtual keyboard

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


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





--- Comment #12 from Jens Petersen [EMAIL PROTECTED]  2008-09-05 03:00:09 EDT 
---
cvs admin done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461131] Review Request: sim - Simple Instant Messenger

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


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





--- Comment #9 from Marcela Maslanova [EMAIL PROTECTED]  2008-09-05 03:25:19 
EDT ---
Shouldn't you use also this part?
desktop-file-install --vendor=fedora\
--add-category=Network\
--delete-original\
--dir=$RPM_BUILD_ROOT/%{_datadir}/applications\
$RPM_BUILD_ROOT/%{_datadir}/applications/kde/%{name}.desktop

You can follow this guide lines:
https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop

I'm not sure if I have correct the category.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459073] Review Request: iok - Indic onscreen virtual keyboard

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


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





--- Comment #13 from Parag AN(पराग) [EMAIL PROTECTED]  2008-09-05 03:30:15 
EDT ---
Package Change Request
==
Package Name: iok
New Branches: 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459073] Review Request: iok - Indic onscreen virtual keyboard

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


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





--- Comment #14 from Jens Petersen [EMAIL PROTECTED]  2008-09-05 03:44:05 EDT 
---
added

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459073] Review Request: iok - Indic onscreen virtual keyboard

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


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





--- Comment #15 from Parag AN(पराग) [EMAIL PROTECTED]  2008-09-05 03:46:44 
EDT ---
Jens,
Thanks for review and 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457277] Review Request: pentaho-reporting-flow-engine - Reporting Engine

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


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





--- Comment #3 from Caolan McNamara [EMAIL PROTECTED]  2008-09-05 03:54:30 
EDT ---
silly me, fixed as:

http://people.redhat.com/caolanm/jfreereport/pentaho-reporting-flow-engine.spec
http://people.redhat.com/caolanm/jfreereport/pentaho-reporting-flow-engine-0.9.3-2.fc9.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461237] New: Review Request: libmusicbrainz3 - Library for accessing MusicBrainz servers

2008-09-05 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: libmusicbrainz3 - Library for accessing MusicBrainz 
servers

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

   Summary: Review Request: libmusicbrainz3 - Library for
accessing MusicBrainz servers
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://people.redhat.com/bnocera/libmusicbrainz3/libmusicbrainz3.spec
SRPM URL:
http://people.redhat.com/bnocera/libmusicbrainz3/libmusicbrainz3-3.0.1-1.fc10.src.rpm
Description:

The MusicBrainz client library allows applications to make metadata
lookup to a MusicBrainz server, generate signatures from WAV data and
create CD Index Disk ids from audio CD roms.

This library uses the new XML service from Musicbrainz, whereas
libmusicbrainz itself uses the deprecated RDF service.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461237] Review Request: libmusicbrainz3 - Library for accessing MusicBrainz servers

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


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





--- Comment #1 from Bastien Nocera [EMAIL PROTECTED]  2008-09-05 05:03:59 EDT 
---
*Sigh*. It's already in the repo...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461237] Review Request: libmusicbrainz3 - Library for accessing MusicBrainz servers

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


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


Bastien Nocera [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 426751] Review Request: ghc-X11 - A Haskell binding to the X11 graphics library.

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


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





--- Comment #21 from Jens Petersen [EMAIL PROTECTED]  2008-09-05 05:22:00 EDT 
---
Created an attachment (id=315852)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=315852)
ghc-X11.spec

Here is a slightly revised spec file without the macros embedded using the
latest macros I attached to bug 460304 - tested and builds on rawhide.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461106] Review Request: libnotifymm - C++ interface for libnotify

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


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





--- Comment #2 from Denis Leroy [EMAIL PROTECTED]  2008-09-05 05:42:02 EDT ---
[EMAIL PROTECTED], rawhide package includes an API-breaking patch.

Here's the fix, but this will only build on rawhide. For F-9, comment out the
first patch from the spec file.

Spec URL: http://www.poolshark.org/src/libnotifymm.spec
SRPM URL: http://www.poolshark.org/src/libnotifymm-0.6.1-2.fc10.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] New: Review-Request: justmoon - Just Moon is lunar observing software for Linux

2008-09-05 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: justmoon - Just Moon is lunar observing software for 
Linux
Alias: justmoon-review

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

   Summary: Review-Request: justmoon - Just Moon is lunar
observing software for Linux
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


SRPMS: http://netbsd.sk/~lkundrak/SRPMS/justmoon-0.3.3-1.el5.src.rpm
SPECS: http://netbsd.sk/~lkundrak/SPECS/justmoon.spec

Description:

This package contains a lunar observing software with
GTK+-based user interface that fits the GNOME Desktop environment,
and QT-based user interface that fits the KDE Desktop environment.
They are is supplemented with package that contains data files.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457535] Review Request: myanmar3-unicode-fonts - Myanmar3 unicode font

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


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





--- Comment #11 from Minto Joseph [EMAIL PROTECTED]  2008-09-05 07:46:08 EDT 
---
New Package CVS Request
===
Package Name: myanmar3-unicode-fonts
Short Description: Myanmar3 unicode font
Owners: mintojoseph
Branches: F-8 F-9
InitialCC: fonts-sig
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461260] New: Review Request: hunspell-is - Icelandic hunspell dictionary

2008-09-05 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: hunspell-is - Icelandic hunspell dictionary

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

   Summary: Review Request: hunspell-is - Icelandic hunspell
dictionary
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://people.redhat.com/caolanm/hunspell/hunspell-is.spec
SRPM URL:
http://people.redhat.com/caolanm/hunspell/hunspell-is-0.20060928-1.fc9.src.rpm
Description: Icelandic dictionaries for hunspell, as used by OpenOffice.org,
firefox, etc

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 456298] Review Request: netbeans-resolver - Resolver subproject of xml-commons patched for NetBeans

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


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


Victor G. Vasilyev [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|needinfo?([EMAIL PROTECTED] |
   |un.com) |




--- Comment #8 from Victor G. Vasilyev [EMAIL PROTECTED]  2008-09-05 08:05:28 
EDT ---
The fifth release is prepared for review.

Spec URL:
http://nbi.netbeans.org/files/documents/210/2050/netbeans-resolver.spec
SRPM URL:
http://nbi.netbeans.org/files/documents/210/2167/netbeans-resolver-6.1-5.fc10.src.rpm

Changes (In reply to comment #7):
 2 issues to be resolved.
  1.13 Summary and description
 Change the description so it does not extend pass 80 chars/line
- The description is formatted
  MUST: The License field in the package spec file must match the actual
 license.
 No. The license field says ASL 2.0, where as the actual license is ASL 1.1.
- The license version is fixed

The rpmlint shows no errors and no warnings against both SRPM and RPM.

Successful koji build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=808496

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 199154] Review Request: Slony-1 (postgresql-slony-engine)

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


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





--- Comment #35 from Itamar Reis Peixoto [EMAIL PROTECTED]  2008-09-05 
08:11:14 EDT ---
the name needs to be fixed, in some parts of the code have Slony-I name

Slony-I
slony1
postgresql-slony1

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453701] Review Request: minisat2 - A minimalistic, open-source SAT solver

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


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





--- Comment #21 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:18:57 EDT ---
minisat2-2.0-7.20070721.fc9 has been pushed to the Fedora 9 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458405] Review Request: pisg - Perl IRC Statistics Generator

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


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





--- Comment #6 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:18:50 EDT ---
pisg-0.72-3.fc8 has been pushed to the Fedora 8 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457026] Review Request: rubygem-arrayfields - Arrayfields RubyGem

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


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





--- Comment #13 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:19:10 EDT ---
rubygem-arrayfields-4.5.0-3.fc8 has been pushed to the Fedora 8 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457263] Review Request: liblayout - positioning library

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


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


Jason Tibbitts [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]




--- Comment #8 from Jason Tibbitts [EMAIL PROTECTED]  2008-09-05 08:20:38 EDT 
---
Great, APPROVED.

Sorry I didn't realize the dependency chain prevented Mattias from doing a
complete review initially.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460101] Review Request: perl-HTML-RewriteAttributes - Concise attribute rewriting

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


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





--- Comment #7 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:21:51 EDT ---
perl-HTML-RewriteAttributes-0.03-1.fc9 has been pushed to the Fedora 9 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455067] Review Request: ferm - For Easy Rule Making

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


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





--- Comment #21 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:20:48 EDT ---
ferm-2.0.2-3.fc9 has been pushed to the Fedora 9 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459409] Review Request: E - Equational Theorem Prover

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


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


Fedora Update System [EMAIL PROTECTED] changed:

   What|Removed |Added

 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455032] Review Request: iwl5000-firmware - Firmwa re for Intel® PRO /Wireless 5000 A/G/N network adaptors

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


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





--- Comment #12 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:22:07 EDT ---
iwl5000-firmware-5.4.A.11-2 has been pushed to the Fedora 8 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457277] Review Request: pentaho-reporting-flow-engine - Reporting Engine

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


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


Jason Tibbitts [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460101] Review Request: perl-HTML-RewriteAttributes - Concise attribute rewriting

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


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


Fedora Update System [EMAIL PROTECTED] changed:

   What|Removed |Added

 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455032] Review Request: iwl5000-firmware - Firmwa re for Intel® PRO /Wireless 5000 A/G/N network adaptors

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


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


Fedora Update System [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ON_QA   |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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454469] Review Request: php-pear-Var-Dump - rovides methods for dumping structured information about a variable

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


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


Fedora Update System [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #8 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:21:16 EDT ---
php-pear-Var-Dump-1.0.3-2.fc8 has been pushed to the Fedora 8 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update php-pear-Var-Dump'.  You can
provide feedback for this update here:
http://admin.fedoraproject.org/updates/F8/FEDORA-2008-7275

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] Review-Request: justmoon - Just Moon is lunar observing software for Linux

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


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


Marek Mahut [EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457839] Review Request: rubygem-cobbler - Provides Ruby bindings to interact with a Cobbler server.

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


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





--- Comment #14 from Darryl L. Pierce [EMAIL PROTECTED]  2008-09-05 08:29:13 
EDT ---
Not sure what's up with the weird timestamp. I built the gem locally and then
unpacked it and am getting none of what you're seeing. Is it possible the RPM
process is repackaging the gem, like it does with JARs?

I took your advice, ran koji local and all built properly.

Spec: http://mcpierce.fedorapeople.org/rubygem-cobbler.spec
SRPM: http://mcpierce.fedorapeople.org/rubygem-cobbler-0.0.2-4.fc9.src.rpm
GEM: http://mcpierce.fedorapeople.org/cobbler-0.0.2.gem

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] Review-Request: justmoon - Just Moon is lunar observing software for Linux

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


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


Marek Mahut [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #1 from Marek Mahut [EMAIL PROTECTED]  2008-09-05 08:30:02 EDT ---
[X]  RPMlink
  mixed-use-of-spaces-and-tabs - cosmetic issue
  rpm-buildroot-usage - please move desktop-file-install to %install
[OK] source files match upstream url
  3007c84e3b5232c0dba9f5c0b5c3a669 justmoon-0.3.3.tar.gz
[OK] package meets naming and packaging guideline
[OK] specfile is sane
[OK] spec file is legible
[OK] dist tag is present
[OK] build root is correct
[OK] license is Fedora compatible
[OK] %clean is present
[X] builds in rawhide
  http://koji.fedoraproject.org/koji/taskinfo?taskID=808442

  [...]

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459409] Review Request: E - Equational Theorem Prover

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


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





--- Comment #18 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:20:19 EDT ---
E-0.999.006-2.fc8 has been pushed to the Fedora 8 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 427674] Review Request: themonospot - An Avi parser and content descriptor

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


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





--- Comment #89 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:17:52 EDT ---
themonospot-0.7.1.1-1.fc8 has been pushed to the Fedora 8 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453701] Review Request: minisat2 - A minimalistic, open-source SAT solver

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


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


Fedora Update System [EMAIL PROTECTED] changed:

   What|Removed |Added

 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460615] Review Request: bti - Bash Twitter Idiocy

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


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


Fedora Update System [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ON_QA




--- Comment #8 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
08:18:45 EDT ---
bti-005-1.fc8 has been pushed to the Fedora 8 testing repository.  If problems
still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update bti'.  You can provide feedback
for this update here:
http://admin.fedoraproject.org/updates/F8/FEDORA-2008-7252

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 456298] Review Request: netbeans-resolver - Resolver subproject of xml-commons patched for NetBeans

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


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





--- Comment #9 from Lillian Angel [EMAIL PROTECTED]  2008-09-05 08:53:48 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457160] Review Request: Zorba - General purpose XQuery processor implemented in C++

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


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





--- Comment #5 from David Timms [EMAIL PROTECTED]  2008-09-05 08:59:11 EDT ---
Did you post updated spec and srpm somewhere ? Please provide those links each
time your spec is upgraded {make it easy for your reviewers to confirm
suggested changes have occurred}.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] Review-Request: justmoon - Just Moon is lunar observing software for Linux

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


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





--- Comment #2 from Lubomir Rintel [EMAIL PROTECTED]  2008-09-05 08:59:15 EDT 
---
Your internets is broken

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] Review-Request: justmoon - Just Moon is lunar observing software for Linux

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


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





--- Comment #3 from Lubomir Rintel [EMAIL PROTECTED]  2008-09-05 09:00:56 EDT 
---
SRPMS: http://netbsd.sk/~lkundrak/SRPMS/justmoon-0.3.3-2.el5.src.rpm
SPECS: http://netbsd.sk/~lkundrak/SPECS/justmoon.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459871] Review Request: samefile - An utility to find identical files on the file system

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


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





--- Comment #9 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
09:08:16 EDT ---
samefile-2.12-3.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/samefile-2.12-3.fc9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459871] Review Request: samefile - An utility to find identical files on the file system

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


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





--- Comment #10 from Fedora Update System [EMAIL PROTECTED]  2008-09-05 
09:08:20 EDT ---
samefile-2.12-3.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/samefile-2.12-3.fc8

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] Review-Request: justmoon - Just Moon is lunar observing software for Linux

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


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





--- Comment #4 from Marek Mahut [EMAIL PROTECTED]  2008-09-05 09:12:46 EDT ---
Please also correct the desktop file in %files, thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] Review-Request: justmoon - Just Moon is lunar observing software for Linux

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


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





--- Comment #5 from Marek Mahut [EMAIL PROTECTED]  2008-09-05 09:26:21 EDT ---
Great, looks correct! 

 - http://koji.fedoraproject.org/koji/taskinfo?taskID=808659

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] Review-Request: justmoon - Just Moon is lunar observing software for Linux

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


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





--- Comment #6 from Lubomir Rintel [EMAIL PROTECTED]  2008-09-05 09:29:36 EDT 
---
Just in case anyone is interested what marek was reviewing (locally at FUDCon,
since bugzilla was returning 500 errors).

SRPMS: http://netbsd.sk/~lkundrak/SRPMS/justmoon-0.3.3-3.el5.src.rpm
SPECS: http://netbsd.sk/~lkundrak/SPECS/justmoon.spec

New Package CVS Request
===
Package Name: justmoon
Short Description: Lunar observing software
Owners: lkundrak
Branches: F-8 F-9 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453026] Review Request: xmp - A multi-format module player

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


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


Karol Trzcionka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #5 from Karol Trzcionka [EMAIL PROTECTED]  2008-09-05 09:45:47 
EDT ---
Please add script to generate tarball according to
http://fedoraproject.org/wiki/Packaging/SourceURL#When_Upstream_uses_Prohibited_Code
but at now it is not a blocker.
others:
+ rpmlint returns only:
audacious-plugin-xmp.x86_64: W: no-documentation
xmms-xmp.x86_64: W: no-documentation
it might be ignored
+ naming is OK
+ specfile named correct
+ package meets Guidelines
+ License is correct (files which cannot be shipped by Fedora are removed)
+ License tag is ok
+ COPYING file correctly provided
+ I think it is American English
+ specfile is clear for me
+ tarball is changed, so I don't compare md5sum
+ building checked before on koji
+ N/A
+ compiled on koji fine, so I think BRs are correct
+ N/A
+ N/A
+ OK, using macros instead of hardcoded paths
+ correct own all files
+ there are no duplicates in %files
+ %defattr used
+ correct %clean section
+ macros are used correctly
+ nonfree file is removed
+ N/A
+ %doc doesn't affect the runtime of app
+ N/A
+ N/A
+ N/A
+ N/A
+ N/A
+ there are not any .la files
+ N/A
+ correct owns all files
+ correct %install section
+ RPMs have valid UTF-8 files

SHOULD Items:
+ N/A (license is included)
+ N/A (no translations)
+ chacked on koji by Charles
+ like above
? I cannot check working
+ N/A
? probably it is not needed to requires it with full version
+ N/A (no .pc files)
+ dependencies are correct

I can approve but change comment about removing a 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 456298] Review Request: netbeans-resolver - Resolver subproject of xml-commons patched for NetBeans

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


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





--- Comment #10 from Victor G. Vasilyev [EMAIL PROTECTED]  2008-09-05 
09:32:13 EDT ---
New Package CVS Request
===
Package Name: netbeans-resolver
Short Description: Resolver subproject of xml-commons patched for NetBeans
Owners: victorv
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455989] Review Request: xfce-mcs-plugins-extra - Extra plugins for the Xfce mcs manager

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


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





--- Comment #8 from Rahul Sundaram [EMAIL PROTECTED]  2008-09-05 09:37:02 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461250] Review-Request: justmoon - Just Moon is lunar observing software for Linux

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


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





--- Comment #7 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 09:36:28 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458449] Review Request: perl-Log-TraceMessages - Perl extension for trace messages used in debugging

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


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





--- Comment #3 from David Timms [EMAIL PROTECTED]  2008-09-05 09:49:07 EDT ---
Hi,

(In reply to comment #2)
 Initial cc: perl-sig is at cvs creation step. (once package is approved).
 
 I didn't get what you meant by Data::Dumper
TraceMessages.pm has:

=cut
sub d($) {
return '' if not $On;
require Data::Dumper;
my $s = $_[0];

I'm not sure where/if that is automatically available.

 Extracted perl dependencies seems clean to me.
 
 Provides: perl(Log::TraceMessages) = 1.4
 Requires(rpmlib): rpmlib(CompressedFileNames) = 3.0.4-1
 rpmlib(PayloadFilesHavePrefix) = 4.0-1 rpmlib(VersionedDependencies) =
 3.0.3-1
 Requires: perl(:MODULE_COMPAT_5.8.8) perl(Exporter) perl(FileHandle)
 perl(strict) perl(vars)
 Vérification des fichiers non empaquetés: /usr/lib/rpm/check-files
 /var/tmp/perl-Log-TraceMessages-1.4-1.fc8.kwizart-root-builder
If you consider they are fine, that's OK, since I have no experience in this
area !

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457263] Review Request: liblayout - positioning library

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


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





--- Comment #9 from Caolan McNamara [EMAIL PROTECTED]  2008-09-05 10:19:49 
EDT ---
New Package CVS Request
===
Package Name: liblayout
Short Description: Java Positioning Library
Owners: caolanm
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457277] Review Request: pentaho-reporting-flow-engine - Reporting Engine

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


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





--- Comment #4 from Jason Tibbitts [EMAIL PROTECTED]  2008-09-05 10:26:13 EDT 
---
OK, builds and installs fine now.  rpmlint says:

  pentaho-reporting-flow-engine.src:87: W: libdir-macro-in-noarch-package 
   %attr(-,root,root) %{_libdir}/gcj/%{name}
which comes from rpmlint not parsing the conditional arch-ness of this package.

  pentaho-reporting-flow-engine-javadoc.x86_64: W: non-standard-group
   Development/Documentation
which is OK because we don't care what goes in group.  And:

  pentaho-reporting-flow-engine-debuginfo.x86_64: E: description-line-too-long 
   This package provides debug information for package 
   pentaho-reporting-flow-engine.
which could use a line wrap.

I'll go ahead and apprrove this and you can tweak that description when you
check in.

* source files match upstream:
   9ddd78ea9239cefbad21e81621bdc4f2c1f1a9a42a247d38100d49882d90d475  
   pentaho-reporting-flow-engine-0.9.3.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK (javadoc description needs a line wrap).
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* debuginfo package looks complete.
X rpmlint has one valid, but minor complaint.
* final provides and requires are sane:
  pentaho-reporting-flow-engine-0.9.3-2.fc10.x86_64.rpm
   pentaho-reporting-flow-engine.jar.so()(64bit)
   pentaho-reporting-flow-engine = 0.9.3-2.fc10
   pentaho-reporting-flow-engine(x86-64) = 0.9.3-2.fc10
  =
   /bin/sh
   flute
   java
   java-gcj-compat = 1.0.31
   jcommon
   jcommon-serializer
   jpackage-utils
   libfonts = 0.3.4
   libformula = 0.1.18
   libgcc_s.so.1()(64bit)
   libgcc_s.so.1(GCC_3.0)(64bit)
   libgcj_bc.so.1()(64bit)
   liblayout = 0.2.9
   librepository = 0.1.6
   libz.so.1()(64bit)
   pentaho-libxml
   sac

  pentaho-reporting-flow-engine-javadoc-0.9.3-2.fc10.x86_64.rpm
   pentaho-reporting-flow-engine-javadoc = 0.9.3-2.fc10
   pentaho-reporting-flow-engine-javadoc(x86-64) = 0.9.3-2.fc10
  =
   jpackage-utils
   pentaho-reporting-flow-engine = 0.9.3-2.fc10

* %check is not present; no test suite.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets are OK (rebuild-gcj-db).
* no pre-built jars
* single jar, named after the package
* jarfiles are under _javadir.
* javadocs are under _javadocdir.
* ant called properly.
* gcj called properly.

APPROVED, just line-wrap the javadoc description at your leisure.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461277] New: Package Review Request: radial

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

Summary: Package Review Request: radial

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

   Summary: Package Review Request: radial
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com,
[EMAIL PROTECTED]
   Estimated Hours: 0.0
Classification: Fedora


Created an attachment (id=315908)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=315908)
Source RPM of radial package.

Description of problem:
Need review of new package radial for astronomy spin.

Version-Release number of selected component (if applicable):
radial-1.0-1

Additional info:
Already reviewed at HackFest at FudCon Brno.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459148] Review Request: txt2rss - Convert from txt to rss

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


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





--- Comment #7 from Rahul Sundaram [EMAIL PROTECTED]  2008-09-05 10:44:37 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459148] Review Request: txt2rss - Convert from txt to rss

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


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





--- Comment #8 from Rakesh Pandit [EMAIL PROTECTED]  2008-09-05 10:48:57 EDT 
---
New Package CVS Request
===
Package Name: txt2rss
Short Description: Convert from txt to rss
Owners: rakesh
Branches: F-8 F-9
InitialCC: rakesh
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461277] Package Review Request: radial

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


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


Marek Mahut [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
  Alias||radial-review




--- Comment #1 from Marek Mahut [EMAIL PROTECTED]  2008-09-05 10:50:59 EDT ---
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457277] Review Request: pentaho-reporting-flow-engine - Reporting Engine

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


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





--- Comment #5 from Caolan McNamara [EMAIL PROTECTED]  2008-09-05 10:51:54 
EDT ---
New Package CVS Request
===
Package Name: pentaho-reporting-flow-engine
Short Description: A flow reporting engine
Owners: caolanm
Branches:
InitialCC:

Thanks tibbs, this is the last package in a chain which has taken nearly 6
months from the first review to the last, and I appreciate being able to see
the back of 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461131] Review Request: sim - Simple Instant Messenger

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


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





--- Comment #10 from Pavel Alexeev [EMAIL PROTECTED]  2008-09-05 11:03:17 EDT 
---
Marcela Maslanova, thank you. Before I include desktop file as regular file. I
think this is more correct way.

http://hubbitus.net.ru/rpm/Fedora9/sim/sim-0.9.5-3.SVN20080904rev2258.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461277] Package Review Request: radial

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


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


Rakesh Pandit [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #2 from Rakesh Pandit [EMAIL PROTECTED]  2008-09-05 11:02:40 EDT 
---
Changed NEW to 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457277] Review Request: pentaho-reporting-flow-engine - Reporting Engine

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


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





--- Comment #6 from Jason Tibbitts [EMAIL PROTECTED]  2008-09-05 11:08:33 EDT 
---
Sorry it took so long.  Once the java guidelines were done, I had intended to
run through all of them but I stopped at the point where others had assigned
themselves to the tickets.  I hope that whatever you needed these packages for
doesn't miss the freeze.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 426867] Review Request: scala - Hybrid functional/object-oriented language

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


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





--- Comment #64 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-05 11:12:21 EDT 
---
(In reply to comment #63)
 Scala isn't Java, and even thought scaladoc has some similarities with javadoc
 they aren't the same, so naming the subpackage -javadoc wouldn't be
 appropriate.

Okay, then I wait for the rest issues to be 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459880] Review Request: anyremote2html - WEB interface for anyRemote

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


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





--- Comment #8 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-05 11:23:15 EDT 
---
Well,

(In reply to comment #7)
 (In reply to comment #6)
- which I could download from the URL does not coincide with
  the one in your srpm.
 Updated to coincide with each other.

Please don't do this way.
As the upstream you should not modify the tarball once formally released
without
changing the version, unless people who want to download and compile your
source by themselves
will get confused.

Please restore 0.5.2 tarball you released first and if you want to modify the
tarball itself please change the version of the tarball.

Note:
Some upstream maintainers change the tarballs in the srpms during review
requests
with_out_ releasing the tarballs formally and when the reviews are passed they
release the final tarballs as formal ones.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 246525] Review Request: libMini - A high-performance terrain rendering library

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


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





--- Comment #17 from Rick L Vinyard Jr [EMAIL PROTECTED]  2008-09-05 11:27:53 
EDT ---
I've been working with upstream to get the autotools files into the project.
There should be a new release coming up that incorporates them.

As far as the DXT code goes I don't think it will be a problem. The DXT code is
in the squish library, not the mini library. When building, mini provides an
option to build against the squish library, but we're disabling that option.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 246524] Review Request: libquikgrid - The QuikGrid generator and contouring routine library

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


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


Rick L Vinyard Jr [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|needinfo?([EMAIL PROTECTED] |
   |edu)|




--- Comment #15 from Rick L Vinyard Jr [EMAIL PROTECTED]  2008-09-05 11:30:39 
EDT ---
Still planning to work on it soon. I need to contact upstream and see if
they'll incorporate the build tools.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455989] Review Request: xfce-mcs-plugins-extra - Extra plugins for the Xfce mcs manager

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


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





--- Comment #9 from Christoph Wickert [EMAIL PROTECTED]  2008-09-05 11:32:48 
EDT ---
New Package CVS Request
===
Package Name: xfce-mcs-plugins-extra
Short Description: Extra plugins for the Xfce mcs manager
Owners: cwickert, kevin
Branches: F-8 F-9
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461131] Review Request: sim - Simple Instant Messenger

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


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





--- Comment #12 from Patrice Dumas [EMAIL PROTECTED]  2008-09-05 11:40:59 EDT 
---
Sorry, it should better be
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#PreReleasePackages

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461131] Review Request: sim - Simple Instant Messenger

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


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





--- Comment #11 from Patrice Dumas [EMAIL PROTECTED]  2008-09-05 11:40:08 EDT 
---
The %release seems, at first glance to be incorrect, please see:
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Snapshot_packages

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460912] Review Request: DeviceKit-power - Power Management Service

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


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


Matthias Clasen [EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #3 from Matthias Clasen [EMAIL PROTECTED]  2008-09-05 12:06:39 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460912] Review Request: DeviceKit-power - Power Management Service

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


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





--- Comment #4 from Matthias Clasen [EMAIL PROTECTED]  2008-09-05 12:10:15 
EDT ---
Builds fine in mock.
rpmlint says:

[EMAIL PROTECTED] ~]$ rpmlint
/var/lib/mock/fedora-rawhide-i386/result/DeviceKit-power-*.rpm
DeviceKit-power.i386: W: non-conffile-in-etc
/etc/udev/rules.d/95-devkit-power-csr.rules
DeviceKit-power.i386: W: non-conffile-in-etc
/etc/dbus-1/system.d/org.freedesktop.DeviceKit.Power.conf
DeviceKit-power.i386: W: non-conffile-in-etc
/etc/udev/rules.d/95-devkit-power-wup.rules
DeviceKit-power.i386: W: non-conffile-in-etc
/etc/udev/rules.d/95-devkit-power-hid.rules
4 packages and 0 specfiles checked; 0 errors, 4 warnings.

All ignorable warnings

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 199154] Review Request: Slony-1 (postgresql-slony-engine)

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


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





--- Comment #36 from Devrim GUNDUZ [EMAIL PROTECTED]  2008-09-05 12:17:48 EDT 
---
It is more or less fixed in current code, I believe.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458548] Review Request: uriparser - URI parsing library - RFC 3986

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


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


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #13 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-05 12:16:15 EDT 
---
For 0.7.1-4:

* Group
  - Usually Group tag for the main package of this type
is System Environment/Libraries (and -devel subpackage
has Development/Libraries)

! %setup
  - -n %{name}-%{version} is redundant as the default of
%setup directory is this.

* autotools call

   419  + cd doc
..
   443  config.status: creating Makefile
   444  + make -j8
   445  cd .  /bin/sh /builddir/build/BUILD/uriparser-0.7.1/missing --run
aclocal-1.10 
   446  /builddir/build/BUILD/uriparser-0.7.1/missing: line 54: aclocal-1.10:
command not found
   447  WARNING: `aclocal-1.10' is missing on your system.  You should only
need it if
   448   you modified `acinclude.m4' or `configure.in'.  You might want
   449   to install the `Automake' and `Perl' packages.  Grab them from
   450   any GNU archive site.
   451  cd .  /bin/sh /builddir/build/BUILD/uriparser-0.7.1/missing --run
autoconf
..

   - Automated autotool calls after configure is not desired. Usually
 timestamps on some files (aclocal.m4 and so on) are wrong.
 You can avoid this by touching some files.

* Timestamp
  - Please consider to use

make install DESTDIR=$RPM_BUILD_ROOT INSTALL=install -p

to keep timestamps on installed files as much as possible.
This method usually works for recent autotool based Makefiles.

* Documents install

# doc folder make install does nothing
install -d $RPM_BUILD_ROOT%{_docdir}/%{name}-%{version}/html/
install -p -m 0644 doc/html/*
$RPM_BUILD_ROOT%{_docdir}/%{name}-%{version}/html/

  - Well, this does not do anything (and can be removed completely) ... 
Actually in -devel package the directory 
%{_docdir}/%{name}-%{version}/html/ does not exist.

This is because of %doc invocation. As build.log shows
what %doc actually does (when the list in %doc does not 
begin with /) is:

   663  Processing files: uriparser-0.7.1-4.fc10
   664  Executing(%doc): /bin/sh -e /var/tmp/rpm-tmp.hz0MmE
   665  + umask 022
   666  + cd /builddir/build/BUILD
   667  + cd uriparser-0.7.1
   668  +
DOCDIR=/builddir/build/BUILDROOT/uriparser-0.7.1-4.fc10.i386/usr/share/doc/uriparser-0.7.1
   669  + export DOCDIR
   670  + rm -rf
/builddir/build/BUILDROOT/uriparser-0.7.1-4.fc10.i386/usr/share/doc/uriparser-0.7.1
   671  + /bin/mkdir -p
/builddir/build/BUILDROOT/uriparser-0.7.1-4.fc10.i386/usr/share/doc/uriparser-0.7.1
   672  + cp -pr THANKS AUTHORS COPYING
/builddir/build/BUILDROOT/uriparser-0.7.1-4.fc10.i386/usr/share/doc/uripars
er-0.7.1
   673  + exit 0

See the line 670. %doc once removes all files under 
$RPM_BUILD_ROOT%{_docdir}/%{name}-%{version}/.

* Documents
  - Please consider to add the following file to %doc.

ChangeLog


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461131] Review Request: sim - Simple Instant Messenger

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


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





--- Comment #13 from Pavel Alexeev [EMAIL PROTECTED]  2008-09-05 12:26:40 EDT 
---
I'm understand what Release should start from 0. and followed by spec
releases, but not completely understand about svn/cvs information in release:

It is meant what version should be:
sim-0.9.5-0.3.20080904svn ?
Or I can add revision number like this:
sim-0.9.5-0.3.20080904svn2258rev ?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461131] Review Request: sim - Simple Instant Messenger

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


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





--- Comment #14 from Patrice Dumas [EMAIL PROTECTED]  2008-09-05 12:35:57 EDT 
---
both are correct. There is a limit in the number of character, if
I remember well, however, to be verified.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459148] Review Request: txt2rss - Convert from txt to rss

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


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


Kevin Fenzi [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #9 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:38:56 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 456298] Review Request: netbeans-resolver - Resolver subproject of xml-commons patched for NetBeans

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


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


Kevin Fenzi [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #11 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:40:08 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455989] Review Request: xfce-mcs-plugins-extra - Extra plugins for the Xfce mcs manager

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


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


Kevin Fenzi [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #10 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:37:42 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 448201] Review Request: pyvnc2swf - Vnc screen recorder

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


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





--- Comment #15 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:42:39 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457690] Review Request: python-flickrapi - python module for flickr api

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


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





--- Comment #11 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:48:16 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457535] Review Request: myanmar3-unicode-fonts - Myanmar3 unicode font

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


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





--- Comment #12 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:46:52 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460244] Review Request: alt-ergo - Alt-Ergo automatic theorem prover

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


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


Kevin Fenzi [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]




--- Comment #7 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:50:28 EDT ---
You should be able to build/test this against rawhide however, right? 

I fired off a scratch build against rawhide and it failed: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=808966

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458446] Review Request: perl-HTML-FromText - Convert plain text to HTML

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


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





--- Comment #9 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:51:39 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461105] Review Request: labrea - A sticky honeypot

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


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





--- Comment #7 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:52:44 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457263] Review Request: liblayout - positioning library

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


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





--- Comment #10 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:53:40 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460732] Review Request: ocaml-reins - Library of OCaml persistent data structures

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


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





--- Comment #7 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 13:00:00 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459138] Review Request: ctemplate - A simple but powerful template language for C++

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


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





--- Comment #12 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:57:48 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457277] Review Request: pentaho-reporting-flow-engine - Reporting Engine

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


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





--- Comment #7 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 12:56:34 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461054] Review Request: qrq - Morse telegraphy trainer

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


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





--- Comment #5 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 13:01:31 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 447457] Review Request: gupnp-av - UPnP-AV is a collection of helpers for building AV upnp apps

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


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





--- Comment #10 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-05 13:02:43 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460244] Review Request: alt-ergo - Alt-Ergo automatic theorem prover

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


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





--- Comment #8 from Alan Dunn [EMAIL PROTECTED]  2008-09-05 13:09:48 EDT ---
(In reply to comment #7)
 You should be able to build/test this against rawhide however, right? 
 
 I fired off a scratch build against rawhide and it failed: 
 http://koji.fedoraproject.org/koji/taskinfo?taskID=808966

You are correct on both counts (should've realize re: rawhide for testing, but
it's not a full substitute I suppose...) - was missing prelink from
BuildRequires:

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

New SRPM and spec file at
http://www.duke.edu/~amd34/alt-ergo/

(or take from Koji)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457839] Review Request: rubygem-cobbler - Provides Ruby bindings to interact with a Cobbler server.

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


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





--- Comment #15 from Jason Tibbitts [EMAIL PROTECTED]  2008-09-05 13:26:36 
EDT ---
rpm doesn't do any kind of repacking of game.  I asked folks on IRC to download
the GEM URL above and they also saw the bizarre timestamping issue.

The package, however, is OK now.

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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 460912] Review Request: DeviceKit-power - Power Management Service

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


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





--- Comment #5 from Matthias Clasen [EMAIL PROTECTED]  2008-09-05 13:28:33 
EDT ---


package name: ok
spec file name: ok
packaging guidelines: ok
license: ok
license field: says MIT, but COPYING is GPL. What gives ?
license: file: I notice the first line:

   Copyright (C) 2008 David Zeuthen [EMAIL PROTECTED] 

   Maybe you want to add yourself there ?

spec language: ok
spec legibility: ok
upstream sources: ok
buildable: ok
excludearch: n/a
build deps: ok
locale handling: n/a
shared libs: ok
relocatable: n/a
directory ownership: ok
duplicate files: ok 
file permissions: ok
%clean: ok
macro use: consistent
content: permissable
large docs: n/a
header files: n/a
static libs: n/a
pc files: n/a
versioned shared libs: n/a
devel-base dep: ok
libtool archives: ok
gui apps: n/a
directory ownership: must not own
  /usr/share/PolicyKit/policy (owned by PolicyKit)
  /usr/share/dbus-1/interfaces (owned by dbus)
%install: ok
utf8 filenames: ok


In summary: 
 - fix the license situation
 - fix directory ownership

then its good to go

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461139] Review Request: arabeyes-core-fonts - Core Arabic fonts from Arabeyes.org

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


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





--- Comment #2 from Nicolas Mailhot [EMAIL PROTECTED]  2008-09-05 13:40:16 
EDT ---
Hi Subhodip,

I'll try to do a full review this week-end.

In the meanwhile:
1. Please split your package in two, since you have two different fonts from
two different sources. This will provide more user flexibility

2. Please follow the workflow described on
http://fedoraproject.org/wiki/Font_package_lifecycle
and in particular create a wiki page per font packages that can be referenced
in release notes and other documentation

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 456341] Review Request: netbeans-platform8 - NetBeans 6.1 Platform 8

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


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


Lillian Angel [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag|needinfo?([EMAIL PROTECTED] |
   |ect.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 456298] Review Request: netbeans-resolver - Resolver subproject of xml-commons patched for NetBeans

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


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


Victor G. Vasilyev [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #12 from Victor G. Vasilyev [EMAIL PROTECTED]  2008-09-05 
13:42:34 EDT ---
Successful koji build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=809078

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 456337] Review Request: NetBeans 6.1 - Integrated Development Environment (IDE)

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


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


Bug 456337 depends on bug 456298, which changed state.

Bug 456298 Summary: Review Request:  netbeans-resolver - Resolver subproject of 
xml-commons patched for NetBeans
https://bugzilla.redhat.com/show_bug.cgi?id=456298

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457839] Review Request: rubygem-cobbler - Provides Ruby bindings to interact with a Cobbler server.

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


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





--- Comment #16 from Darryl L. Pierce [EMAIL PROTECTED]  2008-09-05 13:48:45 
EDT ---
New Package CVS Request
===
Package Name: rubygem-cobbler
Short Description: Provides Ruby bindings to interact with a Cobbler server.
Owners: mcpierce
Branches: F-9 F-10
InitialCC: mcpierce

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 459908] Review Request: freedink - Adventure and role-playing game (engine)

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


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


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #4 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-05 13:51:47 EDT 
---
Well,
* Same with bug 459916, the license of src/freedink_xpm.c is unclear.
* This package installs LiberationSans-Regular.ttf. However this font is
already
  provided by other Fedora package and shipping duplicate fonts like this way
  is not allowed.
  Please modify the source so that this package uses system widely provided
  fonts.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 456341] Review Request: netbeans-platform8 - NetBeans 6.1 Platform 8

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


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


Lillian Angel [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||needinfo?([EMAIL PROTECTED]
   ||un.com)




--- Comment #11 from Lillian Angel [EMAIL PROTECTED]  2008-09-05 13:58:00 EDT 
---
Rpmlint:

$ rpmlint  /notnfs/langel/rpm/RPMS/noarch/netbeans-platform8-*
netbeans-platform8.noarch: W: dangerous-command-in-%post ln
netbeans-platform8.noarch: W: dangerous-command-in-%preun rm
netbeans-platform8-harness.noarch: W: dangerous-command-in-%post ln
netbeans-platform8-harness.noarch: W: dangerous-command-in-%preun rm

Can the file linking be moved to %build or %install? It is also possibly these
create files that end up being unowned by the package.

The rm calls are ok.


These are ok:
netbeans-platform8.noarch: W: class-path-in-manifest
/usr/share/netbeans/platform8/modules/org-netbeans-modules-javahelp.jar
netbeans-platform8.noarch: W: class-path-in-manifest
/usr/share/netbeans/platform8/modules/org-netbeans-modules-autoupdate-services.jar
netbeans-platform8.noarch: W: class-path-in-manifest
/usr/share/netbeans/platform8/modules/org-netbeans-libs-jsr223.jar
netbeans-platform8.noarch: W: class-path-in-manifest
/usr/share/netbeans/platform8/modules/org-netbeans-libs-jna.jar
netbeans-platform8.noarch: W: class-path-in-manifest
/usr/share/netbeans/platform8/modules/org-jdesktop-layout.jar

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



http://fedoraproject.org/wiki/Packaging/Guidelines

1 Packaging Guidelines

* 1.1 Naming
ok
* 1.2 Legal
ok
* 1.4 Writing a package from scratch
ok
* 1.5 Modifying an existing package
ok
* 1.6 Filesystem Layout
o 1.6.1 Libexecdir
* 1.7 Use rpmlint
o 1.7.1 Rpmlint Errors
* 1.8 Changelogs
* 1.9 Tags
ok
* 1.10 BuildRoot tag
ok
* 1.11 Requires
ok
* 1.13 Summary and description
ok
* 1.14 Encoding
ok
* 1.15 Documentation
ok
* 1.16 Compiler flags
ok
* 1.17 Debuginfo packages
n/a
* 1.18 Exclusion of Static Libraries
n/a
* 1.19 Duplication of system libraries
n/a
* 1.20 Beware of Rpath
n/a
* 1.21 Configuration files
ok
* 1.22 Initscripts
ok
* 1.23 Desktop files
ok
* 1.24 Macros
ok
* 1.25 Handling Locale Files
n/a
* 1.26 Timestamps
n/a
* 1.27 Parallel make
n/a
* 1.28 Scriptlets requirements
n/a
* 1.29 Running scriptlets only in certain situations
n/a
* 1.30 Scriplets are only allowed to write in certain directories
n/a
* 1.31 Conditional dependencies
n/a
* 1.32 Build packages with separate user accounts
n/a
* 1.33 Relocatable packages
n/a
* 1.34 Code Vs Content
ok
* 1.35 File and Directory Ownership
Fix permission. Listed below.
* 1.36 Users and Groups
ok
* 1.37 Web Applications
ok
* 1.38 Conflicts
n/a
* 1.39 No External Kernel Modules
n/a
* 1.40 No Files or Directories under /srv
n/a
* 1.41 Application Specific Guidelines
n/a


http://fedoraproject.org/wiki/Packaging/ReviewGuidelines

MUST Items:

 MUST: rpmlint must be run on every package. The output should be posted in
the review.
See comments above

- MUST: The package must be named according to the Package Naming Guidelines .
ok
- MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption on Package Naming Guidelines
.
ok
- MUST: The package must meet the Packaging Guidelines .
ok
- MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
ok
- MUST: The License field in the package spec file must match the actual
license.
ok
- MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.
ok
- MUST: The spec file must be written in American English.
ok
- MUST: The spec file for the package MUST be legible. If the reviewer is
unable to read the spec file, it will be impossible to perform a review. Fedora
is not the place for entries into the Obfuscated Code Contest
(http://www.ioccc.org/).
ok
- MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
ok
- MUST: The package must successfully compile and build into binary rpms on at
least one supported architecture.
ok
- MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. 
ok
- MUST: 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 

[Bug 461303] New: Review Request: perl-Data-Section - Read multiple hunks of data out of your DATA section

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

Summary: Review Request: perl-Data-Section - Read multiple hunks of data out of 
your DATA section

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

   Summary: Review Request: perl-Data-Section - Read multiple
hunks of data out of your DATA section
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://berrange.fedorapeople.org/review/perl-Data-Section/perl-Data-Section.spec
SRPM URL:
http://berrange.fedorapeople.org/review/perl-Data-Section/perl-Data-Section-0.005-1.fc9.src.rpm
Description: 
Data::Section provides an easy way to access multiple named chunks of line-
oriented data in your module's DATA section. It was written to allow
modules to store their own templates, but probably has other uses.



This new package is a requirement to upgrade perl-Module-CPANTS-Analyse to
latest upstream release

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461305] New: Review Request: pam_kcoda - coda auth/cred/session PAM module

2008-09-05 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: pam_kcoda - coda auth/cred/session PAM module

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

   Summary: Review Request: pam_kcoda - coda auth/cred/session PAM
module
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://nhorman.fedorapeople.org/pam_kcoda.spec
SRPM URL: http://nhorman.fedorapeople.org/pam_kcoda-0.4-1.fc8.src.rpm
Description: This is a PAM module that allows a user to obtain coda tokens when
authenticated against a krb5 realm.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


  1   2   >