[Bug 860515] Review Request: mate-themes - MATE Desktop themes

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860515

--- Comment #3 from Jens Petersen  ---
For F18 and F19 can't build until mate-settings-daemon is built.

http://koji.fedoraproject.org/koji/getfile?taskID=4536295&name=root.log&offset=-4000

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860515] Review Request: mate-themes - MATE Desktop themes

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860515

--- Comment #4 from Dan Mashal  ---
Will fix asap

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860249] Review Request: adobe-sourcecodepro-fonts - A set of mono-spaced OpenType fonts designed for coding environments

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860249

--- Comment #16 from Tobias Florek  ---
hi,

spec: http://www.math.hu-berlin.de/~florek/adobe-source-code-pro-fonts.spec
srpm:
http://www.math.hu-berlin.de/~florek/adobe-source-code-pro-fonts-1.009-3.fc17.src.rpm

but honestly, i don't care what package gets chosen. if a known packager's
package comes first, no problem. i can send diffs when a new version comes out
anyway.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856114] Review Request: qxmpp - Qt XMPP Libraries

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856114

--- Comment #11 from Fedora Update System  ---
Package qxmpp-0.7.3-2.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing qxmpp-0.7.3-2.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-14949/qxmpp-0.7.3-2.fc17
then log in and leave karma (feedback).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856114] Review Request: qxmpp - Qt XMPP Libraries

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856114

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856545] Review Request: python-amqpclt - Versatile AMQP client

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856545

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856545] Review Request: python-amqpclt - Versatile AMQP client

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856545

--- Comment #26 from Fedora Update System  ---
python-amqpclt-0.4-1.fc17 has been pushed to the Fedora 17 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826520] Review Request: hiera - A simple hierarchical database supporting plugin data sources

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826520

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826520] Review Request: hiera - A simple hierarchical database supporting plugin data sources

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826520

--- Comment #13 from Fedora Update System  ---
hiera-1.0.0-2.fc17 has been pushed to the Fedora 17 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830458] The afraid-dyndns package is competely broekn

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830458

Paul Flo Williams  changed:

   What|Removed |Added

 CC||p...@frixxon.co.uk
  Component|Package Review  |afraid-dyndns

--- Comment #3 from Paul Flo Williams  ---
(In reply to comment #1)
> I fixed the two issues above along with a number of other fixes rather some
> time ago but I no longer have a proper facility for packaging this stuff for
> Fedora... if you can package it, the latest code is at:
> 
> https://github.com/ekkis/afraid-dyndns

You will need to orphan the package, following this procedure:

https://fedoraproject.org/wiki/Orphaned_package_that_need_new_maintainers

(In reply to comment #2)
> In the hope that something is done, I'm changing the package on this bug to
> "Package Review" (I hope it means what I think it means).

I've changed the component back to afraid-dyndys, as the package is officially
still in Fedora, even in its non-working state. If Erick drops ownership, it
can be re-built by someone else.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860249] Review Request: adobe-source-code-pro-fonts - A set of mono-spaced OpenType fonts designed for coding environments

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860249

Paul Flo Williams  changed:

   What|Removed |Added

 CC||p...@frixxon.co.uk
Summary|Review Request: |Review Request:
   |adobe-sourcecodepro-fonts - |adobe-source-code-pro-fonts
   |A set of mono-spaced|- A set of mono-spaced
   |OpenType fonts designed for |OpenType fonts designed for
   |coding environments |coding environments

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860146] Review Request: vsqlite++ - A C++ Wrapper for SQLite

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860146

--- Comment #10 from Vinzenz Feenstra  ---
SPEC:
https://raw.github.com/vinzenz/fedora-packages/vsqlite++/0.3.9-3/vsqlite++/vsqlite++.spec
SRPM:
https://github.com/downloads/vinzenz/fedora-packages/vsqlite++-0.3.9-3.fc17.src.rpm

ChangeLog:
* Fri Sep 28 2012 Vinzenz Feenstra  - 0.3.9-3
- 

Koji build results for vsqlite++-0.3.9-3.fc17.src.rpm:

F17: http://koji.fedoraproject.org/koji/taskinfo?taskID=4536518

F18: http://koji.fedoraproject.org/koji/taskinfo?taskID=4536519

Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=4536517

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860146] Review Request: vsqlite++ - A C++ Wrapper for SQLite

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860146

