[Bug 475144] Review Request: metalink - CLI Metalink generation tool

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


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





--- Comment #3 from Ant Bryan anthonybr...@gmail.com  2008-12-13 03:08:07 EDT 
---
Thanks for the comments, Jason. I will work on these issues with upstream.

So, don't change the one thing rpmlint complains about?

And yes, I do need a sponsor.

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

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


[Bug 475934] Review Request: m6812-elf-binutils - Cross Compiling GNU binutils targeted at m68hc12

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


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





--- Comment #7 from Ralf Corsepius rc040...@freenet.de  2008-12-13 03:07:19 
EDT ---
 http://www.gnu-m68hc11.org/m68hc11_inst_ptc.php
/me thinks, this page is obsolete, for 2 reasons:

* the person listed as contact (Stephane C.), is the m68hc11/12 target's
maintainer of the FSF toolchain.
* Almost all packages referenced on this page are _way_ outdated.

May-be you should contact him before proceeding with 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.

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


[Bug 470325] Review Request: qd - Double-Double and Quad-Double Arithmetic

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


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


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

   Flag|needinfo?(jussi.leht...@iki |
   |.fi)|




--- Comment #2 from Jussi Lehtola jussi.leht...@iki.fi  2008-12-13 04:06:40 
EDT ---
(In reply to comment #1)
 I would posit that this shouldn't go in until we can either get a shared
 library out of this or at least get some kind of upstream confirmation that 
 you
 really can't build it that way.  You can ship the static librar(y|ies), if you
 like, in a separate -static package.  And the headers should go in a -devel
 package, as rpmlint dutifully complains.

I have contacted upstream about this. It seems that the support to build shared
libraries was added in version 2.1, but removed in 2.1.213 with libtool.

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

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


[Bug 466649] Review Request: adeona-0.2.1 - Stolen Laptop Tracker

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


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


Gianluca Sforna gia...@gmail.com changed:

   What|Removed |Added

 CC||gia...@gmail.com




--- Comment #2 from Gianluca Sforna gia...@gmail.com  2008-12-13 04:06:46 EDT 
---
Fedora is trying to consolidate all packages that are using crypto stuff to use
NSS. https://fedoraproject.org/wiki/FedoraCryptoConsolidation
Can you double check if the program can support an alternate crypto library? if
not, it would be nice if you could propose the change upstream (bonus points
for providing them a patch :))

Apart from this, the spec is clean, maybe a bit too much :) since I can't see
anything in the %files section...

it also fails to build in much with:

