[Bug 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby

2011-02-05 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=667954

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-02-05 03:21:39

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 675388] New: Review Request: xmlada - full XML stack for Ada

2011-02-05 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: xmlada - full XML stack for Ada

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

   Summary: Review Request: xmlada - full XML stack for Ada
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pa...@zhukoff.net
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://landgraf.fedorapeople.org/packages/requested/xmlada/xmlada.spec
SRPM URL:
http://landgraf.fedorapeople.org/packages/requested/xmlada/xmlada-3.2.1-3.fc14.src.rpm

Description: 
XML/Ada includes support for parsing XML files, including DTDs,
full support for SAX, and an almost complete support for the core part of the
DOM. It includes support for validating XML files with XML schemas

I've built it for my next package gprbuild and other packages from Ada stack
(such as AWS).

koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=2763602 (for dist f13
and f14 is also OK).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675388] Review Request: xmlada - full XML stack for Ada

2011-02-05 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=675388

--- Comment #1 from Pavel Zhukov pa...@zhukoff.net 2011-02-05 05:59:38 EST ---
rpmlint output:
xmlada.src: W: spelling-error %description -l en_US schemas - schema, sachems,
schemes
xmlada.src: W: invalid-url Source0:
http://libre.adacore.com/xmlada-gpl-3.2.1-src.tgz HTTP Error 404: Not Found
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
xmlada.x86_64: W: spelling-error %description -l en_US schemas - schema,
sachems, schemes
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
xmlada-devel.x86_64: W: spelling-error %description -l en_US xml - XML, cml,
ml
xmlada-devel.x86_64: W: spelling-error %description -l en_US ada - Ada, ads,
ad
xmlada-devel.x86_64: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 3 warnings.
1 packages and 0 specfiles checked; 0 errors, 0 warnings.




-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675402] New: Review Request: qpass - password manager

2011-02-05 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: qpass - password manager

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

   Summary: Review Request: qpass - password manager
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mateuszpie...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://qpass.sourceforge.net/qpass.spec
SRPM URL:
http://downloads.sourceforge.net/project/qpass/packages/Fedora/qpass-1.0.1-1.fc14.src.rpm
Description: Easy to use password manager which enables you to store your
password and logins in encrypted database(using AES-256). Each entry can be
named and described.

This is my own application.
This is my first package and I need a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


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

2011-02-05 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=448613

Bug 448613 depends on bug 672153, which changed state.

Bug 672153 Summary: Add a -source subpackage
https://bugzilla.redhat.com/show_bug.cgi?id=672153

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|NEW |CLOSED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 669311] Review Request: mupdf - A lightweight PDF viewer and toolkit written in portable C

2011-02-05 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=669311

--- Comment #25 from Mohamed El Morabity pikachu.2...@gmail.com 2011-02-05 
08:06:46 EST ---
(In reply to comment #24)
 Mohamed, 
 I am sorry for irascibility
No problem, in fact I didn't felt any irascibility here :)

 So, Do I rebuild mupdf to use static libraries and build mudpf-static package?
Yes. Just keep in mind that since we will only provide a static library, no
mupdf-static subpackage is needed; the mupdf-devel package will embed the
static library.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675104] Review Request: tweepy - Twitter library for python

2011-02-05 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=675104

--- Comment #3 from Ratnadeep Debnath rtn...@gmail.com 2011-02-05 08:28:27 
EST ---
I updated the SPEC file for tweepy taking into account the above suggestions. I
also did a mock build on my system and it completed successfully. I have
uploaded the latest SPEC and SRPM files.

Spec URL: http://rtnpro.fedorapeople.org/Packages/SPECS/tweepy.spec
SRPM URL:
http://rtnpro.fedorapeople.org/Packages/SRPMS/tweepy-1.7.1-2.fc14.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 635256] Review Request: qtop - tool for monitoring PBS systems

2011-02-05 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=635256

--- Comment #11 from Michael Schwendt mschwe...@gmail.com 2011-02-05 08:30:38 
EST ---
Please enter your real name in your bugzilla account preferences. That will
also improve the current
http://fedoraproject.org/PackageReviewStatus/NEEDSPONSOR.html list.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 633333] Review Request: iosum - An I/O bandwidth and syscall summarizer

