[Bug 451925] Review Request: synce-hal - Connection framework and dccm-implementation

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

Summary: Review Request: synce-hal - Connection framework and 
dccm-implementation


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


[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, or are watching someone who is.

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


[Bug 225671] Merge Review: curl

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

Summary: Merge Review: curl


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 02:11 EST ---
curl-7.18.2-1.fc8 has been submitted as an update for Fedora 8

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 452078] New: Review Request: node - Simple node front end, modelled after the node shells of TheNet and G8BPQ nodes

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




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

   Summary: Review Request: node - Simple node front end, modelled
after the node shells of TheNet and G8BPQ nodes
   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: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://lucilanga.fedorapeople.org/node.spec
SRPM URL: http://lucilanga.fedorapeople.org/node-0.3.2-2.fc9.src.rpm
Description: This is a simple node frontend for Linux kernel AX.25, NETROM,
ROSE and TCP. It's based on pms.c by Alan Cox (GW4PTS) but has been
heavily modified since. It's probably not very well tested, not
pretty, not very flexible and it is certainly not ready! However
I think it's already somewhat usable

Few notes:
This package is not under active development.
We need to provide this package because all AX25-HOWTO refers it, and I think 
it would help users.
rpmlint is silent with a few exceptions:

E: zero-length /var/ax25/node/loggedin
this is an empty file node cannot create on it's own

W: non-standard-dir-in-var ax25
/var/ax25 this is the standard location directory for ax25 applications. I 
think we should get an exception for that in rpmlint.

Package does not have a standard configure script, here configure is just a 
bash script.
so sh configure was put to inhibit rpmlint warning of not using %configure 
macro.

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

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


[Bug 445687] Review Request: portreserve - TCP port reservation utility

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

Summary: Review Request: portreserve - TCP port reservation utility


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 04:33 EST ---
(In reply to comment #1)
 - Build on mock (x86_64) failed:

I just tried a mock build of portreserve-0.0.1-2.fc8 for fedora-9-x86_64 and it
built fine.

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

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


[Bug 444257] Review Request: nted - Musical score editor

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

Summary: Review Request: nted - Musical score editor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 04:41 EST ---
Ping? Michel, you can proceed now with the CVS stuff.


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

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


[Bug 452087] New: Review Request: collectl - utility to collect performace data

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




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

   Summary: Review Request: collectl -  utility to collect
performace data
   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: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://fedorapeople.org/~kzak/collectl.spec
SRPM URL: http://fedorapeople.org/~kzak/collectl-2.6.4-1.fc8.src.rpm
Description: A utility to collect linux performance data

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

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


[Bug 452087] Review Request: collectl - utility to collect performace data

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

Summary: Review Request: collectl -  utility to collect performace data


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   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, or are watching someone who is.

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


[Bug 452087] Review Request: collectl - utility to collect performace data

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

Summary: Review Request: collectl -  utility to collect performace data


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 05:30 EST ---
New Package CVS Request
===
Package Name: collectl
Short Description: A utility to collect linux performance data
Owners: kzak, sharkcz
Branches: 
InitialCC: sharkcz
Cvsextras Commits: yes

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

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


[Bug 450323] Review Request: coq - Coq proof management system

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

Summary: Review Request: coq - Coq proof management system


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 05:38 EST ---
Build failure is limited to ppc64 architecture, potential fix for now would be
an architecture exclusion, like with:

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

(That version was made with an altered SPEC file that just includes the line
ExcludeArch ppc64.)

Perhaps the problem is with OCaml support on ppc64? I remember reading that
there were some difficulties about this in the recent past.

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

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


[Bug 452087] Review Request: collectl - utility to collect performace data

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

Summary: Review Request: collectl -  utility to collect performace data


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 05:24 EST ---
OK  source files match upstream:
4b90523b13cfb5f7c41bfa4fe07c542ed8150b67  
collectl-2.6.4-1.src.tar.gz
OK  package meets naming and versioning guidelines.
OK  specfile is properly named, is cleanly written and uses macros 
consistently.
OK  dist tag is present.
OK  build root is correct.
OK  license field matches the actual license.
OK  license is open source-compatible. License texts are included in 
package.
OK  latest version is being packaged.
OK  BuildRequires are proper.
N/A compiler flags are appropriate.
OK  %clean is present.
OK  package builds in mock (Rawhide/x86_64).
N/A debuginfo package looks complete.
OK  rpmlint is silent.
OK  final provides and requires look sane.
N/A %check is present and all tests pass.
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.
OK  correct initscript scriptlets are present.
OK  code, not content.
OK  documentation is small, so no -docs subpackage is necessary.
OK  %docs are not necessary for the proper functioning of the package.
OK  no headers.
OK  no pkgconfig files.
OK  no libtool .la droppings.
OK  not a GUI app.

this package is 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, or are watching someone who is.

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


[Bug 444744] Review Request: perl-Lingua-Preferred - Pick a language based on user's preferences

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

Summary: Review Request: perl-Lingua-Preferred - Pick a language based on 
user's preferences


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: perl-Lingua-|Review Request: perl-Lingua-
   |Preferred - Perl extension  |Preferred - Pick a language
   |to choose a language|based on user's preferences




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 06:52 EST ---
Thx for the review, I will update the Summary, but i didn't get what to  with
the URL. If I stay as it is, maybe it can get fixed later. But if I comment the
URL, this suggest the problem in is the spec (missing URL field) whereas the
problem seems releated to CPAN...
This is my point of view. But I will follow advices.

I have email cpan admin, but without answer yet. I will resent a mail to the
owner of the module.

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

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


[Bug 444744] Review Request: perl-Lingua-Preferred - Pick a language based on user's preferences

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

Summary: Review Request: perl-Lingua-Preferred - Pick a language based on 
user's preferences


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 06:54 EST ---
New Package CVS Request
===
Package Name: perl-Lingua-Preferred
Short Description: Pick a language based on user's preferences
Owners: kwizart
Branches: F-8 F-9 EL-4 EL-5
InitialCC: perl-sig
Cvsextras Commits: yes

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

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


[Bug 448613] Review Request: perl-EV - Wrapper for the libev high-performance event loop library

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

Summary: Review Request: perl-EV - Wrapper for the libev high-performance event 
loop library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 07:58 EST ---
I've got an answer from the upstream developer.
perl-AnyEvent doesn't need need any dependencies (some will need to be
filtered). So I guess perl-EV will BR perl-AnyEvent (or at least Requires at
runtime).
Once perl-EV is installed, perl-AnyEvent will elect this module (instead of
perl(Glib) or others ,because it is the fastest.
And I guess the dependent modules will only requires perl(AnyEvent), so i still
need to sort this out, and submit perl-AnyEvent for review first...

About the libev be built internally, here is the upstream answer:
--
EV comes with it's own copy of libev and cannot possibly links against a
preinstalled system libev (they are ABI-incompatible).

Therefore, EV itself has no external dependencies (but of course it is
arch-dependent).
--
I don't know if it is how perl arch dependant modules will works.

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

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


[Bug 187243] Review Request: lazarus : IDE and RAD tool for the free pascal compiler (fpc)

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

Summary: Review Request: lazarus : IDE and RAD tool for the free pascal 
compiler (fpc)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 08:45 EST ---
New Package CVS Request
===
Package Name: lazarus
Short Description: Lazarus Component Library and IDE for Freepascal
Owners: joost
Branches: F-9
InitialCC: joost
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 452105] New: Review Request: php-pear-Auth-RADIUS - Wrapper Classes for the RADIUS PECL

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




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

   Summary: Review Request: php-pear-Auth-RADIUS - Wrapper Classes
