[Bug 221010] Review Request: librfid - Free Software RFID library

2006-12-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: librfid - Free Software RFID library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 03:18 EST ---
(In reply to comment #1)
 Minor:
 * I really don't think you need to add the same docs that are in the main
 package to the devel package.  This isn't a blocker, but I would drop them.
 
 +1 APPROVED
Done :)


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221010] Review Request: librfid - Free Software RFID library

2006-12-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: librfid - Free Software RFID library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 03:19 EST ---
(In reply to comment #2)
 One thing to consider:
 
 The %{_bindir}/send_script in the main package has a very generic
 file name and bears the risk of creating a conflict with other
 packages. The upstream authors are advised to choose a more unique
 file name.
 

Will inform the upstream authors.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221010] Review Request: librfid - Free Software RFID library

2006-12-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: librfid - Free Software RFID library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220945] Review Request: perl-Text-CharWidth - Get number of occupied columns of a string on terminal

2006-12-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-Text-CharWidth - Get number of occupied columns 
of a string on terminal


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 03:27 EST ---
It definitely would require the package to be installed.  But this was discussed
in September on fedora-extras-list and I filed an upstream RFE, but it was
rejected.  rpmlint site is broken at the moment, but when it comes back up, the
RFE can be seen at http://rpmlint.zarb.org/cgi-bin/trac.cgi/ticket/35

See also http://thread.gmane.org/gmane.linux.redhat.fedora.maintainers/2883/

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220766] Review Request: ScientificPython - a collection of Python modules that are useful for scientific computing

2006-12-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: ScientificPython -  a collection of Python modules 
that are useful for scientific computing


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 03:31 EST ---
fixed and built in the development tree, awaiting signing.

-jef

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221015] Review Request: ksplash-engine-moodin - Moodin is a splash engine for KDE Desktop

2006-12-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: ksplash-engine-moodin - Moodin is a splash engine for 
KDE Desktop


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 03:52 EST ---
Can't do an official Review as i can't sponsor you.

MUST :
- The .la libtool archives must be removed in the spec.
- The %{name}.desktop file  must be properly installed with desktop-file-install
in the %install section, see
http://fedoraproject.org/wiki/Packaging/Guidelines#desktop



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221010] Review Request: librfid - Free Software RFID library

2006-12-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: librfid - Free Software RFID library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 04:27 EST ---
(I'm the upstream author):

The send_script program has been renamed into librfid_send_script in our SVN
tree. The next release of the library will reflact that change.

Thanks for letting us know.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 194563] Review Request: conman - the console manager

2006-12-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: conman - the console manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 05:51 EST ---
(In reply to comment #13)
 Built for FC4 and 5, closing ticket.

Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 193846] Review Request: ganglia - Ganglia Distributed Monitoring System

2006-12-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: ganglia - Ganglia Distributed Monitoring System


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 05:53 EST ---
(In reply to comment #8)
 Packages all built, closing.

Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 198586] Review Request: ip6sic - IPv6 Stack Integrity Checker

2006-12-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: ip6sic - IPv6 Stack Integrity Checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 05:55 EST ---
(In reply to comment #3)
 Built for FC5, closing ticket.

Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 194355] Review Request: imlib

2006-12-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: imlib


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|133779  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 05:57 EST ---
Fixed typo...

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 165689] Review Request: SquidGuard: filter, redirector and access controller plugin for squid

2006-12-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: SquidGuard: filter, redirector and access controller 
plugin for squid


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  QAContact|[EMAIL PROTECTED]  |fedora-package-
   ||[EMAIL PROTECTED]
OtherBugsDependingO||163779
  nThis||
 CC|fedora-package- |
   |[EMAIL PROTECTED]   |




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 06:21 EST ---
This should now block FE-ACCEPT.

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

2006-12-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: mono-debugger


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 06:25 EST ---
Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 189727] Review Request: Scribes

2006-12-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: Scribes


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 06:28 EST ---
Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 195412] Review Request: obconf

2006-12-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: obconf


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 06:30 EST ---
Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 195292] Review Request: Openbox

2006-12-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: Openbox


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 06:32 EST ---
Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 174915] Review Request: gnome-sudoku - A numerical logic puzzle game

2006-12-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-sudoku - A numerical logic puzzle game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC|fedora-extras-  |
   |[EMAIL PROTECTED] |
 CC||fedora-package-
   ||[EMAIL PROTECTED]
 CC|fedora-package- |
   |[EMAIL PROTECTED]   |
 CC||fedora-extras-
   ||[EMAIL PROTECTED]

