[Bug 426827] Review Request: perl-DateTime-Precise - Perform common time and date operations with additional GPS operations

2008-01-31 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-DateTime-Precise - Perform common time and date 
operations with additional GPS operations


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


[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, 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 426827] Review Request: perl-DateTime-Precise - Perform common time and date operations with additional GPS operations

2008-01-31 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-DateTime-Precise - Perform common time and date 
operations with additional GPS operations


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 03:34 EST ---
New Package CVS Request
===
Package Name: perl-DateTime-Precise
Short Description: Perform common time and date operations with additional GPS
operations
Owners: xavierb
Branches: F-8 EL-4 EL-5
InitialCC: 
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 357661] Review Request: sunbird - Mozilla Sunbird Calendar

2008-01-31 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: sunbird - Mozilla Sunbird Calendar
Alias: sunbird-review

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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 03:55 EST ---
Felix: The langpack issue with sunbird is an upstream issue. I suspect it might
be  the case with lightning too. Could you please check?

I'll search for a fix or file an upstream bug once my vacation is over
(2008-02-06). I'd do it now if I hadn't trouble typing due to a kind of a stupid
injury of a hand.

-- 
Configure 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 430859] Review Request: netmonitor - The free linux network bandwidth monitor

2008-01-31 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: netmonitor - The free linux network bandwidth monitor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 04:00 EST ---
Fixed.
Spec URL: http://kurzawa.nonlogic.org/rpm/netmonitor/netmonitor.spec
SRPM URL: 
http://kurzawa.nonlogic.org/rpm/netmonitor/netmonitor-0.5-5.fc8.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, 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 431032] New: Review Request: ghost-diagrams - A program that generates patterns from tiles

2008-01-31 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=431032

   Summary: Review Request: ghost-diagrams - A program that
generates patterns from tiles
   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://ivazquez.fedorapeople.org/packages/ghost-diagrams/ghost-diagrams.spec
SRPM URL: 
http://ivazquez.fedorapeople.org/packages/ghost-diagrams/ghost-diagrams-0.8-1.src.rpm
Description: Ghost Diagrams is a program that takes sets of tiles and tries to 
find patterns into which they may be formed. The patterns it finds when given 
randomly chosen tiles are often surprising.

-- 
Configure 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 430073] Review Request: python-igraph - Python bindings for igraph

2008-01-31 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-igraph - Python bindings for igraph


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 04:56 EST ---
change Source URL
http://pypi.python.org/packages/source/i/igraph/igraph-%{version}.tar.gz


-- 
Configure 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 430859] Review Request: netmonitor - The free linux network bandwidth monitor

2008-01-31 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: netmonitor - The free linux network bandwidth monitor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 05:07 EST ---
Review:
+ package builds in mock (rawhide i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream.
fefca04be4b7272955fe5a5c93232842  netmonitor-0.5.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.
+ Compiler flags used correctly.
+ 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.
+ 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, 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 431032] Review Request: ghost-diagrams - A program that generates patterns from tiles

2008-01-31 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: ghost-diagrams - A program that generates patterns 
from tiles


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


[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 430859] Review Request: netmonitor - The free linux network bandwidth monitor

2008-01-31 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: netmonitor - The free linux network bandwidth monitor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 05:26 EST ---
New Package CVS Request
===
Package Name: netmonitor
Short Description: The free linux network bandwidth monitor
Owners: kurzawa
Branches: F-7 F-8
InitialCC:
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 431032] Review Request: ghost-diagrams - A program that generates patterns from tiles

2008-01-31 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: ghost-diagrams - A program that generates patterns 
from tiles


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 05:53 EST ---
--delete-original not needed in desktop-file-install
Do you got any icon to install for this desktop file?

-- 
Configure 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 430859] Review Request: netmonitor - The free linux network bandwidth monitor

2008-01-31 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: netmonitor - The free linux network bandwidth monitor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 05:57 EST ---
More pedantic things:

if you change make install line to

make install DESTDIR=$RPM_BUILD_ROOT INSTALL=%{__install} -p

the timestamps will be preserved (which is good thing).



-- 
Configure 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 226422] Merge Review: slrn

2008-01-31 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: slrn


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 06:38 EST ---
Source URL fixed.

-- 
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 430978] Review Request: libsoup22 - Backward compatibility package for libsoup 2.2 API

2008-01-31 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: libsoup22 - Backward compatibility package for libsoup 
2.2 API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||430756
  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 430859] Review Request: netmonitor - The free linux network bandwidth monitor

2008-01-31 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: netmonitor - The free linux network bandwidth monitor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 07:12 EST ---
Fixed.
Spec URL: http://kurzawa.nonlogic.org/rpm/netmonitor/netmonitor.spec
SRPM URL: 
http://kurzawa.nonlogic.org/rpm/netmonitor/netmonitor-0.5-6.fc8.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, 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 226422] Merge Review: slrn

2008-01-31 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: slrn


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


[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 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 431044] New: Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

2008-01-31 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=431044

   Summary: Review Request: perl-ccom - Perl module for context-
sensitive phonetic string replacement
   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 URL: http://labs.linuxnetz.de/bugzilla/perl-ccom.spec
SRPM URL: http://labs.linuxnetz.de/bugzilla/perl-ccom-1.4.1-1.src.rpm
A perl module for context-sensitive phonetic string replacement to modify
strings according to predefined replacement rules in such a way that words
with the same spelling (e.g. tail and tale) are converted to the same
string. This can, for example, be used to implement error-tolerant search
routines in address databases. It contains phonetic rules for German only,
but the software has been prepared for multi-language support.

-- 
Configure 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 226422] Merge Review: slrn

2008-01-31 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: slrn


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 07:31 EST ---
thanks for CVS update.

APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, 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 226147] Merge Review: mktemp

2008-01-31 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: mktemp


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 08:21 EST ---
i have overlooked it. It's fine to obsolte mktemp now. Thanks

-- 
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 431047] Review Request: latencytop - System latency monitor

2008-01-31 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: latencytop - System latency monitor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 08:31 EST ---
The package builds successfully in mock on my x86_64 system.

$ rpmlint /var/lib/mock/fedora-development-x86_64/result/
latencytop.x86_64: W: no-documentation

