[Bug 498214] Review Request: gausssum - A GUI application for analysis of output of quantum computations

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


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





--- Comment #8 from Fedora Update System   
2009-05-04 02:46:10 EDT ---
gausssum-2.1.6-3.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/gausssum-2.1.6-3.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 498214] Review Request: gausssum - A GUI application for analysis of output of quantum computations

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


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





--- Comment #7 from Fedora Update System   
2009-05-04 02:45:22 EDT ---
gausssum-2.1.6-3.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/gausssum-2.1.6-3.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 497338] Review Request: python-cclib - A library for processing results of computational chemistry packages

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


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





--- Comment #12 from Fedora Update System   
2009-05-04 02:44:28 EDT ---
python-cclib-0.91-4.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/python-cclib-0.91-4.fc11

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

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


[Bug 497338] Review Request: python-cclib - A library for processing results of computational chemistry packages

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


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





--- Comment #11 from Fedora Update System   
2009-05-04 02:43:25 EDT ---
python-cclib-0.91-4.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/python-cclib-0.91-4.fc10

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

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


[Bug 498866] New: Review Request: perl-Data-FormValidator-Constraints-DateTime - D::FV constraints for dates and times

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

Summary: Review Request: perl-Data-FormValidator-Constraints-DateTime - D::FV 
constraints for dates and times

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

   Summary: Review Request:
perl-Data-FormValidator-Constraints-DateTime - D::FV
constraints for dates and times
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Data-FormValidator-Constra
ints-DateTime/
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-Data-FormValidator-Constraints-DateTime.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Data-FormValidator-Constraints-DateTime-1.09-1.fc12.src.rpm

Description:
This package provides constraint routines for Data::FormValidator for
dealing with dates and times. It provides an easy mechanism for validating
dates of any format (using strptime(3)) and transforming those dates (as
long as you 'untaint' the fields) into valid DateTime objects, or into
strings that would be properly formatted for various database engines.

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

*rt-0.09

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

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


[Bug 497283] Review Request: perl-Syntax-Highlight-Engine-Kate - Port to Perl of the syntax highlight engine of the Kate texteditor

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


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





--- Comment #3 from Ralf Corsepius   2009-05-04 02:33:05 
EDT ---
(In reply to comment #2)
> The spec was recreated because it is not noarch. 
Please elaborate. I don't see any arch-specific portions inside of this
package.
IMO, this package should be a "noarch" packages.


> The problem in test, which I see is:
> PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" 
> "test_harness(0,
> 'blib/lib', 'blib/arch')" t/*.t   
>   
> t/Kate.t .. 448/? Use of uninitialized value $ch in concatenation (.) or 
> string
> at
> /home/marca/rpmbuild/BUILD/Syntax-Highlight-Engine-Kate-0.04/blib/lib/Syntax/Highlight/Engine/Kate/Template.pm
> line 534. 
>   
I am also seeing these, but don't have a "quick fix" for it (and no time to
investigate further), sorry.


> I can't still get rid off double 
> perl(Syntax::Highlight::Engine::Kate::Alerts).  

The rpmlint warning stems from these two files inside of the sources:
lib/Syntax/Highlight/Engine/Kate/Alerts
vs.
lib/Syntax/Highlight/Engine/Kate/Alerts.pm

They conflict.

I would suspect "Alerts" to be an outdated version of "Alerts.pm", which likely
was packaged into the tarball by mistake. I'd say, this is something to contact
upstream about to clarify.

For the moment, I'd propose to remove "Alerts" in %prep and to watch how far
you will get. I tried
rm -f lib/Syntax/Highlight/Engine/Kate/Alerts
and found it appears to work.

Further issue: Most permissions of the files inside of the tarball are wrong.
Proposal: Add
find -type f -exec chmod -x {} \;
to %prep

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

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


[Bug 498800] Review Request: perl-Text-Textile - A humane web text generator

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


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


Iain Arnell  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #6 from Iain Arnell   2009-05-04 01:58:06 EDT ---
Thanks for the 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 498772] Review Request: perl-Catalyst-Plugin-Cache - Flexible caching support for Catalyst

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


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


Iain Arnell  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #7 from Iain Arnell   2009-05-04 01:57:53 EDT ---
Thanks for the 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 498781] Review Request: perl-Catalyst-Plugin-Setenv - Catalyst::Plugin::Setenv Perl module

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


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


Iain Arnell  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #6 from Iain Arnell   2009-05-04 01:57:58 EDT ---
Thanks for the 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 498566] Review Request: perl-Config-JFDI - Just * Do it: A Catalyst::Plugin::ConfigLoader-style layer over Config::Any

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


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


Iain Arnell  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Iain Arnell   2009-05-04 01:57:46 EDT ---
Thanks for the 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 498785] Review Request: perl-Term-Prompt - Perl extension for prompting a user for information

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


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


Iain Arnell  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #6 from Iain Arnell   2009-05-04 01:58:03 EDT ---
Thanks for the 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 498800] Review Request: perl-Text-Textile - A humane web text generator

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


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





--- Comment #5 from Fedora Update System   
2009-05-04 01:53:23 EDT ---
perl-Text-Textile-2.03-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/perl-Text-Textile-2.03-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 498800] Review Request: perl-Text-Textile - A humane web text generator

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


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





--- Comment #4 from Fedora Update System   
2009-05-04 01:53:18 EDT ---
perl-Text-Textile-2.03-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/perl-Text-Textile-2.03-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 498785] Review Request: perl-Term-Prompt - Perl extension for prompting a user for information

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


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





--- Comment #4 from Fedora Update System   
2009-05-04 01:50:40 EDT ---
perl-Term-Prompt-1.04-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/perl-Term-Prompt-1.04-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 498862] New: Review Request: clthreads - POSIX threads C++ access library

2009-05-03 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: clthreads - POSIX threads C++ access library

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

   Summary: Review Request: clthreads - POSIX threads C++ access
library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: oget.fed...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://oget.fedorapeople.org/review/clthreads.spec
SRPM URL: http://oget.fedorapeople.org/review/clthreads-2.4.0-2.fc10.src.rpm
Description: 
Clthreads is a C++ wrapper library around the POSIX threads API.

rpmlint:
   clthreads-devel.x86_64: W: no-documentation
Not much we can do about it.

koji rawhide build:
   http://koji.fedoraproject.org/koji/taskinfo?taskID=1334670

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

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


[Bug 498781] Review Request: perl-Catalyst-Plugin-Setenv - Catalyst::Plugin::Setenv Perl module

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


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





