[Bug 492583] Review Request: pidgin-gfire - Xfire plugin for Pidgin

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


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





--- Comment #7 from Andreas Osowski th0...@mkdir.name  2009-05-21 03:36:00 
EDT ---
All right, I fixed that stuff now.

Spec URL:
http://fedora.mkdir.name/packages/pidgin-gfire-0.8.1/pidgin-gfire.spec

SRPM URL:
http://fedora.mkdir.name/packages/pidgin-gfire-0.8.1/pidgin-gfire-0.8.1-3.fc10.src.rpm

* Wed May 21 2009 Andreas Osowski th0...@mkdir.name - 0.8.1-3
- Added Requires: libpurple
- Changed the files entry for the pixmaps

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

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


[Bug 492583] Review Request: pidgin-gfire - Xfire plugin for Pidgin

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


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


Andreas Osowski th0...@mkdir.name changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #8 from Andreas Osowski th0...@mkdir.name  2009-05-21 03:38:14 
EDT ---
New Package CVS Request
===
Package Name: pidgin-gfire
Short Description: Xfire plugin for Pidgin
Owners: th0br0
Branches: F-10 F-11

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

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


[Bug 501433] Review Request: perl-DateTime-Calendar-Mayan - Mayan Long Count Calendar

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


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


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

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 501433] Review Request: perl-DateTime-Calendar-Mayan - Mayan Long Count Calendar

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


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





--- Comment #2 from Chris Weyl cw...@alumni.drew.edu  2009-05-21 03:43:30 EDT 
---
New Package CVS Request
===
Package Name: perl-DateTime-Calendar-Mayan
Short Description: Mayan Long Count Calendar
Owners: cweyl
Branches: F-9 F-10 F-11 devel
InitialCC: perl-sig

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

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


[Bug 189611] Review Request: milter-regex regular expression based sendmail milter

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


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


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

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #6 from Paul Howarth p...@city-fan.org  2009-05-21 04:09:34 EDT 
---
Package Change Request
==
Package Name: milter-regex
New Branches: EL-4 EL-5
Owners: pghmcfc

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

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


[Bug 456542] Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH

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


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


Jan ONDREJ ondr...@salstar.sk changed:

   What|Removed |Added

   Flag||needinfo?(walt...@redhat.co
   ||m)




--- Comment #14 from Jan ONDREJ ondr...@salstar.sk  2009-05-21 04:41:18 EDT 
---
Colin, Adel, ping?

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

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


[Bug 481224] Review Request: rabbitmq-server - An AMQP server written in Erlang

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


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





--- Comment #26 from Peter Lemenkov lemen...@gmail.com  2009-05-21 04:42:08 
EDT ---
(In reply to comment #23)

 Peter, maybe we can start an Erlang SIG or something like that? 

Already did :)

https://fedoraproject.org/wiki/Erlang

However, there is not much lavuable info on that page (no guidelines yet). But
feel free to modify it.

 There are a lot of erlang packages which would be nice to have in Fedora, 
 and it would be easier to package them up if this is a joined effort. 

Agree.

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

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


[Bug 481224] Review Request: rabbitmq-server - An AMQP server written in Erlang

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


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