--- Comment #11 from Vinzenz Feenstra  ---
New Package SCM Request
===
Package Name: vsqlite++ 
Short Description: Well designed C++ sqlite 3.x wrapper library
Owners: evilissimo
Branches: f17 f18
InitialCC: sochotni

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 845403] Review Request: onesixtyone - An efficient SNMP scanner - unretire

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845403

Michal Ambroz  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from Michal Ambroz  ---
Oops - sorry 

Package Change Request
==
Package Name: onesixtyone
Unretire Branches: master f17 f16 el5 el6
New Branches: f18
Owners: rebus

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 845403] Review Request: onesixtyone - An efficient SNMP scanner - unretire

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845403

--- Comment #6 from Jon Ciesla  ---
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 860515] Review Request: mate-themes - MATE Desktop themes

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860515

Rex Dieter  changed:

   What|Removed |Added

 Depends On||853692
   ||(mate-settings-daemon)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 853692] Review Request: mate-settings-daemon - MATE Desktop settings daemon

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853692

Rex Dieter  changed:

   What|Removed |Added

 Blocks||860515 (mate-themes)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 853692] Review Request: mate-settings-daemon - MATE Desktop settings daemon

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853692

--- Comment #20 from Rex Dieter  ---
fyi, imported, building for rawhide 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 853692] Review Request: mate-settings-daemon - MATE Desktop settings daemon

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853692

Rex Dieter  changed:

   What|Removed |Added

 Blocks|860515 (mate-themes)|

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860515] Review Request: mate-themes - MATE Desktop themes

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860515

Rex Dieter  changed:

   What|Removed |Added

 Depends On|853692  |
   |(mate-settings-daemon)  |

--- Comment #5 from Rex Dieter  ---
naming: ok

license: ok

sources: ok
7fda787ad446c43f13baceedcbd8de50  mate-themes-1.4.0.tar.xz


1. SHOULD: include
BuildArch: noarch
as seems to be all architecture-independent content


2. scriptets: NOT ok
MUST: add icon scriplets for each icon theme installed.  for example,

%post
touch --no-create %{_datadir}/icons/Fog &> /dev/null || :
touch --no-create %{_datadir}/icons/PrintLarge &> /dev/null || :
touch --no-create %{_datadir}/icons/Quid &> /dev/null || :
...

%postun
if [ $1 -eq 0 ] ; then
touch --no-create %{_datadir}/icons/Fog &> /dev/null || :
touch --no-create %{_datadir}/icons/PrintLarge &> /dev/null || :
touch --no-create %{_datadir}/icons/Quid &> /dev/null || :
...
gtk-update-icon-cache %{_datadir}/icons/Fog &> /dev/null || :
gtk-update-icon-cache %{_datadir}/icons/PrintLarge &> /dev/null || :
gtk-update-icon-cache %{_datadir}/icons/Quid &> /dev/null || :
fi

%posttrans
gtk-update-icon-cache %{_datadir}/icons/Fog &> /dev/null || :
gtk-update-icon-cache %{_datadir}/icons/PrintLarge &> /dev/null || :
gtk-update-icon-cache %{_datadir}/icons/Quid &> /dev/null || :
...


3. MUST fix BuildRequires.  Current BuildRequires is way too much, and missing
a couple items, this minimal set seems to work for me:

BuildRequires: icon-naming-utils mate-common mate-doc-utils
# aka gtk2-engines-devel
BuildRequires: pkgconfig(gtk-engines-2)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860249] Review Request: adobe-source-code-pro-fonts - A set of mono-spaced OpenType fonts designed for coding environments

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860249

--- Comment #17 from Matt Rose  ---
(In reply to comment #16)
> hi,
> 
> but honestly, i don't care what package gets chosen. if a known packager's
> package comes first, no problem. i can send diffs when a new version comes
> out anyway.

I'm not a known packager either, I'm willing to maintain the package going
forward, as it is the only really nice monospace font I've been able to find
for Fedora, and fonts matter.

But I've only started the process of becoming a fedora contributor by following
the steps outlined in the wiki, basically I just need a sponsor at this point,
so either way is fine.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860249] Review Request: adobe-source-code-pro-fonts - A set of mono-spaced OpenType fonts designed for coding environments

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860249

