[Bug 659368] Review Request: luabind - A library that helps create bindings between C++ and Lua

2011-02-08 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=659368

--- Comment #8 from Dmitrij S. Kryzhevich  2011-02-09 00:44:03 
EST ---
Rawhide build is completed. Future F15 will be supplied with it.
There is no F14 luabind still.

-- 
Configure 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 676216] New: Review Request: sugar-calendario - agenda/calendar for sugar

2011-02-08 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: sugar-calendario - agenda/calendar for sugar

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

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


Spec URL: http://yaderv.fedorapeople.org/rpm/calendario/sugar-calendario.spec
SRPM URL:
http://yaderv.fedorapeople.org/rpm/calendario/sugar-calendario-3-1.fc14.src.rpm 
Description: 
Calendario is daily activity for sugar, this will allow children 
to introduce activities per day so they are reminded of past, present 
and future activities. 
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 676159] Review Request: crlibm - Correctly Rounded mathematical library

2011-02-08 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=676159

Pavel Zhukov  changed:

   What|Removed |Added

 CC||pa...@zhukoff.net
 Blocks|674008  |505154(FE-SCITECH)

--- Comment #1 from Pavel Zhukov  2011-02-08 23:39:49 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 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

2011-02-08 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=674008

Pavel Zhukov  changed:

   What|Removed |Added

 Depends on|676159  |

-- 
Configure 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 505154] Tracker: Review Requests for Science and Technology related packages

2011-02-08 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=505154

Pavel Zhukov  changed:

   What|Removed |Added

 Depends on||676159

-- 
Configure 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 676188] New: Review Request: qmidiarp - an arpeggiator, sequencer and MIDI LFO for ALSA

2011-02-08 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: qmidiarp -  an arpeggiator, sequencer and MIDI LFO for 
ALSA

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

   Summary: Review Request: qmidiarp -  an arpeggiator, sequencer
and MIDI LFO for ALSA
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: brendan.jones...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Hi 

a while back qmidiarp existed as a fairly simple MIDI arpeggiator. It has
recently been re-imagined and extended and is currently under heavy
development.

SRPM and SPEC file can be found here:

http://bsjones.fedorapeople.org/qmidiarp-0.3.9-1.fc14.src.rpm
http://bsjones.fedorapeople.org/qmidiarp.spec

Rpmlint output:

rpmlint ../SRPMS/qmidiarp-0.3.9-1.fc14.src.rpm 
qmidiarp.src: W: spelling-error Summary(en_US) arpeggiator -> arpeggiate,
arpeggio, appoggiatura
qmidiarp.src: W: spelling-error %description -l en_US arpeggiators ->
arpeggiate, arpeggios, arpeggio
qmidiarp.src: W: spelling-error %description -l en_US arp -> rap, Arp, ar
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

I will need to be sponsored as this will be my first package

thanks

Brendan Jones
FAS: bsjones

-- 
Configure 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 668243] Review Request: libqb - An IPC library for high performance servers.

2011-02-08 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=668243

--- Comment #22 from Angus Salkeld  2011-02-08 20:29:47 
EST ---
Here is another package (csync2) to review: 
https://bugzilla.redhat.com/show_bug.cgi?id=676187

-- 
Configure 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 676187] New: Csync2 is a cluster synchronization tool.

2011-02-08 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Csync2 is a cluster synchronization tool.

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

   Summary: Csync2 is a cluster synchronization tool.
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: asalk...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Description:
Csync2 is a cluster synchronization tool. It can be used to keep files on 
multiple hosts in a cluster in sync. Csync2 can handle complex setups with 
much more than just 2 hosts, handle file deletions and can detect conflicts.
It is expedient for HA-clusters, HPC-clusters, COWs and server farms.

SPEC: http://asalkeld.fedorapeople.org/csync2.spec
SRPM: http://asalkeld.fedorapeople.org/csync2-1.34-1.fc14.src.rpm
koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2799170

Other reviews:
https://bugzilla.redhat.com/show_bug.cgi?id=668243
https://bugzilla.redhat.com/show_bug.cgi?id=675947

$ rpmlint csync2
csync2.x86_64: W: non-conffile-in-etc /etc/csync2_ssl_cert.pem
csync2.x86_64: W: non-conffile-in-etc /etc/csync2_ssl_key.pem

If we _really_ need this fixed then I'll need to patch the code
to read then from their new location.

csync2.x86_64: W: no-manual-page-for-binary csync2-compare

Upstream does not provide a man page for this utility

1 packages and 0 specfiles checked; 0 errors, 3 warnings.

-- 
Configure 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 676175] New: Review Request: fpaste-server - Fedora Pastebin

2011-02-08 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: fpaste-server - Fedora Pastebin

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

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


Spec URL: http://herlo.fedorapeople.org/rpms/fpaste-server.spec
SRPM URL: http://herlo.fedorapeople.org/rpms/fpaste-server-0.1-1.src.rpm

Description: fpaste-server is simply put, a pastebin.
Like many a pastebin application before it,
fpaste-server strives to make it easy to
share code and other information collaboratively.

-- 
Configure 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 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

2011-02-08 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=674008

Tim Niemueller  changed:

   What|Removed |Added

 Depends on||676159

--- Comment #7 from Tim Niemueller  2011-02-08 18:32:06 EST 
---
Hi Rosen. Thanks for tuning in and the thorough list of packaged libraries. For
packaging it would be much more pleasant if you could make tarball releases. Is
that planned or could you think about it?

About the libraries:
collada-dom: keep as-is for now, once collada-dom is available as an updated
package make an RPM, propose for inclusion and eventually use it.

convexdecomposition: keep internal version, contains non-upstreamed patches,
lib seems to be meant for inclusion (no build system), and is linked statically
(i.e. no confusion/errant deps for other software)

ann/flann: using system-wide version with my patch already

fparser: keep the included version because of the local changes. If upstream
becomes responsive revisit issue.

crlibm: I have packaged it and it is up for review as bug #676159.

minizip, pcre, zlib, qhull: I have made some changes to ensure system-version
is used. I have also added statements to %prep stage to delete these
directories in the source code before building.

New SRPM is at
http://fedorapeople.org/~timn/robotics/openrave-0.2.18-0.3.svn1975.fc14.src.rpm.
Spec file has been changed in place.

-- 
Configure 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 676159] Review Request: crlibm - Correctly Rounded mathematical library

2011-02-08 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=676159

Tim Niemueller  changed:

   What|Removed |Added

 Blocks||674008