+ make install DESTDIR=/builddir/build/BUILDROOT/adeona-0.2.1-2.fc10.x86_64
/builddir/build/BUILD/adeona/scripts/installadeona.sh
ESC[HESC[2J
mkdir: 
cannot create directory `/usr/local/adeona'
: Permission denied
mkdir: 
cannot create directory `/usr/local/adeona/resources'
: No such file or directory
mkdir: cannot create directory `/usr/local/adeona/logs': No such file or
directory
mkdir: cannot create directory `/usr/local/adeona/scripts': No such file or
directory
Could not create installation directory. Please check your permissions.

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

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


[Bug 466649] Review Request: adeona-0.2.1 - Stolen Laptop Tracker

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


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





--- Comment #3 from Gianluca Sforna gia...@gmail.com  2008-12-13 04:07:24 EDT 
---
fails to build in _mock_...

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

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


[Bug 470325] Review Request: qd - Double-Double and Quad-Double Arithmetic

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


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





--- Comment #3 from Conrad Meyer kon...@tylerc.org  2008-12-13 04:13:18 EDT 
---
The lack of a shared library isn't the blocker (though it would be nice). The
issue is, as tibbs says: You can ship the static librar(y|ies), if you
like, in a separate -static package.  And the headers should go in a -devel
package, as rpmlint dutifully complains.

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

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


[Bug 470325] Review Request: qd - Double-Double and Quad-Double Arithmetic

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


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





--- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi  2008-12-13 04:28:09 
EDT ---
(In reply to comment #3)
 The lack of a shared library isn't the blocker (though it would be nice). The
 issue is, as tibbs says: You can ship the static librar(y|ies), if you
 like, in a separate -static package.  And the headers should go in a -devel
 package, as rpmlint dutifully complains.

Very well.

In accordance with the Packaging Guidelines, I packaged everything in qd-devel,
which provides qd-static.

rpmlint output:
qd-devel-debuginfo.x86_64: E: empty-debuginfo-package
2 packages and 1 specfiles checked; 1 errors, 0 warnings.


Spec URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-devel.spec
SRPM URL:
http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-devel-2.3.7-2.fc10.src.rpm

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

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


[Bug 470325] Review Request: qd - Double-Double and Quad-Double Arithmetic

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


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





--- Comment #5 from Conrad Meyer kon...@tylerc.org  2008-12-13 04:52:09 EDT 
---
(In reply to comment #4)
 rpmlint output:
 qd-devel-debuginfo.x86_64: E: empty-debuginfo-package
 2 packages and 1 specfiles checked; 1 errors, 0 warnings.

Add a line to the top of your spec:

  %define debug_package %{nil}

(Since rpmbuild doesn't get useful debuginfo from static libs.)

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

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


[Bug 470325] Review Request: qd - Double-Double and Quad-Double Arithmetic

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


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





--- Comment #6 from Jussi Lehtola jussi.leht...@iki.fi  2008-12-13 05:00:43 
EDT ---
Done.

Spec URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-devel.spec
SRPM URL:
http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-devel-2.3.7-3.fc10.src.rpm

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

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


[Bug 476346] New: Review Request: python-polybori - Framework for Boolean Rings

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

Summary: Review Request: python-polybori - Framework for Boolean Rings

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

   Summary: Review Request: python-polybori - Framework for
Boolean Rings
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/python-polybori.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-polybori-0.5-1.fc9.src.rpm
Description:
PolyBoRi is a special purpose computer algebra system for computations in
Boolean Rings. The core is a C++ library, which provides high-level data
types for Boolean polynomials and related structures. As a unique
approach, binary decision diagrams are used as internal storage type for
polynomial structures. On top of this, we provide a Python interface for
parsing of complex polynomial systems, as well as for sophisticated and
extendable strategies for Groebner base computation.

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

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


[Bug 476346] Review Request: python-polybori - Framework for Boolean Rings

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


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





--- Comment #1 from Conrad Meyer kon...@tylerc.org  2008-12-13 05:48:57 EDT 
---
Koji scratch build:

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

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

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


[Bug 476329] Review Request: cudd - CU Decision Diagram Package

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


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


Conrad Meyer kon...@tylerc.org changed:

   What|Removed |Added

 Blocks||476346




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

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


[Bug 476346] Review Request: python-polybori - Framework for Boolean Rings

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


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


Conrad Meyer kon...@tylerc.org changed:

   What|Removed |Added

 Depends on||476329




--- Comment #2 from Conrad Meyer kon...@tylerc.org  2008-12-13 05:50:31 EDT 
---
Obviously (from scratch build) this depends on cudd. Adding blocker 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.

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


[Bug 476324] Review Request: perl-URI-Find - Find URIs in plain text

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


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


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

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review+




--- Comment #1 from Parag AN(पराग) panem...@gmail.com  2008-12-13 05:54:42 
EDT ---
Review:
+ package builds in mock (rawhide i386).
koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=996047
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
339279e268cf37e629f54c1091f99a13  URI-Find-0.16.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test outputs
All tests successful.
Files=2, Tests=382,  0 wallclock secs ( 0.08 usr  0.01 sys +  0.09 cusr  0.04
csys =  0.22 CPU)
+ Package perl-URI-Find-0.16-1.fc11 =
  Provides: perl(URI::Find) = 0.16 perl(URI::Find::Schemeless) = 1.8
  Requires: perl = 0:5.005 perl(Carp) perl(URI) perl(URI::Find) perl(URI::URL)
perl(base) perl(constant) perl(strict) perl(vars)

APPROVED.

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

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


[Bug 474818] Review Request: perl-Class-XSAccessor - Generate fast XS accessors without runtime compilation

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


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


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

   What|Removed |Added

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




--- Comment #2 from Parag AN(पराग) panem...@gmail.com  2008-12-13 05:56:46 
EDT ---
build failed because of missing BR perl(Test::More)
Can't locate Test/More.pm

See http://koji.fedoraproject.org/koji/taskinfo?taskID=996515

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

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


[Bug 435276] Review Request: kBuild - A cross-platform build enviroment

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


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


Xavier Lamien lxt...@gmail.com changed:

   What|Removed |Added

 CC||lxt...@gmail.com




--- Comment #24 from Xavier Lamien lxt...@gmail.com  2008-12-13 06:01:52 EDT 
---
ping ?

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

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


[Bug 476350] New: Review Request: python-networkx - Creates and Manipulates Graphs and Networks

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

Summary: Review Request: python-networkx - Creates and Manipulates Graphs and 
Networks

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

   Summary: Review Request: python-networkx - Creates and
Manipulates Graphs and Networks
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/python-networkx.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-networkx-0.99-1.fc9.src.rpm
Description:
NetworkX is a Python package for the creation, manipulation, and
study of the structure, dynamics, and functions of complex networks.

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

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


[Bug 476350] Review Request: python-networkx - Creates and Manipulates Graphs and Networks

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


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





--- Comment #1 from Conrad Meyer kon...@tylerc.org  2008-12-13 07:29:20 EDT 
---
Koji scratch build:

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

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

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


[Bug 476351] New: Review Request: palp - A Package for Analyzing Lattice Polytopes

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

Summary: Review Request: palp - A Package for Analyzing Lattice Polytopes

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

   Summary: Review Request: palp - A Package for Analyzing Lattice
Polytopes
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/palp.spec
SRPM URL: http://konradm.fedorapeople.org/fedora/SRPMS/palp-1.1-1.fc9.src.rpm
Description:
PALP contains routines for vertex and facet enumeration, computation of
incidences and symmetries, as well as completion of the set of lattice
points in the convex hull of a given set of points. In addition, there
are procedures specialised to reflexive polytopes such as the enumeration
of reflexive subpolytopes, and applications to toric geometry and string
theory, like the computation of Hodge data and fibration structures for
toric Calabi-Yau varieties.

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

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


[Bug 476351] Review Request: palp - A Package for Analyzing Lattice Polytopes

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


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





--- Comment #1 from Conrad Meyer kon...@tylerc.org  2008-12-13 07:50:49 EDT 
---
Koji scratch build:

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

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

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


[Bug 470737] Review Request: pen - Load balancer for simple tcp based protocols such as http or smtp

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


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





--- Comment #3 from Fabian Affolter fab...@bernewireless.net  2008-12-13 
08:59:08 EDT ---
Package Review

==



Key:

 - = N/A

 x = Check

 ! = Problem

 ? = Not evaluated



=== REQUIRED ITEMS ===

 [x] Package is named according to the Package Naming Guidelines.

 [x] Spec file name must match the base package %{name}, in the format
%{name}.spec.

 [x] Package meets the Packaging Guidelines

 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.

 Tested on: F9/i386 
 [x] Rpmlint output:

 Source RPM:


 [...@laptop024 SRPMS]$ rpmlint pen-0.18.0-2.fc10.src.rpm 
 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
 Binary RPM(s):


 [...@laptop024 i386]$ rpmlint pen*
 2 packages and 0 specfiles checked; 0 errors, 0 warnings.
 [x] Package is not relocatable.

 [x] Buildroot is correct

 master   : %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

 spec file: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

 [X] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.

 [x] License field in the package spec file matches the actual license.

 License type: GPLv2

 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.



 [x] Spec file is legible and written in American English.

 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.

 Upstream source: 
96f6d39e7e7cca11a647e795550f3829  pen-0.18.0.tar.gz
 Build source:
96f6d39e7e7cca11a647e795550f3829  pen-0.18.0.tar.gz
 [x] Package is not known to require ExcludeArch.

 [x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.

 [-] The spec file handles locales properly.  %find_lang used for locales.

 [x] %{optflags} or RPM_OPT_FLAGS are honoured.

 [-] ldconfig called in %post and %postun if required.

 [x] %install starts with rm -rf %{buildroot} or $RPM_BUILD_ROOT.

 [x] Package must own all directories that it creates.

 [x] Package requires other packages for directories it uses.

 [x] Package does not own files or directories owned by other packages.

 [x] Package does not contain duplicates in %files.

 [x] Permissions on files are set properly. %defattr(-,root,root,-) is in every
%files section.

 [x] Package uses nothing in %doc for runtime.

 [x] Package has a %clean section, which contains rm -rf %{buildroot} or
$RPM_BUILD_ROOT.

 [x] Package consistently uses macros.

 [x] Package contains code, or permissable content.



 [-] Large documentation files are in a -doc subpackage, if required.

 [-] Header files in -devel subpackage, if present.

 [-] Fully versioned dependency in subpackage, if present.

 [-] Static libraries in -devel subpackage, if present.

 [-] Package requires pkgconfig, if .pc files are present.

 [-] Development .so files in -devel subpackage, if present.

 [-] Package does not contain any libtool archives (.la).

 [x] -debuginfo subpackage is present and looks complete.

 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.



=== SUGGESTED ITEMS ===

 [x] Timestamps preserved with cp and install.

 [x] Uses parallel make (%{?_smp_mflags})

 [x] Latest version is packaged.

 [x] Package does not include license text files separate from upstream.

 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.

 [x] Reviewer should test that the package builds in mock.

 Tested on: F9/i386

 [x] Package should compile and build into binary RPMs on all supported
architectures.

 Tested on: 

   [x] F10/i386 
http://koji.fedoraproject.org/koji/taskinfo?taskID=996602
 [-] Package functions as described.

 [!] Scriptlets must be sane, if used.

 [-] The placement of pkgconfig(.pc) files is correct.

 [-] File based requires are sane.

 [x] Changelog in allowed format



Some remarks:

- Don't mix $RPM_BUILD_ROOT and %{buildroot}
 
https://fedoraproject.org/wiki/Packaging/Guidelines#Using_.25.7Bbuildroot.7D_and_.25.7Boptflags.7D_vs_.24RPM_BUILD_ROOT_and_.24RPM_OPT_FLAGS

- You should report upstream about the missing pen.conf in the release tarball.

- What's about a restart of apache after the installation?
  https://fedoraproject.org/wiki/Packaging/SysVInitScript

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


[Bug 476357] New: Review Request: libicns - Library for manipulating Macintosh icns files

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

Summary: Review Request: libicns - Library for manipulating Macintosh icns files

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

   Summary: Review Request: libicns - Library for manipulating
Macintosh icns files
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: musur...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.webalice.it/musuruan/RPMS/reviews/libicns.spec
SRPM URL:
http://www.webalice.it/musuruan/RPMS/reviews/libicns-0.6.0-1.fc10.src.rpm
Description: libicns is a library providing functionality for easily reading
and 
writing Macintosh icns files

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

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


[Bug 474768] Review Request: jpilot-backup - An enhanced backup plugin for J-Pilot

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


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





--- Comment #5 from Thorsten Leemhuis fed...@leemhuis.info  2008-12-13 
09:27:31 EDT ---
(In reply to comment #4)

*
 - summary should not start with A or An
 fixed

Really? I still see Summary: A enhanced backup plugin for J-Pilot ;-)

*
 - Remove the *.la files.  They should not be included.
 as you can see in the -4 spec, I manually remove it but have provisions in
 place to build a -static subpackage. I think just removing is better in this
 case but would like comments.

Removing definitely is better. Static subpackages are hightly discouraged in
Fedora; see packaging guidelines for details.


(quoting from the spec or rpmlint)

* 
 License: GPLv2
Looks more like GPLv2+ to me (or did I miss something?). 

*
 Requires: gdbm
Why is that needed? RPM automatically adds a requires for libgdbm.so.2(),
hence please remove it if there there are no good reason for it.


*
$ rpmlint rpmbuild/RPMS/x86_64/jpilot-backup-* 
jpilot-backup-0.53-4.fc10.src.rpm 
jpilot-backup.src: W: strange-permission jpilot-backup.spec 0600
3 packages and 0 specfiles checked; 0 errors, 1 warnings.

Please fix

* 
looks good apart from the above

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

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


[Bug 476359] Review Request: compiz-fusion-unsupported - Additional Compiz Fusion plugins for Compiz

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


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





--- Comment #1 from leigh scott leigh123li...@googlemail.com  2008-12-13 
09:35:39 EDT ---
I also need a sponsor .

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

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


[Bug 476359] New: Review Request: compiz-fusion-unsupported - Additional Compiz Fusion plugins for Compiz

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

Summary: Review Request: compiz-fusion-unsupported - Additional Compiz Fusion 
plugins for Compiz

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

   Summary: Review Request: compiz-fusion-unsupported - Additional
Compiz Fusion plugins for Compiz
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: leigh123li...@googlemail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://www.linux-ati-drivers.homecall.co.uk/compiz-fusion-unsupported.spec
SRPM URL:
http://www.linux-ati-drivers.homecall.co.uk/compiz-fusion-unsupported-0.7.8-1.fc10.src.rpm
Description: This package contains additional plugins from the Compiz Fusion
Project

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

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


[Bug 476360] New: Review Request: rubygem-tlsmail - This library enables pop or smtp via ssl/tls

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

Summary: Review Request: rubygem-tlsmail - This library enables pop or smtp via 
ssl/tls

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

   Summary: Review Request: rubygem-tlsmail - This library enables
pop or smtp via ssl/tls
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dpie...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://mcpierce.fedorapeople.org/rubygem-tlsmail.spec
SRPM URL: http://mcpierce.fedorapeople.org/rubygem-tlsmail-0.0.1-1.fc10.src.rpm
Description: This library enables pop or smtp via ssl/tls by dynamically
replacing these classes to these in ruby 1.9.

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

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


[Bug 475884] Review Request: gbdfed - Bitmap Font Editor

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


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





--- Comment #5 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:54:25 EDT ---
gbdfed-1.4-1.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 475884] Review Request: gbdfed - Bitmap Font Editor

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


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


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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 474908] Review Request: xmms2 - A modular audio framework and plugin architecture

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


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


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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


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

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


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





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:55:44 EDT ---
qle-0.0.10-5.fc8 has been pushed to the Fedora 8 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 467489] Review Request: nwsclient - NetWorkSpaces Client for Python

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


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





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:55:57 EDT ---
nwsclient-1.6.3-2.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


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

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


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





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:54:13 EDT ---
ibp-0.21-1.fc9 has been pushed to the Fedora 9 stable repository.  If problems
still persist, please make note of it in this bug report.

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

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


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

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


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





--- Comment #27 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:56:31 EDT ---
laf-plugin-1.0-4.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 474908] Review Request: xmms2 - A modular audio framework and plugin architecture

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


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





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:54:33 EDT ---
xmms2-0.5-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems
still persist, please make note of it in this bug report.

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

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


[Bug 467489] Review Request: nwsclient - NetWorkSpaces Client for Python

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


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


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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 473222] Review Request: perl-Catalyst-Plugin-Session-Store-File - File storage backend for session data

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


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





--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:55:19 EDT ---
perl-Catalyst-Plugin-Session-Store-File-0.13-1.fc9 has been pushed to the
Fedora 9 stable repository.  If problems still persist, please make note of it
in this bug report.

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

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


[Bug 475082] Review Request: guiloader - GuiXml Loader Library

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


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


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

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:55:11 EDT ---
guiloader-2.12.0-2.fc9 has been pushed to the Fedora 9 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing-newkey update guiloader'.  You can
provide feedback for this update here:
http://admin.fedoraproject.org/updates/F9/FEDORA-2008-11197

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

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


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

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


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





--- Comment #28 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:57:46 EDT ---
laf-plugin-1.0-4.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


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

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


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


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

   What|Removed |Added

 Resolution|RAWHIDE |NEXTRELEASE




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

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


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

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


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





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:58:18 EDT ---
ibp-0.21-1.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


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

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


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





--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:57:17 EDT ---
qle-0.0.10-5.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 472418] Review Request: xmbdfed - Bitmap Font Editor

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


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