for the RADIUS PECL
   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: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://xulchris.fedorapeople.org/packages/php-pear-Auth-RADIUS.spec
SRPM URL: 
http://xulchris.fedorapeople.org/packages/php-pear-Auth-RADIUS-1.0.6-1.fc9.src.rpm

Description:
Wrapper Classes for the RADIUS PECL.

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

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


[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP

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

Summary: Review Request: php-pear-Auth - provides methods for creating an 
authentication system using PHP
Alias: php-pear-Auth

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||452105




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

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


[Bug 442714] Review Request: sat-solver - Satisfyability Solver library which can be used to compute inter-package dependencies.

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

Summary: Review Request: sat-solver - Satisfyability Solver library which can 
be used to compute inter-package dependencies.


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 09:19 EST ---
I can include them and remove that patch that disables the test suite at 
import-I can include them and remove that patch that disables the test suite at 
import-time, if needed

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

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


[Bug 442714] Review Request: sat-solver - Satisfyability Solver library which can be used to compute inter-package dependencies.

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

Summary: Review Request: sat-solver - Satisfyability Solver library which can 
be used to compute inter-package dependencies.


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 09:26 EST ---
Uhm, sorry for the weird comment.. it seems that konqueror4 is not much usable.

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

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


[Bug 452108] New: Review Request: cfdg-fe - A frontend for cfdg

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




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

   Summary: Review Request: cfdg-fe - A frontend for cfdg
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: low
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


SPEC: http://zanoni.jcomserv.net/fedora/cfdg-fe/cfdg-fe.spec
SRPM: http://zanoni.jcomserv.net/fedora/cfdg-fe/cfdg-fe-0.1-1.fc9.src.rpm

A front end to cfdg

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

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


[Bug 452108] Review Request: cfdg-fe - A frontend for cfdg

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

Summary: Review Request: cfdg-fe - A frontend for cfdg


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||452107




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

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


[Bug 452107] Review Request: cfdg - Context Free Design Grammar

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

Summary: Review Request: cfdg - Context Free Design Grammar


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||452108
  nThis||




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

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


[Bug 452107] New: Review Request: cfdg - Context Free Design Grammar

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




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

   Summary: Review Request: cfdg - Context Free Design Grammar
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: low
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


SPEC: http://zanoni.jcomserv.net/fedora/cfdg/cfdg.spec
SRPM: http://zanoni.jcomserv.net/fedora/cfdg/cfdg-2.1-1.fc9.src.rpm

Context Free is a program that generates images from written instructions 
called a grammar. The program follows the instructions in a few seconds to 
create images that can contain millions of shapes.

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

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


[Bug 187243] Review Request: lazarus : IDE and RAD tool for the free pascal compiler (fpc)

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

Summary: Review Request: lazarus : IDE and RAD tool for the free pascal 
compiler (fpc)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 10:44 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 452087] Review Request: collectl - utility to collect performace data

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

