[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


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

   What|Removed |Added

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




--- Comment #1 from Parag AN(पराग) panem...@gmail.com  2009-04-29 02:02:09 
EDT ---
Review:
+ package builds in mock (rawhide i586).
koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1327573
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
78d560ad4fb67cb8ceb0d84c4de40ed0  App-Nopaste-0.10.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
All tests successful.
Files=1, Tests=8,  0 wallclock secs ( 0.04 usr  0.01 sys +  0.17 cusr  0.01
csys =  0.23 CPU)
+ Package perl-App-Nopaste-0.10-1.fc12.noarch =
  Provides: perl(App::Nopaste) = 0.10 perl(App::Nopaste::Command)
perl(App::Nopaste::Service) perl(App::Nopaste::Service::Gist)
perl(App::Nopaste::Service::Husk) perl(App::Nopaste::Service::Mathbin)
perl(App::Nopaste::Service::PastebinCom) perl(App::Nopaste::Service::Pastie)
perl(App::Nopaste::Service::Rafb) perl(App::Nopaste::Service::Shadowcat)
perl(App::Nopaste::Service::Snitch)
Requires: /usr/bin/env perl(App::Nopaste) perl(App::Nopaste::Command)
perl(Module::Pluggable) perl(Moose) perl(WWW::Mechanize) perl(base)
perl(strict) perl(warnings)

Suggestion:
1) you don't need following
# autoreq doesn't catch this
Requires:   perl(MooseX::Getopt)
# necessary for optional modules
Requires:   perl(Clipboard)
Requires:   perl(Config::INI::Reader)
Requires:   perl(Git)
Requires:   perl(WWW::Pastebin::PastebinCom::Create)
Requires:   perl(WWW::Pastebin::RafbNet::Create)

Remove them and then import new SRPM. I see yum auto processed all these
Requires

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 497634] Review Request: perl-App-Daemon - Start an Application as a Daemon

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=497634


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

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|jan.kle...@hp.com
   Flag||fedora-review+




--- Comment #3 from Jan Klepek jan.kle...@hp.com  2009-04-29 02:05:16 EDT ---
requires seems reasonable for me, I have compared it with modules which are
used in Daemon.pm and it fits.

Ok, i didn't find anything wrong (except that Log::Log4perl is required two
times in source code, but this imho doesn't affect runtime)

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 226483] Merge Review: tcsh

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=226483


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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi  2009-04-29 02:04:43 
EDT ---
- Time stamps are not conserved. 

for i in Fixes WishList; do
iconv -f iso-8859-1 -t utf-8  $i  ${i}_
mv ${i}_ $i
done

should be

for i in Fixes WishList; do
iconv -f iso-8859-1 -t utf-8 $i  ${i}_  \
touch -r $i ${i}_  \
mv ${i}_ $i
done

- Defattr should be
 (-,root,root,-)
not
 (-,root,root)


Fix these before building the new release. The package has been

APPROVED

-- 
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 490872] Review Request: jnettop - Network traffic tracker

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=490872





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-04-29 02:11:18 EDT ---
jnettop-0.13.0-5.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/jnettop-0.13.0-5.fc11

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

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


[Bug 490872] Review Request: jnettop - Network traffic tracker

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=490872





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-04-29 02:11:41 EDT ---
jnettop-0.13.0-5.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/jnettop-0.13.0-5.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 490872] Review Request: jnettop - Network traffic tracker

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=490872





--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-04-29 02:10:33 EDT ---
jnettop-0.13.0-5.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/jnettop-0.13.0-5.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 497759] Review Request: funcshell - A shell interface to Func

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=497759





--- Comment #2 from Silas Sewell si...@sewell.ch  2009-04-29 02:16:54 EDT ---
1). Updated Source0 to match tar.gz found in the source rpm.

[si...@silas ~]$ curl --silent
http://cloud.github.com/downloads/silas/funcshell/funcshell-0.0.1.tar.gz |
md5sum 
1c4ffd7be2aee86743b5aa5f64cc7971  -

2). I maintain other packages:
https://admin.fedoraproject.org/pkgdb/users/packages/silas

Just a quick clarification, when I say I am the creator of this package I
mean I wrote funcshell (my original wording was somewhat ambiguous).

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

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


[Bug 490591] Review Request: php-ezc-Template - eZ Components Template

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=490591