2011-02-05 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=63

--- Comment #21 from Michael Schwendt mschwe...@gmail.com 2011-02-05 08:31:04 
EST ---
Please enter your real name in your bugzilla account preferences. That will
also improve the current
http://fedoraproject.org/PackageReviewStatus/NEEDSPONSOR.html list.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 673589] Review Request: UpTools - C++ library for hpc, networking, db, memory, etc.

2011-02-05 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=673589

--- Comment #10 from Michael Schwendt mschwe...@gmail.com 2011-02-05 08:33:24 
EST ---
Please enter your real name in your bugzilla account preferences. That will
also improve the current
http://fedoraproject.org/PackageReviewStatus/NEEDSPONSOR.html list.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 584090] Review Request: mashpodde - mashpodder is a podcatching client based on BashPodder

2011-02-05 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=584090

--- Comment #10 from Michael Schwendt mschwe...@gmail.com 2011-02-05 08:32:44 
EST ---
ping rootoutc...@hushmail.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2011-02-05 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=673585

--- Comment #3 from Michael Schwendt mschwe...@gmail.com 2011-02-05 08:30:17 
EST ---
Please enter your real name in your bugzilla account preferences. That will
also improve the current
http://fedoraproject.org/PackageReviewStatus/NEEDSPONSOR.html list.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 483859] Review Request: libg3d - Library for 3D file/object viewer

2011-02-05 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=483859

--- Comment #18 from Guido Grazioli guido.grazi...@gmail.com 2011-02-05 
08:32:04 EST ---
There were no releases of this library in two years, and i could not find any
scm for the sources, only the tarballs in their site. Did you contact upstream
recently in order to see if the project is still alive?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675104] Review Request: tweepy - Twitter library for python

2011-02-05 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=675104

Pavel Zhukov pa...@zhukoff.net changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|pa...@zhukoff.net

--- Comment #4 from Pavel Zhukov pa...@zhukoff.net 2011-02-05 08:35:09 EST ---
I'll review it



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674188] Review Request: libzeitgeist - Library to access zeitgeist; needed by synapse

2011-02-05 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=674188

Michael Schwendt mschwe...@gmail.com changed:

   What|Removed |Added

 CC||mschwe...@gmail.com

--- Comment #2 from Michael Schwendt mschwe...@gmail.com 2011-02-05 08:56:19 
EST ---
This needs some more work, not limited to these issues or those found by Thomas
already:

Run rpmlint not just on the src.rpm, but also on the built rpms.


 #BuildRequires:  
 Requires:   zeitgeist

https://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires

This explicit Requires asks for a comment in the spec file. Is it a run-time
dependency only? Is it an install-time dependency?


 %{_libdir}/pkgconfig

You include the directory, which belongs into the pkgconfig package. To fix
this, specify the exact .pc file name to include in your package:

  %{_libdir}/pkgconfig/zeitgeist-1.0.pc

That would also help with noticing updates/upgrades which bump the version in
that file name and would break compatibility with dependencies that look for
exactly that .pc file.


 %doc %{_docdir}/%{name}

This is duplicated in both packages. Additionally, the HTML docs are
duplicated, too, in several places.


 %docdir %{_datadir}/gtk-doc/html/zeitgeist-1.0

What does this do?  It doesn't mark any directory as %doc, and a few lines
below you include the directory once more:

%doc %{_datadir}/gtk-doc/html/zeitgeist-1.0


 %doc AUTHORS ChangeLog COPYING COPYING.GPL INSTALL MAINTAINERS NEWS README
 %doc %{_docdir}/%{name}

That creates duplicates, too. Plus, the INSTALL file is the one that is
irrelevant to RPM package users:

$ rpm -qpd libzeitgeist-0.3.2-1.fc14.i686.rpm 
/usr/share/doc/libzeitgeist-0.3.2/AUTHORS
/usr/share/doc/libzeitgeist-0.3.2/COPYING
/usr/share/doc/libzeitgeist-0.3.2/COPYING.GPL
/usr/share/doc/libzeitgeist-0.3.2/ChangeLog
/usr/share/doc/libzeitgeist-0.3.2/INSTALL
/usr/share/doc/libzeitgeist-0.3.2/MAINTAINERS
/usr/share/doc/libzeitgeist-0.3.2/NEWS
/usr/share/doc/libzeitgeist-0.3.2/README
/usr/share/doc/libzeitgeist/AUTHORS
/usr/share/doc/libzeitgeist/COPYING
/usr/share/doc/libzeitgeist/INSTALL
/usr/share/doc/libzeitgeist/MAINTAINERS
/usr/share/doc/libzeitgeist/README


 %{_datadir}/vala/vapi

As a general hint on including directories, consider adding a trailing slash
for increased readability and clearness:

%{_datadir}/vala/vapi/

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675104] Review Request: tweepy - Twitter library for python

2011-02-05 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=675104

--- Comment #5 from Pavel Zhukov pa...@zhukoff.net 2011-02-05 09:06:22 EST ---
Upstream sources has *py in its name. It's ok
rpmlint output is clean. It's ok

-- Please remove %clean section because F12 is currently unsupported or add
BuildRoot tag if you want build package for EPEL

-- Please add python2-devel if you package for Python2 only, python3-devel if
you package for Python3 only. A package that has both python2 and python3 files
will need to BuildRequire both.
Please see http://fedoraproject.org/wiki/Packaging:Python#BuildRequires

-- Please remove python from BuildRequires because python-devel already has
python in deplist

-- Each source file contains point to LICENSE but there is not LICENSE file. So
you SHOULD query upstream to include it.See
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text for
additional info

-- There's also no need to use %{__rm}  for rm command, although there is
currently no guideline forbidding it. Please replace pointless macros or give
reasons to use it.





-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 571019] Review Request: libmtag - An advanced C music tagging library with a simple API

2011-02-05 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=571019

--- Comment #10 from Felipe Contreras felipe.contre...@gmail.com 2011-02-05 
09:52:44 EST ---
(In reply to comment #9)
 Hmm, it seems the .src.rpm is 404 too. Please fix that, I need to src.rpm to
 complete the review. In the mean time I've done most of the review based on 
 the
 .spec file:

What's the point of providing the .spec file if it's generated from the .spec
and tarball? (both have been provided)

Anyway, libmtag:
http://people.freedesktop.org/~felipec/fedora/libmtag.spec
http://people.freedesktop.org/~felipec/fedora/libmtag-0.3.6-1.fc14.src.rpm

libmtag-python:
http://people.freedesktop.org/~felipec/fedora/libmtag-python.spec
http://people.freedesktop.org/~felipec/fedora/libmtag-python-0.3.1-1.fc14.src.rpm

Here's a koji build for the first, I don't know how to do it for the second:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2763785

 Needs work:
 - source matches upstream
   I could not verify this because of the lack of a .src.rpm

rpmbuild -ba libmtag.spec

 - summary
   Please drop the An from the summary, as a general rule we try to not have
   Summaries start with A, An or The.

Done.

 - %doc (main package)
   Please add README and NEWS to the main package %doc, they seem useful
   to have around to me.

Done.

(In reply to comment #8)
 python bindings would make a fine second package submission to show some more
 of your packaging skills.
 
 I'll go and review this package now, but, once reviewed, I would still like to
 see something more of your packaging skills before sponsoring you.

Done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 611048] Review Request: python-amqplib - Client library for AMQP

2011-02-05 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=611048

Brian Pepple bdpep...@gmail.com 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675426] New: Review Request: imap-upload - A tool for uploading a local mbox file to IMAP4 server

2011-02-05 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: imap-upload - A tool for uploading a local mbox file 
to IMAP4 server

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

   Summary: Review Request: imap-upload - A tool for uploading a
local mbox file to IMAP4 server
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jsauc...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://jfsaucier.fedorapeople.org/packages/imap-upload.spec
SRPM URL:
http://jfsaucier.fedorapeople.org/packages/imap-upload-1.2-1.fc14.src.rpm
Description: 
IMAP Upload is a tool for uploading a local mbox file to IMAP4 server. The
most stable way to migrate to Gmail. Features include :

- Read messages stored in mbox format which is used by many mail clients
such as Thunderbird.

- Preserve the delivery time of the message.

