[Bug 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mohamed El Morabity  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #13 from Mohamed El Morabity  2010-11-17 
02:44:54 EST ---
Looks OK now :)

This package is APPROVED.

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


[Bug 614299] Review Request: python-ordereddict - Py2.7's new collections.OrderedDict that works in Python 2.4-2.6.

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Jens Petersen  2010-11-17 01:55:15 
EST ---
https://fedoraproject.org/wiki/Package_SCM_admin_requests#New_Packages

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


[Bug 457343] Review Request: jquery - Fast, concise library that simplifies how you use javascript

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Garcia  changed:

   What|Removed |Added

 CC||webwha...@gmail.com

--- Comment #2 from Steven Garcia  2010-11-17 01:46:43 EST 
---
It seems as though no efforts have been made on this for over two years and two
months. This review appears to be abandoned or forgotten. I would very much
like to maintain the jquery package. I have rewritten the jquery spec using an
existing working model MochiKit. Please, allow me to resubmit the jquery
package for review. Can someone give me a hint as to if this is possible and
how I can regain review submission for this package?

Spec URL: http://aikiframework.org/files/jquery/jquery.spec
SRPM URL: http://aikiframework.org/files/jquery/jquery-1.4.4-1.fc13.src.rpm

Description:
jQuery is a new type of JavaScript library.

jQuery is a fast, concise, JavaScript Library that simplifies how you traverse
HTML documents, handle events, perform animations, and add Ajax interactions
to your web pages. jQuery is designed to change the way that you write
JavaScript.

If possible, this would be my third package review submission to Fedora and I
need a sponsor. I set the jquery package to require the sizzle package which
can be seen here https://bugzilla.redhat.com/show_bug.cgi?id=654151.

Ran successful rpmlint on spec, srpm and rpm
$ rpmlint SPECS/jquery.spec \
SRPMS/jquery-1.4.4-1.fc13.src.rpm \
RPMS/noarch/jquery-1.4.4-1.fc13.noarch.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

Ran successful mock rebuild
$ mock --rebuild SRPMS/jquery-1.4.4-1.fc13.src.rpm 
INFO: mock.py version 1.1.6 starting...
State Changed: init plugins
INFO: selinux disabled
State Changed: start
INFO: Start(SRPMS/jquery-1.4.4-1.fc13.src.rpm)  Config(fedora-13-i386)
State Changed: lock buildroot
State Changed: clean
INFO: chroot (/var/lib/mock/fedora-13-i386) unlocked and deleted
State Changed: init
State Changed: lock buildroot
Mock Version: 1.1.6
INFO: Mock Version: 1.1.6
INFO: enabled root cache
State Changed: unpacking root cache
INFO: enabled yum cache
State Changed: cleaning yum metadata
INFO: enabled ccache
State Changed: running yum
State Changed: setup
State Changed: build
INFO: Done(SRPMS/jquery-1.4.4-1.fc13.src.rpm) Config(default) 0 minutes 26
seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-13-i386/result

Ran successful koji scratch builds
https://koji.fedoraproject.org/koji/tasks?owner=fosdevel&state=all
$ koji build --scratch dist-f13 jquery-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605788
$ koji build --scratch dist-f13-kde jquery-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605777
$ koji build --scratch dist-f13-updates-candidate jquery-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605785
$ koji build --scratch dist-f14 jquery-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605775
$ koji build --scratch dist-f14-gobject jquery-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605780
$ koji build --scratch dist-f14-kde jquery-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605774
$ koji build --scratch dist-f14-updates-candidate jquery-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605783
$ koji build --scratch dist-f15 jquery-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605782

A JavaScript library similar in packaging was approved and has
existed in Fedora for quite a while. The name of the package is MochiKit which
can be seen here http://pkgs.fedoraproject.org/gitweb/?p=MochiKit.git;a=tree.
Here is a link to the approved review
https://bugzilla.redhat.com/show_bug.cgi?id=176528.

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