--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 09:59:51 EDT ---
xmbdfed-4.7-3.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 472418] Review Request: xmbdfed - Bitmap Font Editor

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


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


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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


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

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


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





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:00:19 EDT ---
perl-Catalyst-Component-InstancePerContext-0.001001-1.fc9 has been pushed to
the Fedora 9 stable repository.  If problems still persist, please make note of
it in this bug report.

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

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


[Bug 472418] Review Request: xmbdfed - Bitmap Font Editor

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


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





--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:00:01 EDT ---
xmbdfed-4.7-3.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 467489] Review Request: nwsclient - NetWorkSpaces Client for Python

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


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





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:05:24 EDT ---
nwsclient-1.6.3-2.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 474908] Review Request: xmms2 - A modular audio framework and plugin architecture

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


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





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:05:01 EDT ---
xmms2-0.5-2.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 473218] Review Request: perl-Catalyst-Log-Log4perl - Log::Log4perl logging for Catalyst

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


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





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:02:20 EDT ---
perl-Catalyst-Log-Log4perl-1.00-1.fc9 has been pushed to the Fedora 9 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


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

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


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





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:08:17 EDT ---
ibp-0.21-2.fc8 has been pushed to the Fedora 8 stable repository.  If problems
still persist, please make note of it in this bug report.

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

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


