[Bug 187317] Review Request: mindi

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

Summary: Review Request: mindi


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 03:19 EST ---
ping ?

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

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


[Bug 230560] Review Request: wqy-bitmapfont - a fine-tuned CJK bitmap font

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

Summary: Review Request: wqy-bitmapfont - a fine-tuned CJK bitmap font


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 03:25 EST ---
I have reviewed this package on FC6. Everything seems fine.

Maybe need to change to this:
Release: 6%{?dist}

I think it is good to make this rpm even work on old linux version which haven't
chkfontpath.

But the old Requires is wrong, should be:
Requires: freetype, %{_datadir}/fonts
Prereq: chkfontpath
Not chkfontconfig, as this package don't exist in FC6. Name changed?

The rpmlint output was this:
W: wqy-bitmapfont dangerous-command-in-%post cp
E: wqy-bitmapfont no-prereq-on chkfontpath
W: wqy-bitmapfont dangerous-command-in-%preun rm
E: wqy-bitmapfont no-prereq-on chkfontpath

The no-prereq-on will disappear after add Prereq: chkfontpath.

Maybe you can look at bitmap-fonts package's spec file? It is for pcf files too.


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

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


[Bug 233522] Review Request: nginx - http and reverse proxy server

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

Summary: Review Request: nginx -   http and reverse proxy server


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 03:45 EST ---
Hi Jeremy,

- zlib-devel is already required by openssl-devel
- You can even drop the whole Requires line. rpm is smart enough to calculate 
that by itself.
- Replace /usr/sbin/nginx in your %install section with %{_sbindir}/nginx
- You're missing a forward slash in 
--lock-path=%{_localstatedir}lock/subsys/%{name}

Nothing that should block this review, but I hope it helps.


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

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


[Bug 228450] Review Request: zhcon - A Fast Console CJK System Using FrameBuffer

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

Summary: Review Request: zhcon - A Fast Console CJK System Using FrameBuffer


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 03:57 EST ---
Hello, Zhu.

This bug still block FE-NEEDSPONSOR, however according
to your comment, are you already sponsored??

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

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


[Bug 198816] Review Request: gfs-utils - global file system userland utilities

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

Summary: Review Request: gfs-utils - global file system userland utilities


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 04:07 EST ---
Hi Chris,

Please respond, or I'll have to close this ticket in a week.

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

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


[Bug 209511] Review Request: gnbd - global network block device

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

Summary: Review Request: gnbd - global network block device


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 04:08 EST ---
Hi Chris,

Please respond, or I'll have to close this ticket in a week.

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

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


[Bug 221906] Review Request: gmediaserver - UPnP compatible media server for the GNU system

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