[Bug 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Cédric OLIVIER  2010-11-17 
01:42:43 EST ---
Thanks.

So you can find here an updated package with the last release and modifications
according your last comment.

Spec file :
http://cedric.olivier.free.fr/rpms/rec-applet-0.2.3-3/rec-applet.spec
SRPM file:
http://cedric.olivier.free.fr/rpms/rec-applet-0.2.3-3/rec-applet-0.2.3-3.fc13.src.rpm

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

[Bug 644899] Review Request: python-PyWavelets - Python wavelet transforms module

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Chen Lei  2010-11-17 01:27:27 EST ---
several issues:
1.
Name:  python-PyWavelets
->
Name:  python-pywt

See
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Addon_Packages_.28python_modules.29

2.
BuildRequires: python-setuptools is useless for this package.

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


[Bug 652544] Review Request: erlang-rpm-macros - Macros for simplifying building of Erlang packages

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2010-11-17 01:21:02

--- Comment #9 from Peter Lemenkov  2010-11-17 01:21:02 EST 
---
Ok, the package is available in Rawhide so I'm closing this.

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


[Bug 639263] Review Request: erlang-rebar - Erlang Build Tools

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 639263 depends on bug 639278, which changed state.

Bug 639278 Summary: Review Request: erlang-lfe - Lisp Flavoured Erlang
https://bugzilla.redhat.com/show_bug.cgi?id=639278

   What|Old Value   |New Value

 Status|ASSIGNED|MODIFIED
 Status|MODIFIED|ON_QA
 Resolution||RAWHIDE
 Status|ON_QA   |CLOSED

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


[Bug 639278] Review Request: erlang-lfe - Lisp Flavoured Erlang

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||RAWHIDE
Last Closed||2010-11-17 01:19:27

--- Comment #18 from Peter Lemenkov  2010-11-17 01:19:27 
EST ---
Ok, the package is available in Rawhide so I'm closing this.

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


[Bug 573910] Review Request: dcmtk - Offis DICOM Toolkit

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Andreas Loening  changed:

   What|Removed |Added

   Flag|needinfo?(loen...@alum.mit. |
   |edu)|

--- Comment #12 from Andreas Loening  2010-11-17 01:06:18 
EST ---
Hi Peter,

I've had precious little time to devote to side projects, so if Mario would be
interested in being the lead on the packaging that'd be quite okay with me.

Mario, did you ever get the soname issue worked out?

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


[Bug 554103] Review Request: fgrun - Graphical frontend for launching FlightGear flight simulator

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from pankaj pandey  2010-11-17 00:53:10 EST 
---
I'm unable to package for fedora as the some tools (koji) don't work behind
proxy (c#6). I thought Fabrice may take it up, but since it was quite long back
i'm not sure if he's still around. If thats the case then i think the review
request may need to be dropped. I'll be glad if there's anything i can do.

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


[Bug 506425] Review Request: lightsquid - a light, small and fast log analyzer for squid proxy

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Ruediger Landmann  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||r.landm...@redhat.com
 AssignedTo|nob...@fedoraproject.org|r.landm...@redhat.com
   Flag||needinfo?(alek...@oscada.or
   ||g.ua)

--- Comment #38 from Ruediger Landmann  2010-11-16 
23:39:01 EST ---
Hi Aleksey: just a few more things to look at here:

 - = N/A
 / = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [!] Rpmlint output is clean:

===

$ rpmlint SPECS/lightsquid.spec 
SPECS/lightsquid.spec: W: no-%build-section
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

* OK

$ rpmlint SRPMS/lightsquid-1.8-7.fc14.src.rpm 
lightsquid.src: W: spelling-error Summary(en_US) analizer -> analyzer,
canalize, nationalizer

* Please correct to "analyzer"; "Small" would also be better than "Little"
here:

%description
%{name} Small and fast Squid log analyzer.

* Also, I suggest:

Summary: Light, small, and fast log analyzer for Squid proxy

* Also, for the subpackage, I suggest:

%package apache
Summary: Web controls for %{name}
Group: Applications/Internet
Requires: %{name} = %{version}-%{release}
Requires: httpd
%description apache
Configuration files and scripts for Apache.



lightsquid.src: W: spelling-error %description -l en_US analizer -> analyzer,
canalize, nationalizer
lightsquid.src: W: strange-permission lightsquid.spec 0600L

* Should be 644

lightsquid.src: W: no-%build-section
1 packages and 0 specfiles checked; 0 errors, 4 warnings.



$ rpmlint RPMS/noarch/lightsquid-1.8-7.fc14.noarch.rpm 
lightsquid.noarch: W: spelling-error Summary(en_US) analizer -> analyzer,
canalize, nationalizer
lightsquid.noarch: W: spelling-error %description -l en_US analizer ->
analyzer, canalize, nationalizer
lightsquid.noarch: E: executable-marked-as-config-file
/etc/cron.daily/lightsquid

* Remove:

%config(noreplace) %{_sysconfdir}/cron.daily/lightsquid

this is not a config file (yes, rpmlint will then complain
"non-conffile-in-etc", but this is expected)

lightsquid.noarch: E: script-without-shebang /etc/cron.daily/lightsquid

* Please fix

lightsquid.noarch: W: no-manual-page-for-binary lightparser.pl

* Unfortunate, but not a blocker

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

$ rpmlint RPMS/noarch/lightsquid-apache-1.8-7.fc14.noarch.rpm 
lightsquid-apache.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

===

 [/] Package is named according to the Package Naming Guidelines.
 [/] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [/] Package meets the Packaging Guidelines including the Language specific
items
 [/] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [!] License field in the package spec file matches the actual license.
 License type in spec file: GPLv2

* Files in package state "GNU General Public License as published by the Free
Software Foundation; either version 2 of the License, or (at your option) any
later version", therefore GPLv2+



 [/] 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.
* GPL is in gnugpl.txt
 [/] Spec file is legible and written in American English.
 [!] Sources used to build the package matches the upstream source, as provided
in the spec URL.

* md5sums do not match:
$ md5sum SOURCES/lightsquid-1.8.tgz 
9b7de3d45da769ae058ac06318c56c54  SOURCES/lightsquid-1.8.tgz
$ md5sum ~/Download/lightsquid-1.8.tgz 
310a7e16017290232183753d73f3559d  /home/rlandmann/Download/lightsquid-1.8.tgz

* Please express Source0 as:
Source0: http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz

http://fedoraproject.org/wiki/Packaging/SourceURL#Sourceforge.net



 [/] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested: http://koji.fedoraproject.org/koji/taskinfo?taskID=2605683
 [/] Package is not known to require ExcludeArch
 [/] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly (with the %find_lang macro)
 [-] ldconfig called in %post and %postun if required.
 [/] Package does not bundle copies of system libraries
 [/] P

[Bug 644899] Review Request: python-PyWavelets - Python wavelet transforms module

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chen Lei  changed:

   What|Removed |Added

   Flag||fedora-review?

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


[Bug 522821] Review Request: bluetile - A modern tiling window manager with a gentle learning curve

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jens Petersen  changed:

   What|Removed |Added

   Flag||needinfo?(peter...@redhat.c
   ||om)

Bug 522821 depends on bug 633194, which changed state.

Bug 633194 Summary: Review Request: ghc-glade - Haskell binding to libglade2
https://bugzilla.redhat.com/show_bug.cgi?id=633194

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|MODIFIED
 Status|MODIFIED|ON_QA
 Resolution||ERRATA
 Status|ON_QA   |CLOSED

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


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

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jens Petersen  changed:

   What|Removed |Added

   Flag||fedora-review?

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


[Bug 607584] Review Request: wordgroupz - A vocabulary building application

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 607584] Review Request: wordgroupz - A vocabulary building application

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #41 from Fedora Update System  
2010-11-16 22:41:40 EST ---
wordgroupz-0.3.1-2.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/wordgroupz-0.3.1-2.fc13

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


[Bug 607584] Review Request: wordgroupz - A vocabulary building application

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #42 from Fedora Update System  
2010-11-16 22:41:47 EST ---
wordgroupz-0.3.1-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/wordgroupz-0.3.1-2.fc14

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


[Bug 593800] Review Request: python-keyring - keyring module for python

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 593800] Review Request: python-keyring - keyring module for python

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #48 from Fedora Update System  
2010-11-16 22:36:54 EST ---
python-keyring-0.4-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/python-keyring-0.4-1.fc13

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


[Bug 593800] Review Request: python-keyring - keyring module for python

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #47 from Fedora Update System  
2010-11-16 22:36:47 EST ---
python-keyring-0.4-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/python-keyring-0.4-1.fc14

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


[Bug 569671] Review Request: elliptics - Distributed hash table storage

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

  Status Whiteboard||StalledSubmitter

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


