[Bug 688033] Review Request: gujarati-typing-booster - Gujarati typing booster table ime for ibus

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


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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2011-03-16 02:00:16 EDT 
---
Thanks for scratch build.

Review:
+ Package following guidelines.
+ sha1sum matches as
fd38b89a1a19f60e212a6b082a744aa8727640b3  gujarati-typing-booster-0.0.1.tar.gz
fd38b89a1a19f60e212a6b082a744aa8727640b3 
../SOURCES/gujarati-typing-booster-0.0.1.tar.gz


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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 684398] Review Request: Webilder - Configurable and automatic desktop wallpaper changer

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


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

--- Comment #11 from Pavel Zhukov pa...@zhukoff.net 2011-03-16 02:33:12 EDT 
---
(In reply to comment #7)
 It looks OK, even though there are still two warnings from rpmlint:
 
 $ rpmlint gnome-applet-webilder-0.7-1.fc14.x86_64.rpm
 gnome-applet-webilder.x86_64: E: no-binary
You cannot use noarch because _libdir of gnome-applets is arch-dependency. So,
You can ignore this error, 
 $ rpmlint gnome-applet-webilder-0.7-1.fc14.src.rpm 
 gnome-applet-webilder.src: W: %ifarch-applied-patch Patch0:
 Webilder-lib64.patch
If upstream fix libdir error for X86_64 You can remove patch from package and
fix this warning.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 489226] Review Request: perl-local-lib - Create and use a local lib/ for perl modules

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


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

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

   What|Removed |Added

 CC||iarn...@gmail.com
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #5 from Iain Arnell iarn...@gmail.com 2011-03-16 03:15:57 EDT ---
Package Change Request
==
Package Name: perl-local-lib
New Branches: el5 el6
Owners: iarnell
InitialCC: perl-sig


Chris isn't interested in epel and has no problem with others branching his
packages.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688056] New: Review Request: sir - A simple application for resizing images

2011-03-16 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: sir - A simple application for resizing images

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

   Summary: Review Request: sir - A simple application for
resizing images
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kumarpraveen.nit...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://kumarpraveen.fedorapeople.org/sir/sir.spec
SRPM URL: http://kumarpraveen.fedorapeople.org/sir/sir-2.1.1-1.fc14.src.rpm
Description: SIR (Simple Image Resizer) is a simple application for resizing
images,inspired by GTPY - ImageResizer But it use C++/QT and QImage class to
convert the images.it also can rotate your images.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688056] Review Request: sir - A simple application for resizing images

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


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

Praveen Kumar kumarpraveen.nit...@gmail.com 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

Tomas Mraz tm...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #38 from Tomas Mraz tm...@redhat.com 2011-03-16 03:44:45 EDT ---
Michal, I have set it again when I posted the previous comment. But then you
reset it again as you posted comment 37. If you get the mid-air collision,
always copy your comment text and load the bug from scratch to avoid 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 522821] Review Request: bluetile - A modern tiling window manager with a gentle learning curve

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


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

--- Comment #20 from Jens Petersen peter...@redhat.com 2011-03-16 04:47:11 
EDT ---
Thanks for testing.

Spec: http://petersen.fedorapeople.org/bluetile/bluetile.spec
SRPM: http://petersen.fedorapeople.org/bluetile/bluetile-0.5.3-2.fc14.src.rpm

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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675947] Review Request: sanlock - a shared disk lock manager

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


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

Angus Salkeld asalk...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WORKSFORME
Last Closed||2011-03-16 04:50:43

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676187] Review Request: csync2 - Cluster sync tool

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


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

Angus Salkeld asalk...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WORKSFORME
Last Closed||2011-03-16 04:49:37

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #39 from Michal Novotny minov...@redhat.com 2011-03-16 05:10:05 
EDT ---
(In reply to comment #38)
 Michal, I have set it again when I posted the previous comment. But then you
 reset it again as you posted comment 37. If you get the mid-air collision,
 always copy your comment text and load the bug from scratch to avoid this.

Well, this is the strange thing about mid-air collisions. I did experience it
just once - for comment 35 but not comment 37 and also sometimes I'm not given
the mid-air collision and the flags are just changed without any attention from
the site user. I'm not the only one that experienced this AFAIK but
unfortunately I'm not sure when it exactly happens, i.e. how reproducible this
is and what are the exact steps to do that to file a report against Bugzilla
itself.

Michal

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 652216] Review Request: rubygem-i18n - New wave Internationalization support for Ruby

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


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

--- Comment #14 from Vít Ondruch vondr...@redhat.com 2011-03-16 05:48:52 EDT 
---
f14 seems to be ok. Thank you very much.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 634052] Review Request: ghc-ConfigFile - configuration file library for Haskell

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


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

Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

--- Comment #8 from Jens Petersen peter...@redhat.com 2011-03-16 06:18:20 EDT 
---
Built for rawhide and now for f13, f14, f15.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 681558] Review Request: libvirt-snmp - SNMP support for libvirt

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


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

Michal Privoznik mpriv...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #9 from Michal Privoznik mpriv...@redhat.com 2011-03-16 06:47:18 
EDT ---
New Package SCM Request
===
Package Name: libvirt-snmp
Short Description: Provide full snmp functionality for libvirtd
Owners: mprivozn
Branches: f15
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688056] Review Request: sir - A simple application for resizing images

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


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

Arun SAG saga...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||saga...@gmail.com
 AssignedTo|nob...@fedoraproject.org|saga...@gmail.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688121] New: Review Request: mod_flvx - FLV progressive download streaming for the Apache HTTP Server

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

Summary: Review Request: mod_flvx - FLV progressive download streaming for the 
Apache HTTP Server

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

   Summary: Review Request: mod_flvx - FLV progressive download
streaming for the Apache HTTP Server
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: redhat-bugzi...@linuxnetz.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://labs.linuxnetz.de/bugzilla/mod_flvx.spec
SRPM URL: http://labs.linuxnetz.de/bugzilla/mod_flvx-0-0.1.20100525git.src.rpm
Description:
FLV streaming means it can be sought to any position during video, and
browser (Flash player) will buffer only from this position to the end.
Thus streaming allows to skip boring parts or see video ending without
loading the whole file, which simply saves bandwidth. Even H264 is more
efficient, FLV is still a common container format for videos, because
H264 is supported by Flash since version 9.115.

For using FLV streaming on the web, a pseudo-streaming compliant Flash
player, such as Flowplayer, is needed. Streaming requires that the FLV
has embedded key-frame markers (meta-data), that can be injected by any
supported tool, e.g. flvtool2.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688121] Review Request: mod_flvx - FLV progressive download streaming for the Apache HTTP Server

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


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

Robert Scheck redhat-bugzi...@linuxnetz.de changed:

   What|Removed |Added

 CC||tcall...@redhat.com
 Blocks||182235(FE-Legal)