[Bug 475884] Review Request: gbdfed - Bitmap Font Editor

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


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





--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:08:42 EDT ---
gbdfed-1.4-1.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


[Bug 464050] Package Review Request for projectM-libvisual

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


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





--- Comment #16 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:06:04 EDT ---
projectM-libvisual-1.2.0-4.fc10 has been pushed to the Fedora 10 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 473218] Review Request: perl-Catalyst-Log-Log4perl - Log::Log4perl logging for Catalyst

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


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





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:04:26 EDT ---
perl-Catalyst-Log-Log4perl-1.00-1.fc10 has been pushed to the Fedora 10 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


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

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


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





--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:03:29 EDT ---
qle-0.0.10-5.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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


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

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


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





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:02:04 EDT ---
perl-Catalyst-Component-InstancePerContext-0.001001-1.fc10 has been pushed to
the Fedora 10 stable repository.  If problems still persist, please make note
of it in this bug report.

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

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


[Bug 473222] Review Request: perl-Catalyst-Plugin-Session-Store-File - File storage backend for session data

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


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





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2008-12-13 10:02:28 EDT ---
perl-Catalyst-Plugin-Session-Store-File-0.13-1.fc10 has been pushed to the
Fedora 10 stable repository.  If problems still persist, please make note of it
in this bug report.

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

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


[Bug 466649] Review Request: adeona-0.2.1 - Stolen Laptop Tracker

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


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


David Nalley da...@gnsa.us changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG




--- Comment #4 from David Nalley da...@gnsa.us  2008-12-13 11:22:06 EDT ---
Gianluca - 

It appears that upstream has abandoned this project as OpenDHT has essentially
been abandoned as well. They claim that they will start a new project to work
around OpenDHT but they haven't released any code for this project. I should
have closed this ticket a while ago. 
I apologize, I should have been more proactive in closing this ticket. 
Thanks

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

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


[Bug 475934] Review Request: m6812-elf-binutils - Cross Compiling GNU binutils targeted at m68hc12

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


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





--- Comment #8 from Brennan Ashton bash...@brennanashton.com  2008-12-13 
11:38:23 EDT ---
Yes, I have seen people using the 4.3 gcc, so I am working on building that
right now.  I will do some tests on actual hardware to verify.  That other
project has gone though states of being maintained and not it looks like, and
many of the important links are broken.

Right now I am having issues with getting gcc to compile, there has been some
change in the Fedora gcc, and it has broken even the good avr-gcc package
building.

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

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


[Bug 461912] Review Request: puzzles - A collection of one-player puzzle games

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


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


Sergio Pascual sergio.pa...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #13 from Sergio Pascual sergio.pa...@gmail.com  2008-12-13 
11:46:51 EDT ---
Sorry for the delay, somehow I missed your updates.

This package is APPROVED

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

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


[Bug 472229] Review Request: PyQwt - Python bindings for Qwt

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 CC||mtas...@ioa.s.u-tokyo.ac.jp




--- Comment #2 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2008-12-13 
12:21:48 EDT ---
Some notes:

* BuildRequires/Requires 
  - build.log shows

80  Failed to import numarray: PyQwt will be build without support for
numarray.
81  Failed to find Numeric2: PyQwt will be build without support for
Numeric.
.
86   'disable_numarray': False,
87   'disable_numeric': False,
88   'disable_numpy': False,
 
These message seems contradictory.
Perhaps BuildRequires: python-numarray python-numeric
is needed.

  - Also please check if numpy (or python-numarray, python-numeric)
are also needed for Requires (not BuildRequires). 
For python module related packages, writing a package in 
BuildRequires does not mean that the package is installed
at runtime.

  - And check if PyQwt-devel does not require qwt-devel.