[Bug 570394] Review Request: perl-SQL-Beautify - An SQL Beautifier Written in Perl

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Jason Tibbitts  2010-11-16 22:29:32 EST 
---
Was the license ever clarified?

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


[Bug 593800] Review Request: python-keyring - keyring module for python

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Ratnadeep Debnath  changed:

   What|Removed |Added

   Flag|needinfo?(rtn...@gmail.com) |

--- Comment #46 from Ratnadeep Debnath  2010-11-16 22:22:15 
EST ---
Sorry for the absence. :(
I got busy here with submission of my research paper for review, plus college
seminars.

I am submitting a scratch build for my packages for f15. 
Do I need to make a SCM request for f15? I cannot switch to f15 in my
fedora-git folder.

$fedpkg switch-branch f15 
Unable to switch to another branch: Unknown remote branch f15

Apart from that, for pushing the package to bodhi, please check if the fields
chosen are correct :
Type : newpackage
Request : Testing
Bugs : #593800
and others default values

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


[Bug 611328] Review Request: hydra - Fast login cracker

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

   Flag||needinfo?(haeb...@gmail.com
   ||)

--- Comment #6 from Jason Tibbitts  2010-11-16 22:13:37 EST 
---
There's been plenty of commentary on this ticket but no response at all from
the submitter.  Marcus, please respond or we'll have to close this ticket out.

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


[Bug 654151] Review Request: sizzle - JavaScript CSS selector engine

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Steven Garcia  2010-11-16 21:56:35 EST 
---
Here are some reference notes for anyone interested in reviewing this package
(sizzle). A JavaScript library similar in packaging was approved and has
existed in Fedora for quite a while. The name of the package is MochiKit which
can be seen here http://pkgs.fedoraproject.org/gitweb/?p=MochiKit.git;a=tree.
Here is a link to the approved review
https://bugzilla.redhat.com/show_bug.cgi?id=176528.

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


[Bug 532241] Review Request: emacs-identica-mode - Identica mode for emacs.

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Arun SAG  2010-11-16 21:03:10 EST ---
Updated to the latest version: (the info file needs fix)

SRPM URL:
http://sagarun.fedorapeople.org/SRPMS/emacs-identica-mode-1.0-1.fc13.src.rpm

Spec URL: http://sagarun.fedorapeople.org/SPECS/emacs-identica-mode.spec

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


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

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jens Petersen  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|peter...@redhat.com

--- Comment #1 from Jens Petersen  2010-11-16 21:03:00 EST 
---
Created attachment 460968
  --> https://bugzilla.redhat.com/attachment.cgi?id=460968
haddock.spec-1.patch

Patch to clean up a bit:

- fix base package group
- use upstream main webpage
- don't need filelist file for base package

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


[Bug 654151] Review Request: sizzle - JavaScript CSS selector engine

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Steven Garcia  2010-11-16 20:58:06 EST 
---
Here is the link to Source1:

Conf URL: http://aikiframework.org/files/sizzle/sizzle.conf

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


[Bug 654151] New: Review Request: sizzle - JavaScript CSS selector engine

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: sizzle - JavaScript CSS selector engine

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

   Summary: Review Request: sizzle - JavaScript CSS selector
engine
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: webwha...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://aikiframework.org/files/sizzle/sizzle.spec
SRPM URL: http://aikiframework.org/files/sizzle/sizzle-1.4.4-1.fc13.src.rpm

Description: A pure-JavaScript CSS selector engine designed to be easily
dropped in to a host library.

This is my second package review submission to Fedora and I need a sponsor.

Ran successful rpmlint on spec, srpm and rpm
$ rpmlint SPECS/sizzle.spec \
SRPMS/sizzle-1.4.4-1.fc13.src.rpm \
RPMS/noarch/sizzle-1.4.4-1.fc13.noarch.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

Ran successful mock rebuild
$ mock --rebuild SRPMS/sizzle-1.4.4-1.fc13.src.rpm 
INFO: mock.py version 1.1.6 starting...
State Changed: init plugins
INFO: selinux disabled
State Changed: start
INFO: Start(SRPMS/sizzle-1.4.4-1.fc13.src.rpm)  Config(fedora-13-i386)
State Changed: lock buildroot
State Changed: clean
INFO: chroot (/var/lib/mock/fedora-13-i386) unlocked and deleted
State Changed: init
State Changed: lock buildroot
Mock Version: 1.1.6
INFO: Mock Version: 1.1.6
INFO: enabled root cache
State Changed: unpacking root cache
INFO: enabled yum cache
State Changed: cleaning yum metadata
INFO: enabled ccache
State Changed: running yum
State Changed: setup
State Changed: build
INFO: Done(SRPMS/sizzle-1.4.4-1.fc13.src.rpm) Config(default) 0 minutes 24
seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-13-i386/result

Ran successful koji scratch builds
https://koji.fedoraproject.org/koji/tasks?owner=fosdevel&state=all
$ koji build --scratch dist-f13 sizzle-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605578
$ koji build --scratch dist-f13-kde sizzle-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605576
$ koji build --scratch dist-f13-updates-candidate sizzle-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605572
$ koji build --scratch dist-f14 sizzle-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605573
$ koji build --scratch dist-f14-gobject sizzle-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605565
$ koji build --scratch dist-f14-kde sizzle-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605570
$ koji build --scratch dist-f14-updates-candidate sizzle-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605567
$ koji build --scratch dist-f15 sizzle-1.4.4-1.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2605564

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


[Bug 654151] Review Request: sizzle - JavaScript CSS selector engine

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steven Garcia  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

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


[Bug 648100] Review Request: ghc-xml - A simple XML library

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jens Petersen  changed:

   What|Removed |Added

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

--- Comment #1 from Jens Petersen  2010-11-16 20:20:18 EST 
---
Will try to look at this soon.

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


[Bug 634037] Review Request: ghc-MissingH - Large utility library

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jens Petersen  2010-11-16 20:16:01 EST 
---
ping-a-ling?

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


[Bug 532241] Review Request: emacs-identica-mode - Identica mode for emacs.

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michel Alexandre Salim  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||michel+...@michelsylvain.in
   ||fo
 AssignedTo|nob...@fedoraproject.org|michel+...@michelsylvain.in
   ||fo
   Flag||fedora-review?

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


[Bug 654139] Review Request: ease - GNOME desktop presentation application

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Arun SAG  changed:

   What|Removed |Added

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

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


[Bug 654139] Review Request: ease - GNOME desktop presentation application

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Michel Alexandre Salim  
2010-11-16 19:37:56 EST ---
I'm taking a slightly unusual approach to specifying BuildRequires:, 
specifying pkgconfig modules instead of the usual -devel package names. 
This aligns better with configure.ac (so one can easily verify that the 
two are in sync), and build problems due to API changes would abort the 
build earlier (during root FS setup rather than during the rebuild phase).

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


[Bug 654139] Review Request: ease - GNOME desktop presentation application

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Michel Alexandre Salim  
2010-11-16 19:31:29 EST ---
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2605434

rpmlint:
SRPM:
ease.src:53: E: hardcoded-library-path in /usr/lib
==> this is due to the rpath fix, adding /lib64 and /usr/lib64 to the system
libdirs so that libtool does not add RPATHs for /usr/lib64 in the linked
binaries

ease.src: W: no-cleaning-of-buildroot %clean
ease.src: W: no-buildroot-tag
ease.src: W: no-%clean-section

==> only targeting recent Fedora releases

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

ease-*.x86_64.rpm:
ease.x86_64: W: no-manual-page-for-binary ease
ease-devel.x86_64: W: no-documentation
3 packages and 0 specfiles checked; 0 errors, 2 warnings.

note: rpmlint-ing the installed RPM results in a lot of
undefined-non-weak-symbol warnings; this might shed some light into the issue:
http://sourceware.org/ml/libc-alpha/2003-05/msg00034.html

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


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

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Thomas Spura  changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org
 Blocks||177841(FE-NEEDSPONSOR)

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


[Bug 654139] New: Review Request: ease - GNOME desktop presentation application

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: ease - GNOME desktop presentation application

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

   Summary: Review Request: ease - GNOME desktop presentation
application
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: michel+...@michelsylvain.info
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://salimma.fedorapeople.org/specs/gnome/ease.spec
SRPM URL: http://salimma.fedorapeople.org/specs/gnome/ease-0.4-1.fc14.src.rpm
Description:
Ease is a modern desktop presentation application for GNOME, with
Clutter-based visuals.

With a simple design, Ease is painless to use. It is intended to
integrate well into your current GNOME-based desktop.

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


[Bug 648679] Review Request: jabber-roster - A Python tool for listing your Jabber roster contacts

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Thomas Spura  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||toms...@fedoraproject.org
 AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org
   Flag||fedora-review+

--- Comment #1 from Thomas Spura  2010-11-16 
19:18:27 EST ---
rpmlint:
$ rpmlint ./jabber-roster-0.1.0-1.fc13.src.rpm
./noarch/jabber-roster-0.1.0-1.fc13.noarch.rpm 
jabber-roster.src: W: no-buildroot-tag
jabber-roster.noarch: W: no-manual-page-for-binary jabber-roster
2 packages and 0 specfiles checked; 0 errors, 2 warnings.

- no libs
- noarch ok
- no *.la
- BR/R ok
- License ok
- macros everywhere
- %files ok


SHOULD:
- please query upstream to include a proper license file instead of links to
the web.

_


APPROVED

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


[Bug 226405] Merge Review: selinux-doc

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Garrett Holmstrom  changed:

   What|Removed |Added

   Flag||fedora-review?

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


[Bug 553649] Review Request: rhn-custom-info - Set and list custom values for RHN-enabled machines

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Garrett Holmstrom  2010-11-16 
19:01:13 EST ---
Created attachment 460960
  --> https://bugzilla.redhat.com/attachment.cgi?id=460960
Review for new package rhn-custom-info-5.4.2-1.fc14

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


[Bug 553649] Review Request: rhn-custom-info - Set and list custom values for RHN-enabled machines

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Garrett Holmstrom  changed:

   What|Removed |Added

   Flag||fedora-review?

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


[Bug 226419] Merge Review: sip

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Garrett Holmstrom  changed:

   What|Removed |Added

   Flag||fedora-review?

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


[Bug 553649] Review Request: rhn-custom-info - Set and list custom values for RHN-enabled machines

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Garrett Holmstrom  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|gho...@fedoraproject.org

--- Comment #3 from Garrett Holmstrom  2010-11-16 
18:59:56 EST ---
I meant to come back to this one, but it somehow managed to slip through a
crack somewhere.  Sorry.  I see only a couple minor issues at the moment, so
just fix those up and you should be good to go.  I will attach the full review
shortly.

- Changelog in prescribed format

Please add a hyphen to the beginning of line 74.  Sorry for nitpicking.  ;-)