--- Comment #5 from Fedora Update System   
2009-05-04 01:49:52 EDT ---
perl-Catalyst-Plugin-Setenv-0.03-1.fc10 has been submitted as an update for
Fedora 10.
http://admin.fedoraproject.org/updates/perl-Catalyst-Plugin-Setenv-0.03-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 498772] Review Request: perl-Catalyst-Plugin-Cache - Flexible caching support for Catalyst

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


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





--- Comment #6 from Fedora Update System   
2009-05-04 01:49:09 EDT ---
perl-Catalyst-Plugin-Cache-0.08-2.fc10 has been submitted as an update for
Fedora 10.
http://admin.fedoraproject.org/updates/perl-Catalyst-Plugin-Cache-0.08-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 498781] Review Request: perl-Catalyst-Plugin-Setenv - Catalyst::Plugin::Setenv Perl module

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


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





--- Comment #4 from Fedora Update System   
2009-05-04 01:49:48 EDT ---
perl-Catalyst-Plugin-Setenv-0.03-1.fc11 has been submitted as an update for
Fedora 11.
http://admin.fedoraproject.org/updates/perl-Catalyst-Plugin-Setenv-0.03-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 498785] Review Request: perl-Term-Prompt - Perl extension for prompting a user for information

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


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





--- Comment #5 from Fedora Update System   
2009-05-04 01:50:45 EDT ---
perl-Term-Prompt-1.04-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/perl-Term-Prompt-1.04-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 498772] Review Request: perl-Catalyst-Plugin-Cache - Flexible caching support for Catalyst

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


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





--- Comment #5 from Fedora Update System   
2009-05-04 01:49:05 EDT ---
perl-Catalyst-Plugin-Cache-0.08-2.fc11 has been submitted as an update for
Fedora 11.
http://admin.fedoraproject.org/updates/perl-Catalyst-Plugin-Cache-0.08-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 498838] Review Request: perl-Any-Moose - Use Moose or Mouse automagically

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


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


Chris Weyl  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 498838] Review Request: perl-Any-Moose - Use Moose or Mouse automagically

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


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





--- Comment #2 from Chris Weyl   2009-05-04 01:29:47 EDT 
---
New Package CVS Request
===
Package Name: perl-Any-Moose
Short Description: Use Moose or Mouse automagically
Owners: cweyl
Branches: F-9 F-10 F-11 devel
InitialCC: perl-sig

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

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


[Bug 481717] Review Request: ugene - genome analysis suite

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


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


Ivan Efremov  changed:

   What|Removed |Added

   Flag|needinfo?(iefre...@unipro.r |
   |u)  |




--- Comment #26 from Ivan Efremov   2009-05-04 01:22:08 EDT 
---
I'll update CVS branch during next 2-3 days

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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] Review Request: perl-Wx-Perl-DataWalker - implement subclass that shows relatively simple structure

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


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





--- Comment #3 from Parag AN(पराग)   2009-05-04 01:09:58 
EDT ---
waiting for perl(YAML::XS) >= 0.32
build failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=1334582

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

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


[Bug 481759] Review Request: python-Apptools - Enthough Tool Suite Application Tools

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


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





--- Comment #15 from Rakesh Pandit   2009-05-04 01:10:45 
EDT ---
Yeah very much. I just confirmed that debian packages also had same issues.

I had a chit chat with upstream and confirmed will be working on these issues
in future releases.

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

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


[Bug 497661] Review Request: perl-MIME-EncWords - Deal with RFC 2047 encoded words

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


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


Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #2 from Parag AN(पराग)   2009-05-04 01:05:52 
EDT ---
I also think description you included should have changed to 
This package is aimed to be another implementation of MIME::Words so
that it will achieve more exact conformance with RFC 2047 (former RFC
1522) specifications. Additionally, it contains some improvements.
Following synopsis and descriptions are inherited from its inspirer,
then added descriptions on improvements (**) or changes and
clarifications (*).*

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

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


[Bug 497661] Review Request: perl-MIME-EncWords - Deal with RFC 2047 encoded words

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


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


Bug 497661 depends on bug 497660, which changed state.

Bug 497660 Summary: Review Request: perl-MIME-Charset - Charset Informations 
for MIME
https://bugzilla.redhat.com/show_bug.cgi?id=497660

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



--- Comment #1 from Parag AN(पराग)   2009-05-04 00:58:43 
EDT ---
Review:
+ package builds in mock (rawhide i586).
koji Build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1334537
+ rpmlint output for SRPM and for RPM.
perl-MIME-EncWords.noarch: E: useless-provides perl(MIME::EncWords)
=> This package provides 2 times the same capacity. It should only provide it

+ source files match upstream url
2009449d0af39275d45e45a4afcaf58f  MIME-EncWords-1.010.101.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=2, Tests=74,  0 wallclock secs ( 0.04 usr  0.00 sys +  0.14 cusr  0.02
csys =  0.20 CPU)
+ Package perl-MIME-EncWords-1.010.101-1.fc12.noarch =>
Provides: perl(MIME::EncWords) perl(MIME::EncWords) = 1.010.101
Requires: perl >= 0:5.005 perl(Exporter) perl(strict) perl(vars)
+ Not a GUI application

suggestions:
1) make sure to remove unnecessary provides perl(MIME::EncWords)
2) you don't need 
Requires:   perl(MIME::Charset) >= 1.006.2


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 495564] Review Request: libguestfs - Access and modify virtual machine disk images

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


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


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

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


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


Peter Lemenkov  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 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer library

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


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


Peter Lemenkov  changed:

   What|Removed |Added

 AssignedTo|rjo...@redhat.com   |nob...@fedoraproject.org




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

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


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

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


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


Peter Lemenkov  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 490849] Review Request: php-ezc-Archive- eZ Components Archive

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


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


Peter Lemenkov  changed:

   What|Removed |Added

 Status|NEW |ON_QA




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

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


[Bug 498371] Review Request: fedora-devshell - Fedora Developer's Toolbox

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


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


Yaakov Nemoy  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #10 from Yaakov Nemoy   2009-05-04 
00:32:35 EDT ---
Updated and built.

https://koji.fedoraproject.org/koji/buildinfo?buildID=100713

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

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


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

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


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


Peter Lemenkov  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 498839] Review Request: hunspell-so - Somali spelling dictionary

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


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


Parag AN(पराग)  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(पराग)   2009-05-04 00:05:11 
EDT ---
Review:
+ package builds in mock (rawhide i586).
koji Build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1334539
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
334c3dd1cbcc797dbc9f5a0f4dbaf2e3  so...@dictionaries.addons.mozilla.org3.xpi
+ 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.
+ Not a GUI application

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 426753] Review Request: xmonad - A tiling window manager

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


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