This is expected. Upstream tarball contains no documentation.

-- 
Configure 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 431047] New: Review Request: latencytop - System latency monitor

2008-01-31 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=431047

   Summary: Review Request: latencytop - System latency monitor
   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://michich.fedorapeople.org/latencytop/latencytop.spec
SRPM URL: 
http://michich.fedorapeople.org/latencytop/latencytop-0.3-1.fc9.src.rpm
Description:
LatencyTOP is a tool for software developers (both kernel and userspace), aimed 
at identifying where in the system latency is happening, and what kind of 
operation/action is causing the latency to happen so that the code can be 
changed to avoid the worst latency hiccups.

-- 
Configure 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 431047] Review Request: latencytop - System latency monitor

2008-01-31 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: latencytop - System latency monitor


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


[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 314871] Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In

2008-01-31 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: gimp-lqr-plugin - GIMP LiquidRescale Plug-In


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 09:44 EST ---
Carlo Baldassi wrote:
 Ok, I have changed the licensing of the plugin from GPLv2 strict to
 GPLv2 or later, which I hope solves the issue.

I've been studying the licenses and as far as I can tell the two packages are
now fully compatible. I'll post SRPMS later today.

Regards,
Alex

-- 
Configure 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 225793] Merge Review: gftp

2008-01-31 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: gftp


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 09:41 EST ---
Looks good.

APPROVED

All issues fixed, Closing out this Merge Review

-- 
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 431047] Review Request: latencytop - System latency monitor

2008-01-31 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: latencytop - System latency monitor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 10:02 EST ---
CONFIG_LATENCYTOP is a very new feature merged into Linux after 2.6.24 was
released. Currently one has to run a self-built kernel from Linus's git tree to
use it.

I'll fix issue #1 and contact upstream about #2.

-- 
Configure 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 429202] Review Request: liblqr-1 - LiquidRescale library

2008-01-31 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: liblqr-1 - LiquidRescale library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 09:59 EST ---
Once the license issue on bug 314871 is resolved, I will try to
start to check this 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 431047] Review Request: latencytop - System latency monitor

2008-01-31 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: latencytop - System latency monitor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 09:43 EST ---
The issue I see with this package on F8 machine when tried to run latencytop =
Please enable the CONFIG_LATENCYTOP configuration in your kernel.
Exiting...

Is CONFIG_LATENCYTOP is enabled in newer builds of kernel. If not then how can
one test functionality of this package?

About Packaging issues =
1)Better recreate latencytop-standard-cflags.patch that will remove unnecessary
space caught in last few lines of patch.
  
2) Try to ask upstream to include some README and COPYING file. If this will
take some time then ask them to include it in next release. 
Note :- This is not a blocker for review


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 428894] Review Request: rudesocket - Library (C++ API) for creating client sockets

2008-01-31 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: rudesocket - Library (C++ API) for creating client 
sockets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 10:05 EST ---
Thanks for the review!

New Package CVS Request
===
Package Name: rudesocket
Short Description: Library (C++ API) for creating client sockets
Owners: homeless
Branches: F-7 F-8
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 428410] Review Request: LADR - Library for Automated Deduction Research

2008-01-31 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: LADR - Library for Automated Deduction Research


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 10:10 EST ---
Sent a patch to upstream to add GPL statements to Python scripts and fix the
readme manual link. They intend to include this patch in the next release
(hopefully February).

Not sure I understand the requirement, eg. there can be no %_bindir/attack
because something at some point might need to be called %_bindir/attack? This is
however a specialized package so I could certainly ask upstream if they can be
renamed, or easier probably to just rename them ladr_* for the purposes of
putting this on Fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 429084] Review Request: vamp-plugin-sdk - An API for audio analysis and feature extraction plugins

2008-01-31 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: vamp-plugin-sdk - An API for audio analysis and 
feature extraction plugins


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 10:03 EST ---
vamp-plugin-sdk-1.1b-3.fc7 has been submitted as an update for Fedora 7

-- 
Configure 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 431013] Review Request: perl-CSS-Squish - Compact many CSS files into one big file

2008-01-31 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-CSS-Squish -  Compact many CSS files into one big 
file


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 09:56 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
d992129bf96368a504fe9c67b6d3e622  CSS-Squish-0.07.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 output is
t/00-use..ok
t/01-basicok
t/02-edge-cases...ok
t/03-skip-httpok
t/04-recursionok
t/05-basic-loop-preventionok
t/06-server-relative-urls.skipped
all skipped: Functionality non-existant.  (and tests incomplete.)
t/07-basic-extra-rootsok
t/99-pod-coverage.ok
t/99-pod..ok
All tests successful, 1 test skipped.
Files=10, Tests=15,  1 wallclock secs ( 0.58 cusr +  0.12 csys =  0.70 CPU)
+ Package  perl-CSS-Squish-0.07-1.fc9 -
  Provides: perl(CSS::Squish) = 0.07
  Requires: perl(:MODULE_COMPAT_5.8.8) perl(File::Spec) perl(Scalar::Util)
perl(URI) perl(URI::file) 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, 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 429084] Review Request: vamp-plugin-sdk - An API for audio analysis and feature extraction plugins

2008-01-31 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: vamp-plugin-sdk - An API for audio analysis and 
feature extraction plugins


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 10:02 EST ---
vamp-plugin-sdk-1.1b-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 210007] Review Request: libtune - standard API to access the kernel tunables

2008-01-31 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: libtune - standard API to access the kernel tunables


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 10:51 EST ---
You're right: I've tried that with the tunables-base-data package and got the
same result.
I'm not an rpm expert, but I'm wondering whether this wouldn't be because
version .5 contains exactly the same files as version .4 + a brand new file. So
rpm doens't doesn't see that as a conflict?

I'll try to test something on my side: make version .5 with a file that has the
same name as in .4, but with a different content.

-- 
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 429749] Review Request: libvncserver - Library to make writing a vnc server easy

2008-01-31 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: libvncserver - Library to make writing a vnc server 
easy


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 11:29 EST ---
tibbs, ping, anything else? 

(I'd like to incorporate this BR into the kdenetwork-4.0.1 build I'm comitting
later today, if possible).

-- 
Configure 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 428410] Review Request: LADR - Library for Automated Deduction Research