Summary: Review Request: collectl -  utility to collect performace data


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 10:49 EST ---
cvs done.

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

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


[Bug 444744] Review Request: perl-Lingua-Preferred - Pick a language based on user's preferences

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

Summary: Review Request: perl-Lingua-Preferred - Pick a language based on 
user's preferences


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 10:47 EST ---
cvs done.

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

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


[Bug 191473] Review Request: kdiff3: Compare + merge 2 or 3 files or directories

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

Summary: Review Request: kdiff3: Compare + merge 2 or 3 files or directories


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 12:03 EST ---
Created an attachment (id=309861)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=309861action=view)
removing two lines from doc/nl/index.docbook allows build in C4


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 191473] Review Request: kdiff3: Compare + merge 2 or 3 files or directories

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

Summary: Review Request: kdiff3: Compare + merge 2 or 3 files or directories


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 12:01 EST ---
Quick and dirty patched spec which allows build for EL-4. Works for me in 
C4.6/i386.

Explanation: build fails due to doxygen failing to understand something in
doc/nl/docindex. I've just removed the offending two lines. I apologize towards
.nl users and I would be happy to see a proper fix. I simply was in need of a
functional C4 version, I have no idea about proper doxygen usage and this hack
looked cleaner than removing the whole translation (in this language).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 443469] Review Request: pytrainer - A tool to log all your sport excursions

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

Summary: Review Request: pytrainer - A tool to log all your sport excursions


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|NEW
   Flag|needinfo?([EMAIL PROTECTED]|
   |net)|




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 12:04 EST ---
Spec URL:
https://rpm.silfreed.net:8002/index.cgi/file/beb4427ffcf1/pytrainer/pytrainer.sp
SRPM URL:
http://www.silfreed.net/download/repo/packages/pytrainer/pytrainer-1.5.0.0.1-3.s

%changelog
* Thu Jun 19 2008 Douglas E. Warner [EMAIL PROTECTED] 1.5.0.0.1-3
- updating to support xulrunner or firefox
- removing empty doc line
- added patch to use sqlite3

I've emailed the maintainer about adding a LICENSE or COPYING file; I'll let you
know once that's in.

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

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


[Bug 448312] Review Request: lostlabyrinth-sounds - Lost Labyrinth sounds

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

Summary: Review Request: lostlabyrinth-sounds - Lost Labyrinth sounds


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 12:21 EST ---
Thanks for the review! Imported, build and pushed as F-8 and F-9 update, 
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, or are watching someone who is.

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


[Bug 448311] Review Request: lostlabyrinth - Lost Labyrinth is a coffeebreak dungeon crawling game

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

Summary: Review Request: lostlabyrinth - Lost Labyrinth is a coffeebreak 
dungeon crawling game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 12:23 EST ---
Thanks for the review! Imported, build and pushed as F-8 and F-9 update, 
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, or are watching someone who is.

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


[Bug 448313] Review Request: lostlabyrinth-graphics - Lost Labyrinth graphics

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

Summary: Review Request: lostlabyrinth-graphics - Lost Labyrinth graphics


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 12:22 EST ---
Thanks for the review! Imported, build and pushed as F-8 and F-9 update, 
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, or are watching someone who is.

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


[Bug 448292] Review Request: fbpanel - a lightweight X11 desktop panel

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

Summary: Review Request: fbpanel - a lightweight X11 desktop panel


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 12:49 EST ---
fbpanel-4.12-5.fc9 has been submitted as an update for Fedora 9

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

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


[Bug 452150] New: Review Request: swarp - Tool that resamples and co-adds together FITS images

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




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

   Summary: Review Request: swarp - Tool that resamples and co-adds
together FITS images
   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: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://sergiopr.fedorapeople.org/swarp.spec
SRPM URL: http://sergiopr.fedorapeople.org/swarp-2.17.1-1.fc9.src.rpm
Description: SWarp is a program that resamples and co-adds together FITS images 
using any arbitrary astrometric projection defined in the WCS standard.

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

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


[Bug 448292] Review Request: fbpanel - a lightweight X11 desktop panel

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

Summary: Review Request: fbpanel - a lightweight X11 desktop panel


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 12:53 EST ---
fbpanel-4.12-5.fc8 has been submitted as an update for Fedora 8

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

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