- Requires correct, justified where necessary

If you are building this against EL4 you should Require python with a versioned
dependency, and then only after testing that you're building on < EL5.  If you
aren't building against EL4 then you should instead drop that Requires
directive altogether since rpmbuild will pick that up on its own.

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


[Bug 560852] Review Request: devmon-templates - Templates for Devmon SNMP Device Monitoring for Hobbit/BigBrother

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 560852 depends on bug 560853, which changed state.

Bug 560853 Summary: Review Request: devmon - SNMP Device Monitoring for 
Hobbit/BigBrother
https://bugzilla.redhat.com/show_bug.cgi?id=560853

   What|Old Value   |New Value

 Resolution||DEFERRED
 Status|NEW |CLOSED

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


[Bug 560853] Review Request: devmon - SNMP Device Monitoring for Hobbit/BigBrother

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Colin Coe  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||DEFERRED
Last Closed||2010-11-16 18:50:38

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


[Bug 560852] Review Request: devmon-templates - Templates for Devmon SNMP Device Monitoring for Hobbit/BigBrother

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Colin Coe  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||DEFERRED
Last Closed||2010-11-16 18:49:52

--- Comment #2 from Colin Coe  2010-11-16 18:49:52 EST ---
I need to cancel this and associated requests as I won't have the time to
maintain any more packages.

Thank you for looking though.

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


[Bug 561073] Review Request: gist - Absolutely the best command line gister

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Jason Tibbitts  2010-11-16 18:43:30 EST 
---
Unfortunately pings on tickets don't really work because very few people will
see them (and it makes it look as if there's been recent activity, which has
the opposite effect from what you want).