2008-01-31 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: LADR - Library for Automated Deduction Research


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 11:43 EST ---
Well, the names like renamer or attack are too generic and
such naming should be avoided for binaries under %_bindir.

Renaming to ladr_XXX or so is one of the possible solutions.

-- 
Configure 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 409511] Review Request: libmlx4 - Mellanox ConnectX InfiniBand HCA Userspace Driver

2008-01-31 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: libmlx4 - Mellanox ConnectX InfiniBand HCA Userspace 
Driver


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 11:46 EST ---
New Package CVS Request
===
Package Name: libmlx4
Short Description: device-specific userspace driver for Mellanox ConnectX HCAs
for use with the libibverbs library
Owners: [EMAIL PROTECTED]
Branches: F-7 F-8
InitialCC: 
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 429749] Review Request: libvncserver - Library to make writing a vnc server easy

2008-01-31 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: libvncserver - Library to make writing a vnc server 
easy


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 11:36 EST ---
I've just been a bit time-limited over the past few days.  I'll try to gather
some time later today to finish this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 240543] Review Request: dsmidiwifi - remote controll midi with your Nintendo DS

2008-01-31 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: dsmidiwifi - remote controll midi with your Nintendo DS
Alias: dsmidiwifi

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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 11:59 EST ---
Oh I forgot this: Thank you for the review Jason.

-- 
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 240543] Review Request: dsmidiwifi - remote controll midi with your Nintendo DS

2008-01-31 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: dsmidiwifi - remote controll midi with your Nintendo DS
Alias: dsmidiwifi

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 11:58 EST ---
New Package CVS Request
===
Package Name: dsmidiwifi
Short Description: DS music interface
Owners: till
Branches: 
InitialCC: 
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 187243] Review Request: lazarus : IDE and RAD tool for the free pascal compiler (fpc)

2008-01-31 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

 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 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 428435] Review Request: shezhu - Shezhu Resource Sharing System

2008-01-31 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: shezhu - Shezhu Resource Sharing System


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 12:24 EST ---
Thanks. Got a new version that fixes the users and groups (using the
boilerplate), maps paths properly to %{_libdir}, %{_var} etc., also fixes the
ownership of files in /usr/sbin and capital letters. Will make SRPM and SPEC
available here in the next few days - will post comment with new link.

-- 
Configure 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 350341] Review Request: postfix-logwatch - A postfix log analyzer for logwatch

2008-01-31 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: postfix-logwatch - A postfix log analyzer for logwatch
Alias: postfix-logwatch

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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 12:40 EST ---
Till: Would you like me to take another look at the last posted package here, or
would you like to spin up an updated one? 

Let me know and we can get this review moving... 

-- 
Configure 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 240543] Review Request: dsmidiwifi - remote controll midi with your Nintendo DS

2008-01-31 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: dsmidiwifi - remote controll midi with your Nintendo DS
Alias: dsmidiwifi

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 12:45 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 426827] Review Request: perl-DateTime-Precise - Perform common time and date operations with additional GPS operations

2008-01-31 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-DateTime-Precise - Perform common time and date 
operations with additional GPS operations


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 12:53 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 429084] Review Request: vamp-plugin-sdk - An API for audio analysis and feature extraction plugins

2008-01-31 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: vamp-plugin-sdk - An API for audio analysis and 
feature extraction plugins


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 12:57 EST ---
Looks like cvs is done here... clearing flag. 

-- 
Configure 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 409511] Review Request: libmlx4 - Mellanox ConnectX InfiniBand HCA Userspace Driver

2008-01-31 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: libmlx4 - Mellanox ConnectX InfiniBand HCA Userspace 
Driver


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 12:50 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 316141] Review Request: gridengine - Grid Engine - Distributed Computing Management software

2008-01-31 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: gridengine - Grid Engine - Distributed Computing 
Management software


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 13:00 EST ---
I will recheck whole code later, however I may have did some misunderstanding.

(In reply to comment #16)
 Well, it seems with ltree really being LGPLv2+, comment #13 shouldn't apply, 
It seems so (from my quick glance). Anyway I will recheck the whole code.

-- 
Configure 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 430509] Review Request: synce-kpm - SynCE KDE PDA Manager

2008-01-31 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-kpm - SynCE KDE PDA Manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 13:08 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 428894] Review Request: rudesocket - Library (C++ API) for creating client sockets

2008-01-31 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: rudesocket - Library (C++ API) for creating client 
sockets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 12:56 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 430859] Review Request: netmonitor - The free linux network bandwidth monitor

2008-01-31 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: netmonitor - The free linux network bandwidth monitor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 13:11 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 307791] Review Request: twitux - a Twitter client for the Gnome desktop

2008-01-31 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: twitux - a Twitter client for the Gnome desktop


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 13:14 EST ---
@Michel: no exceptions, please. Better do something to fix it upstream (move
schemas away from /etc to /usr/share/gconf/schemas like it's done in kde).

-- 
Configure 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 273701] Review Request: gnome-main-menu - Gnome Main Menu

2008-01-31 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: gnome-main-menu - Gnome Main Menu


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 13:58 EST ---
Any updates on this, or updated packages for testing on 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 431098] New: Review Request: gambas2 - IDE based on a basic interpreter with object extensions

2008-01-31 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=431098

   Summary: Review Request: gambas2 - IDE based on a basic
interpreter with object extensions
   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://www.auroralinux.org/people/spot/review/new/gambas2.spec
SRPM URL: 
http://www.auroralinux.org/people/spot/review/new/gambas2-2.0.0-2.20080128svn1043.fc9.src.rpm
Description: 
Gambas2 is a free development environment based on a Basic interpreter
with object extensions, like Visual Basic (but it is NOT a clone !).
With Gambas2, you can quickly design your program GUI, access MySQL or
PostgreSQL databases, pilot KDE applications with DCOP, translate your
program into many languages, create network applications easily, and so
on...

A couple of notes from the packager:

- The subpackaging is done a little... uniquely here, because of the nature of 
Gambas2. The IDE actually lets you generate RPMs for the code that you write, 
and it uses these subpackages for dependencies.