[Bug 445224] Review Request: stapitrace - user space instruction trace

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

Summary: Review Request: stapitrace - user space instruction trace


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 13:10 EST ---
(In reply to comment #7)
 source files match upstream:
  ab74f8ed90b8ba9a1d3904892622fc80 (tarball matches tarball found in upstream 
 SRPM)
 specfile is properly named, is cleanly written and uses macros consistently.
 dist tag is present.
 build root is correct
 license field matches the actual license. (though itrace.c has GPL v2.1 
 listed,
 which doesn't exist.  That should probably be fixed upstream)
 license is open source-compatible.
 license text not included upstream.
 latest version is being packaged.
 BuildRequires are proper.
 compiler flags are appropriate.
 %clean is present.
 rpmlint is silent. 
 owns the directories it creates.
 doesn't own any directories it shouldn't.
 no duplicates in %files.
 file permissions are appropriate.
 no scriptlets present.
 code, not content.
 documentation is small, so no -docs subpackage is necessary.
 %docs are not necessary for the proper functioning of the package.
 no headers.
 no pkgconfig files.
 no libtool .la droppings.
 
 Needs fixing:
 
 package meets naming and versioning guidelines.
 package builds in mock:
 http://koji.fedoraproject.org/koji/getfile?taskID=664952name=build.log
 package installs properly. (couldn't check)
 debuginfo package looks complete. (couldn't check)
 final provides and requires are sane (couldn't check)
 if shared libraries are present, make sure ldconfig is run

So, the only issue under the Needs fixing category is the build error you got
in mock.  I'll look into that one.

Also, did you have any response to my questions (in previous comment) about how
I should be referencing the source:  tarball vs. 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, or are watching someone who is.

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


[Bug 448867] Review Request: perl-parent - Establish an ISA relationship with base classes at compile time

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

Summary: Review Request: perl-parent - Establish an ISA relationship with base 
classes at compile time
Alias: perl-parent

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 13:16 EST ---
Imported and building.  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, or are watching someone who is.

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


[Bug 438543] Review Request: Synopsis - Source-code introspection tool.

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

Summary: Review Request: Synopsis - Source-code introspection tool.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED]|
   |ca) |




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 13:22 EST ---
Sorry for being late again. I'm presently being stuck, waiting for my sys admin
to resolve an issue on the machine hosting the Synopsis 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, or are watching someone who is.

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


[Bug 443469] Review Request: pytrainer - A tool to log all your sport excursions

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

Summary: Review Request: pytrainer - A tool to log all your sport excursions


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 13:27 EST ---
Recently changed my paths around a bit; new Spec URL:
https://rpm.silfreed.net:8002/file/beb4427ffcf1/pytrainer/pytrainer.spec

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

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


[Bug 448292] Review Request: fbpanel - a lightweight X11 desktop panel

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

Summary: Review Request: fbpanel - a lightweight X11 desktop panel


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 13:42 EST ---
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, or are watching someone who is.

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


[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader

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

Summary: Review Request: multiget - A multi-thread http/ftp file downloader


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 14:05 EST ---
From the reply of the upstream we can admit that this program is licensed under
GPLv2.

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

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


[Bug 445151] Review Request: merkaartor - openstreetmap editor

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

Summary: Review Request: merkaartor - openstreetmap editor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 14:40 EST ---
Ok, some other points which I've not pointed out before :

* qt4 is a Requires, not a BuildRequires.

* Mock build perfectly without BuildRequires:  xorg-x11-proto-devel
xorg-x11-xtrans-devel, are they usefull ?

* The software doesn't use make install, so you can remove this line : make
INSTALL_ROOT=$RPM_BUILD_ROOT install.

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

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


[Bug 442714] Review Request: sat-solver - Satisfyability Solver library which can be used to compute inter-package dependencies.

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

Summary: Review Request: sat-solver - Satisfyability Solver library which can 
be used to compute inter-package dependencies.


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 14:50 EST ---
SPEC URL: http://rpm.binaryhelix.org/SPECS/libs/sat-solver.spec
SRPM URL: http://rpm.binaryhelix.org/SRPMS/sat-solver-0.9.0-3.fc9.src.rpm

* Thu Jun 19 2008 Lorenzo Villani [EMAIL PROTECTED] - 0.9.0-3
- Using %%cmake properly
- Proper handling of %%{_includedir}/satsolver



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

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


[Bug 447738] Review Request: libzypp - ZYpp is a Linux software management engine

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

Summary: Review Request: libzypp - ZYpp is a Linux software management engine


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 14:53 EST ---
Spec URL: http://rpm.binaryhelix.org/SPECS/libs/libzypp.spec
SRPM URL: http://rpm.binaryhelix.org/SRPMS/libzypp-5.0.0.0-3.fc9.src.rpm

* Thu Jun 19 2008 Lorenzo Villani [EMAIL PROTECTED] - 5.0.0.0-3
- Using %%cmake properly


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

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


[Bug 447740] Review Request: zypper - easy to use command line package manager

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

Summary: Review Request: zypper - easy to use command line package manager


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 14:54 EST ---
Spec URL: http://rpm.binaryhelix.org/SPECS/system/zypper.spec
SRPM URL: http://rpm.binaryhelix.org/SRPMS/zypper-0.12.0-2.fc9.src.rpm

* Thu Jun 19 2008 Lorenzo Villani [EMAIL PROTECTED] - 0.12.0-2
- Using %%cmake to build
- Using %%find_lang



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

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


[Bug 450408] Review Request: trickle - Portable lightweight userspace bandwidth shaper

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

Summary: Review Request: trickle - Portable lightweight userspace bandwidth 
shaper


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 15:15 EST ---
trickle-1.07-3.fc9 has been submitted as an update for Fedora 9

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

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


[Bug 450408] Review Request: trickle - Portable lightweight userspace bandwidth shaper

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

Summary: Review Request: trickle - Portable lightweight userspace bandwidth 
shaper


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 15:17 EST ---
trickle-1.07-3.fc8 has been submitted as an update for Fedora 8

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

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


[Bug 226309] Merge Review: postgresql-jdbc

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

Summary: Merge Review: postgresql-jdbc


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 449339] Review Request: wine-doors - One-click Windows application installer / Wine frontend

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

Summary: Review Request: wine-doors - One-click Windows application installer / 
Wine frontend


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX
   Flag||fedora-review-




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 15:56 EST ---
As Nicolas said, the package :

- have to download scripts to works, it's quite dangerous and a sign that the
method is not mature (well, it's a choice)
- conduce the user to keep Windows' software and don't search open sourced
alternatives which could be considered as against the Fedora's philosophy 

so, I decided to drop the review and send the package upstream for people who
really want 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, or are watching someone who is.

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


[Bug 452172] New: Review Request: mysqltuner - MySQL high performance tuning script

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




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

   Summary: Review Request: mysqltuner - MySQL high performance
tuning script
   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: fedora-package-review@redhat.com,[EMAIL PROTECTED]


http://scop.fedorapeople.org/packages/mysqltuner.spec
http://scop.fedorapeople.org/packages/mysqltuner-0.9.1-1.src.rpm

MySQLTuner is a MySQL high performance tuning script written in perl
that will provide you with a snapshot of a MySQL server's health.
Based on the statistics gathered, specific recommendations will be
provided that will increase a MySQL server's efficiency and
performance.  The script gives you automated MySQL tuning that is on
the level of what you would receive from a MySQL DBA.

Note: intentionally not using %{?dist} as there's nothing really to build in 
the package that would change between rebuilds on different distros.

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

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


[Bug 452087] Review Request: collectl - utility to collect performace data

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

Summary: Review Request: collectl -  utility to collect performace data


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


[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, or are watching someone who is.

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


[Bug 449962] Review Request: tex-fonts-hebrew - Support using (Culmus) Hebrew fonts in TeXLive

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

Summary: Review Request: tex-fonts-hebrew -  Support using (Culmus) Hebrew 
fonts in TeXLive


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 16:35 EST ---
tex-fonts-hebrew-0.1-9.fc9 has been submitted as an update for Fedora 9

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

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


[Bug 450470] RDMA kernel stack initializing package

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

Summary: RDMA kernel stack initializing package


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 16:44 EST ---
I built, installed, and tested this package on an F8 x86_64 system and it 
seems to work correctly.  I do *not* understand udev well enough to say  
whether the rules are correct but they do look reasonable -- esp. compared
the the OFED docs.

Here is a quick review:

GOOD:
+ rpmlint reports a few ignorable warnings and an error:
rdma.noarch: W: no-documentation
rdma.noarch: W: non-conffile-in-etc /etc/rdma/fixup-mtrr.awk
rdma.noarch: W: non-conffile-in-etc /etc/udev/rules.d/90-rdma.rules
rdma.noarch: W: no-url-tag
rdma.noarch: E: malformed-line-in-lsb-comment-block #
+ naming looks OK
+ license is OK and correctly not included
+ spec is legible, macros look sane
+ compiles and installs on F8 x86_64
+ dir ownership looks OK

NEEDSWORK:
+ please remove the blank line which fixes the rpmlint error:

=== trivial patch ===
--- rdma.ORIG   2008-06-19 16:29:58.0 -0400
+++ rdma2008-06-19 16:30:18.0 -0400
@@ -12,7 +12,6 @@
 # Required-Stop: $network $srpd $opensm
 # Short-Description: Loads and unloads the InfiniBand and iWARP kernel modules
 # Description: Loads and unloads the InfiniBand and iWARP kernel modules
-#
 ### END INIT INFO
 
 CONFIG=/etc/rdma/rdma.conf
=== trivial patch ===



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

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


[Bug 434727] Review Request: libpst - utilities to convert Outlook .pst files to other formats

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

Summary: Review Request: libpst - utilities to convert Outlook .pst files to 
other formats


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 17:03 EST ---
./common/cvs-import.sh /tmp/libpst-0.6.14-1.src.rpm
cd devel; make build

That worked, but trying to import into the F-10 branch does not:

./common/cvs-import.sh -b F-10 /tmp/libpst-0.6.14-1.src.rpm
Checking out module: 'libpst'
ERROR: libpst/F-10 does not exist!

Is that supposed to work, or is there some other requirement?

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

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


[Bug 434727] Review Request: libpst - utilities to convert Outlook .pst files to other formats

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

Summary: Review Request: libpst - utilities to convert Outlook .pst files to 
other formats


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 17:13 EST ---
There is no F-10 branch because there is no F-10 release. devel is a magic
branch because it is always there, but right now, it is what will become F-10.

F-8 and F-9 are the other active branches right 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, or are watching someone who is.

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


[Bug 450476] InfiniBand Connection Management library

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

Summary: InfiniBand Connection Management library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 17:22 EST ---
Here is a quick review:

GOOD:
+ source matches upstream (SHA1SUM):
bd18cd1c105275feb25461f659a3ad94e7c5db8c  libibcm-1.0.2.tar.gz
bd18cd1c105275feb25461f659a3ad94e7c5db8c  libibcm-1.0.2.tar.gz.UP
+ license is correct and correctly included
+ builds in mock for F8 x86_64
+ rpmlint output is just some ignore-able warnings:
libibcm.x86_64: E: zero-length /usr/share/doc/libibcm-1.0.2/ChangeLog
libibcm-devel.x86_64: W: no-documentation
libibcm-static.x86_64: W: no-documentation
+ naming OK
+ spec legible and macros look sane
+ shared libs (ldconfig) looks good
+ permissions OK
+ dir ownership OK
+ static libs are correctly in *-static


I don't see any blockers here so its 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, or are watching someone who is.

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


[Bug 450476] InfiniBand Connection Management library

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

Summary: InfiniBand Connection Management library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 17:25 EST ---
Ooops, I overlooked the empty ChangeLog file!  Please just remove it from 
the 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, or are watching someone who is.

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


[Bug 187243] Review Request: lazarus : IDE and RAD tool for the free pascal compiler (fpc)

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

Summary: Review Request: lazarus : IDE and RAD tool for the free pascal 
compiler (fpc)


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 18:46 EST ---
Thanks for all the help, especially to Jason Tibbitts, offcourse.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 187243] Review Request: lazarus : IDE and RAD tool for the free pascal compiler (fpc)

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

Summary: Review Request: lazarus : IDE and RAD tool for the free pascal 
compiler (fpc)


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 18:50 EST ---
lazarus-0.9.24-4.fc9 has been submitted as an update for Fedora 9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 452172] Review Request: mysqltuner - MySQL high performance tuning script

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

Summary: Review Request: mysqltuner - MySQL high performance tuning script


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   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, or are watching someone who is.

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


[Bug 450409] Review Request: rcssserver3d - Robocup Soccer Simulation Server 3D

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

Summary: Review Request: rcssserver3d - Robocup Soccer Simulation Server 3D


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|182235  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 19:28 EST ---
OK! I've updated my package to the latest CVS code. It doesn't have any NVidia
licensed files anymore, so the legal problem is resolved now! So, FE-Legal
should not be needed anymore (?!).

SRPM:
http://www.assembla.com/spaces/hedayat/documents/dOrnuopLqr3BySab7jnrAJ/download/rcssserver3d-0.6-0.1.20080620cvs.fc9.src.rpm

SPEC:
http://www.assembla.com/spaces/hedayat/documents/dC7Shyoh0r3z6aab7jnrAJ/download/rcssserver3d.spec

(
OR
SRPM:
http://assembla.com/file/hedayat/1_rcssserver3d-0.6-0.1.20080620cvs.fc9.src.rpm
SPEC: http://assembla.com/file/hedayat/2_rcssserver3d.spec
)

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

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


[Bug 430429] Review Request: python-storm - An object-relational mapper (ORM) for Python

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

Summary: Review Request: python-storm - An object-relational mapper (ORM) for 
Python


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 19:36 EST ---
! needs work.
OK no problemo
?? can't say/don't understand.

[OK] rpmlint .src.rpm = quiet

$ rpmbuild -ba:
+ /usr/lib/rpm/find-debuginfo.sh /home/davidt/src/redhat/BUILD/storm-0.12
find: debug: No such file or directory
from what others have indicated, this is normal for no-arch python.

[OK] rpmlint /home/davidt/src/redhat/RPMS/noarch/python-storm*.rpm
python-storm-mysql.noarch: W: no-documentation
python-storm-postgresql.noarch: W: no-documentation
python-storm-sqlite.noarch: W: no-documentation

[OK] included source matches upstream: md5sum storm*bz*
python-storm-0.12-1.fc9.src/storm-0.12.tar.bz2 
976a332dadd214612c359df63360fc51  storm-0.12.tar.bz2.from_www
976a332dadd214612c359df63360fc51  python-storm-0.12-1.fc9.src/storm-0.12.tar.bz2
[OK] package name proceeded with python since it;s a python library.
[OK] spec named %name.spec
[OK] source tarball contains no prebuilt binaries/libraries.
[OK] files placed into FHS locations {x4 rpms}
[OK] changelog in standard format
[OK] correctly omits Packager, vendor, copyright, prereq 
, includes license tag.
[OK] summary 80 chars, no ending period
[OK] source0 is correct:
http://launchpad.net/storm/trunk/0.12/+download/storm-0.12.tar.bz2
[OK] buildroot set at second most proferred location.
[OK] %install correctly erases buildroot before build
[OK] mock build succeeds.
[OK] rpmdiff between default build and mock build shows only time
{T} differences for folders and the pre-compiled .py[co] files
[OK] description is column limited to 80 chars, no manual/doc info.
[OK] charset is ascii
[OK] docs {license, readme, todo} included. readme points to web for docs.
[OK] debuginfo not expected in noarch application.
[OK] no static libraries, rpath, self copies of already packaged libaries
etc, since pure python project.
[OK] no config, initscripts, desktop files since it's a db devel library.
[OK] variable style is used consistently
[OK] not multilingual
[OK] timestamps are kept
[OK] make is python based, so smp_flags not required.
[OK] no scriptlets, no conditional dependencies
[OK] is library code not content
[OK] provides backend for each of the backend subpackages seems to make sense.
[OK] dirs/files owned by main package, except the individual backend 
files {whose directory is created by the main package}.
[OK] not a web app, shoudln't conflict with other packages.
[OK] python sitelib is correctly included at top of spec.
[OK] eggs are build
[OK] no files in %{_bindir} and %{_sbindir}.
[OK] %install setup.py install -O1 --skip-build. as requested for python 
packages.
=
[? ] license indication matches upstream web site
Storm is licensed under the [WWW] LGPL 2.1. Contributions must 
have copyright assigned to Canonical.
- copyright assigned seems to infer additional requirements ?
- ... Lesser General Public License as
# published by the Free Software Foundation; either version 2.1 of
# the License, or (at your option) any later version
so LGPLv2+ seems correct.
- eg: cache.py has no licence header - should it be requested ?
- tz.py: PSF License not sure what that is ?
not in http://fedoraproject.org/wiki/Licensing

[??] %files: perhaps should own only it's python-storm dir and below,
according to:
http://fedoraproject.org/wiki/Packaging/Python
%{python_sitelib}/modulename/*.py
 
[??] python eggs: do we need to do anything with them ?

[??] are/should the unit tests be run as part of the package build ?
upstream suggests these as important, but is that something fedora packages
usually do ?

[??] %files: %exclude %{python_sitelib}/storm/cextensions.c  
is this the accepted way to do this ?

[ !] the main package includes the same files that are in the 3 backend sub 
packages

[ !] description missed a 't' in 'erm maintainability'.

[??] no time yet to test built package installation functionality aka:
- SHOULD: The reviewer should test that the package functions as described.
A package should not segfault instead of running, for example.

Michel: pretty close, just a fwe minor tweaks, I think.

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

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


[Bug 450470] RDMA kernel stack initializing package

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

Summary: RDMA kernel stack initializing package


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 21:18 EST ---
New rpm uploaded over the top of the old rpm (meaning you might need to flush
your web cache/proxy to get the new rpm) that solves the rpmlint error (empty
line removed from rdma.init).

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

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


[Bug 450476] InfiniBand Connection Management library

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

Summary: InfiniBand Connection Management library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 21:20 EST ---
ChangeLog file removed from %doc list.  New packages uploaded over top of the
old packages, so you might need to refresh your web/proxy cache to get the new
files from the site.

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

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


[Bug 452172] Review Request: mysqltuner - MySQL high performance tuning script

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

Summary: Review Request: mysqltuner - MySQL high performance tuning script


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 21:44 EST ---
This package is about as simple as you can get; a single file.

I guess you know that's not what the dist tag is about, but I assume you can
keep the versioning straight.

Only one observation:

The applied patch seems to be a good idea, except that if you actually pass the
--skipversion option you are warned about it.  Perhaps it would be better to
accept and ignore it.  I don't think this is a blocker, though, since things
still run.

* source files match upstream:
   6d13fdfda710c5d1ef777a31177f31fb691af11a4f953e9fed07a6cdceff7a32  
   mysqltuner.pl.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text not included upstream.
* latest version is being packaged.
* BuildRequires are proper (none).
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* rpmlint has acceptable complaints.
* final provides and requires are sane:
   mysqltuner = 0.9.1-1
  =
   /usr/bin/perl
   mysql
   perl(Getopt::Long)
   perl(diagnostics)
   perl(strict)
   perl(warnings)
   which

* %check is not present; no test suite upstream.  Seems to run OK, but I didn't 
   let it mess with my database server.

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, or are watching someone who is.

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


[Bug 452150] Review Request: swarp - Tool that resamples and co-adds together FITS images

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

Summary: Review Request: swarp - Tool that resamples and co-adds together FITS 
images


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


[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, or are watching someone who is.

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


[Bug 452150] Review Request: swarp - Tool that resamples and co-adds together FITS images

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

Summary: Review Request: swarp - Tool that resamples and co-adds together FITS 
images


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED




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

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


[Bug 452150] Review Request: swarp - Tool that resamples and co-adds together FITS images

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

Summary: Review Request: swarp - Tool that resamples and co-adds together FITS 
images


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 22:03 EST ---
Two observations:

The pdf file is 1.5MB and three times the size of the rest of the package.  Did
you consider putting it in a separate package?   I don't think it's a
requirement but you know best how this package will be used.

%{__mkdir_p} %{buildroot}%{_datadir}/%{name} at the end of %install seems to be
superfluous, as that directory is created (and swarp.xsl is installed there) by
the install process.  It seems to be harmless, though.

I don't think either of these are blockers, though.

* source files match upstream:
   34e815d44f3b1c3c7bfdeb01bbdcf32d1eac239527a0937c3ed2b6f7be34bcee  
   swarp-2.17.1.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
 build root is OK.
 license field matches the actual license.
 license is open source-compatible.
 license text not included upstream.
 license text included in package.
* latest version is being packaged.
* BuildRequires are proper (none).
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
   swarp = 2.17.1-1.fc10
  =
   (no non-glibc dependencies)
* %check is not present; no test suite upstream.  There is a test directory 
  containing one fits file, but it's not immediately clear what should be done 
  with it.  I ran the executable and gave it the test file and it did... 
  something, without crashing, and didn't complain about it.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
? documentation is 3x the rest of the package.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

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, or are watching someone who is.

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


[Bug 452195] New: Review Request: Presto - A tilemap engine using the Allegro game programming library

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




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

   Summary: Review Request: Presto - A tilemap engine using the
Allegro game programming library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: 
http://www.hypersonicsoft.org/projects/downloads/Presto/misc/presto.spec
SRPM URL: 
http://www.hypersonicsoft.org/projects/downloads/Presto/misc/presto-0.1.1-1.fc9.src.rpm
Description:
Presto is a general-use tilemap engine coded in C that uses Allegro for 
graphics rendering, and therefore is intended for use in games using Allegro.  
It can handle rectangular tiles of any height and width (and different height 
from width), loading tilemaps from files, tile blending, and the capability to 
change most of these elements on the fly.

This is my first package, and I need 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, or are watching someone who is.

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


[Bug 452195] Review Request: Presto - A tilemap engine using the Allegro game programming library

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

Summary: Review Request: Presto - A tilemap engine using the Allegro game 
programming library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 452195] Review Request: Presto - A tilemap engine using the Allegro game programming library

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

Summary: Review Request: Presto - A tilemap engine using the Allegro game 
programming library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 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, or are watching someone who is.

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


[Bug 452195] Review Request: Presto - A tilemap engine using the Allegro game programming library

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

Summary: Review Request: Presto - A tilemap engine using the Allegro game 
programming library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 23:21 EST ---
I forgot to mention that I'm the upstream developer for this package.  I have a
project page on my website for the library, found here:
http://www.hypersonicsoft.org/projects/showproject.php?id=29.  I'm an
experienced C programmer, and I definitely appreciate anyone who is willing to
help me with this first 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, or are watching someone who is.

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


[Bug 452195] Review Request: Presto - A tilemap engine using the Allegro game programming library

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

Summary: Review Request: Presto - A tilemap engine using the Allegro game 
programming library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-19 23:59 EST ---
- Descriptions should be wrapped at 79 characters
- Examples, README, and COPYING should be in %doc
- Changelog entries should be separated by a newline
- Fails to build in mock on F8/i386 and F9/i386
- -devel is missing a Requires of allegro-devel, and should have no 
BuildRequires


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

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


[Bug 427718] Review Request: Megazeux - Text-character driven game creation system

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

Summary: Review Request: Megazeux - Text-character driven game creation system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |201449
  nThis||
 Status|NEEDINFO|CLOSED
 Resolution||NOTABUG
   Flag|needinfo?([EMAIL PROTECTED]|
   |m)  |




--- Additional Comments From [EMAIL PROTECTED]  2008-06-20 00:01 EST ---
Why on earth is this ticket private? 

In any case, it's been way too long without any response.  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, or are watching someone who is.

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


[Bug 450476] InfiniBand Connection Management library

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

Summary: InfiniBand Connection Management library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-20 01:11 EST ---
Thank you -- its 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, or are watching someone who is.

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