For example, QwtList.sip contains:
-
34  %MappedType QListQwtPickerMachine::Command
35  {
36  %TypeHeaderCode
37  #include qlist.h
38  #include qwt_picker_machine.h
39  %End
-

* Timestamps
  - 'INSTALL=install -p' argument to make install does not
make sense for this package because this makefile is not
based on autotools but based on python
(actually when installing files cp -f is used)

* Permission of scripts
-
# non-executable script
chmod 755 %{buildroot}%{python_sitearch}/PyQt4/Qwt5/grace.py
-
  - If this script is not meant to be called directly by
users (but is meant to be called internally from programs
or so), then this script should not have executable
permission and the shebang should be removed.

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

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


[Bug 476359] Review Request: compiz-fusion-unsupported - Additional Compiz Fusion plugins for Compiz

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


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


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

 Blocks||177841




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

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


[Bug 474984] Review Request: bandwidthd - Tracks network usage and builds html and graphs

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


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


Jan ONDREJ ondr...@salstar.sk changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #14 from Jan ONDREJ ondr...@salstar.sk  2008-12-13 12:48:57 EDT 
---
Thanks to you. :)

New Package CVS Request
===
Package Name: bandwidthd
Short Description: Tracks network usage and builds html and graphs
Owners: ondrejj
Branches: F-9 F-10 EL-4 EL-5
InitialCC:

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

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


[Bug 474981] Review Request: jcalendar - A Java date chooser bean for graphically picking a date

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 CC||mtas...@ioa.s.u-tokyo.ac.jp




--- Comment #2 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2008-12-13 
13:28:39 EDT ---
Some notes

* Versioning
  - As this does not seem to be a pre-release of version
1.3.2, you should not use 0. for release number.

* License
  - For this package the license tag must be LGPLv2+

* BuildRequires/Requires
  - Please follow
https://fedoraproject.org/wiki/Packaging/Java#BuildRequires_and_Requires

! Timestamp
  - Preserving the original timestamp on jcalendar-license.txt
is preferred.

* Directory ownership issue
  - This package should not own %_javadir.

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

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


[Bug 465897] Review Request: Judy - General purpose dynamic array library

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


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


Charles R. Anderson c...@wpi.edu changed:

   What|Removed |Added

   Flag|needinfo?(c...@wpi.edu)  |




--- Comment #14 from Charles R. Anderson c...@wpi.edu  2008-12-13 13:35:56 
EDT ---
Spec URL: http://cra.fedorapeople.org/Judy/Judy.spec
SRPM URL: http://cra.fedorapeople.org/Judy/Judy-1.0.4-4.fc10.src.rpm
Scratch Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=996794

* Sat Dec 13 2008 Charles R. Anderson c...@wpi.edu 1.0.4-4
- for Judy1 man page fix, patch Makefile.{am,in} instead of
  relying on autotools to regenerate the latter.
- Add README.Fedora with upstream's license explanation.

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

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


[Bug 474981] Review Request: jcalendar - A Java date chooser bean for graphically picking a date

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


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





--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2008-12-13 
13:45:39 EDT ---


(In reply to comment #2)
 ! Timestamp
   - Preserving the original timestamp on jcalendar-license.txt
 is preferred.

like...

sed -i -e 's|\r||' jcalendar-license.txt  tmp
touch -r jcalendar-license.txt tmp
mv -f tmp jcalendar-license.txt

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

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


[Bug 225798] Merge Review: gimp-help

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


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


Orcan 'oget' Ogetbil orcanba...@yahoo.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #7 from Orcan 'oget' Ogetbil orcanba...@yahoo.com  2008-12-13 
14:11:01 EDT ---
Package is in rawhide. We can close the bug now.

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

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


[Bug 457210] Review Request: dpkg - Package maintenance system for Debian Linux

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


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


Orcan 'oget' Ogetbil orcanba...@yahoo.com changed:

   What|Removed |Added

 CC||orcanba...@yahoo.com




--- Comment #23 from Orcan 'oget' Ogetbil orcanba...@yahoo.com  2008-12-13 
14:08:18 EDT ---
re-ping?

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

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


[Bug 476247] Review Request: log4cpp - logging library for c++

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #7 from Jesse Keating jkeat...@redhat.com  2008-12-13 14:39:08 
EDT ---
All right, looks good now.  Approving.

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

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


[Bug 476374] New: Review Request: OASA - python library for manipulation of chemical formats

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

Summary: Review Request: OASA - python library for manipulation of chemical 
formats

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

   Summary: Review Request: OASA - python library for manipulation
of chemical formats
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: henrique...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://lspooky.fedorapeople.org/oasa/0.12.5/oasa.spec
SRPM URL:
http://lspooky.fedorapeople.org/oasa/0.12.5/oasa-0.12.5-1.fc10.src.rpm
Description: OASA is a python library that handles chemical formats. OASA is,
also, a dependency for the BKChem package (Bug 452413) that is, too, in
approval process.

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

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


[Bug 457210] Review Request: dpkg - Package maintenance system for Debian Linux

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


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


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag|needinfo?(leigh123li...@goo |
   |glemail.com)|




--- Comment #24 from leigh scott leigh123li...@googlemail.com  2008-12-13 
14:42:23 EDT ---
I haven't had time to do any work on dpkg lately , I hope to get some free time
in the new year .

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

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


[Bug 452413] Review Request: BkChem - Chemical drawing program

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


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





--- Comment #21 from Henrique LonelySpooky Junior henrique...@gmail.com  
2008-12-13 14:50:59 EDT ---
Here we go.
BKChem worked perfectly with python-pmw from fedora. OASA is now on a separate
package awaiting to be reviewed (Bug 476374) and all the tests with this new
release of BKChem (using pithon-pmw and the new package of OASA) are going OK.

SPEC: http://lspooky.fedorapeople.org/bkchem/0.12.5/bkchem.spec
SRPM:
http://lspooky.fedorapeople.org/bkchem/0.12.5/bkchem-0.12.5-1.fc10.src.rpm

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

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


[Bug 476324] Review Request: perl-URI-Find - Find URIs in plain text

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


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


Chris Weyl cw...@alumni.drew.edu changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 476351] Review Request: palp - A Package for Analyzing Lattice Polytopes

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jkeat...@redhat.com
 AssignedTo|nob...@fedoraproject.org|jkeat...@redhat.com
   Flag||fedora-review?,
   ||needinfo?(kon...@tylerc.org
   ||)




--- Comment #2 from Jesse Keating jkeat...@redhat.com  2008-12-13 15:03:32 
EDT ---
* Name fine
* Summary/description OK
* Group OK
* Source matches upstream
* Buildroot sane
* Make uses smp flags
* Install has clean
* Consistent use of macros
* %clean section OK
* defatter exists
* Changelog sane
* Build in mock for rawhide
* rpmlint is silent
* License is not OK
  Source refers to http://www.gnu.org/copyleft/gpl.html as it's license, which