The upstream source seems to be properly versioned now; it looks like they've
recently gone to 2.0.0.  Care to update?  Unfortunately a properly versioned
package is actually older (rpm-wise) than the version you've submitted, but
that shouldn't cause any real problems.  (This is why we usually recommend
using a version of 0 and release of 0.20100201 instead unless you're sure that
upstream will never switch to using regular version numbers.)

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


[Bug 528461] Review Request: ocaml-fieldslib - OCaml library for folding over record fields

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jason Tibbitts  2010-11-16 18:40:15 EST 
---
No problem; just making sure it didn't fall through the cracks (again).

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


[Bug 226419] Merge Review: sip

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Rex Dieter  2010-11-16 18:42:08 EST 
---
git's master branch has the latest sip (which should be available upstream),
and I *think* defattr fixed too, fwiw.

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


[Bug 226419] Merge Review: sip

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Rex Dieter  2010-11-16 18:42:48 EST 
---
nvm, defattr fail there too. :(  fixing that now, easyfix.

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


[Bug 513874] Review Request: perl-Math-Calc-Units - Human-readable unit-aware calculator

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #19 from Jose Pedro Oliveira  2010-11-16 
18:37:07 EST ---
Ruben,

Do you mind if I request an EL-6 branch for this perl module and for
perl-Nagios-Plugin? I don't mind co-maintaining them.

Regards,
jpo

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


[Bug 226419] Merge Review: sip

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Garrett Holmstrom  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #1 from Garrett Holmstrom  2010-11-16 
18:31:27 EST ---
Below are the issues I found.  I will attach a full review shortly.

- License field in spec is correct
  sipgen/parser.c is GPLv3+ with exceptions
  sipgen/parser.h is GPLv3+ with exceptions

A quick IRC discussion indicated that this likely needs to appear in the
License field somewhere.  I'm hoping spot gets back to me for confirmation
about it to be sure, though perhaps it would be better to just ask legal to be
extra certain.

- License files installed when any subpackage combination is installed
  sip-macros package does not pull in license files

It looks like you create macros.sip yourself based on the program's API
version.  What license does it have?  Can you include that license along with
the sip-macros package?

- Sources match upstream unless altered to fix permissibility issues
  Upstream MD5:  No longer on upstream's server
  Your MD5:  0a591ef6e59aa16e56822d3eb9fe21b8  sip-4.10.5.tar.gz

Unfortunately, upstream seems to clean out old tarballs quite frequently.  I
wouldn't block the review because of that, though.

- Each %files section contains %defattr
  The python3 packages lack %defattr macros

Please add %defattr macros to the python3 packages' %files sections.

I hope that helps!

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


[Bug 226419] Merge Review: sip

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Garrett Holmstrom  2010-11-16 
18:31:59 EST ---
Created attachment 460953
  --> https://bugzilla.redhat.com/attachment.cgi?id=460953
Review for F14 package sip-4.10.5-2.fc14

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


[Bug 645354] Review Request: django-tracking - Django site visitor tracking, including basic blacklisting

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|django-tracking-0.2.7-1.fc1 |django-tracking-0.2.7-1.fc1
   |4   |3

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


[Bug 645354] Review Request: django-tracking - Django site visitor tracking, including basic blacklisting

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Fedora Update System  
2010-11-16 18:19:51 EST ---
django-tracking-0.2.7-1.fc14 has been pushed to the Fedora 14 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 645354] Review Request: django-tracking - Django site visitor tracking, including basic blacklisting

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Fedora Update System  
2010-11-16 18:20:49 EST ---
django-tracking-0.2.7-1.fc13 has been pushed to the Fedora 13 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 560852] Review Request: devmon-templates - Templates for Devmon SNMP Device Monitoring for Hobbit/BigBrother

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Jason Tibbitts  2010-11-16 18:21:49 EST 
---
Is this package still needed?  It looks like it's been merged into the devmon
package you submitted.

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


[Bug 649777] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #14 from Fedora Update System  
2010-11-16 18:15:28 EST ---
jorbis-0.0.17-4.fc14 has been pushed to the Fedora 14 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update jorbis'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/jorbis-0.0.17-4.fc14

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


[Bug 650180] Review Request: tkabber - Client for the Jabber instant messaging system

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|tkabber-0.11.1-2.svn1948.fc |tkabber-0.11.1-2.svn1948.fc
   |14  |13

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


[Bug 650180] Review Request: tkabber - Client for the Jabber instant messaging system

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Fedora Update System  
2010-11-16 18:15:46 EST ---
tkabber-0.11.1-2.svn1948.fc13 has been pushed to the Fedora 13 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 645354] Review Request: django-tracking - Django site visitor tracking, including basic blacklisting

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|django-tracking-0.2.7-1.fc1 |django-tracking-0.2.7-1.fc1
   |2   |4

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


[Bug 226419] Merge Review: sip

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Garrett Holmstrom  changed:

   What|Removed |Added

 CC||gho...@fedoraproject.org,
   ||rdie...@math.unl.edu
 AssignedTo|nob...@fedoraproject.org|gho...@fedoraproject.org

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


[Bug 612076] Review Request: ghc-feldspar-language - Functional Embedded Language for DSP and PARallelism

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #18 from Fedora Update System  
2010-11-16 18:13:52 EST ---
ghc-feldspar-language-0.3.1-3.fc13 has been pushed to the Fedora 13 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update ghc-feldspar-language'.  You
can provide feedback for this update here:
https://admin.fedoraproject.org/updates/ghc-feldspar-language-0.3.1-3.fc13

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


[Bug 645354] Review Request: django-tracking - Django site visitor tracking, including basic blacklisting

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||django-tracking-0.2.7-1.fc1
   ||2
 Resolution||ERRATA
Last Closed||2010-11-16 18:17:23

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


[Bug 579662] Review Request: emacs-lookup - Search Interface with Electronic Dictionaries for Emacs

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #13 from Fedora Update System  
2010-11-16 18:15:57 EST ---
emacs-lookup-1.4.1-3.fc13 has been pushed to the Fedora 13 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update emacs-lookup'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/emacs-lookup-1.4.1-3.fc13

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


[Bug 645354] Review Request: django-tracking - Django site visitor tracking, including basic blacklisting

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Fedora Update System  
2010-11-16 18:17:18 EST ---
django-tracking-0.2.7-1.fc12 has been pushed to the Fedora 12 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 652533] Review Request: pgRouting - Provides routing functionality to PostGIS/PostgreSQL

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #17 from Fedora Update System  
2010-11-16 18:14:12 EST ---
pgRouting-1.03-3.fc14 has been pushed to the Fedora 14 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update pgRouting'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/pgRouting-1.03-3.fc14

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