--- Comment #27 from Peter Lemenkov lemen...@gmail.com  2009-05-21 04:42:45 
EDT ---
(In reply to comment #26)

 However, there is not much lavuable info on that page (no guidelines yet). 

s/lavuable/valuable/

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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





--- Comment #6 from leigh scott leigh123li...@googlemail.com  2009-05-21 
05:55:28 EDT ---
Spec: http://leigh123linux.fedorapeople.org/review/gmixer/3/gmixer.spec

SRPM:
http://leigh123linux.fedorapeople.org/review/gmixer/3/gmixer-1.3-2.fc11.src.rpm

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

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


[Bug 496718] Review Request: mingw32-plotmm - MinGW GTKmm plot widget for scientific applications

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


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


Bug 496718 depends on bug 492130, which changed state.

Bug 492130 Summary: Review Request: mingw32-gtkmm24 - MinGW Windows C++ 
interface for GTK2 (a GUI library for X)
https://bugzilla.redhat.com/show_bug.cgi?id=492130

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 06:28:08 EDT ---
mingw32-plotmm-0.1.2-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/mingw32-plotmm-0.1.2-2.fc11

--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 06:32:24 EDT ---
mingw32-plotmm-0.1.2-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mingw32-plotmm-0.1.2-2.fc10

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

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


[Bug 496718] Review Request: mingw32-plotmm - MinGW GTKmm plot widget for scientific applications

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


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


Bug 496718 depends on bug 492130, which changed state.

Bug 492130 Summary: Review Request: mingw32-gtkmm24 - MinGW Windows C++ 
interface for GTK2 (a GUI library for X)
https://bugzilla.redhat.com/show_bug.cgi?id=492130

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 06:28:08 EDT ---
mingw32-plotmm-0.1.2-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/mingw32-plotmm-0.1.2-2.fc11

--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 06:32:24 EDT ---
mingw32-plotmm-0.1.2-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mingw32-plotmm-0.1.2-2.fc10

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

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


[Bug 481224] Review Request: rabbitmq-server - An AMQP server written in Erlang

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


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





--- Comment #28 from Hubert Plociniczak hub...@lshift.net  2009-05-21 
06:46:55 EDT ---
Spec URL: http://dev.lshift.net/hubert/rabbitmq-server.spec
SRPM URL: http://dev.lshift.net/hubert/rabbitmq-server-1.5.5-1.src.rpm

This release contains fixes for the problems mentioned above.
Tested on koji for F-11 and EL-5 and works fine now.
rpmlint is not silent but we fixed 'missing-lsb-keyword' error and the rest can
be ignored.

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

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


[Bug 491614] Review Request: mingw32-libglademm24 - C++ wrapper for libglade

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


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


Bug 491614 depends on bug 492130, which changed state.

Bug 492130 Summary: Review Request: mingw32-gtkmm24 - MinGW Windows C++ 
interface for GTK2 (a GUI library for X)
https://bugzilla.redhat.com/show_bug.cgi?id=492130

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 06:46:38 EDT ---
mingw32-libglademm24-2.6.7-6.fc11 has been submitted as an update for Fedora
11.
http://admin.fedoraproject.org/updates/mingw32-libglademm24-2.6.7-6.fc11

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

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


[Bug 225809] Merge Review: gmp

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


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


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 CC||jussi.leht...@iki.fi




--- Comment #24 from Jussi Lehtola jussi.leht...@iki.fi  2009-05-21 06:45:41 
EDT ---
tibbs: ping?

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

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


[Bug 225796] Merge Review: giflib

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


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


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jussi.leht...@iki.fi




--- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi  2009-05-21 06:51:58 
EDT ---
Michael: are you still reviewing this?

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

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


[Bug 501924] New: Review Request: mingw32-tcl - MinGW Windows Tool Command Language

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

Summary: Review Request: mingw32-tcl - MinGW Windows Tool Command Language
Alias: mingw32-tcl

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

   Summary: Review Request: mingw32-tcl - MinGW Windows Tool
Command Language
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: t.sai...@alumni.ethz.ch
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com,
fedora-mi...@lists.fedoraproject.org
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://sailer.fedorapeople.org/mingw32-tcl.spec
SRPM URL: http://sailer.fedorapeople.org/mingw32-tcl-8.5.6-1.fc11.src.rpm
Description:
MinGW Windows C++ wrapper for the glade library.

Approved MinGW packaging guidelines are here:
http://fedoraproject.org/wiki/Packaging/MinGW

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

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


[Bug 481224] Review Request: rabbitmq-server - An AMQP server written in Erlang

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


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


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #29 from Peter Lemenkov lemen...@gmail.com  2009-05-21 07:01:54 
EDT ---
Ok

+ md5 is correct

[pe...@sulaco SOURCES]$ md5sum rabbitmq-server-1.5.5.tar.gz*
1dceb98bb57cd6acef90f58e96a7fce4  rabbitmq-server-1.5.5.tar.gz
1dceb98bb57cd6acef90f58e96a7fce4  rabbitmq-server-1.5.5.tar.gz.1
[pe...@sulaco SOURCES]$

+ rpmlint is not silent but these messages already explained (see
https://bugzilla.redhat.com/show_bug.cgi?id=481224#c18 )

[pe...@sulaco SPECS]$ rpmlint ../RPMS/ppc/rabbitmq-server-1.5.5-1.ppc.rpm 
rabbitmq-server.ppc: E: no-binary
rabbitmq-server.ppc: W: only-non-binary-in-usr-lib
rabbitmq-server.ppc: W: non-standard-uid /var/lib/rabbitmq rabbitmq
rabbitmq-server.ppc: W: non-standard-gid /var/lib/rabbitmq rabbitmq
rabbitmq-server.ppc: E: non-standard-dir-perm /var/lib/rabbitmq 0750
rabbitmq-server.ppc: W: non-standard-uid /var/log/rabbitmq rabbitmq
rabbitmq-server.ppc: W: non-standard-gid /var/log/rabbitmq rabbitmq
rabbitmq-server.ppc: E: non-standard-dir-perm /var/log/rabbitmq 0750
1 packages and 0 specfiles checked; 3 errors, 5 warnings.
[pe...@sulaco SPECS]$

Ok, this package is


APPROVED.

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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


Igor Jurišković juriskovic.i...@gmail.com changed:

   What|Removed |Added

   Flag||needinfo?




--- Comment #7 from Igor Jurišković juriskovic.i...@gmail.com  2009-05-21 
06:59:25 EDT ---
This is completely different package. Above full review doesnt count on this
one. Will make new one after these few errors are fixed. Ok, lets see;


- https://fedoraproject.org/wiki/Packaging/Python#System_Architecture - says if
you are installing something to global site_packages directory use following:
%{!?python_sitelib: %global python_sitelib %(%{__python} -c from
distutils.sysconfig import get_python_lib; print get_python_lib())}

Please change python_sitearch with python_sitelib as stated at link above.

Dont forget to change all %{python_sitearch} with new one.



- remove --vendor.



- desktop file has Encoding. It is assumed that encoding is UTF-8 by default so
that line is useless. Please remove it.


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

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


[Bug 501924] Review Request: mingw32-tcl - MinGW Windows Tool Command Language

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


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





--- Comment #1 from Thomas Sailer t.sai...@alumni.ethz.ch  2009-05-21 
07:04:07 EDT ---
Description should be:
MinGW Windows Tool Command Language

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

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


[Bug 225794] Merge Review: ghostscript-fonts

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


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


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 CC||jussi.leht...@iki.fi
 AssignedTo|nob...@fedoraproject.org|jussi.leht...@iki.fi
   Flag||fedora-review?




--- Comment #8 from Jussi Lehtola jussi.leht...@iki.fi  2009-05-21 07:10:20 
EDT ---
Assigning.

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

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


[Bug 481224] Review Request: rabbitmq-server - An AMQP server written in Erlang

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


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


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |




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

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


[Bug 225794] Merge Review: ghostscript-fonts

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


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





--- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi  2009-05-21 07:35:26 
EDT ---
- This is a legacy font package, so Font guidelines are not valid here.

- Change
 Requires(post): /usr/bin/mkfontscale /usr/bin/mkfontdir
to
 Requires(post): xorg-x11-font-utils
as this is the package that provides those since Fedora Core 2 (2004).

- I'm not totally sure you need
 Requires(post): fontconfig
 Requires(postun): fontconfig
as you already have Requires: fontconfig. Besides, this is probably
automatically picked up by rpm. Doesn't hurt having them, though.

- Change references to /etc to %{_sysconfdir}

- Setting umask is probably not necessary as this is done by rpm.

- Replace `which mkfontdir` with plain mkfontdir.

- Drop
 %dir %{catalogue}
in %files section, as
 $ rpm -qf /etc/X11/fontpath.d/
 filesystem-2.4.19-1.fc10.i386
is already owned on every installation.

- I can't find a single mention of a license in the tarball!

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

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


[Bug 481224] Review Request: rabbitmq-server - An AMQP server written in Erlang

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


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


Hubert Plociniczak hub...@lshift.net changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #30 from Hubert Plociniczak hub...@lshift.net  2009-05-21 
07:46:57 EDT ---
New Package CVS Request
===
Package Name: rabbitmq-server
Short Description: An AMQP server written in Erlang
Owners: hubert
Branches: F-9 F-10 F-11 EL-5
InitialCC:

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag|needinfo?   |




--- Comment #8 from leigh scott leigh123li...@googlemail.com  2009-05-21 
07:56:36 EDT ---
(In reply to comment #7)
 This is completely different package. Above full review doesnt count on this
 one. Will make new one after these few errors are fixed. Ok, lets see;
 
 
 - https://fedoraproject.org/wiki/Packaging/Python#System_Architecture - says 
 if
 you are installing something to global site_packages directory use following:
 %{!?python_sitelib: %global python_sitelib %(%{__python} -c from
 distutils.sysconfig import get_python_lib; print get_python_lib())}
 
 Please change python_sitearch with python_sitelib as stated at link above.
 
 Dont forget to change all %{python_sitearch} with new one.
 
 
 
 - remove --vendor.
 
 
 
 - desktop file has Encoding. It is assumed that encoding is UTF-8 by default 
 so
 that line is useless. Please remove it.
   

Ok, here's the new spec * SRPM

Spec: http://leigh123linux.fedorapeople.org/review/gmixer/4/gmixer.spec


SRPM:
http://leigh123linux.fedorapeople.org/review/gmixer/4/gmixer-1.3-3.fc11.src.rpm

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

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


[Bug 498723] Review Request: eZ Publish

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


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


Randall Berry randyn3...@gmail.com changed:

   What|Removed |Added

 CC||randyn3...@gmail.com




--- Comment #3 from Randall Berry randyn3...@gmail.com  2009-05-21 08:08:09 
EDT ---
I'll take a crack at this since nobody else has.
If my findings are wrong please correct them.

Key:
[*] Pass
[x] Fail
[-] Not applicable
[?] Questions (see comments)

[x]  MUST: rpmlint must be run on every package. The output should be
 posted in the review.[1]

rpmlint not silent.
4 packages and 1 specfiles checked; 292 errors, 2557 warnings.
(see attachement http://fpaste.org/paste/12523 )

[*]  MUST: The package must be named according to the Package Naming
Guidelines.

[*]  MUST: The spec file name must match the base package %{name},
 in the format %{name}.spec unless your package has an exemption. [2] .

[*]  MUST: The package must meet the Packaging Guidelines.

[*]  MUST: The package must be licensed with a Fedora approved
 license and meet the Licensing Guidelines .

[*]  MUST: The License field in the package spec file must match
 the actual license. [3]

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

[*]  MUST: The spec file must be written in American English. [5]

[*]  MUST: The spec file for the package MUST be legible. [6]

[?]  MUST: The sources used to build the package must match the upstream
 source, as provided in the spec URL. Reviewers should use md5sum for
 this task. If no upstream URL can be specified for this package,
 please see the Source URL Guidelines for how to deal with this.

The source URL
(http://ez.no/content/download/261295/1832505/version/3/file/ezpublish-4.1.0-gpl.tar.gz)
causes an error. 4.1.1 is also the current version 4.1.0 seems to be
un-available.

[-]  MUST: The package MUST successfully compile and build into binary
 rpms on at least one primary architecture. [7]

php no binary build.

[-]  MUST: If the package does not successfully compile, build or work on
 an architecture, then those architectures should be listed in the spec
 in ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug
 filed in bugzilla, describing the reason that the package does not
 compile/build/work on that architecture. The bug number MUST be
 placed in a comment, next to the corresponding ExcludeArch line. [8]

[-]  MUST: All build dependencies must be listed in BuildRequires,
 except for any that are listed in the exceptions section of the
 Packaging Guidelines ; inclusion of those as BuildRequires is
 optional. Apply common sense.

[*]  MUST: The spec file MUST handle locales properly. This is done by
 using the %find_lang macro. Using %{_datadir}/locale/[ ] is strictly
 forbidden.[9]

[-]  MUST: Every binary RPM package (or subpackage) which stores shared
 library files (not just symlinks) in any of the dynamic linker's
 default paths, must call ldconfig in %post and %postun. [10]

[-]  MUST: If the package is designed to be relocatable, the packager
 must state this fact in the request for review, along with the
 rationalization for relocation of that specific package.
 Without this, use of Prefix: /usr is considered a blocker.

[x]  MUST: A package must own all directories that it creates. If it does not
 create a directory that it uses, then it should require a package which
 does create that directory. Refer to the Guidelines for examples.

Package does not own ALL directories it creates. Each directory
including %{_datadir}/%{name}/ and all subdirectories therein that the
package creates must be prefixed by %dir not just the one
directory.

[*]  MUST: A package must not contain any duplicate files in the %files
listing.

[x]  MUST: Permissions on files must be set properly. Executables should
 be set with executable permissions, for example. Every %files section
 must include a %defattr(...) line.

Some questionable perms found by rpmlint. (see attachment)

[*]  MUST: Each package must have a %clean section, which contains
 rm -rf %{buildroot} ( or $RPM_BUILD_ROOT ).

[*]  MUST: Each package must consistently use macros, as described in the
 macros section of Packaging Guidelines .

[*]  MUST: The package must contain code, or permissable content.
 This is described in detail in the code vs. content section
 of Packaging Guidelines.

[*]  MUST: Large documentation files should go in a -doc subpackage.
(The definition of large is left up to the packager's best judgement,
 

[Bug 498723] Review Request: eZ Publish

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


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


Jon Ciesla l...@jcomserv.net changed:

   What|Removed |Added

 CC||l...@jcomserv.net




--- Comment #4 from Jon Ciesla l...@jcomserv.net  2009-05-21 08:39:15 EDT ---
Minor quibble, 

[-]  MUST: The package MUST successfully compile and build into binary
 rpms on at least one primary architecture. [7]

php no binary build.

What is meant here is the product RPM as opposed to SRPM, not binary executable
as opposed to script.  Essentially, rpmbuild -ba foo.spec must produce
something in ../RPMS/*/.

FYI. :)

Also, for the Requires, if any of those pear packages exist in Fedora as RPMS,
they should be Required.  If not, and this software needs them, they need to be
packaged and included in Fedora before this can be.  I see that this has a
blocking bug, so maybe that's going on, I didn't dig that far.

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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





--- Comment #9 from Igor Jurišković juriskovic.i...@gmail.com  2009-05-21 
08:49:09 EDT ---
Two more things:

--
in %files section change:
%{python_sitelib}/gmixer-1.0-py2.6.egg-info
to:
%{python_sitelib}/gmixer-1.0-py2.5.egg-info

Because thats the right name of the file.
--
remove %doc in files section. You have no documentation so its useless.
--



I have to go now for few hours. When I get back I'll make full review.

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

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


[Bug 493250] Review Request: perl-Goo-Canvas -- Goo::Canvas Perl interface to the GooCanvas

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


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





--- Comment #19 from Liang Suilong liangsuil...@gmail.com  2009-05-21 
09:06:40 EDT ---
Mamoru Tasaka

Thank you. Your patch works so well. The koji build is excellent. 

https://koji.fedoraproject.org/koji/taskinfo?taskID=1368171

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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





--- Comment #10 from leigh scott leigh123li...@googlemail.com  2009-05-21 
09:12:31 EDT ---
(In reply to comment #9)
 Two more things:
 
 --
 in %files section change:
 %{python_sitelib}/gmixer-1.0-py2.6.egg-info
 to:
 %{python_sitelib}/gmixer-1.0-py2.5.egg-info
 
 Because thats the right name of the file.
 --
 remove %doc in files section. You have no documentation so its useless.
 --
 
 
 
 I have to go now for few hours. When I get back I'll make full review.  

Ok, I have made the recommended changes.


%{python_sitelib}/gmixer-1.0-py2.6.egg-info

is correct for F11  F12, changing it to 

 %{python_sitelib}/gmixer-1.0-py2.5.egg-info

would be wrong as well, so I changed it to 

%{python_sitelib}/gmixer-1.0-py?.?.egg-info

This will enable it to build with python 2.5 or 2.6 


Spec: http://leigh123linux.fedorapeople.org/review/gmixer/5/gmixer.spec


SRPM:
http://leigh123linux.fedorapeople.org/review/gmixer/5/gmixer-1.3-4.fc11.src.rpm

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

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


[Bug 501957] New: Review Request: perl-Tk--ProgressBar-Mac - Mac ProgressBar for Perl::Tk

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

Summary: Review Request: perl-Tk--ProgressBar-Mac - Mac ProgressBar for Perl::Tk

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

   Summary: Review Request: perl-Tk--ProgressBar-Mac - Mac
ProgressBar for Perl::Tk
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: david.hanneq...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://hvad.cfppa-cibeins.com/perl-Tk-ProgressBar-Mac/perl-Tk-ProgressBar-Mac.spec
SRPM URL:
http://hvad.cfppa-cibeins.com/perl-Tk-ProgressBar-Mac/perl-Tk-ProgressBar-Mac-1.2-1.fc10.src.rpm
Description: This widget provides a dynamic image that looks just like a Mac OS
9 progress bar.  Packed around it are four Frames, north, south, east and west,
within which you can stuff additional widgets. For example, see how
Tk::Copy::Mac uses several Labels and a CollapsableFrame widget to create a
reasonable facsimile of a Macintosh copy dialog.

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

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


[Bug 501958] New: Review Request: scotch - Graph, mesh and hypergraph partitioning library

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

Summary: Review Request: scotch - Graph, mesh and hypergraph partitioning 
library

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

   Summary: Review Request: scotch - Graph, mesh and hypergraph
partitioning library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dakin...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://deji.fedorapeople.org/scotch.spec
SRPM URL: http://deji.fedorapeople.org/scotch-5.1.6-1.fc11.src.rpm
Description: Scotch is a software library for graph and mesh/hypergraph
partitioning and sparse matrix ordering.

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

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


[Bug 484798] Review Request: xiphos - Bible study and research tool

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


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





--- Comment #9 from Deji Akingunola dakin...@gmail.com  2009-05-21 09:17:45 
EDT ---
Xiphos-3.1, which includes fixes for the compile-time warnings in comment #3
have now being released.

Spec URL: http://deji.fedorapeople.org/xiphos.spec
SRPM URL: http://deji.fedorapeople.org/xiphos-3.1-1.fc11.src.rpm

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

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


[Bug 501924] Review Request: mingw32-tcl - MinGW Windows Tool Command Language

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


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


Erik van Pienbroek erik-fed...@vanpienbroek.nl changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||erik-fed...@vanpienbroek.nl
 AssignedTo|nob...@fedoraproject.org|erik-fed...@vanpienbroek.nl
   Flag||fedora-review?




--- Comment #2 from Erik van Pienbroek erik-fed...@vanpienbroek.nl  
2009-05-21 09:22:54 EDT ---
This one is quite hard to read, due to the frequent use of macros. However, I
don't think it can be done any other way, so I'll do my best to read through
them.



The %files list contains some redundant entries:
 %dir %{_mingw32_libdir}/%{name1}%{majorver}
 %dir %{_mingw32_libdir}/dde*
 %dir %{_mingw32_libdir}/reg*
 %{_mingw32_libdir}/%{name1}%{majorver}/*
 %{_mingw32_libdir}/dde*/*
 %{_mingw32_libdir}/reg*/* 

These can be simplified to:
 %{_mingw32_libdir}/%{name1}%{majorver}/
 %{_mingw32_libdir}/dde*/
 %{_mingw32_libdir}/reg*/



For readability, you might want to move this piece of code to the top of the
.spec file:
 # don't run make test by default
 %{?_without_check: %define _without_check 0}
 %{!?_without_check: %define _without_check 1}



Is the rename of the import libraries from .a to .dll.a really necessary?
AFAIK, this is only needed for libtool based libraries (which tcl isn't)



The native tcl package is now at version 8.5.7. You might want to update to
that version too. See
http://cvs.fedoraproject.org/viewvc/rpms/tcl/devel/tcl.spec?view=log for the
complete list of changes

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

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


[Bug 501960] New: Review Request: webinject - Web/HTTP Test Tool

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

Summary: Review Request: webinject - Web/HTTP Test Tool

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

   Summary: Review Request: webinject - Web/HTTP Test Tool
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: david.hanneq...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://hvad.cfppa-cibeins.com/webinject/webinject.spec
SRPM URL: http://hvad.cfppa-cibeins.com/webinject/webinject-1.41-1.fc10.src.rpm
Description: WebInject is a free tool for automated testing of web applications
and web services. It can be used to test individual system  components that
have HTTP interfaces (JSP, ASP, CGI, PHP, AJAX, Servlets, HTML Forms, XML/SOAP
Web Services, REST, etc), and can be used as a test harness to create a suite
of [HTTP level] automated functional, acceptance, and regression tests. A test
harness allows you to run many test cases and collect/report your  results.
WebInject offers real-time results display and may also be used for monitoring
system response times.

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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





--- Comment #11 from leigh scott leigh123li...@googlemail.com  2009-05-21 
10:12:31 EDT ---
The gmixer-1.3-4 srpm builds at Koji on all targets on all active dists.

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

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

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

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

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

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


[Bug 489686] Review Request: armadillo - fast C++ matrix library with interfaces to LAPACK and ATLAS

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


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





--- Comment #19 from Chitlesh GOORAH chitl...@gmail.com  2009-05-21 10:26:54 
EDT ---
#1: Ralf is right here, use the macro %cmake instead of cmake.

#2: Also before creating the SRPM, change the permissions of both the spec file
and the sources to 0644, currently :
chitlesh(rpmbuild)[1]$rpmlint
/home/chitlesh/rpmbuild/SRPMS/armadillo-0.6.11-2.fc10.src.rpm
armadillo.src: W: strange-permission armadillo-0.6.11.tar.gz 0640
armadillo.src: W: strange-permission armadillo.spec 0640

#3: Swap the docs_user and LICENSE.txt from %name-devel to %name as they are
user specific.

#4: 
remove /usr/share/doc/armadillo-devel-0.6.11/examples/Makefile.cmake after make
install

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

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


[Bug 493247] Review Request: perl-Gnome2-Wnck -- Perl interface to the Window Navigator Construction Kit

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


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





--- Comment #11 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
10:31:48 EDT ---
Would you write CVS request?

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

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


[Bug 470155] Review Request: xword - Reads and writes crossword puzzles in the Across Lite file format

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Blocks|177841(FE-NEEDSPONSOR)  |201449(FE-DEADREVIEW)
 Resolution||NOTABUG
   Flag|fedora-review?, |
   |needinfo?(alexes...@yahoo.c |
   |om) |




--- Comment #24 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
10:34:15 EDT ---
Once closing.

If someone wants to import this package into Fedora,
please file a new review request and mark this bug as
a duplicate of the new one, thank you.

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

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


[Bug 470696] Review Request: rubygem-passenger - Passenger Ruby on Rails deployment system

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag||needinfo?(kana...@kanarip.c
   ||om)




--- Comment #32 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
10:35:00 EDT ---
ping?

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

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


[Bug 492895] Review Request: xml-security-c - C++ Implementation of W3C security standards for XML

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag||needinfo?(antti.andreimann@
   ||mail.ee)




--- Comment #7 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
10:36:27 EDT ---
ping?

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

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


[Bug 479546] Review Request: jailkit - Jailkit limits user accounts to specific files and/or commands

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG
   Flag|fedora-review?, |
   |needinfo?(dignan.patr...@gm |
   |ail.com)|




--- Comment #26 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
10:35:55 EDT ---
Once closing.

If someone wants to import this package into Fedora, please
file a new review request and mark this as a duplicate of
the new one, thank you.

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

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


[Bug 493247] Review Request: perl-Gnome2-Wnck -- Perl interface to the Window Navigator Construction Kit

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


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





--- Comment #12 from Liang Suilong liangsuil...@gmail.com  2009-05-21 
10:38:38 EDT ---
New Package CVS Request
===
Package Name: perl-Gnome2-Wnck
Short Description: Perl interface to the Window Navigator Construction Kit
Owners: liangsuilong
Branches: F-10 F-10 devel
InitialCC: liangsuilong
Cvsextras Commits: yes

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

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


[Bug 500249] Pre-Review Request: plexus-digest - Plexus Digest / Hashcode Components

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


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





--- Comment #5 from Andrew Overholt overh...@redhat.com  2009-05-21 10:51:45 
EDT ---
(In reply to comment #4)
 I wrote it based on spot's comment:
 
 https://bugzilla.redhat.com/show_bug.cgi?id=500246#c8
 
 DO you think that was not meant for the whole plexus, but just for
 plexus-classworlds?  

Yes, I think it just applies to plexus-classworlds where that LICENSE.txt file
exists.  There's nothing similar in this package to warrant a  and Plexus, is
there?

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

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


[Bug 502024] New: Review Request: xsd - W3C XML schema to C++ data binding compiler

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

Summary: Review Request: xsd - W3C XML schema to C++ data binding compiler
Alias: xsdcpp

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

   Summary: Review Request: xsd - W3C XML schema to C++ data
binding compiler
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: antti.andreim...@mail.ee
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, mtas...@ioa.s.u-tokyo.ac.jp,
ka...@smartlink.ee, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://anttix.org/fedora/pkg/xsd.spec
SRPM URL: http://anttix.org/fedora/pkg/xsd-3.2.0-1.fc10.src.rpm

Description: 
CodeSynthesis XSD is an open-source, cross-platform W3C XML Schema to
C++ data binding compiler. Provided with an XML instance specification
(XML Schema), it generates C++ classes that represent the given
vocabulary as well as parsing and serialization code.
You can then access the data stored in XML using types and functions
that semantically correspond to your application domain rather than
dealing with intricacies of reading and writing XML.

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1368376

Known issues:
1. The doc sub-package has a two Postscript files that rpmlint grunts about
   not being in UTF-8 encoding. I couldn't find any reasonable converter that
   can convert Postscript files to another encoding and AFAIK many printers
   do not support UTF-8 at all.

2. The main package contains both: the xsd schema compiler and the header files
   needed to build the generated code. Rpmlint does not like it at all.
   However there is no value in separating headers from the compiler:
   if You use the compiler to generate C++ code, You most certainly want to
   build the generated files as well.

3. Compiler binary is renamed to xsdcxx to avoid a name conflict. The same 
   naming is currently used in the debian project
   (http://packages.debian.org/source/lenny/xsd).

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

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


[Bug 492895] Review Request: xml-security-c - C++ Implementation of W3C security standards for XML

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


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


Antti Andreimann antti.andreim...@mail.ee changed:

   What|Removed |Added

   Flag|needinfo?(antti.andreimann@ |
   |mail.ee)|




--- Comment #8 from Antti Andreimann antti.andreim...@mail.ee  2009-05-21 
11:04:01 EDT ---
Thank You for Your encouragement.
I have discarded the idea to package every little piece of this xsd stuff
separately. I found an easy to use bundle that includes all the components
and a build script. The same bundle is used by debian to build the xsd compiler
package present in lenny.
So I have created a new xsd package for Fedora here:
https://bugzilla.redhat.com/show_bug.cgi?id=502024

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

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


[Bug 501924] Review Request: mingw32-tcl - MinGW Windows Tool Command Language

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


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





--- Comment #3 from Thomas Sailer t.sai...@alumni.ethz.ch  2009-05-21 
11:16:32 EDT ---
(In reply to comment #2)

Thanks for taking! That was quick!

 This one is quite hard to read, due to the frequent use of macros. However, I
 don't think it can be done any other way, so I'll do my best to read through
 them.

I modified the native tcl package spec file. 

 The %files list contains some redundant entries:

Fixed.

 For readability, you might want to move this piece of code to the top of the
 .spec file:
  # don't run make test by default
  %{?_without_check: %define _without_check 0}
  %{!?_without_check: %define _without_check 1}

This is again from the native spec file. I kept it there to minimize the
differences. So the %check section could probably completely go...

 Is the rename of the import libraries from .a to .dll.a really necessary?
 AFAIK, this is only needed for libtool based libraries (which tcl isn't)

I don't think the renaming is strictly necessary. I did it to make it extra
clear that the .a files are implibs, not static libraries...

 The native tcl package is now at version 8.5.7. You might want to update to
 that version too. See

Woohoo. Just when you think you're up to date :)
But there does not seem to be a successful build of 8.5.7 in koji. So I'm ahead
now :)

Updated:
Spec URL: http://sailer.fedorapeople.org/mingw32-tcl.spec
SRPM URL: http://sailer.fedorapeople.org/mingw32-tcl-8.5.7-1.fc11.src.rpm

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

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


[Bug 492397] Review Request: websvn - Online subversion repository browser

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


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


Bug 492397 depends on bug 499341, which changed state.

Bug 499341 Summary: Review Request: php-pear-Text-Diff - Engine for performing 
and rendering text diffs
https://bugzilla.redhat.com/show_bug.cgi?id=499341

   What|Old Value   |New Value

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



--- Comment #7 from Jan Klepek jan.kle...@hp.com  2009-05-21 11:53:56 EDT ---
Hi Xavier,

Could you share latest spec file and SRPMS for (hope so) final review?

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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





--- Comment #12 from Igor Jurišković juriskovic.i...@gmail.com  2009-05-21 
11:56:16 EDT ---
# MUST: rpmlint must be run on every package. The output should be posted in
the review. YES

# MUST: The package must be named according to the Package Naming Guidelines.
YES

# MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. YES

# MUST: The package must meet the Packaging Guidelines. YES 

# MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines. YES

# MUST: The License field in the package spec file must match the actual
license. YES

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

# MUST: The spec file must be written in American English. YES

# MUST: The spec file for the package MUST be legible. YES

# MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this. YES

# MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. YES

# MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. N/A

# MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense. YES

# MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden. N/A

# MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun. N/A

# MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. N/A

# MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. NO

# MUST: A Fedora package must not list a file more than once in the spec file's
%files listings. YES

# MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line. YES

# MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT). YES

# MUST: Each package must consistently use macros. YES

# MUST: The package must contain code, or permissable content. YES

# MUST: Large documentation files must go in a -doc subpackage. (The definition
of large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity). N/A

# MUST: If a package includes something as %doc, it must not affect the runtime
of the application. To summarize: If it is in %doc, the program must run
properly if it is not present. N/A

# MUST: Header files must be in a -devel package. N/A

# MUST: Static libraries must be in a -static package. N/A

# MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
(for directory ownership and usability). N/A

# MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel
package. N/A

# MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name} =
%{version}-%{release} N/A

# MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built. N/A

# MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section. If you feel that your packaged GUI application does not need
a .desktop file, you must put a comment in the spec file with your explanation.
YES

# MUST: Packages must not own files or directories already owned by other
packages. The rule of thumb here 

[Bug 478609] Review Request: djmount - Mounts UPnP Audio/Video/Photo shares as FUSE filesystems.

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


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


MartinG grons...@gmail.com changed:

   What|Removed |Added

 CC||grons...@gmail.com




--- Comment #7 from MartinG grons...@gmail.com  2009-05-21 12:08:46 EDT ---
I would also like to see this in Fedora

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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


Igor Jurišković juriskovic.i...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




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

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


[Bug 493246] Review Request: Shutter -- a feature-rich screenshot program.

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


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


Jan Klepek jan.kle...@hp.com changed:

   What|Removed |Added

   Flag||needinfo?(liangsuil...@gmai
   ||l.com)




--- Comment #7 from Jan Klepek jan.kle...@hp.com  2009-05-21 11:56:18 EDT ---
Hi, 

any news?

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #14 from leigh scott leigh123li...@googlemail.com  2009-05-21 
12:10:49 EDT ---
New Package CVS Request
===
Package Name: gmixer
Short Description: Just a simple audio mixer
Owners: leigh123linux
Branches: F-9 F-10 F-11
InitialCC:

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

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


[Bug 502054] New: Review Request: perl-Lingua-Flags - Provide small flag icons

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

Summary: Review Request: perl-Lingua-Flags - Provide small flag icons
Alias: perl-Lingua-Flags

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

   Summary: Review Request: perl-Lingua-Flags - Provide small flag
icons
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Lingua-Flags/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~iarnell/review/perl-Lingua-Flags.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Lingua-Flags-0.05-1.fc12.src.rpm

Description:
Module with small flags icons. You can get them in base64 inline HTML image
or as a gif image.

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1368784

*rt-0.09

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

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


[Bug 497639] Review Request: gmixer - Just a simple audio mixer

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


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





--- Comment #13 from leigh scott leigh123li...@googlemail.com  2009-05-21 
12:09:39 EDT ---
Hi Igjurisk,


Thank you for reviewing gmixer for me.

Leigh

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

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


[Bug 478759] Review Request: perl-SystemPerl - SystemPerl Perl module

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


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





--- Comment #18 from Chitlesh GOORAH chitl...@gmail.com  2009-05-21 12:23:57 
EDT ---
Updated:
Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl.spec
SRPM URL:
http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl-1.320-1.fc10.src.rpm

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

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


[Bug 491430] Review Request: sslogger - A keystroke logging utility for privileged user escalation

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks||182235(FE-Legal)




--- Comment #17 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
12:23:58 EDT ---
Some pre-remarks:

 About license
  Well, actually almost all the files in this tarball are
  under GPLv3+, however sslogger.c is under BSD with advertising,
  which conflicts with GPLv3+:
  https://fedoraproject.org/wiki/Licensing

  For this package, actually sl.c (under GPLv3+) actually calls
  execvp() for sslogger, which is written in sslogger.c (BSD with
  advertising), licenses really conflict.

* Please consider to use %?dist macro:
  https://fedoraproject.org/wiki/Packaging/DistTag

* BuildRoot does not honor Fedora's packaging guidelines:
  https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

* Source0 must be specified with full URL:
  https://fedoraproject.org/wiki/Packaging/SourceURL

* Some needed Requires(pre) or so are missing:
 
https://fedoraproject.org/wiki/Packaging/SysVInitScript#Initscripts_in_spec_file_scriptlets

* Fedora specific compilation flags are not correctly
  honored:
  https://fedoraproject.org/wiki/Packaging/Guidelines#Compiler_flags
  https://fedoraproject.org/wiki/Packaging/Debuginfo

* Please use macros for standard directories. For example,
  you should use %{_localstatedir} for /var:
  https://fedoraproject.org/wiki/Packaging/RPMMacros

* For binary names
  - In Fedora other rpm already uses the name %{_bindir}/sl
  - And IMO the name %{_bindir}/replay is too generic.
  - Also I recommend to change %log_dir to %_localstatedir/log/sslogger
or so.

* Fedora considers that deleting user/group automatically by
  rpm scriptlets is dangerous and this must be done manually
  by sysadmin


%post
chown -R %{suser}.%{sgroup} %{log_dir}

* - is actually useless. This scriptlet is executed only when this rpm
  is installed or upgraded and does nothing when this rpm
  is actually used.

* Fedora uses %{_defaultdocdir}/%{name}-%{version} as directory
  to install documents
  - By the way currently the directory %{_docdir}/sslogger/ itself
is not owned by this package:
   
https://fedoraproject.org/wiki/Packaging/Guidelines#File_and_Directory_Ownership
https://fedoraproject.org/wiki/Packaging/UnownedDirectories

Setting FE-Legal

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

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


[Bug 492165] Review Request: rotoscope - A free rotoscoping application.

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks|177841(FE-NEEDSPONSOR)  |201449(FE-DEADREVIEW)
 Resolution||NOTABUG
   Flag|needinfo?(yansta...@googlem |
   |ail.com)|




--- Comment #6 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
12:37:55 EDT ---
Once closing.

If someone wants to import this package into Fedora, please
file a new review request and mark this bug as a duplicate
of the new one.

Thank you!

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

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


[Bug 492094] Review Request: darksnow - GTK Interface for darkice

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


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





--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
12:38:30 EDT ---
ping again?

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

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


[Bug 489598] Review Request: codelite - a powerful open-source, cross platform code editor for C/C++

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


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





--- Comment #11 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
12:39:04 EDT ---
ping again, someone?

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

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


[Bug 493479] Review Request: woodardworks-laconic-fonts - An artistic and minimal sans-serif font family

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


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


Nicolas Mailhot nicolas.mail...@laposte.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|nicolas.mail...@laposte.net
   Flag||fedora-review?,
   ||needinfo?(tcall...@redhat.c
   ||om)




--- Comment #1 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-05-21 
12:39:07 EDT ---
Seems I'm a slacker at heart. Sorry for the delay. Anyway:

1. the source is in macosx zip format. Since the people in charge of rpm and
unzip refused to fix their support for osx zip archives, you need to perform
the un-zipping manually to avoid leaving a stray __MACOSX directory behind in
the build root (annoys people doing full-distro rebuilds). Use something like:

%setup -q -c -T
unzip -j -L -q %{SOURCE0}

(the gfs font packages have the same problem)

2. Microsoft considers faces in modern font families should only differ on
weight, width or slope) and a lot of software is confused by anything else.
Therefore, the shadow face should be split in a
woodardworks-laconic-shadow-fonts subpackage IMHO, making it a multi-family
package.

3. the font is definitely sans-serif, your fontconfig file should declare it
there

The rest of the packaging seems fine

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

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


[Bug 473046] Review Request: miniupnpc - command line tool to control NAT in UPnP-enabled routers as Linksys, D-Link etc

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


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





--- Comment #24 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
12:41:11 EDT ---
ping again?

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

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


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

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks|177841(FE-NEEDSPONSOR)  |201449(FE-DEADREVIEW)
 Resolution||NOTABUG
   Flag|needinfo?(a...@unix.sh)  |




--- Comment #7 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
12:40:18 EDT ---
Once closing.

If someone wants to import this package into Fedora, please
file a new review request and mark this bug as a duplicate
of the new one.

Thank you!

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

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


[Bug 502054] Review Request: perl-Lingua-Flags - Provide small flag icons

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


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





--- Comment #1 from Iain Arnell iarn...@gmail.com  2009-05-21 12:43:36 EDT ---
To quote https://fedoraproject.org/wiki/Package_Maintainers_Flags_Policy

EOF
When a package contains flag images that are used (in the user interface, in
documentation, etc.) to represent locations, countries, nations, other kinds of
geopolitical entities, languages, other kinds of ethnocultural concepts,
religions, political movements and institutions, and the like, and where such
use is not technically or substantively essential to the package, those flag
images must be placed in an -flags subpackage. The -flags subpackage cannot be
Required by the main package. 
EOF

This package may contain flag images, but they are not used ... to represent
anything at all. 

And even if they were used, I'm pretty sure that such use is both technically
and substantively essential to this package.

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

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


[Bug 467324] Review Request: mingw32-portablexdr - MinGW Windows PortableXDR XDR / RPC library

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


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





--- Comment #16 from Peter Robinson pbrobin...@gmail.com  2009-05-21 13:05:22 
EDT ---
Is this covered under the Sun re-licensing that also affected other RPC
implementations in Fedora?

http://www.redhat.com/archives/fedora-devel-list/2009-May/msg01673.html

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

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


[Bug 495399] Review Request: NotCourier-Sans-sfd-fonts - NotCourier Sans is a re-interpretation of Nimbus Mono.

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


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


Nicolas Mailhot nicolas.mail...@laposte.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|nicolas.mail...@laposte.net
   Flag||fedora-review?,
   ||needinfo?(sanjay.an...@gmai
   ||l.com)




--- Comment #3 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-05-21 
13:18:59 EDT ---
First review:

1. W: file-not-utf8
/usr/share/doc/NotCourier-Sans-sfd-fonts-20080906/FONTLOG.txt
⇒ please use iconv to fix this file's encoding (you have an example in sil
andika, make sure you've the right source encoding though)

2. please use lowercase package naming only as requested by the fonts packaging
guidelines

3. please add the oflb- foundry prefix to the package name

4. unless the fonts authors asked you to, I don't thing the -sfd postfix is
really needed

5. your URL does not work

6. it seems upstreams versions this font properly, so you can use a real
version instead of a timestamp (current is 1.1)

7. I'd drop the first § of your description

8. the font is derived from a GPL font so it must stay GPL. Upstream has
already fixed this in its last release (do not forget to update your %doc)

9. Be very careful to use the exact name the font declares in your fontconfig
file. For version 1.1, it will be NotCourierSans

10. since the fonts are missing an italic face, I wouldn't stack them at 60 in
fontconfig, 61 or 62 would be better

Anyway that's all for now, thank you for continuing to package Fedora fonts

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

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


[Bug 502065] New: Review Request: slashem - Super Lotsa Added Stuff Hack - Extended Magic

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

Summary: Review Request: slashem - Super Lotsa Added Stuff Hack - Extended Magic
Alias: slashem

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

   Summary: Review Request: slashem - Super Lotsa Added Stuff Hack
- Extended Magic
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://slashem.sourceforge.net/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~iarnell/review/slashem.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/slashem-0.0.8-0.1.E0F1.fc12.src.rpm

Description:
From the land before 3DFX, before VGA graphics and DOOM, before the IBM PC, way
back in the dark ages of Unixland, there was a game. They called it Rogue.
People played it, and found it good. From this basis, Hack was born. Soon Hack
became Nethack, because it was developed by many people (and has nothing to do
with hacking the internet). And people played this on many machines, from
Unices to Macs to PCs, due to the amazing power of Open Source Code.

But the DevTeam, the reclusive masterminds of Nethack, are a rather quiet
bunch, gracing the world with new versions as they see fit, and when they see
fit. Which is usually a new version every good number of years.

And there was much gnashing of teeth.

But because of the Freely Available Source Code Phenomenon, people began making
their own versions of Nethack to tide themselves between magical releases.

SLASH'EM is the (continuing) saga of one such variant...

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1368974

*rt-0.09

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

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


[Bug 467324] Review Request: mingw32-portablexdr - MinGW Windows PortableXDR XDR / RPC library

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


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





--- Comment #17 from Tom spot Callaway tcall...@redhat.com  2009-05-21 
13:43:47 EDT ---
Not at the moment, we had to request the license changes on a per package, per
file basis from Sun, and the list we sent them was before I was aware of this
one. We have already sent them an additional list of the affected files in this
package and requested the same relicensing, but we have not yet heard back.

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

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


[Bug 499975] Review Request: guitarix - Mono amplifier to JACK

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


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





--- Comment #1 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-05-21 
13:43:14 EDT ---
Spec URL: http://oget.fedorapeople.org/review/guitarix.spec
SRPM URL: http://oget.fedorapeople.org/review/guitarix-0.04.4-1.fc10.src.rpm

Changelog: 0.04.4-1
- Update to 0.04.4
- Drop upstreamed patches

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

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


[Bug 495399] Review Request: NotCourier-Sans-sfd-fonts - NotCourier Sans is a re-interpretation of Nimbus Mono.

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


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


Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

   Flag|needinfo?(sanjay.an...@gmai |
   |l.com)  |




--- Comment #4 from Ankur Sinha sanjay.an...@gmail.com  2009-05-21 13:52:45 
EDT ---
(In reply to comment #3)
 First review:
 
 1. W: file-not-utf8
 /usr/share/doc/NotCourier-Sans-sfd-fonts-20080906/FONTLOG.txt
 ⇒ please use iconv to fix this file's encoding (you have an example in sil
 andika, make sure you've the right source encoding though)
 
 2. please use lowercase package naming only as requested by the fonts 
 packaging
 guidelines
 
 3. please add the oflb- foundry prefix to the package name
 
 4. unless the fonts authors asked you to, I don't thing the -sfd postfix is
 really needed
 
 5. your URL does not work
 
 6. it seems upstreams versions this font properly, so you can use a real
 version instead of a timestamp (current is 1.1)
 
 7. I'd drop the first § of your description
 
 8. the font is derived from a GPL font so it must stay GPL. Upstream has
 already fixed this in its last release (do not forget to update your %doc)
 
 9. Be very careful to use the exact name the font declares in your fontconfig
 file. For version 1.1, it will be NotCourierSans
 
 10. since the fonts are missing an italic face, I wouldn't stack them at 60 in
 fontconfig, 61 or 62 would be better
 
 Anyway that's all for now, thank you for continuing to package Fedora fonts  

bah... they edited it all on may 17.. I'll redo the package..

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

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


[Bug 493250] Review Request: perl-Goo-Canvas -- Goo::Canvas Perl interface to the GooCanvas

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


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





--- Comment #20 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-05-21 
14:01:25 EDT ---
Well

* Please address the issues of
  - directory ownership (mentioned in my comment 4)
  - and changing tetris word on script name and
in documents (mentioned in spot's comment 5)

* About patching
---
patch -p0 -b --suffix .warning  %{SOURCE1}
---
  - This can be replaced by
---
%patch1 -p0 -b .warning
---

* Make build log more verbose
  - This package also shows build log like:
---
77  /usr/bin/perl -MExtUtils::Command -e mkpath blib/lib/Goo/Canvas
78  [ CC xs/goocanvas.c ]
79  [ CC xs/goocanvasbounds.c ]
80  [ CC xs/goocanvasellipse.c ]
81  [ CC xs/goocanvasgroup.c ]
---
which is not so useful.
Please make build.log more verbose, to show if Fedora specific
compilation flags are correctly honored, for example.
For this package the following works.
---
make %{?_smp_mflags} NOECHO=
---

And please change the release number.

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

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


[Bug 501924] Review Request: mingw32-tcl - MinGW Windows Tool Command Language

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


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





--- Comment #4 from Erik van Pienbroek erik-fed...@vanpienbroek.nl  
2009-05-21 14:38:17 EDT ---
(In reply to comment #3)
 Thanks for taking! That was quick!

I'm waiting for somebody to review my review requests, so it's best to do other
reviews in return :)

  For readability, you might want to move this piece of code to the top of the
  .spec file:
   # don't run make test by default
   %{?_without_check: %define _without_check 0}
   %{!?_without_check: %define _without_check 1}
 
 This is again from the native spec file. I kept it there to minimize the
 differences. So the %check section could probably completely go...

Yeah, I've also seen it in the native spec file. However, I still think such
pieces of code need to be near the top of .spec files as it helps people who
are manually rebuilding the package to find out there's an option to enable the
testsuite. Right now, it's hidden somewhere in the .spec file and easily
overlooked.

  Is the rename of the import libraries from .a to .dll.a really necessary?
  AFAIK, this is only needed for libtool based libraries (which tcl isn't)
 
 I don't think the renaming is strictly necessary. I did it to make it extra
 clear that the .a files are implibs, not static libraries...

Have you tried compiling TCL-based applications (or other libraries) against
this package to test whether the compiler can find the .dll.a file? (Normally
libtool takes care of that, but as TCL isn't libtool based it's best to verify
this)

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

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


[Bug 478743] Review Request: saga - SAGA is a free, hybrid, cross-platform GIS software

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


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


Jerry James loganje...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Blocks||201449(FE-DEADREVIEW)
 Resolution||NOTABUG
   Flag|fedora-review?, |
   |needinfo?(cristian.bal...@g |
   |mail.com)   |




--- Comment #7 from Jerry James loganje...@gmail.com  2009-05-21 17:01:06 EDT 
---
The submitter has not responded.  I am closing this bug.

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

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


[Bug 478918] Review Request: globus-gssapi-error - Globus Toolkit - GSSAPI Error Library

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


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


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

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




--- Comment #6 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-05-21 
17:01:15 EDT ---
I have reviewed the package. Everything seems good.

- rpmlint can be ignored:
   globus-gssapi-error-devel.x86_64: W: no-documentation

- koji rawhide build is fine:
   http://koji.fedoraproject.org/koji/taskinfo?taskID=1369237

--
This package (globus-gssapi-error) is APPROVED by oget
--

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

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


[Bug 472791] Review Request: fontbox - A Java library for parsing font files

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


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


Jerry James loganje...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Blocks||201449(FE-DEADREVIEW)
 Resolution||NOTABUG
   Flag|fedora-review?, |
   |needinfo?(mefos...@gmail.co |
   |m)  |




--- Comment #7 from Jerry James loganje...@gmail.com  2009-05-21 17:03:19 EDT 
---
The submitter has not responded.  I am closing this bug.

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

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


[Bug 502101] New: Review Request: agedu - An utility for tracking down wasted disk space

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

Summary: Review Request: agedu - An utility for tracking down wasted disk space

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

   Summary: Review Request: agedu - An utility for tracking down
wasted disk space
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jussi.leht...@iki.fi
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://theory.physics.helsinki.fi/~jzlehtol/rpms/agedu.spec

SRPM URL:
http://theory.physics.helsinki.fi/~jzlehtol/rpms/agedu-0-1.r8442.fc10.src.rpm

Upstream URL: http://www.chiark.greenend.org.uk/~sgtatham/agedu/

Description:
Agedu is a program that helps you to track down wasted disk space by creating
a graphical representation of last access times and occupied disk space of
files and directories.

rpmlint output is clean.

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

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


[Bug 498490] Review Request: smem - Reports application memory usage in a meaningful way

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


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





--- Comment #22 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 17:20:54 EDT ---
smem-0.1-4.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/smem-0.1-4.fc11

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

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


[Bug 498490] Review Request: smem - Reports application memory usage in a meaningful way

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


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





--- Comment #23 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 17:29:51 EDT ---
smem-0.1-4.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/smem-0.1-4.fc10

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

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


[Bug 498490] Review Request: smem - Reports application memory usage in a meaningful way

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


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


Michal Schmidt mschm...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #24 from Michal Schmidt mschm...@redhat.com  2009-05-21 17:32:34 
EDT ---
Package imported, built on devel, F-11 and F-10, updates submitted. Closing
this review. Thanks everyone.

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

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


[Bug 478918] Review Request: globus-gssapi-error - Globus Toolkit - GSSAPI Error Library

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


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


Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #7 from Mattias Ellert mattias.ell...@fysast.uu.se  2009-05-21 
17:39:58 EDT ---
Thank you for the review.


New Package CVS Request
===
Package Name: globus-gssapi-error
Short Description: Globus Toolkit - GSSAPI Error Library
Owners: ellert
Branches: F-9 F-10 F-11 EL-4 EL-5
InitialCC:

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

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


[Bug 500713] Review Request: 389-ds-base - renamed from fedora-ds-base

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


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





--- Comment #5 from Rich Megginson rmegg...@redhat.com  2009-05-21 18:00:27 
EDT ---
389-ds-base-devel.i386: W: only-non-binary-in-usr-lib
- This is the only remaining warning.  I believe this is because the devel
package installs two shared library symlinks in /usr/lib/dirsrv:
/usr/lib/dirsrv/libslapd.so
/usr/lib/dirsrv/libns-dshttpd.so

This is the usual convention for -devel packages, to install symlinks called
.so that point to the real shared lib.  I don't know why rpmlint doesn't like
it, and I would like this error waived.

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

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


[Bug 502117] New: Review Request: mod_log_post - Module for the Apache web server to log all HTTP POST messages

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

Summary: Review Request: mod_log_post - Module for the Apache web server to log 
all HTTP POST messages

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

   Summary: Review Request: mod_log_post - Module for the Apache
web server to log all HTTP POST messages
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: redhat-bugzi...@linuxnetz.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://labs.linuxnetz.de/bugzilla/mod_log_post.spec
SRPM URL: http://labs.linuxnetz.de/bugzilla/mod_log_post-0.1.0-1.src.rpm
Description:
mod_log_post can be used for logging all HTTP POST messages. The module
is based on mod_security but in difference it never returns any error
messages to the visitors of your websites. Logging of POST data can be
very useful for debugging purposes or analyses. As the module is loaded
and run after the SSL decryption, it even can log POST data transmitted
before via SSL to the Apache web server.

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

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


[Bug 478759] Review Request: perl-SystemPerl - SystemPerl Perl module

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


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





--- Comment #19 from Brennan Ashton bash...@brennanashton.com  2009-05-21 
19:14:27 EDT ---
I will evaluate this new package tonight.

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

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


[Bug 499132] Review Request: gnome-applet-alarm-clock - Alarm clock for your GNOME panel

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


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





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:24:14 EDT ---
gnome-applet-alarm-clock-0.2.5-1.fc11 has been pushed to the Fedora 11 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 496718] Review Request: mingw32-plotmm - MinGW GTKmm plot widget for scientific applications

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


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





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:23:13 EDT ---
mingw32-plotmm-0.1.2-2.fc11 has been pushed to the Fedora 11 stable repository.
 If problems still persist, please make note of it in this bug report.

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

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


[Bug 499132] Review Request: gnome-applet-alarm-clock - Alarm clock for your GNOME panel

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


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


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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||0.2.5-1.fc11
 Resolution||NEXTRELEASE




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

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


[Bug 497688] Review Request: php-pecl-memcached - Extension to work with the Memcached caching daemon

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


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


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

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:24:31 EDT ---
php-pecl-memcached-0.1.5-1.fc10 has been pushed to the Fedora 10 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update php-pecl-memcached'.  You can
provide feedback for this update here:
http://admin.fedoraproject.org/updates/F10/FEDORA-2009-5290

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

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


[Bug 499125] Review Request: glista - Simple personal to-do list manager or task tracking tool

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


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


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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||0.4-2.fc10
 Resolution||NEXTRELEASE




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

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


[Bug 498492] Review Request: lxshortcut - Small utility to edit application shortcuts

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


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





--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:24:00 EDT ---
lxshortcut-0.1-1.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 499125] Review Request: glista - Simple personal to-do list manager or task tracking tool

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


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





--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:23:24 EDT ---
glista-0.4-2.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 467239] Review Request: globus-gss-assist - Globus Toolkit - GSSAPI Assist library

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


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





--- Comment #15 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:26:47 EDT ---
globus-gss-assist-4.0-1.fc11 has been pushed to the Fedora 11 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 499279] Review Request: xfce4-cddrive-plugin - Xfce panel plugin to open or close a CD-ROM drive tray

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


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





--- Comment #5 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:25:10 EDT ---
xfce4-cddrive-plugin-0.0.1-1.fc11 has been pushed to the Fedora 11 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 499281] Review Request: xfce4-cellmodem-plugin - Cell Modem monitor plugin for the Xfce panel

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


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





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:26:33 EDT ---
xfce4-cellmodem-plugin-0.0.5-1.fc10 has been pushed to the Fedora 10 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 499281] Review Request: xfce4-cellmodem-plugin - Cell Modem monitor plugin for the Xfce panel

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


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


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

   What|Removed |Added

   Fixed In Version|0.0.5-1.fc10|0.0.5-1.fc11




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

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


[Bug 496718] Review Request: mingw32-plotmm - MinGW GTKmm plot widget for scientific applications

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


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


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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||0.1.2-2.fc11
 Resolution||NEXTRELEASE




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

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


[Bug 499281] Review Request: xfce4-cellmodem-plugin - Cell Modem monitor plugin for the Xfce panel

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


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


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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||0.0.5-1.fc10
 Resolution||NEXTRELEASE




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

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


[Bug 467239] Review Request: globus-gss-assist - Globus Toolkit - GSSAPI Assist library

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


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


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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||4.0-1.fc10
 Resolution||NEXTRELEASE




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

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


[Bug 498130] Review Request: lxinput - Keyboard and mouse settings dialog for LXDE

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


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





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:25:21 EDT ---
lxinput-0.1-1.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 498130] Review Request: lxinput - Keyboard and mouse settings dialog for LXDE

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


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





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:26:17 EDT ---
lxinput-0.1-1.fc11 has been pushed to the Fedora 11 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 499282] Review Request: xfce4-notifyd - Simple notification daemon for Xfce

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


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





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:28:27 EDT ---
xfce4-notifyd-0.1.0-2.fc11 has been pushed to the Fedora 11 stable repository. 
If problems still persist, please make note of it in this bug report.

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

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


[Bug 499125] Review Request: glista - Simple personal to-do list manager or task tracking tool

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


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





--- Comment #13 from Fedora Update System upda...@fedoraproject.org  
2009-05-21 19:27:28 EDT ---
glista-0.4-2.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


  1   2   >