--- Comment #5 from Fedora Update System upda...@fedoraproject.org  
2009-04-29 02:24:11 EDT ---
php-ezc-Template-1.3.2-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/php-ezc-Template-1.3.2-1.fc11

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

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


[Bug 490591] Review Request: php-ezc-Template - eZ Components Template

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=490591





--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-04-29 02:24:16 EDT ---
php-ezc-Template-1.3.2-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/php-ezc-Template-1.3.2-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 490591] Review Request: php-ezc-Template - eZ Components Template

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=490591





--- Comment #4 from Fedora Update System upda...@fedoraproject.org  
2009-04-29 02:24:06 EDT ---
php-ezc-Template-1.3.2-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/php-ezc-Template-1.3.2-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 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


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

   What|Removed |Added

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




--- Comment #10 from Parag AN(पराग) panem...@gmail.com  2009-04-29 02:30:38 
EDT ---

Review:
+ package builds in mock (rawhide i586).
koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1327590
+ rpmlint output for SRPM and for RPM
tucan.noarch: E: explicit-lib-dependency librsvg2
== looks false message
+ source files match upstream url
21cc03fe952da8a0ae44ba66956a0766  tucan-0.3.6.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ tucan desktop file installed correctly.
+ GUI App.

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 493236] Review Request: xmlfy - Convert text/UTF-8 based output into XML format

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=493236





--- Comment #13 from Jan Klepek jan.kle...@hp.com  2009-04-29 02:36:59 EDT ---
Hi Arthur,

I will take a look at your latest package during today/tommorow.

For informal package review, you should follow
http://fedoraproject.org/wiki/Package_Review_Process

Just pickup any review request which is not yet assigned and perform review 
(for example, you could do review on this
https://bugzilla.redhat.com/show_bug.cgi?id=497759, please be aware that there
are specific checks for perl/python/php packages
http://fedoraproject.org/wiki/Packaging/Guidelines#Application_Specific_Guidelines
)

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

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


[Bug 497634] Review Request: perl-App-Daemon - Start an Application as a Daemon

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=497634


Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

___
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 #2 from Iain Arnell iarn...@gmail.com  2009-04-29 02:39:52 EDT ---
???!!?!?!

Without those explicit requires in the spec, I get an rpm that doesn't require
any of those packages and results in a broken installation - MooseX-Getopt is
absolutely required - the nopaste command definitely doesn't work without 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 497634] Review Request: perl-App-Daemon - Start an Application as a Daemon

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=497634





--- Comment #4 from Iain Arnell iarn...@gmail.com  2009-04-29 02:41:03 EDT ---
New Package CVS Request
===
Package Name: perl-App-Daemon
Short Description: Start an Application as a Daemon
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 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 #6 from Remi Collet fed...@famillecollet.com  2009-04-29 02:39:44 
EDT ---
@tom : thanks for this.

I will work on a build whithout hsieh_hash.c (I don't understand what you mean
about %pre, should be deleted in %prep and Makefile patched to not try to use
it)

Rehards

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

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


[Bug 497862] Review Request: python-paida - Pure Python scientific analysis package

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=497862





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-04-29 02:47:11 EDT ---
python-paida-3.2.1_2.10.1-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/python-paida-3.2.1_2.10.1-2.fc11

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

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


[Bug 497862] Review Request: python-paida - Pure Python scientific analysis package

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=497862





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-04-29 02:46:13 EDT ---
python-paida-3.2.1_2.10.1-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/python-paida-3.2.1_2.10.1-2.fc10

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

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


[Bug 497634] Review Request: perl-App-Daemon - Start an Application as a Daemon

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=497634


Iain Arnell iarn...@gmail.com 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 475017] Review Request: l2fprod-common - In JavaSE missing Swing components, inspired from modern user interfaces

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=475017





--- Comment #19 from Sandro Mathys s...@sandro-mathys.ch  2009-04-29 02:56:51 
EDT ---
Spec URL: http://red.fedorapeople.org/SRPMS/l2fprod-common.spec
SRPM URL:
http://red.fedorapeople.org/SRPMS/l2fprod-common-7.3-4.20090428cvs.fc11.src.rpm

* License
- Removing file in %prep
- Notified upstream

* CVS directory
- Removing in %prep