--- Comment #18 from Matt Rose  ---
(In reply to comment #14)
> (In reply to comment #12)
> > (In reply to comment #9)
> > > (In reply to comment #7)

> 
> Note that "raw" URLs would have been more helpful. I'm probably not the only
> one who uses save link/wget/curl/etc on them just to get github's html/xml.
> 


Yeah, I realized that I didn't post the raw URLs from github the first time.  I
fixed that in comment #13

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860249] Review Request: adobe-source-code-pro-fonts - A set of mono-spaced OpenType fonts designed for coding environments

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860249

jos...@joshuajensen.org changed:

   What|Removed |Added

 CC||jos...@joshuajensen.org

--- Comment #19 from jos...@joshuajensen.org ---
Love it!  Lets add Source Sans Pro too...
http://www.google.com/webfonts/specimen/Source+Sans+Pro

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832698] Review Request: CERT Triage tools - a gdb extension similar to microsoft's !exploitable

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832698

Josh Bressers  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW

--- Comment #5 from Josh Bressers  ---
This review needs to stop. I found some GPL code in this package. I'm going to
consult Fontana on what's next.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857705] Review Request: dataquay - Simple RDF for C++ and Qt applications

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857705

Matthew Miller  changed:

   What|Removed |Added

 CC||mat...@redhat.com
   Assignee|nob...@fedoraproject.org|mat...@redhat.com
  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 858060] qpid-snmpd - SNMP agent for qpid broker

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=858060

Irina Boverman  changed:

   What|Removed |Added

 Blocks||701335

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 858841] Review Request: owncloud - Private file sync and share server

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=858841

--- Comment #4 from Gregor Tätzner  ---
(In reply to comment #3)
> just a few drive-by comments:
> - is that main package functional without any of those database
> sub-packages?
No, not really. Though you can't install owncloud without at least one database
sub-package because the main pkg requires: %{name}-database =
%{version}-%{release}

> - you also need to require rsyslog (for dir /etc/rsyslog.d), 
>   note: there's also syslog-ng, you're fine to uninstall rsyslog from your
> system
I think I will switch back to the internal logging mechanism of owncloud. It
doesn't require any kind of syslog, so we don't have to bother with the correct
dependencies :) 

> - the svg-editor is also LGPLv3 and MIT
oc 4.0 does not yet include svg-edit, but I've added a note

thanks for the comment

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857705] Review Request: dataquay - Simple RDF for C++ and Qt applications

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857705

--- Comment #1 from Matthew Miller  ---
Note: https://bugzilla.redhat.com/show_bug.cgi?id=857709

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857705] Review Request: dataquay - Simple RDF for C++ and Qt applications

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857705

--- Comment #2 from Matthew Miller  ---
rpmlint spelling warnings only; they're noise and can be ignored.

  dataquay.x86_64: W: spelling-error %description -l en_US datastore -> data
store, data-store, devastator
  dataquay.x86_64: W: spelling-error %description -l en_US datastores -> data
stores, data-stores, devastator
  dataquay.x86_64: W: spelling-error %description -l en_US schemas -> schema,
sachems, schemes
  dataquay.x86_64: W: spelling-error %description -l en_US metadata -> meta
data, meta-data, metatarsal
  dataquay.x86_64: W: spelling-error %description -l en_US hoc -> ho, choc,
hock
  dataquay.x86_64: W: spelling-error %description -l en_US runtime -> run time,
run-time, rudiment

I agree with your comment that the actual license text is more like MIT than
BSD. 


All of the MUST items are met.


Package builds in mock, with selinux off -- see bug #857709. If we're going to
put this in before that's fixed, maybe a comment in the spec file should
reference the bug?

The package description notes that the datastore is changeable at compile time.
I think it should note that the Fedora package is built with Redland (and maybe
that the docs recommend that for general-purpose packaging?).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 755510] Review Request: gnome-shell-extension-system-monitor-applet - Gnome shell system monitor extension

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=755510

Taunus  changed:

   What|Removed |Added

 CC||codezi...@suomi24.fi

--- Comment #39 from Taunus  ---
This works on F17:

F-17 RPM URL:
http://dl.dropbox.com/u/25699833/Fedora/F-17/gnome-shell-extension-system-monitor-applet/gnome-shell-extension-system-monitor-applet-v18-0.1.git96a05d5.fc17.noarch.rpm

The current package gnome-shell-extension-systemMonitor.noarch from fedora rpm
repositories does not work at all.

About the extension: it is difficult to tell which core is running fast with
multi core processor.