--- Comment #1 from Robert Scheck redhat-bugzi...@linuxnetz.de 2011-03-16 
07:44:13 EDT ---
Tom, as already talked with you on IRC, mod_flvx is not being a legal issue
from your point of view, because there's nothing FLV-related itself inside.
May you just state here that officially and lift FE-Legal afterwards? Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683487] Review Request: R-ACCLMA - R contrib package for ACC LMA Graph Plotting

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


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

--- Comment #3 from Madhavprasad Pai mk...@redhat.com 2011-03-16 07:53:11 EDT 
---

Hello Jose',

Thank you for reviewing my request.

I have read https://fedoraproject.org/wiki/PackageMaintainers/Join

I shall take up your suggestion and read other review requests at
http://fedoraproject.org/PackageReviewStatus/NEW.html so that I can help
others.

Thanks again and regards,
-- Pai

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 489226] Review Request: perl-local-lib - Create and use a local lib/ for perl modules

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


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

--- Comment #6 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 08:25:24 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 681558] Review Request: libvirt-snmp - SNMP support for libvirt

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


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

--- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 08:25:39 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 447368] Review Request: schroot - Execute commands in a chroot environment

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


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

--- Comment #25 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 08:24:55 EDT 
---
This SCM request is not valid; it lists no owners.  These are processed by a
script, so please do not deviate from the formats given in
https://fedoraproject.org/wiki/Package_SCM_admin_requests

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #40 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 08:26:01 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 684915] Review Request: php-pecl-igbinary - Replacement for the standard PHP serializer

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


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

--- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 08:26:40 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #41 from Michal Novotny minov...@redhat.com 2011-03-16 08:38:59 
EDT ---
(In reply to comment #40)
 Git done (by process-git-requests).

Thanks a lot!

Michal

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #42 from Michal Novotny minov...@redhat.com 2011-03-16 08:49:24 
EDT ---
Oh, just one thing I can't find answer to. If I put the master (dist-rawhide)
branch to koji using `fedpkg build` I'm getting BuildError: package
php-libvirt not in list for tag dist-f16. Shouldn't master branch go to the
dist-rawhide that should be devel branch by default? Do I have to create a new
SCM request later to support dist-f16 branch too? I'm unable to find those
information on
http://fedoraproject.org/wiki/PackageMaintainers/Join#Get_a_Fedora_Account :(

Also, when trying to sign the package for bodhi I was getting error:

$ fedpkg update
Creating a new update for  php-libvirt-0.4.1-2.fc15 
Password for minovotn: 
Creating a new update for  php-libvirt-0.4.1-2.fc15 
php-libvirt-0.4.1-2.fc15 not tagged as an update candidate
$

What should I do about this?

Thanks,
Michal

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #43 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 08:52:45 EDT ---
php-libvirt-0.4.1-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/php-libvirt-0.4.1-2.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #44 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 08:56:11 EDT ---
php-libvirt-0.4.1-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/php-libvirt-0.4.1-2.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #45 from Tomas Mraz tm...@redhat.com 2011-03-16 09:08:01 EDT ---
No, for rawhide the branch request is not needed. Could you try to build it
again?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 678678] Review Request: rubygem-vte - Ruby binding of vte

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


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

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||vondr...@redhat.com
 AssignedTo|nob...@fedoraproject.org|vondr...@redhat.com

--- Comment #2 from Vít Ondruch vondr...@redhat.com 2011-03-16 09:07:05 EDT 
---
Taking this one for 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 539387] Review Request: InsightToolkit - Medical imaging processing library

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


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

--- Comment #13 from Per Inge Mathisen per.mathi...@gmail.com 2011-03-16 
09:26:24 EDT ---
Created attachment 485731
  -- https://bugzilla.redhat.com/attachment.cgi?id=485731
Updated spec file for version 3.20 with fixes for lib/lib64 issue

I took the latest spec file posted in this ticket and updated it to work
against version 3.20, and worked around a showstopper issue related to it
putting 64bit libraries in /usr/lib for install but expecting them in
/usr/lib64 for packaging. The workaround might be ugly but at least I got an
RPM out of it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #46 from Michal Novotny minov...@redhat.com 2011-03-16 09:34:47 
EDT ---
(In reply to comment #45)
 No, for rawhide the branch request is not needed. Could you try to build it
 again?

Oh, tried to build the master branch again and now it passed.

Thanks,
Michal

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688121] Review Request: mod_flvx - FLV progressive download streaming for the Apache HTTP Server

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


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

Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 Blocks|182235(FE-Legal)|

--- Comment #2 from Tom spot Callaway tcall...@redhat.com 2011-03-16 
09:34:42 EDT ---
Yep. This code makes no attempt to do anything with the FLV format, not even
parsing it, so there are no legal concerns that I am aware of at this time.

Lifting FE-Legal.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688157] New: Review Request: cpupowerutils - Tools to determine and set CPU power related settings

2011-03-16 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: cpupowerutils - Tools to determine and set CPU power 
related settings

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

   Summary: Review Request: cpupowerutils - Tools to determine and
set CPU power related settings
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: psab...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://psabata.fedorapeople.org/packages/cpupowerutils/cpupowerutils.spec
SRPM URL:
http://psabata.fedorapeople.org/packages/cpupowerutils/cpupowerutils-009-0.1.p1.fc14.src.rpm
Description:
This tool is to make access to the Linux kernel's processor power subsystems
like CPU frequency switching (cpufreq) or CPU sleep states (cpuidle) for users
and userspace tools easier.

Note:
This package intends to replace cpufrequtils.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688157] Review Request: cpupowerutils - Tools to determine and set CPU power related settings

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

 Blocks||685158

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680921] Review Request: perl-PathTools - PathTools Perl module (Cwd, File::Spec)

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


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680921] Review Request: perl-PathTools - PathTools Perl module (Cwd, File::Spec)

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


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

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-03-16 
09:56:12 EDT ---
perl-PathTools-3.33-1.fc15,perl-5.12.3-155.fc15 has been submitted as an update
for Fedora 15.
https://admin.fedoraproject.org/updates/perl-PathTools-3.33-1.fc15,perl-5.12.3-155.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 630644] Review Request: perl-Bio-SamTools - Bio::SamTools Perl module

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


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

--- Comment #15 from Petr Pisar ppi...@redhat.com 2011-03-16 10:30:30 EDT ---
Standalone spec file and spec file in SRPM differ. I will use the standalone
one in this review.

Spec file changes:

--- perl-Bio-SamTools.spec 2011-03-13 02:06:15.0 +0100
+++ perl-Bio-SamTools.spec.1 2011-03-14 22:52:48.0 +0100
@@ -1,24 +1,28 @@
 Name:  perl-Bio-SamTools
-Version: 1.21
-Release: 3%{?dist}
+Version: 1.27
+Release: 1%{?dist}
 Summary: Bio::SamTools Perl module