Jens Petersen  changed:

   What|Removed |Added

   Flag|needinfo?(peter...@redhat.c |
   |om) |




--- Comment #44 from Jens Petersen   2009-05-04 00:03:40 
EDT ---
(In reply to comment #43)
> New update

Thanks!

> 1) carries around a ppc workaround that really should go in macros.ghc

Yeah we could add something in the macros to help with this, but it is a bit
tedious: I hope we can fix the real problem on ppc soon.

> 2) only builds on fedora 12 (yay?)

I hope to backport latest ghc to f11 soon.  We might have to wait for a new
gtk2hs release.

> 3) converts the value added default config into a patch directly against the
> upstream source. what this patch does is replaces the xmonad line of the
> default verbose config with an expansion that loads xterm with the man page
> showing. in the future, i'll probably add a comment to the user

Good idea.

I think better to rename the patch to xmonad-config-show-manpage.patch
or something like that.

> 4) includes just xmonad-start without doing xmonad-session.

Good - I think the name in the .desktop file should just be "xmonad" or
"XMonad" though not "xmonad-start".

> 5) renumbers the source to make more sense
> 6) includes other changes proposed here

Ok

Otherwise looks pretty good to me now.  I am going to take it for a spin and
then do the 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 498790] Review Request: firebird - SQL relational database management system

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #4 from Kevin Fenzi   2009-05-03 23:56:35 EDT ---
Please use your fedora account system name in Owners. ;) 

cvs done.

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

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


[Bug 498800] Review Request: perl-Text-Textile - A humane web text generator

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #3 from Kevin Fenzi   2009-05-03 23:58:43 EDT ---
cvs done.

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

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


[Bug 498785] Review Request: perl-Term-Prompt - Perl extension for prompting a user for information

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #3 from Kevin Fenzi   2009-05-03 23:54:39 EDT ---
cvs done.

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

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


[Bug 498703] Review Request: perl-Term-Completion - Read one line of user input, with convenience functions

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #3 from Kevin Fenzi   2009-05-03 23:52:16 EDT ---
cvs done.

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

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


[Bug 498566] Review Request: perl-Config-JFDI - Just * Do it: A Catalyst::Plugin::ConfigLoader-style layer over Config::Any

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #3 from Kevin Fenzi   2009-05-03 23:50:28 EDT ---
cvs done.

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

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


[Bug 498781] Review Request: perl-Catalyst-Plugin-Setenv - Catalyst::Plugin::Setenv Perl module

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #3 from Kevin Fenzi   2009-05-03 23:53:45 EDT ---
cvs done.

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

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


[Bug 498654] Review Request: uget - A download manager

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #3 from Kevin Fenzi   2009-05-03 23:51:22 EDT ---
cvs done.

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

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


[Bug 498772] Review Request: perl-Catalyst-Plugin-Cache - Flexible caching support for Catalyst

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #4 from Kevin Fenzi   2009-05-03 23:53:03 EDT ---
cvs done.

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

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


[Bug 498371] Review Request: fedora-devshell - Fedora Developer's Toolbox

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #9 from Kevin Fenzi   2009-05-03 23:47:37 EDT ---
cvs done.

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

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


[Bug 498214] Review Request: gausssum - A GUI application for analysis of output of quantum computations

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Kevin Fenzi   2009-05-03 23:46:17 EDT ---
cvs done.

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

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


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

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #3 from Kevin Fenzi   2009-05-03 23:44:27 EDT ---
cvs done.

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

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


[Bug 497339] Review Request: qmforge - Analysis tools for quantum mechanical calculations

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #10 from Kevin Fenzi   2009-05-03 23:43:37 EDT ---
cvs done.

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

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


[Bug 492927] Review Request: xcowsay - xcowsay displays a cute cow and message

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #15 from Kevin Fenzi   2009-05-03 23:39:45 EDT ---
cvs done.

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

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


[Bug 497338] Review Request: python-cclib - A library for processing results of computational chemistry packages

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #10 from Kevin Fenzi   2009-05-03 23:42:44 EDT ---
cvs done.

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

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


[Bug 494148] Review Request: soci - The database access library for C++ programmers

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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #27 from Kevin Fenzi   2009-05-03 23:41:48 EDT ---
cvs done.

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

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


[Bug 482763] Review Request: compat-guichan07 - compatibility libraries for guichan 0.7.1

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Kevin Fenzi   2009-05-03 23:38:38 EDT ---
Note that you always get a devel branch. Make sure and follow 
https://fedoraproject.org/wiki/How_to_remove_a_package_at_end_of_life
for the devel branch. 

cvs done.

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

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


[Bug 497766] Review Request: paintdotnet - A mono port of the Paint.NET image editor

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


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





--- Comment #13 from Eric Moret   2009-05-03 23:35:30 EDT 
---
Should now build on x86_64. New files at:
SRPM URL:
ftp://ftp.zouric.com/public/linux/paintdotnet/paintdotnet-0.1.63-1.fc10.src.rpm
SPEC URL: ftp://ftp.zouric.com/public/linux/paintdotnet/paintdotnet.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 491430] Review Request: sslogger - A keystroke logging utility for privileged user escalation

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


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





--- Comment #13 from Ed Brand   2009-05-03 23:36:48 EDT 
---
OK, apologies for the delay. Had a series of personal issues to resolve.

In reading through the comments, here is a summary of what we've got..
a/ no issue
b/ no issue
c/ no issue
d/ no issue
e/ no issue
f/ no issue
g/ * W: non-standard-uid /usr/bin/sslogger slogger
h/ * E: setuid-binary /usr/bin/sslogger slogger 06555
i/ * E: non-standard-dir-perm /var/log/sl 0750
j/ in progress...
k/ no issue
l/ convince me of the need of %pre, %post, %prein
m/ no issue
n/ no issue
README ... in progress


So lets start a dialogue on the suid, user/group, and permission:

Here is what I was looking to accomplish:
  - Create a binary which allows the user to gain access to another user's
account, including root, whilst logging all keystrokes. 
  - Create group of users "sloggers"  whose members are permitted to
review/audit other users activities. Hence the /i 750 


Breaking it down,
g/ I need a suid user/group, as I am reluctant to create a suid root binary.
True the Makefile installed as suid root which the spec file later changes
permission to the other "non standard" user/group in the %pre/%post. This is
related to /k. 
h/ The log files are written at a lower privilege level and not as root.  
i/ The permission on the directory are set to exclude users that are not in the
audit group "sloggers"
l/ Needed to create a non-root logging audit directory. As stated, I am
reluctant to crate a suid/sgid root binary.  

I am open to suggestions to resolve the above.