-- 
Configure 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 676159] New: Review Request: crlibm - Correctly Rounded mathematical library

2011-02-08 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: crlibm - Correctly Rounded mathematical library

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

   Summary: Review Request: crlibm - Correctly Rounded
mathematical library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: t...@niemueller.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~timn/misc/crlibm.spec
SRPM URL: http://fedorapeople.org/~timn/misc/crlibm-1.0-0.1.beta4.fc14.src.rpm
Description:
CRlibm is a free mathematical library (libm) which provides:
* implementations of the double-precision C99 standard elementary functions,
* correctly rounded in the four IEEE-754 rounding modes,
* with a comprehensive proof of both the algorithms used and their
  implementation,
* sufficiently efficient in average time, worst-case time, and memory
  consumption to replace existing libms transparently.

Project URL: http://lipforge.ens-lyon.fr/www/crlibm/

Koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=2796232

rpmlint:
crlibm.src: W: spelling-error %description -l en_US libm -> limb, lib, lib m
crlibm.src: W: spelling-error %description -l en_US libms -> limbs, lib's, lib
ms
crlibm.x86_64: W: spelling-error %description -l en_US libm -> limb, lib, lib m
crlibm.x86_64: W: spelling-error %description -l en_US libms -> limbs, lib's,
lib ms
- That is a false positive. In mentions libm (math lib) in the description.

crlibm.x86_64: W: incoherent-version-in-changelog 1.0beta4-1
['1.0-0.1.beta4.fc14', '1.0-0.1.beta4']
- I do not include the dist tag in the changelog entry and consider rpmlint
broken in this regard.
3 packages and 1 specfiles checked; 0 errors, 5 warnings.

-- 
Configure 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 676146] New: Review request: trace-gui - GUI tools from trace-cmd

2011-02-08 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: trace-gui - GUI tools from trace-cmd

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

   Summary: Review request: trace-gui - GUI tools from trace-cmd
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jonstan...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec file: http://jstanley.fedorapeople.org/trace-gui.spec
SRPM: http://jstanley.fedorapeople.org/trace-gui-1.0.4-1.fc13.src.rpm

trace-cmd is a user interface to Ftrace. Instead of needing to use the
debugfs directly, trace-cmd will handle of setting of options and
tracers and will record into a data file.

This package is the GUI components of trace-cmd, that is not packaged in RHEL,
built for EPEL only.
-
This is meant only for EPEL - Fedora does not have trace-cmd packaged (I'll
submit a separate review to get it in Fedora, and package the latest upstream
release there, this tracks the RHEL release of trace-cmd)

RHEL packages the upstream trace-cmd, but without the GUI components
(trace-graph, trace-view, and kernelshark). This package includes ONLY those
components, and has a requirement on trace-cmd being installed.

Note that during the build process, the entire package is built. During
%install, I remove the parts that are shipped in RHEL.

This package only applies to RHEL6, since it depends on Ftrace in the kernel.

-- 
Configure 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 672203] Review Request: erlang-riak_err - Enhanced SASL Error Logger for Riak

2011-02-08 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=672203

--- Comment #8 from Fedora Update System  2011-02-08 
18:00:26 EST ---
erlang-riak_err-1.0.0-0.1.20110105git429f757.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 652576] Review Request: ghc-texmath - Haskell texmathml library

2011-02-08 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=652576

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|ghc-texmath-0.4-3.fc13  |ghc-texmath-0.4-3.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 652576] Review Request: ghc-texmath - Haskell texmathml library

2011-02-08 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=652576

--- Comment #13 from Fedora Update System  
2011-02-08 17:59:30 EST ---
ghc-texmath-0.4-3.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 672203] Review Request: erlang-riak_err - Enhanced SASL Error Logger for Riak

2011-02-08 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=672203

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version||erlang-riak_err-1.0.0-0.1.2
   ||0110105git429f757.fc14
 Resolution|RAWHIDE |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 663244] Review Request: CUnit - A unit testing framework for C

2011-02-08 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=663244

--- Comment #23 from Fedora Update System  
2011-02-08 17:54:15 EST ---
CUnit-2.1.2-6.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 652582] Review Request: pandoc - Markdown markup converter

2011-02-08 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=652582

--- Comment #13 from Fedora Update System  
2011-02-08 17:57:23 EST ---
pandoc-1.6.0.1-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 652582] Review Request: pandoc - Markdown markup converter

2011-02-08 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=652582

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|pandoc-1.6.0.1-2.fc14   |pandoc-1.6.0.1-2.fc13

-- 
Configure 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 652576] Review Request: ghc-texmath - Haskell texmathml library

2011-02-08 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=652576

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||ghc-texmath-0.4-3.fc13
 Resolution||ERRATA
Last Closed||2011-02-08 17:57:52

-- 
Configure 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 663244] Review Request: CUnit - A unit testing framework for C

2011-02-08 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=663244

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||CUnit-2.1.2-6.fc14
 Resolution||ERRATA
Last Closed||2011-02-08 17:54:07

-- 
Configure 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 663244] Review Request: CUnit - A unit testing framework for C

2011-02-08 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=663244

--- Comment #22 from Fedora Update System  
2011-02-08 17:54:02 EST ---
CUnit-2.1.2-6.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 652576] Review Request: ghc-texmath - Haskell texmathml library

2011-02-08 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=652576

Bug 652576 depends on bug 648100, which changed state.

Bug 648100 Summary: Review Request: ghc-xml - A simple XML library
https://bugzilla.redhat.com/show_bug.cgi?id=648100

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #12 from Fedora Update System  
2011-02-08 17:57:47 EST ---
ghc-texmath-0.4-3.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 652582] Review Request: pandoc - Markdown markup converter

2011-02-08 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=652582

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||pandoc-1.6.0.1-2.fc14
 Resolution||ERRATA
Last Closed||2011-02-08 17:57:04

-- 
Configure 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 663244] Review Request: CUnit - A unit testing framework for C

2011-02-08 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=663244

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|CUnit-2.1.2-6.fc14  |CUnit-2.1.2-6.fc13

-- 
Configure 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 652582] Review Request: pandoc - Markdown markup converter

2011-02-08 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=652582

Bug 652582 depends on bug 648100, which changed state.

Bug 648100 Summary: Review Request: ghc-xml - A simple XML library
https://bugzilla.redhat.com/show_bug.cgi?id=648100

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #12 from Fedora Update System  
2011-02-08 17:56:58 EST ---
pandoc-1.6.0.1-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 673919] Review Request: monkey - Very small, fast and scalable web server for Linux.