[EMAIL PROTECTED] changed:

   What|Removed |Added

  QAContact|[EMAIL PROTECTED]  |fedora-package-
   ||[EMAIL PROTECTED]
OtherBugsDependingO||163779
  nThis||
 CC|fedora-extras-  |
   |[EMAIL PROTECTED] |




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 06:37 EST ---
Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
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 175291] Review Request: perl-PatchReader - utilities to read and manipulate patches and CVS

2006-12-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-PatchReader - utilities to read and manipulate 
patches and CVS


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC|fedora-extras-  |
   |[EMAIL PROTECTED] |
 CC||fedora-package-
   ||[EMAIL PROTECTED]

[EMAIL PROTECTED] changed:

   What|Removed |Added

  QAContact|[EMAIL PROTECTED]  |fedora-package-
   ||[EMAIL PROTECTED]
OtherBugsDependingO||163779
  nThis||
 CC|fedora-package- |
   |[EMAIL PROTECTED]   |




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 06:39 EST ---
Please do not remove the FE-ACCEPT blocker.  Thanks.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
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 221039] Review Request: aria2 - High speed download utility with resuming and segmented downloading.

2006-12-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: aria2 - High speed download utility with resuming and 
segmented downloading.


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 07:10 EST ---
(In reply to comment #2)
 download accellerators with segmented downloading of one file with multiple
 connections from one URL are evil.

Nevertheless, they can be used by a lot of people and also metalink feature is
quite nice :)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude

2006-12-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: libpreludedb - Provides an abstraction layer for the 
libprelude


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 07:22 EST ---
For 0.9.11-2:

* Directories/files ownership issue
- Well, this issue becomes rather complicate when one package
  creates several subpackages like this package. You have to
  make it sure that the directories created by the installation
  of your packages are actually owned by some package for
  every situation.

  - For example, now we check /usr/lib/libpreludedb/plugins/sql/.
This directory is owned by -mysql package, however, this
is actually wrong.

When we install main + -sqlite package (with yum), it succeeds
without -mysql package. The installation of -sqlite creates
/usr/lib/libpreludedb/plugins/sql/ , however in this situation
this package is not owned by any package.

The following can happen:
--
[EMAIL PROTECTED] ~]$ rpm -qa --qf '%{NAME}\n'  libpreludedb\* | sort
libpreludedb
libpreludedb-devel
libpreludedb-python
libpreludedb-sqlite
[EMAIL PROTECTED] ~]$ LANG=C rpm -qf /usr/lib/libpreludedb/plugins/sql/
file /usr/lib/libpreludedb/plugins/sql is not owned by any package
--
To resolve this unwilling ownership issue, you should either
= make -sqlite package require -mysql package *if this is proper*
  (i.e. it is really true that -sqlite package requires -mysql
   package).
= or change the owner of the directory from -mysql package to
  main package.

  - Some other directories have this issue. Please check it.

* rpmlint issue
-
W: libpreludedb macro-in-%changelog python_sitearch (for srpm)
-
  Macros are expanded on %changelog. To avoild this, please use:
-
- fixed x86_64 arch build problem with %%python_sitearch
-

* Requirement
  - For devel package
-
Requires: libpreludedb = %{version}-%{release}, gnutls-devel, automake
-
Well, gnutls-devel is insufficient as
/usr/bin/libpreludedb-config --libs require libprelude.so.
gnutls-devel should be libprelude-devel.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221039] Review Request: aria2 - High speed download utility with resuming and segmented downloading.

2006-12-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: aria2 - High speed download utility with resuming and 
segmented downloading.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 07:33 EST ---
I am not sponsored, it is prereview.

-Rpmlint:
(S)RPM: W: aria2 summary-ended-with-dot High speed download utility with
resuming and segmented downloading.

Summary is:
Summary:High speed download utility with resuming and segmented
downloading.
It should be:
Summary:High speed download utility with resuming and segmented
downloading