Thanks for the 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 755510] Review Request: gnome-shell-extension-system-monitor-applet - Gnome shell system monitor extension

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=755510

--- Comment #40 from Taunus  ---
And I did not try the other packages

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856238] Review Request: scratch - Programming language learning environment for stories, games, music and art

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856238

--- Comment #9 from Matthew Miller  ---
Update on the Scratch.image bundle -- in Squeak (and Smalltalk in general),
this is actually the preferred form of modification. Regenerating the whole
thing from scratch (heh) is just not done. This is similar to etoys, which is
included already. (See
http://www.redhat.com/archives/fedora-packaging/2009-January/msg00126.html)

Therefore, I think this is okay. Will run it by the packaging committee,
though.

I will, though, also include the Squeak 2.0 image from which the Scratch image
is derived in the source RPM, so that it's all together. This plus the existing
Scratch image plus the .changes file found in the source constitutes upstream's
preferred form of modification.


Also, on the Apache 2.0 license issue: the license text implies that GPLv3 may
be used; that would be compatible with Apache 2.0, making that a non-issue.
I've asked upstream to clarify.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856238] Review Request: scratch - Programming language learning environment for stories, games, music and art

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856238

--- Comment #10 from Matthew Miller  ---
Updated packages:

Spec URL: http://mattdm.org/misc/fedora/scratch.spec
SRPM URL: http://mattdm.org/misc/fedora/scratch-1.4.0.7-2.fc17.mattdm.src.rpm


As noted above, there is still a licensing issue to resolve. Additionally,
sound does not work, but we'd like to fix that after the package is in rawhide.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 858025] python-django-helpdesk - A Django powered ticket tracker for small enterprise

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=858025

--- Comment #5 from Luis Bazan  ---
SPEC:
http://lbazan.fedorapeople.org/python-django-helpdesk.spec

SRPM:
http://lbazan.fedorapeople.org/python-django-helpdesk-0.1.7b-3.fc17.src.rpm

css file contain nothing does nothing and I take the desition to remove this
file.

Best Regards!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 859671] Review Request: opencpn - A free and open source software for marine navigation

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=859671

--- Comment #7 from Volker Fröhlich  ---
We found the software to use several parts of GDAL, that do not belong to the
public API. This needs work 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 861498] New: Review Request: jdbi - A SQL convenience library for Java

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=861498

Bug ID: 861498
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: jdbi - A SQL convenience library for
Java
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/jdbi.spec
SRPM URL: http://gil.fedorapeople.org/jdbi-2.39.1-1.fc16.src.rpm
Description: jDBI is designed to provide convenient tabular data access in
Java(TM). It uses the Java collections framework for query
results, provides a convenient means of externalizing SQL
statements, and provides named parameter support for any database
being used.
Fedora Account System Username: gil

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 815060] Review Request: classmate - Java introspection library

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=815060

gil cattaneo  changed:

   What|Removed |Added

 Blocks||861498

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861498] Review Request: jdbi - A SQL convenience library for Java

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=861498

gil cattaneo  changed:

   What|Removed |Added

 Depends On||815060

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 815060] Review Request: classmate - Java introspection library

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=815060

gil cattaneo  changed:

   What|Removed |Added

 Blocks||852330

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 852330] Review Request: hibernate - Relational persistence and query service

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852330

gil cattaneo  changed:

   What|Removed |Added

 Depends On||815060

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861502] New: Review Request: metrics - Java library which gives you what your code does in production

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=861502

Bug ID: 861502
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: metrics - Java library which gives you
what your code does in production
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/metrics.spec
SRPM URL: http://gil.fedorapeople.org/metrics-2.1.3-1.fc16.src.rpm
Description: Metrics is a Java library which gives you unparalleled insight
into what your code does in production.

Developed by Yammer to instrument their JVM-based back-end services,
Metrics provides a powerful toolkit of ways to measure the behavior
of critical components in your production environment.

With modules for common libraries like Jetty, Logback, Log4j,
Apache HttpClient, Ehcache, JDBI, Jersey and reporting back-ends like
Ganglia and Graphite, Metrics provides you with full-stack visibility.

For more information, please see the documentation.

This package provides the Metrics Core Library.
Fedora Account System Username: gil

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 825347] Review Request: jersey - JAX-RS (JSR 311) production quality Reference Implementation

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=825347

gil cattaneo  changed:

   What|Removed |Added

 Blocks||861502

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861502] Review Request: metrics - Java library which gives you what your code does in production

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=861502