Summary: Review Request: gmediaserver - UPnP compatible media server for the 
GNU system
Alias: gmediaserver

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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 04:10 EST ---
(In reply to comment #8)
  * %define mediadir /var/gmediaserver
please read http://www.pathname.com/fhs/pub/fhs-2.3.html and determine the
proper location... perhaps /var/lib/gmediaserver?
 Change to /srv/gmediaserver (consulted on IRC - #fedora-devel)

Seems reasonable to me.

Init file still needs some work.  Here is the relevent rpmlint output:

E: gmediaserver wrong-line-in-lsb-tag # gmediaserver  Start/stop gmediaserver
service
E: gmediaserver wrong-line-in-lsb-tag #
E: gmediaserver unknow-lsb-tag # chkconfig: - 85 15
E: gmediaserver unknow-lsb-tag # description:   GMediaServer is a UPnP
compatible media server for the GNU system. It is part \
E: gmediaserver wrong-line-in-lsb-tag #
E: gmediaserver unknow-lsb-tag # processname: gmediaserver
E: gmediaserver unknow-lsb-tag # config: /etc/sysconfig/gmediaserver
E: gmediaserver unknow-lsb-tag # pidfile: /var/run/gmediaserver.pid
E: gmediaserver missing-mandatory-lsb-tag Provides
E: gmediaserver missing-mandatory-lsb-tag Description
E: gmediaserver missing-mandatory-lsb-tag Short-Description

And here is an example init with the LSB info:

#!/bin/bash
#
# Startup script for the ntop program

# chkconfig: - 81 19
# description: A network traffic probe similar to the UNIX top command
# processname: ntop
# pidfile: /var/run/ntop.pid
# config: /etc/ntop.conf
#
### BEGIN INIT INFO
# Provides:  ntop
# Required-Start:$local_fs $network $syslog
# Should-Start:
# Required-Stop:
# Default-Start: 3 4 5
# Default-Stop:  0 1 2 6
# Short-Description: Start ntop daemon
# Description:   Start ntop, a network traffic probe similar to the UNIX \
#top command
### END INIT INFO



E: gmediaserver executable-marked-as-config-file /etc/sysconfig/gmediaserver
E: gmediaserver script-without-shebang /etc/sysconfig/gmediaserver

this should be installed mode 644

E: gmediaserver executable-marked-as-config-file /etc/logrotate.d/gmediaserver
W: gmediaserver spurious-executable-perm /etc/logrotate.d/gmediaserver

this should be installed mode 644

 install -D -m755 %{SOURCE1} $RPM_BUILD_ROOT%{_initrddir}/%{name}
 cp %{SOURCE1} $RPM_BUILD_ROOT%{_initrddir}/%{name}
 chmod +x $RPM_BUILD_ROOT%{_initrddir}/%{name}

the install then cp is redundant.
use -p to preserve the timestamp.  If you want, you can use 'cp -p' or more
efficiently:
install -D -m 0755 -p %{SOURCE} $RPM_BUILD_ROOT%{_initrddir}/%{name}

 install -D $RPM_BUILD_ROOT%{_bindir}/%{name} 
$RPM_BUILD_ROOT%{_sbindir}/%{name}
 rm $RPM_BUILD_ROOT%{_bindir}/%{name}

just use 'mv' - it will keep the timestamp, mode, ownership, etc of the original
file
you might want a comment here too, explaining that the make install put the file
in /usr/bin but for a server it should be /usr/sbin/(In reply to comment #8)


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

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


[Bug 232752] Review Request: g3data - Program for extracting the data from scanned graphs

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

Summary: Review Request: g3data - Program for extracting the data from scanned 
graphs


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 04:14 EST ---
done

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

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


[Bug 228450] Review Request: zhcon - A Fast Console CJK System Using FrameBuffer

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

Summary: Review Request: zhcon - A Fast Console CJK System Using FrameBuffer


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 04:33 EST ---
I have already get sponsored. I have reviewed the wqy-bitmapfont package too.
Have the review get past? If so, we can add the fedora-cvs tag.

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

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


[Bug 228450] Review Request: zhcon - A Fast Console CJK System Using FrameBuffer

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

Summary: Review Request: zhcon - A Fast Console CJK System Using FrameBuffer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778, 177841  |
  nThis||
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 04:41 EST ---
The okay.

---
This package (zhcon) is APPROVED by me.
---

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

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


[Bug 228450] Review Request: zhcon - A Fast Console CJK System Using FrameBuffer

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

Summary: Review Request: zhcon - A Fast Console CJK System Using FrameBuffer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 225984] Merge Review: lftp

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

Summary: Merge Review: lftp


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
   Flag|fedora-review-  |fedora-review?, needinfo?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 04:55 EST ---
mbarabas, can you please create a new spec based on the comments above?

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

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


[Bug 230560] Review Request: wqy-bitmapfont - a fine-tuned CJK bitmap font

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

Summary: Review Request: wqy-bitmapfont - a fine-tuned CJK bitmap font


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 05:01 EST ---
Well, the newest spec file is what I can see on
http://wenq.org/release/08src/wqy-bitmapfont.spec , right?

Then:
* At least this spec file cannot be accepted as this
  rpm creates some non-owned files. All files created or
  installed by this rpm should be owned by this rpm.
  i.e. Generally using cp or ln rm on scriptlets are 
   generally prohibited. This means that file ownership
   is somewhat wrong.

  If you have some strong reason why you want to support
  older versions, then IMO you should introduce %if macro, for example,
  and split the case accordingly.

* /usr/X11R6/ directory is no longer used.
  mkfontdir is now under %{_bindir}.

* Please use proper macros (i.e. %{_bindir} %{_sysconfdir},...)

* Please check what rpm owns %{_datadir}/fonts/wenquanyi/ .


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

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


[Bug 231707] Review Request: perl-CGI-Session - Persistent session data in CGI applications

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

Summary: Review Request: perl-CGI-Session - Persistent session data in CGI 
applications
Alias: perl-CGI-Session

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 05:38 EST ---
Thanks for the review, package has been built for FC-5, FC-6 and rawhide.

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

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


[Bug 232612] Review Request: perl-Class-Observable - Allow other classes and objects to respond to events in yours

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

Summary: Review Request: perl-Class-Observable - Allow other classes and 
objects to respond to events in yours


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 05:40 EST ---
Thanks for the review, package has been built for FC-5, FC-6 and rawhide.

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

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


[Bug 231758] Review Request: perl-Workflow - Simple, flexible system to implement workflows

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

Summary: Review Request: perl-Workflow - Simple, flexible system to implement 
workflows
Alias: perl-Workflow

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


Bug 231758 depends on bug 232612, which changed state.

Bug 232612 Summary: Review Request: perl-Class-Observable - Allow other classes 
and objects to respond to events in yours
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=232612

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 231736] Review Request: perl-FileHandle-Fmode - FileHandle::Fmode Perl module

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

Summary: Review Request: perl-FileHandle-Fmode - FileHandle::Fmode Perl module


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 05:42 EST ---
New Package CVS Request
===
Package Name: perl-FileHandle-Fmode
Short Description: FileHandle::Fmode Perl module
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com

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

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


[Bug 232615] Review Request: perl-Mail-RFC822-Address - Perl extension for validating email addresses according to RFC822

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

Summary: Review Request: perl-Mail-RFC822-Address - Perl extension for 
validating email addresses according to RFC822


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 05:58 EST ---
New Package CVS Request
===
Package Name: perl-Mail-RFC822-Address
Short Description: Perl extension for validating addresses according to RFC822
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com

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

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


[Bug 191036] Review Request: libmp4v2 a library for handling the mp4 container format

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

Summary: Review Request: libmp4v2 a library for handling the mp4 container 
format


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:06 EST ---
Simply because I missed it. Since I'm not the reported or the assignee, it
doesn't appear in my bugzilla front page...

Closing now at last :-)

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

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


[Bug 232626] Review Request: perl-MD5 - Perl interface to the MD5 Message-Digest Algorithm

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

Summary: Review Request: perl-MD5 - Perl interface to the MD5 Message-Digest 
Algorithm


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:06 EST ---
New Package CVS Request
===
Package Name: perl-MD5
Short Description: Perl interface to the MD5 Message-Digest Algorithm
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com



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

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


[Bug 232617] Review Request: perl-Sys-SigAction - Perl extension for Consistent Signal Handling

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

Summary: Review Request: perl-Sys-SigAction - Perl extension for Consistent 
Signal Handling
Alias: perl-Sys-SigAction

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:07 EST ---
New Package CVS Request
===
Package Name: perl-Sys-SigAction
Short Description: Read extension for Consistent Signal Handling
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com



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

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


[Bug 232613] Review Request: perl-Data-Password - Perl extension for assesing password quality

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

Summary: Review Request: perl-Data-Password - Perl extension for assesing 
password quality


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:11 EST ---
Thanks for the review. Built for 5, 6 and rawhide.

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

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


[Bug 233352] Review Request: funtools - FITS library and utilities

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

Summary: Review Request: funtools - FITS library and utilities


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:16 EST ---
Yes, the problem is there. I have tested the srpm with multiprocessor machines
and I get the same failed build.

This is the rpm with the parallel make flag removed

http://t-rex.fis.ucm.es/~spr/funtools.spec
http://t-rex.fis.ucm.es/~spr/funtools-1.3.0-0.3.b29.fc6.src.rpm

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

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


[Bug 232618] Review Request: perl-XML-Filter-BufferText - Filter to put all characters() in one event

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