-License: GPL+ or Artistic
+License: Artistic 2.0
 Group:  Development/Libraries
 URL:  http://search.cpan.org/dist/Bio-SamTools/
-Source0:
http://search.cpan.org/CPAN/authors/id/L/LD/LDS/Bio-SamTools-1.21.tar.gz
-#Patch0:  %%{name}-%%{version}-includes.patch
+Source0:
http://search.cpan.org/CPAN/authors/id/L/LD/LDS/Bio-SamTools-%{version}.tar.gz
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 BuildRequires: perl = 0:5.008
-#BuildRequires: perl(Bio::Root::Version) = 1.005009001
-BuildRequires:  perl-bioperl
-BuildRequires:  perl(ExtUtils::CBuilder)
-BuildRequires:  perl(Module::Build)
-BuildRequires:  samtools-devel = 0.1.8
-BuildRequires:  zlib-devel
-#Requires:  perl(Bio::Root::Version) = 1.005009001
+BuildRequires: perl(Bio::PrimarySeq)
+BuildRequires: perl(Bio::SeqFeature::Lite)
+BuildRequires: perl(ExtUtils::CBuilder)
+BuildRequires: perl(ExtUtils::MakeMaker)
+BuildRequires: perl(Module::Build)
+BuildRequires: perl(Exporter)
+BuildRequires: perl(File::Spec)
+BuildRequires: perl(File::Temp)
+BuildRequires: perl(Cwd)
+BuildRequires: perl(Test)
+BuildRequires: samtools-devel = 0.1.8
+BuildRequires: zlib-devel
+
 Requires:  perl-bioperl
 Requires:  perl(:MODULE_COMPAT_%(eval `%{__perl} -V:version`; echo
$version))
-Requires:  samtools-devel = 0.1.8

 %{?perl_default_filter}

@@ -32,7 +36,6 @@
 sed -i 's|= find_sam(|= (%{_includedir}/samtools, %{_libdir}|' Build.PL

 %build
-#export SAMTOOLS=/usr/include/samtools
 %{__perl} Build.PL installdirs=vendor optimize=$RPM_OPT_FLAGS 
 ./Build

@@ -60,6 +63,13 @@
 %{_bindir}/bamToGBrowse.pl

 %changelog
+* Mon Mar 14 2011 Adam Huffman bl...@verdurin.com - 1.27-1
+- new upstream version 1.27
+- remove unnecessary runtime samtools-devel R
+- more fine-grained module BR for bioperl and others
+- fix license
+- remove commented cruft
+
 * Sun Mar 13 2011 Adam Huffman bl...@verdurin.com - 1.21-3
 - better fix for samtools headers from Petr Pisar


Source file is original. Ok.

 License is GPL+ or Artistic 2.0 (LICENSE, lib/Bio/DB/Sam.pm).
 FIX: Correct License tag to reflect Artistic license version. See
 https://fedoraproject.org/wiki/Licensing:Main#Good_Licenses for license names.
 
-License: GPL+ or Artistic
+License: Artistic 2.0

FIX: The package is dual-licensed. User can select between GPL+ and Artistic
2.0. Please mention both licenses in the License tag.

  TODO: You can remove BuildRoot definition and it's purge in %install
  section and %clean section as this is obsolete in Fedora.
 
 I'm hoping to build for EPEL5 - isn't this still needed there?

Yes, it's needed for EPEL5. Then keeping the BuildRoot code makes sense if you
want one spec file for Fedora and EPEL.

Build time dependencies are Ok.

All tests pass. Ok.

 FIX: Remove samtools-devel from Requires. Or is there any reason?
-Requires:  samtools-devel = 0.1.8
Ok.

 FIX: Is the samtools-devel needed at run time?
$ rpmlint perl-Bio-SamTools.spec ../SRPMS/perl-Bio-SamTools-1.27-1.fc14.src.rpm
../RPMS/x86_64/perl-Bio-SamTools-*1.27-1.fc14.x86_64.rpm 
perl-Bio-SamTools.x86_64: W: no-manual-page-for-binary bamToGBrowse.pl
3 packages and 1 specfiles checked; 0 errors, 1 warnings.
rpmlint Ok.

$ rpm -q -lv -p ../RPMS/x86_64/perl-Bio-SamTools-1.27-1.fc14.x86_64.rpm 
-rwxr-xr-x1 rootroot 1913 bře 16 15:03
/usr/bin/bamToGBrowse.pl
drwxr-xr-x2 rootroot0 bře 16 15:03
/usr/lib64/perl5/Bio
drwxr-xr-x2 rootroot0 bře 16 15:03
/usr/lib64/perl5/Bio/DB
drwxr-xr-x2 rootroot0 bře 16 15:03
/usr/lib64/perl5/Bio/DB/Bam
-rw-r--r--1 rootroot11125 bře 16 15:03
/usr/lib64/perl5/Bio/DB/Bam/AlignWrapper.pm
-rw-r--r--1 rootroot17097 bře 16 15:03
/usr/lib64/perl5/Bio/DB/Bam/Alignment.pm
-rw-r--r--1 rootroot  335 bře 16 15:03
/usr/lib64/perl5/Bio/DB/Bam/FetchIterator.pm
-rw-r--r--1 rootroot 2499 bře 16 15:03
/usr/lib64/perl5/Bio/DB/Bam/Pileup.pm
-rw-r--r--1 rootroot 1993 bře 16 15:03
/usr/lib64/perl5/Bio/DB/Bam/PileupWrapper.pm
-rw-r--r--1 rootroot 4596 bře 16 15:03
/usr/lib64/perl5/Bio/DB/Bam/Query.pm
-rw-r--r--1 rootroot  402 bře 16 15:03

[Bug 688183] Review Request: main package name here - short summary here

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


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

Mike Ma zht...@gmail.com changed:

   What|Removed |Added

 CC||zht...@gmail.com
 Blocks||177841(FE-NEEDSPONSOR)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688183] New: Review Request: main package name here - short summary here

2011-03-16 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: main package name here - short summary here

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

   Summary: Review Request: main package name here - short
summary here
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: zht...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ekd123.fedorapeople.org/iceplayer.spec
SRPM URL: http://ekd123.fedorapeople.org/iceplayer-4.0.4-7.fc14.src.rpm
Description: A simple media player. It can download lyrics and show them
automatically, and supports themes, ID3, Equalizer and more.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688033] Review Request: gujarati-typing-booster - Gujarati typing booster table ime for ibus

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


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

Pravin Satpute psatp...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Pravin Satpute psatp...@redhat.com 2011-03-16 10:38:28 
EDT ---
Parag thanks for review,
added git 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688183] Review Request: main package name here - short summary here

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


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

--- Comment #1 from Mike Ma zht...@gmail.com 2011-03-16 10:41:26 EDT ---
mike@mike-laptop SPECS$ rpmlint iceplayer.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