2011-02-08 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=673919

Guillermo Gómez  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|guillermo.go...@gmail.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 647076] Review Request: jackbeat - audio sequencer

2011-02-08 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=647076

Brendan Jones  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG
   Flag|needinfo?(brendan.jones.it@ |
   |gmail.com)  |
Last Closed||2011-02-08 17:13:36

--- Comment #21 from Brendan Jones  2011-02-08 
17:13:36 EST ---
Closing for the time being



-- 
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 673919] Review Request: monkey - Very small, fast and scalable web server for Linux.

2011-02-08 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=673919

Guillermo Gómez  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||guillermo.go...@gmail.com
   Flag||fedora-review?

--- Comment #1 from Guillermo Gómez  2011-02-08 
16:49:12 EST ---
I'll review in parallel with Toshio.

-- 
Configure 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 676129] New: Review Request: qconf - Allows you to have a nice configure script for your qmake-based project

2011-02-08 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: qconf - Allows you to have a nice configure script for 
your qmake-based project

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

   Summary: Review Request: qconf - Allows you to have a nice
configure script for your qmake-based project
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Linux
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dr...@land.ru
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Hello. It is my second package for Fedora.

Spec URL: https://github.com/drizt/psi-plus/blob/master/qconf.spec
SRPM URL:
https://github.com/drizt/psi-plus/blob/master/qconf-1.4-1.fc14.src.rpm

QConf allows you to have a nice configure script for your
qmake-based project. It is intended for developers who don't need
(or want) to use the more complex GNU autotools. With qconf/qmake,
it is easy to maintain a cross-platform project that uses a
familiar configuration interface on unix.

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

Expected results:

Additional info:
Really this package is need only for psi-plus package. I will create Psi+
review request after qconf package will be added to Fedora repo.

-- 
Configure 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 622904] Review Request: rubygem-state_machine - Adds support for creating state machines for attributes on any Ruby class

2011-02-08 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=622904

--- Comment #12 from Guillermo Gómez  2011-02-08 
16:37:24 EST ---
New Package SCM Request
===
Package Name: rubygem-state_machine
Short Description: Adds support for creating state machines for attributes on
any Ruby class
Owners: gomix
Branches: el6 el5
InitialCC: gomix

-- 
Configure 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-08 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

--- Comment #5 from Golo Fuchert  2011-02-08 16:31:40 EST 
---
Just to prevent confusion: In the third point (splitting up) with "both
packages" I meant GNU parallel and moreutils parallel, using them together
would only be possible by renaming.

-- 
Configure 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 676124] Review Request: racket - replacement package for renamed plt-scheme

2011-02-08 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=676124

Gérard Milmeister  changed:

   What|Removed |Added

   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=647242

-- 
Configure 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 676124] New: Review Request: racket - replacement package for renamed plt-scheme

2011-02-08 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: racket - replacement package for renamed plt-scheme

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

   Summary: Review Request: racket - replacement package for
renamed plt-scheme
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: g...@bluewin.ch
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://gemi.fedorapeople.org/racket.spec
SRPM URL: http://gemi.fedorapeople.org/racket-5.0.2-1.fc14.src.rpm
Description: This is based on the plt-scheme package that I maintained.
PLT Scheme has been renamed to racket, so a new package is needed. The SPEC
file is almost the same. Any discussions that targeted the plt-scheme package
should therefore be redirected to this 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 674006] Review Request: openni - Library for human-machine Natural Interaction

2011-02-08 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=674006

--- Comment #4 from Tim Niemueller  2011-02-08 16:30:02 EST 
---
(In reply to comment #3)
> libraries should be deleted explicitly in %prep.  This should include

Done.

> It’s not a problem now, but if you do add version suffixes to the libraries,
> you’ll probably have to un and re-register the libraries on upgrade.  The
> .so.suffix could change with new api versions.  Also, is niReg creating a file
> somewhere?  If so, your package should own it if niReg -u doesn’t delete it 
> (so
> it doesn’t get left behind on uninstall).  

The libraries are plugins, therefore they don't need version numbers.


> I don’t really understand your rationale for leaving the changelog entry
> different than the package version number.  Changelog E-V-R doesn’t have
> anything to do with the source tarball, and the actual version of the package
> works out to 1.0.0.25-0.2.git4c9ff978.  There shouldn’t be a .fc14 in there
> either.

I tried to mute rpmlint. But I agree and have removed the dist tag from
changelog entries.

> [-] MUST: The package must meet the Packaging Guidelines .
> Bundled library and changelog notes above

Done.

> [-] MUST: 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 must be included in %doc.
> 
> Is there any reason the GPL text is also included?

No, removed.

> [-] MUST: Packages must NOT bundle copies of system libraries.
> See above notes.

It didn't, I patched them out. GL and Libs are deleted in %prep now, glh
headers are C++ template headers, which are directly compiled into objects. The
libraries are BSD licensed and do not seem to be available stand-alone but only
as part of some NVidia SDK.

> [-] MUST: Packages containing GUI applications must include a %{name}.desktop
> file, and that file must be properly installed with desktop-file-install in 
> the
> %install section. If you feel that your packaged GUI application does not need
> a .desktop file, you must put a comment in the spec file with your 
> explanation.
> The niViewer stuff in the samples is creating GUIs, you might want to note 
> that
> you don’t want to include .desktop files.

Added note in %files section.


New SRPM at
http://fedorapeople.org/~timn/robotics/openni-1.0.0.25-0.3.git4c9ff978.fc14.src.rpm,
spec changed in place.

Changed license to include BSD for glh headers. Renamed samples to examples
sub-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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 654007] Review Request: python-simplemediawiki - Extremely low-level wrapper to the MediaWiki API

2011-02-08 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=654007

Guillermo Gómez  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CURRENTRELEASE
Last Closed||2011-02-08 16:28:33

-- 
Configure 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-08 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

Pavel Zhukov  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution||NEXTRELEASE
Last Closed|2011-02-02 15:59:50 |2011-02-08 16:19:32

-- 
Configure 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-08 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 #36 from Fedora Update System  
2011-02-08 16:16:14 EST ---
mupdf-0.7-7.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/mupdf-0.7-7.fc13

-- 
Configure 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-08 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

--- Comment #4 from Golo Fuchert  2011-02-08 15:51:03 EST 
---
This is a tough one. What I learned so far:
- The developer of GNU parallel says that the first version of his program 
  would date back to 2001, but version control was introduced much later. He 
  argues that renaming on a per-distribution basis would cause problems between
  machines running different distributions, since GNU parallel runs itself on
  remote machines while distributing the work load.