- The examples/ directory needs to be world-writeable, or the IDE examples 
functionality doesn't work at all. It is purposefully owned by nobody to 
minimize any possible security concerns.

- Network based components don't quite work right on x86_64 yet, but given that 
the previous version of Gambas didn't work at all on x86_64, this is a big step 
in the right direction. Upstream hopes to have this resolved by 2.1.

- Doesn't work on PPC, or any other big-endian arch.

- Only intended for F9 or higher targets (aka, this won't build right on F8 or 
older).

-- 
Configure 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 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-31 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: R-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 14:31 EST ---
Only items I see:

* Your changelog says you're at -4, but you forgot to increment release to 4.
Just make sure that your %changelog matches. :)

Good:

- rpmlint checks return:
R-Matrix.x86_64: W: incoherent-version-in-changelog 0.999375-4 0.999375-1.fc9
R-Matrix.x86_64: W: one-line-command-in-%post 
/usr/lib/rpm/R-make-search-index.sh
R-Matrix.x86_64: W: one-line-command-in-%postun 
/usr/lib/rpm/R-make-search-index.sh
R-Matrix-devel.x86_64: W: no-documentation
No blockers there, just be careful with matching n-v-r and changelog.

- package meets naming guidelines
- package meets packaging guidelines
- license (GPLv2+) OK, text in %doc, matches source (explanation in spec)
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 
- devel package ok
- no .la files
- devel requires base package n-v-r 

APPROVED (just fix the release before committing).

Get your account here, and follow the rest of the steps:
http://fedoraproject.org/wiki/PackageMaintainers/Join#head-a601c13b0950a89568deafa65f505b4b58ee869b

-- 
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 429749] Review Request: libvncserver - Library to make writing a vnc server easy

2008-01-31 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: libvncserver - Library to make writing a vnc server 
easy


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 14:50 EST ---
OK, rpmlint is down to:
  libvncserver-devel.x86_64: W: no-documentation
which is OK.
The errant permissions are fixed.
Everything else still looks good.

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 217197] Review Request: MyBashBurn - burn data and songs.

2008-01-31 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: MyBashBurn - burn data and songs.
Alias: MyBashBurn

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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:00 EST ---
OK, the license is fixed, but the tarball still does not match the upstream 
tarball.

If you have a Source0: url of 
http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz
then I simply must be able to download that tarball and get exactly, byte for
byte, what you are including in the srpm.  It is not acceptable for you to
modify that tarball in any way unless it contains content which we cannot
legally distribute.

If you are packaging something out of upstream SVN, we have rules about how you
document that.  You certainly cannot use the same version name because what you
want to distribute is not the same as what upstream ships with that version.  So
please see http://fedoraproject.org/wiki/Packaging/NamingGuidelines.  And if you
are going to package an SVN checkout then we also have rules about how you
document that so that I can make the same SVN checkout as you made:
http://fedoraproject.org/wiki/Packaging/SourceURL.

But honestly, do you have some compelling reason not to just ship what upstream
ships?  Just use their tarball in place of whatever you are trying to package.

-- 
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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:08 EST ---
MUST:
 - Package meets naming guidelines - OK
 - Spec file matches base package name. - OK
 - Spec has consistant macro usage. - ***
 - Meets Packaging Guidelines. - OK
 - License - GPLv2+ - OK
 - License field in spec matches - OK
 - License file included in package - ***
 - Spec in American English - OK
 - Spec is legible. - OK
 - Sources match upstream md5sum:
794208083d405ece123ad59a02f3e233 - OK
 - Package needs ExcludeArch - N/A
 - BuildRequires correct - OK
 - Spec handles locales/find_lang - OK
 - Package has %defattr and permissions on files is good. - OK
 - Package has a correct %clean section. - OK
 - Package has correct buildroot - OK
 - Package is code or permissible content. - OK
 - Doc subpackage needed/used. - N/A
 - Packages %doc files don't affect runtime. - OK, once they're added

 - Headers/static libs in -devel subpackage. - N/A, nuked
 - Spec has needed ldconfig in post and postun - OK
 - .pc files in -devel subpackage/requires pkgconfig - N/A
 - .la files are removed. - OK

 - Package compiles and builds on at least one arch. - once issue #1 addressed,
yes (tested x86_64)
 - Package has no duplicate files in %files. - OK
 - Package doesn't own any directories other packages own. - OK
 - Package owns all the directories it creates. - OK
 - No rpmlint output.
upstart.x86_64: W: conffile-without-noreplace-flag /etc/event.d/logd

This may be OK, if this isn't meant to be user modified.

 - final provides and requires are sane: ***

SHOULD Items:

 - Should build in mock. - OK (tested x86_64)
 - Should build on all supported archs - didn't test ppc
 - Should function as described. - ***
 - Should have sane scriptlets. - N/A
 - Should have dist tag - OK
 - Should package latest version - OK

Issues:
1) %configure --enable-compat=sysv --sbindir=/sbin --libdir=/lib

Needs to be --libdir=/%{_lib}, otherwise build will fail on x86_64, ppc64.

2) No docs, including licenses

Probably should include ChangeLog, AUTHORS, COPYING, NEWS, README, TODO, HACKING

3) Should this package require event-compat-sysv? Should it obsolete/provide
sysvinit?

4) Re: functionality. I think that will go in a separate bug.

-- 
Configure 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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||431106




-- 
Configure 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 429416] Review Request: event-compat-sysv - sysVinit compatibility for upstart

2008-01-31 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: event-compat-sysv - sysVinit compatibility for upstart


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||431106




-- 
Configure 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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn|431106  |




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:13 EST ---
5) The runlevel man page ends up screwed up somehow.

-- 
Configure 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 225806] Merge Review: glibc

2008-01-31 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: glibc


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:19 EST ---
OK, this was not as bad as I'd originally thought it would be. Here is a
commented diff between the existing devel spec and my cleaned up spec (which I
will attach to this bug ticket):

# First section
# There is no need to define macros that have the same values as Name, Version 
# and Release. Also, lets use %{glibcdate} to generate glibcsrcdir.

@@ -1,9 +1,6 @@
 %define glibcdate 20080131T0821
-%define glibcname glibc
-%define glibcsrcdir glibc-20080131T0821
+%define glibcsrcdir glibc-%{glibcdate}
 %define glibc_release_tarballs 0