Thanks

-Ed

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

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


[Bug 498838] Review Request: perl-Any-Moose - Use Moose or Mouse automagically

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


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


Iain Arnell  changed:

   What|Removed |Added

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




--- Comment #1 from Iain Arnell   2009-05-03 23:27:25 EDT ---
+ source files match upstream.  
  d159b8b99180599ecc618078c93f3501da8c1469  Any-Moose-0.07.tar.gz

+ package meets naming and versioning guidelines.
+ specfile is properly named, is cleanly written and uses macros consistently.
+ summary is OK.
+ description is OK.
+ dist tag is present.
+ build root is OK.
+ license field matches the actual license.
  GPL+ or Artistic

+ license is open source-compatible.
+ license text is included.
+ latest version is being packaged.
+ BuildRequires are proper.
+ compiler flags are appropriate.
+ %clean is present.
+ package builds in mock
  http://koji.fedoraproject.org/koji/taskinfo?taskID=1334395

+ package installs properly.
+ rpmlint has no complaints:
  perl-Any-Moose.src: I: checking
  perl-Any-Moose.noarch: I: checking
  2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ final provides and requires are sane:
  perl(Any::Moose) = 0.07
  perl-Any-Moose = 0.07-1.fc12
=
  rpm -qp --requires perl-Any-Moose-0.07-1.fc12.noarch.rpm 
  perl(:MODULE_COMPAT_5.10.0)  
  perl(Mouse) >= 0.20
  perl(strict)  
  perl(warnings)  
  rpmlib(CompressedFileNames) <= 3.0.4-1
  rpmlib(FileDigests) <= 4.6.0-1
  rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  rpmlib(VersionedDependencies) <= 3.0.3-1

+ %check is present and all tests pass.
  All tests successful.
  Files=13, Tests=49,  5 wallclock secs ( 0.09 usr  0.01 sys +  4.76 cusr  0.14
csys =  5.00 CPU)
  Result: PASS

+ no shared libraries are added to the regular linker search paths.
+ owns the directories it creates.
+ doesn't own any directories it shouldn't.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no generically named files
+ code, not content.
+ documentation is small, so no -doc subpackage is necessary.
+ %docs are not necessary for the proper functioning of the 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 498838] Review Request: perl-Any-Moose - Use Moose or Mouse automagically

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


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


Iain Arnell  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




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

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


[Bug 493236] Review Request: xmlfy - Convert text/UTF-8 based output into XML format

2009-05-03 Thread bugzilla
Please do not reply directly 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 #15 from arthurguru   2009-05-03 23:24:59 
EDT ---
Packet delayed - I use the far less luxurious option of internet dial-up -
though not by choice.  

Is there anything outstanding?

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

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


[Bug 498136] Review Request: gst-mixer - advanced mixer for GNOME (old gnome-volume-control)

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


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #8 from Kevin Fenzi   2009-05-03 23:16:37 EDT ---
Andreas: Sorry for hijacking here, but we really need to have this review done. 
I hope you don't mind I take over?

OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name. 
OK - Spec has consistant macro usage. 
OK - Meets Packaging Guidelines. 
OK - License
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
3d519bc7d812aed8f6e4288b6d3cdf26  gnome-media-2.26.0.tar.bz2
3d519bc7d812aed8f6e4288b6d3cdf26  gnome-media-2.26.0.tar.bz2.orig
OK - Package needs ExcludeArch
OK - BuildRequires correct
OK - Spec handles locales/find_lang
OK - Package has %defattr and permissions on files is good. 
OK - Package has a correct %clean section. 
OK - Package has correct buildroot
OK - Package is code or permissible content. 
OK - Packages %doc files don't affect runtime. 
OK - Package has rm -rf RPM_BUILD_ROOT at top of %install

OK - Package is a GUI app and has a .desktop file

OK - Package compiles and builds on at least one arch. 
OK - Package has no duplicate files in %files. 
OK - Package doesn't own any directories other packages own. 
OK - Package owns all the directories it creates. 
OK - Package obey's FHS standard (except for 2 exceptions)
See below - No rpmlint output. 
OK - final provides and requires are sane.

SHOULD Items:

OK - Should build in mock. 
OK - Should have sane scriptlets. 
OK - Should have dist tag
OK - Should package latest version
OK - Should not use file requires outside of /etc, /bin, /sbin, /usr/bin, or
/usr/sbin

Issues: 

1. Should the 
Requires:   gnome-media
be versioned? 
Or will it cause any issue for this to be out of step with gnome-media?
I suspect not, but though I would mention it. 

2. rpmlint says: 

gst-mixer.src:163: W: macro-in-%changelog defines

Minor issue... change to %%defines in changelog? :) 

gst-mixer.x86_64: W: non-conffile-in-etc /etc/gconf/schemas/gst-mixer.schemas

Can be ignored. 

I don't see any blockers here, so this package is APPROVED. 

Fix the %%define and address if gnome-media requires should be versioned before
import?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-05-03 Thread bugzilla
Please do not reply directly 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 #6 from arthurguru   2009-05-03 23:17:43 
EDT ---
Hi Silas,

I'm using f7 which still uses python 2.5 (r25:51908, Apr 10 2007) (which is why
I stressed "on my machine"). 

I'm building against the source archive as:
% rpmbuild -ba funcshell.spec
The build works, but the egg-info file is not present in %{buildroot}, which
then fails the packaging part of the spec file.

Not familiar with python history but 2 years is a long time in software. If its
a version issue and f7 is not reaching end-of-life status with looming f11 then
maybe some advanced packaging syntax could be used in the specfile.
E.g.
BuildRequires: python >= 

Kind Regards,
Arthur

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

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


[Bug 225622] Merge Review: boost

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


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


Denis Arnaud  changed:

   What|Removed |Added

 CC||denis.arnaud_fed...@m4x.org