- The problem for the parallel of moreutils seems to be that a lot of other 
  things depend on this. So they are not wanting to rename either.
- The Fedora packager of moreutils thinks that splitting up moreutils into a 
  core package and a parallel sub-package is a bad idea, since it might be
  the case that on a multiuser system both packages ought to be used.
- Personally, I really don't like the idea of a conflict between GNU parallel 
  and the moreutils package, since moreutils is a collection of different tools
  and I can think of users that want to use tools from that, but the GNU
  implementation of parallel.

So, what is the least bad solution? I don't know yet...

-- 
Configure 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-08 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

Fedora Update System  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 669311] Review Request: mupdf - A lightweight PDF viewer and toolkit written in portable C

2011-02-08 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 #35 from Fedora Update System  
2011-02-08 15:42:47 EST ---
mupdf-0.7-6.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/mupdf-0.7-6.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 553055] Review Request: gpxe - Open Source network bootloader

2011-02-08 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=553055

--- Comment #13 from Terje Røsten  2011-02-08 15:39:57 
EST ---
Hi Jeroen,
what is status here now? Do you still want this included?

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

[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-02-08 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=612796

Jason Tibbitts  changed:

   What|Removed |Added

  Status Whiteboard||StalledSubmitter

--- Comment #33 from Jason Tibbitts  2011-02-08 15:21:45 EST 
---
Not sure why this was returned to the review queue instead of just being
closed, but I've marked it stalled.

-- 
Configure 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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

Rex Dieter  changed:

   What|Removed |Added

 CC||rdie...@math.unl.edu
   Flag|fedora-review?  |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.
___
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-08 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 #34 from Bill Nottingham  2011-02-08 15:05:48 
EST ---
Git done (by process-git-requests).

-- 
Configure 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 663384] Review Request: scap-workbench - GUI tool that provides scanning, tailoring, editing and validation functionality for SCAP content

2011-02-08 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=663384

--- Comment #10 from Bill Nottingham  2011-02-08 15:04:32 
EST ---
f15 is not a valid branch - please resubmit with correct branches.

-- 
Configure 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 666222] Review Request: python-speaklater - implements a lazy string for python useful for use with gettext

2011-02-08 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=666222

--- Comment #10 from Bill Nottingham  2011-02-08 15:05:32 
EST ---
f15 is not a valid branch - please resubmit with correct branches.

-- 
Configure 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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #26 from Bill Nottingham  2011-02-08 15:03:11 
EST ---
Git done (by process-git-requests).

-- 
Configure 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 596138] Review Request: nss-gui - A graphical user interface for NSS security databases

2011-02-08 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=596138

--- Comment #22 from Bill Nottingham  2011-02-08 15:02:32 
EST ---
Git done (by process-git-requests).

-- 
Configure 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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #25 from Cosimo Cecchi  2011-02-08 14:57:38 EST 
---
New Package SCM Request
===
Package Name: abattis-cantarell-fonts
Short Description: Cantarell, a Humanist sans-serif font family
Owners: cosimoc
Branches: 
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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

Cosimo Cecchi  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 668591] Review Request: python26-markupsafe - Implements a XML/HTML/XHTML Markup safe string for Python

2011-02-08 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=668591

Steve Traylen  changed:

   What|Removed |Added

 Blocks||675628

-- 
Configure 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 675628] Review Request: python26-jinja2 - General purpose template engine

2011-02-08 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=675628

Steve Traylen  changed:

   What|Removed |Added

 CC||steve.tray...@cern.ch
 Depends on||668591

-- 
Configure 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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #24 from Stephen John Smoogen  2011-02-08 
14:01:14 EST ---

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


[Bug 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #23 from Cosimo Cecchi  2011-02-08 13:48:32 EST 
---
Stephen, thanks for the spec patch.
Here are updated URLs for the spec and the srpm, which should be good to go.

SPEC:
http://people.gnome.org/~cosimoc/cantarell-pkg/abattis-cantarell-fonts.spec
SRPM:
http://people.gnome.org/~cosimoc/cantarell-pkg/abattis-cantarell-fonts-0.0.1-2.fc15.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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #22 from Stephen John Smoogen  2011-02-08 
13:35:44 EST ---
[This review was done with the patch above so that the rpmlint would hush. The
only rpmlint issues left were aspell not knowing how to spell words and COPYING
not being Unix eol]

Once more unto the breach, dear friends, once more;
Or close the wall up with our Fedoran dead.

Package Review
==

Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[x]  Package is named according to the Package Naming Guidelines. [1]
[x]  Spec file name must match the base package %{name}, in the format
  %{name}.spec.
[x]  Spec file is legible and written in American English.
[x]  Spec file lacks Packager, Vendor, PreReq tags.
[x]  Spec uses macros instead of hard-coded directory names.
[x]  Package consistently uses macros.
[-]  Macros in Summary, %description expandable at SRPM build time.
[-]  PreReq is not used.
[x]  Requires correct, justified where necessary.
[x]  All build dependencies are listed in BuildRequires, except for 
 any that are listed in the exceptions section of Packaging 
 Guidelines. [2]
[x]  Buildroot is correct 
 (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)).
[x]  Package run rm -rf %{buildroot}
 (or $RPM_BUILD_ROOT) and the beginning of %install.
[-]  Package use %makeinstall only when 
 ``make install DESTDIR=...'' doesn't work.
[x]  Package has a %clean section, which contains
 rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
[x]  The spec file handles locales properly.
[x]  Changelog in prescribed format.
[x]  Rpmlint output is silent.
[x]  License field in the package spec file matches the actual
 license.
[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]  License file installed when any subpackage combination is 
 installed.
[x]  Package is licensed with an open-source compatible license and 
 meets other legal requirements as defined in the legal section 
 of Packaging Guidelines. [3,4]
[x]  Sources contain only permissible code or content.
[x]  Sources used to build the package matches the upstream source, 
 as provided in the spec URL.
 MD5SUM this package : d47d1ade6e292d90fc5f345db1ee67e7
 MD5SUM upstream package : d47d1ade6e292d90fc5f345db1ee67e7
[x]  Compiler flags are appropriate.
[-]  %build honors applicable compiler flags or justifies otherwise.
[-]  ldconfig called in %post and %postun if required.
[-]  Package must own all directories that it creates.
[x]  Package does not own files or directories owned by other packages.
[x]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[x]  Permissions on files are set properly.
[-]  Each %files section contains %defattr.
[-]  No %config files under /usr.
[-]  %config files are marked noreplace or the reason is justified.
[-]  Package contains a properly installed %{name}.desktop using 
 desktop-file-install file if it is a GUI application. [5]