[Bug 635450] Review Request: docky - MacOS-like docker app

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #19 from Fedora Update System  
2010-11-16 18:13:57 EST ---
docky-2.0.7-3.fc14 has been pushed to the Fedora 14 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update docky'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/docky-2.0.7-3.fc14

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


[Bug 650180] Review Request: tkabber - Client for the Jabber instant messaging system

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version||tkabber-0.11.1-2.svn1948.fc
   ||14
 Resolution|NEXTRELEASE |ERRATA

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


[Bug 652835] Review Request: ktikz - Editor for the TikZ language

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System  
2010-11-16 18:12:09 EST ---
ktikz-0.10-1.fc13 has been pushed to the Fedora 13 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update ktikz'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/ktikz-0.10-1.fc13

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


[Bug 650180] Review Request: tkabber - Client for the Jabber instant messaging system

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Fedora Update System  
2010-11-16 18:12:28 EST ---
tkabber-0.11.1-2.svn1948.fc14 has been pushed to the Fedora 14 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 528461] Review Request: ocaml-fieldslib - OCaml library for folding over record fields

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Richard W.M. Jones  2010-11-16 18:05:27 
EST ---
Yeah sorry I meant to, but then obviously I didn't.  Give me a little
longer.  This package is important for a Fedora 15 proposed feature ...

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


[Bug 560071] Review Request: php-pecl-augeas - PHP bindings to the Augeas API

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Jason Tibbitts  2010-11-16 18:06:41 EST 
---
Pedro, have you done any work to obtain a sponsor? 
http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group has
some pointers.

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


[Bug 554103] Review Request: fgrun - Graphical frontend for launching FlightGear flight simulator

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

  Status Whiteboard||StalledSubmitter

--- Comment #12 from Jason Tibbitts  2010-11-16 18:03:57 EST 
---
Is anyone (besides Hans and I) listening?

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


[Bug 620385] Review Request: BEDTools - A flexible suite of utilities for comparing genomic features

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Adam Huffman  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution||CURRENTRELEASE
Last Closed||2010-11-16 17:58:51

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


[Bug 533558] Review Request: gtkwhiteboard - GTK Wiimote Whiteboard

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Paulo Roma Cavalcanti  2010-11-16 
17:06:40 EST ---
Lets keep the warnings, then:

Spec URL: http://orion.lcg.ufrj.br/RPMS/SPECS/gtkwhiteboard.spec

SRPM URL: http://roma.fedorapeople.org/srpms/gtkwhiteboard-1.3-4.fc14.src.rpm

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


[Bug 652257] Review Request: focuswriter - A fullscreen, distraction-free writing program

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Golo Fuchert  changed:

   What|Removed |Added

 CC||packa...@golotop.de

--- Comment #1 from Golo Fuchert  2010-11-16 16:54:05 EST 
---
Some initial comments on the package:
- No need to BuildRequire gcc-c++
  http://fedoraproject.org/wiki/Packaging:Guidelines#Exceptions_2
- %setup -q is sufficient, -n %{name}-%{version} is the standard argument
- PREFIX=%{_prefix} would be better than PREFIX=/usr since the standard
  might be changed one day.
- COPYING, ChangeLog and README should be included as %doc, COPYING is a must.
- Don't forget to update the icon cache when installing new icons:
http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache

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


[Bug 549228] Review Request: fosfor - A media player for Linux that uses the Clutter and GStreamer toolkits

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

   Keywords|Reopened|
  Status Whiteboard|StalledSubmitter|

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


[Bug 549228] Review Request: fosfor - A media player for Linux that uses the Clutter and GStreamer toolkits

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Hicham HAOUARI  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed|2009-12-28 06:45:44 |2010-11-16 16:41:39

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