gil cattaneo  changed:

   What|Removed |Added

 Depends On||861498, 825347

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861498] Review Request: jdbi - A SQL convenience library for Java

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=861498

gil cattaneo  changed:

   What|Removed |Added

 Blocks||861502

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 815060] Review Request: classmate - Java introspection library

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=815060

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785462] Review Request: sinfo - Show an overview of the available computers an their current load

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=785462

--- Comment #10 from sophiekovalev...@fedoraproject.org ---
There are the new spec and rpm.

Spec URL: http://sophiekovalevsky.fedorapeople.org/sinfo.spec
SRPM URL: http://sophiekovalevsky.fedorapeople.org/sinfo-0.0.47-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 845403] Review Request: onesixtyone - An efficient SNMP scanner - unretire

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845403

--- Comment #7 from Michal Ambroz  ---
Hello,
seems that package is still blocked for build.
I have created ticket to engineering team to unblock the package in koji.
https://fedorahosted.org/rel-eng/ticket/5340

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 827649] Review Request: supercat - colorized cat

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=827649

--- Comment #16 from Fabian Affolter  ---
How about those two warnings?

supercat.x86_64: W: manual-page-warning /usr/share/man/man1/spc.1.gz 112:
warning: macro `r':reverse' not defined
supercat.x86_64: W: manual-page-warning /usr/share/man/man1/spc.1.gz 123:
warning: macro `r':RE.' not defined

run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install is
only needed if supporting EPEL5

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 853514] Review Request: rubygem-clouddb - Ruby interface into the Rackspace Cloud DB service

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853514

Guillermo Gómez  changed:

   What|Removed |Added

 CC||guillermo.go...@gmail.com
   Assignee|nob...@fedoraproject.org|guillermo.go...@gmail.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 853514] Review Request: rubygem-clouddb - Ruby interface into the Rackspace Cloud DB service

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853514

Guillermo Gómez  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #1 from Guillermo Gómez  ---
Unneeded macros...

%global gem_dir /usr/share/rubygems
%global gem_instdir %{gem_dir}/gems/%{gem_name}-%{version}
%global gem_docdir %{gem_dir}/doc/%{gem_name}-%{version}
%global gem_libdir %{gem_instdir}/lib

All of them are provided by rubygems-devel.

Source0:http://rubygems.org/downloads/clouddb-0.0.1.gem 

Please use macros for name and version, sample:

Source0:   http://rubygems.org/downloads/%{gem_name}-%{version}.gem 

The rest looks good to me :) Please fix to go a second round.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 853514] Review Request: rubygem-clouddb - Ruby interface into the Rackspace Cloud DB service

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853514

--- Comment #2 from Alejandro_Perez  ---
Changes done as suggested thanks
rpmlint rubygem-clouddb.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
rpmlint SRPMS/rubygem-clouddb-0.0.1-2.fc17.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
rpmlint RPMS/noarch/rubygem-clouddb-0.0.1-2.fc17.noarch.rpm 
rubygem-clouddb.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832635] Review Request: pencil - A sketching and GUI prototyping tool

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832635

--- Comment #14 from Truong Anh Tuan  ---
(In reply to comment #13)

All issues have been addressed. Upstream developers have updated the sources
and released as the new one 2.0.1.

The new .spec file: http://tuanta.fedorapeople.org/pencil/pencil.spec
The new .src file:
http://tuanta.fedorapeople.org/pencil/pencil-2.0.1-1.fc17.src.rpm

Please review them for me. Thanks.

See some other comments below.

> [!]: 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.
> [!]: MUST License field in the package spec file matches the actual license.
>  Note: Checking patched sources after %prep for licenses. No licenses
>  found. Please check the source files for licenses manually.

This has been fixed.

> [!]: SHOULD Package has no %clean section with rm -rf %{buildroot} (or
>  $RPM_BUILD_ROOT)
>  Note: Clean is needed only if supporting EPEL5

Updated.

> [!]: 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.
> See: http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text

Has been fixed. Keep only one license file (COPYING in root folder)

> Rpmlint
> ---
> Checking: pencil-2.0-2.fc19.src.rpm
>   pencil-2.0-2.fc19.noarch.rpm
> pencil.src: W: invalid-url Source0:
> http://evoluspencil.googlecode.com/files/pencil-2.0.tar.gz HTTP Error 404:
> Not Found

I think it's normal with source in Google code. You can try to download
directly: http://evoluspencil.googlecode.com/files/pencil-2.0.1.tar.gz

> pencil.noarch: W: no-manual-page-for-binary pencil

There is no manual page in upstream source (it may be not necessary). Is it ok?

> Issues:
> 
> - please clarify with upstream, which license is is the one to use, ask
> upstream to delete the other
> - there are several files from windows included. please delete them:
> /usr/share/pencil/chrome/icons/default/main-window.ico
> /usr/share/pencil/chrome/icons/default/pencilMainWindow.ico
> /usr/share/pencil/icons/default/main-window.ico
> /usr/share/pencil/skin/classic/images/pencil.ico
> /usr/share/pencil/skin/classic/pencil.ico

Those files have been removed.

> - what are those files for:
> /usr/share/pencil/platform/WINNT
> /usr/share/pencil/platform/WINNT/chrome.manifest

Removed.

> - settings: path to external bitmap editor: is that correctable? currently,
> it points to /usr/local/bin/gimp-2.7 which is clearly wrong for several
> reasons.

Updated.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 854345] Review Request: smartsim - Digital logic circuit design and simulation package

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854345