Summary: Review Request: perl-XML-Filter-BufferText - Filter to put all 
characters() in one event


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:18 EST ---
Thanks for the review. Package has been built for FC-5, FC-6 and rawhide.

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

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


[Bug 232619] Review Request: perl-XML-SAX-Writer - SAX2 Writer

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

Summary: Review Request: perl-XML-SAX-Writer - SAX2 Writer


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


Bug 232619 depends on bug 232618, which changed state.

Bug 232618 Summary: Review Request: perl-XML-Filter-BufferText - Filter to put 
all characters() in one event
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=232618

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 232622] Review Request: perl-XML-Validator-Schema - Validate XML against a subset of W3C XML Schema

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

Summary: Review Request: perl-XML-Validator-Schema - Validate XML against a 
subset of W3C XML Schema


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


Bug 232622 depends on bug 232618, which changed state.

Bug 232618 Summary: Review Request: perl-XML-Filter-BufferText - Filter to put 
all characters() in one event
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=232618

   What|Old Value   |New Value

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



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

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


[Bug 232619] Review Request: perl-XML-SAX-Writer - SAX2 Writer

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

Summary: Review Request: perl-XML-SAX-Writer - SAX2 Writer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:18 EST ---
Thanks for the review. Package has been built for FC-5, FC-6 and rawhide.

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

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


[Bug 232620] Review Request: perl-XML-Filter-XInclude - XInclude as a SAX Filter

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

Summary: Review Request: perl-XML-Filter-XInclude - XInclude as a SAX Filter


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


Bug 232620 depends on bug 232619, which changed state.

Bug 232619 Summary: Review Request: perl-XML-SAX-Writer - SAX2 Writer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=232619

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 232624] Review Request: perl-ConfigReader - Read directives from a configuration file

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

Summary: Review Request: perl-ConfigReader - Read directives from a 
configuration file


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:21 EST ---
Thanks for the review. Package has been built for FC-5, FC-6 and rawhide.

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

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


[Bug 232616] Review Request: perl-Proc-ProcessTable - Perl extension to access the unix process table

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

Summary: Review Request: perl-Proc-ProcessTable - Perl extension to access the 
unix process table


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:21 EST ---
Thanks for the review. Built for 5, 6 and rawhide.

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

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


[Bug 232625] Review Request: perl-LockFile-Simple - Simple file locking scheme

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

Summary: Review Request: perl-LockFile-Simple - Simple file locking scheme


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:24 EST ---
Thanks for the review. Package has been built for FC-5, FC-6 and rawhide.

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

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


[Bug 225695] Merge Review: diffstat

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

Summary: Merge Review: diffstat


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:25 EST ---
I've removed that sentence.  I haven't added a 'Requires' tag for diffutils
because there is no dependency there -- it is entirely possible that you might
want to run diffstat on a patch you've received by email without having
diffutils installed, and that works just fine.

Tagged and built as 1.43-5.fc7.

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

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


[Bug 225693] Merge Review: dialog

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

Summary: Merge Review: dialog


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:31 EST ---
 It should be Release: 0.1.%{SubVersion}svn%{?dist}

From http://fedoraproject.org/wiki/Packaging/NamingGuidelines:

Release Tag for Post-Release Snapshot Packages:

*

  %{X}.%{alphatag} 

Where %{X} is the build number from any previous stable package build,
incremented by one (if no previous stable package build, use 1), and %{alphatag}
is the checkout string, as described above.

Example (post-release cvs):

 kismet-1.0-1 (this is the formal release of kismet 1.0)
 kismet-1.0-2 (this is a bugfix build to the 1.0 release)
 kismet-1.0-3.20050515cvs (move to a post-release cvs checkout)
 kismet-1.0-4.20050515cvs (bugfix to the post-release cvs checkout)
 kismet-1.0-5.20050517cvs (new cvs checkout, note the increment of %{X})

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

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


[Bug 228450] Review Request: zhcon - A Fast Console CJK System Using FrameBuffer

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

Summary: Review Request: zhcon - A Fast Console CJK System Using FrameBuffer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs-




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:46 EST ---
Please use the CVS request template found here:

http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure

And reset the fedora-cvs flag with an entry using that.

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

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


[Bug 231736] Review Request: perl-FileHandle-Fmode - FileHandle::Fmode Perl module

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

Summary: Review Request: perl-FileHandle-Fmode - FileHandle::Fmode Perl module


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 232193] Review Request: perl-SVG - Generate stand-alone or inline SVG

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

Summary: Review Request: perl-SVG - Generate stand-alone or inline SVG


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:51 EST ---
Updated as per comment #2
spec: http://allele5.biol.berkeley.edu/~alex/fedora/perl-SVG.spec
SRPM: http://allele5.biol.berkeley.edu/~alex/fedora/perl-SVG-2.33-2.src.rpm

I also added the disttag.

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

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


[Bug 232615] Review Request: perl-Mail-RFC822-Address - Perl extension for validating email addresses according to RFC822

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

Summary: Review Request: perl-Mail-RFC822-Address - Perl extension for 
validating email addresses according to RFC822


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 226670] Merge Review: zisofs-tools

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

Summary: Merge Review: zisofs-tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |MODIFIED




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 06:53 EST ---
 release value is suitable but rather strange (why 3.2.2 and not 4, 5, 6.. ? )

This was from automatic rebuilds, as you can see by the changelog entries...

please check zisofs-tools-1.0.7-1.fc7

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

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


[Bug 232626] Review Request: perl-MD5 - Perl interface to the MD5 Message-Digest Algorithm

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

Summary: Review Request: perl-MD5 - Perl interface to the MD5 Message-Digest 
Algorithm


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 232617] Review Request: perl-Sys-SigAction - Perl extension for Consistent Signal Handling

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

Summary: Review Request: perl-Sys-SigAction - Perl extension for Consistent 
Signal Handling
Alias: perl-Sys-SigAction

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 233584] New: Review Request: perl-Text-Shellwords - A thin wrapper around the shellwords.pl package

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




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

   Summary: Review Request: perl-Text-Shellwords -  A thin wrapper
