[Bug 497720] Review Request: hunspell-ln - Lingala hunspell dictionaries

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


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


Caolan McNamara  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 497827] Review Request: shortrpm - Short-circuit binary RPM package build

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


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


Lubomir Rintel  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Lubomir Rintel   2009-04-29 03:24:07 EDT ---
Hm, testsuite was failing since %a seems to be ignored with most recent glibcs
even when the source is not an ISO C source, contrary to what was documented.
The package was updated to 1.1, imported and built.

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

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


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

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


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





--- Comment #3 from Parag AN(पराग)   2009-04-29 03:35:13 
EDT ---
Argh! my bad. I forgot to remove them from SPEC while testing and installing
this package locally.

Please add Requires: 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 478918] Review Request: globus-gssapi-error - Globus Toolkit - GSSAPI Error Library

2009-04-29 Thread bugzilla
Please do not reply directly 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  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |gssapi-error - Globus   |globus-gssapi-error -
   |Toolkit - GSSAPI Error  |Globus Toolkit - GSSAPI
   |Library |Error Library




--- Comment #4 from Mattias Ellert   2009-04-29 
03:35:29 EDT ---
The upstream name of this package (gssapi-error) fails the "to common name"
rule. Therefore I have changed it to have the globus- prefix like all the other
globus packages. Patch for this is submitted upstream.

New version:

http://www.grid.tsl.uu.se/repos/globus/info/new/globus-gssapi-error-2.5-2.fc10.src.rpm
http://www.grid.tsl.uu.se/repos/globus/info/new/globus-gssapi-error.spec

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

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


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

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


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


Iain Arnell  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 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

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


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





--- Comment #4 from Iain Arnell   2009-04-29 03:39:50 EDT ---
New Package CVS Request
===
Package Name: perl-App-Nopaste
Short Description: Easy access to any pastebin
Owners: iarnell
Branches: F-10 F-11
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 497660] Review Request: perl-MIME-Charset - Charset Informations for MIME

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


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


Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Parag AN(पराग)   2009-04-29 04:03:27 
EDT ---
APPROVED.

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

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


[Bug 497660] Review Request: perl-MIME-Charset - Charset Informations for MIME

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


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


Xavier Bachelot  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Xavier Bachelot   2009-04-29 04:06:44 
EDT ---
Thanks.

New Package CVS Request
===
Package Name: perl-MIME-Charset
Short Description: Charset Informations for MIME
Owners: xavierb
Branches: F-9 F-10 F-11 EL-5
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 494820] Review Request: python-pytools - A collection of tools for python

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


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


Ramakrishna Reddy  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 497682] Review Request: libmemcached - Client library and command line tools for memcached server

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


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





--- Comment #7 from Remi Collet   2009-04-29 04:09:33 
EDT ---
 Response from Brian
Thanks for the heads up. I'll look into adding a configure switch to disable
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 497991] Review Request: hunspell-as - Assamese hunspell dictionaries

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


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


Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #8 from Parag AN(पराग)   2009-04-29 04:14:45 
EDT ---
Built for F-11 and devel.

-- 
Configure 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 494819] Review Request: python-bitarray - Efficient Array of Booleans --C Extensions

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


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


Ramakrishna Reddy  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 495021] Review Request: python-posix_ipc - POSIX IPC primitives for Python

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


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


Ramakrishna Reddy  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 494853] Review Request: python-achoo - A fluent interface for testing Python objects

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


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


Ramakrishna Reddy  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 453855] Review Request: globus-openssl-module - Globus Toolkit - Globus OpenSSL Module Wrapper

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


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





--- Comment #16 from Fedora Update System   
2009-04-29 04:47:41 EDT ---
globus-openssl-module-0.6-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/globus-openssl-module-0.6-1.fc9

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

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


[Bug 453855] Review Request: globus-openssl-module - Globus Toolkit - Globus OpenSSL Module Wrapper

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


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





--- Comment #15 from Fedora Update System   
2009-04-29 04:47:33 EDT ---
globus-openssl-module-0.6-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/globus-openssl-module-0.6-1.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 491618] Review Request: mingw32-libsqlite3x - MinGW Windows C++ wrapper for the sqlite database library

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


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


Mattias Ellert  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #5 from Mattias Ellert   2009-04-29 
05:27:38 EDT ---
Do you still need BuildRequires: doxygen?

Anyway, that you can check post-review.

Package 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 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

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


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


Mattias Ellert  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se
   Flag||fedora-review?




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

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


[Bug 498029] Review Request: kio-ftps - An ftps KIO slave for KDE

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


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





--- Comment #2 from Jiri Reischig   2009-04-29 06:05:02 
EDT ---
(In reply to comment #0)

I try compile and install the package and it's working good.
Thank you very much.
I hope that the package will be included in the next Fedora version.

-- 
Configure 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 453857] Review Request: globus-gsi-sysconfig - Globus Toolkit - Globus GSI System Config Library

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


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


Thomas Sailer  changed:

   What|Removed |Added

 CC||t.sai...@alumni.ethz.ch
 AssignedTo|nob...@fedoraproject.org|t.sai...@alumni.ethz.ch

Bug 453857 depends on bug 453850, which changed state.

Bug 453850 Summary: Review Request: globus-openssl - Openssl Library (virtual 
GPT glue package)
https://bugzilla.redhat.com/show_bug.cgi?id=453850

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

Bug 453857 depends on bug 453853, which changed state.

Bug 453853 Summary: Review Request: globus-gsi-openssl-error - Globus Toolkit - 
Globus OpenSSL Error Handling
https://bugzilla.redhat.com/show_bug.cgi?id=453853

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



--- Comment #5 from Thomas Sailer   2009-04-29 
06:02:24 EDT ---
Sources match upstream:
$ diff -urN globus_gsi_sysconfig-2.2/ globus_gsi_sysconfig-2.2-dl/
is silent.

Scratch Build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1328247

$ rpmlint globus-gsi-sysconfig*.rpm
globus-gsi-sysconfig-devel.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

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

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


[Bug 498029] Review Request: kio-ftps - An ftps KIO slave for KDE

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


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





--- Comment #3 from Jiri Reischig   2009-04-29 06:10:55 
EDT ---
(In reply to comment #0)

Please, do you know how can I insert to the KDE new CA key or site selfsigned
key which will be acepted in this kio-ftps?

-- 
Configure 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 492259] Review Request: tucan - Manager for downloads and uploads at hosting sites

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


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


Simon Wesp  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #11 from Simon Wesp   2009-04-29 
06:11:52 EDT ---
thank you for your review!

New Package CVS Request
===
Package Name: tucan
Short Description: Manager for downloads and uploads at hosting sites
Owners: cassmodiah
Branches: F-11 F-10
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 453857] Review Request: globus-gsi-sysconfig - Globus Toolkit - Globus GSI System Config Library

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


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