I tried rpmlint, there is no error or warning.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688183] Review Request: main package name here - short summary here

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


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

William Lima wl...@primate.com.br changed:

   What|Removed |Added

 CC||wl...@primate.com.br

--- Comment #2 from William Lima wl...@primate.com.br 2011-03-16 10:43:53 EDT 
---
Please fix your 'Review Summary'.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 678554] Review Request: iceplayer - a simple media player

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


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

Mike Ma zht...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CANTFIX
Last Closed||2011-03-16 10:42:15

--- Comment #19 from Mike Ma zht...@gmail.com 2011-03-16 10:42:15 EDT ---
I have re-opened a Review about this..
Thanks for all helps!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688183] Review Request: iceplayer - A simple music player for Linux

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


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

Mike Ma zht...@gmail.com changed:

   What|Removed |Added

Summary|Review Request: main   |Review Request: iceplayer -
   |package name here - short |A simple music player for
   |summary here   |Linux

--- Comment #3 from Mike Ma zht...@gmail.com 2011-03-16 10:49:57 EDT ---
Everything up-to-date.

New SRPM: http://ekd123.fedorapeople.org/iceplayer-4.0.4-9.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683725] Review Request: marathi-typing-booster - Marathi Typing Booster Table IMEngine for ibus

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-03-16 
10:51:58 EDT ---
marathi-typing-booster-0.0.1-1.fc15 has been submitted as an update for Fedora
15.
https://admin.fedoraproject.org/updates/marathi-typing-booster-0.0.1-1.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683725] Review Request: marathi-typing-booster - Marathi Typing Booster Table IMEngine for ibus

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


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

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 10:53:19 EDT ---
marathi-typing-booster-0.0.1-1.fc14 has been submitted as an update for Fedora
14.
https://admin.fedoraproject.org/updates/marathi-typing-booster-0.0.1-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 684112] Review Request: hindi-typing-booster - Hindi Typing Booster Table IMEngine for ibus

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


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

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-03-16 
10:55:01 EDT ---
hindi-typing-booster-0.0.1-1.fc15 has been submitted as an update for Fedora
15.
https://admin.fedoraproject.org/updates/hindi-typing-booster-0.0.1-1.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688033] Review Request: gujarati-typing-booster - Gujarati typing booster table ime for ibus

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


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

--- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 10:58:03 EDT 
---
There is no SCM request to process here.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688033] Review Request: gujarati-typing-booster - Gujarati typing booster table ime for ibus

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


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

Pravin Satpute psatp...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Pravin Satpute psatp...@redhat.com 2011-03-16 11:05:56 
EDT ---
ohh missed that :( 

New Package SCM Request
===
Package Name: gujarati-typing-booster
Short Description:  Gujarati typing booster table IMEngine for ibus
Owners: pravins
Branches: f14, f15, devel
InitialCC: i18n-team
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 684112] Review Request: hindi-typing-booster - Hindi Typing Booster Table IMEngine for ibus

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-03-16 
11:14:31 EDT ---
hindi-typing-booster-0.0.1-1.fc14 has been submitted as an update for Fedora
14.
https://admin.fedoraproject.org/updates/hindi-typing-booster-0.0.1-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688033] Review Request: gujarati-typing-booster - Gujarati typing booster table ime for ibus

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


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

--- Comment #5 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 11:15:10 EDT 
---
Git done (by process-git-requests).

I'm curious, though; where did you find any documentation that mentions a
Cvsextras Commits line?  That's not something we use in an SCM 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 678678] Review Request: rubygem-vte - Ruby binding of vte

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


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

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review?

--- Comment #3 from Vít Ondruch vondr...@redhat.com 2011-03-16 11:14:52 EDT 
---
* Installation original gem
  - This is apparently upstream bug, comparing the 'vte-0.90.8/ext/vte/depend'
with the 'gtksourceview2-0.90.8/ext/gtksourceview2/depend', which I
reviewed
for you previously. So I am wondering if this bug is reported upstream?
  - This gem repackaging stuff seems to me pretty ugly. I would suggest the
other
way around, i.e. install the gem without the '--with-pkg-config-dir' which
is
working just fine and later move the generated ruby-vte.pc into appropriate
place. Or is there something I am missing?

* gtk2 dependency
  - Upstream specifies gtk2 = 0.90.8 runtime dependency while you allow
dependency on 0.90.7. I just want to ensure that this is intentional, 
not by mistake.

* rpmlint output:
  - rubygem-vte.src: W: strange-permission vte-0.90.8.gem 0600L

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 688033] Review Request: gujarati-typing-booster - Gujarati typing booster table ime for ibus

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


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

--- Comment #6 from Pravin Satpute psatp...@redhat.com 2011-03-16 11:33:29 
EDT ---
Yes, you are right.

It was there 2-3 years back and unfortunately i copied it from my very old
package request and forgot to remove that line. Next time will take care for
same, will use latest one

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 447368] Review Request: schroot - Execute commands in a chroot environment

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


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

Zach Carter z.car...@f5.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #26 from Zach Carter z.car...@f5.com 2011-03-16 11:42:28 EDT ---
Package Change Request
==
Package Name: zachcarter
New Branches: el6
Owners: zachcarter

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 447368] Review Request: schroot - Execute commands in a chroot environment

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


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

--- Comment #28 from Zach Carter z.car...@f5.com 2011-03-16 11:43:11 EDT ---
Package Change Request
==
Package Name: schroot
New Branches: el6
Owners: zachcarter

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 447368] Review Request: schroot - Execute commands in a chroot environment

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


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

--- Comment #27 from Zach Carter z.car...@f5.com 2011-03-16 11:42:52 EDT ---
crap, sorry

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688223] New: Review Request: libdeltacloud - C API for accessing deltacloud

2011-03-16 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: libdeltacloud - C API for accessing deltacloud

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

   Summary: Review Request: libdeltacloud - C API for accessing
deltacloud
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: clala...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://people.redhat.com/clalance/libdeltacloud/rpmreview/libdeltacloud.spec
SRPM URL:
http://people.redhat.com/clalance/libdeltacloud/rpmreview/libdeltacloud-0.7-1.fc14.src.rpm
Description: Libdeltacloud is a library for accessing deltacloud via a
convenient C API.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688223] Review Request: libdeltacloud - C API for accessing deltacloud

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


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

--- Comment #1 from Chris Lalancette clala...@redhat.com 2011-03-16 11:41:18 
EDT ---
I also ran this through rpmlint:

[clalance@localhost libdeltacloud (master)]$ rpmlint libdeltacloud.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[clalance@localhost libdeltacloud (master)]$ rpmlint
~/rpmbuild/SRPMS/libdeltacloud-0.7-1.fc14.src.rpm 
libdeltacloud.src: W: spelling-error Summary(en_US) deltacloud - delta cloud,
delta-cloud, deltaic
libdeltacloud.src: W: spelling-error %description -l en_US deltacloud - delta
cloud, delta-cloud, deltaic
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
[clalance@localhost libdeltacloud (master)]$ 