-%define glibcversion 2.7.90
-%define glibcrelease 5
 %define run_glibc_tests 1
 %define auxarches i586 i686 athlon sparcv9v sparc64v alphaev6
 %define xenarches i686 athlon

# We really don't want to use hacks here, especially 
# _unpackaged_files_terminate_build. glibc doesn't even
# need it. Drop the summary period, and put the version/release here.

@@ -21,11 +18,13 @@
 %endif
 %define rtkaioarches %{ix86} x86_64 ia64 ppc ppc64 s390 s390x
 %define debuginfocommonarches %{ix86} alpha alphaev6 sparc sparcv9 sparcv9v
sparc64 sparc64v
-%define _unpackaged_files_terminate_build 0
-Summary: The GNU libc libraries.
+
+Summary: The GNU libc libraries
 Name: glibc
-Version: %{glibcversion}
-Release: %{glibcrelease}
+Version: 2.7.90
+# Normally, we'd embed the checkout tag in the release, but since glibc changes
+# checkout tag with every release, it is somewhat pointless to do so.
+Release: 6
 # GPLv2+ is used in a bunch of programs, LGPLv2+ is used for libraries.
 # Things that are linked directly into dynamically linked programs
 # and shared libraries (e.g. crt files, lib*_nonshared.a) have an additional

# Add the upstream URL.
# Use name rather than glibcname.
# Fix the buildroot to comply with Fedora standards
# Drop the ancient obsoletes, conflicts. We only keep conflicts around for 3 
# releases.
# Get rid of AutoReq: false.
# Use BuildRequires instead of BuildPreReq

@@ -33,48 +32,56 @@
 # libraries without restrictions.
 License: LGPLv2+ and LGPLv2+ with exceptions and GPLv2+
 Group: System Environment/Libraries
+URL: http://sources.redhat.com/glibc/
 Source0: %{glibcsrcdir}.tar.bz2
 %if %{glibc_release_tarballs}
 Source1: %(echo %{glibcsrcdir} | sed s/glibc-/glibc-linuxthreads-/).tar.bz2
 Source2: %(echo %{glibcsrcdir} | sed s/glibc-/glibc-libidn-/).tar.bz2
 %define glibc_release_unpack -a1 -a2
 %endif
-Source3: %{glibcname}-fedora-%{glibcdate}.tar.bz2
-Patch0: %{glibcname}-fedora.patch
+Source3: %{name}-fedora-%{glibcdate}.tar.bz2
+Patch0: %{name}-fedora.patch
 Patch1: %{name}-ia64-lib64.patch
-Buildroot: %{_tmppath}/glibc-%{PACKAGE_VERSION}-root
-Obsoletes: zoneinfo, libc-static, libc-devel, libc-profile, libc-headers,
-Obsoletes: gencat, locale, ldconfig, locale-ja, glibc-profile
+Buildroot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+# These obsoletes are ancient.
+# Obsoletes: zoneinfo, libc-static, libc-devel, libc-profile, libc-headers,
+# Obsoletes: gencat, locale, ldconfig, locale-ja, glibc-profile
 Provides: ldconfig
 # The dynamic linker supports DT_GNU_HASH
 Provides: rtld(GNU_HASH)
-Autoreq: false
+# Autoreq: false
 Requires: glibc-common = %{version}-%{release}
-%ifarch sparc
-Obsoletes: libc
-%endif
 # Require libgcc in case some program calls pthread_cancel in its %%post
-Prereq: basesystem, libgcc
+Requires(pre): basesystem, libgcc
 # This is for building auxiliary programs like memusage, nscd
 # For initial glibc bootstraps it can be commented out
-BuildPreReq: gd-devel libpng-devel zlib-devel texinfo, libselinux-devel = 
1.33.4-3
-BuildPreReq: audit-libs-devel = 1.1.3, sed = 3.95, libcap-devel, gettext
-BuildPreReq: /bin/ps, /bin/kill, /bin/awk
+BuildRequires: gd-devel libpng-devel zlib-devel texinfo, libselinux-devel =
1.33.4-3
+BuildRequires: audit-libs-devel = 1.1.3, sed = 3.95, libcap-devel, gettext
+BuildRequires: /bin/ps, /bin/kill, /bin/awk
 # This is to ensure that __frame_state_for is exported by glibc
 # will be compatible with egcs 1.x.y
-BuildPreReq: gcc = 3.2
-Conflicts: rpm = 4.0-0.65
-Conflicts: glibc-devel  2.2.3
-Conflicts: gcc4 = 4.0.0-0.6
-%ifarch x86_64 %{ix86}
+BuildRequires: gcc = 3.2
+# Last seen in RHL, no need to state this anymore.
+# Conflicts: rpm = 4.0-0.65
+# Last seen in RHL, no need to state this anymore.
+# Conflicts: glibc-devel  2.2.3
+# Last seen in FC-3, no need to state this anymore.
+# Conflicts: gcc4 = 4.0.0-0.6
+# %ifarch x86_64 %{ix86}
 # Need gdb that understands DW_CFA_val_expression
-Conflicts: gdb  6.3.0.0-1.111
-%endif
+# Last seen in preFC-5 era, no need to state this anymore.
+# Conflicts: gdb  6.3.0.0-1.111
+# %endif
 # Earlier shadow-utils packages had too restrictive permissions on
 # 

[Bug 429749] Review Request: libvncserver - Library to make writing a vnc server easy

2008-01-31 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: libvncserver - Library to make writing a vnc server 
easy


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:07 EST ---
thanks!  excellent review (as usual).

New Package CVS Request
===
Package Name: libvncserver
Short Description: Library to make writing a vnc server easy
Owners: rdieter
Branches: F-7 F-8 EL-5
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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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


[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 225806] Merge Review: glibc

2008-01-31 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: glibc


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:20 EST ---
Created an attachment (id=293646)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=293646action=view)
New glibc spec file, Fedora cleanups.


-- 
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 429416] Review Request: event-compat-sysv - sysVinit compatibility for upstart

2008-01-31 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: event-compat-sysv - sysVinit compatibility for upstart


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:42 EST ---
More rpmlint:
event-compat-sysv.noarch: W: incoherent-version-in-changelog 1 0.3.9-1.fc9