-The package is named according to the Package Naming Guidelines.
-The spec file name is in the format %{name}.spec
-I think that the package meets the Packaging Guidelines.
-License GPL - ok
-In spec the actual license is written
-The text of the license is included in %doc
-The spec file is written in American English.
-The spec file for the package is legible.
-md5sum source in srpm and from url is the same
-SRPM succesfully compiles with mock (Fc6, arch i386)
-All build dependencies are listed in BuildRequires. There are not any packages
from ExceptionList
-The spec file handle locales properly. This is done by using the %find_lang
macro. There are not used %{_datadir}/locale/*
-RPM package have not shared library files
-The package is not designed to be relocatable.
-The package own all directories that it creates. 
-There are not any duplicate files in %files listing.
-Permissions on files are set properly.
-Package have a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
-The package consistently uses macros
-The package contains code, or permissable content.
-There are not large documentation files
-Files in %doc do not affect the runtime of the application.
-There are not header files or static libraries.
-There are not pkgconfig(.pc) files.
-There are not any library files with a suffix
-There are not a devel package
-There are not any .la libtool archives, these should be removed in the spec.
-It is not a GUI application
-Packages does not own files or directories already owned by other packages. 

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221039] Review Request: aria2 - High speed download utility with resuming and segmented downloading.

2006-12-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: aria2 - High speed download utility with resuming and 
segmented downloading.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221039] Review Request: aria2 - High speed download utility with resuming and segmented downloading.

2006-12-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: aria2 - High speed download utility with resuming and 
segmented downloading.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica

2006-12-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: libscigraphica - A library of gtk+ widgets for 
SciGraphica


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 08:33 EST ---
Builds cleanly in mock
rpmlint is quiet (warning on the devel package, but it can be ignored)

Review

Bad:
Release:2 - must have the .%{?dist} tag
Requires:   %name = %{version}-%{release} - inconsistent naming - should be 
%{name}
%{_libdir}/scigraphica/ - this takes the ownership of the directory which should
be owned by scigraphical

Good:
UTF-8, American English, contains documentation for the main package
pkgconfig in R for the -devel package
MD5SUM matches upstream
No .la or .a files packaged

Unsure:
Does the -devel need ldconfig to be run?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221039] Review Request: aria2 - High speed download utility with resuming and segmented downloading.

2006-12-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: aria2 - High speed download utility with resuming and 
segmented downloading.


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 08:38 EST ---
(In reply to comment #4)
 Summary is:
 Summary:High speed download utility with resuming and segmented
 downloading.
 It should be:
 Summary:High speed download utility with resuming and segmented
 downloading


Fixed.

Spec URL: http://ecik.nonlogic.org/aria/aria2.spec
SRPM URL: http://ecik.nonlogic.org/aria/aria2-0.9.0-2.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220968] Review Request: scigraphica - Scientific application for data analysis and technical graphics

2006-12-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: scigraphica - Scientific application for data analysis 
and technical graphics


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 08:38 EST ---
rpmlint is giving 2 warnings for the main package - looks like you need a devel
subpackage adding

W: scigraphica devel-file-in-non-devel-package
/usr/share/pixmaps/scigraphica/stock_pixmaps.h
W: scigraphica devel-file-in-non-devel-package
/usr/share/pixmaps/scigraphica/pixmaps.h



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221027] Review Request: LabPlot - Data Analysis and Visualization

2006-12-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: LabPlot - Data Analysis and Visualization


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude

2006-12-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: libpreludedb - Provides an abstraction layer for the 
libprelude


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 08:59 EST ---
* dirowner problem
I changed the owner of some directories to the main package, which seems to be
the smartest solution here.

* escaped the macro in %changelog

* changed the requirements in -devel

find the new packages here:
http://people.redhat.com/tscherf/fedora-extra/libpreludedb-0.9.11-3.src.rpm
http://people.redhat.com/tscherf/fedora-extra/libpreludedb.spec

Thanks.




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221039] Review Request: aria2 - High speed download utility with resuming and segmented downloading.

2006-12-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: aria2 - High speed download utility with resuming and 
segmented downloading.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 08:59 EST ---
- MUST: rpmlint is silent
- MUST: Package is named well
- MUST: Spec file name matches the package name
- MUST: Package meets packaging guidelines
- MUST: Pachage is licensed with GPL
- MUST: License field matches actual license
- MUST: COPYING file is included in %doc
- MUST: Spec is written in English, most probably American (I am not that
familiar with the differences to be 100% sure)
- MUST: I can understand the spec file
- MUST: MD5 sums match
- MUST: Package build in mock (fc6/i386)
- MUST: I can't check other architectures
- MUST: All deps are listed in BR
- MUST: %find_lang is used
- MUST: Package does not contain shared libs
- MUST: Package is not relocatable
- MUST: Directory ownership is ok
- MUST: There are no duplicates in %files
- MUST: Permissions are OK
- MUST: Clean section is present
- MUST: Macros are used consistently
- MUST: Package contains code
- MUST: There is no large documentation
- MUST: %doc is not required at runtime
- MUST: There is no headers nor static libs
- MUST: No pkgconfig files
- MUST: There is no shared libs
- MUST: There is no -devel subpackage
- MUST: There is no libtool archives
- MUST: Not a GUI app
- MUST: Pachage does not own dirs owned by other packages

SHOULD Items:
- SHOULD: License is included
- SHOULD: There are not translantions
- SHOULD: Package builds in mock (fc6/i386)
- SHOULD: The package should compile and build into binary rpms on all supported
architectures.
- SHOULD: Package works (smoke tested on fc6/i386)
- SHOULD: Scriptlets are not used
- SHOULD: There is no devel subpackage
- SHOULD: There are no pkgconfig files

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220968] Review Request: scigraphica - Scientific application for data analysis and technical graphics

2006-12-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: scigraphica - Scientific application for data analysis 
and technical graphics


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 09:22 EST ---
Builds cleanly in mock

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221045] Review Request: gtk-rezlooks-engine - Rezlooks GTK2 engine

2006-12-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: gtk-rezlooks-engine - Rezlooks GTK2 engine


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 09:32 EST ---
What's the license of the Source1? files?  Those archives don't appear to
contain a license.

But otherwise:
 * rpmlint output is clean
 * source in srpm is as spec file specifies
 * builds on i386
 * installs and works on i386

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221054] New: Review Request: gerbv - A gerber file viewer

2006-12-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/bugzilla/show_bug.cgi?id=221054

   Summary: Review Request: gerbv - A gerber file viewer
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://xgoat.com/rpm/gerbv/gerbv.spec
SRPM URL: http://xgoat.com/rpm/gerbv/gerbv-1.0.2-1.src.rpm
Description: 
Gerber Viewer (gerbv) is a viewer for Gerber files. Gerber files are
generated from PCB CAD systems and sent to PCB manufacturers as a
basis for the manufacturing process.

This is my first package, and I need a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221054] Review Request: gerbv - A gerber file viewer

2006-12-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: gerbv - A gerber file viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 09:54 EST ---
Added FE-NEEDSPONSOR

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221031] Review Request: PyOpenGL - Cross platform Python binding to OpenGL

2006-12-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: PyOpenGL - Cross platform Python binding to OpenGL


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 10:00 EST ---
New URLs:
Spec URL: http://rafalzaq.webd.pl/fedora/PyOpenGL/PyOpenGL.spec
SRPM URL: 
http://rafalzaq.webd.pl/fedora/PyOpenGL/PyOpenGL-3.0.0-0.1.a5.fc6.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221031] Review Request: PyOpenGL - Cross platform Python binding to OpenGL

2006-12-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: PyOpenGL - Cross platform Python binding to OpenGL


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 10:04 EST ---
How alpha is this software?


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221045] Review Request: gtk-rezlooks-engine - Rezlooks GTK2 engine

2006-12-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: gtk-rezlooks-engine - Rezlooks GTK2 engine


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 10:28 EST ---
I have found that unfortunately themes rarely contain a license. However
Gilouche and Graphite are linked to from the main gnome-look.org page for the
theme and there the license is specified to be GPL.  The rest of the themes are
from the same author and all appear next to each other on his website
(http://people.os-zen.net/rezza/themes/).

So I assumed GPL. I will try to get in touch with him to clear up any licensing
issues.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221054] Review Request: gerbv - A gerber file viewer

2006-12-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: gerbv - A gerber file viewer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 11:10 EST ---
* In build log:

gerbv.c:1526: warning: call to __builtin___snprintf_chk will
always overflow destination buffer


* Gerber file format support fails to load test files.

* Drill file format support crashes with buffer overflow.

* Passing files at the command-line results in 100% buffer
overflow crash:

*** buffer overflow detected ***: gerbv terminated
=== Backtrace: =
/lib/libc.so.6(__chk_fail+0x41)[0x87c161]
/lib/libc.so.6[0x87bba3]
/lib/libc.so.6(__snprintf_chk+0x37)[0x87ba87]
gerbv[0x8055686]
gerbv[0x805625a]
/lib/libglib-2.0.so.0[0x12a5e1]


* Directories %{_datadir}/gerbv and %{_datadir}/gerbv/scheme
are not included:

$ rpmls -p /home/qa/tmp/rpm/RPMS/gerbv-1.0.2-1.i386.rpm
-rwxr-xr-x  /usr/bin/gerbv
-rw-r--r--  /usr/share/applications/fedora-gerbv.desktop
-rw-r--r--  /usr/share/gerbv/scheme/init.scm
-rw-r--r--  /usr/share/icons/hicolor/48x48/apps/gerbv.png
-rw-r--r--  /usr/share/man/man1/gerbv.1.gz


* Scriptlets for GTK icon cache are missing:
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets?action=showredirect=ScriptletSnippets#head-7103f6c38d1b5735e8477bdd569ad73ea2c49bda


* No %doc files are included. Not even the GPL license.

* Manual pages are marked %doc automatically.

* Upstream project site contains the following warning: If anyone is interested
to takeover this project, please contact Stefan. Else this project will risk to
rot away yet again.


Hints:

* It is commonly considered better style to make Summary in
headline-style:

Summary: Gerber file viewer from the gEDA toolkit

* Second sentence in %description is missing punctuation.

* Manual pages contains unexpanded ${prefix}.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude

2006-12-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: libpreludedb - Provides an abstraction layer for the 
libprelude


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 12:31 EST ---
In Japan (EST+14h) new year started. A Happy new year!

Well, one more issue:
---
%doc %{_defaultdocdir}/%{name}-%{version}/*
---

I watched HTML docs under this directory and it seems
they should be -devel package.

= Now directories ownership issue are correctly fixed.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude

2006-12-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: libpreludedb - Provides an abstraction layer for the 
libprelude


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 12:45 EST ---
In Germany we still have a couple of hours before the New Year starts, but
nevertheless, wish you a very happy New Year in Japan. :)

You are right with the docs, the html stuff should go into -devel. Done.

Find new packages here:
http://people.redhat.com/tscherf/fedora-extra/libpreludedb-0.9.11-3.src.rpm
http://people.redhat.com/tscherf/fedora-extra/libpreludedb.spec



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220979] Review Request: tesseract - Raw OCR Engine

2006-12-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: tesseract - Raw OCR Engine


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 12:52 EST ---
I had to change URLs, here are correct:
SPEC: http://karlik.nonlogic.org/tesseract/tesseract.spec
SRPM: http://karlik.nonlogic.org/tesseract/tesseract-1.02-2.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude

2006-12-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: libpreludedb - Provides an abstraction layer for the 
libprelude


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 13:15 EST ---
Okay.

As I get 404 for http://prelude-ids.org/%{name}-%{version}.tar.gz,
Please change to 
http://prelude-ids.org/download/releases/%{name}-%{version}.tar.gz

Other things are okay.

---
   This package (libpreludedb) is APPROVED by me.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2006-12-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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 13:42 EST ---
(In reply to comment #7)
 builds in mock in fc6 and devel on x86.
 I'm going through this now, doing a detailed review.
 
 PROBLEM #1:
 poker2d desktopfile has the EXEC in the wrong location.
 should be EXEC=%{_exec_prefix}/games/poker2d
 because %{_exec_prefix}/games/poker2d

Fixed this.

 
 because /usr/games is not in the standard executable path.
 
 PROBLEM #2:
 On my fc-6 system gnome seems to be unable to recognize that poker2D.xpm is
 there. Even if I create a custom panel launcher and attemnpt to select the
 poker2D.xpm from the pixmap directory in the icon chooser interface, it throws
 an icon '[EMAIL PROTECTED]' not found error dialog. Do we have to do somethng 
 silly and
 convert this to png?   

I asked in #fedora-packaging about icons placed in pixmap directory, and this is
the reply:

15:27:20   XulChris | if upstream installs in icon in %{_datadir}/pixmaps/
should it be moved to %{_datadir}/icons/ instead and run gtk-update-icon-cache?
   15:28:02   XulChris | or should it just be left in pixmaps/ w/o
running gtk-update-icon-cache? 
  17:49:19 ---| User: *** rdieter_away is now
known as rdieter1
18:05:45   rdieter1 |  XulChris: imo, move it, yadda, and ping upstream to
do the same.

So perhaps moving the icon to the icons/ directory will fix it in GNOME? Not
sure since I run KDE, I will try moving the icons in the spec let me know if it
fixes it in GNOME.



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221059] New: Review Request: cowsay - configurable speaking/thinking cow

2006-12-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/bugzilla/show_bug.cgi?id=221059

   Summary: Review Request: cowsay - configurable speaking/thinking
cow
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://ecik.nonlogic.org/cowsay/cowsay.spec
SRPM URL: http://ecik.nonlogic.org/cowsay/cowsay-3.03-1.src.rpm
Description:
cowsay is a configurable talking cow, written in Perl.  It operates
much as the figlet program does, and it written in the same spirit
of silliness.
It generates ASCII pictures of a cow with a message. It can also generate
pictures of other animals.

rpmlint gives following output:
W: cowsay non-conffile-in-etc /etc/bash_completion.d/cowsay.bashcomp
but that same output is given on rpmlint.rpm so it looks like we can ignore it.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2006-12-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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 13:47 EST ---
(In reply to comment #10)
 PROBLEM #3: poker-network-devel needs to require pkgconfig
 
 MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' (for
 directory ownership and usability).

Fixed.

 
 
 Clarification:
 is %{_libdir}/%{name}/poker-interface a shared library? I don't think it is 
 but
 I just want to be sure.
 
 

I will ask upstream to make sure. Will get back to you on this item as well.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2006-12-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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 13:50 EST ---
(In reply to comment #11)
 poker2d appears to work.  I haven't had a chance to test the server related
 subpackages. 
 
 I'm not a kde user, so forgive me if I sound ignorant, but the payload for
 poker2d-kde looks very thin to me.  Is the payload really only suppose to be:
 /usr/share/apps/khotkeys/poker2d.khotkeys
 /usr/share/doc/poker2d-kde-1.0.32
 /usr/share/doc/poker2d-kde-1.0.32/COPYING

This is done on purpose.  Otherwise a user would have to install kdebase to pick
up the /usr/share/apps/khotkeys/ directory when they install poker2d which is
probably not what people want to do if they do not have KDE installed.  Having a
package with a single file in it I deemed a better alternative than requiring
kdebase just to install poker2d, and poker2d does not need kdebase to run.

So basically its just a directory ownership issue.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220889] Review Request: fakechroot - Gives a fake chroot environment

2006-12-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: fakechroot - Gives a fake chroot environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 13:53 EST ---
Well, a very quick check:

* Please add a license documentation.
  Note: LICENSE documentation says that this is licensed under
  LGPL, not GPL.
* Can the files under test/ can be used for %check?
* What are other scripts under scripts/ directories?
* %{_libdir}/fakechroot/ is not owned by any packages.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220789] Review Request: fail2ban - Ban IPs that make too many password failures

2006-12-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: fail2ban - Ban IPs that make too many password failures


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude

2006-12-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: libpreludedb - Provides an abstraction layer for the 
libprelude


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 14:54 EST ---
ok, changed url. thanks for the hint.

will upload the package to the buildsystem.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221045] Review Request: gtk-rezlooks-engine - Rezlooks GTK2 engine

2006-12-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: gtk-rezlooks-engine - Rezlooks GTK2 engine


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 15:13 EST ---
I have just received an e-mail from the author confirming that the themes are
under the GPL license.

So everything is okay there as well.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 178922] Review Request: asterisk - The Open Source PBX

2006-12-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: asterisk - The Open Source PBX


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 15:38 EST ---
Have you had a chance to look at upgrading to 1.4.0 final?

Are there any major blockers to getting this into Extras now?  This review is
getting rather long, so I'm not quite sure...  :-)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 200498] Review Request: btsco-kmod

2006-12-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: btsco-kmod


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 16:28 EST ---
It appears that btsco is depreciated now... 
the new 'plugz' bluetooth-alsa project no longer requires a kernel module. 

See: 
http://bluetooth-alsa.sourceforge.net/history.html

Perhaps this should be closed now?
Might you consider packaging the new 'plugz' and 'sbc' packages?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 200499] Review Request: btsco-kmod-common

2006-12-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: btsco-kmod-common


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 16:28 EST ---
It appears that btsco is depreciated now... 
the new 'plugz' bluetooth-alsa project no longer requires a kernel module. 

See: 
http://bluetooth-alsa.sourceforge.net/history.html

Perhaps this should be closed now?
Might you consider packaging the new 'plugz' and 'sbc' packages?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221065] New: Review Request: warzone2100 - Innovative 3D real-time strategy

2006-12-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/bugzilla/show_bug.cgi?id=221065

   Summary: Review Request: warzone2100 - Innovative 3D real-time
strategy
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://karlik.nonlogic.org/warzone/warzone2100.spec
SRPM URL: http://karlik.nonlogic.org/warzone/warzone2100-2.0.5-1.src.rpm
Description:
Warzone 2100 was an innovative 3D real-time strategy game back in 1999, and
most will agree it didn't enjoy the commercial success it should have had. The
game's source code was liberated on December 6th, 2004, under a GPL license
(see COPYING in this directory for details). Soon after that, the Warzone 2100
ReDev project was formed to take care of its future.

I am looking for a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221065] Review Request: warzone2100 - Innovative 3D real-time strategy

2006-12-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: warzone2100 - Innovative 3D real-time strategy


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221065] Review Request: warzone2100 - Innovative 3D real-time strategy

2006-12-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: warzone2100 - Innovative 3D real-time strategy


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 17:54 EST ---
This is not an official review, I'm looking for a sponsor.

* rpmlint returns only no-documentation in warzone2100-data package
* package and specfile are named well
* package is licensed under GPL (which is included)
* specfile is legible
* source file match upstream
* package successfully build in mock (FC6 i386)- BuildRequires are proper
* no locales
* no duplicates in %files

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 178922] Review Request: asterisk - The Open Source PBX

2006-12-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: asterisk - The Open Source PBX


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 18:19 EST ---
I'm upgrading my phone system today, so pardon the noise.  :-)

Issue #1:
In the script you use to modify the source from Digium, you have this:

# tar xf asterisk-1.4.0-beta4.tar.gz

That should have been this:

# tar zxf asterisk-1.4.0-beta4.tar.gz

Other than that, following the directions does indeed give me identical sources
(according to diff -urN) to the source included in the src.rpm.

Issue #2:
I'd suggest changing %post to something like this:

/sbin/chkconfig --add asterisk || :
/sbin/service asterisk condrestart || :

If there is a good way to do a graceful restart here, that would be even better.

I'd also prefer it if %pre and %preun looked like this:

%pre
/usr/sbin/fedora-groupadd 31 -r asterisk || :
/usr/sbin/fedora-useradd  31 -r -s /sbin/nologin -d /var/lib/asterisk -M \
 -c 'Asterisk User' -g asterisk asterisk || :

%preun
if [ $1 = 0 ]; then
/sbin/service asterisk stop || :
/sbin/chkconfig --del asterisk || :
fi

That way we see failures, but they don't make rpm fail.  When I installed the
packages on my phone system (which had old asterisk/zaptel/libpri packages of
mine, which might be the problem), I got the following error:

  Installing: zaptel   ### [ 9/13] 
id: asterisk: No such user
usermod: user asterisk does not exist
error: %pre(asterisk-zaptel-1.4.0-6.beta4.fc6.i386) scriptlet failed, exit 
status 6
error:   install: %pre scriptlet failed (2), skipping
asterisk-zaptel-1.4.0-6.beta4.fc6

Since the asterisk %pre was redirected to /dev/null, I have no idea if there was
a failure there.  Again, the real problem might have been caused by old
packages, but, since I don't have diagnostic output, I can't be sure.  I would
definitely test an install on a clean system though to be safe.

So far those are the only problems that I've found.  The package builds fine in
mock (i386 and x86_64), and the handful of rpmlint warnings can be safely 
ignored.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica

2006-12-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: libscigraphica - A library of gtk+ widgets for 
SciGraphica


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 18:22 EST ---
(In reply to comment #6)

 Bad:
 Release:  2 - must have the .%{?dist} tag
 Requires: %name = %{version}-%{release} - inconsistent naming - should be 
 %{name}

Fixed

 %{_libdir}/scigraphica/ - this takes the ownership of the directory which 
 should
 be owned by scigraphical
 
Maybe not. It just seems to design mistake on the part of upstream to name that
directory scigraphica, its a superset of things scigraphica itself installs
under it. I've modified scigraphica not to own that directory, but just install
it own stuff under it.

 Unsure:
 Does the -devel need ldconfig to be run?
I believe the package guildline says it has to be done for the package and its
subpackage that contains a *.so.

Corrected spec and srpms below, thanks.

Spec URL: ftp://czar.eas.yorku.ca/pub/scigraphica/libscigraphica.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/scigraphica/libscigraphica-2.1.1-3.src.rpm

 

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica

2006-12-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: libscigraphica - A library of gtk+ widgets for 
SciGraphica


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 18:25 EST ---
The scigraphica directory must belong to the application and not the library.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220968] Review Request: scigraphica - Scientific application for data analysis and technical graphics

2006-12-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: scigraphica - Scientific application for data analysis 
and technical graphics


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 18:28 EST ---
(In reply to comment #2)
 rpmlint is giving 2 warnings for the main package - looks like you need a 
 devel
 subpackage adding
 
 W: scigraphica devel-file-in-non-devel-package
 /usr/share/pixmaps/scigraphica/stock_pixmaps.h
 W: scigraphica devel-file-in-non-devel-package
 /usr/share/pixmaps/scigraphica/pixmaps.h
 
The rpmlint warnings can be ignored as those *.h are just listings of the
contained pixmaps, I don't think they are being used to build anything.
I've corrected the package not to own %{_libdir}/%{name} (already owned by
libscigraphical) but it sub-directory that it creates.

Spec URL: ftp://czar.eas.yorku.ca/pub/scigraphica/scigraphica.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/scigraphica/scigraphica-2.1.0-2.src.rpm




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2006-12-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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 18:35 EST ---
(In reply to comment #14)
 (In reply to comment #10)
  Clarification:
  is %{_libdir}/%{name}/poker-interface a shared library? I don't think it is 
  but
  I just want to be sure.
  
  
 
 I will ask upstream to make sure. Will get back to you on this item as well.

dachary |  poker-interface is a shared library (really a python module but a
shared library)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2006-12-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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 19:32 EST ---
(In reply to comment #16)
 
 dachary |  poker-interface is a shared library (really a python module but a
 shared library)

Considering where it is placed on disk.. doesn't this mean you need to run
ldconfig in the postinstall scriptlet?


What is confusing is why its placed in /usr/lib/poker-network/poker-interface
instead of under /usr/lib/python2.4/site-packages/pokerclient2d ?

I don't think its really meant to be in the general ldconfig path. If its meant
as a library to be used from python bindings only, its best to place it into the
python module tree of interest. There are several examples of this sort of .so
inclusion in the python module directory already in Fedora space, python-numeric
being one specific case.

And shouldn't the filename end with the .so extention for clarity?


-jef

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 197754] Review Request: perl-Perl6-Bible

2006-12-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-Perl6-Bible


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 20:59 EST ---
Imported into CVS, branches created, and built.

Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209219] Review Request: libpreludedb-sqlite - Sqlite plugin for libprelude

2006-12-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: libpreludedb-sqlite - Sqlite plugin for libprelude


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209220] Review Request: libpreludedb-pgsql - pgsql plugin for libprelude

2006-12-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: libpreludedb-pgsql - pgsql plugin for libprelude


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Resolution|NOTABUG |DUPLICATE
OtherBugsDependingO|163776  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 23:13 EST ---


*** This bug has been marked as a duplicate of 209215 ***

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude

2006-12-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: libpreludedb - Provides an abstraction layer for the 
libprelude


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 23:13 EST ---
*** Bug 209220 has been marked as a duplicate of this bug. ***

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude

2006-12-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: libpreludedb - Provides an abstraction layer for the 
libprelude


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 23:14 EST ---
*** Bug 209219 has been marked as a duplicate of this bug. ***

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209219] Review Request: libpreludedb-sqlite - Sqlite plugin for libprelude

2006-12-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: libpreludedb-sqlite - Sqlite plugin for libprelude


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Resolution|NOTABUG |DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2006-12-31 23:14 EST ---


*** This bug has been marked as a duplicate of 209215 ***

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 218887] Review Request: fxload - Firmware loader

2006-12-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: fxload - Firmware loader


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221016] Review Request: Moodin - Splash engine for KDE

2006-12-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: Moodin - Splash engine for KDE


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 208113] Review Request: freepops - a tool to get html mail through a pop daemon

2006-12-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: freepops -  a tool to get html mail through a pop 
daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-01 02:32 EST ---
NOTE:

One month has passed since my comment #11.
Unless I can receive some response from Thibaud within one
week, I will close this bug as NOTABUG.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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