- Automatic retry when the connection was aborted which happens frequently
on Gmail.

- Can write out failed messages in mbox format.



rpmlint give no warning/error on spec, srpm and rpm, just said the package
didn't have a man page, I will try to submit this bug in upstream.


Here are the koji scratch build, they all build fine :

Rawhide : http://koji.fedoraproject.org/koji/taskinfo?taskID=2763385
F14 : http://koji.fedoraproject.org/koji/taskinfo?taskID=2763387
F13 : http://koji.fedoraproject.org/koji/taskinfo?taskID=2763391


Thank you!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675104] Review Request: tweepy - Twitter library for python

2011-02-05 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=675104

--- Comment #6 from Ratnadeep Debnath rtn...@gmail.com 2011-02-05 11:06:09 
EST ---
(In reply to comment #5)

 -- Please remove %clean section because F12 is currently unsupported or add
 BuildRoot tag if you want build package for EPEL
Currently, I am packaging it only for Fedora. If it is error-free, I will
package it for EPEL.

 -- Please add python2-devel if you package for Python2 only, python3-devel if
 you package for Python3 only. A package that has both python2 and python3 
 files
 will need to BuildRequire both.
 Please see http://fedoraproject.org/wiki/Packaging:Python#BuildRequires
As far as I know, python2-devel doesn't exist in yum repository, there exists
python-devel ( for python2) and python3-devel.

 -- Please remove python from BuildRequires because python-devel already has
 python in deplist
done.

 -- Each source file contains point to LICENSE but there is not LICENSE file. 
 So
 you SHOULD query upstream to include it.See
 http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text for
 additional info
There is a LICENSE file in the root of the src dir.
 -- There's also no need to use %{__rm}  for rm command, although there is
 currently no guideline forbidding it.
Do I remove the %{__rm} macro from %install section also?

 Please replace pointless macros or give
 reasons to use it.
Please mention the pointless macros.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 623606] Review Request: gxneur - GTK front-end for X Neural Switcher

2011-02-05 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=623606

--- Comment #13 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 
2011-02-05 11:21:24 EST ---
Martin, thank you very much for the review! Can I make response review for you?

(In reply to comment #12)
 Hi Pavel,
 
 the package looks almost fine. Please drop Requires: xneur = %{version}. This
 dependency is automatically resolved by the soname of libxneur. Upstream seems
 to bump the soname with every new release. 
 See also http://fedoraproject.org/wiki/PackagingDrafts/ExplicitRequires
 
Unfortunately we can't do that! As you can see in last update of xneur it is
from SVN and no soname bump was made. In last particular update I agree it only
build fix and is not so important. But previous svn build (it in changelog of
xneur if you want see) was to fix error what I ask upstream author. So, full
version dependency should be used to pull correct version!

http://hubbitus.net.ru/rpm/Fedora13/gxneur/gxneur-0.12.0-2.svn859.fc13.src.rpm
http://hubbitus.net.ru/rpm/Fedora13/gxneur/gxneur.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675426] Review Request: imap-upload - A tool for uploading a local mbox file to IMAP4 server

2011-02-05 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=675426

--- Comment #1 from Michael Schwendt mschwe...@gmail.com 2011-02-05 11:47:29 
EST ---
 Gmail

Note that Google Mail would be less ambiguous with regard to the Gmail
trademark and lawsuits like this:
http://www.theregister.co.uk/2008/06/24/no_access_to_gmail_in_germany/

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 673589] Review Request: UpTools - C++ library for hpc, networking, db, memory, etc.

2011-02-05 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=673589

Ralf Corsepius rc040...@freenet.de changed:

   What|Removed |Added

 CC||rc040...@freenet.de

--- Comment #11 from Ralf Corsepius rc040...@freenet.de 2011-02-05 11:46:07 
EST ---
Please increment your submitted package's %release each time you modify your
submission and provide URLS to your updated SPEC and srpm - ATM, it's
impossible for reviewers to tell whether they are looking at the right package.


Besides this, your package currently fails to build in mock:

[configure]
...
checking whether the C compiler works... no
onfigure: error: in `/builddir/build/BUILD/UpTools-8.5.4':
configure: error: C compiler cannot create executables
See `config.log' for more details