Remove the spaces around the '-' in the version-release should fix this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 429416] Review Request: event-compat-sysv - sysVinit compatibility for upstart

2008-01-31 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: event-compat-sysv - sysVinit compatibility for upstart


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn|431106  |
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:42 EST ---
MUST items:
 - Package meets naming guidelines - OK
 - Spec file matches base package name. - ***
 - Spec has consistant macro usage.  - OK
 - Meets Packaging Guidelines. - ***
 - License - ***
 - License field in spec matches - ***
 - License file included in package - ***
 - Spec in American English - OK
 - Spec is legible. - OK
 - Sources match upstream md5sum:
22d66ef8bc9d167eb822bbfecb584107 - OK
 - Package needs ExcludeArch - see note about buildarch below
 - BuildRequires correct - OK
 - Package has %defattr and permissions on files is good. - OK
 - Package has a correct %clean section. - OK
 - Package has correct buildroot - OK
  %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 - Package is code or permissible content. - OK
 - Doc subpackage needed/used. - N/A
 - Package compiles and builds on at least one arch. - OK
 - Package has no duplicate files in %files. - OK
 - Package doesn't own any directories other packages own. - OK
 - Package owns all the directories it creates. - OK
 - No rpmlint output. ***
 - final provides and requires are sane: ***

SHOULD Items:
 - Should build in mock. - OK (tested on x86_64)
 - Should build on all supported archs - it's noarch
 - Should function as described. 
 - Should have dist tag - OK
 - Should package latest version - OK

Issues:

1) Doesn't match upstream tarball; then again, 'example-jobs' is a pretty
awful name, so this should probably be ignored. Then again, this maybe
should move to initscripts at some point?

2) Package should have 'BuildArchitecture: noarch' and be built as a noarch
package - there's nothing arch specific about it, AFAICT

3) There's no license information in the source at all. Probably should be
cleaned up somewhere.

4) Explicit Provides for SysVinit seems wrong, especially w/o an obsolete. Need
to figure out whether the obsolete goes here or in upstart.

5) More rpmlint:
event-compat-sysv.noarch: W: non-conffile-in-etc /etc/event.d/rc2
event-compat-sysv.noarch: W: non-conffile-in-etc /etc/event.d/rc3
event-compat-sysv.noarch: W: non-conffile-in-etc /etc/event.d/rc0
...

Possibly should be marked %config(noreplace), as the admin may want to customize
these. Note: %config(noreplace) makes upgrading *very* hard in the case of event
file format changes.

6) See bug 431120.

-- 
Configure 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-01-31 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-01-31 15:50 EST ---
So this is back in the pool.  Perhaps I'll find the time to look at it if nobody
beats me to it.  If the package in comment 34 is the one you'd like reviewed,
please say so or go ahead and drop another package.

However, some things I didn't see mentioned before.  One warning from
desktop-file-install during the package build:
+ desktop-file-install --vendor fedora --dir
/var/tmp/lazarus-0.9.24-1.fc9-root-mockbuild/usr/share/applications
lazarus/install/lazarus.desktop
/var/tmp/lazarus-0.9.24-1.fc9-root-mockbuild/usr/share/applications/fedora-lazarus.desktop:
warning: value lazarus.png for key Icon in group Desktop Entry is an icon
name with an extension, but there should be no extension as described in the
Icon Theme Specification if the value is not an absolute path

and come rpmlint complaints:
  lazarus.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 10)
I'm not generally concerned about these.

  lazarus.src: W: invalid-license GPL, MPL
  lazarus.src: W: invalid-license modified LGPL
Please use proper License tags according to
http://fedoraproject.org/wiki/Licensing and
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines.  If the modified
LGPL in use doesn't fit any existing tags and something like LGPLv2+ with
exceptions isn't close enough then you will need to have the licensing experts
assign a new tag for you.

  lazarus.x86_64: E: non-executable-script 
   /usr/lib64/lazarus/components/lazreport/doc/cvs2cl.pl 0644
  lazarus.x86_64: E: non-executable-script 
   /usr/lib64/lazarus/examples/trayicon/createbundle.sh 0644
  lazarus.x86_64: E: non-executable-script 
   /usr/lib64/lazarus/components/lazreport/tools/localize.sh 0644
Are you sure these shouldn't be executable?  What will call them?

  lazarus.x86_64: E: non-executable-script 
   /usr/lib64/lazarus/tools/install/cross_unix/debian_crosswin32/postrm 0644
I thought that according to comments above that this would be going away.

  lazarus.x86_64: W: ldd-failed /usr/lib64/lazarus/tools/svn2revisioninc
I think this was covered in a discussion above about fpc output always being
static, and should be OK.

-- 
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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||431106




-- 
Configure 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 225806] Merge Review: glibc

2008-01-31 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: glibc


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:56 EST ---
(In reply to comment #2)

 +# Conflicts: kernel  2.6.9
 +# However, just to be safe, lets make this a Requires:
 +Requires: kernel = 2.6.9

This is not right for install in chroots. No package should ever
requires the kernel.


-- 
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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:58 EST ---
The problem with #3 is that something still needs to obsolete sysvinit to a)
force the upgrade b) get sysvinit off the system.

-- 
Configure 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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:49 EST ---
1) Will fix soon.

2) Ditto.

3) event-compat-sysv provides sysvinit. It requires upstart.

4) Broken in part due to the lack of a sysvinit-utils package. Still working
that out.

5) I'll look into this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 429416] Review Request: event-compat-sysv - sysVinit compatibility for upstart

2008-01-31 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: event-compat-sysv - sysVinit compatibility for upstart


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 15:59 EST ---
1) I don't think it belongs in initscripts. If anything it could merge into
upstart but I chose not to do this since technically a sysadmin could opt to
write his own events from scratch and not install any event scripts at all
(you'd have to assume he enjoyed pain, but its possible).

2) Whoops! I'll fix that.

3) Given the haphazard packaging, I wouldn't be surprised if it was public
domain, and the contents of these is so light I'm not certain its even
copyrightable material (IANAL), but I'll ask upstream.

4) It goes here, IMHO. The logic behind this is that it is in fact these scripts
which create the sysv-compatible environment, not upstart itself.