--- Comment #33 from Denis Arnaud   2009-05-03 
21:27:44 EDT ---
Besides, for Boost 1.37 (Fedora 11 / Rawhide), there seems to be an issue with
the dependency on libicu, as in the build.log of RMOL
(http://koji.fedoraproject.org/koji/getfile?taskID=1334520&name=root.log):

DEBUG backend.py:554:  /usr/bin/yum --installroot
/var/lib/mock/dist-f12-build-444978-72210/root/  resolvedep  'libicu'
'gsl-devel >= 1.8' 'ghostscript' 'cppunit-devel >= 1.10' 'texlive-latex'
'boost-devel >= 1.34' 'doxygen' 'texlive-dvips'
DEBUG util.py:280:  Executing command: /usr/bin/yum --installroot
/var/lib/mock/dist-f12-build-444978-72210/root/  resolvedep  'libicu'
'gsl-devel >= 1.8' 'ghostscript' 'cppunit-devel >= 1.10' 'texlive-latex'
'boost-devel >= 1.34' 'doxygen' 'texlive-dvips'
DEBUG util.py:256:  0:libicu-4.2-0.1.d03.fc12.x86_64
DEBUG util.py:256:  0:gsl-devel-1.12-3.fc11.x86_64
DEBUG util.py:256:  0:ghostscript-8.64-6.fc12.x86_64
DEBUG util.py:256:  0:cppunit-devel-1.12.1-2.fc11.x86_64
DEBUG util.py:256:  0:texlive-latex-2007-42.fc11.x86_64
DEBUG util.py:256:  0:boost-devel-1.37.0-6.fc11.x86_64
DEBUG util.py:256:  1:doxygen-1.5.8-2.fc11.x86_64
DEBUG util.py:256:  0:texlive-dvips-2007-42.fc11.x86_64
DEBUG util.py:319:  Child returncode was: 0
DEBUG backend.py:554:  /usr/bin/yum --installroot
/var/lib/mock/dist-f12-build-444978-72210/root/  install  'libicu' 'gsl-devel
>= 1.8' 'ghostscript' 'cppunit-devel >= 1.10' 'texlive-latex' 'boost-devel >=
1.34' 'doxygen' 'texlive-dvips'
DEBUG util.py:280:  Executing command: /usr/bin/yum --installroot
/var/lib/mock/dist-f12-build-444978-72210/root/  install  'libicu' 'gsl-devel
>= 1.8' 'ghostscript' 'cppunit-devel >= 1.10' 'texlive-latex' 'boost-devel >=
1.34' 'doxygen' 'texlive-dvips'
DEBUG util.py:256:  boost-1.37.0-6.fc11.x86_64 from build has depsolving
problems
DEBUG util.py:256:--> Missing Dependency: libicuuc.so.40()(64bit) is needed
by package boost-1.37.0-6.fc11.x86_64 (build)
DEBUG util.py:256:  boost-1.37.0-6.fc11.x86_64 from build has depsolving
problems
DEBUG util.py:256:--> Missing Dependency: libicui18n.so.40()(64bit) is
needed by package boost-1.37.0-6.fc11.x86_64 (build)
DEBUG util.py:256:  Error: Missing Dependency: libicuuc.so.40()(64bit) is
needed by package boost-1.37.0-6.fc11.x86_64 (build)
DEBUG util.py:256:   You could try using --skip-broken to work around the
problem
DEBUG util.py:256:  Error: Missing Dependency: libicui18n.so.40()(64bit) is
needed by package boost-1.37.0-6.fc11.x86_64 (build)
DEBUG util.py:256:   You could try running: package-cleanup --problems
DEBUG util.py:256:  package-cleanup --dupes
DEBUG util.py:256:  rpm -Va --nofiles --nodigest
DEBUG util.py:256:  The program package-cleanup is found in the yum-utils
package.
DEBUG util.py:319:  Child returncode was: 1

-- 
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 496635] Review Request: monodevelop-debugger-mdb - Mono Debugger Addin for MonoDevelop

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


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


Ryan Bair  changed:

   What|Removed |Added

 Attachment #342187|0   |1
is obsolete||
 Attachment #342189|0   |1
is obsolete||




--- Comment #12 from Ryan Bair   2009-05-03 20:01:51 EDT 
---
Created an attachment (id=342262)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=342262)
Update spec file

Going the (seemingly) preferred route of fixing the install process in the spec
as opposed to the source itself. I feel pretty good with this one. Comments?

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

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


[Bug 498849] New: Review Request: klatexformula - Application for easy image creating from a LaTeX equation

2009-05-03 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: klatexformula - Application for easy image creating 
from a LaTeX equation

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

   Summary: Review Request: klatexformula - Application for easy
image creating from a LaTeX equation
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: alekc...@googlemail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: 
http://nucleo.fedorapeople.org/pkg-reviews/klatexformula/klatexformula.spec
SRPM URL: 
http://nucleo.fedorapeople.org/pkg-reviews/klatexformula/klatexformula-3.0.0-1.fc10.src.rpm

Description:
This application is especially designed for generating an image
(e.g. PNG) from a LaTeX equation, this using a nice GUI. Support for
Copy-Paste, Save as image/pdf/ps and drag-and-drop is included. In
addition to supporting a whole GUI-based user interface, this package
includes the library KLFBackend to integrate KLatexFormula functionality
into your programs.

$ rpmlint klatexformula-3.0.0-1.fc10.i386.rpm
klatexformula-debuginfo-3.0.0-1.fc10.i386.rpm
klatexformula-devel-3.0.0-1.fc10.i386.rpm
4 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint klatexformula-3.0.0-1.fc10.src.rpm
klatexformula.src: W: %ifarch-applied-patch Patch0: klatexformula-lib64.patch
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

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

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

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


[Bug 498846] New: Review Request: R-RM2 - Revenue Management and Pricing for R

2009-05-03 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: R-RM2 - Revenue Management and Pricing for R

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

   Summary: Review Request: R-RM2 - Revenue Management and Pricing
for R
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: denis.arnaud_fed...@m4x.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://denisarnaud.fedorapeople.org/R/RM2/R-RM2.spec
SRPM URL: http://denisarnaud.fedorapeople.org/R/RM2/R-RM2-0.0-1.fc10.src.rpm
[FE-NEEDSPONSOR] That is my fourth package for Fedora, but the second one for
R,
and I thus still need a sponsor.

Description: RM2 is a simple package that implements functions used in revenue
management and pricing environments.

Web site: http://cran.r-project.org/web/packages/RM2/
  

Additional notes:
 * I followed the R-specific guidelines
(https://fedoraproject.org/wiki/Packaging/R).
 * That package requires the R-msm package, which I submitted for
review/approval separately (https://bugzilla.redhat.com/show_bug.cgi?id=498845)


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

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


[Bug 498845] Review Request: R-msm - Multi-state Markov and hidden Markov models in continuous time

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


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


Denis Arnaud  changed:

   What|Removed |Added

 Blocks||498846




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

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


[Bug 498846] Review Request: R-RM2 - Revenue Management and Pricing for R

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


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


Denis Arnaud  changed:

   What|Removed |Added

 Depends on||498845




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

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


[Bug 498845] Review Request: R-msm - Multi-state Markov and hidden Markov models in continuous time

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


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


Denis Arnaud  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 498845] Review Request: R-msm - Multi-state Markov and hidden Markov models in continuous time

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


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


Denis Arnaud  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 498845] New: Review Request: R-msm - Multi-state Markov and hidden Markov models in continuous time