(the warnings are wrong, since the name of the API that we are targeting is
indeed deltacloud)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] New: Review Request: perl-HTTP-Date - Date conversion routines

2011-03-16 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-HTTP-Date - Date conversion routines

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

   Summary: Review Request: perl-HTTP-Date - Date conversion
routines
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ppi...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ppisar.fedorapeople.org/perl-HTTP-Date/perl-HTTP-Date.spec
SRPM URL:
http://ppisar.fedorapeople.org/perl-HTTP-Date/perl-HTTP-Date-6.00-1.fc16.src.rpm
Description:
This module provides functions that deal the date formats used by the HTTP
protocol (and then some more). Only the first two functions, time2str() and
str2time(), are exported by default.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

--- Comment #1 from Petr Pisar ppi...@redhat.com 2011-03-16 11:44:49 EDT ---
This package collides with perl-libwww-perl-5.837-3.fc15.noarch because this is
new version of module that has been unbundled from perl-libwww-perl-6. I will
add proper Conflicts.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688033] Review Request: gujarati-typing-booster - Gujarati typing booster table ime for ibus

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


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

Pravin Satpute psatp...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-03-16 11:48:26

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688033] Review Request: gujarati-typing-booster - Gujarati typing booster table ime for ibus

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


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

--- Comment #7 from Pravin Satpute psatp...@redhat.com 2011-03-16 11:48:09 
EDT ---
built for master, f15 and f14.

Thanks for quickly processing git 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
 AssignedTo|nob...@fedoraproject.org|psab...@redhat.com
   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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676879] Review Request: mpiexec - MPI job launcher that uses the PBS task interface directly

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


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

Doug Ledford dledf...@redhat.com changed:

   What|Removed |Added

 CC||dledf...@redhat.com,
   ||fenla...@redhat.com

--- Comment #11 from Doug Ledford dledf...@redhat.com 2011-03-16 11:53:28 EDT 
---
Sorry to take so long, but sometimes you're just busy.

So, here's what I see from reading up on the mpiexec website.

First, this doesn't necessarily work with lots of batch systems.  It works with
OpenPBS, PBS Pro, and Torque.  However, these are all three forks of OpenPBS
where OpenPBS appears to be dead, PBS Pro is (I guess) some group taking
OpenPBS and delivering ongoing service and support around it, and Torque is an
actively developed fork of OpenPBS.

Second, it's not intended to be multi-MPI friendly.  Not really anyway.  I know
the web page talks about a lot of MPI packages, but in truth, there are only a
few MPI families, with lots of forks along those families.  There is the mpich
family, which includes mpich, mpich2, mvapich, mvapich2, intel mpi, etc.  Then
there is the lam family, which includes lam and openmpi.  I don't know of any
other open source mpi families that are still alive.  There might be other
closed source mpis out there, but we don't care about those.  In any case, the
mpiexec website basically calls out that lam/openmpi get things right on their
own so there is no need to use mpiexec there and recommends that you don't
(side note: this does not surprise me in the least, the mpich family of job
starting daemons has always been nothing more than a bunch of scripts calling
rsh, hardly what I would call robust or well designed, more like a quick and
dirty job to get things running in the early days and then they never went back
and did things right later).

So, for all the talk on the web site about how many mpis this supports, and how
many PBSes this supports, it really only supports one mpi family and one pbs
family.  Given that, this *absolutely* does not belong in the main path.  I
know that's already been fixed, but I'm putting this here so that someone
doesn't get the idea in the future to undo that fix.

Now, what's more, is I'm not entirely certain that this will work transparently
with different mpich mpis from a single build.  You have to specify the default
communication method in the configure script, as well as a few other options. 
I haven't looked into it, but I know that mvapich and mvapich2 are enough
different from mpich and mpich2 that I'm not certain that the same build
parameters will work with both.  If it doesn't, then you might have to build it
more than once in the spec file with different options, create an mpiexec base
package, and then mpiexec-%{mpiname} sub packages that have the files specific
to that particular mpi implementation.

As an example, if needed, you could create an mpiexec shell script and place it
in the directory specified in your environment modules file.  This shell script
could then execute %{mpidir}/bin/mpiexec-pbs.  You would then place the mpiexec
binary this build process spits out into %{mpidir}/bin/mpiexec-pbs for each of
the mpis you intend to support and place the files into subpackages of mpiexec
specific to each mpi implementation.  That way, if you need different options
for different mpis, it can be done.  However, as I haven't tried to use this
program, I don't know if this is even necessary.  Before the package goes into
Fedora as is though, this needs to be tested.  It's much easier to fix this
before it hits repos than it is after.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

--- Comment #2 from Petr Pisar ppi...@redhat.com 2011-03-16 11:55:37 EDT ---
Updated package with Conflicts is on the same URL.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 426537] Review Request: perl-Test-YAML-Valid - Lets you test the validity of YAML files in unit tests

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


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

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

   What|Removed |Added

 CC||p...@city-fan.org

--- Comment #7 from Paul Howarth p...@city-fan.org 2011-03-16 12:02:41 EDT ---
I would like to request EPEL-4 and EPEL-5 branches of this package. I am
willing to maintain/co-maintain if the current maintainer isn't interested.

This is needed for a Test::CPAN::Meta::YAML package I'm preparing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 684915] Review Request: php-pecl-igbinary - Replacement for the standard PHP serializer

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


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

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

   What|Removed |Added

 Status|NEW |MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 684915] Review Request: php-pecl-igbinary - Replacement for the standard PHP serializer

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


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

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-03-16 
12:08:24 EDT ---
php-pecl-igbinary-1.1.1-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/php-pecl-igbinary-1.1.1-1.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688239] New: Review Request: perl-File-Listing - Parse directory listing

2011-03-16 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-File-Listing - Parse directory listing

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

   Summary: Review Request: perl-File-Listing - Parse directory
listing
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ppi...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://ppisar.fedorapeople.org/perl-File-Listing/perl-File-Listing.spec
SRPM URL:
http://ppisar.fedorapeople.org/perl-File-Listing/perl-File-Listing-6.01-1.fc16.src.rpm
Description:
This module exports a single function called parse_dir(), which can be used
to parse directory listings.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688239] Review Request: perl-File-Listing - Parse directory listing

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


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Depends on||688222

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Blocks||688239

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-review?

