[Bug 879624] Review Request: jbossws-native - JBossWS Native

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879624

--- Comment #3 from Marek Goldmann mgold...@redhat.com ---
Hi Gil, thanks for review.

Only ASL 2.0, EPL, BSD and MIT licenses require including the license text in
the packages. Please take a look at:
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text

-- 
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 879624] Review Request: jbossws-native - JBossWS Native

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879624

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks|652183 (FE-JAVASIG) |
  Flags||fedora-cvs?

--- Comment #4 from Marek Goldmann mgold...@redhat.com ---
New Package SCM Request
===
Package Name: jbossws-native
Short Description: JBossWS Native
Owners: goldmann
Branches: f17 f18

-- 
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 866325] Review Request: ugene - genome analysis suite

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866325

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 866325] Review Request: ugene - genome analysis suite

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866325

--- Comment #21 from Fedora Update System upda...@fedoraproject.org ---
ugene-1.11.3-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/ugene-1.11.3-1.fc18

-- 
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 866325] Review Request: ugene - genome analysis suite

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866325

--- Comment #22 from Fedora Update System upda...@fedoraproject.org ---
ugene-1.11.3-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/ugene-1.11.3-1.fc17

-- 
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 879903] Review Request: xorg-x11-drv-armsoc - Xorg X11 armsocdrm driver

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879903

Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from Peter Robinson pbrobin...@gmail.com ---
Thanks for the review

New Package GIT Request
===
Package Name: xorg-x11-drv-armsoc
Short Description: Xorg X11 armsocdrm driver
Owners: pbrobinson dgilmore
Branches: F-17 F-18
InitialCC:

-- 
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 758734] Review Request: fatrat-subtitlesearch - FatRat plugin enabling OpenSubtitles.org and Sublight.si integration

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=758734

--- Comment #20 from Jan Vcelak jvce...@redhat.com ---
You are right, I didn't. Sorry for that, I'm quite busy recently. Will do it
soon. :-)

-- 
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 772362] Review Request: sigil - Free, Open Source WYSIWYG ebook editor

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=772362

Hans de Goede hdego...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2012-11-26 05:00:40

--- Comment #34 from Hans de Goede hdego...@redhat.com ---
Imported, build for f16+ and bodhi tickets created for f16+, closing.

-- 
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 866156] Review Request: giada - audio looper for JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866156

--- Comment #9 from Ismael Olea ism...@olea.org ---

You are right, I've been using the r2 one.

Using r3 I got this:

  /usr/bin/ld: cannot find -lXext


after:

mock init -r fedora-rawhide-i386
mock --install -r fedora-rawhide-i386 rtaudio-devel-4.0.11-3.fc19.i686.rpm
rtaudio-4.0.11-3.fc19.i686.rpm
mock --no-clean giada-0.5.2-3.fc18.src.rpm -r fedora-rawhide-i386


I can build it with:
mock --install -r fedora-rawhide-i386 libXext-devel
mock --no-clean giada-0.5.2-3.fc18.src.rpm -r fedora-rawhide-i386

-- 
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 758734] Review Request: fatrat-subtitlesearch - FatRat plugin enabling OpenSubtitles.org and Sublight.si integration

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=758734

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

   What|Removed |Added

 Status|NEW |MODIFIED

-- 
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 758734] Review Request: fatrat-subtitlesearch - FatRat plugin enabling OpenSubtitles.org and Sublight.si integration

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=758734

--- Comment #21 from Fedora Update System upda...@fedoraproject.org ---
fatrat-subtitlesearch-1.2.0-0.3.beta1.fc18 has been submitted as an update for
Fedora 18.
https://admin.fedoraproject.org/updates/fatrat-subtitlesearch-1.2.0-0.3.beta1.fc18

-- 
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 758734] Review Request: fatrat-subtitlesearch - FatRat plugin enabling OpenSubtitles.org and Sublight.si integration

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=758734

--- Comment #22 from Fedora Update System upda...@fedoraproject.org ---
fatrat-subtitlesearch-1.2.0-0.3.beta1.fc17 has been submitted as an update for
Fedora 17.
https://admin.fedoraproject.org/updates/fatrat-subtitlesearch-1.2.0-0.3.beta1.fc17

-- 
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 758734] Review Request: fatrat-subtitlesearch - FatRat plugin enabling OpenSubtitles.org and Sublight.si integration

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=758734

--- Comment #23 from Fedora Update System upda...@fedoraproject.org ---
fatrat-subtitlesearch-1.2.0-0.3.beta1.fc16 has been submitted as an update for
Fedora 16.
https://admin.fedoraproject.org/updates/fatrat-subtitlesearch-1.2.0-0.3.beta1.fc16

-- 
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 758734] Review Request: fatrat-subtitlesearch - FatRat plugin enabling OpenSubtitles.org and Sublight.si integration

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=758734

Jan Vcelak jvce...@redhat.com changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
   Fixed In Version||fatrat-subtitlesearch-1.2.0
   ||-0.3.beta1.fc16
 Resolution|--- |NEXTRELEASE
Last Closed||2012-11-26 05:20:57

--- Comment #24 from Jan Vcelak jvce...@redhat.com ---
Finished. Thanks to everyone involved.

-- 
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 877403] Review Request: svnkit - Pure Java Subversion client library

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=877403

--- Comment #12 from Brendan Jones brendan.jones...@gmail.com ---
I'm running into some build errors in fedora-rawhide-x86_64

Some logs for you here: http://bsjones.fedorapeople.org/reviews/svnkit/

-- 
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 866156] Review Request: giada - audio looper for JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866156

--- Comment #10 from Brendan Jones brendan.jones...@gmail.com ---
Thats' strange because traditionally libXpm has pulled in libXext as a
dependancy. See http://koji.fedoraproject.org/koji/rpminfo?rpmID=3227992