2009-05-03 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: R-msm - Multi-state Markov and hidden Markov models in 
continuous time

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

   Summary: Review Request: R-msm - Multi-state Markov and hidden
Markov models in continuous time
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: denis.arnaud_fed...@m4x.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://denisarnaud.fedorapeople.org/R/msm/R-msm.spec
SRPM URL: http://denisarnaud.fedorapeople.org/R/msm/R-msm-0.8.2-1.fc10.src.rpm
[FE-NEEDSPONSOR] That is my third package for Fedora, but the first one for R,
and I thus still need a sponsor.

Description: Functions for fitting general continuous-time Markov and hidden
Markov multi-state models to longitudinal data.  Both Markov transition rates
and the hidden Markov output process can be modelled in terms of covariates.  A
variety of observation schemes are supported, including processes observed at
arbitrary times, completely-observed processes, and censored states.

Web site: http://cran.r-project.org/web/packages/msm/
  

Additional notes:
 * I followed the R-specific guidelines
(https://fedoraproject.org/wiki/Packaging/R).
 * I need that package, as it is a dependency of another package I would like
to submit, namely R-RM2 (for which I shall open a distinct package review
request).


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

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


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

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


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





--- Comment #5 from Xavier Bachelot   2009-05-03 19:03:45 
EDT ---
(In reply to comment #3)
> 1] rpmlint is not silent on actual package

websvn.noarch: W: non-standard-uid /var/cache/websvn apache
websvn needs this to be able to temporarily cache some generated files. The
owner is indeed apache.

websvn.noarch: W: dangling-relative-symlink /usr/share/websvn/temp
../../../var/tmp
I believe this warning is bogus. /var/tmp exists on the file system. I've tried
to fix this, but failed. I'd love to see it fixed though and I welcome any
suggestion. 

> 2] do you really need patch for replacing 3 lines? sed with comment in %prep
> will be imho better readable.

Well, the patch is perfectly legible imo. And also, I'd rather not use sed for
this kind of thing, it would be too easy to break things w/o noticing. I'd
rather keep a patch and possibly rebase it if the underlying code changes,
rather than silently breaking it with sed.

> 3] I think that it would be good to add webserver into requires. Because 
> websnv
> wouldn't work without webserver.
Right, and also the package wouldn't even install because the apache user
wouldn't exist. I'll add :
Requires(pre): httpd