is now version 3, so I do believe the license needs to be GPLv3+

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

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


[Bug 476324] Review Request: perl-URI-Find - Find URIs in plain text

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


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





--- Comment #2 from Chris Weyl cw...@alumni.drew.edu  2008-12-13 15:02:22 EDT 
---
New Package CVS Request
===
Package Name: perl-URI-Find
Short Description: Find URIs in plain text
Owners: cweyl
Branches: F-9 F-10 devel
InitialCC:

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

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


[Bug 476351] Review Request: palp - A Package for Analyzing Lattice Polytopes

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


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


Conrad Meyer kon...@tylerc.org changed:

   What|Removed |Added

   Flag|needinfo?(kon...@tylerc.org |
   |)   |




--- Comment #3 from Conrad Meyer kon...@tylerc.org  2008-12-13 15:17:34 EDT 
---
New URLs:

http://konradm.fedorapeople.org/fedora/SPECS/palp.spec
http://konradm.fedorapeople.org/fedora/SRPMS/palp-1.1-2.fc9.src.rpm

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

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


[Bug 476351] Review Request: palp - A Package for Analyzing Lattice Polytopes

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #4 from Jesse Keating jkeat...@redhat.com  2008-12-13 16:03:05 
EDT ---
Looks good to me!

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

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


[Bug 476351] Review Request: palp - A Package for Analyzing Lattice Polytopes

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


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


Conrad Meyer kon...@tylerc.org changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #5 from Conrad Meyer kon...@tylerc.org  2008-12-13 16:10:49 EDT 
---
Thanks!

New Package CVS Request
===
Package Name: palp
Short Description: A Package for Analyzing Lattice Polytopes
Owners: konradm
Branches: F-10 F-9
InitialCC:

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

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


[Bug 476374] Review Request: OASA - python library for manipulation of chemical formats

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

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




--- Comment #1 from Jesse Keating jkeat...@redhat.com  2008-12-13 16:08:01 
EDT ---
Taking review.

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

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


[Bug 476374] Review Request: OASA - python library for manipulation of chemical formats

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(henrique...@gmail
   ||.com)




--- Comment #2 from Jesse Keating jkeat...@redhat.com  2008-12-13 16:20:42 
EDT ---
* Python defines correct
* Name not correct.  This is a python module, and should be named python-oasa. 
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Addon_Packages_.28python_modules.29
* Licensing not correct.  Header files have 'or any later version' in them,
therefore license should be GPLV2+
  oasa/elements.py - No license header
  oasa/subsearch_data.py - No license header
  oasa/tests.py - No license header
  oasa/graph/multi_thread.py - No license header
* Summary could drop the OASA is a
* URL is OK
* Source matches upstream
* Buildroot fine
* Requires on python is redundant, don't put that in.
* Description is OKish.
* Consistent macro usage
* Correct cleaning at %install
* Good %clean section
* defatters in %files
* %files is a bit broad.  I'd rather see something more like
http://fedoraproject.org/wiki/Packaging/Python#Byte_Compiled_Files
* changelog should have a revision listed.
* rpmlint output:
  oasa.noarch: W: no-documentation
  oasa.noarch: E: description-line-too-long OASA is a free python library for  
manipulating and analyzing chemical structures.
  oasa.noarch: W: incoherent-version-in-changelog 0.12.5 ['0.12.5-1.fc11',
'0.12.5-1']
  oasa.src: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line 3)

** SUMMARY **
Fix license, report unlicensed files to upstream
Fix name
Fix summary
Fix Requires
Potentially fix %files
Fix rpmlint output

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

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


[Bug 475098] Review Request: python-Traits - Explicitly typed attributes for Python

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

 CC||jkeat...@redhat.com
 AssignedTo|nob...@fedoraproject.org|jkeat...@redhat.com
   Flag||fedora-review?




--- Comment #1 from Jesse Keating jkeat...@redhat.com  2008-12-13 16:28:01 
EDT ---
Taking review.

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

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


[Bug 461619] Review Request: system-switch-displaymanager - A display manager switcher for GDM, KDM, XDM and WDM

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


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


Ngo Than t...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG




--- Comment #13 from Ngo Than t...@redhat.com  2008-12-13 16:36:25 EDT ---
it's build in f9/f10

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

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


[Bug 475098] Review Request: python-Traits - Explicitly typed attributes for Python

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||needinfo?(rakesh.pan...@gma
   ||il.com)




--- Comment #2 from Jesse Keating jkeat...@redhat.com  2008-12-13 16:49:53 
EDT ---
* python define correct
* Name/Summary/Group/URL all correct
* Buildroot def is fine
* Builds Ok in mock
* License is wrong, there are multiple licenses in this package, due to various
image files.  BSD 3-clause, Eclipse Public License, LGPL, GPLv2
  Many files in enthought/traits/protocols/ have no license headers, and are in
fact from a different project, PyProtocols
  Many files in enthought/traits/tests/ have no license headers
  enthought/traits/ui/editors_gen.py is licensed under GPLv2(thus +)
  MANY things in examples/ have no license information
  Most things in integrationtests/ have no license information
* buildroot is sane
* %prep,%build,%files,%clean all seem fine
* %files OK
* changelog OK

* rpmlint output:
python-Traits.i386: E: non-standard-executable-perm
/usr/lib/python2.6/site-packages/enthought/traits/ctraits.so 0775
python-Traits.i386: E: non-standard-executable-perm
/usr/lib/python2.6/site-packages/enthought/traits/protocols/_speedups.so 0775

** SUMMARY **
This needs a close examination of the licenses of the files and a correct
License field.
Permissions need to be fixed.

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

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


[Bug 468226] Review Request: python-tw-jquery - ToscaWidgets wrapping for jQuery

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

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




--- Comment #1 from Jesse Keating jkeat...@redhat.com  2008-12-13 16:59:48 
EDT ---
Taking review.

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

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


[Bug 468189] Review Request: rear - Relax and Recovery (disaster recovery framework)

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


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