[Bug 533558] Review Request: gtkwhiteboard - GTK Wiimote Whiteboard

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Jason Tibbitts  2010-11-16 16:39:18 EST 
---
For some reason python programmers like to put the '#!' line on files that are
not supposed to be executed directly.  I've no idea at all why they do this,
but it causes the rpmlint complaints you tried to fix.  You can either accept
the complaint (they're not sufficiently problematic to block acceptance of the
package) or you can fix them by removing the first line of the python source
files (with sed -i -e '1d'.  Making the files executable when they aren't
executables isn't the right solution.

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


[Bug 226405] Merge Review: selinux-doc

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Garrett Holmstrom  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|gho...@fedoraproject.org

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


[Bug 226405] Merge Review: selinux-doc

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Garrett Holmstrom  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||gho...@fedoraproject.org

--- Comment #1 from Garrett Holmstrom  2010-11-16 
16:31:02 EST ---
Most of this package's problems simply arise from its age.  Upstream seems to
be dead; is this package still relevant?  If it is no longer relevant maybe it
would be better to just retire it.  I will attach a full review shortly, but I
will also present the things that need to be fixed here.

- License files included in package %docs or not included in upstream source
- License files installed when any subpackage combination is installed

Just fix these with "%doc LICENSE".

- Sources match upstream unless altered to fix permissibility issues
  Upstream MD5:  ???
  Your MD5:  5836fbb58dbd20586415d7f4baa0b55b  selinux-doc-1.26.tgz

Is upstream dead?  rpmlint and I get 404s from the URI in the spec file, and I
failed to find a new upstream.

- Relocatability is justified

Does "Prefix: %{_prefix}" need to be there for some reason?  I would drop it,
but if it is important to the package then the spec file should say why.

- Has dist tag

This is not a blocker, but adding a dist tag wouldn't be a bad idea to get this
package in line with most of the others.

- Correct BuildRoot tag on < F10/EL6
- Correct %clean section on < F13/EL6

Since this package is only building on F12 and up you can dump both the
BuildRoot tag and the %clean section if you want.  This also isn't required,
though the buildroot given in the spec file is somewhat unusual.

- Text files encoded in ASCII or UTF-8
  README.HIERARCHY contains non-ISO extended-ASCII characters

Perhaps the file's encoding could be converted?

- File timestamps preserved by file ops

This isn't mandatory, but would you mind adding -p switches to cp commands?

One other thing that seemed off to me is the Group field.  Shouldn't it be
"Documentation"?

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


[Bug 226405] Merge Review: selinux-doc

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Garrett Holmstrom  2010-11-16 
16:31:57 EST ---
Created attachment 460934
  --> https://bugzilla.redhat.com/attachment.cgi?id=460934
Review for F14 package selinux-doc-1.26-5

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


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

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Golo Fuchert  2010-11-16 16:23:47 EST 
---
After having a closer look at the package I would like to point out the
following things:
- The files TODO and README are included in the package. However, they are 
  empty (already in the source tarball). I see no point in including these 
  files then. Maybe you can ask upstream if this is a mistake.
- If you update the version of the software you should reset the release number 
  to 1
- As far as I can tell it is not necessary to require xneur, this dependency is 
  found automagically during the build process.
- The icon tag in the .desktop file should not have an explicit file extension 
  (.png here)

Furthermore, Martin (from comment #3) hinted me at the following points:
- the autotools (autoconf, automake, autoheader, libtool, ...) are only needed
when existing configure.ac or Makefile.am files are changed. This is not the
case here and so they don't have to be included.
- It would be better to be more precise in the %file section. Excessive
globbing might lead to unwanted files being packaged. So it would be better to
make the following changes:
%{_bindir}/* =>%{_bindir}/gxneur
%{_mandir}/man?/*=>%{_mandir}/man1/gxneur.1*

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


[Bug 552113] Review Request: libwiiuse - library to use wiiremotes via bluetooth

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Jason Tibbitts  2010-11-16 16:09:29 EST 
---
Cool, thanks.

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


[Bug 634909] Review Request: v8 - JavaScript Engine

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Toshio Ernie Kuratomi  changed:

   What|Removed |Added

 CC||a.bad...@gmail.com

--- Comment #8 from Toshio Ernie Kuratomi  2010-11-16 
15:49:28 EST ---
I'm not committing to reviewing this but I noticed a couple things:

1) I see that we're installing jsmin.py... If this is a derivative of the jsmin
stuff that Douglas Crockford did then we can't do that.  jsmin is not under an
fsf free license (the silly "must be used for good, not evil clause does that
to us).

2) usually we don't make shared libraries if upstream is not making shared
libraries due to being afraid that we'll choose a version that upstream will
later stomp on.  The same idea (upstream reusing the version number we use for
a later, API incompatible, release of their own) would seem to apply here. 
However, I do see that Debian is providing versioned dynamic libs (Their
patches are different, though... perhaps we should adopt their approach?
http://packages.debian.org/experimental/libv8-2.4.7)  The best outcome would be
for upstream to start providing versioned dynamic libraries.  Have you
contacted them about this?  The debian patches are to the build files so they
could be an appropriate starting point for that discussion.

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


[Bug 533558] Review Request: gtkwhiteboard - GTK Wiimote Whiteboard

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Paulo Roma Cavalcanti  2010-11-16 
15:38:45 EST ---
(In reply to comment #5)
> Thanks.  This looks pretty good.  rpmlint says:
> 
>   gtkwhiteboard.noarch: E: explicit-lib-dependency python-xlib
> Which is bogus.
> 
>   gtkwhiteboard.noarch: E: script-without-shebang
>/usr/lib/python2.7/site-packages/gtkwhiteboard-1.3/linuxWiimoteLib.py
> Why is this file executable?  Actually, why are any of the .py files
> executable?  They're not intended to be run directly.  This should be fixed.

Just to make rpmlint happy. Otherwise, we get:


[cascavel:/home/mock/fedora-14-x86_64/result] rpmlint
gtkwhiteboard-1.3-4.fc14.noarch.rpm
gtkwhiteboard.noarch: E: explicit-lib-dependency python-xlib
gtkwhiteboard.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/gtkwhiteboard-1.3/gtkwhiteboard.py 0644
/usr/bin/python
gtkwhiteboard.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/gtkwhiteboard-1.3/mousecontrol.py 0644
/usr/bin/python
gtkwhiteboard.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/gtkwhiteboard-1.3/perspective.py 0644
/usr/bin/env
gtkwhiteboard.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/gtkwhiteboard-1.3/whiteboard.py 0644
/usr/bin/python
gtkwhiteboard.noarch: W: no-manual-page-for-binary gtkwhiteboard
1 packages and 0 specfiles checked; 5 errors, 1 warnings.

What should I do? Keep the files 755 or use 644 and get those warnings?

> 
>   gtkwhiteboard.noarch: W: no-manual-page-for-binary gtkwhiteboard
> It would be nice to have a manual page but it's not mandatory.
> 
> Where did whii.png come from?  I am concerned that a picture of a Wii
> controller with the Wii logo visible as an icon may pose a legal issue. 
> Perhaps I'm just being paranoid, but I know Nintendo is fond of lawsuits so it
> would be good to have the legal folks give their OK.  I suspect that if this 
> is
> an issue, fixing it should be easy since the icon does not come from upstream
> at all.  If it came from someplace on the net, that may also pose a copyright
> issue.

I think I just cut and paste from here:

http://computer.yourdictionary.com/images/computer/_WII.GIF

The picture is really nice for an icon ...

> 
> The desktop file has a problem:
> 
> /builddir/build/BUILDROOT/gtkwhiteboard-1.3-3.fc14.x86_64/usr/share/applications/gtkwhiteboard.desktop:
> error: (will be fatal in the future): value "GNOME" in key "Categories" in
> group "Desktop Entry" requires another category to be present among the
> following categories: GTK

This is easy to fix. 

> 
> * source files match upstream.  sha256sum:
>   9610f498bb1711aff2898fdaa9533319fbb2fb74c7b2588c8387b969ed510e2c
>gtkwhiteboard-1.3.zip
> * package meets naming and versioning guidelines.
> * specfile is properly named, is cleanly written and uses macros consistently.
> * summary is OK.
> * description is OK.
> * dist tag is present.
> * license field matches the actual license (for the code, at least).
> * license is open source-compatible.
> * license text included in package (in README file).
> * latest version is being packaged.
> * BuildRequires are proper.
> * package builds in mock (14, x86_64).
> * package installs properly.
> X rpmlint has valid complaints.
> * final provides and requires are sane:
>gtkwhiteboard = 1.3-3.fc14
>   =
>/bin/sh  
>/usr/bin/env  
>/usr/bin/python  
>pybluez  
>python(abi) = 2.7
>python-xlib  
>wxPython  
> 
> * owns the directories it creates.
> * doesn't own any directories it shouldn't.
> * no duplicates in %files.
> X file permissions are odd.
> * no generically named files
> * code, not content.
> * documentation is small, so no -doc subpackage is necessary.
> * %docs are not necessary for the proper functioning of the package.
> X desktop file has an error.

I am looking forward to hearing from you, so I can produce the final .src.rpm
version.

Thanks.

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


[Bug 552113] Review Request: libwiiuse - library to use wiiremotes via bluetooth

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Simo Sorce  2010-11-16 15:34:59 EST ---
I've got distracted and haven't updated it.
Will see if I can find out some time in the week end.

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


[Bug 552113] Review Request: libwiiuse - library to use wiiremotes via bluetooth

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Jason Tibbitts  2010-11-16 15:05:34 EST 
---
Did anything ever happen with this package?  I see plenty of commentary but no
updated srpm.

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


[Bug 553210] Review Request: nanny - Parental Control

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jason Tibbitts  2010-11-16 15:07:29 EST 
---
Did anything ever happen with this ticket?  Could potential reviewers get an
updated package to look at?

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


[Bug 553893] Review Request: rubygem-tiny_mce - TinyMCE in Ruby on Rails applications

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks||201449(FE-DEADREVIEW)
 Resolution||NOTABUG
Last Closed||2010-11-16 15:01:14

--- Comment #3 from Jason Tibbitts  2010-11-16 15:01:14 EST 
---
The ticket this depends on was closed due to lack of response.

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


[Bug 567241] Review Request: taxbird 0.15 -- Need Sponsor

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks|177841(FE-NEEDSPONSOR)  |201449(FE-DEADREVIEW)
 Resolution||NOTABUG
   Flag|needinfo?(dirk-gottsch...@i |
   |t-internet-service.de)  |
Last Closed||2010-11-16 14:59:31

--- Comment #6 from Jason Tibbitts  2010-11-16 14:59:31 EST 
---
The ticket this depends on was closed due to lack of response; this should be
closed as well.

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


[Bug 528461] Review Request: ocaml-fieldslib - OCaml library for folding over record fields

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Jason Tibbitts  2010-11-16 14:43:02 EST 
---
Did you want to move forward with this?  I know it sat in the queue for a
while, and if you no longer wish to continue with the package then feel free to
just close this ticket.

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


[Bug 533558] Review Request: gtkwhiteboard - GTK Wiimote Whiteboard

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 Blocks||182235(FE-Legal)
 AssignedTo|nob...@fedoraproject.org|ti...@math.uh.edu
   Flag||fedora-review?

--- Comment #5 from Jason Tibbitts  2010-11-16 14:39:41 EST 
---
Thanks.  This looks pretty good.  rpmlint says:

  gtkwhiteboard.noarch: E: explicit-lib-dependency python-xlib
Which is bogus.

  gtkwhiteboard.noarch: E: script-without-shebang
   /usr/lib/python2.7/site-packages/gtkwhiteboard-1.3/linuxWiimoteLib.py
Why is this file executable?  Actually, why are any of the .py files
executable?  They're not intended to be run directly.  This should be fixed.

  gtkwhiteboard.noarch: W: no-manual-page-for-binary gtkwhiteboard
It would be nice to have a manual page but it's not mandatory.

Where did whii.png come from?  I am concerned that a picture of a Wii
controller with the Wii logo visible as an icon may pose a legal issue. 
Perhaps I'm just being paranoid, but I know Nintendo is fond of lawsuits so it
would be good to have the legal folks give their OK.  I suspect that if this is
an issue, fixing it should be easy since the icon does not come from upstream
at all.  If it came from someplace on the net, that may also pose a copyright
issue.

The desktop file has a problem:

/builddir/build/BUILDROOT/gtkwhiteboard-1.3-3.fc14.x86_64/usr/share/applications/gtkwhiteboard.desktop:
error: (will be fatal in the future): value "GNOME" in key "Categories" in
group "Desktop Entry" requires another category to be present among the
following categories: GTK

* source files match upstream.  sha256sum:
  9610f498bb1711aff2898fdaa9533319fbb2fb74c7b2588c8387b969ed510e2c
   gtkwhiteboard-1.3.zip
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* license field matches the actual license (for the code, at least).
* license is open source-compatible.
* license text included in package (in README file).
* latest version is being packaged.
* BuildRequires are proper.
* package builds in mock (14, x86_64).
* package installs properly.
X rpmlint has valid complaints.
* final provides and requires are sane:
   gtkwhiteboard = 1.3-3.fc14
  =
   /bin/sh  
   /usr/bin/env  
   /usr/bin/python  
   pybluez  
   python(abi) = 2.7
   python-xlib  
   wxPython  

* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
X file permissions are odd.
* no generically named files
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
X desktop file has an error.

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


[Bug 519009] Review Request: gnome-do-docklets - Docklets for gnome-do

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #34 from Fedora Update System  
2010-11-16 14:05:38 EST ---
gnome-do-docklets-0.8.2-3.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/gnome-do-docklets-0.8.2-3.fc13

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


[Bug 499992] Review Request: mingw32-webkitgtk - MinGW Windows web content engine library

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #20 from Jason Tibbitts  2010-11-16 14:03:14 EST 
---
I may have led you astray; Fedora doesn't need it and EL6 doesn't need it, but
el5, being ancient, still needs BuildRoot, %clean and so on.

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


[Bug 519009] Review Request: gnome-do-docklets - Docklets for gnome-do

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 519009] Review Request: gnome-do-docklets - Docklets for gnome-do

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #33 from Fedora Update System  
2010-11-16 14:04:44 EST ---
gnome-do-docklets-0.8.2-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/gnome-do-docklets-0.8.2-3.fc14

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


[Bug 499992] Review Request: mingw32-webkitgtk - MinGW Windows web content engine library

2010-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Erik van Pienbroek  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #19 from Erik van Pienbroek  
2010-11-16 13:57:26 EST ---
Thanks for the review!

New Package SCM Request
===
Package Name: mingw32-webkitgtk
Short Description: MinGW Windows web content engine library
Owners: epienbro
Branches: f14 el5 el6
InitialCC:

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


  1   2   >