around the shellwords.pl package
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://allele5.biol.berkeley.edu/~alex/fedora/perl-Text-Shellwords.spec
SRPM URL: 
http://allele5.biol.berkeley.edu/~alex/fedora/perl-Text-Shellwords-1.08-1.src.rpm
 info here
Description: 
This is a thin wrapper around the shellwords.pl package, which comes
preinstalled with Perl.  This module imports a single subroutine,
shellwords().  The shellwords() routine parses lines of text and
returns a set of tokens using the same rules that the Unix shell does
for its command-line arguments.  Tokens are separated by whitespace,
and can be delimited by single or double quotes.  The module also
respects backslash escapes.

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

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


[Bug 232622] Review Request: perl-XML-Validator-Schema - Validate XML against a subset of W3C XML Schema

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

Summary: Review Request: perl-XML-Validator-Schema - Validate XML against a 
subset of W3C XML Schema


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 226725] Review Request: netgen - LVS netlist comparison tool

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

Summary: Review Request: netgen - LVS netlist comparison tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:06 EST ---
(In reply to comment #10)
 (In reply to comment #9)
  make install still fails for some reason, but I solved it by manually 
 creating
  %{_libdir}/%{name} in the build root before running make install in 
 the %install
  section.
 
 Can i have a look why it's failing on x86_64 arch?

The problem is simply that the Makefile in lib/ does not check for the existence
of $prefix/%{_libdir}/%{name} before attempting to copy the file. I can create
an account on my x86_64 machine if you want to check for your self, but the
problem can easily be fixed either by adding the directory to the install-dirs
rule in the top level Makefile, or by manually running mkdir -p
%{buildroot}%{_libdir}/%{name} before running make install.

On the other hand, it seems kind of strange to me to install the scripts in
libdir in the first place. Wouldn't the correct location be libexec?



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

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


[Bug 232622] Review Request: perl-XML-Validator-Schema - Validate XML against a subset of W3C XML Schema

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

Summary: Review Request: perl-XML-Validator-Schema - Validate XML against a 
subset of W3C XML Schema


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:06 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
4f9decc25d2b484df80ee4adfa912f12  XML-Validator-Schema-1.08.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test
PERL_DL_NONLAZY=1 /usr/bin/perl -MExtUtils::Command::MM -e test_harness(0,
'blib/lib', 'blib/arch') t/*.t
t/01basic...ok
t/02yaml

== Testing against XML::SAX::PurePerl ==




== Testing against XML::LibXML::SAX ==

ok
t/03types...ok
t/04model...ok
t/05xerces..skipped
all skipped: Test requires $XERCES_DOMCOUNT
t/06passthruskipped
all skipped: Test requires XML::SAX::Writer
t/07locator.ok
1/2 skipped: These tests require XML::SAX::ExpatXS
All tests successful, 2 tests and 1 subtest skipped.
Files=7, Tests=658,  4 wallclock secs ( 2.66 cusr +  0.24 csys =  2.90 CPU)

+ Provides: perl(XML::Validator::Schema) = 1.08
perl(XML::Validator::Schema::AllModelNode)
perl(XML::Validator::Schema::Attribute)
perl(XML::Validator::Schema::AttributeLibrary)
perl(XML::Validator::Schema::ChoiceModelNode)
perl(XML::Validator::Schema::ComplexTypeNode)
perl(XML::Validator::Schema::ElementLibrary)
perl(XML::Validator::Schema::ElementNode)
perl(XML::Validator::Schema::ElementRefNode)
perl(XML::Validator::Schema::Library) perl(XML::Validator::Schema::ModelNode)
perl(XML::Validator::Schema::Node) perl(XML::Validator::Schema::Parser)
perl(XML::Validator::Schema::RootNode)
perl(XML::Validator::Schema::SequenceModelNode)
perl(XML::Validator::Schema::SimpleType)
perl(XML::Validator::Schema::SimpleTypeNode)
perl(XML::Validator::Schema::TypeLibrary) perl(XML::Validator::Schema::Util)

APPROVED.


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

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


[Bug 233584] Review Request: perl-Text-Shellwords - A thin wrapper around the shellwords.pl package

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

Summary: Review Request: perl-Text-Shellwords -  A thin wrapper around the 
shellwords.pl package


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 233585] New: Review Request: perl-XML-Writer - A simple Perl module for writing XML documents

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




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

   Summary: Review Request: perl-XML-Writer - A simple Perl module
for writing XML documents
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://allele5.biol.berkeley.edu/~alex/fedora/perl-XML-Writer.spec
SRPM URL: 
http://allele5.biol.berkeley.edu/~alex/fedora/perl-XML-Writer-0.602-1.src.rpm 
here
Description: 
XML::Writer is a simple Perl module for writing XML documents: it
takes care of constructing markup and escaping data correctly, and by
default, it also performs a significant amount of well-formedness
checking on the output, to make certain (for example) that start and
end tags match, that there is exactly one document element, and that
there are not duplicate attribute names.

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

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


[Bug 232620] Review Request: perl-XML-Filter-XInclude - XInclude as a SAX Filter

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

Summary: Review Request: perl-XML-Filter-XInclude - XInclude as a SAX Filter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 232620] Review Request: perl-XML-Filter-XInclude - XInclude as a SAX Filter

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

Summary: Review Request: perl-XML-Filter-XInclude - XInclude as a SAX Filter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:12 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
6fd41f32027f3dbf6993fcef0ee81578  XML-Filter-XInclude-1.0.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test
PERL_DL_NONLAZY=1 /usr/bin/perl -MExtUtils::Command::MM -e test_harness(0,
'blib/lib', 'blib/arch') t/*.t
t/00basic...ok
t/01xincludeok
All tests successful.
Files=2, Tests=3,  1 wallclock secs ( 0.20 cusr +  0.03 csys =  0.23 CPU)
+ Provides: perl(XML::Filter::XInclude) = 1.0
APPROVED.

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

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


[Bug 233585] Review Request: perl-XML-Writer - A simple Perl module for writing XML documents

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

Summary: Review Request: perl-XML-Writer - A simple Perl module for writing XML 
documents


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:19 EST ---
Kindly avoid hardcoding version number in Source URL tarball name. Instead use
%{version}
Anyway you can do that at time of importing package in CVS.

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

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


[Bug 233585] Review Request: perl-XML-Writer - A simple Perl module for writing XML documents

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

Summary: Review Request: perl-XML-Writer - A simple Perl module for writing XML 
documents


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:20 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
c715d6fd90ac775316cc313815ba3b77  XML-Writer-0.602.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test
PERL_DL_NONLAZY=1 /usr/bin/perl -MExtUtils::Command::MM -e test_harness(0,
'blib/lib', 'blib/arch') t/*.t
t/01_main.ok
t/pod-coverageskipped
all skipped: Test::Pod::Coverage required for testing pod coverage
t/pod.skipped
all skipped: Test::Pod 1.00 required for testing POD
All tests successful, 2 tests skipped.
Files=3, Tests=213,  1 wallclock secs ( 0.30 cusr +  0.03 csys =  0.33 CPU)

+ Provides: perl(XML::Writer) = 0.602 perl(XML::Writer::Namespaces)
perl(XML::Writer::_String)
APPROVED.


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

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


[Bug 231911] Review Request: jomolhari-fonts - Jomolhari a Bhutanese style font for Tibetan and Dzongkha

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

Summary: Review Request: jomolhari-fonts - Jomolhari a Bhutanese style font for 
Tibetan and Dzongkha


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:25 EST ---
Spec URL: http://manta.univ.gda.pl/~mgarski/fe/jomolhari-fonts.spec
SRPM URL: http://manta.univ.gda.pl/~mgarski/fe/jomolhari-fonts-0.003-2.src.rpm

- Extend description section

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

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


[Bug 233584] Review Request: perl-Text-Shellwords - A thin wrapper around the shellwords.pl package

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

Summary: Review Request: perl-Text-Shellwords -  A thin wrapper around the 
shellwords.pl package


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:25 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
72284605020ed5135dbed65a8d3b70c6  Text-Shellwords-1.08.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test
PERL_DL_NONLAZY=1 /usr/bin/perl -Iblib/lib -Iblib/arch test.pl
1..6
# Running under perl version 5.008008 for linux
# Current time local: Fri Mar 23 07:11:51 2007
# Current time GMT:   Fri Mar 23 11:11:51 2007
# Using Test.pm version 1.25
ok 1
ok 2
ok 3
ok 4
ok 5
ok 6
+ Provides: perl(Text::Shellwords) = 1.08
APPROVED.


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

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


[Bug 233588] New: Review Request: perl-Graph - A Perl module for dealing with graphs, the abstract data structure

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




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

   Summary: Review Request: perl-Graph - A Perl module for dealing
with graphs, the abstract data structure
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://allele5.biol.berkeley.edu/~alex/fedora/perl-Graph.spec
SRPM URL: 
http://allele5.biol.berkeley.edu/~alex/fedora/perl-Graph-0.81-1.src.rpm
Description: 
This is Graph, a Perl module for dealing with graphs, the abstract
data structures. 
 
This is a full rewrite of the Graph module 0.2xx series as discussed
in the book Mastering Algorithms with Perl, written by Jarkko
Hietaniemi (the undersigned), John Macdonald, and Jon Orwant,
and published by O'Reilly and Associates.  This rewrite is not
fully compatible with the 0.2xx series.

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

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


[Bug 232018] Review Request: perl-YAML-Syck - Fast, lightweight YAML loader and dumper

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

Summary: Review Request: perl-YAML-Syck - Fast, lightweight YAML loader and 
dumper


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 232018] Review Request: perl-YAML-Syck - Fast, lightweight YAML loader and dumper

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

Summary: Review Request: perl-YAML-Syck - Fast, lightweight YAML loader and 
dumper


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:44 EST ---
Mock build log showed
*** WARNING ***

This release breaks compatibility with versions earlier than version 0.60 of
YAML::Syck and YAML.pm when serializing blessed references.

See the COMPATIBILITY file for more information.

As current version of perl-YAML is 0.62, I assume this package will not break
any package dependency in rawhide and in FC6 also.

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

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


[Bug 232423] Review Request: perl-Class-Std - Support for creating standard inside-out classes

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

Summary: Review Request: perl-Class-Std - Support for creating standard 
inside-out classes


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 232423] Review Request: perl-Class-Std - Support for creating standard inside-out classes

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

Summary: Review Request: perl-Class-Std - Support for creating standard 
inside-out classes


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:50 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
dc01fecfc8d5886e41d2e8bc2f29a7a2  Class-Std-v0.0.8.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test is Successful.
t/00.load..# Testing Class::Std v0.0.8
ok
t/access...ok
t/access_automethodok
t/access_name..ok
t/automethod...ok
t/begin-coercion...ok
t/coercionsok
t/cumulative...ok
t/dump.ok
t/perlattrsok
t/pod-coverage.ok
t/pod..ok
t/runtime..ok
t/simple...ok
t/startok
All tests successful.
Files=15, Tests=276,  2 wallclock secs ( 0.99 cusr +  0.19 csys =  1.18 CPU)
+ Provides: perl(Class::Std) perl(Class::Std::SCR)

APPROVED.


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

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


[Bug 233588] Review Request: perl-Graph - A Perl module for dealing with graphs, the abstract data structure

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

Summary: Review Request: perl-Graph - A Perl module for dealing with graphs, 
the abstract data structure


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 233588] Review Request: perl-Graph - A Perl module for dealing with graphs, the abstract data structure

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

Summary: Review Request: perl-Graph - A Perl module for dealing with graphs, 
the abstract data structure


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 07:59 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
8400bd615c74de11644746709cbe7fe3  Graph-0.81.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test is Successful.
+ Provides: perl(Graph) = 0.81 perl(Graph::AdjacencyMap)
perl(Graph::AdjacencyMap::Heavy) perl(Graph::AdjacencyMap::Light)
perl(Graph::AdjacencyMap::Vertex) perl(Graph::AdjacencyMatrix)
perl(Graph::Attribute) perl(Graph::BitMatrix) perl(Graph::Directed)
perl(Graph::MSTHeapElem) = 0.01 perl(Graph::Matrix) perl(Graph::SPTHeapElem) =
0.01 perl(Graph::TransitiveClosure) perl(Graph::TransitiveClosure::Matrix)
perl(Graph::Traversal) perl(Graph::Traversal::BFS) perl(Graph::Traversal::DFS)
perl(Graph::Undirected) perl(Graph::UnionFind)

APPROVED.


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

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


[Bug 233585] Review Request: perl-XML-Writer - A simple Perl module for writing XML documents

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

Summary: Review Request: perl-XML-Writer - A simple Perl module for writing XML 
documents


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review+  |fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 08:11 EST ---
Pushing back to review, because of this:

1.)

t/pod-coverageskipped
all skipped: Test::Pod::Coverage required for testing pod coverage
t/pod.skipped
all skipped: Test::Pod 1.00 required for testing POD

=

BR: perl(Test::Pod::Coverage)
BR: perl(Test::Pod)


2.
BuildRequires:  perl = 1:5.6.1
= Superfluous

3. Missing
BuildRequires: perl(ExtUtils::MakeMaker)

Parag, Alex, these remarks also seem to apply to most other perl-packages Parag
nodded off today.

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

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


[Bug 232193] Review Request: perl-SVG - Generate stand-alone or inline SVG

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

Summary: Review Request: perl-SVG - Generate stand-alone or inline SVG


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 08:10 EST ---
Review:
- rpmlint is ok
- name ok
- spec name ok
- packaging guidelines met
- license ok
- license matches actual license in package
- spec file in American English and legible
- source matches upstream
- builds, installs, runs, and uninstalls cleanly
- BuildRequires ok
- no locale to worry about
- no relocation to worry about
- no duplicated files
- files perms ok
- clean section ok
- macro useage ok
- package contains code
- no large doc
- %doc is only doc
- builds in mock

This package is approved.

I'll sponsor you.

Cheers,
 C


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

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


[Bug 225695] Merge Review: diffstat

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

Summary: Merge Review: diffstat


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 08:22 EST ---
Everything looks good so this package is APPROVED.

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

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


[Bug 233596] New: Review Request: coherence - Python framework to participate in digital living networks

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




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

   Summary: Review Request: coherence - Python framework to
participate in digital living networks
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://ftp.es6.freshrpms.net/tmp/extras/coherence/coherence.spec
SRPM URL: 
http://ftp.es6.freshrpms.net/tmp/extras/coherence/coherence-0.1.0-1.src.rpm
Description:
Coherence is a framework written in Python enabling applications to participate
in digital living networks, such as the UPnP universe.

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

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


[Bug 233597] New: Review Request: pigment - Media Center Toolkit

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




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

   Summary: Review Request: pigment - Media Center Toolkit
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://ftp.es6.freshrpms.net/tmp/extras/pigment/pigment.spec
SRPM URL: 
http://ftp.es6.freshrpms.net/tmp/extras/pigment/pigment-0.1.4-1.src.rpm
Description:
Pigment is a toolkit for writing Media Center software.

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

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


[Bug 233598] New: Review Request: elisa - Media Center

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




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

   Summary: Review Request: elisa - Media Center
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://ftp.es6.freshrpms.net/tmp/extras/elisa/elisa.spec
SRPM URL: http://ftp.es6.freshrpms.net/tmp/extras/elisa/elisa-0.1.4.2-1.src.rpm
Description:
Media center solution using the GStreamer multimedia framework.

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

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


[Bug 228301] Review Request: python-nevow - Web application construction kit written in Python

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

Summary: Review Request: python-nevow - Web application construction kit 
written in Python


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||233598
  nThis||




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

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


[Bug 228299] Review Request: python-metar - Coded METAR weather reports parser for Python

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

Summary: Review Request: python-metar - Coded METAR weather reports parser for 
Python


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||233598
  nThis||




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

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


[Bug 228298] Review Request: python-louie - Dispatches signals between Python objects in a wide variety of contexts

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

Summary: Review Request: python-louie - Dispatches signals between Python 
objects in a wide variety of contexts


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||233598
  nThis||




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

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


[Bug 233596] Review Request: coherence - Python framework to participate in digital living networks

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

Summary: Review Request: coherence - Python framework to participate in digital 
living networks


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||233598
  nThis||




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

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


[Bug 233597] Review Request: pigment - Media Center Toolkit

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

Summary: Review Request: pigment - Media Center Toolkit


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||233598
  nThis||




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

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


[Bug 228303] Review Request: python-tag - Python bindings for TagLib to read and write music files tags

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

Summary: Review Request: python-tag - Python bindings for TagLib to read and 
write music files tags


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||233598
  nThis||




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

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


[Bug 233598] Review Request: elisa - Media Center

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

Summary: Review Request: elisa - Media Center


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||228298, 228299, 228301,
   ||228303, 233596, 233597




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 08:37 EST ---
Note that this package depends on a few python packages as well as the pigment
toolkit, which will need to be reviewed first (see the bug dependencies).

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

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


[Bug 226725] Review Request: netgen - LVS netlist comparison tool

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

Summary: Review Request: netgen - LVS netlist comparison tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 08:48 EST ---
Created an attachment (id=150746)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=150746action=view)
install.log

make install fails on x86_64. 

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

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


[Bug 233602] New: Review Request: pykdeextentions - A collection of python packages to support KDE applications

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




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

   Summary: Review Request: pykdeextentions - A collection of python
packages to support KDE applications
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/pykdeextentions.spec
SRPM URL:
http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.testing/pykdeextensions-0.4.0-1.src.rpm

Description:
A collection of software and Python packages to support
the creation and installation of KDE applications.

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

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


[Bug 233603] New: Review Request: guidance - System administration tools

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




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

   Summary: Review Request: guidance - System administration tools
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/guidance.spec
SRPM URL: 
http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.testing/guidance-0.7.1-1.src.rpm

Description:
Guidance is a collection of system administration tools for Linux/KDE systems.

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

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


[Bug 229490] Review Request: pypar2 - graphical frontend to par2cmdline

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

Summary: Review Request: pypar2 - graphical frontend to par2cmdline


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||laurent.rineau__fedora_extra
   ||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 08:58 EST ---
As the packager of par2cmdline, I always wanted a GUI for par2cmdline in 
Extra. gpar2 (from parchive.sf.net) was too badly written to be accepted in 
Extras (see deferred bug #190991 and bug #190992). PyPar2 will hopfully be a 
solution.

I will be quite busy in my job, until summer. I offered Maxime to co-maintain 
par2cmdline. Seems that he accepted (actually par2cmdline required no 
intervention in the past year but the FC6 mass-rebuild). I will try to find 
out in the Wiki what is the procedure to make somebody co-maintain a 
package...

Congratulations, Maxime!


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

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


[Bug 226215] Merge Review: openobex

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

Summary: Merge Review: openobex


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |MODIFIED




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:07 EST ---
right

please check openobex-1.3-5.fc7

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

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


[Bug 233603] Review Request: guidance - System administration tools

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

Summary: Review Request: guidance - System administration tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||233602




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:10 EST ---

Depends on pykdeextentions (see bug #233602),

Not very polished packaging-wise yet,
But wanted to get this out there for feedback.


FIXME/TODO items: 

1. guidance-power-manager wants a 'notify' pkg that I haven't found yet (for
fedora):

$/usr/share/apps/guidance/guidance-power-manager.py
Traceback (most recent call last):
  File /usr/share/apps/guidance/guidance-power-manager.py, line 47, in ?
from notify import *
ImportError: No module named notify

2. included rpath patch doesn't work as-is, not sure if it's worth bothering
with to fix.

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

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


[Bug 233602] Review Request: pykdeextentions - A collection of python packages to support KDE applications

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

Summary: Review Request: pykdeextentions - A collection of python packages to 
support KDE applications


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||233603
  nThis||




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

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


[Bug 226453] Merge Review: system-config-boot

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

Summary: Merge Review: system-config-boot


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 08:36 EST ---
 1. Minor: might include a copy of the GPL.
done

 2. Since redhat/fedora is upstream for this, can you make a note in the spec
 as suggested in:

http://www.fedoraproject.org/wiki/Packaging/SourceURL#head-413e1c297803cfa9de0cc4c56f3ac384bff5dc9e

done

 3. I assume the reason it only builds on ix86/x86_64 is that it only 
 understands
 lilo/grub? Might be worth filing a bug and noting it in the spec and see if
 some ppc folk are interested in contributing yaboot support.

what about sparc, s390 and the others?

 4. Please use one of the preferred buildroots, such as:
  %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

done

 5. Do not use %makeinstall. See:
 http://www.fedoraproject.org/wiki/PackagingDrafts/MakeInstall

done

 6. The desktop file is missing a valid Main Category, see:
 http://standards.freedesktop.org/menu-spec/latest/apa.html
 Suggest: System or Settings be added.
 Without this, this tool shows up under a Other menu in Xfce.

Categories=System;Application;SystemSetup;X-Red-Hat-Base;

 7. The buildrequires are not all needed, suggest changing:
 BuildRequires: python = 0:2.2, perl, gettext, glibc-devel, gcc,
 desktop-file-utils, yelp, perl-XML-Parser

done

 8. Shouldn't the firstboot package own
 %dir /usr/share/firstboot/
 %dir /usr/share/firstboot/modules
 and not this package?

done

 9. 2 outstanding bugs. Might look if either can be resolved easily.

bug #134548 and bug #181749 are not easily fixable

 10. rpmlint says:

 a) E: system-config-boot no-binary
 I assume this is not noarch since it can be only run on ix86/x86_64?

yep

 b)
 W: system-config-boot conffile-without-noreplace-flag 
 /etc/pam.d/system-config-boot
 W: system-config-boot conffile-without-noreplace-flag
 /etc/security/console.apps/system-config-boot
 Suggest: should those be (noreplace)?

done

 c) W: system-config-boot no-documentation 
 No docs available?

no .)

 d)
 Suggest: remove the #!/usr/bin/python from those.

done

 e) W: system-config-boot unversioned-explicit-obsoletes redhat-config-boot
 Suggest: add a version here? or just remove it at this point?

added version

 f)
 W: system-config-boot rpm-buildroot-usage %prep rm -rf $RPM_BUILD_ROOT
 E: system-config-boot no-cleaning-of-buildroot %install
 Suggest: Move the rm from prep to the top of install?

done

 g)
 W: system-config-boot macro-in-%changelog dist
 W: system-config-boot macro-in-%changelog dist
 
 Suggest: Change occurances of %dist in the changelog with %%dist

done

 h) E: system-config-boot-debuginfo empty-debuginfo-package
 I guess you need to add
  %define debug_package %{nil}
 if this really has to be an arch package.

done

please check system-config-boot-0.2.15-1.fc7


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

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


[Bug 220635] Review Request: livecd-tools - Tools for creating Live CD's

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

Summary: Review Request: livecd-tools - Tools for creating Live CD's


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:20 EST ---
Good stuff (still), APPROVED.

When/if you get round-tuit:

* maybe(?)/SHOULD include a better URL, like
URL:  http://fedoraproject.org/wiki/FedoraLiveCD
until something better comes along.


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

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


[Bug 232193] Review Request: perl-SVG - Generate stand-alone or inline SVG

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

Summary: Review Request: perl-SVG - Generate stand-alone or inline SVG


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:20 EST ---
New Package CVS Request
===
Package Name: perl-SVG
Short Description: An extension to generate stand-alone or inline SVG
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6
InitialCC: 

Some of these may already exist because of an obsolete package.

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

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


[Bug 226237] Merge Review: pcmciautils

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

Summary: Merge Review: pcmciautils


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:30 EST ---
done... please check pcmciautils-014-6.fc7

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

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


[Bug 225693] Merge Review: dialog

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

Summary: Merge Review: dialog


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|MODIFIED|ASSIGNED




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

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


[Bug 226453] Merge Review: system-config-boot

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

Summary: Merge Review: system-config-boot


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|MODIFIED|ASSIGNED




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

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


[Bug 225693] Merge Review: dialog

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

Summary: Merge Review: dialog


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 226215] Merge Review: openobex

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

Summary: Merge Review: openobex


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|MODIFIED|ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 226670] Merge Review: zisofs-tools

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

Summary: Merge Review: zisofs-tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|MODIFIED|ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 225693] Merge Review: dialog

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

Summary: Merge Review: dialog


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|RAWHIDE |




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

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


[Bug 226513] Merge Review: units

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

Summary: Merge Review: units


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED])|




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:41 EST ---
please check units-1.86-5.fc7

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

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


[Bug 226201] Merge Review: nmap

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

Summary: Merge Review: nmap


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]   |[EMAIL PROTECTED]
   Flag|fedora-review-  |fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:44 EST ---
please check nmap-4.20-3.fc7

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

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


[Bug 226670] Merge Review: zisofs-tools

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

Summary: Merge Review: zisofs-tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL

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

Summary: Review Request: paragui - Graphical User Interface based on SDL


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:59 EST ---
Hint to self: fix:
/usr/share/aclocal/paragui.m4:12: warning: underquoted definition of 
AM_PATH_PARAGUI


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

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


[Bug 226670] Merge Review: zisofs-tools

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

Summary: Merge Review: zisofs-tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag|fedora-review?  |needinfo?, fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:42 EST ---
Legend: OK - passed, NA = not available/not applicable, MUSTFIX - failed

MUST
OK - package meets naming guidelines
OK - package meets packaging guidelines
OK - license (GPL ) OK, text in %doc, matches source
OK - spec file legible, in am. english
OK - source matches upstream, latest available version, sha1sum is
77222a47c923d937b6af06362b8c1e6bf54259fe  zisofs-tools-1.0.7.tar.bz2
OK - package compiles on devel (x86_64)
NA - no missing BR
OK - no unnecessary BR
NA - no locales
OK - not relocatable
OK - owns all files/directories that it creates, does not take ownership of
foreign files/directories
OK - no duplicate files
OK - permissions ok
OK - %clean ok
OK - macro use consistent
OK - code, not content
OK - no need for -docs
OK - nothing in %doc affects runtime
OK - no need for .desktop file, program is a cmdline
NA - no devel / pkconfig / libtool


SHOULD
OK - package compiles in mock
OK - package builds in mock
NA - no scriptlets
OK - does not segfault when run

APPROVED


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

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


[Bug 232622] Review Request: perl-XML-Validator-Schema - Validate XML against a subset of W3C XML Schema

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

Summary: Review Request: perl-XML-Validator-Schema - Validate XML against a 
subset of W3C XML Schema


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 10:13 EST ---
New Package CVS Request
===
Package Name: perl-XML-Validator-Schema
Short Description: Validate XML against a subset of W3C XML Schema
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com

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

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


[Bug 232620] Review Request: perl-XML-Filter-XInclude - XInclude as a SAX Filter

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

Summary: Review Request: perl-XML-Filter-XInclude - XInclude as a SAX Filter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 10:14 EST ---
New Package CVS Request
===
Package Name: perl-XML-Filter-XInclude
Short Description: XInclude as a SAX Filter
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com

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

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


[Bug 232423] Review Request: perl-Class-Std - Support for creating standard inside-out classes

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

Summary: Review Request: perl-Class-Std - Support for creating standard 
inside-out classes


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 10:15 EST ---
New Package CVS Request
===
Package Name: perl-Class-Std
Short Description: Support for creating standard inside-out classes
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com

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

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


[Bug 226201] Merge Review: nmap

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

Summary: Merge Review: nmap


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
   Flag|fedora-review?  |fedora-review-




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 10:59 EST ---
(4.20-4.fc7 is in CVS, actually...)

1. Not fixed, but it's not (AFAIK) a blocker.

2. Fixed, thanks.  (It's a blocker now, too.)

3. Fixed (rpmlint on SRPM is now silent), thanks.

4. `make install`-y line looks better and appears to work fine.

5. nmap-4.20-nostrip.patch fixed.

A couple (minor) problems creeped in, however.

1. Versions in %changelog shouldn't have %{?dist} on the end; see the last point
in this section:

http://fedoraproject.org/wiki/Packaging/DistTag#head-5fff04551074fb311a96352be7e7cc3006c90f25

Totally logical mistake, I fully admit. :-)

2. We now have an rpmlint warning on the binary package:

W: nmap incoherent-version-in-changelog 4.20-4 2:4.20-4.fc7

This is due to Karsten Hopp's changelog entry -- it should have contained the
Epoch used (as you and Florian have been doing).

In a totally random comment, in running `rpmdiff` on the RPMs produced by your
changes (against ones from before), it seems you may have fixed a bug that I
hadn't noticed.  It looks like before your 1.35 CVS change to nmap.spec, nmap
was using its own libpcap.  Your BR change from libpcap to libpcap-devel fixed
that, and added a dependency on libpcap.  This, in my opinion, is a very good
thing -- thanks.

Fix the new 1 and 2, and I think nmap's good to go.

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

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


[Bug 233386] Review Request: perl-Gtk2-Ex-Carp - GTK+ friendly die() and warn() functions

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

Summary: Review Request: perl-Gtk2-Ex-Carp - GTK+ friendly die() and warn() 
functions
Alias: perl-Gtk2-Ex-Carp

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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 10:58 EST ---
Nope, nothing missing -- make test fails here as mock (correctly) runs with
$DISPLAY unset.  New spec/srpm to follow.

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

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


[Bug 233386] Review Request: perl-Gtk2-Ex-Carp - GTK+ friendly die() and warn() functions

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

Summary: Review Request: perl-Gtk2-Ex-Carp - GTK+ friendly die() and warn() 
functions
Alias: perl-Gtk2-Ex-Carp

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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 11:00 EST ---
SRPM URL: http://home.comcast.net/~ckweyl/perl-Gtk2-Ex-Carp-0.01-2.fc6.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-Gtk2-Ex-Carp.spec

Description:
This module exports four functions, of which two override the standard
die() and warn() functions, and two which allow for extended error
reporting. When called, these functions display a user-friendly message
dialog window.
New spec/srpm posted.

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

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


  1   2   >