[Bug 789249] New: Review Request: Horizontal or vertical bar-graph audio levels meter

[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: Horizontal or vertical bar-graph audio levels meter

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

   Summary: Review Request: Horizontal or vertical bar-graph audio
levels meter
   Product: Fedora
   Version: 16
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: brendan.jones...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


jkmeter is a horizontal or vertical bar-graph level
meter based on the ideas of mastering guru Bob Katz.
See  and
follow the links on 'level practices'.

This is the type of meter you want for live recording,
mixing and mastering. It probably makes no sense to
use it on all tracks of a DAW, where keeping digital
level within limits is the main purpose of metering.

This is another package being moved from the Planet CCRMA repo.

SRPM: http://bsjones.fedorapeople.org/jkmeter-0.6.1-2.fc16.src.rpm
SPEC: http://bsjones.fedorapeople.org/jkmeter.spec

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

[Bug 788006] Review Request: ghc-temporary - portable temp file library

[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=788006

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 788006] Review Request: ghc-temporary - portable temp file library

[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=788006

--- Comment #5 from Fedora Update System  2012-02-10 
02:26:58 EST ---
ghc-temporary-1.1.2.3-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/ghc-temporary-1.1.2.3-1.fc16

-- 
Configure 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 788006] Review Request: ghc-temporary - portable temp file library

[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=788006

--- Comment #6 from Fedora Update System  2012-02-10 
02:27:07 EST ---
ghc-temporary-1.1.2.3-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/ghc-temporary-1.1.2.3-1.fc15

-- 
Configure 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 789240] New: Reveiw Request: freqtweak - Realtime audio frequency spectral manipulation

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Reveiw Request: freqtweak - Realtime audio frequency spectral 
manipulation

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

   Summary: Reveiw Request: freqtweak - Realtime audio frequency
spectral manipulation
   Product: Fedora
   Version: 16
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: brendan.jones...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


SRPM: http://bsjones.fedorapeople.org/freqtweak-0.7.2-2.fc16.src.rpm
SPEC: http://bsjones.fedorapeople.org/freqtweak.spec

FreqTweak is a tool for FFT-based realtime audio spectral manipulation
and display. It provides several algorithms for processing audio data
in the frequency domain and a highly interactive GUI to manipulate the
associated filters for each. It also provides high-resolution spectral
displays in the form of scrolling-raster spectragrams and energy vs
frequency plots displaying both pre- and post-processed spectra.

this is another package being moved from Planet CCRMA into Fedora.

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

[Bug 789240] Review Request: freqtweak - Realtime audio frequency spectral manipulation

[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=789240

Brendan Jones  changed:

   What|Removed |Added

Summary|Reveiw Request: freqtweak - |Review Request: freqtweak -
   |Realtime audio frequency|Realtime audio frequency
   |spectral manipulation   |spectral manipulation

-- 
Configure 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 788815] Review Request: python-ZSI - python Zolera Soap Infrastructure

[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=788815

--- Comment #3 from Haïkel Guémar  2012-02-10 02:08:56 
EST ---
Awesome !

i've mistakenly pasted the wrong download url in my review (i'm sorry, it's my
fault), it should be:
http://downloads.sourceforge.net/project/pywebsvcs/ZSI/ZSI-%{version}/ZSI-%{version}.tar.gz

A quick test:
urlgrabber
http://downloads.sourceforge.net/project/pywebsvcs/ZSI/ZSI-2.0/ZSI-2.0.tar.gz

The mock build fails due to tests
http://koji.fedoraproject.org/koji/taskinfo?taskID=3777577

Since tests doesn't modify their sys.path, they fail to find the ZSI module
(which is not installed yet). To fix that, you can add the ZSI directory to
PYTHONPATH environment variable. PyXML will also be needed as a BR.
I have one more failing tests (the TCtimes one).

Here's a scratch build of a slightly modified version of your package that
builds inside mock.
http://koji.fedoraproject.org/koji/taskinfo?taskID=3777593

As soon as you fix the download url, i'll approve this package. Save that, it's
all green for me.

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

[Bug 579389] Review Request: perl-HTTP-Parser-XS - A fast, primitive HTTP request parser

[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=579389

Emmanuel Seyman  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Emmanuel Seyman  
2012-02-10 01:43:39 EST ---
I knew I was going to do it wrong.

Package Change Request
==
Package Name: perl-HTTP-Parser-XS
New Branches: el6
Owners: eseyman bochecha
InitialCC: perl-sig

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

[Bug 756448] Review Request: mingw-ftplib - MinGW package for ftplib

[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=756448

--- Comment #9 from Fedora Update System  2012-02-10 
00:49:09 EST ---
mingw-ftplib-3.1-4.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/mingw-ftplib-3.1-4.fc16

-- 
Configure 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 756448] Review Request: mingw-ftplib - MinGW package for ftplib

[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=756448

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 787561] Review Request: torsocks - A transparent socks proxy for use with tor

[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=787561

--- Comment #4 from Eric Christensen  2012-02-10 
00:28:21 EST ---
Spec URL: http://sparks.fedorapeople.org/Packages/torsocks.spec

SRPM URL: http://sparks.fedorapeople.org/Packages/torsocks-1.2-1.fc16.src.rpm

Got all that other stuff cleaned up.

-- 
Configure 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 789212] New: Review Request: ibus-fep - IBus client for text terminals

[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: ibus-fep - IBus client for text terminals

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

   Summary: Review Request: ibus-fep - IBus client for text
terminals
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: du...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Depends on: 787128
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://ueno.fedorapeople.org/ibus-fep/ibus-fep.spec
SRPM URL: http://ueno.fedorapeople.org/ibus-fep/ibus-fep-1.4.1-1.fc16.src.rpm
Description:
ibus-fep is an IBus client that runs on text terminals such as xterm.

-- 
Configure 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 508817] Review Request: Limesurvey - An open source survey application

[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=508817

Bug 508817 depends on bug 549590, which changed state.

Bug 549590 Summary: Review Request: php-pChart - A PHP class to build charts.
https://bugzilla.redhat.com/show_bug.cgi?id=549590

   What|Old Value   |New Value

 Resolution||WONTFIX
 Status|ASSIGNED|CLOSED

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

[Bug 549590] Review Request: php-pChart - A PHP class to build charts.

[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=549590

Eric Christensen  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
Last Closed|2010-09-05 09:31:36 |2012-02-10 00: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 787128] Review Request: libfep - Library to implement FEP (front end processor) on ANSI terminals

[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=787128

Daiki Ueno  changed:

   What|Removed |Added

 Blocks||789212

-- 
Configure 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 682489] Review Request: drupal6-acidfree - A full-featured media management system for Drupal

[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=682489

Eric Christensen  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
   Flag|needinfo?(eric@christensenp |
   |lace.us),   |
   |needinfo?(eric@christensenp |
   |lace.us)|
Last Closed||2012-02-10 00:20:35

-- 
Configure 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 508817] Review Request: Limesurvey - An open source survey application

[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=508817

Eric Christensen  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
Last Closed||2012-02-10 00:17:23

-- 
Configure 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 721179] Review Request: rubygem-extlib - Support library for DataMapper and Merb

[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=721179

Shawn Starr  changed:

   What|Removed |Added

   Priority|medium  |urgent
  Status Whiteboard||opennebula-dep
   Severity|medium  |urgent

--- Comment #35 from Shawn Starr  2012-02-10 00:08:52 
EST ---
Please find packages here: updated for Ruby packaging specifications and an
additional rspec compat fix in test.

SRPM:
http://fedorapeople.org/~spstarr/packages/rubygem-extlib-0.9.15-4.fc17.src.rpm

SPEC:
http://fedorapeople.org/~spstarr/packages/rubygem-extlib.spec

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

[Bug 728051] Review Request: rubygem-stringex - Extensions to ruby String class

[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=728051

--- Comment #2 from Shawn Starr  2012-02-09 23:46:19 
EST ---
Updated package new version 1.3.0 fixes for Ruby package guidelines

Spec URL: http://fedorapeople.org/~spstarr/packages/rubygem-stringex.spec
SRPM URL:
http://fedorapeople.org/~spstarr/packages/rubygem-stringex-1.3.0-1.fc17.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 784958] Review Request: ktp-text-ui - Telepathy text chat handler

[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=784958

--- Comment #16 from Kevin Kofler  2012-02-09 23:19:52 
EST ---
Ah, that's the simkete style, the other style shipped in the package. That one
is clearly MIT. But it's unrelated to the Fadomatik code in the renkoo style,
which is missing its copyright notice and at least a reference to the license
(but since it's the MIT license, a full copy of the exact license is better,
it's very short and there are subtle differences in wording between the
variants).

-- 
Configure 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 719854] Review Request: rubygem-xmlparser-0.6.81-1 - Ruby bindings to the Expat XML parsing library

[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=719854

Shawn Starr  changed:

   What|Removed |Added

  Status Whiteboard||opennebula-dep

-- 
Configure 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 719854] Review Request: rubygem-xmlparser-0.6.81-1 - Ruby bindings to the Expat XML parsing library

[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=719854

--- Comment #42 from Shawn Starr  2012-02-09 23:14:28 
EST ---
Can I get an update please? This is now urgent for OpenNebula and Fedora 17
timeframes.

-- 
Configure 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 784958] Review Request: ktp-text-ui - Telepathy text chat handler

[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=784958

--- Comment #15 from nucleo  2012-02-09 23:13:07 EST 
---
https://projects.kde.org/projects/extragear/network/telepathy/ktp-text-ui/repository/revisions/master/entry/data/styles/simkete/Contents/README

-- 
Configure 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 784958] Review Request: ktp-text-ui - Telepathy text chat handler

[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=784958

--- Comment #14 from Kevin Kofler  2012-02-09 23:07:24 
EST ---
Where did you find that license? The license I see is this:
https://projects.kde.org/projects/extragear/network/telepathy/ktp-text-ui/repository/revisions/master/entry/data/styles/renkoo.AdiumMessageStyle/Contents/Resources/Renkoo
LICENSE.txt
Either way, the fadomatic.js code needs to be properly attributed to its true
author.

-- 
Configure 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 588428] Review Request: rubygem-addressable - Improved URI/URL handling

[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=588428

--- Comment #28 from Shawn Starr  2012-02-09 23:07:59 
EST ---
I have an updated package for review:

* Koji scratch build pass 

SPEC: http://fedorapeople.org/~spstarr/packages/rubygem-addressable.spec

SRPM:
http://fedorapeople.org/~spstarr/packages/rubygem-addressable-2.2.6-3.fc17.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 789192] Package Change Request - orc 'The Oil Run-Time Compiler' - SCM Request For EL6

[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=789192

Randall "Randy" Berry  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 784952] Review Request: ktp-filetransfer-handler - Telepathy file transfer handler

[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=784952

nucleo  changed:

   What|Removed |Added

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

--- Comment #5 from nucleo  2012-02-09 22:53:49 EST ---
Hope this time not too short.

MUST Items:
+ rpmlint output
  $ $ rpmlint ktp-filetransfer-handler-0.3.0-2.fc16.i686.rpm
ktp-filetransfer-handler-0.3.0-2.fc16.src.rpm
ktp-filetransfer-handler-debuginfo-0.3.0-2.fc16.i686.rpm
  3 packages and 0 specfiles checked; 0 errors, 0 warnings.
+ named and versioned according to the Package Naming Guidelines.
  Package name match the upstream tarball name
ktp-filetransfer-handler-0.3.0.tar.bz2
+ spec file name ktp-filetransfer-handler.spec matches base package name
+ complies with all the legal guidelines:
  + License: GPLv2+ valid, matches actual license (main.cpp GPLv2+, some
headers LGPLv2+)
  + No known patent problems
  + No emulator, no firmware, no binary-only or prebuilt components
+ COPYING (GNU GENERAL PUBLIC LICENSE Version 2) packaged as %doc
+ source matches upstream:
  MD5: 8e0f2929e7bdeb0c9c364007f20ddbd0  ktp-filetransfer-handler-0.3.0.tar.bz2
  SHA1: dd409ecc375fa082165d8b9f539736894bd052da 
ktp-filetransfer-handler-0.3.0.tar.bz2
  SHA256: 813d09ce5e5cd09e663d48e88fda974f4d222a3cf9862010788b639a294304d7 
ktp-filetransfer-handler-0.3.0.tar.bz2
+ builds on at least one arch
  build from mock is in F16 kde-unstable repo
+ no known non-working arches, so no ExcludeArch needed
+ no missing BuildRequires (builds in mock)
+ locales are handled properly by using %find_lang %{name} --all-name
--with-kde macro
+ ldconfig call not needed (no shared libraries in dynamic linker's default
paths)
+ no duplicated system libraries
+ package not relocatable (no Prefix tag)
+ directory ownership correct (doesn't own directories owned by another
package, owns all package-specific directories)
+ no duplicate files in %files
+ permissions correct, %defattr(-,root,root,-) not needed now, executables have
executable permissions
+ macros used where possible (%{name}, %{version}, %{buildroot}, %{cmake_kde4},
%{_target_platform}, %{cmake_kde4}, %{_kde4_libexecdir}, %{_datadir})
+ no non-code content (only binary and texts files that binary needs)
+ no large documentation files, so no -doc package needed
+ no %doc files required at runtime
+ no header files which would need to be in a -devel subpackage
+ no static libraries, so no -static package needed
+ no devel symlinks which would need to be in a -devel subpackage
+ devel packages must require the base package (no -devel package)
+ no .la files
+ no .desktop file needed in /usr/share/applications for this KDE Telepathy
internal module
+ desktop-file-validate call not needed
+ all filenames are valid UTF-8
+ other packaging guidelines:
  + complies with the Filesystem Hierarchy Standard (all files in
%{_kde4_libexecdir} and %{_datadir})
  - proper changelog, tags, BuildRequires, Summary, Description
  + no non-UTF-8 characters
  + all relevant documentation included as %doc (COPYING AUTHORS NEWS README
TODO)
  + RPM_OPT_FLAGS are used in %{cmake_kde4} macro
  + debuginfo package is valid (contains stripped symbols from ELF binary and
source code related to it)
  + no rpaths (no check-rpaths error)
  + no configuration files, so %config guideline doesn't apply
  + no init scripts, so init script guideline doesn't apply
  + timestamps are preserved
  + %{?_smp_mflags} used
  + not a web application, so web application guideline doesn't apply
  + no conflicts

SHOULD Items:
+ license already included upstream
+ no translations for description and summary provided by upstream
+ package builds in mock (built for kde-unstable)
- successfully tested the package functionality (no testing yet)
+ scriptlets are sane (no scriptlets needed)
+ subpackages other than devel should require the base package using a fully
versioned dependency (no subpackages)
+ no .pc files, so "placement of .pc files" is irrelevant
+ no file dependencies
+ no binaries/scripts that needs man pages

So please add some useful description from README.

-- 
Configure 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 789192] New: Package Change Request - orc 'The Oil Run-Time Compiler' - SCM Request For EL6

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Package Change Request - orc 'The Oil Run-Time Compiler' - SCM Request 
For EL6

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

   Summary: Package Change Request - orc 'The Oil Run-Time
Compiler' - SCM Request For EL6
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: randyn3...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Package Change Request
==
Package Name: orc
New Branches: el6 
Owners: dp67


This is one of the packages needed for 'gnuradio' one of the foundation
packages for Ham Radio applications. Sorry for not posting this request to the
original package review but I could not find it. If the original review is
necessary and someone could point me to it I'd much appreciate 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 789192] Package Change Request - orc 'The Oil Run-Time Compiler' - SCM Request For EL6

[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=789192

Randall "Randy" Berry  changed:

   What|Removed |Added

   Flag||fedora-review?

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

[Bug 787128] Review Request: libfep - Library to implement FEP (front end processor) on ANSI terminals

[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=787128

--- Comment #3 from Daiki Ueno  2012-02-09 22:17:52 EST ---
Thanks for taking this review.

I've put the updated package at:
Spec URL: http://ueno.fedorapeople.org/libfep/libfep.spec
SRPM URL: http://ueno.fedorapeople.org/libfep/libfep-0.0.4-1.fc17.src.rpm

It is rebased against the latest upstream tarball, which now includes man
pages.

-- 
Configure 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 788815] Review Request: python-ZSI - python Zolera Soap Infrastructure

[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=788815

--- Comment #2 from Tim Fenn  2012-02-09 21:44:13 EST ---
(In reply to comment #1)
> 
> Notes:
> * we can safely ignore rpmlint output since python-ZSI has existing EPEL5
> branch, the same for spelling errors.
> * fix upstream download url (the one in the spec is not working)
> http://belnet.dl.sourceforge.net/sourceforge/pywebsvcs/ZSI-2.0.tar.gz

Done.

> * prefer macro style over shell style s/$RPM_BUILD_ROOT/%{buildroot}/

Fixed.

> * you should ask upstream to include license text in source package (if they
> still answer)

Will do - I'll also ask re. the failed test (see below)

> * Though i won't consider this as a blocker, i strongly recommend you to run
> tests in %check section. All the more important since ZSI upstream is not
> responsive. For the few tests, that fails, i suggests you that you override
> tests exit code with "|| :". 
> That will help you monitoring ZSI health state during each release.

Done - one test fails, which I'll mention to upstream (there is also an alpha
2.1 release, it may be fixed there?)

> * i need to confirm that you're ok with the fact that ZSI is not actively
> maintained by upstream ? That means that you might have to fix by yourself
> FTBFS ou issues reported by users.
> 

It seems like there has been some work on the code in the past few months? 
I'll look into this further and try to contact upstream.  I'm mostly getting
this out to avoid breaking a dependency for another package I maintain (apbs) -
if upstream is silent here, I'll discuss with upstream of apbs about switching
to a different SOAP library.  If both of these approaches fail, I'll do my best
to handle problems that arise.

> 
> As long as you do the required changes above and you confirm that you will
> maintain it knowing that upstream might not be helpful, i may approve this
> package.

I understand - and thanks for the help getting this review done promptly!

New files:

Spec URL: https://sites.google.com/site/timfenn/python-ZSI.spec
SRPM URL: https://sites.google.com/site/timfenn/python-ZSI-2.0-11.fc16.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 756448] Review Request: mingw-ftplib - MinGW package for ftplib

[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=756448

--- Comment #8 from Jon Ciesla  2012-02-09 21:32:21 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 755093] Review Request: mactel-boot - boot tools for Intel Apple hardware

[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=755093

--- Comment #38 from Jon Ciesla  2012-02-09 21:31:42 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 579389] Review Request: perl-HTTP-Parser-XS - A fast, primitive HTTP request parser

[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=579389

--- Comment #6 from Jon Ciesla  2012-02-09 21:34:59 EST 
---
Love to, but it needs to be a Package Change request, not New Package.

https://fedoraproject.org/wiki/Package_SCM_admin_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 787858] Review Request: bashmount - a menu-driven bash script for mounting removable media

[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=787858

--- Comment #11 from Jon Ciesla  2012-02-09 21:33:02 EST 
---
Git done (by process-git-requests).

Added f17.

-- 
Configure 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 789108] Review Request: python-qserve - Job queue server

[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=789108

--- Comment #3 from Jon Ciesla  2012-02-09 21:33: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 784958] Review Request: ktp-text-ui - Telepathy text chat handler

[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=784958

--- Comment #13 from nucleo  2012-02-09 21:10:49 EST 
---
Created attachment 560743
  --> https://bugzilla.redhat.com/attachment.cgi?id=560743
copyright file from ubuntu deb

MIT License also used for files in data/styles/simkete/, license text file 
data/styles/simkete/Contents/README (can be added in %dos) so maybe it is
enough for us?

This Adium style is released under MIT/X11 license:

Copyright (c) 2009,2011 Matěj Cepl 

Permission is hereby granted, free of charge, to any person obtaining a copy of
this software and associated documentation files (the "Software"), to deal in
the Software without restriction, including without limitation the rights to
use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies
of
the Software, and to permit persons to whom the Software is furnished to do so,
subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS
FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Or for modified Footer.html needed yet one MIT license with fadomatic.js
author?

-- 
Configure 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 756448] Review Request: mingw-ftplib - MinGW package for ftplib

[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=756448

Michael Cronenworth  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Michael Cronenworth  2012-02-09 20:47:21 
EST ---
New Package SCM Request
===
Package Name: mingw-ftplib
Short Description: MinGW Library of FTP routines
Owners: mooninite
Branches: f16 f17
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 756448] Review Request: mingw-ftplib - MinGW package for ftplib

[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=756448

--- Comment #6 from Michael Cronenworth  2012-02-09 20:46:28 
EST ---
I've added a copying file. Interesting Fact: The native Fedora package does not
ship 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 784958] Review Request: ktp-text-ui - Telepathy text chat handler

[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=784958

--- Comment #12 from nucleo  2012-02-09 20:35:27 EST 
---
Looks like
https://projects.kde.org/projects/extragear/network/telepathy/ktp-text-ui/repository/revisions/master/entry/data/styles/renkoo.AdiumMessageStyle/Contents/Resources/Footer.html
is not exactly the same as
https://github.com/phl/Fadomatic/blob/master/fadomatic.js 
For example in fadomatic.js no functions initStyle and scrollToBottom  from
Footer.html.

-- 
Configure 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 784958] Review Request: ktp-text-ui - Telepathy text chat handler

[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=784958

nucleo  changed:

   What|Removed |Added

 CC||alekc...@googlemail.com

--- Comment #11 from nucleo  2012-02-09 20:26:16 EST 
---
There is error:

cp: cannot stat
`./data/styles/renkoo.AdiumMessageStyle/Contents/Resources/Renkoo': No such
file or directory
cp: cannot stat `LICENSE.txt': No such file or directory

This should fix it:

%doc ./data/styles/renkoo.AdiumMessageStyle/Contents/Resources/Renkoo\
LICENSE.txt

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

[Bug 785436] Review Request: php-horde-Horde-Exception - Horde Exception Handler

[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=785436

--- Comment #7 from Nick Bebout  2012-02-09 20:19:40 
EST ---
I made the changes you suggested via email.

-- 
Configure 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 785436] Review Request: php-horde-Horde-Exception - Horde Exception Handler

[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=785436

Bug 785436 depends on bug 785424, which changed state.

Bug 785424 Summary: Review Request: php-channel-horde - Adds pear.horde.org 
channel to PEAR
https://bugzilla.redhat.com/show_bug.cgi?id=785424

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #6 from Nick Bebout  2012-02-09 20:19:22 
EST ---
I think this one is ready to 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 784952] Review Request: ktp-filetransfer-handler - Telepathy file transfer handler

[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=784952

nucleo  changed:

   What|Removed |Added

 CC||alekc...@googlemail.com

--- Comment #4 from nucleo  2012-02-09 20:03:01 EST ---
Part of README file can be added in %description:

Telepathy-KDE file transfer handler. It basically does two thigs:
 - Send files to your contact
 - Receive files from your contact

-- 
Configure 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 787878] Review Request: mingw-icu - MinGW compilation of International Components for Unicode Tools

[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=787878

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #12 from Fedora Update System  
2012-02-09 19:56:42 EST ---
mingw-icu-4.8.1.1-2.fc16 has been pushed to the Fedora 16 testing repository.

-- 
Configure 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 781775] Review Request: sevmgr - C++ Simulation-Oriented Discrete Event Management Library

[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=781775

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||sevmgr-0.2.0-1.fc16
 Resolution||ERRATA
Last Closed||2012-02-09 19:57:39

--- Comment #9 from Fedora Update System  2012-02-09 
19:57:39 EST ---
sevmgr-0.2.0-1.fc16 has been pushed to the Fedora 16 stable repository.

-- 
Configure 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 742729] Review Request: php-pecl-mysqlnd-ms - A replication and load balancing plugin for mysqlnd

[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=742729

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #13 from Fedora Update System  
2012-02-09 19:56:35 EST ---
php-pecl-mysqlnd-ms-1.2.2-1.fc16 has been pushed to the Fedora 16 testing
repository.

-- 
Configure 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 786668] Review Request: python-sqlite3dbm - SQLite-backed dictionary conforming to the dbm interface

[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=786668

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #11 from Fedora Update System  
2012-02-09 19:55:26 EST ---
python-sqlite3dbm-0.1.4-2.fc16 has been pushed to the Fedora 16 testing
repository.

-- 
Configure 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 785424] Review Request: php-channel-horde - Adds pear.horde.org channel to PEAR

[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=785424

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||php-channel-horde-1.0-1.fc1
   ||6
 Resolution||ERRATA
Last Closed||2012-02-09 19:53:54

--- Comment #14 from Fedora Update System  
2012-02-09 19:53:54 EST ---
php-channel-horde-1.0-1.fc16 has been pushed to the Fedora 16 stable
repository.

-- 
Configure 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 736262] Review Request: proxyfuzz - man-in-the-middle non-deterministic network fuzzer

[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=736262

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||proxyfuzz-20110923-1.fc16
 Resolution||ERRATA
Last Closed||2012-02-09 19:54:22

--- Comment #15 from Fedora Update System  
2012-02-09 19:54:22 EST ---
proxyfuzz-20110923-1.fc16 has been pushed to the Fedora 16 stable repository.

-- 
Configure 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 781775] Review Request: sevmgr - C++ Simulation-Oriented Discrete Event Management Library

[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=781775

--- Comment #8 from Fedora Update System  2012-02-09 
19:53:16 EST ---
sevmgr-0.2.0-1.fc15 has been pushed to the Fedora 15 stable repository.

-- 
Configure 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 783061] Review Request: omniORB - A robust high performance CORBA ORB for C++ and Python

[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=783061

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||omniORB-4.1.6-1.fc15
 Resolution||ERRATA
Last Closed||2012-02-09 19:47:02

--- Comment #9 from Fedora Update System  2012-02-09 
19:47:02 EST ---
omniORB-4.1.6-1.fc15 has been pushed to the Fedora 15 stable repository.

-- 
Configure 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 783061] Review Request: omniORB - A robust high performance CORBA ORB for C++ and Python

[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=783061

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|omniORB-4.1.6-1.fc15|omniORB-4.1.6-1.fc16

--- Comment #10 from Fedora Update System  
2012-02-09 19:48:44 EST ---
omniORB-4.1.6-1.fc16 has been pushed to the Fedora 16 stable repository.

-- 
Configure 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 784954] Review Request: ktp-kded-integration-module - KDE integration for telepathy

[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=784954

nucleo  changed:

   What|Removed |Added

 CC||alekc...@googlemail.com

--- Comment #6 from nucleo  2012-02-09 19:41:13 EST ---
Right URL:
https://projects.kde.org/projects/extragear/network/telepathy/ktp-kded-module

-- 
Configure 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 784946] Review Request: ktp-approver - KDE Channel Approver for Telepathy

[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=784946

nucleo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||alekc...@googlemail.com
 AssignedTo|nob...@fedoraproject.org|alekc...@googlemail.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 784944] Review Request: ktp-accounts-kcm - KDE Configuration Module for Telepathy Instant Messaging Accounts

[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=784944

nucleo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
Configure 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 784956] Review Request: ktp-send-file - A File manager plugin to launch file transfer jobs

[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=784956

nucleo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||alekc...@googlemail.com
 AssignedTo|nob...@fedoraproject.org|alekc...@googlemail.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 784951] Review Request: ktp-contact-list - Telepathy contact list application

[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=784951

nucleo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||alekc...@googlemail.com
 AssignedTo|nob...@fedoraproject.org|alekc...@googlemail.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 784955] Review Request: ktp-presence-applet - Plasma applet for managing your Telepathy account presence

[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=784955

nucleo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||alekc...@googlemail.com
 AssignedTo|nob...@fedoraproject.org|alekc...@googlemail.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 784949] Review Request: ktp-auth-handler - Provide UI/KWallet Integration

[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=784949

nucleo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||alekc...@googlemail.com
 AssignedTo|nob...@fedoraproject.org|alekc...@googlemail.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 738556] Review Request: gogoc - IPv6 TSP client for gogo6

[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=738556

--- Comment #15 from shrek-m  2012-02-09 18:56:37 EST ---
Thanks for your work, gogoc-1.2-8 SRPM works great under fc16  :)

gogoc-1.2-8.fc16.i686.rpm
gogoc-debuginfo-1.2-8.fc16.i686.rpm


i have absolutely no knowledge with rpm-packaging :(
would it be useful if the radvd-maintainers could be sponsors or
co-maintainers, if they have time ?

# rpm -q -changelog radvd
* Mo Okt 10 2011 Jiri Skala  - 1.8.2-2
- fixes CVE-2011-3602
[...]
* Mi Jan 13 2010 Jan Gorig  - 1.5-1
- updated do latest upstream version
- fixed #554125 - added error message
[...]


 gogoc has a radvd dependency 
[root@macmini ~]# rpm -e --nodeps radvd
[root@macmini ~]# LANG=C rpm -Uvh rpmbuild/RPMS/i686/gogoc-1.2-8.fc16.i686.rpm 
error: Failed dependencies:
 radvd is needed by gogoc-1.2-8.fc16.i686


oops, is this correct that gogoc has no dependencies in yum if it is installed
with rpm ?

[root@macmini ~]# LANG=C yum deplist gogoc
Loaded plugins: refresh-packagekit, security, upgrade-helper
Finding dependencies: 
[root@macmini ~]#
/



ok, no sponsor yet :(
if this package would be in "rawhide / testing" what would i do ?


rpmsave is only created if gogoc.conf was changed, this is correct.
# LANG=C rpm -e gogoc
warning: /etc/gogoc/gogoc.conf saved as /etc/gogoc/gogoc.conf.rpmsave



a few remarks as a user and admin:


0)
a gogoc-package in EPEL with an additional SysV init script for
RHEL5 would be ok until 2017/2020
RHEL6 would be ok until 2020/2023
RHEL7 should have systemd too ?!
http://en.wikipedia.org/wiki/Red_Hat_Enterprise_Linux



1)
my understanding is that you should add  "gcc-c++" ?
BuildRequires:  kernel-headers, openssl-devel, systemd-units, gcc-c++

 gcc-c++ was not installed on my fc16 
make[1]: Leaving directory
`/root/rpmbuild/BUILD/gogoc-1.2/gogoc-1_2-RELEASE/gogoc-pal'
Building gogoc-config module ...
make[1]: Entering directory
`/root/rpmbuild/BUILD/gogoc-1.2/gogoc-1_2-RELEASE/gogoc-config'
mkdir -p objs
g++ -O2 -I. -I../gogoc-pal/out_inc -I../gogoc-pal/defs -Wall -D_REENTRANT
-DNDEBUG   -c -o objs/namevalueparser.o src/namevalueparser.cc
make[1]: g++: Command not found
make[1]: *** [objs/namevalueparser.o] Error 127
make[1]: Leaving directory
`/root/rpmbuild/BUILD/gogoc-1.2/gogoc-1_2-RELEASE/gogoc-config'
make: *** [check-gogoc-config] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.jfjLA8 (%build)
/

# yum list gcc*
# yum install -y gcc-c++

and rpmbuild is ok.



2)
oops, debian-patches (why?) in the changelog but not in SOURCES ?
or is this i.e. the  "lp418176_client_v4.patch" ?
"Author: Philippe Gauthier 
Description: Fix crash on 64-bit architectures occuring when the client_v4
option is not set to "auto"."


[root@macmini ~]# rpm -q --changelog gogoc
* Di Sep 20 2011 Juan Orti Alcaine  - 1.2-8
- fixed debug symbols

* Mo Sep 19 2011 Juan Orti Alcaine  - 1.2-7
- spec file clean up
- systemd service fix

* Mi Sep 14 2011 Juan Orti Alcaine  - 1.2-6
- add Debian patches
- add systemd service
- license changed to BSD

[root@macmini ~]# ls  rpmbuild/SOURCES
gogoc-1.2-debug_info.patch
gogoc-1.2-dirpath.patch
gogoc-1.2-gcc_4.6_compile_fix.patch
gogoc-1.2-gogoc_conf_5.patch
gogoc-1.2-kill_radvd.patch
gogoc-1.2-lp418176_client_v4.patch
gogoc-1.2-mkdir.patch
gogoc-1_2-RELEASE.tar.gz
gogoc-1.2-selinux_fix.patch
gogoc.service



3)
- i do not know if this is the correct place for samples but it would be great
if you could move the original  "gogoc.conf.sample"  into
/usr/share/doc/gogoc-1.2/
it is always useful if a admin should need a new /etc/gogoc/gogoc.conf  ;)

- i do not know if the linux.sh should be under /etc/gogoc/template/  or under
/usr/share/doc/gogoc-1.2/


a) your rpm file list
[root@macmini ~]# rpm -ql gogoc
/etc/gogoc
/etc/gogoc/gogoc.conf
/etc/gogoc/template
/etc/gogoc/template/linux.sh
/etc/sysconfig/gogoc
/lib/systemd/system/gogoc.service
/usr/bin/gogoc
/usr/share/doc/gogoc-1.2
/usr/share/doc/gogoc-1.2/CLIENT-LICENSE.TXT
/usr/share/doc/gogoc-1.2/README
/usr/share/gogoc
/usr/share/gogoc/template
/usr/share/gogoc/template/README
/usr/share/gogoc/template/linux.sh
/usr/share/man/man5/gogoc.conf.5.gz
/usr/share/man/man8/gogoc.8.gz
/var/lib/gogoc
/var/lib/gogoc/gogockeys.pub
/var/lib/gogoc/tsp-broker-list.txt
/var/lib/gogoc/tsp-last-server.txt
/var/log/gogoc
/var/run/gogoc
/var/run/gogoc/gogoc-rtadvd.conf


b) vs. gogo6-sourcecode - `make`
[root@srv01 ~]# ls  /usr/local/gogoc/ /usr/local/gogoc/bin/
/usr/local/gogoc/man/ /usr/local/gogoc/template/

/usr/local/gogoc/:
bin  gogoc-rtadvd.conf  man  template

/usr/local/gogoc/bin/:
gogoc   gogoc.conf.sample  tsp-broker-list.txt
gogoc.conf  gogockeys.pub  tsp-last-server.txt

/usr/local/gogoc/man/:
man5  man8

/usr/local/gogoc/template/:
linux.sh  README



4)
CAP_SYS_MODULE (deprecated)
i do not know if the warnings in /var/log/messages are a gogoc pr

[Bug 789108] Review Request: python-qserve - Job queue server

[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=789108

Ian Weller  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Ian Weller  2012-02-09 18:40:23 EST ---
New Package SCM Request
===
Package Name: python-qserve
Short Description: Job queue server
Owners: ianweller
Branches: f16 f17
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 787858] Review Request: bashmount - a menu-driven bash script for mounting removable media

[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=787858

Jamie Nguyen  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #10 from Jamie Nguyen  2012-02-09 18:31:17 
EST ---
New Package SCM Request
===
Package Name: bashmount
Short Description: A menu-driven bash script for mounting removable media
Owners: jamielinux
Branches: f16
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 579389] Review Request: perl-HTTP-Parser-XS - A fast, primitive HTTP request parser

[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=579389

Emmanuel Seyman  changed:

   What|Removed |Added

 CC||emmanuel.seyman@club-intern
   ||et.fr
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #5 from Emmanuel Seyman  
2012-02-09 17:50:30 EST ---
New Package SCM Request
===
Package Name: perl-HTTP-Parser-XS
Short Description: A fast, primitive HTTP request parser
Owners: eseyman bochecha
Branches: el6
InitialCC: perl-sig

I've been asked by Mathieu Bridon (bochacha) to create a el6 branch for this
module. Since I'm a helpful guy, I'ld be really glad if someone did this for
me. :-)

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

[Bug 789108] Review Request: python-qserve - Job queue server

[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=789108

Eric Christensen  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Eric Christensen  2012-02-09 
17:29:49 EST ---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated



 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm < 4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[-]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf is only needed if supporting EPEL5
[x]: 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 is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.

rpmlint python-qserve-0.2.6-1.fc17.src.rpm

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


rpmlint python-qserve-0.2.6-1.fc17.noarch.rpm

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


[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
/home/christensene/rpmbuild/789108/qserve-0.2.6.zip :
  MD5SUM this package : 7aad2ca65283fa8bfa674765c4a2a85b
  MD5SUM upstream package : 7aad2ca65283fa8bfa674765c4a2a85b

[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD If the source package does not include license text(s) as a
 separate file from upstream, the packager SHOULD query upstream to
 include it.
[ ]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[-]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD SourceX is a working URL.
[!]: SHOULD Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[-]: SHOULD Package should compile and build into binary rpms on all supported
 architectures.
[-]: SHOULD %check is present and all tests pass.
[-]: SHOULD Packages should try to preserve timestamps of original installed
 files.
[x]: SHOULD Spec use %global instead of %define.


Generated by fedora-review 0.1.2
External plugins:


APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- Yo

[Bug 789138] Review Request: moc - console audio player for LINUX/UNIX - FE-NEEDSPONSOR

[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=789138

--- Comment #1 from Antonio Trande  2012-02-09 17:21:51 
EST ---
*** Bug 788690 has been marked as a duplicate of this bug. ***

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

[Bug 789138] New: Review Request: moc - console audio player for LINUX/UNIX - FE-NEEDSPONSOR

[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: moc - console audio player for LINUX/UNIX - 
FE-NEEDSPONSOR

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

   Summary: Review Request: moc - console audio player for
LINUX/UNIX - FE-NEEDSPONSOR
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: trp...@katamail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://sagitter.fedorapeople.org/moc.spec
SRPM URL: http://sagitter.fedorapeople.org/moc-2.4.4-2.src.rpm
Description: MOC (music on console) is a console audio player for LINUX/UNIX
designed to be powerful and easy to use. You just need to select a file from
some directory using the menu similar to Midnight Commander, and MOC will start
playing all files in this directory beginning from the chosen file.
Homepage:http://moc.daper.net

Info: moc needs ffmpeg package from rpmfusion-free repository to work.
This is my first package so i need a sponsor.
Regards.

-- 
Configure 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 784944] Review Request: ktp-accounts-kcm - KDE Configuration Module for Telepathy Instant Messaging Accounts

[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=784944

Kevin Kofler  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|alekc...@googlemail.com

--- Comment #7 from Kevin Kofler  2012-02-09 17:20:49 
EST ---
You can't unassign yourself from an already approved 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 785943] Review Request: python-gunicorn - Python WSGI application server

[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=785943

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 785943] Review Request: python-gunicorn - Python WSGI application server

[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=785943

--- Comment #11 from Fedora Update System  
2012-02-09 17:20:34 EST ---
python-gunicorn-0.13.4-3.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/python-gunicorn-0.13.4-3.el6

-- 
Configure 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 785943] Review Request: python-gunicorn - Python WSGI application server

[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=785943

--- Comment #10 from Fedora Update System  
2012-02-09 17:19:30 EST ---
python-gunicorn-0.13.4-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/python-gunicorn-0.13.4-3.fc15

-- 
Configure 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 785943] Review Request: python-gunicorn - Python WSGI application server

[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=785943

--- Comment #9 from Fedora Update System  2012-02-09 
17:18:21 EST ---
python-gunicorn-0.13.4-3.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/python-gunicorn-0.13.4-3.fc16

-- 
Configure 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 787858] Review Request: bashmount - a menu-driven bash script for mounting removable media

[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=787858

--- Comment #9 from Ian Weller  2012-02-09 16:31:27 EST ---
Good review, Jamie -- I'm going to go ahead and sponsor you now.

If you have any questions of where to go from here just let me know.

-- 
Configure 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 787128] Review Request: libfep - Library to implement FEP (front end processor) on ANSI terminals

[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=787128

--- Comment #2 from Ian Weller  2012-02-09 16:27:38 EST ---
(So there's not any confusion, I asked Jamie to do a full review of this
package as part of the sponsorship process.)

(In reply to comment #1)
> 2) ChangeLog date format usually has leading zero (Fri Feb 03 2012).

This is up to the packager; there's no rule in the packaging guidelines about
this. (I usually drop the leading zero.)

> [x] MUST: rpmlint must be run on the source rpm and all binary rpms the build
> produces. The output should be posted in the review.

libfep.x86_64: W: no-manual-page-for-binary fep
libfep.x86_64: W: no-manual-page-for-binary fepcli
libfep-devel.x86_64: W: no-documentation

These warnings are all fine.

-- 
Configure 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 755093] Review Request: mactel-boot - boot tools for Intel Apple hardware

[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=755093

Matthew Garrett  changed:

   What|Removed |Added

 CC||mj...@srcf.ucam.org
   Flag||fedora-cvs?

--- Comment #37 from Matthew Garrett  2012-02-09 16:17:56 
EST ---
New Package SCM Request
===
Package Name: mactel-boot
Short Description: Support package for booting Linux on Intel-based Mac
hardware
Owners: mjg59
Branches: f17
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 787128] Review Request: libfep - Library to implement FEP (front end processor) on ANSI terminals

[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=787128

Jamie Nguyen  changed:

   What|Removed |Added

 CC||ja...@tomoyolinux.co.uk

--- Comment #1 from Jamie Nguyen  2012-02-09 16:10:30 
EST ---
Hello, some comments:

1) The "%doc" should be removed from the "%files devel" section.
2) ChangeLog date format usually has leading zero (Fri Feb 03 2012).
3) Single quote the curly brackets: find -exec rm -rf '{}' ';'
4) Perhaps include the ChangeLog file (alongside README etc.)


Review checklist, last updated 2012-02-07
Based on https://fedoraproject.org/wiki/Packaging:ReviewGuidelines
Key: [x] passed, [F] failed, [-] irrelevant

[x] MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
[x] MUST: The package must be named according to the Package Naming Guidelines.
[x] MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
[x] MUST: The package must meet the Packaging Guidelines.
[x] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[x] MUST: The License field in the package spec file must match the actual
license. 
[x] 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.
[x] MUST: The spec file must be written in American English. 
[x] MUST: The spec file for the package MUST be legible. 
[x] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If
no upstream URL can be specified for this package, please see the Source
URL Guidelines for how to deal with this.
[x] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture. 
[-] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed
in bugzilla, describing the reason that the package does not
compile/build/work on that architecture. The bug number MUST be placed in a
comment, next to the corresponding ExcludeArch line. 
[x] MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines;
inclusion of those as BuildRequires is optional. Apply common sense.
[-] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[X] MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths,
must call ldconfig in %post and %postun. 
[x] MUST: Packages must NOT bundle copies of system libraries.
[-] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. 
[x] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which
does create that directory. 
[x] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)
[x] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. 
[x] MUST: Each package must consistently use macros. 
[x] MUST: The package must contain code, or permissable content. 
[-] MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity). 
[x] MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program
must run properly if it is not present. 
[-] MUST: Static libraries must be in a -static package. 
[X] MUST: Development files must be in a -devel package. 
[X] MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency:
   Requires: %{name}%{?_isa} = %{version}-%{release} 
[X] MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built

[Bug 789113] Review Request: libg15 - A Library to handle the LCD and extra keys on the Logitech G15

[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=789113

Thomas Pifer  changed:

   What|Removed |Added

 Blocks||499109(bonding,Bug,interfac
   ||e,multiple),
   ||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 789113] New: Review Request: libg15 - A Library to handle the LCD and extra keys on the Logitech G15

[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: libg15 - A Library to handle the LCD and extra keys on 
the Logitech G15

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

   Summary: Review Request: libg15 - A Library to handle the LCD
and extra keys on the Logitech G15
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: zero...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://zero456.fedorapeople.org/libg15.spec
SRPM URL: http://zero456.fedorapeople.org/libg15-1.2.7-1.fc16.src.rpm
Description: libg15 is a library to handle the LCD and extra keys on the
Logitech G15 Gaming Keyboard and similar devices.

LibG15 currently supports LCD & keys on the following devices:
- Original G15 keyboard
- G11 keyboard
- Z10 speakers
- 2nd revision of the G15 (amber backlight, 6 'G' keys, LCD)
- Gamepanel available on some laptops

This is the first package necessary to get G15 keyboard support in Fedora.

I previously submitted bug 788806 for three of the G15tools packages but didn't
know it was for only one package per review request.  As such I'm starting
fresh.  

This is also my first package and I'm unsure if I still need a sponsor as Jason
Tibbitts was kind enough to sponsor me into a group for space on fedorapeople.

Here is the rpmlint logs for libg15:

$ rpmlint SRPMS/libg15-1.2.7-1.fc16.src.rpm 
libg15.src: W: spelling-error %description -l en_US backlight -> back light,
back-light, fightback
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint -i libg15-1.2.7-1.fc16.x86_64.rpm 
libg15.x86_64: W: spelling-error %description -l en_US backlight -> back light,
back-light, fightback
The value of this tag appears to be misspelled. Please double-check.

$ rpmlint -i libg15-devel-1.2.7-1.fc16.x86_64.rpm 
libg15-devel.x86_64: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.

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

$ rpmlint -i libg15-debuginfo-1.2.7-1.fc16.x86_64.rpm 
1 packages and 0 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 789108] Review Request: python-qserve - Job queue server

[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=789108

Eric Christensen  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||e...@christensenplace.us
 AssignedTo|nob...@fedoraproject.org|e...@christensenplace.us
   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 755093] Review Request: mactel-boot - boot tools for Intel Apple hardware

[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=755093

Josh Boyer  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #36 from Josh Boyer  2012-02-09 15:28:37 EST ---
- Package meets naming and packaging guidelines
- Spec file matches base package name.
- Spec has consistant macro usage.
- Meets Packaging Guidelines.
- License GPLv2+
- License field in spec matches
- License file included in package
- Spec in American English
- Spec is legible.
- Sources match upstream md5sum:

[jwboyer@zod ~]$ wget
http://www.codon.org.uk/~mjg59/mactel-boot/mactel-boot-0.9.tar.bz2
--2012-02-09 15:16:59-- 
http://www.codon.org.uk/~mjg59/mactel-boot/mactel-boot-0.9.tar.bz2
Resolving www.codon.org.uk... 93.93.128.6
Connecting to www.codon.org.uk|93.93.128.6|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 7751 (7.6K) [application/x-bzip2]
Saving to: “mactel-boot-0.9.tar.bz2”

100%[==>] 7,751   48.8K/s   in 0.2s

2012-02-09 15:17:00 (48.8 KB/s) - “mactel-boot-0.9.tar.bz2” saved [7751/7751]

[jwboyer@zod ~]$ md5sum mactel-boot-0.9.tar.bz2 
c03ba709112ab102d6f75b8028bc3db3  mactel-boot-0.9.tar.bz2
[jwboyer@zod ~]$ md5sum ~/rpmbuild/SOURCES/mactel-boot-0.9.tar.bz2 
c03ba709112ab102d6f75b8028bc3db3 
/home/jwboyer/rpmbuild/SOURCES/mactel-boot-0.9.tar.bz2
[jwboyer@zod ~]$ 

- Package correctly uses ExclusiveArch
- BuildRequires correct

  I think you can drop the BR on redhat-lsb here and just use the pre-defined
%{fedora} in place of your %{fedora_release} macro

- Package has %defattr and permissions on files is good.
- Package has a correct %clean section.
- Package has correct buildroot
- Package is code or permissible content.
- Packages %doc files don't affect runtime.

- Package compiles and builds on at least one arch.
[jwboyer@zod SPECS]$ rpmbuild -ba mactel-boot.spec 
Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.WMqeg6
+ umask 022
+ cd /home/jwboyer/rpmbuild/BUILD
+ LANG=C
+ export LANG
+ unset DISPLAY
+ cd /home/jwboyer/rpmbuild/BUILD
+ rm -rf mactel-boot-0.9
+ /usr/bin/bzip2 -dc /home/jwboyer/rpmbuild/SOURCES/mactel-boot-0.9.tar.bz2
+ /bin/tar -xf -
+ STATUS=0
+ '[' 0 -ne 0 ']'
+ cd mactel-boot-0.9
+ /bin/chmod -Rf a+rX,u+w,g-w,o-w .
+ exit 0
Executing(%build): /bin/sh -e /var/tmp/rpm-tmp.K1EYpT
+ umask 022
+ cd /home/jwboyer/rpmbuild/BUILD
+ cd mactel-boot-0.9
+ LANG=C
+ export LANG
+ unset DISPLAY
++ lsb_release -r -s
+ make 'PRODUCTVERSION=Fedora 16' -j4
sed "s/PRODUCTVERSION/Fedora 16/" SystemVersion.plist.in >SystemVersion.plist
cc -g -o hfs-bless bless.c
+ exit 0
Executing(%install): /bin/sh -e /var/tmp/rpm-tmp.lTpLaH
+ umask 022
+ cd /home/jwboyer/rpmbuild/BUILD
+ '[' /home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64 '!=' / ']'
+ rm -rf /home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64
++ dirname /home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64
+ mkdir -p /home/jwboyer/rpmbuild/BUILDROOT
+ mkdir /home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64
+ cd mactel-boot-0.9
+ LANG=C
+ export LANG
+ unset DISPLAY
+ rm -rf /home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64
+ make install
DESTDIR=/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64
install -D hfs-bless
/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64/usr/sbin/hfs-bless
install -D -m 644 bless.1
/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64/usr/share/man/man1/hfs-bless.1
+ install -D -m 644 SystemVersion.plist
/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64/boot/efi/System/Library/CoreServices/SystemVersion.plist
+ echo 'This file is required for booting'
+ touch
/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64/boot/efi/System/Library/CoreServices/boot.efi
+ touch
/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64/boot/efi/.VolumeIcon.icns
+ /usr/lib/rpm/find-debuginfo.sh --strict-build-id
/home/jwboyer/rpmbuild/BUILD/mactel-boot-0.9
extracting debug info from
/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64/usr/sbin/hfs-bless
4 blocks
+ /usr/lib/rpm/check-buildroot
+ /usr/lib/rpm/redhat/brp-compress
+ /usr/lib/rpm/redhat/brp-strip-static-archive /usr/bin/strip
+ /usr/lib/rpm/brp-python-bytecompile /usr/bin/python 1
+ /usr/lib/rpm/redhat/brp-python-hardlink
+ /usr/lib/rpm/redhat/brp-java-repack-jars
Processing files: mactel-boot-0.9-1.fc16.x86_64
Executing(%doc): /bin/sh -e /var/tmp/rpm-tmp.6M0Ctw
+ umask 022
+ cd /home/jwboyer/rpmbuild/BUILD
+ cd mactel-boot-0.9
+
DOCDIR=/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64/usr/share/doc/mactel-boot-0.9
+ export DOCDIR
+ /bin/mkdir -p
/home/jwboyer/rpmbuild/BUILDROOT/mactel-boot-0.9-1.fc16.x86_64

[Bug 789108] New: Review Request: python-qserve - Job queue server

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: python-qserve - Job queue server

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

   Summary: Review Request: python-qserve - Job queue server
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: i...@ianweller.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL:
http://ianweller.fedorapeople.org/SRPMS/python-qserve/0.2.6-1/python-qserve.spec
SRPM URL:
http://ianweller.fedorapeople.org/SRPMS/python-qserve/0.2.6-1/python-qserve-0.2.6-1.fc16.src.rpm

Description:
qserve is a job queue server written in Python.

-- 
Configure 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 788806] Package Review: g15tools - Support for the G15 in Linux

[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=788806

Thomas Pifer  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-02-09 15:12:49

--- Comment #6 from Thomas Pifer  2012-02-09 15:12:49 EST ---
Understood.  Thanks for all your help!  Closing this bug and unblocking 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 788806] Package Review: g15tools - Support for the G15 in Linux

[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=788806

Thomas Pifer  changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR), |
   |499109(bonding,Bug,interfac |
   |e,multiple) |

-- 
Configure 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 755093] Review Request: mactel-boot - boot tools for Intel Apple hardware

[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=755093

Josh Boyer  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|jwbo...@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 705109] Review Request: python-qserve - Job queue server used in python-mwlib

[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=705109

Ian Weller  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
   Flag|needinfo?(linux.n.pkd@gmail |
   |.com)   |
Last Closed||2012-02-09 15:04:02

--- Comment #15 from Ian Weller  2012-02-09 15:04:02 EST ---
Pursuant to the policy for stalled package reviews, I am closing this review
request.

-- 
Configure 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 787128] Review Request: libfep - Library to implement FEP (front end processor) on ANSI terminals

[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=787128

Ian Weller  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||i...@ianweller.org
 AssignedTo|nob...@fedoraproject.org|i...@ianweller.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 787858] Review Request: bashmount - a menu-driven bash script for mounting removable media

[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=787858

--- Comment #8 from Ian Weller  2012-02-09 15:00:00 EST ---
So as we discussed in IRC, before I sponsor you I'm going to have you do a full
review of this package:
  https://bugzilla.redhat.com/show_bug.cgi?id=787128

I've taken the bug, but go ahead and do the review. Let me know if you have any
questions.

-- 
Configure 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 788815] Review Request: python-ZSI - python Zolera Soap Infrastructure

[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=788815

Haïkel Guémar  changed:

   What|Removed |Added

 CC||karlthe...@gmail.com
 AssignedTo|nob...@fedoraproject.org|karlthe...@gmail.com
   Flag||fedora-review?

--- Comment #1 from Haïkel Guémar  2012-02-09 14:52:09 
EST ---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated



 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[!]: MUST Buildroot is not present
 Note: Buildroot is not needed unless packager plans to package for EPEL5
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[!]: MUST Each %files section contains %defattr if rpm < 4.4
 Note: defattr() present in %files section. This is OK if packaging
 for EPEL5. Otherwise not needed
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[x]: 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 is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[!]: MUST Rpmlint output is silent.

rpmlint python-ZSI-2.0-10.fc17.noarch.rpm

python-ZSI.noarch: W: spelling-error Summary(en_US) Zolera -> Cholera
python-ZSI.noarch: W: no-manual-page-for-binary wsdl2dispatch
python-ZSI.noarch: W: no-manual-page-for-binary wsdl2py
1 packages and 0 specfiles checked; 0 errors, 3 warnings.


rpmlint python-ZSI-2.0-10.fc17.src.rpm

python-ZSI.src: W: spelling-error Summary(en_US) Zolera -> Cholera
python-ZSI.src: W: invalid-url Source0:
http://belnet.dl.sourceforge.net/sourceforge/pywebsvcs/ZSI-2.0.tar.gz 
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
/home/haikel/788815/ZSI-2.0.tar.gz :
  MD5SUM this package : bb706337cafe9e2201b06b1bce71ca0f
  MD5SUM upstream package : bb706337cafe9e2201b06b1bce71ca0f

[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
 separate file from upstream, the packager SHOULD query upstream to
 include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
 translations for sup

[Bug 750394] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes

[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=750394

--- Comment #4 from Kapil Arya  2012-02-09 14:56:01 EST ---
I think Neal is going to be the sponsor :).

Neal, can you please confirm this?

I am going through the checklist now. Will update once I am 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 788806] Package Review: g15tools - Support for the G15 in Linux

[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=788806

--- Comment #5 from Jason Tibbitts  2012-02-09 14:40:38 EST 
---
I have sponsored you into a group which gives you enough privileges to get
fedorapeople access.  That should allow you to put your srpms and spec files
where reviewers can actually get them. 
http://fedoraproject.org/wiki/Infrastructure/fedorapeople.org has information
on using the space.

You can also now do scratch builds in our buildsystem.

You'd definitely have to open two more tickets.  At this point it would
probably be easiest to just abandon this ticket and open three more, one for
each package.  Please use the standard format for review tickets: 
https://bugzilla.redhat.com/enter_bug.cgi?product=Fedora&format=fedora-review

If the packages need to be reviewed in some order (i.e. they depend on each
other in some way), you can indicate the ordering using the Depends on: and
Blocks: fields.

-- 
Configure 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 750394] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes

[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=750394

--- Comment #3 from Timothy St. Clair  2012-02-09 14:34:00 
EST ---
Kapil have you run through the checklist?  (Do you have a sponsor, etc.)

http://fedoraproject.org/wiki/PackageMaintainers/Join

-- 
Configure 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 781260] Review Request: leechcraft - A Free Open Source Cross-Platform Modular Internet-Client

[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=781260

Rex Dieter  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|rdie...@math.unl.edu
   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 750394] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes

[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=750394

--- Comment #2 from Kapil Arya  2012-02-09 14:21:34 EST ---
I have updated the SRPM and SPEC file to reflect the newer upstream release.
Here are the URLs:
Spec URL: http://www.ccs.neu.edu/home/kapil/fedora_rpms/dmtcp.spec
SRPM URL:
http://www.ccs.neu.edu/home/kapil/fedora_rpms/dmtcp-1.2.4-1.svn1449.fc16.src.rpm

BTW, any idea on how long before this package gets accepted?

Thanks,
Kapil

-- 
Configure 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 784950] Review Request: ktp-contact-applet - Telepathy contact plasmoid

[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=784950

--- Comment #24 from Matthias Runge  2012-02-09 
14:10:44 EST ---
(In reply to comment #20)
> He made me responsible for errors in existing packages without particular
> examples - nice beginning, true? And all next comments was in the same way -

Plainly wrong. I didn't made you responsible for any error in any package.

I said, you might didn't notice an existing packaging error in a package 
doing such a short review. I can't see from outside, what you exactly did
check. 

I must admit, just asking I you're familiar with packaging guidelines is not
exactly "being excellent to each other". I promise, I'll try better next time!

I tried to be constructive and to point out, how to make this review better;
All (or all -1) contributors here thinking, this special review is just too
short.
My intention was, to outline, one can do it better, and how to improve. 
This point wasn't accepted by repeating, there's nothing wrong with the review.

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

[Bug 784954] Review Request: ktp-kded-integration-module - KDE integration for telepathy

[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=784954

--- Comment #5 from Rex Dieter  2012-02-09 14:09:21 EST 
---
Spec URL:
http://rdieter.fedorapeople.org/rpms/ktp/ktp-kded-integration-module.spec 
SRPM URL:
http://rdieter.fedorapeople.org/rpms/ktp/ktp-kded-integration-module-0.3.0-2.fc16.src.rpm

%changelog
* Thu Feb 09 2012 Rex Dieter  0.3.0-2
- fix Source0 URL
- drop BR: telepathy-qt4-devel
- fix buildRequires typo

-- 
Configure 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 784952] Review Request: ktp-filetransfer-handler - Telepathy file transfer handler

[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=784952

--- Comment #3 from Rex Dieter  2012-02-09 14:05:44 EST 
---
http://rdieter.fedorapeople.org/rpms/ktp/ktp-filetransfer-handler.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/ktp/ktp-filetransfer-handler-0.3.0-2.fc16.src.rpm

%changelog
* Thu Feb 09 2012 Rex Dieter  0.3.0-2
- drop BR: desktop-file-utils telepathy-qt4-devel
- %%doc AUTHORS NEWS README TODO

-- 
Configure 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 784955] Review Request: ktp-presence-applet - Plasma applet for managing your Telepathy account presence

[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=784955

--- Comment #3 from Jon Ciesla  2012-02-09 13:58:16 EST 
---
Git done (by process-git-requests).

Added f17.

-- 
Configure 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 784951] Review Request: ktp-contact-list - Telepathy contact list application

[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=784951

--- Comment #4 from Jon Ciesla  2012-02-09 13:57:34 EST 
---
Git done (by process-git-requests).

Added f17.

-- 
Configure 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 784956] Review Request: ktp-send-file - A File manager plugin to launch file transfer jobs

[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=784956

--- Comment #8 from Jon Ciesla  2012-02-09 13:58:59 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 784950] Review Request: ktp-contact-applet - Telepathy contact plasmoid

[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=784950

--- Comment #23 from Jon Ciesla  2012-02-09 13:56:41 EST 
---
Git done (by process-git-requests).

Added f17.

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