Eric Smith  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #2 from Eric Smith  ---
New Package SCM Request
===
Package Name: smartsim
Short Description: Digital logic circuit design and simulation package
Owners: brouhaha
Branches: f17 f18 el6
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 759823] Review Request: libkdtree++ - C++ template container implementation of kd-tree sorting

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=759823

--- Comment #10 from Eric Smith  ---
Thanks for reviewing this. I really appreciate the recommendations as to how to
fix the problems. I've taken most but not all of the advice, as further
explained below. Here are the updated files:

Spec URL: http://fedorapeople.org/~brouhaha/libkdtree++/libkdtree++.spec
SRPM URL:
http://fedorapeople.org/~brouhaha/libkdtree++/libkdtree++-0.7.0-3.fc17.src.rpm

I don't intend to support EL5 unless someone requests it, so for now I'm not
adding the requires for pkgconfig.

If examples depends on devel, rpmlint gives the errors "E: devel-dependency"
and "E: explicit-lib-dependency libkdtree++-devel". While these conditions are
basically self-explanatory, it's not obvious to me why they are considered to
be errors, and I don't see an explanation of them on the "Common rpmlint
issues" wiki page.

I'm not convinced about installing kdtree.py with 755 permissions, as it isn't
intended to be executed directly. If it's 755, rpmlint gives the error "E:
script-without-shebang". All of the other .py files in my
/usr/lib/python2.7/site-packages directory (from various other Fedora RPM
files) are 644, and only of of them has a shebang.  The RPMs providing these
.py files with 644 perms are:

abrt-addon-python-2.0.13-1.fc17.x86_64
dbus-python-0.83.0-9.fc17.x86_64
python-decorator-3.3.3-1.fc17.noarch
python-setuptools-0.6.27-2.fc17.noarch
python-GnuPGInterface-0.3.2-9.fc17.noarch
sssd-1.8.4-14.fc17.x86_64
python-IPy-0.75-2.fc17.noarch
python-magic-5.11-2.fc17.x86_64
python-setuptools-0.6.27-2.fc17.noarch
sssd-1.8.4-14.fc17.x86_64

Also, installing it with 755 does not seem to eliminate the
unstripped-binary-or-object warning, even with a "%filter_provides_in
%{python_sitearch}/.*\.so$", and I still get an empty-debuginfo-package unless
I keep the macro to disable automatic debuginfo processing.

I'm unclear on whether the Python packaging standard requires 755 in a case
like this, but given that the actual practice is clearly to use 644, for now
I'm keeping 644.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861548] New: Review Request: znerd-oss-parent - Znerd.org OSS Parent

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=861548

Bug ID: 861548
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: znerd-oss-parent - Znerd.org OSS
Parent
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/znerd-oss-parent.spec
SRPM URL: http://gil.fedorapeople.org/znerd-oss-parent-3-1.fc16.src.rpm
Description: Parent for znerd.org OSS Projects.
Fedora Account System Username: gil

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861548] Review Request: znerd-oss-parent - Znerd.org OSS Parent

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=861548

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861548] Review Request: znerd-oss-parent - Znerd.org OSS Parent

2012-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=861548

gil cattaneo  changed:

   What|Removed |Added

 Blocks||861549

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review