--- Comment #3 from Petr Sabata psab...@redhat.com 2011-03-16 12:32:25 EDT ---
Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[x]  Package is named according to the Package Naming Guidelines. [1]
[x]  Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x]  Spec file is legible and written in American English.
[x]  Spec file lacks Packager, Vendor, PreReq tags.
[x]  Spec uses macros instead of hard-coded directory names.
[x]  Package consistently uses macros.
[-]  Macros in Summary, %description expandable at SRPM build time.
[x]  PreReq is not used.
[x]  Requires correct, justified where necessary.
[x]  All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines. [2]
[-]  Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)).
[-]  Package run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) and the beginning of
%install.
[-]  Package use %makeinstall only when ``make install DESTDIR=...'' doesn't
work.
[-]  Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
[-]  The spec file handles locales properly.
[x]  Changelog in prescribed format.
[x]  Rpmlint output is silent.
[x]  License field in the package spec file matches the actual license.
[-]  If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[-]  License file installed when any subpackage combination is installed.
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[3,4]
[x]  Sources contain only permissible code or content.
[x]  Sources used to build the package matches the upstream source, as provided
in the spec URL.
 MD5SUM this package : 41b875ac89c14d7f66515b52cdd12acd
 MD5SUM upstream package : 41b875ac89c14d7f66515b52cdd12acd
[-]  Compiler flags are appropriate.
[-]  %build honors applicable compiler flags or justifies otherwise.
[-]  ldconfig called in %post and %postun if required.
[x]  Package must own all directories that it creates.
[x]  Package does not own files or directories owned by other packages.
[-]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[x]  Permissions on files are set properly.
[x]  Each %files section contains %defattr.
[x]  No %config files under /usr.
[-]  %config files are marked noreplace or the reason is justified.
[-]  Package contains a properly installed %{name}.desktop using
desktop-file-install file if it is a GUI application. [5]
[-]  Package contains a valid .desktop file.
[x]  Package contains code, or permissable content.
[-]  Package contains a SysV-style init script if in need of one.
[x]  File names are valid UTF-8.
[-]  Large documentation files are in a -doc subpackage, if required.
[x]  Package uses nothing in %doc for runtime.
[x]  Package contains no bundled libraries.
[-]  Header files in -devel subpackage, if present.
[-]  Static libraries in -static subpackage, if present.
[x]  Package contains no static executables.
[-]  Package requires pkgconfig, if .pc files are present.
[-]  Development .so files in -devel subpackage, if present.
[-]  Fully versioned dependency in subpackages, if present.
[-]  Package does not contain any libtool archives (.la).
[-]  Useful -debuginfo package or justification otherwise.
[-]  Rpath absent or only used for internal libs.
[x]  Package does not genrate any conflict.
[x]  Package does not contains kernel modules.
[x]  Package is not relocatable.
[-]  Package successfully compiles and builds into binary rpms on at least one
supported architecture.
[x]  Package is not known to require ExcludeArch.
[x]  Package installs properly.
[x]  Package obeys FHS, except libexecdir and /usr/target.
[x]  Package meets the Packaging Guidelines. [6]

=== SUGGESTED ITEMS ===
[?]  Package functions as described.
[x]  Latest version is packaged.
[x]  Package does not include license text files separate from upstream.
[-]  If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[-]  Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x]  SourceX is a working URL.
[-]  SourceX / PatchY prefixed with 

[Bug 688263] New: Review Request: perl-Encode-Locale - Determine the locale encoding

2011-03-16 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-Encode-Locale - Determine the locale encoding

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

   Summary: Review Request: perl-Encode-Locale - Determine the
locale encoding
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ppi...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://ppisar.fedorapeople.org/perl-Encode-Locale/perl-Encode-Locale.spec
SRPM URL:
http://ppisar.fedorapeople.org/perl-Encode-Locale/perl-Encode-Locale-1.01-1.fc16.src.rpm
Description:
In many applications it's wise to let Perl use Unicode for the strings
it processes.  Most of the interfaces Perl has to the outside world is
still byte based.  Programs therefore needs to decode byte strings
that enter the program from the outside and encode them again on the
way out.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676879] Review Request: mpiexec - MPI job launcher that uses the PBS task interface directly

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


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

--- Comment #12 from Christos Triantafyllidis 
christos.triantafylli...@gmail.com 2011-03-16 12:48:19 EDT ---
Hi Doug,
   i would agree with most of the things you are writing but i have the
following comments:
a) As stated in package description, this package supports only the PBS batch
system family. Given that Fedora has only torque in its repositories i would
say that this package supports all PBS batch systems available.

b) Regarding MPI libraries i totally agree that different implementation use
different parameters, namely for packages that are available in fedora at the
moment we have the following libraries:
- MPICH2
- OpenMPI

From those, OpenMPI (as already commented) provides a mpiexec that works as far
as OpenMPI is build against the batch system (namely torque). So i find much
more convenient to have as default the parameters that MPICH needs as this is
actually the one that creates the need for yet another mpiexec. 

Note that this is the DEFAULT communication method, meaning that can be
overridden at any time. I may not know the Fedora packaging internals that well
but i clearly find it a bad idea to have the same code installed multiple times
just to avoid the usage of a parameter. Are you sure this is actually needed to
include it to the distribution?

c) The installation should be totally transparent to anyone who don't intent to
use it as it is enabled by the relevant environment module. For those who
intent to use it, i guess it is not that inconvenient to add their runtime 
parameters if needed.

d) This build (in specific the 0.83 el5 one) has been used extensively in our
Grid cluster on production quality without a single error report.

Finally, i'm not a mpiexec fan... we (as a grid site) are using it and with
current config this seems to make our users/community happy. Given that i'm
building this package for our infrastructure i thought it may be useful for
others. If the resulted package is not usable for us i don't see any reason to
maintain it for fedora :).

Regards,
Christos

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688263] Review Request: perl-Encode-Locale - Determine the locale encoding

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


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Blocks||683784

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688239] Review Request: perl-File-Listing - Parse directory listing

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

--- Comment #4 from Petr Pisar ppi...@redhat.com 2011-03-16 13:03:38 EDT ---
Fixed package:

Spec URL: http://ppisar.fedorapeople.org/perl-HTTP-Date/perl-HTTP-Date.spec
SRPM URL:
http://ppisar.fedorapeople.org/perl-HTTP-Date/perl-HTTP-Date-6.00-1.fc16.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Petr Sabata psab...@redhat.com 2011-03-16 13:09:03 EDT ---
--- perl-HTTP-Date.spec 2011-03-16 16:51:20.0 +0100
+++ perl-HTTP-Date.spec.new 2011-03-16 17:59:13.0 +0100
@@ -7,7 +7,7 @@
 URL:http://search.cpan.org/dist/HTTP-Date/
 Source0:   
http://www.cpan.org/authors/id/G/GA/GAAS/HTTP-Date-%{version}.tar.gz
 BuildArch:  noarch
-BuildArch:  perl(Exporter)
+BuildRequires:  perl(Exporter)
 BuildRequires:  perl(ExtUtils::MakeMaker)
 BuildRequires:  perl(Time::Local)
 BuildRequires:  perl(Time::Zone)


*** 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688239] Review Request: perl-File-Listing - Parse directory listing

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


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