[-]  Package contains a valid .desktop file.
[-]  Package contains code, or permissable content.
[-]  Package contains a SysV-style init script if in need of one.
[x]  File names are valid UTF-8.
[-]  Large documentation files are in a -doc subpackage, if required.
[x]  Package uses nothing in %doc for runtime.
[x]  Package contains no bundled libraries.
[-]  Header files in -devel subpackage, if present.
[-]  Static libraries in -static subpackage, if present.
[x]  Package contains no static executables.
[-]  Package requires pkgconfig, if .pc files are present.
[-]  Development .so files in -devel subpackage, if present.
[-]  Fully versioned dependency in subpackages, if present.
[x]  Package does not contain any libtool archives (.la).
[-]  Useful -debuginfo package or justification otherwise.
[x]  Rpath absent or only used for internal libs.
[x]  Package does not genrate any conflict.
[x]  Package does not contains kernel modules.
[x]  Package is not relocatable.
[x]  Package successfully compiles and builds into binary rpms on
 at least one supported architecture.
[x]  Package is not known to require ExcludeArch.
[x]  Package installs properly.
[x]  Package obeys FHS, except libexecdir and /usr/target.
[x]  Package meets the Packaging Guidelines. [6]

=== SUGGESTED ITEMS ===
[x]  Package functions as described.
[x]  Latest version is packaged.
[x]  Package does not include license text files separate from 
 upstream.
[-]  If the source package does not include license text(s) as a 
 separate file from upstream, the packager SHOULD query 
 upstream to include it.
[-]  Description and summary sections in the package spec file 
   

[Bug 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #21 from Jared Smith  2011-02-08 
13:22:23 EST ---
I'm glad to hear that Gnome is working w/ Dave Crossland on this, and that
we're making progress on getting it packaged.  I'm glad to defer to someone
else's spec file -- heaven knows mine wasn't perfect. It looks like the sources
on the GNOME infrastructure have the updated license text, so I don't see that
as being a blocker any more.

-- 
Configure 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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #20 from Stephen John Smoogen  2011-02-08 
12:56:19 EST ---
Created attachment 477662
  --> https://bugzilla.redhat.com/attachment.cgi?id=477662
Fix to make patch spec to pass reivew

To speed this up am adding patch to get this 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 659368] Review Request: luabind - A library that helps create bindings between C++ and Lua

2011-02-08 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=659368

Jan Kaluža  changed:

   What|Removed |Added

 CC||jkal...@redhat.com

--- Comment #7 from Jan Kaluža  2011-02-08 12:35:51 EST ---
What's the state of this one please? I would like to add package which needs
luabind.

-- 
Configure 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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #19 from Cosimo Cecchi  2011-02-08 12:16:20 EST 
---
(In reply to comment #18)

> 
> Cosimo, could you summarize what you have changed in the spec file? We have 3
> spec files now from 3 people with 2 reviews. Nobody will be interested in 
> doing
> another futile review.

Of course!

- as I mentioned previously, the font is now managed using the GNOME
infrastructure, and released as versioned tarballs on our servers, in sync with
the GNOME release cycle, so I changed the spec to fetch files from there.
- the tarballs contains now both the source files for the font and the binary,
there's no need for having multiple 'SourceX' entries in the spec file for each
.sfd file, so I removed those. 
- the fontconfig file is now shipped inside the tarball too, so there's no need
for having a separate Source entry for it.
- the preferred binary format for this font is OTF now, as that's the format
used by the binaries in the tarball too, so I changed the fontforge build lines
to use that instead of TTF.
- the upstream repository now removed the Oblique variants, so I removed those
from this package.
- the license of the font is now OFL-only, so I removed the reference to GPL in
the spec.

> What we don't have is much/any time left, if this is supposed to make it to 
> F15
> (feature freeze today). I suppose an exception could be granted for a Gnome 3
> dependency, though.

Yeah, I guess so, as this is a very important 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

--- Comment #18 from Michael J Gruber  2011-02-08 
11:48:59 EST ---
(In reply to comment #17)
> So, now Cantarell fonts moved to the GNOME infrastructure for releasing, bug
> tracking and version control. The font will be the GNOME 3 default, so it's 
> now
> very important to have it packaged in Fedora.
> 
> I created another updated SPEC file based on the new release, which you can
> find here
> 
> SPEC:
> http://people.gnome.org/~cosimoc/cantarell-pkg/abattis-cantarell-fonts.spec
> SRPM:
> http://people.gnome.org/~cosimoc/cantarell-pkg/abattis-cantarell-fonts-0.0.1-1.fc15.src.rpm
> 
> I am proposing as the package maintainer for this package.

Cosimo, could you summarize what you have changed in the spec file? We have 3
spec files now from 3 people with 2 reviews. Nobody will be interested in doing
another futile review.

What we don't have is much/any time left, if this is supposed to make it to F15
(feature freeze today). I suppose an exception could be granted for a Gnome 3
dependency, though.

-- 
Configure 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 663384] Review Request: scap-workbench - GUI tool that provides scanning, tailoring, editing and validation functionality for SCAP content

2011-02-08 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=663384

Peter Vrabec  changed:

   What|Removed |Added

 CC||pvra...@redhat.com
   Flag||fedora-cvs?

--- Comment #9 from Peter Vrabec  2011-02-08 11:49:03 EST 
---
New Package SCM Request
===
Package Name: scap-workbench
Short Description: GUI tool that provides scanning, tailoring, editing and
validation functionality for SCAP content
Owners: mbarabas pvrabec 
Branches: f14 f15

-- 
Configure 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 623868] Review Request: abattis-cantarell-fonts - Contemporary humanist sans-serif font

2011-02-08 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=623868

Cosimo Cecchi  changed:

   What|Removed |Added

 CC||ccec...@redhat.com

--- Comment #17 from Cosimo Cecchi  2011-02-08 11:38:31 EST 
---
So, now Cantarell fonts moved to the GNOME infrastructure for releasing, bug
tracking and version control. The font will be the GNOME 3 default, so it's now
very important to have it packaged in Fedora.

I created another updated SPEC file based on the new release, which you can
find here

SPEC:
http://people.gnome.org/~cosimoc/cantarell-pkg/abattis-cantarell-fonts.spec
SRPM:
http://people.gnome.org/~cosimoc/cantarell-pkg/abattis-cantarell-fonts-0.0.1-1.fc15.src.rpm