--- Comment #6 from Thomas Sailer   2009-04-29 
06:08:12 EDT ---
Fedora review globus-gsi-sysconfig-2.2-1.fc10.src.rpm 2009-04-29

+ OK
! needs attention

+ rpmlint output
  See above; the "no documentation" warning can be ignored.
+ Package is named according to Fedora Package Naming Guidelines
+ Specfile name matches the package base name
+ Package follows the Fedora packaging guidelines
+ License meets guidelines and is acceptable to Fedora
  ASL 2.0
+ License matches the actual package license
+ The package contains the license file
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match sources in the srpm
the source repacking makes it impossible to compare md5sum's, thus:
$ diff -urN globus_gsi_sysconfig-2.2/ globus_gsi_sysconfig-2.2-dl/
reports no differences

+ Package scratch-builds
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
n/a %find_lang instead of %{_datadir}/locale/*
+ binary RPM with shared library files must call ldconfig in %post and
%postun
+ Does not use Prefix: /usr
+ Package owns all directories it creates
+ No duplicate files in %files
+ %files has %defattr
+ %clean contains rm -rf $RPM_BUILD_ROOT
+ Consistent use of macros
+ Package must contain code or permissible content
+ Large documentation files should go in -doc subpackage
n/a Files marked %doc should not affect package
+ Header files should be in -devel
n/a Static libraries should be in -static
+ Packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
  indirectly, via R: globus-core
+ libfoo.so must go in -devel
+ -devel must require the fully versioned base
+ Packages should not contain libtool .la files
n/a Packages containing GUI apps must include %{name}.desktop file
+ Packages must not own files or directories owned by other packages
+ %install begins with rm -rf $RPM_BUILD_ROOT
+ Filenames must be valid UTF-8

APPROVED by sailer.

-- 
Configure 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-04-29 Thread bugzilla
Please do not reply directly 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


Thomas Sailer  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Thomas Sailer   2009-04-29 
06:15:37 EDT ---
Thank you for the review.


New Package CVS Request
===
Package Name: mingw32-plotmm
Short Description:  MinGW GTKmm plot widget for scientific applications
Owners: sailer rjones
Branches: 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 491618] Review Request: mingw32-libsqlite3x - MinGW Windows C++ wrapper for the sqlite database library

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


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


Thomas Sailer  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 491618] Review Request: mingw32-libsqlite3x - MinGW Windows C++ wrapper for the sqlite database library

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


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





--- Comment #6 from Thomas Sailer   2009-04-29 
06:16:19 EDT ---
Thank you for the review.


New Package CVS Request
===
Package Name: mingw32-libsqlite3x
Short Description:  MinGW Windows C++ wrapper for the sqlite database
Owners: sailer rjones
Branches: 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 226108] Merge Review: lsof

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


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


Karel Zak  changed:

   What|Removed |Added

   Flag|needinfo?(k...@redhat.com)  |




--- Comment #8 from Karel Zak   2009-04-29 06:21:19 EDT ---
Michal, thanks for your review. I'll fix all in the next rawhide update
(probably together with some others issues).

-- 
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 494148] Review Request: soci - The database access library for C++ programmers

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


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


Michael Schwendt  changed:

   What|Removed |Added

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




--- Comment #22 from Michael Schwendt   2009-04-29 
06:21:57 EDT ---
> soci-3.0.0-5.fc10.src.rpm

APPROVED

So you can squash remaining issues in pkg cvs, which will be more convenient
than going through the upload-bugzilla-review cycle.

[...]

With a caveat, though, as the conditional build options don't seem to work
[yet]:

configure: WARNING: unrecognized options: --disable-backend-empty,
--disable-backend-sqlite3

And although it recognises the other --disable-backend options, it ignores them
and still builds various backends if the needed -devel pkgs are installed. E.g.
--without mysql ends with RPM build errors:
Installed (but unpackaged) file(s) found:
   /usr/include/soci/backends/mysql/soci-mysql.h
   /usr/lib/libsoci_mysql.so
   /usr/lib/libsoci_mysql.so.3
   /usr/lib/libsoci_mysql.so.3.0.0

Unless you can disable the backends at "configure"-time, you could add
"BuildConflicts: mysql-devel" to enforce uninstalling unwanted -devel pkgs. ;o)

[...]

> %define mydocs __tmp_docdir

Not used anywhere.


* The many %{?with_foo:...} switches throughout the spec file could be grouped
together and could be replaced with a much more readable %if/%endif block that
covers everything for each conditional subpkg except the %build/%install
sections. In other words, for e.g. MySQL you would end up with:

%if %{with mysql}
%package mysql
...
%description mysql
...
%files mysql
...
%post mysql -p /sbin/ldconfig
%postun mysql -p /sbin/ldconfig
...
%package mysql-devel
...
%description mysql-devel
...
%files mysql-devel
...
%endif

-- 
Configure 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 453857] Review Request: globus-gsi-sysconfig - Globus Toolkit - Globus GSI System Config Library

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


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


Thomas Sailer  changed:

   What|Removed |Added

   Flag||fedora-review+




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

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


[Bug 453856] Review Request: globus-gsi-cert-utils - Globus Toolkit - Globus GSI Cert Utils Library

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


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


Thomas Sailer  changed:

   What|Removed |Added

 CC||t.sai...@alumni.ethz.ch
 AssignedTo|nob...@fedoraproject.org|t.sai...@alumni.ethz.ch

Bug 453856 depends on bug 453850, which changed state.

Bug 453850 Summary: Review Request: globus-openssl - Openssl Library (virtual 
GPT glue package)
https://bugzilla.redhat.com/show_bug.cgi?id=453850

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

Bug 453856 depends on bug 453853, which changed state.

Bug 453853 Summary: Review Request: globus-gsi-openssl-error - Globus Toolkit - 
Globus OpenSSL Error Handling
https://bugzilla.redhat.com/show_bug.cgi?id=453853

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



--- Comment #5 from Thomas Sailer   2009-04-29 
06:19:36 EDT ---
Sources match upstream:
48987f39031ac79949a9b0913aa42c67  ORIG/globus_gsi_cert_utils-5.5.tar.gz
48987f39031ac79949a9b0913aa42c67  SOURCES/globus_gsi_cert_utils-5.5.tar.gz

$ rpmlint globus-gsi-cert-utils*.rpm globus-gsi-cert-utils.spec
globus-gsi-cert-utils.x86_64: W: no-documentation
globus-gsi-cert-utils-devel.x86_64: W: no-documentation
globus-gsi-cert-utils-progs.x86_64: W: no-documentation
5 packages and 1 specfiles checked; 0 errors, 3 warnings.

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

-- 
Configure 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 453856] Review Request: globus-gsi-cert-utils - Globus Toolkit - Globus GSI Cert Utils Library

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


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





--- Comment #6 from Thomas Sailer   2009-04-29 
06:25:10 EDT ---
Fedora review globus-gsi-cert-utils-5.5-1.fc10.src.rpm 2009-04-29

+ OK
! needs attention

+ rpmlint output
  See above; the "no documentation" warning can be ignored. Source Tarball does
not contain a license file
+ Package is named according to Fedora Package Naming Guidelines
+ Specfile name matches the package base name
+ Package follows the Fedora packaging guidelines
+ License meets guidelines and is acceptable to Fedora
  ASL 2.0
+ License matches the actual package license
n/a The package contains the license file
  none shipped by upstream tarball
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match sources in the srpm
  48987f39031ac79949a9b0913aa42c67  ORIG/globus_gsi_cert_utils-5.5.tar.gz
  48987f39031ac79949a9b0913aa42c67  SOURCES/globus_gsi_cert_utils-5.5.tar.gz
+ Package scratch-builds
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
n/a %find_lang instead of %{_datadir}/locale/*
+ binary RPM with shared library files must call ldconfig in %post and
%postun
+ Does not use Prefix: /usr
+ Package owns all directories it creates
+ No duplicate files in %files
+ %files has %defattr
+ %clean contains rm -rf $RPM_BUILD_ROOT
+ Consistent use of macros
+ Package must contain code or permissible content
+ Large documentation files should go in -doc subpackage
n/a Files marked %doc should not affect package
+ Header files should be in -devel
n/a Static libraries should be in -static
+ Packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
  indirectly, via R: globus-core
+ libfoo.so must go in -devel
+ -devel must require the fully versioned base
+ Packages should not contain libtool .la files
n/a Packages containing GUI apps must include %{name}.desktop file
+ Packages must not own files or directories owned by other packages
+ %install begins with rm -rf $RPM_BUILD_ROOT
+ Filenames must be valid UTF-8

APPROVED by sailer.

-- 
Configure 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 453856] Review Request: globus-gsi-cert-utils - Globus Toolkit - Globus GSI Cert Utils Library

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


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


Thomas Sailer  changed:

   What|Removed |Added

   Flag||fedora-review+




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

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


[Bug 453856] Review Request: globus-gsi-cert-utils - Globus Toolkit - Globus GSI Cert Utils Library

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


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


Mattias Ellert  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-cvs?




--- Comment #7 from Mattias Ellert   2009-04-29 
06:43:00 EDT ---
Thank you for the review.


New Package CVS Request
===
Package Name: globus-gsi-cert-utils
Short Description: Globus Toolkit - Globus GSI Cert Utils 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 453857] Review Request: globus-gsi-sysconfig - Globus Toolkit - Globus GSI System Config Library

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


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


Mattias Ellert  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-cvs?




--- Comment #7 from Mattias Ellert   2009-04-29 
06:43:03 EDT ---
Thank you for the review.


New Package CVS Request
===
Package Name: globus-gsi-sysconfig
Short Description: Globus Toolkit - Globus GSI System Config 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 497807] Review Request: perl-Capture-Tiny - Capture stdin or stdout from Perl, XS or external programs

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


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


Marcela Maslanova  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CURRENTRELEASE




-- 
Configure 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 498186] New: Review Request: perl-JavaScript-Beautifier - beautifier for javascript

2009-04-29 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-JavaScript-Beautifier - beautifier for javascript

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

   Summary: Review Request: perl-JavaScript-Beautifier -
beautifier for javascript
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://mmaslano.fedorapeople.org/review/perl-JavaScript-Beautifier.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-JavaScript-Beautifier-0.04-1.fc11.src.rpm
Description: This module is mostly a Perl-rewrite of
http://github.com/einars/js-
beautify/tree/master/beautify.js


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

-- 
Configure 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 497947] Review Request: libmetalink - A Metalink C library

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


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


Jussi Lehtola  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




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

___
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-04-29 Thread bugzilla
Please do not reply directly 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


Jussi Lehtola  changed:

   What|Removed |Added

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




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

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


[Bug 497947] Review Request: libmetalink - A Metalink C library

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


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


Jussi Lehtola  changed:

   What|Removed |Added

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




--- Comment #3 from Jussi Lehtola   2009-04-29 07:43:05 
EDT ---
A few notes:

- .la files need to be removed, add
 find . -name *.la -exec rm {} \;
to the end of the install phase

- if possible, static library build should be disable (usually %configure
--disable-static)

- devel package needs to Requires: pkgconfig

- you're running autoreconf, but not BRing the packages => the package will not
build in mock. Autoreconf'ing is frowned upon, and should not be done unless it
is specifically necessary.

-- 
Configure 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 498188] New: Review Request: perl-JavaScript-Minifier - Perl extension for minifying JavaScript code

2009-04-29 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-JavaScript-Minifier - Perl extension for 
minifying JavaScript code

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

   Summary: Review Request: perl-JavaScript-Minifier - Perl
extension for minifying JavaScript code
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mmaslano.fedorapeople.org/review/perl-JavaScript-Minifier.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-JavaScript-Minifier-1.05-1.fc11.src.rpm
Description:
This module removes unnecessary whitespace from JavaScript code. The
primary requirement developing this module is to not break working
code: if working JavaScript is in input then working JavaScript is
output. It is ok if the input has missing semi-colons, snips like '++
+' or '12 .toString()', for example. Internet Explorer conditional
comments are copied to the output but the code inside these comments
will not be minified.

-- 
Configure 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 498188] Review Request: perl-JavaScript-Minifier - Perl extension for minifying JavaScript code

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


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


Marcela Maslanova  changed:

   What|Removed |Added

 Blocks||498189




-- 
Configure 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 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

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


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


Mattias Ellert  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #1 from Mattias Ellert   2009-04-29 
07:57:15 EDT ---
Fedora review jorbis-0.0.17-1.fc10.src.rpm 2009-04-29

rpmlint output:

[ell...@ellert jorbis]$ rpmlint 1256512409979681881233819/result/jorbis/*.rpm
jorbis.spec 
jorbis.src:156: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/gcj/%{name}
jorbis.src:157: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/gcj/%{name}/%{name}-%{version}.jar.*
jorbis.src:170: W: libdir-macro-in-noarch-package player
%{_libdir}/gcj/%{name}/%{name}-player*.jar.*
jorbis.src:178: W: libdir-macro-in-noarch-package comment
%{_libdir}/gcj/%{name}/%{name}-comment*.jar.*
jorbis.spec:156: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/gcj/%{name}
jorbis.spec:157: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/gcj/%{name}/%{name}-%{version}.jar.*
jorbis.spec:170: W: libdir-macro-in-noarch-package player
%{_libdir}/gcj/%{name}/%{name}-player*.jar.*
jorbis.spec:178: W: libdir-macro-in-noarch-package comment
%{_libdir}/gcj/%{name}/%{name}-comment*.jar.*
6 packages and 1 specfiles checked; 0 errors, 8 warnings.

* OK
! needs attention

* rpmlint warnings are bogus - rpmlint does not understand the
  conditional BuildArch.

* Package is named according to the guidelines

* The specfile is named after the package

* The package follows the guidelines for a native Java package

* Package license is Fedora approved (LGPLv2+)

* The README file says that the license is LGPL with no version, but
  that is equivalent to LGPLv2+ according to the Fedora License list

* The license file (COPYING.LIB) in the sources is installed as %doc

* The specfile is written in legible English

* Sources matches upstream:

6141edfb16d34412a86c95a97fd4b10f  jorbis-0.0.17.zip
6141edfb16d34412a86c95a97fd4b10f  SRPM/jorbis-0.0.17.zip

* Package builds in mock (Fedora 10)

* BuildRequires are sane

* No shared libraries

* Package owns the directories it creates

! The jars in -comment and -player are also in main (duplicate files)

* Permissions are sane and %files has %defattr

* %clean clears buildroot

* Consistent use of macros

* Documentation in -javadoc package

* %doc is not runtime essential

* subpackages require main with fully versioned dependencies

* Package does not own other's directories

* %install clears buildroot

* Installed filenames are valid UTF-8

-- 
Configure 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 498189] New: Review Request: perl-JavaScript-Minifier-XS - XS based JavaScript minifier

2009-04-29 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-JavaScript-Minifier-XS - XS based JavaScript 
minifier

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

   Summary: Review Request: perl-JavaScript-Minifier-XS - XS based
JavaScript minifier
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
Depends on: 498188
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://mmaslano.fedorapeople.org/review/perl-JavaScript-Minifier-XS.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-JavaScript-Minifier-XS-0.05-1.fc11.src.rpm
Description:
JavaScript::Minifier::XS is a JavaScript "minifier"; its designed to remove
un-necessary whitespace and comments from JavaScript files, which also not
breaking the JavaScript.

-- 
Configure 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 498191] New: Review Request: perl-ExtUtils-InstallPAR - Install .par's into any installed perl

2009-04-29 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-ExtUtils-InstallPAR - Install .par's into any 
installed perl

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

   Summary: Review Request: perl-ExtUtils-InstallPAR - Install
.par's into any installed perl
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InstallPAR.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InstallPAR-0.03-1.fc11.src.rpm
Description:
This module installs PAR distributions (i.e. .par files) into any perl
installation on the system. The PAR::Dist module can install into the
currently running perl by default and provides the necessary parameters to
override any installation directories. Figuring out how to use those
overrides in order to install into an arbitrary perl installation on the
system may be beyond most users, however. Hence this convenience wrapper
using ExtUtils::InferConfig to automatically determine the typical site
installation paths of any perl interpreter than can be executed by the
current user.

-- 
Configure 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 472229] Review Request: PyQwt - Python bindings for Qwt

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


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





--- Comment #15 from Fedora Update System   
2009-04-29 08:08:06 EDT ---
PyQwt-5.1.0-4.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/PyQwt-5.1.0-4.fc9

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

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


[Bug 498130] Review Request: lxinput - Keyboard and mouse settings dialog for LXDE

2009-04-29 Thread bugzilla
Please do not reply directly 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


Jussi Lehtola  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #2 from Jussi Lehtola   2009-04-29 08:12:12 
EDT ---
Whoops, I was too fast:
SHOULD: The package builds in mock. OK

The package has been

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 498130] Review Request: lxinput - Keyboard and mouse settings dialog for LXDE

2009-04-29 Thread bugzilla
Please do not reply directly 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 #1 from Jussi Lehtola   2009-04-29 08:11:30 
EDT ---
rpmlint output is clean.


MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. OK
MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the 
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual license.
OK
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. OK
MUST: Optflags are used and time stamps preserved. OK
MUST: Packages containing shared library files must call ldconfig. OK
MUST: A package must own all directories that it creates or require the package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. OK
MUST: All relevant items are included in %doc. Items in %doc do not affect 
runtime of application. OK
MUST: Header files must be in a -devel package. OK
MUST: Static libraries must be in a -static package. OK
MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. OK
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. OK
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. OK
MUST: Packages does not contain any .la libtool archives. OK
MUST: Desktop files are installed properly. OK
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK

SHOULD: The package builds in mock.

-- 
Configure 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 497682] Review Request: libmemcached - Client library and command line tools for memcached server

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


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





--- Comment #8 from Tom "spot" Callaway   2009-04-29 
08:18:20 EDT ---
I meant %prep, not %pre. :) Sorry!

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

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


[Bug 498193] New: Review Request: perl-ExtUtils-InferConfig - Infer Perl Configuration for non-running interpreters

2009-04-29 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-ExtUtils-InferConfig - Infer Perl Configuration 
for non-running interpreters

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

   Summary: Review Request: perl-ExtUtils-InferConfig - Infer Perl
Configuration for non-running interpreters
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InferConfig.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InferConfig-1.03-1.fc11.src.rpm
Description:This module can determine the configuration and @INC of a perl
interpreter
given its path and that it is runnable by the current user.

-- 
Configure 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 498125] Review Request: lxde-settings-daemon - XSettings Daemon for LXDE

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


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


Jussi Lehtola  changed:

   What|Removed |Added

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




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

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


[Bug 498125] Review Request: lxde-settings-daemon - XSettings Daemon for LXDE

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


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





--- Comment #1 from Jussi Lehtola   2009-04-29 08:02:26 
EDT ---
- Require system-icon-theme instead of fedora-icon-theme, as one should be able
to remove Fedora brandings altogether.

- Is the obsoletes&provides really necessary since there AFAIK is not and
hasn't ever been a lxde-settings package in Fedora?

rpmlint output is clean.

MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. OK
MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the 
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual license.
OK
- lxde-settings-daemon.c is under GPLv2+, other files are under MIT.
- I'm not sure, however, if the MIT license needs to be mentioned since it's
compatible with GPLv2+.

MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. OK
MUST: Optflags are used and time stamps preserved. OK
MUST: Packages containing shared library files must call ldconfig. OK
MUST: A package must own all directories that it creates or require the package
that owns the directory. NEEDSFIX
- Package already Requires: lxde-common which provides /usr/share/lxde =>
remove
 %dir %{_datadir}/lxde/

MUST: Files only listed once in %files listings. OK
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. OK
MUST: All relevant items are included in %doc. Items in %doc do not affect 
runtime of application. OK
MUST: Header files must be in a -devel package. OK
MUST: Static libraries must be in a -static package. OK
MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. OK
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. OK
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. OK
MUST: Packages does not contain any .la libtool archives. OK
MUST: Desktop files are installed properly. OK
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The package builds in mock. OK

-- 
Configure 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 498196] New: Review Request: perl-WebService-Validator-CSS-W3C - Interface to the W3C CSS Validator

2009-04-29 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-WebService-Validator-CSS-W3C - Interface to the 
W3C CSS Validator

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

   Summary: Review Request: perl-WebService-Validator-CSS-W3C -
Interface to the W3C CSS Validator
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://mmaslano.fedorapeople.org/review/perl-WebService-Validator-CSS-W3C.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-WebService-Validator-CSS-W3C-0.2-1.fc11.src.rpm
Description:
This module is an interface to the W3C CSS Validation online service 
http://jigsaw.w3.org/css-validator/, based on its SOAP 1.2 support. 
It helps to find errors in Cascading Style Sheets.

-- 
Configure 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 498197] New: Review Request: zarafa-webaccess-mobile - Zarafa Webaccess Mobile is a simple web interface for mobile phones

2009-04-29 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: zarafa-webaccess-mobile - Zarafa Webaccess Mobile is a 
simple web interface for mobile phones

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

   Summary: Review Request: zarafa-webaccess-mobile - Zarafa
Webaccess Mobile is a simple web interface for mobile
phones
   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, tcall...@redhat.com,
fedora-package-review@redhat.com, kana...@kanarip.com,
fedora-jas...@newnewyork.nl
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: [Will come very soon]
SRPM URL: [Will come very soon]
Description:
The Zarafa Webaccess Mobile is a simple web interface for mobile phone
users unable to use the Zarafa Webaccess based on Ajax and whose mobile
phone is not supported via ActiveSync yet. View your e-mail, calendar
and contacts via your mobile device. Zarafa Webaccess Mobile has a text
interface only and allows users to create simple e-mails, appointments,
contacts and tasks.


I'll add the spec file, once I clarified some library dependency issues with
Jeroen or so. SRPM shipping is for now a legal problem as I'm testing with an
non-official release candidate, nevertheless, there are still some licensing 
issues with Zarafa, Tom is investigating into.

-- 
Configure 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 498194] New: Review Request: zarafa - Zarafa Outlook Sharing and Open Source Collaboration

2009-04-29 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: zarafa - Zarafa Outlook Sharing and Open Source 
Collaboration

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

   Summary: Review Request: zarafa - Zarafa Outlook Sharing and
Open Source Collaboration
   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, tcall...@redhat.com,
fedora-package-review@redhat.com, kana...@kanarip.com,
fedora-jas...@newnewyork.nl
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: [Will come very soon]
SRPM URL: [Will come very soon]
Description:
Zarafa Outlook Sharing is a Microsoft Exchange replacement. The Open Source
Collaboration provides an integration with your existing Linux mail server,
native mobile phone support by ActiveSync compatiblity and a webaccess with
'Look & Feel' similar to Outlook using Ajax. Including an IMAP4 and a POP3
gateway as well as an iCal/CalDAV gateway, Zarafa can combine the usability
with the stability and flexibility of a Linux server.

The proven Zarafa groupware solution is using MAPI objects, provides a MAPI
client library as well as programming interfaces for C++, PHP and Perl. The
other Zarafa related packages need to be installed to gain all the features
and benefits of Zarafa Outlook Sharing and Open Source Collaboration.


I'll add the spec file, once I clarified some library dependency issues with
Jeroen or so. SRPM shipping is for now a legal problem as I'm testing with an
non-official release candidate, nevertheless, there are still some licensing 
issues with Zarafa, Tom is investigating into.

-- 
Configure 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 498195] New: Review Request: zarafa-webaccess - Zarafa Webaccess featuring a 'Look & Feel' similar to Outlook

2009-04-29 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: zarafa-webaccess - Zarafa Webaccess featuring a 'Look 
& Feel' similar to Outlook

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

   Summary: Review Request: zarafa-webaccess - Zarafa Webaccess
featuring a 'Look & Feel' similar to Outlook
   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, tcall...@redhat.com,
fedora-package-review@redhat.com, kana...@kanarip.com,
fedora-jas...@newnewyork.nl
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: [Will come very soon]
SRPM URL: [Will come very soon]
Description:
Zarafa Webaccess features the familiar Outlook 'Look & Feel' interface
and you can keep using the features in Outlook that have always allowed
you to work efficiently. View your e-mail, calendar and contacts via a
web browser. And opening your colleagues calendar or sending a meeting
request is only a piece of cake. The Zarafa Webaccess is using the ajax
technology to give a more interactive feeling to the users.


I'll add the spec file, once I clarified some library dependency issues with
Jeroen or so. SRPM shipping is for now a legal problem as I'm testing with an
non-official release candidate, nevertheless, there are still some licensing 
issues with Zarafa, Tom is investigating into.

-- 
Configure 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 498198] New: Review Request: perl-WebService-Validator-HTML-W3C - Access the W3Cs online HTML validator

2009-04-29 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-WebService-Validator-HTML-W3C - Access the W3Cs 
online HTML validator

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

   Summary: Review Request: perl-WebService-Validator-HTML-W3C -
Access the W3Cs online HTML validator
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://mmaslano.fedorapeople.org/review/perl-WebService-Validator-HTML-W3C.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-WebService-Validator-HTML-W3C-0.24-1.fc11.src.rpm
Description: 
WebService::Validator::HTML::W3C provides access to the W3C's online Markup
validator. As well as reporting on whether a page is valid it also provides
access to a detailed list of the errors and where in the validated document
they occur.

-- 
Configure 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 498199] New: Review Request: dynamic-wallpaper - Generates svg wallpaper based on current weather, season and others

2009-04-29 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: dynamic-wallpaper - Generates svg wallpaper based on 
current weather, season and others

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

   Summary: Review Request: dynamic-wallpaper - Generates svg
wallpaper based on current weather, season and others
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: serein...@online.fr
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://tmp.sereinity.homelinux.org/rpm/dynamic-wallpaper.spec
SRPM URL:
http://tmp.sereinity.homelinux.org/rpm/dynamic-wallpaper-0.3-1.fc10.src.rpm
Description:
Dynamic wallpaper is based on weather-wallpaper and generates svg wallpaper
based on current weather, season, time of day and others. It supports themes.

-- 
Configure 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 484423] Review Request: netbook-launcher - A clutter-based desktop launcher

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


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





--- Comment #9 from Michel Alexandre Salim   
2009-04-29 08:45:06 EDT ---
Thanks! And sorry for the delay. I've been running 1.6.21 and most things work
(apart from, strangely, the Log Out button).

http://salimma.fedorapeople.org/specs/gnome/netbook-launcher.spec
http://salimma.fedorapeople.org/specs/gnome/netbook-launcher-1.6.21-1.fc11.src.rpm

Should we ask the legal team about the netbook trademark?

-- 
Configure 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 498200] Review Request: window-picker-applet - Window picker applet for GNOME

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


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


Michel Alexandre Salim  changed:

   What|Removed |Added

 Blocks||462851(FedoraMini)




-- 
Configure 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 451771] Review Request: maximus - an addon for metacity

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


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


Michel Alexandre Salim  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||michel.syl...@gmail.com




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

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


[Bug 451771] Review Request: maximus - an addon for metacity

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


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


Michel Alexandre Salim  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|michel.syl...@gmail.com




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

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


[Bug 498200] New: Review Request: window-picker-applet - Window picker applet for GNOME

2009-04-29 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: window-picker-applet - Window picker applet for GNOME

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

   Summary: Review Request: window-picker-applet - Window picker
applet for GNOME
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: michel.syl...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://salimma.fedorapeople.org/specs/gnome/window-picker-applet.spec
SRPM URL:
http://salimma.fedorapeople.org/specs/gnome/window-picker-applet-0.4.23-1.fc11.src.rpm
Description:

A gnome-panel applet that displays open windows as icons on the panel,
and has integrated window title-bar functionality. Optimised for use
on netbook-size screens.

-- 
Configure 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 484423] Review Request: netbook-launcher - A clutter-based desktop launcher

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


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


Michel Alexandre Salim  changed:

   What|Removed |Added

 Depends on||462851(FedoraMini)




-- 
Configure 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 484423] Review Request: netbook-launcher - A clutter-based desktop launcher

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


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


Michel Alexandre Salim  changed:

   What|Removed |Added

 Blocks||462851(FedoraMini)
 Depends on|462851(FedoraMini)  |




-- 
Configure 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-04-29 Thread bugzilla
Please do not reply directly 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


Christoph Wickert  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #3 from Christoph Wickert   2009-04-29 
08:58:53 EDT ---
New Package CVS Request
===
Package Name: lxinput
Short Description: Keyboard and mouse settings dialog for LXDE
Owners: cwickert
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 498203] New: Review Request: perl-YAML-LibYAML - YAML::LibYAML Perl module

2009-04-29 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-YAML-LibYAML - YAML::LibYAML Perl module

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

   Summary: Review Request: perl-YAML-LibYAML - YAML::LibYAML Perl
module
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mmaslano.fedorapeople.org/review/perl-YAML-LibYAML.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-YAML-LibYAML-0.32-1.fc11.src.rpm
Description:
Kirill Siminov's "libyaml" is arguably the best YAML
implementation. The C library is written precisely to the YAML 1.1
specification. It was originally bound to Python and was later
bound to Ruby.

-- 
Configure 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 498204] New: Review Request: perl-Wx-Perl-DataWalker - implement subclass that shows relatively simple structure

2009-04-29 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-Wx-Perl-DataWalker - implement subclass that 
shows relatively simple structure

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

   Summary: Review Request: perl-Wx-Perl-DataWalker - implement
subclass that shows relatively simple structure
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mmaslano.fedorapeople.org/review/perl-Wx-Perl-DataWalker.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-Wx-Perl-DataWalker-0.02-1.fc11.src.rpm
Description:
Wx::Perl::DataWalker implements a Wx::Frame subclass that shows a
relatively simple Perl data structure browser. After opening such a frame
and supplying it with a reference to an essentially arbitrary data
structure, you can visually browse it by double-clicking references.

-- 
Configure 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 498194] Review Request: zarafa - Zarafa Outlook Sharing and Open Source Collaboration

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


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


Robert Scheck  changed:

   What|Removed |Added

 Blocks||498195, 498196




-- 
Configure 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 498195] Review Request: zarafa-webaccess - Zarafa Webaccess featuring a 'Look & Feel' similar to Outlook

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


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


Robert Scheck  changed:

   What|Removed |Added

 Depends on||498194




-- 
Configure 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 498196] Review Request: perl-WebService-Validator-CSS-W3C - Interface to the W3C CSS Validator

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


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


Robert Scheck  changed:

   What|Removed |Added

 Depends on||498194




-- 
Configure 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 498194] Review Request: zarafa - Zarafa Outlook Sharing and Open Source Collaboration

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


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


Robert Scheck  changed:

   What|Removed |Added

 Blocks|498196  |498197




-- 
Configure 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 498196] Review Request: perl-WebService-Validator-CSS-W3C - Interface to the W3C CSS Validator

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


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


Robert Scheck  changed:

   What|Removed |Added

 Depends on|498194  |




-- 
Configure 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 498197] Review Request: zarafa-webaccess-mobile - Zarafa Webaccess Mobile is a simple web interface for mobile phones

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


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


Robert Scheck  changed:

   What|Removed |Added

 Depends on||498194




-- 
Configure 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 497441] Review Request: mumble - Voice chat application

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


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





--- Comment #40 from Igor Jurišković   2009-04-29 
09:29:20 EDT ---
(In reply to comment #39)
> Definitely getting there :)
> 
> Right now, all the descriptions are identical. Wouldn't it be better if the
> different packages explained what they contained, instead of just a common
> description of what Mumble is? Unless you already know, it's not very clear
> what "mumble-overlay" does.
Changed for everything except for protocol. Dont now what to write.

> 
> You're calling the package "murmur", which provides "mumble-murmur". If you
> make it provide "mumble-server" instead (or simply make it the "server"
> subpackage), you'll match the naming used in other distros, which means less
> confusion :)
It's called mumble-server now.
> 
> You're currently renaming the binary from murmurd to murmur. The 'd' is kind 
> of
> standard for daemons, and I suggest leaving it.
Changed.

> 
> Read the comment about the input-policy file above :) Unless you know it to
> work in current revisions of Fedora, I recommend dropping it.
To be honest with you I don't even now what it does. I'll put it as
documentation so if somebody wants it, it can get it there.

> 
> There needs to be a post-install for murmur/mumble-server that reloads the 
> dbus
> service.
dbus uses inotify to monitor conf files. They'll get updated automatically.
> 
> You're installing a logrotate, and your murmur.ini contains logging 
> information
> for /var/log/murmur, but this directory isn't created anywhere. Same goes for
> /var/run/murmur (where murmur will try to store it's pid file)  

Are you sure about this? I just untared mumble-1.1.8.tar.gz and opened
scripts/murmur.ini.system and it says this:

logfile=/var/log/mumble-server/mumble-server.log

pidfile=/var/run/mumble-server/mumble-server.pid

murmur.logrotate first line is:
/var/log/mumble-server/mumble-server.log

Files coming in about one hour.

-- 
Configure 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 497947] Review Request: libmetalink - A Metalink C library

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


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





--- Comment #4 from Ruben Kerkhof   2009-04-29 09:37:30 
EDT ---
Ant, have you been sponsored yet?

-- 
Configure 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 497441] Review Request: mumble - Voice chat application

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


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





--- Comment #41 from Rudolf Kastl   2009-04-29 09:36:44 EDT 
---
(In reply to comment #40)
> (In reply to comment #39)
> > Definitely getting there :)
> > 
> > You're installing a logrotate, and your murmur.ini contains logging 
> > information
> > for /var/log/murmur, but this directory isn't created anywhere. Same goes 
> > for
> > /var/run/murmur (where murmur will try to store it's pid file)  
> 
> Are you sure about this? I just untared mumble-1.1.8.tar.gz and opened
> scripts/murmur.ini.system and it says this:
> 
> logfile=/var/log/mumble-server/mumble-server.log
> 
> pidfile=/var/run/mumble-server/mumble-server.pid
> 
> murmur.logrotate first line is:
> /var/log/mumble-server/mumble-server.log

that is correct but you need to mkdir -p
%{buildroot}%{_localstatedir}/log/mumble-server and mkdir -p
%{buildroot}%{_localstatedir}/run/mumble-server

and %dir them both in the files section:

%dir %{_localstatedir}/log/mumble-server
%dir %{_localstatedir}/run/mumble-server

> 
> Files coming in about one hour.

-- 
Configure 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 252108] Review Request: python-html5lib - A python based HTML5 parser/tokenizer

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


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


Ruben Kerkhof  changed:

   What|Removed |Added

   Flag||needinfo?(o...@geek.co.il)




--- Comment #20 from Ruben Kerkhof   2009-04-29 
09:39:55 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 498125] Review Request: lxde-settings-daemon - XSettings Daemon for LXDE

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


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





--- Comment #2 from Christoph Wickert   2009-04-29 
09:41:28 EDT ---
(In reply to comment #1)
> - Require system-icon-theme instead of fedora-icon-theme, as one should be 
> able 
> to remove Fedora brandings altogether.

The reason for choosing fedora-icon-theme over system-icon-theme is that I
explicitly set Fedora in the patch. Changed.

> - Is the obsoletes&provides really necessary since there AFAIK is not and
> hasn't ever been a lxde-settings package in Fedora?

You are correct. I made one as a subpackage of the current lxde-common, but I
never built it. Removed.

> - lxde-settings-daemon.c is under GPLv2+, other files are under MIT.
> - I'm not sure, however, if the MIT license needs to be mentioned since it's
compatible with GPLv2+.

Agreed and removed. BTW: The included COPYING is GPLv3, not v2. Upstream
confirmed this was an error. You want me to replace the file in the package or
leave it as is for now.

> - Package already Requires: lxde-common which provides /usr/share/lxde =>
> remove
>  %dir %{_datadir}/lxde/

That was intentional. I omitted the dir from lxde-common-0.4 and made it
require lxde-settings-daemon instead.


Updated package:
SPEC: http://cwickert.fedorapeople.org/review/lxde-settings-daemon.spec
SRPM:
http://cwickert.fedorapeople.org/review/lxde-settings-daemon-0.4-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 498214] New: Review Request: gausssum - A GUI application for analysis of output of quantum computations

2009-04-29 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: gausssum - A GUI application for analysis of output of 
quantum computations

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

   Summary: Review Request: gausssum - A GUI application for
analysis of output of quantum computations
   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/gausssum.spec

SRPM URL:
http://theory.physics.helsinki.fi/~jzlehtol/rpms/gausssum-2.1.6-1.fc10.src.rpm

Description:
GaussSum is a GUI application that can analyse the output of ADF, GAMESS (US),
GAMESS-UK, Gaussian, Jaguar and PC GAMESS to extract and calculate useful
information. This includes the progress of the SCF cycles, geometry
optimisation, UV-Vis/IR/Raman spectra, MO levels, MO contributions and more.


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 498216] New: Review Request: perl-Perl-Tags - Generate Ctags style tags for Perl sourcecode

2009-04-29 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-Perl-Tags - Generate Ctags style tags for Perl 
sourcecode
Alias: perl-Perl-Tags

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

   Summary: Review Request: perl-Perl-Tags - Generate Ctags style
tags for Perl sourcecode
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Perl-Tags/
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-Perl-Tags.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Perl-Tags-0.26-1.fc12.src.rpm

Description:
Generate (possibly exuberant) Ctags style tags for Perl sourcecode.

Recursively follows use and require statements, up to a maximum of max_level.

The implemented tagger, Perl::Tags::Naive is a more-or-less straight ripoff,
slightly updated, of the original pltags code, and is rather naive. It should
be possible to subclass using something like PPI or Text::Balanced, though be
aware that this is alpha software and the internals are subject to change.

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

*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 498218] Review Request: picturetile - Tiles a bunch of images into one large "photo wall"

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


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


Edwin ten Brink  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 498218] New: Review Request: picturetile - Tiles a bunch of images into one large "photo wall"

2009-04-29 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: picturetile - Tiles a bunch of images into one large 
"photo wall"

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

   Summary: Review Request: picturetile - Tiles a bunch of images
into one large "photo wall"
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ed...@tenbrink-bekkers.nl
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.tenbrink-bekkers.nl/fedora/picturetile.spec
SRPM URL:
http://www.tenbrink-bekkers.nl/fedora/picturetile-20050314-1.fc10.src.rpm

Introduction:
Users need this perl script in order to use the Create Photowall option
(provided by extension PictureTile 0.5.0.0 in F-Spot). The script itself is not
packaged anywhere in Fedora until now. When user click Create Photowall in
F-Spot, only the message "picturetile.pl not found" appears, which is not very
user-friendly.

Unfortunately, F-Spot has a problem with its extensions at the moment,
therefore live testing of the interaction with F-Spot is at the moment
impossible. I already filed a bug upstream, and add I will add a tracker bug
here in Bugzilla as well.

Since it is not required for F-Spot to have this package or vice versa, I
deliberately left out these dependencies.

This is my second package under review, but I have no sponsor up to now. My
first package is under review under bug 497525.


Description of the script:
Takes a bunch of images and tiles them into one large image.
Not so much a collage, as a "photo wall."

The images are selected randomly from the given directory, and packed
together as tightly as possible. Images that are very small are
assumed to be thumbnails and are ignored. No image will be used more
than once, and the images need not be of the same size or aspect ratio.

This script can be used stand-alone from the command-line, or from the
F-Spot plugin "create photowall" (PictureTile).

-- 
Configure 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 498220] New: Review Request: perl-Module-Used - Find modules loaded by Perl code without running it

2009-04-29 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-Module-Used - Find modules loaded by Perl code 
without running it
Alias: perl-Module-Used

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

   Summary: Review Request: perl-Module-Used - Find modules loaded
by Perl code without running it
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Module-Used/
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-Module-Used.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Module-Used-1.2.0-1.fc12.src.rpm

Description:
Modules are found statically based upon use and require statements. If use
of the base or parent is found, both that module and the referenced ones
will be returned. If Moose or Moose::Role are found, this will look for
extends and with sugar will be looked for; presently, this will miss
modules listed in parentheses.

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

*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 498218] Review Request: picturetile - Tiles a bunch of images into one large "photo wall"

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


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


Edwin ten Brink  changed:

   What|Removed |Added

 Depends on||498222




-- 
Configure 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 498125] Review Request: lxde-settings-daemon - XSettings Daemon for LXDE

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


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


Jussi Lehtola  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Jussi Lehtola   2009-04-29 10:01:43 
EDT ---
(In reply to comment #2)
> > - lxde-settings-daemon.c is under GPLv2+, other files are under MIT.
> > - I'm not sure, however, if the MIT license needs to be mentioned since it's
> compatible with GPLv2+.
> 
> Agreed and removed. BTW: The included COPYING is GPLv3, not v2. Upstream
> confirmed this was an error. You want me to replace the file in the package or
> leave it as is for now.

Leave it as it is. The license of the program is the one in the source code.

> > - Package already Requires: lxde-common which provides /usr/share/lxde =>
> > remove
> >  %dir %{_datadir}/lxde/
> 
> That was intentional. I omitted the dir from lxde-common-0.4 and made it
> require lxde-settings-daemon instead.

Oh, okay, no problem then.

**

The package has been

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 476530] Review Request: projxp - Agile project management server

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


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





--- Comment #3 from Darryl L. Pierce   2009-04-29 10:07:02 
EDT ---
Updated release is available:

Spec URL: http://mcpierce.fedorapeople.org/rpms/projxp.spec
SRPM URL: http://mcpierce.fedorapeople.org/rpms/projxp-0.2.0-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 490576] Review Request: bibtex2html - Collection of tools for translating from BibTeX to HTML

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


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





--- Comment #13 from Guido Grazioli   2009-04-29 
10:02:42 EDT ---
Hi, no and yes actually; i'm trying to help packaging an eclipse feature right
now (wtp), which is software i use daily. If someone would take over the
maintenance of this package because i'm not sponsored, its no problem for me.

-- 
Configure 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 476530] Review Request: projxp - Agile project management server

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


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


Darryl L. Pierce  changed:

   What|Removed |Added

 Depends on||472819, 497640




-- 
Configure 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 472819] Review Request: rubygem-rufus-scheduler - Scheduler for Ruby (at, cron and every jobs)

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


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


Darryl L. Pierce  changed:

   What|Removed |Added

 Blocks||476530




-- 
Configure 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 497640] Review Request: rubygem-RedCloth - Textile parser for Ruby.

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


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


Darryl L. Pierce  changed:

   What|Removed |Added

 Blocks||476530




-- 
Configure 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 492259] Review Request: tucan - Manager for downloads and uploads at hosting sites

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


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





--- Comment #12 from Simon Wesp   2009-04-29 
10:10:17 EDT ---
new version available 0.3.7 alpha

@parag

in new version i removed shipped mo-files and built it from source

-- 
Configure 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 498125] Review Request: lxde-settings-daemon - XSettings Daemon for LXDE

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


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


Christoph Wickert  changed:

   What|Removed |Added

   Flag||fedora_requires_release_not
   ||e?




--- Comment #4 from Christoph Wickert   2009-04-29 
10:11:44 EDT ---
New Package CVS Request
===
Package Name: lxde-settings-daemon
Short Description: XSettings Daemon for LXDE
Owners: cwickert
Branches: 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 498130] Review Request: lxinput - Keyboard and mouse settings dialog for LXDE

2009-04-29 Thread bugzilla
Please do not reply directly 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 #4 from Christoph Wickert   2009-04-29 
10:15:01 EDT ---
Sorry, no F-9 please.

-- 
Configure 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 498125] Review Request: lxde-settings-daemon - XSettings Daemon for LXDE

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


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


Christoph Wickert  changed:

   What|Removed |Added

   Flag|fedora_requires_release_not |fedora-cvs?
   |e?  |




-- 
Configure 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 484423] Review Request: netbook-launcher - A clutter-based desktop launcher

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


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





--- Comment #10 from Traxtopel   2009-04-29 10:18:33 EDT 
---
The logout button will not work since it prereqs an older gdm (needs socket),
needs a rewrite. The patch I proposed adds functionality.

-- 
Configure 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 484423] Review Request: netbook-launcher - A clutter-based desktop launcher

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


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





--- Comment #11 from Traxtopel   2009-04-29 10:19:32 EDT 
---
Created an attachment (id=341758)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=341758)
Update logout/etc patch

Workaround for logout button issue.

-- 
Configure 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 497441] Review Request: mumble - Voice chat application

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


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





--- Comment #42 from Igor Jurišković   2009-04-29 
10:23:55 EDT ---
Done. Everything pointed by Thorvald and Rudolf has be fixed or changed.


NEW SPEC: http://78.46.84.75:81/tj/etmain/mumble/mumble.spec
SRPM: http://78.46.84.75:81/tj/etmain/mumble/mumble-1.1.8-7.fc9.src.rpm
DESKTOP MUMBLE: http://78.46.84.75:81/tj/etmain/mumble/mumble.desktop
DESKTOP OVERLAY: http://78.46.84.75:81/tj/etmain/mumble/mumble-overlay.desktop
NEW INIT: http://78.46.84.75:81/tj/etmain/mumble/murmur.init

-- 
Configure 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 487148] Review Request: gearmand - A distributed job system

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


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





--- Comment #5 from Ruben Kerkhof   2009-04-29 10:25:05 
EDT ---
Hi Peter, thanks for the review.
I've fixed all the issues mentioned above, and disabled building with tcmalloc
(from google-perftools-devel) for ppc64.

In the meantime upstream released a new version.

New version here:
SPEC: http://ruben.fedorapeople.org/gearmand.spec
SRPM: http://ruben.fedorapeople.org/gearmand-0.5-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 462535] Review Request: python-foolscap - Next-generation RPC protocol, intended to replace Perspective Broker

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


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


Ruben Kerkhof  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 498216] Review Request: perl-Perl-Tags - Generate Ctags style tags for Perl sourcecode

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


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


Parag AN(पराग)  changed:

   What|Removed |Added

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




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

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


  1   2   3   >