--- Comment #13 from Fabian Affolter fab...@bernewireless.net  2008-12-13 
17:00:28 EDT ---
(In reply to comment #12)
 1/ during the rpmbuild the warnings about file listed twice

http://rpm.org/max-rpm-snapshot/s1-rpm-inside-files-list-directives.html

 2/ rear.spec not in correct permission mode

Use chmod to switch the permission before you use rpmbuild locally, then the
warning will go away. 

One remark...

- leave contrib in %doc away.  On a RPM based system it's not interesting how
to build stuff for a gentoo system.

Did you find a sponsor in the meantime?

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

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


[Bug 468226] Review Request: python-tw-jquery - ToscaWidgets wrapping for jQuery

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


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


Jesse Keating jkeat...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(lmac...@redhat.co
   ||m)




--- Comment #2 from Jesse Keating jkeat...@redhat.com  2008-12-13 17:07:46 
EDT ---
* Python define good
* Source0 is a 404
* Commented out %doc isn't likely commented out.  You need %%doc
* MANY files lack license header info
* PKG-INFO lists the license as UNKNOWN, there is no fallback for unlicensed
files.
* Some files are licensed under MIT /and/ GPL

* rpmlint output:
python-tw-jquery.noarch: W: no-documentation
python-tw-jquery.noarch: W: hidden-file-or-dir
/usr/lib/python2.6/site-packages/tw/jquery/static/css/flexigrid/images/._fhbg.gif
python-tw-jquery.noarch: W: hidden-file-or-dir
/usr/lib/python2.6/site-packages/tw/jquery/static/css/flexigrid/._ie.css
python-tw-jquery.noarch: W: hidden-file-or-dir
/usr/lib/python2.6/site-packages/tw/jquery/static/css/flexigrid/._images
python-tw-jquery.noarch: W: hidden-file-or-dir
/usr/lib/python2.6/site-packages/tw/jquery/static/css/flexigrid/images/._wbg.gif
python-tw-jquery.noarch: E: htaccess-file
/usr/lib/python2.6/site-packages/tw/jquery/static/javascript/validate/demo/captcha/images/.htaccess
python-tw-jquery.noarch: W: hidden-file-or-dir
/usr/lib/python2.6/site-packages/tw/jquery/static/css/flexigrid/._flexigrid.css
2 packages and 0 specfiles checked; 1 errors, 6 warnings.

** SUMMARY **
Needs a license audit.
Needs an updates Source
rpmlint issues need accounting for or fixing.

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

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


[Bug 459800] Review Request: python-py - Innovative python library containing py.test, greenlets and other niceties

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


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





--- Comment #9 from Thomas Moschny thomas.mosc...@gmx.de  2008-12-13 17:24:00 
EDT ---
Spec URL: http://thm.fedorapeople.org/python-py.spec
SRPM URL: http://thm.fedorapeople.org/python-py-0.9.2-5.fc10.src.rpm

%changelog
* Fri Dec 12 2008 Thomas Moschny .. - 0.9.2-5
- Add patch from trunk fixing a subversion 1.5 problem (pylib
  issue66).
- Don't replace doctest compat module (pylib issue67).

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

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


[Bug 476385] New: Review Request: pycrypto - The Python Cryptography Toolkit

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

Summary: Review Request: pycrypto - The Python Cryptography Toolkit

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

   Summary: Review Request: pycrypto - The Python Cryptography
Toolkit
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/pycrypto.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/pycrypto-2.0.1-1.20081213git510428c.fc9.src.rpm
Description:
This is a collection of both secure hash functions (such as MD5 and SHA),
and various encryption algorithms (AES, DES, IDEA, RSA, ElGamal, etc.). The
package is structured to make adding new modules easy.

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

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

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


[Bug 476386] New: Review Request: perl-verilog - Verilog parsing routines

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

Summary: Review Request: perl-verilog  - Verilog parsing routines

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

   Summary: Review Request: perl-verilog  - Verilog parsing
routines
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cgoo...@yahoo.com.au
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-verilog.spec
SRPM URL:
http://chitlesh.fedorapeople.org/RPMS/perl-verilog-3.044-1.fc10.src.rpm
Description:
This package provides functions to support writing utilities
that use the Verilog language.

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

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


[Bug 476374] Review Request: OASA - python library for manipulation of chemical formats

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


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


Henrique LonelySpooky Junior henrique...@gmail.com changed:

   What|Removed |Added

   Flag|needinfo?(henrique...@gmail |
   |.com)   |




--- Comment #3 from Henrique LonelySpooky Junior henrique...@gmail.com  
2008-12-13 18:08:42 EDT ---
Hello, Jesse, thanks for reviewing the package.

* Fix license, report unlicensed files to upstream - DONE and I already warned
Beda about the files without license.
* Fix name - DONE
* Fix summary - DONE
* Fix Requires - DONE
* Potentially fix %files - I still have little experience in packaging, I hope
I have done correctly this time.
* Fix rpmlint output - rpmlint is no longer showing any errors.

SPEC - http://lspooky.fedorapeople.org/oasa/0.12.5/python-oasa.spec
SRPM -
http://lspooky.fedorapeople.org/oasa/0.12.5/python-oasa-0.12.5-1.fc10.src.rpm

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

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


[Bug 476387] New: Review Request: vhd2vl - VHDL to Verilog translator

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

Summary: Review Request: vhd2vl -  VHDL to Verilog translator

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

   Summary: Review Request: vhd2vl -  VHDL to Verilog translator
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cgoo...@yahoo.com.au
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://chitlesh.fedorapeople.org/RPMS/vhd2vl.spec
SRPM URL: http://chitlesh.fedorapeople.org/RPMS/vhd2vl-2.0-1.fc10.src.rpm
Description:
vhd2vl is a VHDL to Verilog translation program.
It targets the translation of synthetisable RTL.
While far from complete it supports a useful
subset of VHDL, sufficient for complex designs.

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

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


[Bug 475132] Review Request: usbmon - Front-end for in-kernel usbmon

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


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


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|ke...@tummy.com
   Flag||fedora-review?




--- Comment #5 from Kevin Fenzi ke...@tummy.com  2008-12-13 18:53:06 EDT ---
I'd be happy to review this and sponsor you... look for a full review in a bit.

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

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


[Bug 475132] Review Request: usbmon - Front-end for in-kernel usbmon

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


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


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

 Blocks|177841  |
   Flag|fedora-review?  |fedora-review+




--- Comment #6 from Kevin Fenzi ke...@tummy.com  2008-12-13 19:13:25 EDT ---
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name. 
OK - Spec has consistant macro usage. 
OK - Meets Packaging Guidelines. 
OK - License (GPLv2)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
cbba81a5b47b811dafd897cd7bd72dee  usbmon-5.2.tar.gz
cbba81a5b47b811dafd897cd7bd72dee  usbmon-5.2.tar.gz.orig


See below - Package needs ExcludeArch
OK - 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 - Package is code or permissible content. 
OK - Packages %doc files don't affect runtime. 
OK - Package has rm -rf RPM_BUILD_ROOT at top of %install

OK - 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. 
OK - final provides and requires are sane

SHOULD Items:

OK - Should build in mock. 
OK - Should build on all supported archs
OK - Should have dist tag
OK - Should package latest version

Issues: 

A few general comments, unrelated to the packaging and thus moot for the
review, 
but I thought I would mention them: 

- You have your upstream url as your people.redhat.com page. 
Perhaps it would be good to use a fedorahosted.org site for this? 
That way you get bug tracking/mailing lists/vcs repo, etc. 
See: https://fedorahosted.org/web/new

- You might add a note about the license version to the .c file. 
No big deal since it's mentioned other places and is clearly your intent, 
but just to be paranoid. ;) 