--- Comment #1 from Petr Sabata psab...@redhat.com 2011-03-16 13:07:20 EDT ---
Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[x]  Package is named according to the Package Naming Guidelines. [1]
[x]  Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x]  Spec file is legible and written in American English.
[x]  Spec file lacks Packager, Vendor, PreReq tags.
[x]  Spec uses macros instead of hard-coded directory names.
[x]  Package consistently uses macros.
[-]  Macros in Summary, %description expandable at SRPM build time.
[x]  PreReq is not used.
[x]  Requires correct, justified where necessary.
[x]  All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines. [2]
[-]  Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)).
[-]  Package run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) and the beginning of
%install.
[-]  Package use %makeinstall only when ``make install DESTDIR=...'' doesn't
work.
[-]  Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
[-]  The spec file handles locales properly.
[x]  Changelog in prescribed format.
[x]  Rpmlint output is silent.
[x]  License field in the package spec file matches the actual license.
[-]  If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[-]  License file installed when any subpackage combination is installed.
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[3,4]
[x]  Sources contain only permissible code or content.
[x]  Sources used to build the package matches the upstream source, as provided
in the spec URL.
 MD5SUM this package : cd0b6fd131127be775d6f64044e7cd5d
 MD5SUM upstream package : cd0b6fd131127be775d6f64044e7cd5d
[-]  Compiler flags are appropriate.
[-]  %build honors applicable compiler flags or justifies otherwise.
[-]  ldconfig called in %post and %postun if required.
[x]  Package must own all directories that it creates.
[x]  Package does not own files or directories owned by other packages.
[-]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[x]  Permissions on files are set properly.
[x]  Each %files section contains %defattr.
[x]  No %config files under /usr.
[-]  %config files are marked noreplace or the reason is justified.
[-]  Package contains a properly installed %{name}.desktop using
desktop-file-install file if it is a GUI application. [5]
[-]  Package contains a valid .desktop file.
[x]  Package contains code, or permissable content.
[-]  Package contains a SysV-style init script if in need of one.
[x]  File names are valid UTF-8.
[-]  Large documentation files are in a -doc subpackage, if required.
[x]  Package uses nothing in %doc for runtime.
[x]  Package contains no bundled libraries.
[-]  Header files in -devel subpackage, if present.
[-]  Static libraries in -static subpackage, if present.
[x]  Package contains no static executables.
[-]  Package requires pkgconfig, if .pc files are present.
[-]  Development .so files in -devel subpackage, if present.
[-]  Fully versioned dependency in subpackages, if present.
[x]  Package does not contain any libtool archives (.la).
[-]  Useful -debuginfo package or justification otherwise.
[x]  Rpath absent or only used for internal libs.
[x]  Package does not genrate any conflict.
[x]  Package does not contains kernel modules.
[x]  Package is not relocatable.
[-]  Package successfully compiles and builds into binary rpms on at least one
supported architecture.
[x]  Package is not known to require ExcludeArch.
[x]  Package installs properly.
[x]  Package obeys FHS, except libexecdir and /usr/target.
[x]  Package meets the Packaging Guidelines. [6]

=== SUGGESTED ITEMS ===
[?]  Package functions as described.
[x]  Latest version is packaged.
[-]  Package does not include license text files separate from upstream.
[-]  If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[-]  Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x]  SourceX is a working URL.
[-]  SourceX / PatchY prefixed with %{name}.
[!]  Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[x]  %check is present and all tests pass.
[-]  Usually, subpackages other than devel should require the base package
using a fully versioned 

[Bug 688239] Review Request: perl-File-Listing - Parse directory listing

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


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Blocks||683784

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688264] New: Review Request: perl-Test-CPAN-Meta-YAML - Validate a META.yml file within a CPAN distribution

2011-03-16 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-Test-CPAN-Meta-YAML - Validate a META.yml file 
within a CPAN distribution

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

   Summary: Review Request: perl-Test-CPAN-Meta-YAML - Validate a
META.yml file within a CPAN distribution
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: p...@city-fan.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://subversion.city-fan.org/repos/cfo-repo/perl-Test-CPAN-Meta-YAML/branches/fedora/perl-Test-CPAN-Meta-YAML.spec
SRPM URL:
http://www.city-fan.org/~paul/extras/perl-Test-CPAN-Meta-YAML/perl-Test-CPAN-Meta-YAML-0.17-2.fc16.src.rpm

Description
This module was written to ensure that a META.yml file, provided with a
standard distribution uploaded to CPAN, meets the specifications that are
slowly being introduced to module uploads, via the use of ExtUtils::MakeMaker,
Module::Build and Module::Install.

EPEL-4/5 builds of this package will require branching/building of
perl-Test-YAML-Valid, which I have requested in Bug #426537

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 555248] Review Request: perl-Pod-Eventual - Read a POD document as a series of trivial events

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


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

--- Comment #20 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 12:53:26 EDT ---
perl-Pod-Eventual-0.093330-1.el4 has been submitted as an update for Fedora
EPEL 4.
https://admin.fedoraproject.org/updates/perl-Pod-Eventual-0.093330-1.el4

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 611372] Review Request: perl-Net-Twitter - Perl interface to Twitter

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


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

--- Comment #8 from Julian C. Dunn jd...@aquezada.com 2011-03-16 13:18:55 EDT 
---
(In reply to comment #7)
 Is someone still working on this? I need the Net::Twitter::Lite module and
 there doesn't seem to be an RPM for it currently. Any chance we'll get one in
 the near future?

Hi Steven,

I would love to, but I need the maintainer of Net::OAuth to push the upgraded
version before I can work on this further. See bug 619644.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688263] Review Request: perl-Encode-Locale - Determine the locale encoding

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 539387] Review Request: InsightToolkit - Medical imaging processing library

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


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

Arnaud Gelas arnaud_ge...@hms.harvard.edu changed:

   What|Removed |Added

 CC||arnaud_ge...@hms.harvard.ed
   ||u

--- Comment #14 from Arnaud Gelas arnaud_ge...@hms.harvard.edu 2011-03-16 
13:26:43 EDT ---
When you set cmake variable, you set twice BUILD_EXAMPLES once to ON, once to
OFF.
I guess you should either set it to ON or OFF.

[..]
   -DBUILD_EXAMPLES:BOOL=ON \
   -DCMAKE_BUILD_TYPE:STRING=Release\
   -DCMAKE_VERBOSE_MAKEFILE=OFF\
   -DBUILD_EXAMPLES=OFF\
   -DBUILD_TESTING=OFF\
   -DITK_USE_REVIEW:BOOL=OFF \
   -DITK_USE_PATENTED:BOOL=OFF \
   -DITK_USE_SYSTEM_TIFF=ON \
   -DITK_USE_SYSTEM_PNG=ON \
   -DITK_USE_SYSTEM_ZLIB=ON \
   -DITK_USE_SYSTEM_LIBXML2=ON \
   -DUSE_FFTWD=ON  .