Updated here (and tested in mock!)

SPEC: http://bsjones.fedorapeople.org/reviews/giada.spec
SRPM: http://bsjones.fedorapeople.org/reviews/giada-0.5.2-4.fc18.src.rpm

-- 
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 845262] Review Request: mhwaveedit - Sound editing program

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=845262

Jørn Lomax northlo...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
Last Closed||2012-11-26 06:25:11

--- Comment #12 from Jørn Lomax northlo...@gmail.com ---
Yes

-- 
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 845262] Review Request: mhwaveedit - Sound editing program

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=845262

Jørn Lomax northlo...@gmail.com changed:

   What|Removed |Added

 Resolution|NOTABUG |NEXTRELEASE

-- 
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 834239] Review Request: monobristol - frontend for britsol in mono

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=834239

Jørn Lomax northlo...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2012-11-26 06:26:02

-- 
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 866012] Review Request: non-daw - a digital audio workstation using JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866012

--- Comment #7 from Jørn Lomax northlo...@gmail.com ---
Good to close this?

-- 
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 866012] Review Request: non-daw - a digital audio workstation using JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866012

--- Comment #8 from Brendan Jones brendan.jones...@gmail.com ---
Not as yet. I've not submitted an update but will soon.

-- 
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 866012] Review Request: non-daw - a digital audio workstation using JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866012

--- Comment #9 from Matthias Runge mru...@redhat.com ---
@Jørn

normally, it get's closed if a package hits stable.
Apparently, there are no builds yet.
https://admin.fedoraproject.org/pkgdb/acls/bugs/non-daw

-- 
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 880155] New: Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

Bug ID: 880155
   Summary: Review Request: PackageKit-Qt - Qt support library for
PackageKit
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: rhug...@redhat.com

Spec URL: http://people.freedesktop.org/~hughsient/temp/PackageKit-Qt.spec
SRPM URL:
http://people.freedesktop.org/~hughsient/temp/PackageKit-Qt-0.8.6-1.fc18.src.rpm
Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4728316
Description: PackageKit-qt is a Qt support library for PackageKit
Fedora Account System Username: rhughes