Now, on to issues about the package: 

1. I have no idea on the ExcludeArch. Does s390 have usb? 
In any case it's not a blocker here as Fedora doesn't have s390 as a primary
arch.
You might ask the s390 list?

I don't see any other blockers here, this is a very simple package, 
and is APPROVED. 

Go ahead and continue the process from
http://fedoraproject.org/wiki/PackageMaintainers/Join#Get_a_Fedora_Account
If you have any questions don't hesitate to contact me via bugzilla, email, 
or on irc (nirik on freenode).

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

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


[Bug 452413] Review Request: BkChem - Chemical drawing program

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Depends on||476374




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

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


[Bug 476374] Review Request: OASA - python library for manipulation of chemical formats

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


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 CC||mtas...@ioa.s.u-tokyo.ac.jp
 Blocks||177841, 452413




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

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


[Bug 476385] Review Request: pycrypto - The Python Cryptography Toolkit

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


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


Mads Kiilerich m...@kiilerich.com changed:

   What|Removed |Added

 CC||m...@kiilerich.com




--- Comment #1 from Mads Kiilerich m...@kiilerich.com  2008-12-13 21:08:27 
EDT ---
This is already packaged in the python-crypto package. The upstream project has
just moved from amk.ca to dlitz.net.

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

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


[Bug 476385] Review Request: pycrypto - The Python Cryptography Toolkit

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


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


Conrad Meyer kon...@tylerc.org changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG




--- Comment #2 from Conrad Meyer kon...@tylerc.org  2008-12-13 21:25:52 EDT 
---
Ah, thanks. Closing then.

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

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


[Bug 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client

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


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





--- Comment #17 from Jerry Amundson jamun...@gmail.com  2008-12-13 21:52:01 
EDT ---
[r...@walnut ~]# rpm -Uvh --test
~jerry/rpmbuild/RPMS/i386/samba4-common-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386.rpm
~jerry/rpmbuild/RPMS/i386/samba4-client-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386.rpm
~jerry/rpmbuild/RPMS/i386/samba4-winbind-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386.rpm
~jerry/rpmbuild/RPMS/i386/samba4-libs-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386.rpm
~jerry/rpmbuild/RPMS/i386/samba4-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386.rpm
Preparing...### [100%]
file /lib/libnss_winbind.so.2 from install of
samba4-winbind-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386 conflicts with file from
package samba-winbind-0:3.2.5-0.23.fc11.i386
file /usr/bin/ntlm_auth from install of
samba4-winbind-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386 conflicts with file from
package samba-winbind-0:3.2.5-0.23.fc11.i386
file /usr/bin/wbinfo from install of
samba4-winbind-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386 conflicts with file from
package samba-winbind-0:3.2.5-0.23.fc11.i386
file /usr/lib/libnss_winbind.so from install of
samba4-winbind-4.0.0-0.6.alpha6.GIT.3508a66.fc11.i386 conflicts with file from
package samba-winbind-0:3.2.5-0.23.fc11.i386

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

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


[Bug 476386] Review Request: perl-verilog - Verilog parsing routines

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


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


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
   Flag||fedora-review?




--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro  2008-12-13 
22:03:13 EDT ---
There are some missing BR:

+ /usr/bin/perl Makefile.PL INSTALLDIRS=vendor 'CP=/bin/cp -p' PREFIX=/usr

%Error: 'flex' must be installed to build

%Error: 'bison' must be installed to build
%Error: Exiting due to above missing dependencies.
error: Bad exit status from /var/tmp/rpm-tmp.pdWnhE (%build)


RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.pdWnhE (%build)


And according to https://fedoraproject.org/wiki/Licensing the license tag
should be either Artistic(clarified) or Artistic 2.0, just Artistic is not
valid. At the first glance, you should use  Artistic(clarified)

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

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


[Bug 476387] Review Request: vhd2vl - VHDL to Verilog translator

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


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


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
   Flag||fedora-review?




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

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


[Bug 476387] Review Request: vhd2vl - VHDL to Verilog translator

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


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





--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro  2008-12-13 
22:42:54 EDT ---
please either use parallel build or make a note about it being non-functional
in the spec.
Anyway, mock build fails:

+ /usr/bin/make -C src
make: Entering directory `/builddir/build/BUILD/vhd2vl/src'
flex -i vhd2vl.l
bison -d -v -t vhd2vl.y
gcc -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector
--param=ssp-buffer-size=4 -m64 -m
tune=generic -O -g -o vhd2vl lex.yy.c vhd2vl.tab.c -lfl
lex.yy.c: In function 'yylex':
vhd2vl.l:144: warning: ignoring return value of 'fwrite', declared with
attribute warn_unused_result
vhd2vl.l: At top level:
lex.yy.c:1670: warning: 'yyunput' defined but not used
lex.yy.c:1713: warning: 'input' defined but not used
vhd2vl.y: In function 'yyparse':
vhd2vl.y:850: error: expected ';' before '}' token
vhd2vl.y:851: error: expected ';' before '}' token
vhd2vl.y:1389: error: expected ';' before '}' token
make: *** [vhd2vl] Error 1
make: Leaving directory `/builddir/build/BUILD/vhd2vl/src'
error: Bad exit status from /var/tmp/rpm-tmp.14SfcI (%build)


RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.14SfcI (%build)

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

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


[Bug 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client

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


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





--- Comment #18 from Matthew Barnes mbar...@redhat.com  2008-12-13 23:14:35 
EDT ---
Yeah, not sure what to do about the library.

File conflicts in other samba4 subpackages were all binaries, which I resolved
by simply appending a '4' to the name.

Ideas?

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

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


  1   2   >