config.log:
...
configure:3622: checking whether the C compiler works
configure:3644: gcc -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic  -no-install
-Wl,-as-needed conftest.c  5
gcc: error: unrecognized option '-no-install'
configure:3648: $? = 1
configure:3686: result: no
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME UpTools   
| #define PACKAGE_TARNAME UpTools
| #define PACKAGE_VERSION 8.5.4
| #define PACKAGE_STRING UpTools 8.5.4
| #define PACKAGE_BUGREPORT bugs-upto...@palermo.edu
| #define PACKAGE_URL
| #define PACKAGE UpTools
| #define VERSION 8.5.4
| /* end confdefs.h.  */
|
| int
| main ()
| {  
|
|   ;
|   return 0;
| }
configure:3691: error: in `/builddir/build/BUILD/UpTools-8.5.4':
configure:3693: error: C compiler cannot create executables
See `config.log' for more details

AFAIS, the configure script is trying to play games with LDFLAGS.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675104] Review Request: tweepy - Twitter library for python

2011-02-05 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=675104

--- Comment #7 from Pavel Zhukov pa...@zhukoff.net 2011-02-05 12:08:48 EST ---
 Please mention the pointless macros.
you can find additional info here:
https://bugzilla.redhat.com/show_bug.cgi?id=669311#c9 macros starting with __
is pointless because they do nothing

 Do I remove the %{__rm} macro from %install section also?
Yes, you do

 There is a LICENSE file in the root of the src dir.
I've check twice: There's no LICENSE file in src dir of your SRPM but it is in
Git repo. Currently, your package doesn't contain license text but it contain
11 points to this one
grep -R LICENSE *  | wc -l  
11
Please contact upstream to clarify this situation (create github's issue, send
mail or other). If LICENSE is MIT you can bring LICENSE file as Source1 or
Patch (with bug id, mail id in comment) for example, you should add it to %doc
list of %files section and I'll aprove tweepy package.

 As far as I know, python2-devel doesn't exist in yum repository, there exists
python-devel ( for python2) and python3-devel.
Ok.sorry. I think PythonPackaging Guidelines is out-of-date

 If it is error-free, I will package it for EPEL.
You have to add
BuildRoot:  %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
to your spec file.


Summary: Please replace pointless macros, add BuildRoot (or remove %clean) and
fix LICENSE issue for approving



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675426] Review Request: imap-upload - A tool for uploading a local mbox file to IMAP4 server

2011-02-05 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=675426

--- Comment #2 from Jean-Francois Saucier jsauc...@gmail.com 2011-02-05 
12:24:47 EST ---
Spec URL: http://jfsaucier.fedorapeople.org/packages/imap-upload.spec
SRPM URL:
http://jfsaucier.fedorapeople.org/packages/imap-upload-1.2-2.fc14.src.rpm


Change Gmail for Google Mail in the package description.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python

2011-02-05 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=619355

--- Comment #11 from Steve Traylen steve.tray...@cern.ch 2011-02-05 13:33:39 
EST ---
Not
 http://cern.ch/straylen/rpms/python26-numpy/python26-numpy-1.5.1-1.el5.src.rpm
 http://cern.ch/straylen/rpms/python26-numpy/python26-numpy.spec

but

http://cern.ch/straylen/rpms/python26-numpy/python26-numpy-1.5.1-2.el5.src.rpm
http://cern.ch/straylen/rpms/python26-numpy/python26-numpy.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 662301] Review Request: plotdrop - A minimal GNOME front-end to Gnuplot

2011-02-05 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=662301

--- Comment #16 from Fedora Update System upda...@fedoraproject.org 
2011-02-05 14:29:17 EST ---
plotdrop-0.5.3-5.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/plotdrop-0.5.3-5.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 662301] Review Request: plotdrop - A minimal GNOME front-end to Gnuplot

2011-02-05 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=662301

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 592487] Review Request: ffgtk - A solution for controlling Fritz!Box or compatible routers

2011-02-05 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=592487