$ rpmlint rpmbuild/*RPMS/PackageKit-Qt-*
4 packages and 0 specfiles checked; 0 errors, 0 warnings.

This package is the Qt code split out from the PackageKit project. The new
package is blocking the release of PackageKit 0.8.6 as projects such as apper
require the Qt interfaces provided by PackageKit-Qt. I'd really appreciate any
help on this package as I'm a bit of a cmake newbie. Thanks.

Richard.

-- 
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 866012] Review Request: non-daw - a digital audio workstation using JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866012

--- Comment #10 from Jørn Lomax northlo...@gmail.com ---
Ahh, my bad. I was just cleaning up some of the packages I got reviewed during
the summer that still weren't closed and were cluttering up my list of bugs.
Guess i was a little hasty :p

-- 
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 879953] Review Request: xs-config - XS/XSX default configuration

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879953

Michael Scherer m...@zarb.org changed:

   What|Removed |Added

 CC||m...@zarb.org
 Blocks||177841 (FE-NEEDSPONSOR)

-- 
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 879954] Review Request: rssh - Restricted shell for use with OpenSSH, allowing only scp and/or sftp

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879954

Michael Scherer m...@zarb.org changed:

   What|Removed |Added

 CC||m...@zarb.org
 Blocks||177841 (FE-NEEDSPONSOR)

-- 
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 879954] Review Request: rssh - Restricted shell for use with OpenSSH, allowing only scp and/or sftp

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879954

Michael Scherer m...@zarb.org changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
Last Closed||2012-11-26 06:37:42

--- Comment #1 from Michael Scherer m...@zarb.org ---
Rssh is already in Fedora, so closing this bug.

-- 
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 868717] Review Request: libtvdb - Library to retrieve TV series information from web

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868717

Dan Vrátil dvra...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #10 from Dan Vrátil dvra...@redhat.com ---
New Package SCM Request
===
Package Name: libtvdb
Short Description: Library to retrieve TV series information from web
Owners: dvratil
Branches: f17 f18
InitialCC:

-- 
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 866156] Review Request: giada - audio looper for JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866156

--- Comment #11 from Ismael Olea ism...@olea.org ---
(In reply to comment #10)
 Thats' strange because traditionally libXpm has pulled in libXext as a
 dependancy. See http://koji.fedoraproject.org/koji/rpminfo?rpmID=3227992

but not libXext-devel, as I can see


 Updated here (and tested in mock!)

works fine now.


Please notice there is probably a typo in description: «giada is an audio
looper or ALSA/JACK using the FLTK toolit» should be «giada is an audio looper
for ALSA/JACK using the FLTK toolit», I suppose.


I'm not into audio but the word «machine» in Summary is inexact, isn't it?


Next is my full review.

-- 
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 880179] New: Review Request: libnetfilter_cthelper - User-space infrastructure for connection tracking helpers

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880179

Bug ID: 880179
   Summary: Review Request: libnetfilter_cthelper - User-space
infrastructure for connection tracking helpers
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: i...@stingr.net

Spec URL:
http://stingray.fedorapeople.org/review/libnetfilter_cthelper/libnetfilter_cthelper.spec
SRPM URL:
http://stingray.fedorapeople.org/review/libnetfilter_cthelper/libnetfilter_cthelper-1.0.0-1.fc17.src.rpm
Description:
This library provides the infrastructure for the user-space helper
infrastructure available since the Linux kernel 3.6.
Fedora Account System Username: stingray

-- 
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 880179] Review Request: libnetfilter_cthelper - User-space infrastructure for connection tracking helpers

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880179

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

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

--- Comment #1 from Peter Lemenkov lemen...@gmail.com ---
I'll review it.

-- 
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 866156] Review Request: giada - audio looper for JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866156

--- Comment #13 from Ismael Olea ism...@olea.org ---
Summarizing:

[?]: Package functions as described.
 * Seems my pulseaudio system is not compatible with rtaudio

[!]: Latest version is packaged.
 * last upstream is 0.5.4


[!]: Patches link to upstream bugs/comments/lists or are otherwise justified.
 * you should contribute desktop file, and the patch if considered useful to
upstream
 and provide the links to those
 See
https://fedoraproject.org/wiki/Packaging:Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment

[!]: Scriptlets must be sane, if used.
 * as desktop file doesn't include a MimeType key it not needs the
 update-destkop-database calls
 See
https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#desktop-database

-- 
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 868717] Review Request: libtvdb - Library to retrieve TV series information from web

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868717

--- Comment #11 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 875359] Review Request: willie - a simple, lightweight, open source, easy-to-use IRC Utility bot, written in Python.

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875359

--- Comment #14 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 879624] Review Request: jbossws-native - JBossWS Native

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879624

--- Comment #5 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 866156] Review Request: giada - audio looper for JACK

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=866156

--- Comment #14 from Ismael Olea ism...@olea.org ---
Other issues:

Maybe you'll want to accept this enhancement for the desktop file:

--- srpm-unpacked/giada.desktop2012-11-26 12:36:35.515736117 +0100
+++ giada.desktop2012-11-26 14:04:20.306506516 +0100
@@ -1,6 +1,8 @@
 [Desktop Entry]
 Name=Giada
-Comment=Realtime software looping sampler
+Name[es]=Giada
+GenericName=Drum machine and loop sequencer
+GenericName[es]=Caja de ritmos y sequenciador de loops
 Icon=giada
 Type=Application
 Exec=giada


After a fast watch at Giada website I think you could add more info into the
descrption to be more descriptive, something like:

Features
Ultra-lightweight internal design
Multi-thread/multi-core support
32 bit floating point audio engine
Experimental VST support
Up to 32 audio channels controlled via computer keyboard
Different playback modes and combinations
Bpm and beat sync with sample-accurate loop engine
Live sampler from external inputs
Live quantizer
Action recorder
Patch-based systemBuilt-in wave editor
Support for all major uncompressed file formats

at your own consideration.

-- 
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 879954] Review Request: rssh - Restricted shell for use with OpenSSH, allowing only scp and/or sftp

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879954

--- Comment #2 from Alex aadav...@learn.senecac.on.ca ---
yes but which release?

-- 
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 879903] Review Request: xorg-x11-drv-armsoc - Xorg X11 armsocdrm driver

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879903

--- Comment #6 from Jon Ciesla limburg...@gmail.com ---
WARNING: dgilmore is not a valid FAS account.

-- 
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 880195] New: Review Request: python-di - Dependency injection library for python unittesters

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880195

Bug ID: 880195
   Summary: Review Request: python-di - Dependency injection
library for python unittesters
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: msi...@redhat.com

Spec URL: http://msivak.fedorapeople.org/newpkg/python-di.spec
SRPM URL: http://msivak.fedorapeople.org/newpkg/python-di-0.1-1.fc17.src.rpm
Description: Python library for dependency injection design patterns (also
known as Inversion of Control). Its purpose is to help isolate units from each
other without changing the code and algorithms much. It is helping mostly in
environments and teams where high unittest coverage is a goal.
Fedora Account System Username: msivak

-- 
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 875359] Review Request: willie - a simple, lightweight, open source, easy-to-use IRC Utility bot, written in Python.

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875359

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 875359] Review Request: willie - a simple, lightweight, open source, easy-to-use IRC Utility bot, written in Python.

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875359

--- Comment #15 from Fedora Update System upda...@fedoraproject.org ---
willie-3.1.2-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/willie-3.1.2-2.fc17

-- 
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 875359] Review Request: willie - a simple, lightweight, open source, easy-to-use IRC Utility bot, written in Python.

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875359

--- Comment #16 from Fedora Update System upda...@fedoraproject.org ---
willie-3.1.2-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/willie-3.1.2-2.fc18

-- 
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 879903] Review Request: xorg-x11-drv-armsoc - Xorg X11 armsocdrm driver

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879903

Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #7 from Peter Robinson pbrobin...@gmail.com ---
New Package GIT Request
===
Package Name: xorg-x11-drv-armsoc
Short Description: Xorg X11 armsocdrm driver
Owners: pbrobinson ausil
Branches: F-17 F-18
InitialCC:

-- 
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 880203] New: strong_parameters - Permitted and required parameters for Action Pack

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880203

Bug ID: 880203
   Summary: strong_parameters - Permitted and required parameters
for Action Pack
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: jstri...@redhat.com

Spec URL: http://data-strzibny.rhcloud.com/rubygem-strong_parameters.spec
SRPM URL:
http://data-strzibny.rhcloud.com/rubygem-strong_parameters-0.1.5-1.fc17.src.rpm
Description: strong_parameters is an implementation of the slice pattern. The
basic idea
is to move mass-assignment protection out of the model and into the controller
where it belongs.
Fedora Account System Username: jstribny
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4728639

-- 
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 879903] Review Request: xorg-x11-drv-armsoc - Xorg X11 armsocdrm driver

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879903

--- Comment #8 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 879954] Review Request: rssh - Restricted shell for use with OpenSSH, allowing only scp and/or sftp

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879954

--- Comment #3 from Michael Scherer m...@zarb.org ---
https://admin.fedoraproject.org/updates/rssh 

2.3.3 

However, the package seems to be orphaned, so someone need to take care of it.
Are you willing to do it ? 

That's the same as pushing the new version, except the procedure is a little
bit different.

( http://fedoraproject.org/wiki/Orphaned_package_that_need_new_maintainers for
the procedure ) ?

-- 
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 880203] rubygem-strong_parameters - Permitted and required parameters for Action Pack

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880203

Josef Stribny jstri...@redhat.com changed:

   What|Removed |Added

Summary|strong_parameters - |rubygem-strong_parameters -
   |Permitted and required  |Permitted and required
   |parameters for Action Pack  |parameters for Action Pack

-- 
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 880203] Review Request: rubygem-strong_parameters - Permitted and required parameters for Action Pack

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880203

Josef Stribny jstri...@redhat.com changed:

   What|Removed |Added

Summary|rubygem-strong_parameters - |Review Request:
   |Permitted and required  |rubygem-strong_parameters -
   |parameters for Action Pack  |Permitted and required
   ||parameters for Action Pack

-- 
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 873813] Review Request: gnome-shell-search-github-repositories - Search your Github Repos from the gnome-shell

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=873813

--- Comment #7 from Michael Scherer m...@zarb.org ---
I am on f18, 64 bits, running gnome-shell. My account is mscherer.

However, i just noted that I gave a wrong password, so maybe that's the issue.

-- 
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 880179] Review Request: libnetfilter_cthelper - User-space infrastructure for connection tracking helpers

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880179

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #2 from Peter Lemenkov lemen...@gmail.com ---
REVIEW:

Legend: + = PASSED, - = FAILED, 0 = Not Applicable

+ rpmlint is mostly silent (except few false positives):

sulaco ~/rpmbuild/SPECS: rpmlint ../RPMS/ppc/libnetfilter_cthelper-*
../SRPMS/libnetfilter_cthelper-1.0.0-1.fc19.src.rpm 
libnetfilter_cthelper.ppc: E: incorrect-fsf-address
/usr/share/doc/libnetfilter_cthelper-1.0.0/COPYING
libnetfilter_cthelper-debuginfo.ppc: W: spelling-error Summary(en_US)
libnetfilter - filibusterer
libnetfilter_cthelper-debuginfo.ppc: W: spelling-error Summary(en_US) cthelper
- ct helper, ct-helper, helper
libnetfilter_cthelper-debuginfo.ppc: W: spelling-error %description -l en_US
libnetfilter - filibusterer
libnetfilter_cthelper-debuginfo.ppc: W: spelling-error %description -l en_US
cthelper - ct helper, ct-helper, helper
libnetfilter_cthelper-devel.ppc: W: spelling-error Summary(en_US) libnetfilter
- filibusterer
libnetfilter_cthelper-devel.ppc: W: spelling-error Summary(en_US) cthelper -
ct helper, ct-helper, helper
libnetfilter_cthelper-devel.ppc: W: spelling-error %description -l en_US
libnetfilter - filibusterer
libnetfilter_cthelper-devel.ppc: W: spelling-error %description -l en_US
cthelper - ct helper, ct-helper, helper
4 packages and 0 specfiles checked; 1 errors, 8 warnings.
sulaco ~/rpmbuild/SPECS: 

+ The package is named according to the  Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format
%{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines.

- The License field in the package spec file MUST matche the actual license
(GPLv2 or later). A correct License tag is GPLv2+.

+ The file, containing the text of the license(s) for the package, is included
in %doc.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package, match the upstream source, as provided
in the spec URL.

sulaco ~/rpmbuild/SOURCES: sha256sum libnetfilter_cthelper-1.0.0.tar.bz2*
07618e71c4d9a6b6b3dc1986540486ee310a9838ba754926c7d14a17d8fccf3d 
libnetfilter_cthelper-1.0.0.tar.bz2
07618e71c4d9a6b6b3dc1986540486ee310a9838ba754926c7d14a17d8fccf3d 
libnetfilter_cthelper-1.0.0.tar.bz2.1
sulaco ~/rpmbuild/SOURCES: 

+ The package successfully compiles and builds into binary rpms on at least one
primary architecture.
+ All build dependencies are listed in BuildRequires.
0 No need to handle locales.
+ The package stores shared library files in some of the dynamic linker's
default paths, and it calls ldconfig in %post and %postun.
+ The package does NOT bundle copies of system libraries.
0 The package is not designed to be relocatable.
+ The package owns all directories that it creates.
+ The package does not list a file more than once in the spec file's %files
listings.
+ Permissions on files are set properly.
0 The package DOESN'T have a %clean section, so it won't build cleanly on
systems with old rpm (EL-4 and EL-5, not sure about EL-6). Beware.
+ The package consistently uses macros.
+ The package contains code, or permissible content.
0 No extremely large documentation files.
+ Anything, the package includes as %doc, does not affect the runtime of the
application.
+ Header files are stored in a -devel package.
0 No static libraries.
+ The pkgconfig(.pc) file is stored in a -devel package.
+ The library file(s) that end in .so (without suffix) is(are) stored in a
-devel package.
+ The -devel package requires the base package using a fully versioned
dependency: Requires: %{name}%{?_isa} = %{version}-%{release}
+ The package does NOT contain any .la libtool archives.
0 Not a GUI application.
+ The package does not own files or directories already owned by other
packages.
0 At the beginning of %install, the package  does not run rm -rf %{buildroot}
(or $RPM_BUILD_ROOT) so it won't build cleanly on systems with old rpm (EL-4
and EL-5, not sure about EL-6). Beware.
+ All filenames in rpm packages are valid UTF-8.


Assuming that you set License tag to GPLv2+ I don't see any issues and this
package is


APPROVED.

-- 
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 879954] Review Request: rssh - Restricted shell for use with OpenSSH, allowing only scp and/or sftp

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879954

--- Comment #4 from Alex aadav...@learn.senecac.on.ca ---
in order for it to be process I have to change the version? or the release?

according to my course it has to be fully maintain not orphaned.

got to check with ctyler first, not sure.

-- 
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 739398] Review Request: openblas - An optimized BLAS library based on GotoBLAS2

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=739398

--- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi ---
(In reply to comment #8)
 I'll be taking a two-week vacation now, so I won't be able to look at this
 until I get back.

Whoops...

Anyway, I have rebased the package to 0.2.5, otherwise the spec is pretty much
the same as the one reviewed above.

I ran into some speed issues with ATLAS, so I'd like to get this package in the
repos now. 

http://theory.physics.helsinki.fi/~jzlehtol/rpms/openblas.spec
http://theory.physics.helsinki.fi/~jzlehtol/rpms/openblas-0.2.5-1.fc18.src.rpm

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

-- 
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 868717] Review Request: libtvdb - Library to retrieve TV series information from web

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868717

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

   What|Removed |Added

 Status|NEW |MODIFIED

-- 
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 868717] Review Request: libtvdb - Library to retrieve TV series information from web

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868717

--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
libtvdb-0.3.0-3.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/libtvdb-0.3.0-3.fc18

-- 
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 880179] Review Request: libnetfilter_cthelper - User-space infrastructure for connection tracking helpers

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880179

Paul P Komkoff Jr i...@stingr.net changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #3 from Paul P Komkoff Jr i...@stingr.net ---
New Package SCM Request
===
Package Name: libnetfilter_cthelper
Short Description: User-space infrastructure for connection tracking helpers
Owners: stingray
Branches: f16 f17 f18 el5 el6

-- 
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 876915] Review Request: erlang-emmap - Erlang mmap interface

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876915

--- Comment #8 from Jon Ciesla limburg...@gmail.com ---
Can you fedpkg clone?

-- 
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 880179] Review Request: libnetfilter_cthelper - User-space infrastructure for connection tracking helpers

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880179

--- Comment #4 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 876915] Review Request: erlang-emmap - Erlang mmap interface

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876915

--- Comment #9 from Peter Lemenkov lemen...@gmail.com ---
(In reply to comment #8)
 Can you fedpkg clone?

Yes, I can. It got stuck during fedpkg import, at file uploading.

-- 
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 868717] Review Request: libtvdb - Library to retrieve TV series information from web

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868717

--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
libtvdb-0.3.0-3.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/libtvdb-0.3.0-3.fc17

-- 
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 880155] Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||rdie...@math.unl.edu
   Assignee|nob...@fedoraproject.org|rdie...@math.unl.edu
  Flags||fedora-review?

--- Comment #1 from Rex Dieter rdie...@math.unl.edu ---
I can help with the review, it's fairly small and should be pretty simple and
straight-forward...

-- 
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 876915] Review Request: erlang-emmap - Erlang mmap interface

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876915

--- Comment #10 from Jon Ciesla limburg...@gmail.com ---
Ok, I've re-created the git repo, try it now.

-- 
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 873813] Review Request: gnome-shell-search-github-repositories - Search your Github Repos from the gnome-shell

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=873813

Michael Scherer m...@zarb.org changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #8 from Michael Scherer m...@zarb.org ---
ok so just after testing, the issue is that I use the mouse to click on the
button, and only with keyboard, this work. So I tested, this work fine.

if you do not forget to remove the macro at the top ( since they are defined in
fedora since a long time, so that just add potential bugs in the futur, IMHO ),
the package is approved. ( also, you forgot to update the spec file on your
website )

-- 
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 880155] Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

--- Comment #2 from Rex Dieter rdie...@math.unl.edu ---
$ rpmlint *.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

naming: ok

sources: ok
41f6fa67e980733d7d8a2cd881bbe5b4  PackageKit-Qt-0.8.6.tar.gz

license: ok
$ licensecheck -r src
src/daemonprivate.h: LGPL (v2 or later)
src/transactionprivate.h: LGPL (v2 or later)
src/common.h: LGPL (v2 or later)
src/bitfield.cpp: *No copyright* UNKNOWN
src/daemonprivate.cpp: LGPL (v2 or later)
src/daemon.cpp: LGPL (v2 or later)
src/transaction.h: LGPL (v2 or later)
src/bitfield.h: *No copyright* UNKNOWN
src/transaction.cpp: LGPL (v2 or later)
src/transactionprivate.cpp: LGPL (v2 or later)
src/daemon.h: LGPL (v2 or later)

macros: ok

builds/installs: mostly ok, but...

1.  MUST: need some Obsoletes (and at least a temporary Provides) here to
handle upgrade from older PackageKit-qt... 

Obsoletes: PackageKit-qt  0.8.6
Provides:  PackageKit-qt = %{version}-%{release}
Provides:  PackageKit-qt%{?_isa} = %{version}-%{release}

and for -devel subpkg:
Obsoletes: PackageKit-qt-devel  0.8.6
Provides:  PackageKit-qt-devel = %{version}-%{release}


that seems to be the only review blocker i see.  there's a handful of smaller
SHOULD things we can do, but i'll be happy to wait until after import to help
implement that.

-- 
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 873813] Review Request: gnome-shell-search-github-repositories - Search your Github Repos from the gnome-shell

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=873813

--- Comment #9 from Ralph Bean rb...@redhat.com ---
Cool.  Thanks :)

Just for posterity, here's one more new release that fixes the ctrl-C issue we
talked about in IRC and removes those python macros.

Spec URL: http://threebean.org/rpm/gnome-shell-search-github-repositories.spec
SRPM URL:
http://threebean.org/rpm/gnome-shell-search-github-repositories-1.0.2-1.fc18.src.rpm

-- 
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 873813] Review Request: gnome-shell-search-github-repositories - Search your Github Repos from the gnome-shell

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=873813

Ralph Bean rb...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #10 from Ralph Bean rb...@redhat.com ---
New Package SCM Request
===
Package Name: gnome-shell-search-github-repositories
Short Description: Search your Github Repos from the gnome-shell
Owners: ralph
Branches: f18
InitialCC:

-- 
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 880155] Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

--- Comment #3 from Richard Hughes rhug...@redhat.com ---
(In reply to comment #2)
 1.  MUST: need some Obsoletes (and at least a temporary Provides) here to
 handle upgrade from older PackageKit-qt... 
 
 Obsoletes: PackageKit-qt  0.8.6
 Provides:  PackageKit-qt = %{version}-%{release}
 Provides:  PackageKit-qt%{?_isa} = %{version}-%{release}
 
 and for -devel subpkg:
 Obsoletes: PackageKit-qt-devel  0.8.6
 Provides:  PackageKit-qt-devel = %{version}-%{release}

Fixed in:

Spec URL: http://people.freedesktop.org/~hughsient/temp/PackageKit-Qt.spec
SRPM URL:
http://people.freedesktop.org/~hughsient/temp/PackageKit-Qt-0.8.6-1.fc18.src.rpm

Thanks for the super-speedy review.

Richard.

-- 
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 880155] Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #4 from Rex Dieter rdie...@math.unl.edu ---
invert the order of your changelog (newest first), and we have a winner. :)

please add me (rdieter) as comaintainer.

APPROVED

-- 
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 847322] Review Request: libnetfilter_cttimeout - Timeout policy tuning for Netfilter/conntrack

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=847322

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2012-11-26 11:20:33

-- 
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 761619] Review Request: cross-binutils - Multiple cross-build binutils

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=761619

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 CC||lemen...@gmail.com
 Resolution|--- |CURRENTRELEASE
Last Closed||2012-11-26 11:23:22

-- 
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 766166] Review Request: cross-gcc - Multiple cross-build gcc

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=766166

Bug 766166 depends on bug 761619, which changed state.

Bug 761619 Summary: Review Request: cross-binutils - Multiple cross-build 
binutils
https://bugzilla.redhat.com/show_bug.cgi?id=761619

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |CURRENTRELEASE

-- 
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 766166] Review Request: cross-gcc - Multiple cross-build gcc

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=766166

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 CC||lemen...@gmail.com
 Resolution|--- |CURRENTRELEASE
Last Closed||2012-11-26 11:24:19

--- Comment #22 from Peter Lemenkov lemen...@gmail.com ---
This package was imported and built for a very long time. All other questions
and issues should be discussed in a separate rhbz tickets.

-- 
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 873813] Review Request: gnome-shell-search-github-repositories - Search your Github Repos from the gnome-shell

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=873813

--- Comment #11 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 879435] Review Request: c-graph - an interactive package that demonstrates the mathematical operation of convolution

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879435

--- Comment #5 from Adrienne Thompson a...@codeartnow.com ---
Having given further thought to the problem, I have decided that my workload
does not allow the task of building the SRPM any time in the near future.

Extrapolating the download statistics for gnu.org, I estimate that since its
release last year, C-Graph's user base is somewhere in the tens of thousands.
GNU C-Graph seems likely to become a popular educational tool for engineering
signals/systems theory and computer vision.

Accordingly, you might consider undertaking the requisite packaging yourselves
as Fedora will find the investment of time and resources worthwhile. Please
advise me should you decide to do so.

-- 
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 880155] Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

--- Comment #5 from Richard Hughes rhug...@redhat.com ---
(In reply to comment #4)
 invert the order of your changelog (newest first), and we have a winner. :)

Gahh, Monday :) Sorry about that. Fixed.

-- 
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 880155] Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

--- Comment #6 from Richard Hughes rhug...@redhat.com ---
New Package SCM Request
===
Package Name: PackageKit-Qt
Short Description: Qt support library for PackageKit
Owners: rhughes rdieter
Branches: f17 f18
InitialCC: rdieter

-- 
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 880155] Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

  Flags||fedora-cvs?

-- 
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 880155] Review Request: PackageKit-Qt - Qt support library for PackageKit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880155

--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 853775] Review Request: gpick - Advanced color picker written in C++ using GTK+ toolkit

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=853775

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

  Flags||fedora-review?
  Flags||needinfo?(alexisis-pristont
   ||a...@hotmail.com)

-- 
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 841266] Review Request: plink - whole genome association analysis toolset

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=841266

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

  Flags||needinfo?(raymondpete@fedor
   ||aproject.org)

--- Comment #6 from Volker Fröhlich volke...@gmx.at ---
Are you still interested in this package?

-- 
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 878653] Review Request: NetworkManager-l2tp - NetworkManager VPN plugin for l2tp

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878653

--- Comment #8 from Ivan Romanov dr...@land.ru ---
License text issue fixed by upstream.

-- 
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 877763] Review Request: gnome-photos - Access, organize and share your photos on GNOME

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=877763

Ivan Romanov dr...@land.ru changed:

   What|Removed |Added

  Flags||fedora-review?

-- 
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 880195] Review Request: python-di - Dependency injection library for python unittesters

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880195

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

 CC||mario.blaetterm...@gmail.co
   ||m

--- Comment #1 from Mario Blättermann mario.blaetterm...@gmail.com ---
The source tarball contains a bundled egg-info. Please remove it before
building your package (in the %prep section):

rm -rf di.egg-info

See https://fedoraproject.org/wiki/Packaging:Python_Eggs#Upstream_Eggs for more
information.

Additionally, you will need python-setuptools in BuildRequires.

Another question: Are you planning to provide this package for EPEL 5?

-- 
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 879954] Review Request: rssh - Restricted shell for use with OpenSSH, allowing only scp and/or sftp

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879954

--- Comment #5 from Alex aadav...@learn.senecac.on.ca ---
yeah i am willing to do it

-- 
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 841266] Review Request: plink - whole genome association analysis toolset

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=841266

Ray Pete raymondp...@fedoraproject.org changed:

   What|Removed |Added

  Flags|needinfo?(raymondpete@fedor |
   |aproject.org)   |

--- Comment #7 from Ray Pete raymondp...@fedoraproject.org ---
Hi Volker,

Indeed I have updated the spec per your requests above. 
I'll submit the new srpm to koji and update the ticket when all is completed.

Regarding the .jar file. This is a gui application, which does not seem to
include src.
I also dealt with a few of the warnings mentioned above, but this is not
included on the website source yet.

Cheers,

Ray

-- 
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 875299] Review Request: slowhttptest - An Application Layer DoS attack simulator

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875299

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at
   Assignee|nob...@fedoraproject.org|volke...@gmx.at
  Flags||fedora-review?

-- 
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 877763] Review Request: gnome-photos - Access, organize and share your photos on GNOME

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=877763

--- Comment #6 from Ivan Romanov dr...@land.ru ---
I couldn't to use this application. When I started it it has no any photos and
I couldn't find any way to add them.

In .spec you use https://live.gnome.org/Design/Apps/Photos but About of
Gnome-Photos used https://live.gnome.org/GnomePhotos.
Probably it should be fixed.

https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Documentation
Any relevant documentation included in the source distribution should be
included in the package as %doc. Irrelevant documentation include build
instructions, the omnipresent INSTALL file containing generic build
instructions

[!]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 GPL (v2 or later) (with incorrect FSF address), LGPL (v2 or later),
 GPL (v2 or later), Unknown or generated. 4 files have unknown
 license. Detailed output of licensecheck in /home/taurus/877763-gnome-
 photos/licensecheck.txt
Use GPLv2+ and LGPLv2+ for License tag

Package Review
==

Key:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed



= MUST items =

C/C++:
[x]: Header files in -devel subpackage, if present.
[x]: Package does not contain any libtool archives (.la)
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package contains no bundled libraries.
[x]: Changelog in prescribed format.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm  4.4
[-]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install if there is
 such a file.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package complies to the Packaging Guidelines
[x]: Spec file lacks Packager, Vendor, PreReq tags.
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[!]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 GPL (v2 or later) (with incorrect FSF address), LGPL (v2 or later),
 GPL (v2 or later), Unknown or generated. 4 files have unknown
 license. Detailed output of licensecheck in /home/taurus/877763-gnome-
 photos/licensecheck.txt
[x]: The spec file handles locales properly.
[x]: Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: Package is named using only allowed ASCII characters.
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: Package do not use a name that already exist
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Package installs properly.
[x]: Package is not relocatable.
[x]: Requires correct, justified where necessary.
[x]: CheckResultdir
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file is legible and written in American English.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: Package contains systemd file(s) if in need.
[x]: File names are valid UTF-8.
[x]: Useful -debuginfo package or justification otherwise.
[-]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 71680 bytes 

[Bug 875299] Review Request: slowhttptest - An Application Layer DoS attack simulator

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875299

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #5 from Volker Fröhlich volke...@gmx.at ---
Antonio, you should actually fill out this form!


There are warnings about comparing unsigned and signed integers. Maybe you can
patch that and send it upstream!

Please ask upstream to provide a license file.

=APPROVED=

Package Review
==

Key:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated

= MUST items =

C/C++:
[x]: Header files in -devel subpackage, if present.
[x]: Package does not contain any libtool archives (.la)
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package contains no bundled libraries.
[x]: Changelog in prescribed format.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm  4.4
[-]: Macros in Summary, %description expandable at SRPM build time.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[-]: Package requires other packages for directories it uses.
[-]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package complies to the Packaging Guidelines
[x]: Spec file lacks Packager, Vendor, PreReq tags.
[-]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 Apache (v2.0). 1 files have unknown license. Detailed output of
 licensecheck in

/media/speicher1/makerpm/rpmbuild/SPECS/875299-slowhttptest/licensecheck.txt
[x]: Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: Package is named using only allowed ASCII characters.
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: Package do not use a name that already exist
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[-]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Package installs properly.
[x]: Package is not relocatable.
[-]: Requires correct, justified where necessary.
[x]: CheckResultdir
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file is legible and written in American English.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: Package contains systemd file(s) if in need.
[x]: File names are valid UTF-8.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[!]: 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]: Dist tag is present.
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: Package functions as described.

Ran one of the tests from the manpage

[x]: Latest version is packaged.
[x]: Package does not include license text 

[Bug 815018] Review Request: nodejs - javascript fast build framework

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=815018

Ron jokeyrh...@gmail.com changed:

   What|Removed |Added

 CC||jokeyrh...@gmail.com

--- Comment #46 from Ron jokeyrh...@gmail.com ---
This is great news. So Node.JS 0.10.x will be in the official repositories?

-- 
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 879932] Review Request: SocketW - It is a streaming socket C++ library designed to be easy to use

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879932

--- Comment #2 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info ---
Thank you for the comments. All addressed.
http://hubbitus.info/rpm/Fedora17/SocketW/SocketW.spec
http://hubbitus.info/rpm/Fedora17/SocketW/SocketW-031026-2.fc17.src.rpm

-- 
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 783294] Review Request: picolisp - Lisp Interpreter

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=783294

--- Comment #14 from Michael Shigorin shigo...@gmail.com ---
Created attachment 652233
  -- https://bugzilla.redhat.com/attachment.cgi?id=652233action=edit
spec accepted into ALT Linux Sisyphus

Here's a brief overview of changes to get it working (basically a closer look
at the INSTALL file and translation into spec lingo without trying to be even
smarter ass):

%define _libdir %_usr/lib

# ...

%install
mkdir -p %buildroot{%_bindir,%_libdir/%name,%_datadir}
cp -a bin/{picolisp,pil} %buildroot%_bindir/
# TODO: emacs subpackage
rm -rf lib/el
# lib/ and lib.l
cp -a lib* %buildroot%_libdir/%name/
# as per INSTALL
ln -s ../lib/%name %buildroot%_datadir/%name

%files
%doc CHANGES CREDITS README
%_bindir/*
%_libdir/%name/
%_datadir/%name/

-- 
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 875087] Review Request: pbsclusterviz - Visualise the status of PBS clusters

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875087

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at
 Blocks||177841 (FE-NEEDSPONSOR)

--- Comment #1 from Volker Fröhlich volke...@gmx.at ---
Hello Paul!

Thank you for your contribution. You must become a packager to maintain
packages in Fedora.

https://fedoraproject.org/wiki/Join_the_package_collection_maintainers

I left a comment on the ticket that you'll need a sponsor first.

vtk is only in EPEL 6. If you want your package to appear in EPEL 5, you'll
have to convince the vtk maintainer to create a build there. :)

---

Some comments:

License is GPLv2+

Use the name and version macro in Source0.

You don't need defattr anymore.

You must own %{_sysconfdir}/pbsclusterviz.d. and
%{python_sitelib}/pbsclusterviz

Drop all version constraints from BuildRequires. No maintained release of
Fedora or EL has older versions. I don't know whether you need those packages
while you're building. But if you need them during operation, you'll have to
make them Requires.

Installed (but unpackaged) file(s) found:
/usr/lib/python2.7/site-packages/pbsclusterviz-0.7a-py2.7.egg-info

Also include AUTHORS and COPYING. The postal address in COPYING is outdated.

-- 
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 863796] Review Request: printrun - RepRap printer interface and tools

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=863796

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at

--- Comment #6 from Volker Fröhlich volke...@gmx.at ---
The last SRPM link is dead.

-- 
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 879928] Review Request: rigsofrods - Vehicle simulator based on soft-body physics

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879928

Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

 Depends On||880364

-- 
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 855605] Review Request: hexchat - GTK IRC Client

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=855605

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at

--- Comment #5 from Volker Fröhlich volke...@gmx.at ---
Don't use macros for make, rm, chmod and install.

Header and source code files have executable permissions.

Please inform upstream to update the FSF address, if you haven't done it yet.

Make the build verbose so the exact command line becomes visible.

Drop the version constraints for glib2, gtk2 and bison. No maintained release
has versions older than that.

-- 
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 879931] Review Request: angelscript - AngelCode Scripting Library

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=879931

--- Comment #4 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info ---
(In reply to comment #1)
 Hm, the spec looks like mine from
 http://fedora.danny.cz/danny/development/SRPMS/repoview/angelscript.html :-)
Absolutely! You may find it copyright in sole changelog entry of my spec file.
So if you have Fedora account in packager group and wish (co)maintain this
packet - welcome.

I had packaged it as dependency to rigsofrods package. And with hope fix come
in upstream bug to interoperability with new version.
Also thank you for the patch. I'll look it. May be you are willing submit it
upstream? Are you author them?

-- 
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 875506] Review Request: python-fdb - Firebird RDBMS bindings for Python

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=875506

--- Comment #12 from Philippe Makowski makowski.fed...@gmail.com ---
thanks

-- 
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 863796] Review Request: printrun - RepRap printer interface and tools

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=863796

--- Comment #7 from Miro Hrončok m...@hroncok.cz ---
Sorry for that. I've only raised the revision number in the link and forget to
change the commit number and date.

Spec URL: https://raw.github.com/hroncok/SPECS/master/printrun.spec
SRPM URL:
https://github.com/downloads/hroncok/SPECS/printrun-0.0-12.20121103git6fa47668f2.fc17.src.rpm

-- 
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 880387] New: Review Request: python-pymongo - Python driver for MongoDB

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880387

Bug ID: 880387
   Summary: Review Request: python-pymongo - Python driver for
MongoDB
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: amcn...@mcnabbs.org

Spec URL: http://aml.cs.byu.edu/~amcnabb/python-pymongo.spec
SRPM URL: http://aml.cs.byu.edu/~amcnabb/python-pymongo-2.3-1.fc17.src.rpm
Description:

This review request is for renaming the pymongo package to python-pymongo. It
also adds initial support for Python 3.

Fedora Account System Username: amcnabb

-- 
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 880387] Review Request: python-pymongo - Python driver for MongoDB

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880387

--- Comment #1 from Andrew McNabb amcn...@mcnabbs.org ---
I submitted this srpm to koji, and it built without any errors:

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

-- 
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 874905] Review Request: php-drush-drush - Command line shell and Unix scripting interface for Drupal

2012-11-26 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=874905

--- Comment #9 from Shawn Iwinski shawn.iwin...@gmail.com ---
(In reply to comment #8)

There has to be some sort of cache somewhere.  I had a system with drush custom
installed in /usr/local/bin and /usr/share/drush.  I removed the custom
install, installed these RPMS, and get the same type error you are seeing
(except showing that it is trying to point to the old location):

# which drush
/usr/bin/drush
# drush --version
-bash: /usr/local/bin/drush: No such file or directory
# /usr/bin/drush --version
drush version 5.7

I found ~/.drush and /tmp/drush-env.  I removed both directories but the
test result was the same.  I rebooted the VM and then the drush command worked
perfectly fine.  I need to figure out a way to make it work without a reboot...

Also, version 5.8.0 was released so I need to update the RPM to that version
too.

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

  1   2   >