* Comments
- Removed (no copy-paste but sort of backup during the last edit...simply
forgot to remove them before uploading)

* Multilue jar files (including symlink)
- Done.

! Using xargs
- Done.

! About creating many symlinks manually
- Great input, thanks. Done.

! use %{_javadir}/%{name}-*.jar in %files
- Done.

FYI: Right now, there's still some stuff from 7.3-3 in 7.3-4 but commented, in
case there's a problem with the new changes.

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

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


[Bug 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 caol...@redhat.com 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 lkund...@v3.sk changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Lubomir Rintel lkund...@v3.sk  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(पराग) panem...@gmail.com  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 mattias.ell...@fysast.uu.se 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 mattias.ell...@fysast.uu.se  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 iarn...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

___
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 iarn...@gmail.com  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(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Parag AN(पराग) panem...@gmail.com  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 xav...@bachelot.org changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Xavier Bachelot xav...@bachelot.org  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 ramkr...@gmail.com 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 fed...@famillecollet.com  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(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #8 from Parag AN(पराग) panem...@gmail.com  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 ramkr...@gmail.com 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 ramkr...@gmail.com 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 ramkr...@gmail.com 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 upda...@fedoraproject.org  
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 upda...@fedoraproject.org  
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 mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #5 from Mattias Ellert mattias.ell...@fysast.uu.se  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 mattias.ell...@fysast.uu.se 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 jiri.reisc...@ecn.cz  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 t.sai...@alumni.ethz.ch 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 t.sai...@alumni.ethz.ch  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 jiri.reisc...@ecn.cz  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 cassmod...@fedoraproject.org changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #11 from Simon Wesp cassmod...@fedoraproject.org  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 t.sai...@alumni.ethz.ch  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 t.sai...@alumni.ethz.ch changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Thomas Sailer t.sai...@alumni.ethz.ch  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 t.sai...@alumni.ethz.ch 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 t.sai...@alumni.ethz.ch  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 k...@redhat.com changed:

   What|Removed |Added

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




--- Comment #8 from Karel Zak k...@redhat.com  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 bugs.mich...@gmx.net changed:

   What|Removed |Added

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




--- Comment #22 from Michael Schwendt bugs.mich...@gmx.net  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 t.sai...@alumni.ethz.ch 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 t.sai...@alumni.ethz.ch 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 t.sai...@alumni.ethz.ch  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 t.sai...@alumni.ethz.ch  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 t.sai...@alumni.ethz.ch 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 mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

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




--- Comment #7 from Mattias Ellert mattias.ell...@fysast.uu.se  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 mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

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




--- Comment #7 from Mattias Ellert mattias.ell...@fysast.uu.se  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 mmasl...@redhat.com 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 jussi.leht...@iki.fi 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 jussi.leht...@iki.fi 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 jussi.leht...@iki.fi changed:

   What|Removed |Added

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




--- Comment #3 from Jussi Lehtola jussi.leht...@iki.fi  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 mmasl...@redhat.com 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 mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se  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 upda...@fedoraproject.org  
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 jussi.leht...@iki.fi changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #2 from Jussi Lehtola jussi.leht...@iki.fi  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 jussi.leht...@iki.fi  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 tcall...@redhat.com  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 jussi.leht...@iki.fi 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 jussi.leht...@iki.fi  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 obsoletesprovides 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 michel.syl...@gmail.com  
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 michel.syl...@gmail.com 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 michel.syl...@gmail.com 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 michel.syl...@gmail.com 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 michel.syl...@gmail.com 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 michel.syl...@gmail.com 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 fed...@christoph-wickert.de changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #3 from Christoph Wickert fed...@christoph-wickert.de  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 redhat-bugzi...@linuxnetz.de 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 redhat-bugzi...@linuxnetz.de 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 redhat-bugzi...@linuxnetz.de 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 redhat-bugzi...@linuxnetz.de 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 redhat-bugzi...@linuxnetz.de 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 redhat-bugzi...@linuxnetz.de 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ć juriskovic.i...@gmail.com  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 ru...@rubenkerkhof.com  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 che...@gmail.com  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 ru...@rubenkerkhof.com changed:

   What|Removed |Added

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




--- Comment #20 from Ruben Kerkhof ru...@rubenkerkhof.com  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 fed...@christoph-wickert.de  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 obsoletesprovides 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


  1   2   3   >