--- Comment #24 from Louis Lagendijk lo...@lagendijk.xs4all.nl 2011-02-05 
14:34:12 EST ---
New Package CVS Request
===
Package Name: ffgtk
Short Description: A solution for controlling Fritz!Box or compatible routers
Owners: llagendijk
Branches: f13 f14 el6
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674188] Review Request: libzeitgeist - Library to access zeitgeist; needed by synapse

2011-02-05 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=674188

--- Comment #4 from Renich Bon Ciric ren...@woralelandia.com 2011-02-05 
15:20:02 EST ---
(In reply to comment #2)
 $ rpm -qpd libzeitgeist-0.3.2-1.fc14.i686.rpm 
 /usr/share/doc/libzeitgeist-0.3.2/AUTHORS
 /usr/share/doc/libzeitgeist-0.3.2/COPYING
 /usr/share/doc/libzeitgeist-0.3.2/COPYING.GPL
 /usr/share/doc/libzeitgeist-0.3.2/ChangeLog
 /usr/share/doc/libzeitgeist-0.3.2/INSTALL
 /usr/share/doc/libzeitgeist-0.3.2/MAINTAINERS
 /usr/share/doc/libzeitgeist-0.3.2/NEWS
 /usr/share/doc/libzeitgeist-0.3.2/README
 /usr/share/doc/libzeitgeist/AUTHORS
 /usr/share/doc/libzeitgeist/COPYING
 /usr/share/doc/libzeitgeist/INSTALL
 /usr/share/doc/libzeitgeist/MAINTAINERS
 /usr/share/doc/libzeitgeist/README

I think this might be the only remaining problem. How do I get rid of 
/usr/share/doc/libzeitgeist/ which is of  no use to us?

I tried deleting this in the %install section but gained nothing.
Do I need to patch the make file?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674188] Review Request: libzeitgeist - Library to access zeitgeist; needed by synapse

2011-02-05 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=674188

--- Comment #5 from Michael Schwendt mschwe...@gmail.com 2011-02-05 16:28:15 
EST ---
 How do I get rid of /usr/share/doc/libzeitgeist/ which is of  no use to us?

Certainly you can adjust the contents of %{buildroot} at the end of %install to
your liking. The issue is fixed in libzeitgeist-0.3.2-2.fc14.src.rpm


The html docs are missing their top directory entry now, however:

$ rpmls -p
/home/misc14/tmp/rpm/RPMS/libzeitgeist-devel-0.3.2-2.fc14.i686.rpm|grep ^d
drwxr-xr-x  /usr/include/zeitgeist-1.0
drwxr-xr-x  /usr/share/doc/libzeitgeist-devel-0.3.2
drwxr-xr-x  /usr/share/vala/vapi

As I've pointed out before, the %docdir entry in your %files section does not
do what you probably think it does.  Directories are not marked as %doc, so a
single entry

  %doc %{_datadir}/gtk-doc/html/zeitgeist-1.0/

would be enough to include that directory *and* everything it it.


 %doc AUTHORS ChangeLog COPYING COPYING.GPL INSTALL MAINTAINERS NEWS README

Here 'INSTALL' can still be dropped.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 669311] Review Request: mupdf - A lightweight PDF viewer and toolkit written in portable C

2011-02-05 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=669311

--- Comment #26 from Pavel Zhukov pa...@zhukoff.net 2011-02-05 16:49:53 EST 
---
roll back to static
http://landgraf.fedorapeople.org/packages/requested/mupdf/mupdf.spec
http://landgraf.fedorapeople.org/packages/requested/mupdf/mupdf-0.7-5.fc14.src.rpm

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



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675478] New: Review Request: gprbuild - Ada project builder

2011-02-05 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: gprbuild - Ada project builder

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

   Summary: Review Request: gprbuild - Ada project builder
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pa...@zhukoff.net
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://landgraf.fedorapeople.org/packages/requested/gprbuild/gprbuild.spec
SRPM URL:
http://landgraf.fedorapeople.org/packages/requested/gprbuild/gprbuild-2010-2.fc14.src.rpm

Description: 
GPRbuild is an advanced software tool designed to help automate the
construction of multi-language systems. It removes complexity from
multi-language development by allowing developers to quickly and easily compile
and link software written in a combination of languages including Ada,
Assembler, C, C++, and Fortran. Easily extendable by users to cover new
toolchains and languages it is primarily aimed at projects of all sizes
organized into subsystems and libraries and is particularly well-suited for
compiled languages.