5) Event files are kind of a special case. I followed the guidelines for init
scripts, but we might need to add something to packaging procedure about how
these really should be done.

6) Already have a fix. Coming soon (see bug).

-- 
Configure 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 225806] Merge Review: glibc

2008-01-31 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: glibc


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 16:05 EST ---
It doesn't require the kernel, it just requires a version of the kernel which 
is new enough for glibc to work with. IMHO requiring is better than conflicting
anyway...

-- 
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 225806] Merge Review: glibc

2008-01-31 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: glibc


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 16:12 EST ---
But it requires the kernel to be installed, doesn't it? Requires
are better except for such cases. 

-- 
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 417711] Review Request: flam3 - Programs to generate and render cosmic recursive fractal flames

2008-01-31 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: flam3 - Programs to generate and render cosmic 
recursive fractal flames


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 16:16 EST ---
* Tue Jan 29 2008 Ian Weller [EMAIL PROTECTED] 2.7.8-3
- Removed config.h properly
- We now own datadir/flam3

SRPM: http://ianweller.org/rpm/flam3/flam3-2.7.8-3.fc8.src.rpm
spec: http://ianweller.org/rpm/flam3/flam3.spec

---

on another note, i've pretty much got qosmic ready to put up as a package, which
is utterly dependent on flam3. should i wait until flam3 is approved and whatnot
before I submit a review request for qosmic or should I do it after I double
check my packaging?

-- 
Configure 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 357661] Review Request: sunbird - Mozilla Sunbird Calendar

2008-01-31 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: sunbird - Mozilla Sunbird Calendar
Alias: sunbird-review

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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 16:20 EST ---
Lubomir: Okay, I found that using 'sunbird -uilocale de' will start sunbird with
German localization as expected. The Fedora Firefox start script in /usr/bin
does some hacks to pass the correct locale to Firefox. Maybe this should be the
same for sunbird?

Lightning seems to be different, though: First off all, its UI is English, too.
But I noticed that I don't see an langpacks for Lightning when I look in the
Add-ons menu. The thunderbird-lightning rpm does not contain any jars which do
contain localizations. Upstream lightning xpi files contain localizations as
calendar-$LANG.jar in the /chrome folder.



-- 
Configure 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 426827] Review Request: perl-DateTime-Precise - Perform common time and date operations with additional GPS operations

2008-01-31 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-DateTime-Precise - Perform common time and date 
operations with additional GPS operations


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 16:25 EST ---
Imported and built for devel, F-8, EL-4 and EL-5.

-- 
Configure 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 428113] Review Request: perl-Net-eBay - Perl Interface to XML based eBay API

2008-01-31 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-Net-eBay - Perl Interface to XML based eBay API


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


Bug 428113 depends on bug 426827, which changed state.

Bug 426827 Summary: Review Request: perl-DateTime-Precise - Perform common time 
and date operations with additional GPS operations
https://bugzilla.redhat.com/show_bug.cgi?id=426827

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 16:01 EST ---
That would be event-compat-sysv :)

It seems strange, but in effect, it is that collection of event files which
enables sysv compatibility, not upstart.

-- 
Configure 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 430859] Review Request: netmonitor - The free linux network bandwidth monitor

2008-01-31 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: netmonitor - The free linux network bandwidth monitor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 17:04 EST ---
Imported and built. Thanks again for review!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 225806] Merge Review: glibc

2008-01-31 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: glibc


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 17:29 EST ---
I would much rather have the versioned Requires than the versioned Conflicts.

This means that all of your installed kernels much be  2.6.9, or glibc hits
this Conflict. With it as a requires, as long as one kernel is = 2.6.9, we meet
this Requires.

But, I do see Patrice's point, this will pull kernel into the default buildroot.

Perhaps it would be best to drop it altogether, since the last Fedora with a
2.6.9 kernel was FC-3, 6 releases back from F-9. We'd leave it as a comment for
anyone foolish enough to try to run modern glibc with old kernels.

-- 
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 225806] Merge Review: glibc

2008-01-31 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: glibc


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 17:44 EST ---
In my opinion it is in such a case that Conflict is right.

-- 
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 429081] Review Request: perl-DateTime-Format-Oracle - Parse and format Oracle dates and timestamps

2008-01-31 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-DateTime-Format-Oracle - Parse and format Oracle 
dates and timestamps


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


[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, 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 429081] Review Request: perl-DateTime-Format-Oracle - Parse and format Oracle dates and timestamps

2008-01-31 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-DateTime-Format-Oracle - Parse and format Oracle 
dates and timestamps


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 18:12 EST ---
New Package CVS Request
===
Package Name: perl-DateTime-Format-Oracle
Short Description: Parse and format Oracle dates and timestamps
Owners: cweyl
Branches: F-7, F-8, devel
InitialCC: perl-sig

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 430509] Review Request: synce-kpm - SynCE KDE PDA Manager

2008-01-31 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-kpm - SynCE KDE PDA Manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 18:45 EST ---
Thanks yet again for another nice 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 427380] Review Request: efibootmgr - EFI Boot Manager

2008-01-31 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: efibootmgr - EFI Boot Manager


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 20:10 EST ---
build will be coordinated with build of elilo package removing efibootmgr.

-- 
Configure 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 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-31 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: R-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 20:20 EST ---
Thanks for the re-review.  I'll fix the changelog.

I already have an account.  I just need cvsextras approval to move forward.   
:-)

Note: This is the point where things got bogged down last time, so maybe
something is wrong with the state of my account (tmoertel).  Please let me
know if you find any problems.

Cheers,
Tom


-- 
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 431032] Review Request: ghost-diagrams - A program that generates patterns from tiles

2008-01-31 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: ghost-diagrams - A program that generates patterns 
from tiles


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 20:54 EST ---
Created an attachment (id=293679)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=293679action=view)
Potential icon source

(In reply to comment #1)
 --delete-original not needed in desktop-file-install

Strictly speaking, no. But I'd rather be pedantic about it.

 Do you got any icon to install for this desktop file?

Nothing official, but I was considering carving one from the attached image
generated by 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 431154] New: Review Request: IMDbPY - Retrieve and manage the data of the IMDb movie database