[...]

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688239] Review Request: perl-File-Listing - Parse directory listing

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


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

--- Comment #2 from Petr Pisar ppi...@redhat.com 2011-03-16 13:26:13 EDT ---
Fixed package on the same URL.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688263] Review Request: perl-Encode-Locale - Determine the locale encoding

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Petr Sabata psab...@redhat.com 2011-03-16 13:32:40 EDT ---
Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[x]  Package is named according to the Package Naming Guidelines. [1]
[x]  Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x]  Spec file is legible and written in American English.
[x]  Spec file lacks Packager, Vendor, PreReq tags.
[x]  Spec uses macros instead of hard-coded directory names.
[x]  Package consistently uses macros.
[-]  Macros in Summary, %description expandable at SRPM build time.
[x]  PreReq is not used.
[x]  Requires correct, justified where necessary.
[x]  All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines. [2]
[-]  Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)).
[-]  Package run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) and the beginning of
%install.
[-]  Package use %makeinstall only when ``make install DESTDIR=...'' doesn't
work.
[-]  Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
[-]  The spec file handles locales properly.
[x]  Changelog in prescribed format.
[x]  Rpmlint output is silent.
[x]  License field in the package spec file matches the actual license.
[-]  If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[-]  License file installed when any subpackage combination is installed.
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[3,4]
[x]  Sources contain only permissible code or content.
[x]  Sources used to build the package matches the upstream source, as provided
in the spec URL.
 MD5SUM this package : e92740d24715c60682325b5bd971b38a
 MD5SUM upstream package : e92740d24715c60682325b5bd971b38a
[-]  Compiler flags are appropriate.
[-]  %build honors applicable compiler flags or justifies otherwise.
[-]  ldconfig called in %post and %postun if required.
[x]  Package must own all directories that it creates.
[x]  Package does not own files or directories owned by other packages.
[-]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[x]  Permissions on files are set properly.
[x]  Each %files section contains %defattr.
[x]  No %config files under /usr.
[-]  %config files are marked noreplace or the reason is justified.
[-]  Package contains a properly installed %{name}.desktop using
desktop-file-install file if it is a GUI application. [5]
[-]  Package contains a valid .desktop file.
[x]  Package contains code, or permissable content.
[-]  Package contains a SysV-style init script if in need of one.
[x]  File names are valid UTF-8.
[-]  Large documentation files are in a -doc subpackage, if required.
[x]  Package uses nothing in %doc for runtime.
[x]  Package contains no bundled libraries.
[-]  Header files in -devel subpackage, if present.
[-]  Static libraries in -static subpackage, if present.
[x]  Package contains no static executables.
[-]  Package requires pkgconfig, if .pc files are present.
[-]  Development .so files in -devel subpackage, if present.
[-]  Fully versioned dependency in subpackages, if present.
[x]  Package does not contain any libtool archives (.la).
[-]  Useful -debuginfo package or justification otherwise.
[-]  Rpath absent or only used for internal libs.
[x]  Package does not genrate any conflict.
[x]  Package does not contain kernel modules.
[x]  Package is not relocatable.
[-]  Package successfully compiles and builds into binary rpms on at least one
supported architecture.
[x]  Package is not known to require ExcludeArch.
[x]  Package installs properly.
[x]  Package obeys FHS, except libexecdir and /usr/target.
[x]  Package meets the Packaging Guidelines. [6]

=== SUGGESTED ITEMS ===
[?]  Package functions as described.
[x]  Latest version is packaged.
[-]  Package does not include license text files separate from upstream.
[-]  If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[-]  Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x]  SourceX is a working URL.
[-]  SourceX / PatchY prefixed with 

[Bug 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

Michal Novotny minov...@redhat.com changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-03-16 13:35:15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688239] Review Request: perl-File-Listing - Parse directory listing

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


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

--- Comment #3 from Petr Sabata psab...@redhat.com 2011-03-16 13:35:42 EDT ---
--- perl-File-Listing.spec  2011-03-16 17:09:02.0 +0100
+++ perl-File-Listing.spec.new  2011-03-16 18:22:42.0 +0100
@@ -20,7 +20,9 @@
 Conflicts:  perl-libwww-perl  6

 # Remove underspecified dependencies
-%filter_from_requires /^perl(HTTP::Date)\s*$/
+%filter_from_requires /^perl(HTTP::Date)\s*$/d
+# Do not provide private modules
+%filter_from_provides /^perl(File::Listing::/d
 %filter_setup

 %description
@@ -54,3 +56,4 @@
 - Specfile autogenerated by cpanspec 1.78.
 - Remove BuildRoot stuff
 - Conflict with perl-libwww-perl-5* and older
+- Do not provide private modules

---

Seems alright.
I'll approve once perl-HTTP-Date makes it into Fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688222] Review Request: perl-HTTP-Date - Date conversion routines

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


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

--- Comment #6 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 14:00:41 EDT 
---
I see no SCM request to process.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 447368] Review Request: schroot - Execute commands in a chroot environment

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


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

--- Comment #29 from Jason Tibbitts ti...@math.uh.edu 2011-03-16 14:00:20 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688157] Review Request: cpupowerutils - Tools to determine and set CPU power related settings

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


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 680308] Review Request: php-pear-MDB2-Schema - XML based database schema manager

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


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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 14:18:09 EDT ---
php-pear-MDB2-Schema-0.8.0-2.el5 has been pushed to the Fedora EPEL 5 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 246537] Review Request: perl-Test-Output - Utilities to test STDOUT and STDERR messages

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


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

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-03-16 
14:18:53 EDT ---
perl-Test-Output-0.10-2.el4 has been pushed to the Fedora EPEL 4 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680308] Review Request: php-pear-MDB2-Schema - XML based database schema manager

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


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

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

   What|Removed |Added

   Fixed In Version|php-pear-MDB2-Schema-0.8.5- |php-pear-MDB2-Schema-0.8.5-
   |1.fc14  |1.el6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 681393] Review Request: perl-Nagios-Plugin-WWW-Mechanize - Login to a web page as a user and get data as a Nagios plugin

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 14:18:02 EDT ---
perl-Nagios-Plugin-WWW-Mechanize-0.13-3.el6 has been pushed to the Fedora EPEL
6 testing repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680308] Review Request: php-pear-MDB2-Schema - XML based database schema manager

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


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

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 14:17:27 EDT ---
php-pear-MDB2-Schema-0.8.5-1.el6 has been pushed to the Fedora EPEL 6 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 496473] Review Request: perl-Mixin-Linewise - Write your linewise code for handles; this does the rest

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


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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 14:18:34 EDT ---
perl-Mixin-Linewise-0.002-2.el4 has been pushed to the Fedora EPEL 4 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-03-16 14:18:43 EDT ---
perl-Dist-CheckConflicts-0.02-2.el4 has been pushed to the Fedora EPEL 4 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


  1   2   >