(In reply to comment #4)
> If the provided config files works with httpd I think it's better to require 
> it
> rather than a generic web server. On the other hand, those _not_ wanting to 
> use
> httpd as web server will be disappointed...
> 
> A possible solution to this (used in the mantis package) is to split the 
> config
> file in a subpackage and let that require httpd.  

It looks a bit too much to me to have a subpackage just for the apache conf and
I'm not sure this would be enough anyway, because some files are owned by the
apache user. Actually, I've never used any other webserver, and I'm not sure
how, say lighttpd, works (user, conf file, etc...). I'll dig the guidelines for
advices. Any pointers welcome.

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

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


[Bug 497663] Review Request: perl-HTML-StripScripts-Parser - XSS filter using HTML::Parser

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


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





--- Comment #5 from Fedora Update System   
2009-05-03 18:26:54 EDT ---
perl-HTML-StripScripts-Parser-1.02-1.fc10 has been submitted as an update for
Fedora 10.
http://admin.fedoraproject.org/updates/perl-HTML-StripScripts-Parser-1.02-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 497663] Review Request: perl-HTML-StripScripts-Parser - XSS filter using HTML::Parser

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


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





--- Comment #7 from Fedora Update System   
2009-05-03 18:27:03 EDT ---
perl-HTML-StripScripts-Parser-1.02-1.fc11 has been submitted as an update for
Fedora 11.
http://admin.fedoraproject.org/updates/perl-HTML-StripScripts-Parser-1.02-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 497663] Review Request: perl-HTML-StripScripts-Parser - XSS filter using HTML::Parser

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


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





--- Comment #6 from Fedora Update System   
2009-05-03 18:26:58 EDT ---
perl-HTML-StripScripts-Parser-1.02-1.fc9 has been submitted as an update for
Fedora 9.
http://admin.fedoraproject.org/updates/perl-HTML-StripScripts-Parser-1.02-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 497663] Review Request: perl-HTML-StripScripts-Parser - XSS filter using HTML::Parser

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


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


Bug 497663 depends on bug 497662, which changed state.

Bug 497662 Summary: Review Request: perl-HTML-StripScripts - Strip scripting 
constructs out of HTML
https://bugzilla.redhat.com/show_bug.cgi?id=497662

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



--- Comment #4 from Xavier Bachelot   2009-05-03 18:24:17 
EDT ---
Imported and built for F-9, F10 and F-11.
Imported and built a slightly older version for EL-5 to be able to fulfil the
build requirements.

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

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


[Bug 496635] Review Request: monodevelop-debugger-mdb - Mono Debugger Addin for MonoDevelop

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


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





--- Comment #11 from Mauricio Henriquez   2009-05-03 
18:05:48 EDT ---
(In reply to comment #10)
> I added the two patches as attachments to the bug. The first patch is against
> the spec file and the second patch is against the unpacked source tarball. 
> Both
> patches fix the assumption that the libdir should be in prefix/lib (/usr/lib)
> which works on x86 but not on x86_64 (where it is /usr/lib64). Also, the 
> x86_64
> build does not work because of the pc file issue described in bug 490025 
> (which
> I also wrote a patch for yesterday). I would mark it as a blocker, but I don't
> seem to have access to do so. 
> 
> I'm not trying to take over your package, just fixing a few miscellaneous
> issues that prevented me from compiling/installing/using/enjoying the package
> myself. 
Is not what I meant, is just that you are making changes to a also newbie guy
package, so I don't have a clue if your patchs are correct or not, thats is way
I was waiting to some sponsor/experienced guys to let what to do with the
packaes. But sure, if in the meantime you test some of your patch and you know
that makes the magic, so then is a step forward, is just that we are going to
wait for someone that really tell us that our changes are correct...Still have
lot of doubts about how koji works, etc...

Thanks!!

Mauricio..

> 
> As a warning, I am new to both RPM packaging and the Fedora guidelines so I
> could have committed a major sin. Please check my patches.

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

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


[Bug 478683] Review Request: perl-Net-Google-AuthSub - Provides interface to interact with sites that implement Google style AuthSub

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


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


Chris Weyl  changed:

   What|Removed |Added

 CC||cw...@alumni.drew.edu
 Blocks||177841(FE-NEEDSPONSOR)




--- Comment #4 from Chris Weyl   2009-05-03 16:40:10 EDT 
---
Marking as blocking FE-NEEDSPONSOR.

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

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


[Bug 497140] Review Request: php-IDNA_Convert - Internationalized domain name to UTF-8 converter

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


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


Jussi Lehtola  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #6 from Jussi Lehtola   2009-05-03 16:43:05 
EDT ---
Oh, this has been left hanging, sorry.

Lukas is right: you are mixing macros which is not allowed. Choose either
$RPM_BUILD_ROOT or %{buildroot} and stick with it, don't mix the two.

Also, I don't like to see macros used for standard unix commands, but this is
maybe more a matter of style. I think it just makes the spec file harder to
read.

I'll trust you to fix the macro issue upon import to CVS. 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 498490] Review Request: smem - Reports application memory usage in a meaningful way

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


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





--- Comment #15 from Matthew Miller   2009-05-03 16:46:05 
EDT ---
(update - can't work on this this weekend but will monday or tuesday.)

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

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


[Bug 481442] Review Request: clutter-perl - Perl bindings to the Clutter OpenGL canvas

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


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


Chris Weyl  changed:

   What|Removed |Added

 CC||cw...@alumni.drew.edu
   Flag||needinfo?(allis...@gmail.co
   ||m)




--- Comment #2 from Chris Weyl   2009-05-03 16:42:33 EDT 
---
Marking needinfo w.r.t. comment #1.

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

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


[Bug 498839] New: Review Request: hunspell-so - Somali spelling dictionary

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

Summary: Review Request: hunspell-so - Somali spelling dictionary

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

   Summary: Review Request: hunspell-so - Somali spelling
dictionary
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: caol...@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://people.redhat.com/caolanm/rpms/hunspell-so.spec
SRPM URL:
http://people.redhat.com/caolanm/rpms/hunspell-so-0.1.2-1.fc11.src.rpm
Description: Somali hunspell dictionary

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

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


[Bug 498390] Review Request: rakudo - Rakudo - A Perl compiler for Parrot

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


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


Chris Weyl  changed:

   What|Removed |Added

 CC||cw...@alumni.drew.edu
   Flag||needinfo?(g...@zimt.uni-siege
   ||n.de)




--- Comment #1 from Chris Weyl   2009-05-03 16:37:07 EDT 
---
Do you have a timeframe in which you plan on submitting this?

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

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


[Bug 498838] Review Request: perl-Any-Moose - Use Moose or Mouse automagically

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


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


Chris Weyl  changed:

   What|Removed |Added

 Blocks||498564




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

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


[Bug 498838] New: Review Request: perl-Any-Moose - Use Moose or Mouse automagically

2009-05-03 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-Any-Moose - Use Moose or Mouse automagically
Alias: perl-Any-Moose

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

   Summary: Review Request: perl-Any-Moose - Use Moose or Mouse
automagically
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Any-Moose
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cw...@alumni.drew.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~cweyl/review/perl-Any-Moose.spec
SRPM URL:
http://fedorapeople.org/~cweyl/review/perl-Any-Moose-0.07-1.fc10.src.rpm

Description:
This module allows one to take advantage of the features Moose/Mouse
provides, while allowing one to let the program author determine if Moose
or Mouse should be used; when use'd, we load Mouse if Moose isn't already
loaded, otherwise we go with Moose.

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

Additional Comment:

This is a new dep of Data::Visitor 0.24

*rt-0.10

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

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


[Bug 498736] Review Request: ucommon - Portable C++ runtime for threads and sockets

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


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





--- Comment #12 from David Sugar   2009-05-03 16:26:53 
EDT ---
rpmlint gave me a couple of useful additional things:

ucommon.src: E: no-cleaning-of-buildroot %install
ucommon.src: W: non-standard-group System/Libraries

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-05-03 Thread bugzilla
Please do not reply directly 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


Jeroen van Meeuwen  changed:

   What|Removed |Added

 Depends on||498837




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

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


[Bug 497140] Review Request: php-IDNA_Convert - Internationalized domain name to UTF-8 converter

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


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


LukasHetzi  changed:

   What|Removed |Added

 CC||l...@gmx.at




--- Comment #5 from LukasHetzi   2009-05-03 16:12:51 EDT ---
rpmlint output: 2 packages and 1 specfiles checked; 0 errors, 0 warnings.
Package naming and spec file name is correct.
License is correct.
Spec file is legible, but doesn't consistently use macros (see below)
Package is correct.

- Do not mix $RPM_BUILD_ROOT and %{buildroot}
- You could copy all php files in one line

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-05-03 Thread bugzilla
Please do not reply directly 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





--- Comment #1 from Robert Scheck   2009-05-03 
15:44:51 EDT ---
Spec URL: http://labs.linuxnetz.de/bugzilla/zarafa-webaccess.spec

If you currently want to test this package, you need to be an approved Zarafa 
beta tester. Needed patch for webaccess is included in the *.nosrc.rpm package. 
Note, that 6.30.0 hasn't been released and I'm updating the snapshot whenever
a new beta preview gets available.

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

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


[Bug 498736] Review Request: ucommon - Portable C++ runtime for threads and sockets

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


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





--- Comment #11 from David Sugar   2009-05-03 15:33:34 
EDT ---
Result of too much last minute editing :)

Added %?dist...and "%{__make} DESTDIR=%{buildroot}/ install" for %install.

Updated %changelog entry to correct format...

I will do the sipwitch one later today (or in the week) as you suggest.

As for ucommon-config.in...hmm...that one I have to think about a bit.  I
actually use pkgconfig, so I do not normally think about ucommon-config :), and
passing --prefix is perhaps a less common use case.  This may be an issue to
resolve for the next upstream patch release...

Updated files:

http://www.gnutelephony.org/specs/ucommon.spec
http://www.gnutelephony.org/specs/ucommon-2.0.5-3.src.rpm

There was a time I used to create a lot of rpm packages, but that was years
ago...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-05-03 Thread bugzilla
Please do not reply directly 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





--- Comment #1 from Robert Scheck   2009-05-03 
15:31:38 EDT ---
Spec URL: http://labs.linuxnetz.de/bugzilla/zarafa-webaccess-mobile.spec

If you currently want to test this package, you need to be an approved Zarafa 
beta tester. Needed patch for webaccess-mobile is included in the *.nosrc.rpm 
package. Note, that 6.30.0 hasn't been released and I'm updating the snapshot
whenever a new beta preview gets available.

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

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


[Bug 481759] Review Request: python-Apptools - Enthough Tool Suite Application Tools

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


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





--- Comment #14 from Jochen Schmitt   2009-05-03 
15:10:52 EDT ---
It may be nice, if you can tell the upstream authors, that this is important to
get a minimum of QA insurance, to make sure, that the package works well on all
of our supoorted plattforms.

Even if you can build an application for a specific plattform, you doesn't have
the warrenty, that the application will run on this specific plattform.

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

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


[Bug 498790] Review Request: firebird - SQL relational database management system

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


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


Philippe Makowski  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #3 from Philippe Makowski   2009-05-03 
14:38:31 EDT ---
New Package CVS Request
===
Package Name: firebird
Short Description: SQL relational database management system
Owners: Philippe Makowski
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 498189] Review Request: perl-JavaScript-Minifier-XS - XS based JavaScript minifier

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


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