I am proposing as the package maintainer for 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676007] New: Review Request: python26-configobj - ConfigObj for Python 2.6

2011-02-08 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: python26-configobj - ConfigObj for Python 2.6

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

   Summary: Review Request: python26-configobj - ConfigObj for
Python 2.6
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: sh...@vecna.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://people.hmdc.harvard.edu/~shuff/epel/python26-configobj.spec
SRPM URL:
http://people.hmdc.harvard.edu/~shuff/epel/python26-configobj-4.6.0-1.src.rpm
Description: This is a port of the existing python-configobj package
(https://admin.fedoraproject.org/pkgdb/acls/name/python-configobj), built
against Python 2.6.  It's intended for EPEL 5 only, since it's unnecessary on
RHEL 6 and impossible on RHEL 4.

I have not submitted packages to Fedora or EPEL before; I'm looking for 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 676007] Review Request: python26-configobj - ConfigObj for Python 2.6

2011-02-08 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=676007

Steve Huff  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
Configure 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 664624] Review Request: libdmapsharing3 - A DMAP client and server library

2011-02-08 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=664624

--- Comment #4 from Cosimo Cecchi  2011-02-08 10:19:36 EST 
---
(In reply to comment #3)
> Based on Michael's comments, I am going to mark this report WONTFIX. Since the
> Rawhide Rhythmbox package has been updated to the version that uses
> libdmapsharing-2.9+, I will build only this libdmapsharing series for Rawhide.
> See http://koji.fedoraproject.org/koji/taskinfo?taskID=2772656. Cosimo, can 
> you
> re-enable the Rhythmbox DAAP plugin once this libdmapsharing build hits
> Rawhide?

Thanks, I just triggered a DAAP-enabled build in Koji.

-- 
Configure 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 675726] Review Request: cdm - Very minimalistic display manager

2011-02-08 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=675726

--- Comment #9 from Peter Lemenkov  2011-02-08 10:03:53 EST 
---
Ok, few more notes:

* xinitrc must be installed into %{_datarootdir}/%{name}/ .
* Missing requires - dialog, xorg-x11-utils, dbus, xorg-x11-xinit
* cdmrc mentions 'awesome' as one of the window managers. Please change it to
some more Fedora-specific.
* Licensing terms are misleading - the LICENSE file contains GPLv3, but the
main script, ./src/cdm, explicitly menions GPLv2+. Please, contact upstream for
clarification.

-- 
Configure 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 675726] Review Request: cdm - Very minimalistic display manager

2011-02-08 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=675726

--- Comment #7 from Mikhail Kulemin  2011-02-08 09:12:15 
EST ---
Created attachment 477617
  --> https://bugzilla.redhat.com/attachment.cgi?id=477617
New spec v3

-- 
Configure 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 675726] Review Request: cdm - Very minimalistic display manager

2011-02-08 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=675726

--- Comment #8 from Mikhail Kulemin  2011-02-08 09:13:02 
EST ---
Fixed

https://koji.fedoraproject.org/koji/getfile?taskID=2784901&name=cdm-0.5.3-3.fc14.src.rpm

spec in attachments

-- 
Configure 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 596138] Review Request: nss-gui - A graphical user interface for NSS security databases

2011-02-08 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=596138

Kai Engert (kaie)  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #21 from Kai Engert (kaie)  2011-02-08 08:49:45 
EST ---
New Package SCM Request
===
Package Name: nss-gui
Short Description: GUI to manage contents of an NSS database
Owners: kengert
Branches: 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 566410] Review Request: snmpcheck - An utility to get information via SNMP protocols

2011-02-08 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=566410

--- Comment #9 from Fabian Affolter  2011-02-08 
08:50:30 EST ---
The latest release of snmpcheck is 1.8

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


[Bug 675976] New: Review Request: jgoodies-common - Common library shared by JGoodies libraries and applications

2011-02-08 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: jgoodies-common - Common library shared by JGoodies 
libraries and applications

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

   Summary: Review Request: jgoodies-common - Common library
shared by JGoodies libraries and applications
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pikachu.2...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://melmorabity.fedorapeople.org/packages/jgoodies-common/jgoodies-common.spec
SRPM URL:
http://melmorabity.fedorapeople.org/packages/jgoodies-common/jgoodies-common-1.1.1-1.fc14.src.rpm
Description:
The JGoodies Common library provides convenience code for other JGoodies
libraries and applications.

Note:
This package is required by latest versions of jgoodies libraries
(jgoodies-forms/looks in the repos).

-- 
Configure 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 675963] Review Request: perl-MDV-Packdrakeng - Simple Archive Extractor/Builder

2011-02-08 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=675963

--- Comment #1 from Petr Pisar  2011-02-08 08:50:21 EST ---
Source tar ball is original. Ok.
Summary verified from lib/MDV/Packdrakeng.pm. Ok.
License verified from lib/MDV/Packdrakeng.pm. Ok.
No C binding, noarch BuildArch Ok.
Description verified from lib/MDV/Packdrakeng.pm. Ok.