2008-01-31 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=431154

   Summary: Review Request: IMDbPY - Retrieve and manage the data of
the IMDb movie database
   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://allisson.fedorapeople.org/packages/IMDbPY/IMDbPY.spec
SRPM URL: 
http://allisson.fedorapeople.org/packages/IMDbPY/IMDbPY-3.4-1.fc8.src.rpm

Description: IMDbPY is a Python package useful to retrieve and manage the data 
of
the IMDb movie database.

IMDbPY is mainly a tool intended for programmers and developers, but
some example scripts are included.

-- 
Configure 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 250634] Review Request: perl-Mail-DKIM - Perl module for DKIM support

2008-01-31 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-Mail-DKIM -   Perl module for DKIM support


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 21:14 EST ---
The latest amavisd-new requires this module.  Leif, either you need to get this
review moving soon, or I'll have to close this and open a review request for my
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 431156] New: Review Request: ruby-openid - Ruby library for verifying and serving OpenID identities

2008-01-31 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=431156

   Summary: Review Request: ruby-openid - Ruby library for verifying
and serving OpenID identities
   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://allisson.fedorapeople.org/packages/ruby-openid/ruby-openid.spec
SRPM URL: 
http://allisson.fedorapeople.org/packages/ruby-openid/ruby-openid-2.0.3-1.fc8.src.rpm

Description: The Ruby OpenID library, with batteries included.

A Ruby library for verifying and serving OpenID identities. 
Ruby OpenID makes it easy to add OpenID authentication to 
your web applications.

-- 
Configure 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 429081] Review Request: perl-DateTime-Format-Oracle - Parse and format Oracle dates and timestamps

2008-01-31 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-DateTime-Format-Oracle - Parse and format Oracle 
dates and timestamps


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 21:25 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 429749] Review Request: libvncserver - Library to make writing a vnc server easy

2008-01-31 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: libvncserver - Library to make writing a vnc server 
easy


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 21:28 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 431032] Review Request: ghost-diagrams - A program that generates patterns from tiles

2008-01-31 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: ghost-diagrams - A program that generates patterns 
from tiles


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 22:01 EST ---
(In reply to comment #2)
 Created an attachment (id=293679)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=293679action=view) [edit]
 Potential icon source

OK
 
 (In reply to comment #1)
  --delete-original not needed in desktop-file-install
 
 Strictly speaking, no. But I'd rather be pedantic about it.
 
Anyway, Not a blocker for review.

  Do you got any icon to install for this desktop file?
 
 Nothing official, but I was considering carving one from the attached image
 generated by it.

Ok.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 431032] Review Request: ghost-diagrams - A program that generates patterns from tiles

2008-01-31 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: ghost-diagrams - A program that generates patterns 
from tiles


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 22:04 EST ---
Review:
+ package builds in mock (rawhide i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream.
e578104d433f48639d9cf73aade1ae9e  ghost-diagrams-0.8.py
+ 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 NOT 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.
+ Desktop file installed correctly.
+ GUI app.

 SHOULD:
Good if some kind of documentation say README included in next release.

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 431047] Review Request: latencytop - System latency monitor

2008-01-31 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: latencytop - System latency monitor


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 22:11 EST ---
waiting for your updated SRPM

-- 
Configure 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 431161] New: Review Request: mathmap - A gimp plugin and commandline tool

2008-01-31 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=431161

   Summary: Review Request: mathmap - A gimp plugin and commandline
tool
   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://people.redhat.com/rnorwood/rpms/mathmap.spec
SRPM URL: http://people.redhat.com/rnorwood/rpms/mathmap-1.2.4-1.fc9.src.rpm
Description: MathMap is a GIMP plug-in which allows distortion of images 
specified
by mathematical formulae.  For each pixel in the generated image, an
expression is evaluated which should return a pixel value.  The
expression can either refer to a pixel in the source image or can
generate pixels completely independent of the source.

-- 
Configure 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 431161] Review Request: mathmap - A gimp plugin and commandline tool

2008-01-31 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: mathmap - A gimp plugin and commandline tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 22:27 EST ---
There are unfortunately a few rpmlint warnings:

mathmap.i386: W: devel-file-in-non-devel-package
/usr/share/gimp/2.0/mathmap/new_template.c
mathmap.i386: W: devel-file-in-non-devel-package
/usr/share/gimp/2.0/mathmap/blender_opmacros.h
mathmap.i386: W: devel-file-in-non-devel-package 
/usr/share/gimp/2.0/mathmap/pools.h
mathmap.i386: W: devel-file-in-non-devel-package
/usr/share/gimp/2.0/mathmap/opmacros.h
mathmap.i386: W: devel-file-in-non-devel-package
/usr/share/gimp/2.0/mathmap/blender_template.c

These seem to be required for the plugin to work at runtime, despite being
source files.

mathmap.i386: W: symlink-should-be-relative /usr/lib/gimp/2.0/plug-ins/mathmap
/usr/bin/mathmap

Not sure if this matters.

mathmap.i386: E: only-non-binary-in-usr-lib

I don't think I can fix this, because the gimp plugin path isn't picked by 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, 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 429416] Review Request: event-compat-sysv - sysVinit compatibility for upstart

2008-01-31 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: event-compat-sysv - sysVinit compatibility for upstart


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-01 00:17 EST ---
New package:

http://www4.ncsu.edu/~cjdahlin/event-compat-sysv-0.3.9-3.fc8.src.rpm
http://www4.ncsu.edu/~cjdahlin/event-compat-sysv.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 429028] Review Request: upstart - an event-driven init daemon

2008-01-31 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: upstart - an event-driven init daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-01 00:18 EST ---
New package:

http://www4.ncsu.edu/~cjdahlin/upstart-0.3.9-3.fc8.src.rpm
http://www4.ncsu.edu/~cjdahlin/upstart.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 431013] Review Request: perl-CSS-Squish - Compact many CSS files into one big file

2008-01-31 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-CSS-Squish -  Compact many CSS files into one big 
file


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-01 00:59 EST ---
New Package CVS Request
===
Package Name: perl-CSS-Squish
Short Description: Compact many CSS files into one big file
Owners: corsepiu
Branches: F-7 F-8
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


  1   2   >