[Bug 472179] Review Request: perl-Test-Block - Specify fine granularity test plans

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 438808] Review Request: php-pear-DB-ldap - A DB compliant interface to LDAP servers

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Remi Collet [EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #4 from Remi Collet [EMAIL PROTECTED]  2008-11-23 03:14:20 EDT ---
Please answer to all your pending reviews, 
there are work todo in comments on each of them

I will be able to end the reviews and to sponsor you.

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

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


[Bug 438810] Review Request: php-pear-HTTP-Download - PHP Class for handling file downloads

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Remi Collet [EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




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

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


[Bug 438811] Review Request: php-pear-HTML-AJAX - PHP and JavaScript AJAX library

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Remi Collet [EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #3 from Remi Collet [EMAIL PROTECTED]  2008-11-23 03:20:58 EDT ---
I must disagree with Lev.

As this package run without HTML_Common (check all the require_once in the
scripts), you don't have to Requires it. So you must also own the
/usr/share/pear/HTML directory.

But %file must be cleared

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

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


[Bug 472664] New: Review Request: perl-HTML-TokeParser-Simple - Easy to use HTML::TokeParser interface

2008-11-23 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-HTML-TokeParser-Simple - Easy to use 
HTML::TokeParser interface

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

   Summary: Review Request: perl-HTML-TokeParser-Simple - Easy to
use HTML::TokeParser interface
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://iarnell.fedorapeople.org/perl-HTML-TokeParser-Simple.spec
SRPM URL:
http://iarnell.fedorapeople.org/perl-HTML-TokeParser-Simple-3.15-1.fc9.src.rpm
Description: 
HTML::TokeParser is an excellent module that's often used for parsing HTML.
However, the tokens returned are not exactly intuitive to parse.  To simplify
this, HTML::TokeParser::Simple allows the user ask more intuitive (read: more
self-documenting) questions about the tokens returned.

This is one of several requirements for perl-Catalyst-Controller-HTML-FormFu.

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

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


[Bug 472666] New: Review Request: perl-DateTime-Format-Natural - Create machine readable date/time with natural parsing logic

2008-11-23 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-DateTime-Format-Natural - Create machine readable 
date/time with natural parsing logic

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

   Summary: Review Request: perl-DateTime-Format-Natural - Create
machine readable date/time with natural parsing logic
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://iarnell.fedorapeople.org/perl-DateTime-Format-Natural.spec
SRPM URL:
http://iarnell.fedorapeople.org/perl-DateTime-Format-Natural-0.73-1.fc9.src.rpm
Description: 
DateTime::Format::Natural takes a string with a human readable date/time
and creates a machine readable one by applying natural parsing logic.

This is one of several requirements for perl-Catalyst-Controller-HTML-FormFu.

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

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


[Bug 438811] Review Request: php-pear-HTML-AJAX - PHP and JavaScript AJAX library

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Lev Shamardin [EMAIL PROTECTED]  2008-11-23 03:34:39 EDT 
---
Agreed, I forgot to check the requires in the sorce code. BTW, may be this
(https://fedoraproject.org/wiki/PackagingDrafts/DirectoryOwnershipImprovement)
is already should be included into the guidelines?

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

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


[Bug 455226] Review Request: php-pecl-runkit - PHP Opcode Analyser

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Remi Collet [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   Flag||needinfo?




--- Comment #3 from Remi Collet [EMAIL PROTECTED]  2008-11-23 03:35:07 EDT ---
Small question :

Do you think it is a good idea to import a very old package which doesn't seem
maintained upstream for quite a long time (june 2006).


Small notes :

Version 0.9 is available, so you cannot use a pre-release version number
(0.9-0.xxx) but a post release (Official 0.9 will have a 0.9-1 number schema)

Please add in comment the cvs export command with the revision or date
retrieved.

Also indicate where the patch come from. If they are needed to build against
latest php version, you should file a bug upstream with your patch attached
(but I see in CSV php 5.3.0 compatibility have been fixed for a few months).

+

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

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


[Bug 472668] New: Review Request: perl-Captcha-reCAPTCHA - Perl implementation of the reCAPTCHA API

2008-11-23 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-Captcha-reCAPTCHA - Perl implementation of the 
reCAPTCHA API

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

   Summary: Review Request: perl-Captcha-reCAPTCHA - Perl
implementation of the reCAPTCHA API
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://iarnell.fedorapeople.org/perl-Captcha-reCAPTCHA.spec
SRPM URL:
http://iarnell.fedorapeople.org/perl-Captcha-reCAPTCHA-0.92-1.fc9.src.rpm
Description: 
reCAPTCHA is a hybrid mechanical turk and captcha that allows visitors who
complete the captcha to assist in the digitization of books.

This is one of several requirements for perl-Catalyst-Controller-HTML-FormFu.

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

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


[Bug 472639] Review Request: Scilab - Numerical Analysis toolkit

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from D Haley [EMAIL PROTECTED]  2008-11-23 03:43:17 EDT ---
Building the documentation will require some things we don't currently have in
the repository. Most notably the JEuclid project. This particular project is
quite large and has a very large number of dependencies, this will probably in
turn require some packages we don't have as well. A quick internet search turns
up no current RPMs for that project.

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

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


[Bug 455226] Review Request: php-pecl-runkit - PHP Opcode Analyser

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Remi Collet [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|needinfo?   |




--- Comment #4 from Remi Collet [EMAIL PROTECTED]  2008-11-23 03:46:34 EDT ---
Also read the PHP Guidelines (and/or read other pecl spec file)
http://fedoraproject.org/wiki/Packaging/PHP

You must have the requires for ABI version

.hu should be removed from release.

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

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


[Bug 438811] Review Request: php-pear-HTML-AJAX - PHP and JavaScript AJAX library

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Remi Collet [EMAIL PROTECTED]  2008-11-23 03:54:10 EDT ---
See
http://fedoraproject.org/wiki/Packaging/Guidelines#FileAndDirectoryOwnership
See 2 (Animal example).

++

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

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


[Bug 472639] Review Request: Scilab - Numerical Analysis toolkit

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Sylvestre Ledru [EMAIL PROTECTED]  2008-11-23 05:24:18 
EDT ---
gluegen is a native library provided by JOGL (libgluegen-rt.so)
To help Scilab finding it, update etc/librarypath.xml and add the path inside.

About jeuclid, the Debian packaging is available:
http://svn.debian.org/viewsvn/debian-science/packages/jeuclid/trunk/debian/
if it 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.

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


[Bug 472673] New: Review Request: pyglet - A cross-platform windowing and multimedia library for Python

2008-11-23 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: pyglet - A cross-platform windowing and multimedia 
library for Python

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

   Summary: Review Request: pyglet - A cross-platform windowing
and multimedia library for Python
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://oget.fedorapeople.org/review/pyglet.spec
SRPM URL: http://oget.fedorapeople.org/review/pyglet-1.1.2-1.fc9.src.rpm
Description: 
pyglet provides an object-oriented programming interface for developing
games and other visually-rich applications for Windows, Mac OS X and
Linux. Some of the features of pyglet are:

* No external dependencies or installation requirements. For most
  application and game requirements, pyglet needs nothing else besides
  Python, simplifying distribution and installation.
* Take advantage of multiple windows and multi-monitor desktops. pyglet
  allows you to use as many windows as you need, and is fully aware of
  multi-monitor setups for use with fullscreen games.
* Load images, sound, music and video in almost any format. pyglet can
  optionally use AVbin to play back audio formats such as MP3,
  OGG/Vorbis and WMA, and video formats such as DivX, MPEG-2, H.264,
  WMV and Xvid.
---

The package have a few issues:

* It depends softly on avbin. Softly means, it uses it if it is present,
otherwise it won't produce errors. avbin is a wrapper around ffmpeg, so it
can't be packaged for Fedora. I am packaging avbin for rpmfusion. pyglet can do
many other things hence I thought I should package it for Fedora.

The question is: Should I remove the last paragraph from the description?

* The source tarball has an examples directory, with bunch of code examples. I
am packaging all these examples in a -examples subpackage. Among them is a game
called astraea. Astraea contains a ttf font file. The Readme file says it is a
free font by I don't think we can include it in Fedora. It is actually this
font:

http://www.myfonts.com/fonts/larabie/saved-by-zero/

Right now, I picked a font that is available in Fedora and made the -examples
subpackage require that font package. Is this a good solution? What is the best
procedure in such cases? 


Please share your opinions.

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

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


[Bug 455226] Review Request: php-pecl-runkit - PHP Opcode Analyser

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Pavel Alexeev [EMAIL PROTECTED]  2008-11-23 05:50:28 EDT 
---
Firstly - thank you for comment.

(In reply to comment #3)
 Small question :
 
 Do you think it is a good idea to import a very old package which doesn't seem
 maintained upstream for quite a long time (june 2006).
This is work now and have not major bugs which must be fixed by upstream.
Furthermore, I do not known any alternatives by functionality yet. So, answer
is yes, I think is it is a very good idea to import this package.


 Small notes :
 
 Version 0.9 is available, so you cannot use a pre-release version number
 (0.9-0.xxx) but a post release (Official 0.9 will have a 0.9-1 number schema)
Hm. It is build from CVS. How I should number its versions - 0.9-1.CVS20080512,
0.9-2.CVS20080512, 0.9-3.CVS20080512? Or 0.9-1.1.CVS20080512,
0.9-1-2.CVS20080512, 0.9-1-3.CVS20080512?

 Please add in comment the cvs export command with the revision or date
 retrieved.
Excuse me, I do not understand. Date of CVS retrieving present in version and
in defined macros
%define CVS 20080512
in spec file. For what and where more I should comment it?

 Also indicate where the patch come from.
This is my patches. And it is reflected in spec changelog.

 If they are needed to build against
 latest php version, you should file a bug upstream with your patch attached
 (but I see in CSV php 5.3.0 compatibility have been fixed for a few months).
Patch not for PHP, patch for this extension to make it compatible with new PHP
ABI.
So, if we speak about file bug to upstream of this pecl extension - this have
not sense, because you said before what it is old and unmaintained.

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

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


[Bug 469470] Review Request: mz - A fast versatile packet generator

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #14 from vivek shah [EMAIL PROTECTED]  2008-11-23 05:57:08 EDT ---
I have modified the package binary and manpage as per your suggestions.
Please assign it to yourself and review it.

Spec URL: http://bonii.fedorapeople.org/spec/mz.spec
SRPM URL: http://bonii.fedorapeople.org/srpms/mz-0.34.5-2.fc9.src.rpm

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

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


[Bug 472668] Review Request: perl-Captcha-reCAPTCHA - Perl implementation of the reCAPTCHA API

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review+




--- Comment #1 from Parag AN(पराग) [EMAIL PROTECTED]  2008-11-23 05:14:59 EDT 
---
Review:
+ package builds in mock (rawhide i386).
koji build= http://koji.fedoraproject.org/koji/taskinfo?taskID=946142
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
36b00fa37fd8a17367e1aa2f3b218f34  Captcha-reCAPTCHA-0.92.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test outputs
All tests successful.
Files=6, Tests=45,  1 wallclock secs ( 0.04 usr  0.01 sys +  0.69 cusr  0.03
csys =  0.77 CPU)
+ Package  perl-Captcha-reCAPTCHA-0.92-1.fc11 =
 Provides: perl(Captcha::reCAPTCHA) = 0.92
 Requires: perl(Carp) perl(HTML::Tiny) perl(LWP::UserAgent) perl(constant)
perl(strict) perl(warnings)

APPROVED.

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

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


[Bug 472676] New: Review Request: partimage

2008-11-23 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: partimage

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

   Summary: Review Request: partimage
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://www.partimage.org/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], [EMAIL PROTECTED],
fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://people.atrpms.net/~pcavalcanti/specs/partimage.spec

SRPM URL:
http://people.atrpms.net/~pcavalcanti/srpms/partimage-0.6.7-2.fc8.src.rpm

Description of problem:


Partimage is a Linux/UNIX partition imaging utility,
which saves partitions, having a supported filesystem, to an image file.
Partimage will only copy data from the used portions of the partition.
The image file can be compressed with gzip or bzip2 to save space,
and can be split into multiple files to be copied on CDs/DVDs.

Partimage was compiled with login disabled,
and supporting the following filesystems:

  ext2fs, ext3fs, fat, fat16, fat32, hfs, hpfs, jfs,
  ntfs, reiserfs, reiserfs-3.5, reiserfs-3.6, ufs, xfs.


Additional Information:

This request has been closed some time ago, and I would like to reopen it.
I have been using partimage for years and have cloned dozens of computers with
it. The spec I am submitting is based on Dag's, Ubuntu and Gentoo. 
I have tested all of its options and everything is working, except the pam
usage via partimaged-passwd. Maybe someone can help me with this.

Also, the buildarch is i386 and ppc, although I run it in x86_64 just fine.

This is the original bug report:

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

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

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


[Bug 472639] Review Request: Scilab - Numerical Analysis toolkit

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Chitlesh GOORAH [EMAIL PROTECTED]  2008-11-23 06:26:37 
EDT ---
Kwizart also was working on a scilab rpm,. To my guess, he needed matio as well
for scilab:
https://bugzilla.redhat.com/show_bug.cgi?id=466737

The following is not a review blocker: 

Is it possible for you to setup a wiki page and file a simple table describing
what are the options/plugins that your package is supporting ?

Perhaps on:
https://fedoraproject.org/wiki/SIGs/SciTech/Scilab

Since scilab can be used by anyone from any field of application, it will be
helpful for other packagers/maintainers to know the actual situation.

It is my case, I'll recommend scilab for electronics on FEL, and will try to
make my best so that fedora has the maximum compatibility with Matlab
environment. Thus your wiki page(which I will help you to) will enable me to
find other packagers and packages missing for the best scilab experience
targeting electronic design with FEL.

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

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


[Bug 472639] Review Request: Scilab - Numerical Analysis toolkit

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Chitlesh GOORAH [EMAIL PROTECTED]  2008-11-23 06:32:35 
EDT ---
This package requires a lot of work before approval. I'll rebuild and bring
forward some review comments later today.

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

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


[Bug 472683] New: Review Request: jpcap - Packet filtering library for Java

2008-11-23 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: jpcap - Packet filtering library for Java

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

   Summary: Review Request: jpcap - Packet filtering library for
Java
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://users.wpi.edu/~dignan/jpcap.spec
SRPM URL: http://users.wpi.edu/~dignan/jpcap-0.7-3.fc10.src.rpm
Description: Jpcap is a Java library for capturing and
sending network packets from Java applications.
This Jpcap package requires Java 1.6 or higher 
and libpcap 0.9 or higher.

This is my first package, so I am looking for a sponsor.

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

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


[Bug 472683] Review Request: jpcap - Packet filtering library for Java

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Patrick Dignan [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||177841




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

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


[Bug 469897] Review Request: swingx - A collection of powerful, useful, and just plain fun Swing components

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Felix Kaechele [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Comment #2 from Felix Kaechele [EMAIL PROTECTED]  2008-11-23 07:23:17 EDT 
---
Looks good so far. Only thing I see is that you didn't comment on the patches
within the SPEC file.  Could you please add a comment wether you filed a bug
with upstream already or if this patches are Fedora specific. Also see
https://fedoraproject.org/wiki/Packaging/Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment

Also I'll be taking this bug for review now.

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

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


[Bug 464781] Review Request: flexdock - Java docking UI element. First package.

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chitlesh GOORAH [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #17 from Chitlesh GOORAH [EMAIL PROTECTED]  2008-11-23 07:30:21 
EDT ---
#
# Copyright  (c) 2008  Daniel Haley
#
# This file and all modifications and additions to the pristine 
#  package are under the same license as the package itself.  
#
# please send bugfixes or comments to [EMAIL PROTECTED]
#


Hmm isn't this a blocker for review ? There should be no other
copyright/license on any fedora spec file except fedora's.

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

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


[Bug 472664] Review Request: perl-HTML-TokeParser-Simple - Easy to use HTML::TokeParser interface

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Iain Arnell [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Iain Arnell [EMAIL PROTECTED]  2008-11-23 07:37:28 EDT ---
New Package CVS Request
===
Package Name: perl-HTML-TokeParser-Simple
Short Description: Easy to use HTML::TokeParser interface
Owners: iarnell
Branches: F-9 F-10
InitialCC: perl-sig

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

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


[Bug 447104] Review Request: qdevelop - development environment for QT4

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #13 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
07:42:00 EDT ---
qdevelop-0.26.1-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/qdevelop-0.26.1-1.fc9

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

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


[Bug 472668] Review Request: perl-Captcha-reCAPTCHA - Perl implementation of the reCAPTCHA API

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Iain Arnell [EMAIL PROTECTED]  2008-11-23 07:39:52 EDT ---
New Package CVS Request
===
Package Name: perl-Captcha-reCAPTCHA
Short Description: Perl implementation of the reCAPTCHA API
Owners: iarnell
Branches: F-9 F-10
InitialCC: perl-sig

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

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


[Bug 472666] Review Request: perl-DateTime-Format-Natural - Create machine readable date/time with natural parsing logic

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Iain Arnell [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Iain Arnell [EMAIL PROTECTED]  2008-11-23 07:38:52 EDT ---
New Package CVS Request
===
Package Name: perl-DateTime-Format-Natural
Short Description: Create machine readable date/time with natural parsing logic
Owners: iarnell
Branches: F-9 F-10
InitialCC: perl-sig

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

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


[Bug 472684] New: Review Request: electric - Sophisticated Java based VLSI CAD System

2008-11-23 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: electric - Sophisticated Java based VLSI CAD System

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

   Summary: Review Request: electric - Sophisticated Java based
VLSI CAD System
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://chitlesh.fedorapeople.org/electric/electric.spec
SRPM URL: http://chitlesh.fedorapeople.org/electric/electric-8.07-1.fc9.src.rpm

Description:
Electric is a sophisticated electrical CAD system that can handle
many forms of circuit design, including custom IC layout (ASICs),
schematic drawing, hardware description language specifications,
and electro-mechanical hybrid layout.

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

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


[Bug 447104] Review Request: qdevelop - development environment for QT4

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #14 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
07:42:59 EDT ---
qdevelop-0.26.1-1.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/qdevelop-0.26.1-1.fc8

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

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


[Bug 461407] Review Request: laf-plugin - Generic plugin framework for Java look-and-feels

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chitlesh GOORAH [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #23 from Chitlesh GOORAH [EMAIL PROTECTED]  2008-11-23 07:48:08 
EDT ---
As for the F8 failed built, the log says:

DEBUG util.py:250:  No Package Found for java-devel = 1:1.6.0

http://koji.fedoraproject.org/koji/getfile?taskID=945622name=root.log

In accordance to: 
http://koji.fedoraproject.org/koji/packageinfo?packageID=1165

F8 has java-devel-1.5.0 from java-1.5.0-gcj-1.5.0.0-17.fc8

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

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


[Bug 472685] New: Review Request: perl-Catalyst-Component-InstancePerContext - Return a new instance a component on each request

2008-11-23 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-Catalyst-Component-InstancePerContext - Return a 
new instance a component on each request

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

   Summary: Review Request:
perl-Catalyst-Component-InstancePerContext - Return a
new instance a component on each request
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://iarnell.fedorapeople.org/perl-Catalyst-Component-InstancePerContext.spec
SRPM URL:
http://iarnell.fedorapeople.org/perl-Catalyst-Component-InstancePerContext-0.001001-1.fc9.src.rpm
Description: Return a new instance a component on each request.

rpmlint shows two warnings, but there's not a lot I can do about long
filenames.

perl-Catalyst-Component-InstancePerContext.src: W: filename-too-long-for-joliet
perl-Catalyst-Component-InstancePerContext-0.001001-1.fc9.src.rpm
perl-Catalyst-Component-InstancePerContext.noarch: W:
filename-too-long-for-joliet
perl-Catalyst-Component-InstancePerContext-0.001001-1.fc9.noarch.rpm

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

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


[Bug 472668] Review Request: perl-Captcha-reCAPTCHA - Perl implementation of the reCAPTCHA API

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Iain Arnell [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 472664] Review Request: perl-HTML-TokeParser-Simple - Easy to use HTML::TokeParser interface

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review+




--- Comment #1 from Parag AN(पराग) [EMAIL PROTECTED]  2008-11-23 06:02:50 EDT 
---
Review:
+ package builds in mock (rawhide i386).
koji build= http://koji.fedoraproject.org/koji/taskinfo?taskID=946153
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
2867824be9cbe3e86e94ca677d042ffe  HTML-TokeParser-Simple-3.15.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test outputs
All tests successful.
Files=7, Tests=188,  0 wallclock secs ( 0.04 usr  0.01 sys +  0.50 cusr  0.04
csys =  0.59 CPU)
+ Package perl-HTML-TokeParser-Simple-3.15-1.fc11 =
 Provides: perl(HTML::TokeParser::Simple) = 3.15
perl(HTML::TokeParser::Simple::Token) = 3.0
perl(HTML::TokeParser::Simple::Token::Comment) = 1.0
perl(HTML::TokeParser::Simple::Token::Declaration) = 1.0
perl(HTML::TokeParser::Simple::Token::ProcessInstruction) = 2.0
perl(HTML::TokeParser::Simple::Token::Tag) = 1.2
perl(HTML::TokeParser::Simple::Token::Tag::End) = 1.0
perl(HTML::TokeParser::Simple::Token::Tag::Start) = 1.0
perl(HTML::TokeParser::Simple::Token::Text) = 1.0

Requires: perl(HTML::Entities) perl(HTML::TokeParser)
perl(HTML::TokeParser::Simple::Token)
perl(HTML::TokeParser::Simple::Token::Comment)
perl(HTML::TokeParser::Simple::Token::Declaration)
perl(HTML::TokeParser::Simple::Token::ProcessInstruction)
perl(HTML::TokeParser::Simple::Token::Tag)
perl(HTML::TokeParser::Simple::Token::Tag::End)
perl(HTML::TokeParser::Simple::Token::Tag::Start)
perl(HTML::TokeParser::Simple::Token::Text) perl(base) perl(strict) perl(vars)

APPROVED.

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

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


[Bug 464781] Review Request: flexdock - Java docking UI element. First package.

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #18 from Chitlesh GOORAH [EMAIL PROTECTED]  2008-11-23 07:33:58 
EDT ---
#001: Missing BuildRequires: ant-commons-logging

#002: Rpmlint issues;

chitlesh(~)[0]$rpmlint
/home/chitlesh/rpmbuild/RPMS/i386/flexdock-0.5.1-4.fc9.i386.rpm
flexdock.i386: W: no-documentation
flexdock.i386: W: non-standard-group Development/Java
flexdock.i386: W: incoherent-version-in-changelog 0.1-4 ['0.5.1-4.fc9',
'0.5.1-4']
flexdock.i386: W: no-soname /usr/lib/libRubberBand.so
flexdock.i386: E: shlib-with-non-pic-code /usr/lib/libRubberBand.so
1 packages and 0 specfiles checked; 1 errors, 4 warnings.

chitlesh(~)[0]$rpmlint
/home/chitlesh/rpmbuild/RPMS/i386/flexdock-debuginfo-0.5.1-4.fc9.i386.rpm
flexdock-debuginfo.i386: E: empty-debuginfo-package

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

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


[Bug 469471] Review Request: skinlf - Java look and feel for swing

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chitlesh GOORAH [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #2 from Chitlesh GOORAH [EMAIL PROTECTED]  2008-11-23 08:00:09 
EDT ---
#001: remove the copyright paragraph, else I won't review this package

#002: avoid renaming spec files : %{name}-X.spec. Always keep the spec filenme
as %{name}.spec. I have seen you did the same for all your packages.

#003: rpmlint issues
chitlesh(~)[0]$rpmlint
/home/chitlesh/rpmbuild/RPMS/i386/skinlf-6.7-2.fc9.i386.rpm
skinlf.i386: W: no-documentation
skinlf.i386: E: description-line-too-long Today many applications must be
skinnable: your CD player, your emailnotifier, even your operating system! Now
with Skin Look And Feel, your Java application is skinnable! Skin Look And Feel
is able to read GTK (The Gimp ToolKit) and KDE (The K Desktop Environment)
Skins to enhance your application GUI controls such as Buttons, Checks, Radios,
Scrollbars, Progress Bar, Lists, Tables, Internal Frames, Colors, Background
Textures, Regular Windows.
skinlf.i386: W: non-standard-group skinlf
skinlf.i386: W: invalid-license Apache
skinlf.i386: E: no-binary

Break the description paragraph into seperate lines.

004: Build errors, possible missing BuildRequires:
 [javac]   ^
[javac]
/home/chitlesh/rpmbuild/BUILD/skinlf-6.7/build/src/com/l2fprod/tools/msstyles/MsStylesToSkinLF.java:78:
warning: com.sun.org.apache.xpath.internal.XPathAPI is Sun proprietary API and
may be removed in a future release  
[javac]   (Element)XPathAPI.
[javac]^

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

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


[Bug 447104] Review Request: qdevelop - development environment for QT4

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
07:38:49 EDT ---
qdevelop-0.26.1-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/qdevelop-0.26.1-1.fc10

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

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


[Bug 472666] Review Request: perl-DateTime-Format-Natural - Create machine readable date/time with natural parsing logic

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review+




--- Comment #1 from Parag AN(पराग) [EMAIL PROTECTED]  2008-11-23 05:07:14 EDT 
---
Review:
+ package builds in mock (development i386).
koji build= http://koji.fedoraproject.org/koji/taskinfo?taskID=946139
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
d5ca703d49ee23e85741d2da1dceeb31  DateTime-Format-Natural-0.73.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test outputs
All tests successful.
Files=9, Tests=248,  8 wallclock secs ( 0.07 usr  0.00 sys +  3.93 cusr  0.07
csys =  4.07 CPU)
+ Package perl-DateTime-Format-Natural-0.73-1.fc11 =
Provides: perl(DateTime::Format::Natural) = 0.73
perl(DateTime::Format::Natural::Base) = 1.15
perl(DateTime::Format::Natural::Lang::Base) = 0.9
perl(DateTime::Format::Natural::Lang::EN) = 1.12

Requires: /usr/bin/perl perl(Carp) perl(Date::Calc) perl(DateTime)
perl(DateTime::Format::Natural) perl(Getopt::Long) perl(List::MoreUtils)
perl(Params::Validate) perl(Term::ReadLine) perl(base) perl(boolean)
perl(constant) perl(strict) perl(warnings)

APPROVED.

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

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


[Bug 466737] Review Request: matio - Library for reading/writing Matlab MAT files

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chitlesh GOORAH [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||472639




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

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


[Bug 472683] Review Request: jpcap - Packet filtering library for Java

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Patrick Dignan [EMAIL PROTECTED]  2008-11-23 07:24:50 EDT 
---
I'd like to note that this library was written by Keita Fujii (see the URL in
the spec file), and I repackaged it to conform with Fedora's standards.

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

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


[Bug 472639] Review Request: Scilab - Numerical Analysis toolkit

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chitlesh GOORAH [EMAIL PROTECTED] changed:

   What|Removed |Added

 Depends on||466737




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

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


[Bug 461407] Review Request: laf-plugin - Generic plugin framework for Java look-and-feels

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #25 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 08:04:27 EDT 
---
(In reply to comment #24)
 (In reply to comment #22)
  The build fails for F8. I cannot determine why from the build logs.
  http://koji.fedoraproject.org/koji/taskinfo?taskID=945616
 
  Something to do with being unable to set up the noarch build architecture. I
  think.
  
  Any advice?
 
 java = 1:1.6 is not available on Fedora

on Fedora 8, I mean.

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

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


[Bug 461407] Review Request: laf-plugin - Generic plugin framework for Java look-and-feels

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #24 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 08:02:12 EDT 
---
(In reply to comment #22)
 The build fails for F8. I cannot determine why from the build logs.
 http://koji.fedoraproject.org/koji/taskinfo?taskID=945616

 Something to do with being unable to set up the noarch build architecture. I
 think.
 
 Any advice?

java = 1:1.6 is not available on Fedora. Fedora 8 uses java icedtea,
which has a virtual Provides java = 1.7.0, which is lower than 1:1.6.0.

On F-9+ Fedora uses java openjdk, which has a virtual Provides java =
1:1.6.0.
So if this package can build with java icedtea you can use BR: java-devel =
1.7
instead (note: 1.7 is lower than 1:1.6.0), however another note that
java icedtea (on Fedora 8) is not available on ppc64.

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

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


[Bug 469470] Review Request: mz - A fast versatile packet generator

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #15 from Patrice Dumas [EMAIL PROTECTED]  2008-11-23 08:44:39 EDT 
---
I don't know if you are speaking about me, but to me the package name is
still too short, si I wouldn't accept th epackage. Besides I have only jumped 
on the review because I try to avoid inflation of short and generic names
in fedora, not because I am interested in doing the review.

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

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


[Bug 472639] Review Request: Scilab - Numerical Analysis toolkit

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from D Haley [EMAIL PROTECTED]  2008-11-23 09:00:08 EDT ---
Is it possible for you to setup a wiki page and file a simple table describing
what are the options/plugins that your package is supporting ?

Perhaps on:
https://fedoraproject.org/wiki/SIGs/SciTech/Scilab

The wiki page has been created.

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

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


[Bug 472639] Review Request: Scilab - Numerical Analysis toolkit

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Sylvestre Ledru [EMAIL PROTECTED]  2008-11-23 09:12:13 
EDT ---
Nice work.
Just a short comment about this, jeuclid can be built with ant (get the
build.xml from the Debian package).

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

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


[Bug 469955] Review Request: fprintd - D-Bus service for Fingerprint reader access

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Bastien Nocera [EMAIL PROTECTED]  2008-11-23 09:32:44 EDT 
---
All fixed.

http://hadess.fedorapeople.org/fprintd/fprintd.spec
http://hadess.fedorapeople.org/fprintd/fprintd-0.1-2.gitaf42ec70f3.fc10.src.rpm

Koji scratch build worked:
http://koji.fedoraproject.org/koji/taskinfo?taskID=946393

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

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


[Bug 469471] Review Request: skinlf - Java look and feel for swing

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from D Haley [EMAIL PROTECTED]  2008-11-23 09:48:55 EDT ---
*sigh* One more time... 

URLS:
Spec: http://www.dhd.selfip.com/427e/skinlf-3.spec
SRPM: http://www.dhd.selfip.com/427e/skinlf-6.7-3.fc9.src.rpm

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

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


[Bug 469471] Review Request: skinlf - Java look and feel for swing

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from D Haley [EMAIL PROTECTED]  2008-11-23 09:47:22 EDT ---
Spec URL:
SRPM URL:

 #001: remove the copyright paragraph, else I won't review this package
Done.

#002: avoid renaming spec files : %{name}-X.spec. Always keep the spec filenme
as %{name}.spec. I have seen you did the same for all your packages.

I'm going to do it again, just to be difficult -- my local copy is always
blahblah.spec, only the bugzilla copies are blahblah-rev.spec. Sorry. How do I
make sure other people on the 'net can see the changes in the spec files after
I have made them? Is there a good way to do this?

#003: rpmlint issues
skinlf.i386: W: no-documentation
Fixed.

skinlf.i386: E: description-line-too-long
Fixed.

skinlf.i386: W: non-standard-group skinlf
Fixed.

004: Build errors, possible missing BuildRequires:
...
warning: com.sun.org.apache.xpath.internal.
XPathAPI is Sun proprietary API and may be removed in a future release  
Not technically errors, these warnings arise due to my usage of the sun xpath
api as a substitute for the org.apache.xpath api (patch #3), which was removed
as of sun JRE1.5 [1]. I can look at this again to see if I can make it work
without the patch, but that will have to wait. The package itself will still
work with the sun API, it just means that I may have to revisit and update
later. I'd rather continue with the current sun API if no-one knows how to
solve this properly, and doesn't object to doing so. Just so we can lift the
block on other bugs


skinlf.i386: W: invalid-license Apache
Not fixed -- I am unsure how to proceed here. If you examine the LICENSE file
in the source, it isn't Apache, contrary to what's on their website [2]. Its a
redistributable with attribution licence, which doesn't appear up in the
rpmlint -iv output. Which licence should I select??


Current RPMlint output:
$ rpmlint -i skinlf-6.7-3.fc9.src.rpm 
skinlf.src: W: invalid-license Apache 2.0


Thanks for taking a look at the package!

[1]
http://java.sun.com/j2se/1.5.0/docs/guide/xml/jaxp/JAXP-Compatibility_150.html#New
[2] https://skinlf.dev.java.net/

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

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


[Bug 469471] Review Request: skinlf - Java look and feel for swing

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from D Haley [EMAIL PROTECTED]  2008-11-23 09:54:08 EDT ---
*sigh* again. There is no www prefix. 

URLS:
Spec: http://dhd.selfip.com/427e/skinlf-3.spec
SRPM: http://dhd.selfip.com/427e/skinlf-6.7-3.fc9.src.rpm

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

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


[Bug 452584] Review Request: mldonkey - Client for several P2P networks

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from Dominik 'Rathann' Mierzejewski [EMAIL PROTECTED]  
2008-11-23 10:36:22 EDT ---
Please move your discussion of the Release tag to fedora-packaging mailing
list. Peter's usage is correct.

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

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


[Bug 472639] Review Request: Scilab - Numerical Analysis toolkit

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Nicolas Chauvet (kwizart) [EMAIL PROTECTED]  2008-11-23 
10:40:19 EDT ---
I've got a spec ready for 5.0.2 and was waiting for 5.0.3 to fix some problem
I've reported upstream. For example, we shouldn't need to patch configure.ac
for the jhall to javahelp2 jar detection (as the problem was reported upstream
by me).
I've also reported a problem with the default search path for the java machine
using alternatives. I will upload a 5.0.3 spec/srpm once I'm back home.

@D Haley
About this:
---
#Scilab installs lots of stuff to /usr/lib/debug  and /usr/src/debug/ for some
reason
rm -rf /%{buildroot}/%{_libdir}/debug
---
Those are debuginfo extracted by rpm. If you have a problem with them , then I
would suggest that you review how your builsystem is done... thoses are
expected to build fine and ends in %{name}-debuginfo rpm package.



@Sylvestre
According to our packaging standard. gluegen should be built separately from
jogl, as we need jogl to be 'buildable' without using 'custom' Jar files at
built time. (every jar file should be removed in prep). For some reason,
separating the gluegen build from jogl isn't that trivial.

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

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


[Bug 469471] Review Request: skinlf - Java look and feel for swing

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 469471 depends on bug 461407, which changed state.

Bug 461407 Summary: Review Request: laf-plugin - Generic plugin framework for 
Java look-and-feels
https://bugzilla.redhat.com/show_bug.cgi?id=461407

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 461407] Review Request: laf-plugin - Generic plugin framework for Java look-and-feels

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #26 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 10:45:26 EDT 
---
By the way if you don't want you don't have to rebuild laf-plugin on F-8
as F-8 support is near end.

Closing as NEXTRELEASE. When you think F-9/10 laf-plugin packages can be
moved from testing to stable, please revisit bodhi and edit (modify) your
push 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.

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


[Bug 471527] Review Request: snmp++ - SNMP C++ library

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #24 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 10:50:11 EDT 
---
Okay, now closing.

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

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


[Bug 457926] Review Request: python-wikimarkup - Python module to format text to Mediawiki syntax

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #32 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 10:51:49 EDT 
---
Okay, now Closing as NEXTRELEASE. 

When you think F-9/10 packages can be
moved from testing to stable, please revisit bodhi and edit (modify) your
push 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.

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


[Bug 459010] Review request: pystatgrab - Python bindings for libstatgrab

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #24 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 10:52:38 EDT 
---
Okay, now Closing as NEXTRELEASE. 

When you think F-9/10 packages can be
moved from testing to stable, please revisit bodhi and edit (modify) your
push 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.

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


[Bug 462813] Review Request: ldtp - Desktop testing framework

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 462813 depends on bug 459010, which changed state.

Bug 459010 Summary: Review request: pystatgrab - Python bindings for libstatgrab
https://bugzilla.redhat.com/show_bug.cgi?id=459010

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 438811] Review Request: php-pear-HTML-AJAX - PHP and JavaScript AJAX library

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 11:49:10 EDT 
---
Then why does this package have to install files under %_datadir/pear/HTML ??

By the way currently on koji-11:
# repoquery --repoid=koji-11 --whatprovides '/usr/share/pear/HTML/*'
php-pear-HTML-QuickForm-advmultiselect-0:1.4.1-1.fc10.noarch
php-pear-HTML-Common-0:1.2.4-2.fc10.noarch
php-pear-HTML-QuickForm-0:3.2.10-1.fc9.noarch
php-pear-HTML-Table-0:1.8.2-1.fc9.noarch
php-pear-HTML-QuickForm-ElementGrid-0:0.1.1-2.fc10.noarch

which all Requires php-pear-HTML-Common directly or indirectly.

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

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


[Bug 459088] Review Request: protobuf - Protocol Buffers - Google's data interchange format

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #35 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 12:01:59 EDT 
---
Okay

+ Now this package itself is good
+ Your pre-review seems good for initial comments

-
This package (protobuf) is APPROVED by mtasaka
-

Please follow the procedure written on:
http://fedoraproject.org/wiki/PackageMaintainers/Join
from Get a Fedora Account.
After you request for sponsorship a mail will be sent to sponsor 
members automatically (which is invisible for you) which notifies 
that you need a sponsor. After that, please also write on
this bug for confirmation that you requested for sponsorship and
your FAS (Fedora Account System) name. Then I will sponsor you.

If you want to import this package into Fedora 9/10, you also have
to look at
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT
(after once you rebuilt this package on koji Fedora rebuilding system).

If you have questions, please ask me.

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

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


[Bug 438811] Review Request: php-pear-HTML-AJAX - PHP and JavaScript AJAX library

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Christopher Stone [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #7 from Christopher Stone [EMAIL PROTECTED]  2008-11-23 12:04:20 
EDT ---
I would recommend removing dos2unix entirely, and just sed which is part of the
core build environment.

Untested:
find docdir -type f -exec sed -i 's/\r//' {} \;

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

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


[Bug 459088] Review Request: protobuf - Protocol Buffers - Google's data interchange format

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #36 from Lev Shamardin [EMAIL PROTECTED]  2008-11-23 12:09:26 EDT 
---
I have filed an application to the packagers group, my FAS login name is
'abbot'.

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

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


[Bug 438811] Review Request: php-pear-HTML-AJAX - PHP and JavaScript AJAX library

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Remi Collet [EMAIL PROTECTED]  2008-11-23 12:15:14 EDT ---
In PEAR extension, the tree is a thematic classification (not a dependency one)
It's an upstream choice that packager cannot change.

Requires must be taken from the PEAR documentation or (best) from the source

In QuickForm.php = require_once 'HTML/Common.php';
So php-pear-HTML-QuickForm must requires php-pear-HTML-Common

In Table.php = require_once 'HTML/Common.php';
So php-pear-HTML-Table must requires php-pear-HTML-Common

In all the script from HTML_AJAX, there is no require/include outside of
HTML/AJAX. So no need to require php-pear-HTML-Common

++

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

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


[Bug 459088] Review Request: protobuf - Protocol Buffers - Google's data interchange format

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #37 from Mamoru Tasaka [EMAIL PROTECTED]  2008-11-23 12:13:47 EDT 
---
Confirmed. Now I am sponsoring you. Please follow Join wiki again.

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

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


[Bug 472696] New: Review Request: vcards-daemon - manage evolution contacts as vcards

2008-11-23 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: vcards-daemon - manage evolution contacts as vcards

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

   Summary: Review Request: vcards-daemon - manage evolution
contacts as vcards
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


A simple daemon which manages a directory of vcards reflecting the contents of
a address book in evolution data server. On user level, it's related to bug
471575 - this part makes vcards available for the extension. However, there are
no technical dependencies.

Source
RPM:http://downloads.sourceforge.net/dt-contacts/vcards-evo-daemon-0.2-1.61.src.rpm?use_mirror=

Spec:
http://downloads.sourceforge.net/dt-contacts/vcards-evo-daemon.spec?use_mirror=

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

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


[Bug 472696] Review Request: vcards-daemon - manage evolution contacts as vcards

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Alec Leamas [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||177841




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

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


[Bug 472696] Review Request: vcards-daemon - manage evolution contacts as vcards

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Alec Leamas [EMAIL PROTECTED]  2008-11-23 12:54:36 EDT ---
This is my second submission for approval;  I need s sponsor. My attempts to
make what's required for being sponsored are documented in bug 471575

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

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


[Bug 472696] Review Request: vcards-daemon - manage evolution contacts as vcards

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Alec Leamas [EMAIL PROTECTED]  2008-11-23 12:54:14 EDT ---
This is my second aubmission for approval;  I need s sponsor. My attempts to
make what's required for being sponsored are documented in bug 471575

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

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


[Bug 472673] Review Request: pyglet - A cross-platform windowing and multimedia library for Python

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Orcan 'oget' Ogetbil [EMAIL PROTECTED]  2008-11-23 
13:40:48 EDT ---
I found out that the font (saved-by-zero) is not free, hence it cannot be
packaged in Fedora:
http://fedoraproject.org/wiki/Licensing/LarabieFontsLicense

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

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


[Bug 460253] Review Request: qle - A QSO Logger and log Editor

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
14:21:09 EDT ---
qle-0.0.10-5.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/qle-0.0.10-5.fc8

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

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


[Bug 463079] Review Request: ibp - A tool to show which IBP beacons are transmitting

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
14:16:44 EDT ---
ibp-0.21-2.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/ibp-0.21-2.fc8

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

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


[Bug 463079] Review Request: ibp - A tool to show which IBP beacons are transmitting

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
14:18:16 EDT ---
ibp-0.21-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/ibp-0.21-1.fc10

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

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


[Bug 463079] Review Request: ibp - A tool to show which IBP beacons are transmitting

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lucian Langa [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 460253] Review Request: qle - A QSO Logger and log Editor

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
14:22:00 EDT ---
qle-0.0.10-5.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/qle-0.0.10-5.fc9

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

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


[Bug 463079] Review Request: ibp - A tool to show which IBP beacons are transmitting

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
14:17:31 EDT ---
ibp-0.21-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/ibp-0.21-1.fc9

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

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


[Bug 460253] Review Request: qle - A QSO Logger and log Editor

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Fedora Update System [EMAIL PROTECTED]  2008-11-23 
14:23:11 EDT ---
qle-0.0.10-5.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/qle-0.0.10-5.fc10

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

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


[Bug 460253] Review Request: qle - A QSO Logger and log Editor

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lucian Langa [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 456138] Review Request: edb - Debugger based on the ptrace API and QT4

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lucian Langa [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




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

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


[Bug 456138] Review Request: edb - Debugger based on the ptrace API and QT4

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lucian Langa [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #8 from Lucian Langa [EMAIL PROTECTED]  2008-11-23 15:38:52 EDT 
---
OK  source files match upstream:
bf13f4ec695dd37bd01c4eff2db63ef9  debugger-0.9.6.tgz
OK  package meets naming and versioning guidelines.
OK  specfile is properly named, is cleanly written and uses macros
consistently.
OK  summary is OK.
OK  description is OK.
OK  dist tag is present.
OK  build root is OK.
OK  license field matches the actual license.
OK  license is open source-compatible.
OK  license text included in package.
OK  BuildRequires are proper.
OK  compiler flags are appropriate.
OK  %clean is present.
OK  package builds in mock (rawhide, x86_64).
OK  package installs properly.
OK  debuginfo package looks complete.
OK  rpmlint is silent.
OK  final provides and requires are sane:
libAnalyzer.so()(64bit)
libBinarySearcher.so()(64bit)
libBookmarks.so()(64bit)
libBreakpointManager.so()(64bit)
libCheckVersion.so()(64bit)
libDebuggerCore.so()(64bit)
libDumpState.so()(64bit)
libELFBinaryInfo.so()(64bit)
libEnvironment.so()(64bit)
libFunctionDB.so()(64bit)
libFunctionFinder.so()(64bit)
libHardwareBreakpoints.so()(64bit)
libHeapAnalyzer.so()(64bit)
libOpcodeSearcher.so()(64bit)
libOpenFiles.so()(64bit)
libReferences.so()(64bit)
libStringSearcher.so()(64bit)
libSymbolViewer.so()(64bit)
edb = 0.9.6-2.fc10
edb(x86-64) = 0.9.6-2.fc10
=
/bin/sh
libQtCore.so.4()(64bit)
libQtGui.so.4()(64bit)
libQtNetwork.so.4()(64bit)
libc.so.6()(64bit)
libgcc_s.so.1()(64bit)
libm.so.6()(64bit)
libpthread.so.0()(64bit)
libstdc++.so.6()(64bit)
OK  %check is not present. I was able to run application and debug a binary
file
N/A no shared libraries are added to the regular linker search paths.
OK  owns the directories it creates.
OK  doesn't own any directories it shouldn't.
OK  no duplicates in %files.
OK  file permissions are appropriate.
N/A scriptlets present look OK
N/A initscript looks OK.
OK  code, not content.
OK  documentation is small, so no -doc subpackage is necessary.
OK  %docs are not necessary for the proper functioning of the package.
OK  no headers.
OK  no pkgconfig files.
OK  no static libraries.
OK  no libtool .la files.
OK  this is a GUI application, desktop file OK

Suggestion: please see if you can provide an icon file for this package. (try
upstream, or try to find a default one)


APPROVED.

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

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


[Bug 472274] Review Request: atari++ - Unix based emulator of the Atari eight bit computers

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Dan Horák [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Dan Horák [EMAIL PROTECTED]  2008-11-23 16:06:39 EDT ---
Imported and built, thanks for the review.

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

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


[Bug 456138] Review Request: edb - Debugger based on the ptrace API and QT4

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Nicoleau Fabien [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #9 from Nicoleau Fabien [EMAIL PROTECTED]  2008-11-23 16:12:05 
EDT ---
Thank you for the review Lucian. I'll ask upstream for an icon file.

New Package CVS Request
===
Package Name: edb
Short Description: A debugger based on the ptrace API and Qt4
Owners: eponyme
Branches: F-8 F-9 F-10
InitialCC:

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

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


[Bug 472713] New: Review Request: Smokeping - keeps track of your network latency

2008-11-23 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: Smokeping - keeps track of your network latency

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

   Summary: Review Request: Smokeping - keeps track of your
network latency
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://files.thaumaturge.org/users/dante/smokeping/smokeping.spec
SRPM URL:
http://files.thaumaturge.org/users/dante/smokeping/smokeping-2.4.2-0.1.src.rpm
Description: 
SmokePing is a deluxe latency measurement tool. It can measure, store and
display latency, latency distribution and packet loss. SmokePing uses RRDtool
to maintain a longterm data-store and to draw pretty graphs, giving up to the
minute information on the state of each network connection.

This is my second package and I have no sponsor. I have based it on a old
spec-file used inhouse at our company. But I still have some troubles with
SELinux-enabled installations since the httpd needs to read and write outside
its normal locations.

Reference: 
My first package (Rancid):
https://bugzilla.redhat.com/show_bug.cgi?id=451189

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

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


[Bug 451189] Review Request: rancid - Really Awesome New Cisco confIg Differ

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Åge Olai Johnsen [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|needinfo?([EMAIL PROTECTED] |
   |org)|




--- Comment #24 from Åge Olai Johnsen [EMAIL PROTECTED]  2008-11-23 19:31:30 
EDT ---
Sorry for the enormous delay! 

I've submited a second review request:
Bug 472713 - Review Request: Smokeping - keeps track of your network latency 
https://bugzilla.redhat.com/show_bug.cgi?id=472713

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

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


[Bug 472713] Review Request: Smokeping - keeps track of your network latency

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||[EMAIL PROTECTED]
 Resolution||NOTABUG




--- Comment #1 from Itamar Reis Peixoto [EMAIL PROTECTED]  2008-11-23 
19:36:14 EDT ---
this package alrealy exists in fedora.

http://koji.fedoraproject.org/koji/packageinfo?packageID=7035

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

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


[Bug 472673] Review Request: pyglet - A cross-platform windowing and multimedia library for Python

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Orcan 'oget' Ogetbil [EMAIL PROTECTED]  2008-11-23 
20:20:00 EDT ---
New files:

Spec URL: http://oget.fedorapeople.org/review/pyglet.spec
SRPM URL: http://oget.fedorapeople.org/review/pyglet-1.1.2-2.fc9.src.rpm

%changelog
- Removed the extra font dependency (reverting to the default system font).
- Added Requires.


I thought the dependency on a totally irrelevant fonts package is not that
necessary, so I made the script fall back to the default system font.

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

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


[Bug 469843] Review Request: unhide - Tool to find hidden processes and TCP/UDP ports from rootkits

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
  Alias||unhide




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

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


[Bug 469527] Review Request: tcping - Check of TCP connection to a given IP/Port

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
  Alias||tcping




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

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


[Bug 469471] Review Request: skinlf - Java look and feel for swing

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rudolf Kastl [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #6 from Rudolf Kastl [EMAIL PROTECTED]  2008-11-23 21:13:58 EDT 
---
build seems to error out aswell without laf-plugin installed so shouldnt that
also have a BuildRequires on it?

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

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


[Bug 469470] Review Request: mz - A fast versatile packet generator

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #16 from vivek shah [EMAIL PROTECTED]  2008-11-23 23:40:00 EDT ---
Now this becomes a bit misleading and confusing, if you see my comments above I
had raised the same question about renaming the binaries and manpage without
renaming the package name and was suggested to rename the binaries and manpage
and get done with it.

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

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


[Bug 472170] Review Request: perl-Check-ISA - DWIM, correct checking of an object's class

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 472170 depends on bug 472167, which changed state.

Bug 472167 Summary: Review Request: perl-asa - Lets your class/object say it 
works like something else
https://bugzilla.redhat.com/show_bug.cgi?id=472167

   What|Old Value   |New Value

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



--- Comment #2 from Chris Weyl [EMAIL PROTECTED]  2008-11-24 00:00:21 EDT ---
Koji scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=947316

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

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


[Bug 469843] Review Request: unhide - Tool to find hidden processes and TCP/UDP ports from rootkits

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


manuel wolfshant [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #3 from manuel wolfshant [EMAIL PROTECTED]  2008-11-24 00:08:54 
EDT ---
I've just taken a look at the project page and as far as I see, upstream is a
bit inconsistent with the name of the source. First version was unhide.tgz,
followed by unhide02112007.tgz while the current version seems to be
unhide20080519.tgz (note the change from day-month-year to year-month-date).
I suggest to follow
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Snapshot_packages in
naming your package (i.e. change
  Version:20080519
  Release:2%{?dist}
to 
  Version:1.0
  Release:2%{?dist}.20080519
thus avoiding problems in the future.

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

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


[Bug 472348] Review Request: hunspell-no - Norwegian hunspell dictionaaries

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review+




--- Comment #1 from Parag AN(पराग) [EMAIL PROTECTED]  2008-11-24 00:14:13 EDT 
---
Review:
+ package builds in mock.
 Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=947321
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream.
dab12f79c6ccc9b92b23733bb29fc234  no_NO-pack2-2.0.10.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Not a GUI app.

APPROVED.

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

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


[Bug 472685] Review Request: perl-Catalyst-Component-InstancePerContext - Return a new instance a component on each request

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review?




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

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


[Bug 472170] Review Request: perl-Check-ISA - DWIM, correct checking of an object's class

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review+




--- Comment #3 from Parag AN(पराग) [EMAIL PROTECTED]  2008-11-24 00:16:16 EDT 
---

Review:
+ package builds in mock (rawhide i386).
koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=947323
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
7d118aadd4069b4287f309482776a2bd  Check-ISA-0.04.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
All tests successful.
Files=3, Tests=83,  0 wallclock secs ( 0.02 usr  0.01 sys +  0.31 cusr  0.02
csys =  0.36 CPU)

+ Package perl-Check-ISA-0.04-1.fc11 =
  Provides: perl(Check::ISA) = 0.04
  Requires: perl(Scalar::Util) perl(Sub::Exporter) perl(constant) perl(strict)
perl(warnings) perl(warnings::register)

APRPOVED.

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

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


[Bug 472685] Review Request: perl-Catalyst-Component-InstancePerContext - Return a new instance a component on each request

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl [EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




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

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


[Bug 472685] Review Request: perl-Catalyst-Component-InstancePerContext - Return a new instance a component on each request

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag|fedora-review?  |fedora-review+




--- Comment #1 from Chris Weyl [EMAIL PROTECTED]  2008-11-24 00:25:39 EDT ---
Everything looks good.

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

Spec looks sane, clean and consistent; license is correct (perl); make test
passes cleanly.

Source tarball matches upstream:

dd3e9c804cb7f8d6755132d92b26a80a 
Catalyst-Component-InstancePerContext-0.001001.tar.gz
dd3e9c804cb7f8d6755132d92b26a80a 
Catalyst-Component-InstancePerContext-0.001001.tar.gz.srpm

Provides, requires and rpmlint looks sane:

= perl-Catalyst-Component-InstancePerContext-0.001001-1.fc9.noarch.rpm
 rpmlint
perl-Catalyst-Component-InstancePerContext.noarch: W:
filename-too-long-for-joliet
perl-Catalyst-Component-InstancePerContext-0.001001-1.fc9.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
 provides for
perl-Catalyst-Component-InstancePerContext-0.001001-1.fc9.noarch.rpm
perl(Catalyst::Component::InstancePerContext) = 0.001001
perl-Catalyst-Component-InstancePerContext = 0.001001-1.fc9
 requires for
perl-Catalyst-Component-InstancePerContext-0.001001-1.fc9.noarch.rpm
perl(:MODULE_COMPAT_5.10.0)  
perl(Catalyst)  
perl(Moose)  
perl(Moose::Role)  
perl(Scalar::Util)  
perl(strict)  
perl(warnings)  

APPROVED

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

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


[Bug 472724] New: Review Request: xjparse - wrapper for the Xerces XML Schema validator

2008-11-23 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: xjparse - wrapper for the Xerces XML Schema validator

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

   Summary: Review Request: xjparse - wrapper for the Xerces XML
Schema validator
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://bashton.fedorapeople.org/xjparse.spec
SRPM URL: http://bashton.fedorapeople.org/xjparse-1.0-1.fc9.src.rpm
Description: 
The xjparse tool is a simple command-line wrapper for the Xerces XML Schema
validator. It accepts several options, notably one which specifies the set of
schemas to be used during validation.

I have scratch-built it for fedora 9 with koji on all arch
http://koji.fedoraproject.org/koji/taskinfo?taskID=947352

This is my first package so I need a sponsorship. More packages are on the way
once this happens.

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

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


[Bug 472724] Review Request: xjparse - wrapper for the Xerces XML Schema validator

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Brennan Ashton [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||177841




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

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


[Bug 472335] Review Request: perl-rpm-build-perl - Perl compiler backend to extract Perl dependencies

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||472726




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

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


[Bug 472726] Review Request: perl-Test-Dependencies - Ensure that your Makefile.PL specifies all module dependencies

2008-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl [EMAIL PROTECTED] changed:

   What|Removed |Added

 Depends on||472335




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

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


  1   2   >