TODO: BuildRequire perl(File::Path) (lib/MDV/Packdrakeng.pm:23) as the package
can dual-live in the future (http://search.cpan.org/~dland/File-Path/).

perl(Compress::Zlib) dependency (lib/MDV/Packdrakeng.pm:144) is optional. Ok.

FIX: BuildRequire perl(Test::More) to proceed all tests (t/01packdrakeng.t:6).
FIX: BuildRequire perl(Digest::MD5) to proceed all tests (t/01packdrakeng.t:7).

TODO: BuildRequire perl(Compress::Zlib) to perform all tests
(t/02methodusage.t:12).

All tests pass. Ok.

$ rpmlint perl-MDV-Packdrakeng.spec
../SRPMS/perl-MDV-Packdrakeng-1.13-1.fc14.src.rpm
../RPMS/noarch/perl-MDV-Packdrakeng-1.13-1.fc14.noarch.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
rpmlint Ok.

$ rpm -q -lv -p ../RPMS/noarch/perl-MDV-Packdrakeng-1.13-1.fc14.noarch.rpm
drwxr-xr-x2 rootroot0 úno  8 14:38
/usr/share/doc/perl-MDV-Packdrakeng-1.13
-rw-r--r--1 rootroot 4065 srp  9  2007
/usr/share/doc/perl-MDV-Packdrakeng-1.13/ChangeLog
-rw-r--r--1 rootroot  228 kvě  6  2007
/usr/share/doc/perl-MDV-Packdrakeng-1.13/README
-rw-r--r--1 rootroot 4664 úno  8 14:38
/usr/share/man/man3/MDV::Packdrakeng.3pm.gz
-rw-r--r--1 rootroot 1831 úno  8 14:38
/usr/share/man/man3/MDV::Packdrakeng::zlib.3pm.gz
drwxr-xr-x2 rootroot0 úno  8 14:38
/usr/share/perl5/MDV
drwxr-xr-x2 rootroot0 úno  8 14:38
/usr/share/perl5/MDV/Packdrakeng
-rw-r--r--1 rootroot30528 srp  9  2007
/usr/share/perl5/MDV/Packdrakeng.pm
-rw-r--r--1 rootroot 7517 srp  9  2007
/usr/share/perl5/MDV/Packdrakeng/zlib.pm
File permissions and layout Ok.

$ rpm -q --requires -p
../RPMS/noarch/perl-MDV-Packdrakeng-1.13-1.fc14.noarch.rpm | sort | uniq -c
  1 perl(base)  
  1 perl(Compress::Zlib)  
  1 perl(File::Path)  
  1 perl(:MODULE_COMPAT_5.12.3)  
  1 perl(POSIX)  
  1 perl(strict)  
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
  2 rpmlib(VersionedDependencies) <= 3.0.3-1
Binary requires Ok.

rpm -q --provides -p ../RPMS/noarch/perl-MDV-Packdrakeng-1.13-1.fc14.noarch.rpm
| sort | uniq -c
  1 perl(MDV::Packdrakeng::zlib)  
  1 perl(MDV::Packdrakeng) = 1.13
  1 perl-MDV-Packdrakeng = 1.13-1.fc14
Binary provides Ok.

$ resolvedeps-f15 ../RPMS/noarch/perl-MDV-Packdrakeng-1.13-1.fc14.noarch.rpm 
Binary dependencies resolvable. Ok.

FIX: Package does not build in F15
(http://koji.fedoraproject.org/koji/taskinfo?taskID=2784614). This is due to
missing BuildRequires for tests.

Othwerwise package is in line with Fedora and perl packaging guidelines.


Please correct all `FIX' prefixed issues, consider fixing `TODO' prefixed
issues, and provide new spec file.
Resolution: Package NOT 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-02-08 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=612796

Fabian Affolter  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 AssignedTo|fab...@bernewireless.net|nob...@fedoraproject.org
   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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675963] Review Request: perl-MDV-Packdrakeng - Simple Archive Extractor/Builder

2011-02-08 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=675963

Petr Pisar  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ppi...@redhat.com
 AssignedTo|nob...@fedoraproject.org|ppi...@redhat.com
   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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 596138] Review Request: nss-gui - A graphical user interface for NSS security databases

2011-02-08 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=596138

--- Comment #20 from Kai Engert (kaie)  2011-02-08 08:03:44 
EST ---
(In reply to comment #19)
> Sorry for taking so long; I have been sick for the past several days.

Hope you're recovering quickly

> APPROVED

Thank you very much

-- 
Configure 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 675963] Review Request: perl-MDV-Packdrakeng - Simple Archive Extractor/Builder

2011-02-08 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=675963

Marcela Mašláňová  changed:

   What|Removed |Added

 Blocks||671389

-- 
Configure 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 675963] New: Review Request: perl-MDV-Packdrakeng - Simple Archive Extractor/Builder

2011-02-08 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-MDV-Packdrakeng - Simple Archive Extractor/Builder

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

   Summary: Review Request: perl-MDV-Packdrakeng - Simple Archive
Extractor/Builder
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


SPEC: http://mmaslano.fedorapeople.org/review/perl-MDV-Packdrakeng.spec
SRPM:
http://mmaslano.fedorapeople.org/review/perl-MDV-Packdrakeng-1.13-1.fc14.src.rpm
Description: 
MDV::Packdrakeng is a simple indexed archive builder and extractor using
standard compression methods.

-- 
Configure 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 675726] Review Request: cdm - Very minimalistic display manager

2011-02-08 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=675726

--- Comment #6 from Peter Lemenkov  2011-02-08 07:42:53 EST 
---
Few notes before review:

* Please, drop commented out lines in the %install section

* Drop the following line as well

install -Dm644 src/xinitrc.skel
$RPM_BUILD_ROOT/%{_docdir}/%{name}-%{version}/xinitrc.example

this file should be installed just as other docs (mark it as %doc in the %files
section).

* The directory %{_datarootdir}/%{name}/ left unowned. Please apply the
following change:

- %{_datarootdir}/%{name}/*
+ %{_datarootdir}/%{name}/

Note - trailing asterisk was 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 668243] Review Request: libqb - An IPC library for high performance servers.

2011-02-08 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=668243

--- Comment #21 from Angus Salkeld  2011-02-08 07:10:00 
EST ---
I have also cleaned up (deleted) the global variables and changed the
include files section to how you suggested.

Here is the new spec & srpm:
http://asalkeld.fedorapeople.org/libqb.spec
http://asalkeld.fedorapeople.org/libqb-0.4.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 675914] Review Request: flush - GTK-based BitTorrent client

2011-02-08 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=675914

--- Comment #4 from Peter Lemenkov  2011-02-08 07:03:43 EST 
---
Unfotrunately it failed to build in Koji. Please, fix this issue.

There are other issues:

1) The directory %{_datadir}/%{name} is left unowned. Please fix the %files
section in the following  way:

- %{_datadir}/%{name}/*
+ %{_datadir}/%{name}/

Note that I stripped trailing asterisk thus claiming ownership on the entire
directory (not only on their contents).

2) The following change *should* be done with patch instead of sed. 

sed -i -e 's|strerror(e.system_error())|EE(e)|g' \
src/create_torrent_dialog.cpp

3) Please, ensure that none of bundled libaries (dbus-c++, libconfig,
libtorrent) are used during building. I suggest you to remove both of then in
the %prep section.

-- 
Configure 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 668243] Review Request: libqb - An IPC library for high performance servers.

2011-02-08 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=668243

--- Comment #20 from Angus Salkeld  2011-02-08 07:00:51 
EST ---
(In reply to comment #17)
> (In reply to comment #16)
> > (In reply to comment #15)
> > > - Did you make sure that there were no dependencies on other headers in 
> > > the
> > > devel
> > >   subpackage? The devel subpackage only requires pkgconfig.
> > 
> > I believe so, we don't need std headers & man do we?
> > Is here a way of checking? I have done rpmlint and koji builds without
> > problems.
> 
> Well, the reason is so that when you use these header files in code, the
> compiler doesn't throw an error because an #include file couldn't be found.
> 
> I ran the following (very very hacky) shell script from /usr/include in an
> extracted version of the RPM:
> 
> for j in $(for i in $(grep -r '#include <' . | \
> sed -e 's/.*#include <\(.*\)>$/\1/'); do find /usr/include | \
> grep $i; done | less); do rpm -qf $j; done | grep -v kernel-headers | \
> grep -v libstdc++-devel | grep -v glibc-headers
> 
> (kernel-headers and glibc-headers are both deps for gcc; libstdc++-devel is a
> dep for gcc-g++, so those don't need to be included.)
> 
> Nothing showed up in the output of that script, so I guess you're good.
> 
> >> - Do you have any other pending package reviews, or have you done any
> >>   informal reviews of other packages?
> 
> > Nope, just this one.
> 
> If you have something else you need to package, I would recommend submitting
> that for review and having me take a look at it. My sponsoring you depends on
> how well I feel you understand the Fedora packaging guidelines. I think you've
> got a pretty good grasp on them, but I generally like to see more than one
> package from any submitter. (Alternatively, you can take any package review, 
> do
> an informal review of it, and link to that bug here.)

Here is another package (sanlock) to review: 
https://bugzilla.redhat.com/show_bug.cgi?id=675947

-- 
Configure 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 675947] New: sanlock - a shared disk lock manager

2011-02-08 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: sanlock - a shared disk lock manager

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

   Summary: sanlock - a shared disk lock manager
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: asalk...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://asalkeld.fedorapeople.org/sanlock.spec
SRPM URL: http://asalkeld.fedorapeople.org/sanlock-1.0-2.fc14.src.rpm

Description: sanlock uses disk paxos to manage leases on shared storage.
Hosts connected to a common SAN can use this to synchronize their
access to the shared disks.

Other Reviews in progress:
https://bugzilla.redhat.com/show_bug.cgi?id=668243


koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2782824

rpmlint -v ./sanlock.spec 
./sanlock.spec: I: checking-url
https://fedorahosted.org/releases/s/a/sanlock/sanlock-1.0.tar.gz (timeout 10
seconds)
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
Configure 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 675914] Review Request: flush - GTK-based BitTorrent client

2011-02-08 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=675914

--- Comment #3 from Peter Lemenkov  2011-02-08 06:44:00 EST 
---
Koji scratch build for Rawhide (still wip at the moment of writing this):

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

-- 
Configure 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 675914] Review Request: flush - GTK-based BitTorrent client

2011-02-08 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=675914

Peter Lemenkov  changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |

--- Comment #2 from Peter Lemenkov  2011-02-08 06:40:24 EST 
---
Unblocking FE-NEEDSPONSOR - I just sponsored Oksana.

-- 
Configure 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 675726] Review Request: cdm - Very minimalistic display manager

2011-02-08 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=675726

Peter Lemenkov  changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |

--- Comment #5 from Peter Lemenkov  2011-02-08 06:39:55 EST 
---
Unblocking FE-NEEDSPONSOR - I just sponsored Mikhail.

-- 
Configure 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 675726] Review Request: cdm - Very minimalistic display manager

2011-02-08 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=675726

Peter Lemenkov  changed:

   What|Removed |Added

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

--- Comment #4 from Peter Lemenkov  2011-02-08 06:28:50 EST 
---
I'll review it

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


[Bug 675914] Review Request: flush - GTK-based BitTorrent client

2011-02-08 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=675914

Peter Lemenkov  changed:

   What|Removed |Added

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

--- Comment #1 from Peter Lemenkov  2011-02-08 06:28:18 EST 
---
I'll review it

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


[Bug 659082] Review Request: redland-bindings - language bindings for redland

2011-02-08 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=659082

--- Comment #9 from Thomas Vander Stichele  2011-02-08 
06:06:47 EST ---
New version at
http://thomas.apestaart.org/download/pkg/fedora-14-i386-updates/redland-bindings-1.0.11.1-5.fc14/

I also took the time to integrate the commited versions of my patches because
upstream commited fixes - should make it easier to update for the next release.
 As a result the package now needs autotools to build, which is temporary.

I think this resolves all problems you had with the review, so I hope you can
approve this one!

-- 
Configure 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-08 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 #11 from Hans de Goede  2011-02-08 04:10:44 
EST ---
Hi,

(In reply to comment #10)
> (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)
> 

The src.rpm is what eventually will get imported into the buildsystem / package
git. By providing it, you allow the reviewer to verify the tarbal as it will
get imported. The reviewer can then verify that this matches the one provided
by upstream.

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

Thanks! Looks good now. I'll approve this as soon as I'm ready to sponsor you
(once the libmtag-python package is also reviewed).

> 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
> 

Thanks, please file a separate review bug for this, using the usual template
and let me know the bz number.

Regards,

Hans

-- 
Configure 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 675914] Review Request: flush - GTK-based BitTorrent client

2011-02-08 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=675914

Oksana  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
Configure 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 675914] New: Review Request: flush - GTK-based BitTorrent client

2011-02-08 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: flush - GTK-based BitTorrent client

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

   Summary: Review Request: flush - GTK-based BitTorrent client
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: oksana.kurysh...@vdel.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.ossportal.ru/sites/default/files/packages/flush.spec
SRPM URL:
http://www.ossportal.ru/sites/default/files/packages/flush-0.9.9-1.fc14.src.rpm
Description: 
Flush - A GTK-based BitTorrent client.
Features:
 * Controlling running instance by command line interface.
 * Running many instances with different configs from the same user.
 * Automatic copying finished downloads to specified directory.
 * Setting custom download path for each file of the torrent.
 * Ability to choose torrent file's character set encoding.
 * Automatic torrents loading from specified directory.
 * Automatic pausing and removing old torrents.
 * Temporary pausing and resuming torrents.
 * Overall and current session statistics.
 * Creating your own torrent files.
 * IP filter.

-- 
Configure 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-08 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

--- Comment #20 from Mathieu Bridon  2011-02-08 
03:39:26 EST ---
(In reply to comment #19)
> If Nicholas wants to produce an updated package I'll finish the review (since
> it's still assigned to me after all this time).

I took over the submission from Nicolas (see comment 16) and produced an
updated package (see comment 17).

The srpm in comment 17 is the one I scratched built in Koji to show that it now
uses the system libev (see comment 18).

-- 
Configure 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