koji: Unavailable (see depend tag)
mock: ok

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675478] Review Request: gprbuild - Ada project builder

2011-02-05 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=675478

Pavel Zhukov pa...@zhukoff.net changed:

   What|Removed |Added

 Depends on||675388

--- Comment #1 from Pavel Zhukov pa...@zhukoff.net 2011-02-05 17:10:24 EST ---


-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 620191] Review Request: clawsker - Dialog to edit Claws Mail's hidden preferences

2011-02-05 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=620191

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

   What|Removed |Added

   Fixed In Version|clawsker-0.7.2-2.fc13   |clawsker-0.7.2-2.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 672015] Review Request: perl-Eval-Closure - Safely and cleanly create closures via string eval

2011-02-05 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=672015

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-02-05 17:20:39 EST ---
perl-Eval-Closure-0.02-1.fc14 has been pushed to the Fedora 14 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 620191] Review Request: clawsker - Dialog to edit Claws Mail's hidden preferences

2011-02-05 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=620191

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

   What|Removed |Added

   Fixed In Version||clawsker-0.7.2-2.fc13
 Resolution|NEXTRELEASE |ERRATA

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 672015] Review Request: perl-Eval-Closure - Safely and cleanly create closures via string eval

2011-02-05 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=672015

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Eval-Closure-0.02-1.fc
   ||14
 Resolution||ERRATA
Last Closed||2011-02-05 17:20:44

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 620191] Review Request: clawsker - Dialog to edit Claws Mail's hidden preferences

2011-02-05 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=620191

--- Comment #18 from Fedora Update System upda...@fedoraproject.org 
2011-02-05 17:21:06 EST ---
clawsker-0.7.2-2.fc14 has been pushed to the Fedora 14 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 620191] Review Request: clawsker - Dialog to edit Claws Mail's hidden preferences

2011-02-05 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=620191

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2011-02-05 17:19:46 EST ---
clawsker-0.7.2-2.fc13 has been pushed to the Fedora 13 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 672015] Review Request: perl-Eval-Closure - Safely and cleanly create closures via string eval

2011-02-05 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=672015

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

   What|Removed |Added

   Fixed In Version|perl-Eval-Closure-0.02-1.fc |perl-Eval-Closure-0.02-1.fc
   |14  |13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 672015] Review Request: perl-Eval-Closure - Safely and cleanly create closures via string eval

2011-02-05 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=672015

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-02-05 17:20:52 EST ---
perl-Eval-Closure-0.02-1.fc13 has been pushed to the Fedora 13 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 670352] Review Request: python-turbojson13 - Python template plugin that supports json

2011-02-05 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=670352

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-02-05 
17:52:09 EST ---
python-turbojson13-1.3-2.el6 has been pushed to the Fedora EPEL 6 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 670352] Review Request: python-turbojson13 - Python template plugin that supports json

2011-02-05 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=670352

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||python-turbojson13-1.3-2.el
   ||6
 Resolution||ERRATA
Last Closed||2011-02-05 17:52:15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675402] Review Request: qpass - password manager

2011-02-05 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=675402

Golo Fuchert packa...@golotop.de changed:

   What|Removed |Added

 CC||packa...@golotop.de
 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from Golo Fuchert packa...@golotop.de 2011-02-05 18:02:35 EST 
---
I have a few initial comments on your package:

- there is a typo in the summary (buil-in)
- have you checked if you really need the version numbers in the 
  (Build)Requires?
  You need them only, if there are older versions available for currently 
  supported versions of Fedora and maybe RHEL (if you want to package for it).
  Additionally you should check if the Requires are really needed, just from 
  looking at the spec file I would assume not (see
http://fedoraproject.org/wiki/PackagingGuidelines#Requires ).
- Lines should not be longer than 80 characters in the description.
- You made a very common mistake in the files section: %{_datadir}/%{name}/*
  This way the package will only own the files _in_ the directory 
  %{_datadir}/%{name}/, if the user now uninstalls the package, the directory 
  will remain on the user's system. So you need to own this by writing 
  %{_datadir}/%{name}/ in the files section. By that, the directory itself
_and_
  the files in the directory are owned by the package.
http://fedoraproject.org/wiki/Packaging:UnownedDirectories#Unowned_Directories
- Write something in the changelog, even for the very first version of the 
  package. For example
  * Sun Feb 6 2011 Your Name your_adr...@email.com 0.1-1
  - initial packaging

  If you don't understand the first line, have a closer look at that topic
  in the Guidelines.
- You need to verify the desktop file:
  http://fedoraproject.org/wiki/PackagingGuidelines#desktop-file-install_usage
- You need to update the icon cache:
  http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache

Don't be upset now, the package looks already quite clear for your first try!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674180] Review Request: knights - A chess board for KDE

2011-02-05 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=674180

Golo Fuchert packa...@golotop.de changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||packa...@golotop.de
 AssignedTo|nob...@fedoraproject.org|packa...@golotop.de

--- Comment #1 from Golo Fuchert packa...@golotop.de 2011-02-05 18:09:02 EST 
---
Knights should really be included in Fedora! I will do the review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674674] Review Request: python-zope-configuration - Zope Configuration Markup Language (ZCML)

2011-02-05 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=674674

Luke Macken lmac...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #6 from Luke Macken lmac...@redhat.com 2011-02-05 18:46:12 EST ---
New Package SCM Request
===
Package Name: python-zope-configuration
Short Description: Zope Configuration Markup Language (ZCML)
Owners: lmacken
Branches: f13 f14 el6 el5

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674676] Review Request: python-zope-deprecation - Zope 3 Deprecation Infrastructure

2011-02-05 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=674676

Luke Macken lmac...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Luke Macken lmac...@redhat.com 2011-02-05 18:45:34 EST ---
New Package SCM Request
===
Package Name: python-zope-deprecation
Short Description: Zope 3 Deprecation Infrastructure
Owners: lmacken
Branches: f13 f14 el6 el5

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675495] New: Review Request: parallel - Shell tool for executing jobs in parallel

2011-02-05 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: parallel - Shell tool for executing jobs in parallel

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

   Summary: Review Request: parallel - Shell tool for executing
jobs in parallel
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: low
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: packa...@golotop.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


SPEC URL: http://golotop.de/parallel.spec
SRPM URL: http://golotop.de/parallel-20110205-1.fc14.src.rpm
Description:
GNU Parallel is a shell tool for executing jobs in parallel using one or more
machines. A job is typically a single command or a small script that has to be
run for each of the lines in the input. The typical input is a list of files, a
list of hosts, a list of users, or a list of tables.

If you use xargs today you will find GNU Parallel very easy to use. If you
write loops in shell, you will find GNU Parallel may be able to replace most of
the loops and make them run faster by running jobs in parallel. If you use ppss
or pexec you will find GNU Parallel will often make the command easier to read.

GNU Parallel also makes sure output from the commands is the same output as you
would get had you run the commands sequentially. This makes it possible to use
output from GNU Parallel as input for other programs.

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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675495] Review Request: parallel - Shell tool for executing jobs in parallel

2011-02-05 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=675495

Golo Fuchert packa...@golotop.de changed:

   What|Removed |Added

   Priority|unspecified |medium

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 651613] Review Request: haddock - Haskell documentation tool

2011-02-05 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=651613

--- Comment #17 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-02-05 
23:17:54 EST ---
The fedorahosted ticket has been closed. The next step would be to build
haddock for devel,f14 and f13.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675402] Review Request: qpass - password manager

2011-02-05 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=675402

--- Comment #2 from Mateusz Piękos mateuszpie...@gmail.com 2011-02-06 
01:45:57 EST ---
Thanks for your reply and helpful information. I think that I corrected
everything what you said. Here is new:
SPEC: http://qpass.sourceforge.net/qpass.spec
SRPM:
http://downloads.sourceforge.net/project/qpass/packages/Fedora/qpass-1.0.1-2.fc14.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 674674] Review Request: python-zope-configuration - Zope Configuration Markup Language (ZCML)

2011-02-05 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=674674

Robin Lee robinlee.s...@gmail.com changed:

   What|Removed |Added

 Blocks||633138

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review