--- Comment #1 from Chris Weyl   2009-05-03 14:30:13 EDT 
---
Let me know when perl-JavaScript-Minifier is in 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 498703] Review Request: perl-Term-Completion - Read one line of user input, with convenience functions

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


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


Chris Weyl  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 498703] Review Request: perl-Term-Completion - Read one line of user input, with convenience functions

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


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





--- Comment #2 from Chris Weyl   2009-05-03 14:31:28 EDT 
---
New Package CVS Request
===
Package Name: perl-Term-Completion
Short Description: Read one line of user input, with convenience functions
Owners: cweyl
Branches: F-9 F-10 F-11 devel
InitialCC: perl-sig

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

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


[Bug 498736] Review Request: ucommon - Portable C++ runtime for threads and sockets

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


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


Mamoru Tasaka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review?




--- Comment #10 from Mamoru Tasaka   2009-05-03 
14:26:46 EDT ---
Assigning.

Before doing full-review
- Why is %install section empty this time?
- Please consider to use %?dist tag:
  https://fedoraproject.org/wiki/Packaging/DistTag
- Is this the expected behavior on ucommon-config.in?

16  if [ "`ldd /bin/sh | grep lib64`" = "" ]
17  then
18  libdir=${exec_prefix}/lib
19  else
20  libdir=${exec_prefix}/lib64
21  fi


56  case "$1" in
57  --prefix=*)
58  prefix=$optarg
59  includedir=$prefix/include
60  libdir=$prefix/lib
61  ;;

  Here when --prefix or --exec_prefix is specified, libdir
  will always points to X/lib even on 64 bits architecture.

- Please follow Fedora's %changelog format:
  https://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs

! For sipwitch please submit a new review request for the
  package and make this review request block the review
  request for sipwitch.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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] Review Request: perl-JavaScript-Minifier-XS - XS based JavaScript minifier

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


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


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

2009-05-03 Thread bugzilla
Please do not reply directly 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 #9 from Chris Weyl   2009-05-03 13:55:01 EDT 
---
Hmm... looks like a small ruby script hardcoded to use
http://www.rafb.net/paste.

Given that both of these packages provide similar base functionality (but this
one supproting significantly more), I wonder if it wouldn't be approprate to
just have perl-App-Nopaste explicitly Conflict: with nopaste; or perhaps
leverage the alternatives system (see, e.g. man update-alternatives).  I
admittedly know a very small amount about the alternatives system, but it would
seem to fit this situation...

Anyways, I've probably hijacked the review tix enough :)  Let me know if
there's anything I can do if you decide to go either of those routes.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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] Review Request: perl-JavaScript-Minifier-XS - XS based JavaScript minifier

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


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


Chris Weyl  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|cw...@alumni.drew.edu




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

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


[Bug 497800] Review Request: perl-Acme-PlayCode - Perl module for enhancement of code

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


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





--- Comment #4 from Chris Weyl   2009-05-03 13:43:05 EDT 
---
Generally speaking, dos2unix (or the ever popular "sed -e 's/\r//'") doesn't
strictly belong in %prep, but it fits far better in there than it does in
%build.  Stripping dos-like EOL characters from source is as much preparing it
as is patching it, or removing errant execute bits, etc.  Absent a very
compelling reason, we wouldn't include a %patch in %build, would we?

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

Spec looks sane, clean and consistent; license is correct (GPL+ or Artistic);
make test passes cleanly.

Source tarballs match upstream (sha1sum):
0034e6c7961d2f9ed8b5e6ad60ac7333941d73d9 Acme-PlayCode-0.11.tar.gz
0034e6c7961d2f9ed8b5e6ad60ac7333941d73d9 Acme-PlayCode-0.11.tar.gz.srpm

Final provides / requires are sane:

0 packages and 1 specfiles checked; 0 errors, 0 warnings.
=> perl-Acme-PlayCode-0.11-2.fc12.noarch.rpm <=
> rpmlint
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
> provides for perl-Acme-PlayCode-0.11-2.fc12.noarch.rpm
perl(Acme::PlayCode) = 0.11
perl-Acme-PlayCode = 0.11-2.fc12
perl(Acme::PlayCode::Plugin::Averything) = 0.10
perl(Acme::PlayCode::Plugin::DoubleToSingle) = 0.10
perl(Acme::PlayCode::Plugin::ExchangeCondition) = 0.10
perl(Acme::PlayCode::Plugin::NumberPlus) = 0.11
perl(Acme::PlayCode::Plugin::PrintComma) = 0.10
> requires for perl-Acme-PlayCode-0.11-2.fc12.noarch.rpm
perl(Data::Dumper)  
perl(List::MoreUtils)  
perl(:MODULE_COMPAT_5.10.0)  
perl(Moose)  
perl(Moose::Role)  
perl(Path::Class)  
perl(PPI)  
perl(PPI::Token::Comment)  
perl(vars)  

=> perl-Acme-PlayCode-0.11-2.fc12.src.rpm <=
> rpmlint
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
> provides for perl-Acme-PlayCode-0.11-2.fc12.src.rpm
> requires for perl-Acme-PlayCode-0.11-2.fc12.src.rpm
dos2unix  
perl(List::MoreUtils)  
perl(Module::Build)  
perl(Moose) >= 0.57
perl(MooseX::Object::Pluggable) >= 0.0008
perl(Path::Class) >= 0.16
perl(PPI) >= 1.201
perl(Test::More)  
perl(Test::Pod)  

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 497800] Review Request: perl-Acme-PlayCode - Perl module for enhancement of code

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


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


Chris Weyl  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




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

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


[Bug 497800] Review Request: perl-Acme-PlayCode - Perl module for enhancement of code

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


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





--- Comment #3 from Chris Weyl   2009-05-03 13:38:22 EDT 
---
Koji (success) http://koji.fedoraproject.org/koji/taskinfo?taskID=1334190

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the 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   >