[Bug 546147] Merge Review: kasumi

2009-12-23 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=546147





--- Comment #7 from fujiwara tfuji...@redhat.com  2009-12-23 03:24:41 EDT ---
(In reply to comment #6)
 Well, how should I think of desktop file for now?  

I think reverting the desktop file in the rpm would be a workaround.
It's a problem when a language specific tool is shown on other language menu
items.
It would be better to remove the .desktop file after ibus-anthy language bar
could show the menu.
That's why I asked to file a bug in ibus-anthy.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549915] Review Request: FreeMat - An interpreted, matrix-oriented development environment

2009-12-23 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=549915





--- Comment #3 from Jussi Lehtola jussi.leht...@iki.fi  2009-12-23 03:24:02 
EDT ---
(In reply to comment #2)
 (In reply to comment #1)
  Ugh. blas.ini in %{_bindir}??
 Yes... it is in the same place that upstream places it (with this strange 644
 permission, already fixed). Is it the wrong place?

Absolutely. It should be placed in %{_datadir}/%{name}/. Please ask upstream to
fix 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 546147] Merge Review: kasumi

2009-12-23 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=546147


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks||550001




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 546147] Merge Review: kasumi

2009-12-23 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=546147


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks|550001  |
 Depends on||550001




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 546147] Merge Review: kasumi

2009-12-23 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=546147





--- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
03:46:29 EDT ---
(In reply to comment #7)
 (In reply to comment #6)
  Well, how should I think of desktop file for now?  
 
 I think reverting the desktop file in the rpm would be a workaround.
 It's a problem when a language specific tool is shown on other language menu
 items.
 It would be better to remove the .desktop file after ibus-anthy language bar
 could show the menu.
 That's why I asked to file a bug in ibus-anthy.  

Filed as  bug 550001

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549863] Review Request: plexus-interpolation - Plexus Interpolation API

2009-12-23 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=549863


Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Alexander Kurtakov akurt...@redhat.com  2009-12-23 
03:51:16 EDT ---
New Package CVS Request
===
Package Name: plexus-interpolation
Short Description: Plexus Interpolation API
Owners: akurtakov
Branches: 
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 521993] New Package for Dogtag PKI:pki-setup

2009-12-23 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=521993





--- Comment #10 from Kevin Wright kwri...@redhat.com  2009-12-23 03:58:22 EDT 
---
removed all Requires for perl packages.

updated spec file and src rpm:

Spec URL: http://people.redhat.com/kwright/pki-setup/pki-setup.spec

SRPM URL:
http://people.redhat.com/kwright/pki-setup/pki-setup-1.3.0-6.fc11.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 522207] New Package for Dogtag PKI: pki-common

2009-12-23 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=522207





--- Comment #9 from Parag AN(पराग) panem...@gmail.com  2009-12-23 05:02:01 
EDT ---
1) Scratch build again failed for rawhide, See log
http://koji.fedoraproject.org/koji/getfile?taskID=1887806name=root.log

May I request you to please check spec file and see if all BuildRequires
packages for this pki-common package is built in rawhide.

This time we are missing dogtag-pki-common-ui package.

2) But I also observed that build
(http://koji.fedoraproject.org/koji/taskinfo?taskID=1887883) is successful if
we remove 

BuildRequires:  dogtag-pki-common-ui

3) I also see few more things can be cleaned up in spec file
for Requires: lines

you can replace following lines
Requires:   %{_javadir}/pki/cmsutil.jar
Requires:   %{_javadir}/pki/nsutil.jar

with simple

Requires:   pki-util

4) There is no rhgb package in Fedora active branches but yes yum will give you
its replacement plymouth-system-theme which is empty(meta) package. So why
would this package need theme 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 521993] New Package for Dogtag PKI:pki-setup

2009-12-23 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=521993


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #11 from Parag AN(पराग) panem...@gmail.com  2009-12-23 05:12:26 
EDT ---
Thanks.

Koji build:-http://koji.fedoraproject.org/koji/taskinfo?taskID=1887911

As issue2 in above comment is not blocking this review. I will approve this
package.

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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 522207] New Package for Dogtag PKI: pki-common

2009-12-23 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=522207





--- Comment #10 from Parag AN(पराग) panem...@gmail.com  2009-12-23 05:18:16 
EDT ---
5) you can change defattr of -javadoc subpackage to
%defattr(-,root,root,-)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 521995] New Package for Dogtag PKI:pki-java-tools

2009-12-23 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=521995


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review+




--- Comment #7 from Parag AN(पराग) panem...@gmail.com  2009-12-23 05:33:28 
EDT ---
Review:
+ package builds in mock (rawhide i686).
koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=1887819
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url (sha1sum)
f8e9e4d1230ed80cfe38a5f28b5e96dc655ba54c  pki-java-tools-1.3.0.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Not a GUI application


Suggestions:
1) don't forget to change defattr of javadoc subpackage to
%defattr(-,root,root,-)

2) again why this package have
Requires:   pki-native-tools
though this is not a blocker.

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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453422] Review Request: songbird - Mozilla based multimedia player

2009-12-23 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=453422


Andreas Petzold andreas.petz...@gmail.com changed:

   What|Removed |Added

 CC||andreas.petz...@gmail.com




--- Comment #93 from Andreas Petzold andreas.petz...@gmail.com  2009-12-23 
05:54:42 EDT ---
Reply to comment 92:

I've downloaded the binary tarball Songbird_1.4.2-1434_linux-i686.tar.gz
yesterday, but it's not working for me on F12:

(songbird-bin:21336): GLib-WARNING **: g_set_prgname() called multiple times

(songbird-bin:21343): GStreamer-WARNING **: Failed to load plugin
'/home/petzold/download/Songbird/gst-plugins/libgstpulse.so':
/usr/lib/libsndfile.so.1: undefined symbol: vorbis_version_string   

(songbird-bin:21343): GStreamer-WARNING **: Failed to load plugin
'/usr/lib/gstreamer-0.10/libgstjpeg.so': /usr/lib/gstreamer-0.10/libgstjpeg.so:
undefined symbol: _gst_debug_get_category   

(songbird-bin:21343): GStreamer-WARNING **: Failed to load plugin
'/usr/lib/gstreamer-0.10/libgstvideo4linux2.so':
/usr/lib/gstreamer-0.10/libgstvideo4linux2.so: undefined symbol:
_gst_debug_get_category 

(songbird-bin:21343): GStreamer-WARNING **: Failed to load plugin
'/usr/lib/gstreamer-0.10/libgstapp.so': /usr/lib/libgstapp-0.10.so.0: undefined
symbol: gst_buffer_list_get_type

(songbird-bin:21343): GStreamer-WARNING **: Failed to load plugin
'/usr/lib/gstreamer-0.10/libgstdeinterlace.so':
/usr/lib/gstreamer-0.10/libgstdeinterlace.so: undefined symbol:
gst_video_format_parse_caps_interlaced  

(songbird-bin:21343): GStreamer-WARNING **: Failed to load plugin
'/usr/lib/gstreamer-0.10/libgstmatroska.so':
/usr/lib/gstreamer-0.10/libgstmatroska.so: undefined symbol:
gst_util_array_binary_search

(songbird-bin:21343): GStreamer-WARNING **: Failed to load plugin
'/usr/lib/gstreamer-0.10/libgstpulse.so':
/usr/lib/gstreamer-0.10/libgstpulse.so: undefined symbol:
gst_message_new_request_state   

(songbird-bin:21343): GStreamer-WARNING **: Failed to load plugin
'/usr/lib/gstreamer-0.10/libgstdv.so': /usr/lib/gstreamer-0.10/libgstdv.so:
undefined symbol: gst_tag_list_new_full 
././songbird-bin: symbol lookup error:
/usr/lib/python2.6/site-packages/gst-0.10/gst/_gst.so: undefined symbol:
gst_task_pool_get_type   
Could not initialize GStreamer: Error re-scanning registry , child terminated
by signal

Googling these error messages, I've found the advise to set LD_BIND_NOW=1 when
running songbird. This seems to help a bit, but songbird is still crashing
randomly.

Scott, what did you do to make it work for you? :-)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 532813] Review Request: gummi - A simple LaTeX editor

2009-12-23 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=532813


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #7 from Fabian Affolter fab...@bernewireless.net  2009-12-23 
06:23:03 EDT ---
New Package CVS Request
===
Package Name: gummi
Short Description: A simple LaTeX editor
Owners: fab
Branches: F-11 F-12
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549915] Review Request: FreeMat - An interpreted, matrix-oriented development environment

2009-12-23 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=549915





--- Comment #4 from Henrique LonelySpooky Junior henrique...@gmail.com  
2009-12-23 06:29:19 EDT ---
(In reply to comment #3)
 Absolutely. It should be placed in %{_datadir}/%{name}/. Please ask upstream 
 to
 fix this.  
Leave it to me.
Can you tell me something about this implicit requirements that my package is
asking for?
By the way, here is a little update in the páckage but I just realized that if
we move %{_datadir}/%{name}-%{version}/help to
%{_docdir}/%{name}-%{version}/help the help section of the software will break.
Is there a more elegant solution to this problem?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549915] Review Request: FreeMat - An interpreted, matrix-oriented development environment

2009-12-23 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=549915





--- Comment #5 from Henrique LonelySpooky Junior henrique...@gmail.com  
2009-12-23 06:41:49 EDT ---
ooops! Sorry. Here goes the files.
SPEC: http://lonelyspooky.com/uploads/rpm/freemat/4.0-2/freemat.spec
SRPM:
http://lonelyspooky.com/uploads/rpm/freemat/4.0-2/FreeMat-4.0-2.fc12.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 266001] Review Request: webunit - Python web testing framework

2009-12-23 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=266001


Fabian Affolter fab...@bernewireless.net 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 266001] Review Request: webunit - Python web testing framework

2009-12-23 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=266001


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|fab...@bernewireless.net
   Flag|fedora-review?  |




--- Comment #6 from Fabian Affolter fab...@bernewireless.net  2009-12-23 
06:52:50 EDT ---
- Do you really need 'python_sitearch'?  I guess that this is a left-over from
the template.
- Can you please preserve the timestamps in the install section?
  https://fedoraproject.org/wiki/Packaging:Guidelines#Timestamps

The biggest issue is still the missing license statement.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549915] Review Request: FreeMat - An interpreted, matrix-oriented development environment

2009-12-23 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=549915





--- Comment #6 from Jussi Lehtola jussi.leht...@iki.fi  2009-12-23 07:31:01 
EDT ---
(In reply to comment #4)
 (In reply to comment #3)
  Absolutely. It should be placed in %{_datadir}/%{name}/. Please ask 
  upstream to
  fix this.  
 Leave it to me.
 Can you tell me something about this implicit requirements that my package is
 asking for?
 By the way, here is a little update in the páckage but I just realized that if
 we move %{_datadir}/%{name}-%{version}/help to
 %{_docdir}/%{name}-%{version}/help the help section of the software will 
 break.
 Is there a more elegant solution to this problem?  

If it breaks, then the files are needed by the main package and thus cannot be
placed in -doc, instead they are placed in the main package.

Now you have
 %dir %{_datadir}/%{name}-%{version}
 %{_datadir}/%{name}-%{version}/toolbox
 %{_datadir}/%{name}-%{version}/help
so change this simply to
 %{_datadir}/%{name}-%{version}
or
 %{_datadir}/%{name}-%{version}/
both of which are equivalent, but the latter is IMHO clearer to the reader of
the spec file.

It's a bit off that the datadir is versioned. Only a few packages do this. Is
there a configure option or whatnot to change the datadir to
%{_datadir}/%{name} instead of using %{_datadir}/%{name}-%{version}?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513497] Review Request: ovirt-viewer - A vnc viewer for oVirt managed vms

2009-12-23 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=513497


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

   Flag||needinfo?(mmo...@redhat.com
   ||)




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513497] Review Request: ovirt-viewer - A vnc viewer for oVirt managed vms

2009-12-23 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=513497





--- Comment #2 from Fabian Affolter fab...@bernewireless.net  2009-12-23 
07:39:35 EDT ---
ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 544581] Review Request: python-minimock - The simplest possible mock library

2009-12-23 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=544581


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|fab...@bernewireless.net
   Flag||fedora-review+




--- Comment #3 from Fabian Affolter fab...@bernewireless.net  2009-12-23 
08:04:32 EDT ---
Package Review
==

Package: 

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

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines
 [x] Spec file name must match the base package %{name}, in the format
%{name}.spec
 [x] Package meets the Packaging Guidelines
 [x] Package successfully compiles and builds into binary RPMs on at least one
supported architecture
 Tested on: F12/i386
 [x] Rpmlint output:
 Source RPM:
 [...@localhost SRPMS]$ rpmlint python-minimock-1.2.5-1.fc12.src.rpm 
 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
 Binary RPM(s):
 [...@localhost noarch]$ rpmlint python-minimock-1.2.5-1.fc12.noarch.rpm 
 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
 [x] Package is not relocatable
 [x] Buildroot is correct
 master   : %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 spec file: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license
 License type: MIT
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc

 [x] Spec file is legible and written in American English
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL
 Upstream source: 404ad9b0b2052686dee28936403aa3e7
 Build source:404ad9b0b2052686dee28936403aa3e7
 [x] Package is not known to require ExcludeArch
 [x] Architecture independent packages have: BuildArch: noarch
 [x] 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.  %find_lang used for locales
 [-] %{optflags} or RPM_OPT_FLAGS are honoured
 [-] ldconfig called in %post and %postun if required
 [x] %install starts with rm -rf %{buildroot} or $RPM_BUILD_ROOT
 [x] Package must own all directories that it creates
 [x] Package requires other packages for directories it uses
 [x] Package does not own files or directories owned by other packages
 [x] Package does not contain duplicates in %files
 [x] Permissions on files are set properly. %defattr(-,root,root,-) is in every
%files section
 [x] Package uses nothing in %doc for runtime
 [x] Package has a %clean section, which contains rm -rf %{buildroot} or
$RPM_BUILD_ROOT
 [-] Included tests passed successfully 
 [x] Package consistently uses macros
 [x] Package contains code, or permissable content
 [x] Included filenames are in UTF-8

 [-] Large documentation files are in a -doc subpackage, if required
 [-] Header files (.h) in -devel subpackage, if present
 [-] Fully versioned dependency in subpackage, if present
 [-] Static libraries (.a) in -static subpackage, if present
 [-] Package requires pkgconfig, if .pc files are present
 [-] Development .so files in -devel subpackage, if present
 [-] Package does not contain any libtool archives (.la)
 [-] -debuginfo subpackage is present and looks complete
 [x] No pre-built binaries (.a, .so*, executable)

 [-] Package contains a properly installed .desktop file if it is a GUI
application
 [-] Follows desktop entry spec
 [-] Valid .desktop Name
 [-] Valid .desktop GenericName
 [-] Valid .desktop Categories
 [-] Valid .desktop StartupNotify
 [-] .desktop file installed with desktop-file-install in %install

=== SUGGESTED ITEMS ===
 [-] Timestamps preserved with cp and install
 [-] Uses parallel make (%{?_smp_mflags})
 [x] Latest version is packaged
 [x] Package does not include license text files separate from upstream
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available
 [x] Reviewer should test that the package builds in mock
 Tested on: F12/i386
 [x] Package should compile and build into binary RPMs on all supported
architectures.
 Tested:  http://koji.fedoraproject.org/koji/taskinfo?taskID=1888127
 [?] Package functions as described
 [-] Scriptlets must be sane, if used
 [-] The placement of 

[Bug 266001] Review Request: webunit - Python web testing framework

2009-12-23 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=266001


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #7 from Fabian Affolter fab...@bernewireless.net  2009-12-23 
08:12:43 EDT ---
Upstream closed the bug [1] about the license as 'wont fix'.  

[1] https://sourceforge.net/tracker/?func=browsegroup_id=20323atid=120323

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 544581] Review Request: python-minimock - The simplest possible mock library

2009-12-23 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=544581


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
08:15:48 EDT ---
Thanks for reviewing this.

New Package CVS Request
===
Package Name:  python-minimock
Short Description: The simplest possible mock library
Owners:tomspur
Branches: 
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476527] Review Request: python-zdaemon - Python Daemon Process Control Library

2009-12-23 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=476527


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #19 from Fabian Affolter fab...@bernewireless.net  2009-12-23 
08:18:58 EDT ---
Looks good, package 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550067] New: Review Request: perl-Template-Tiny - Template Toolkit reimplemented in as little code as possible

2009-12-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-Template-Tiny - Template Toolkit reimplemented in 
as little code as possible

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

   Summary: Review Request: perl-Template-Tiny - Template Toolkit
reimplemented in as little code as possible
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mmaslano.fedorapeople.org/review/perl-Template-Tiny.spec
SRPM URL:
http://mmaslano.fedorapeople.org/review/perl-Template-Tiny-0.09-1.fc13.src.rpm
Description: WARNING: THIS MODULE IS EXPERIMENTAL AND SUBJECT TO CHANGE WITHOUT
NOTICE Template::Tiny is a reimplementation of a partial subset of the Template
Toolkit,  in as few lines of code as possible. It is intended for use in
light-usage, low-memory, or low-cpu templating situations, where you may need
to upgrade to the full feature set 
in the future, or if you want the familiarity of TT-style templates. It is
intended to have fully-compatible template and stash usage, with a limited by
similar Perl API. Unlike Template Toolkit, Template::Tiny will process
templates without a compile phase  (but despite this is still quicker, owing to
heavy use of the Perl regular expression engine.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549884] Review Request: plexus-io - Plexus IO Components

2009-12-23 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=549884


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

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




--- Comment #1 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-12-23 
08:54:25 EDT ---
I made a full review. Here are my comments, questions etc:

* Package does not build in mock. It pulls gcj. We want openjdk. I added that
BR fol the rest of the review.
As far as I know, all java packages need these BR's and R's:
   http://fedoraproject.org/wiki/Packaging:Java#BuildRequires_and_Requires

* Additionally, the guidelines tell us to include these R's for maven:
   http://fedoraproject.org/wiki/Packaging:Java#maven

* rpmlint says
   plexus-io.noarch: W: non-conffile-in-etc /etc/maven/fragments/plexus-io
Is this ignorable? Why?

! javadoc package does not require the main package. (just a warning. I don't
know if this is intentional)

? Shouldn't the release tag just be 0.1.a5%{?dist}

! For macro consistency plexus-io can be replaced by %{name} at two places

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 548877] Review Request: python-virtualenv - Virtual Python Environment builder

2009-12-23 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=548877


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org




--- Comment #3 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
08:57:42 EDT ---
Could you post the new url to the spec file?

The one above should be an old one, because there are still the spaces.


Just some other comments:
- %gobal over %define
  see:
https://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define

- BR: python-setuptools-devel is not needed, python-setuptools is enought

- How about modifying the description a bit?
  You could use %{python_sitelib} instead of
``/usr/lib/python2.4/site-packages``, so the example in the description will
always be uptodate.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 529375] Review Request: emerillon - A map viewer for GNOME

2009-12-23 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=529375





--- Comment #13 from Peter Robinson pbrobin...@gmail.com  2009-12-23 09:02:48 
EDT ---

 %{_bindir}/%{name} is a file, not a directory.  

Good catch, rushed update. Now fixed.

SRPM: http://pbrobinson.fedorapeople.org/emerillon-0.1.0-5.fc12.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 529374] Review Request: ethos - Plugin framework for GLib

2009-12-23 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=529374





--- Comment #13 from Peter Robinson pbrobin...@gmail.com  2009-12-23 09:04:44 
EDT ---
(In reply to comment #12)
 as far as i understand it right, you should use wildcards for every
 versionnumber in your filelist. But I'm not sure, just make sense to me, as 
 far
 as i understand it correctly.

Sorry I don't understand what your asking. Is this a geneeral query, or
something in the review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549884] Review Request: plexus-io - Plexus IO Components

2009-12-23 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=549884





--- Comment #2 from Alexander Kurtakov akurt...@redhat.com  2009-12-23 
09:18:56 EDT ---
(In reply to comment #1)
 I made a full review. Here are my comments, questions etc:
 
 * Package does not build in mock. It pulls gcj. We want openjdk. I added that
 BR fol the rest of the review.
 As far as I know, all java packages need these BR's and R's:
http://fedoraproject.org/wiki/Packaging:Java#BuildRequires_and_Requires
Fixed.
 
 * Additionally, the guidelines tell us to include these R's for maven:
http://fedoraproject.org/wiki/Packaging:Java#maven
Fixed.

 
 * rpmlint says
plexus-io.noarch: W: non-conffile-in-etc /etc/maven/fragments/plexus-io
 Is this ignorable? Why?
This is the place where mvn looks for fragments. And they are clearly non
conffile because users should not edit them.
 
 ! javadoc package does not require the main package. (just a warning. I don't
 know if this is intentional)
Perfectly ok. There are many times when you want to install just the javadoc
for local reference and studing the API without any need to use it.

 
 ? Shouldn't the release tag just be 0.1.a5%{?dist}
Fixed.

 
 ! For macro consistency plexus-io can be replaced by %{name} at two places  
Fixed

Spec URL: http://akurtakov.fedorapeople.org/plexus-io.spec
SRPM URL: http://akurtakov.fedorapeople.org/plexus-io-1.0-0.2.a5.fc12.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 524190] Review Request: flowcanvas - an interactive Gtkmm/Gnomecanvasmm widget for “boxes and lines” environments

2009-12-23 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=524190





--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
09:33:37 EDT ---
Again ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510784] Review Request: dieharder - A random number generator tester and timer

2009-12-23 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=510784





--- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
09:34:07 EDT ---
Again ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549884] Review Request: plexus-io - Plexus IO Components

2009-12-23 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=549884


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-12-23 
09:38:43 EDT ---
Thanks! Good to go.


This package (plexus-io) is APPROVED by oget


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549884] Review Request: plexus-io - Plexus IO Components

2009-12-23 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=549884


Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Alexander Kurtakov akurt...@redhat.com  2009-12-23 
09:42:15 EDT ---
New Package CVS Request
===
Package Name: plexus-io
Short Description: Plexus IO Components
Owners: akurtakov
Branches: 
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 517743] Review Request: PyPE - Lightweight but powerful graphical editor for developers

2009-12-23 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=517743





--- Comment #9 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
10:08:54 EDT ---
Sandro, I don't mind accepting this now with the old version, if it's clear,
which file has which license.

You can choose, between
- adding a comment above the License: field
- refer to a file upstream, where anything is explained
- breack down in the %files section.


See [1] for further information.

[1]
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Multiple_Licensing_Scenarios

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550100] New: Review Request: python-olpcgames - Utilities for developing games on the OLPC platform

2009-12-23 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: python-olpcgames - Utilities for developing games on 
the OLPC platform

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

   Summary: Review Request: python-olpcgames - Utilities for
developing games on the OLPC platform
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: sebast...@when.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://sdz.fedorapeople.org/rpmbuild/python-olpcgames.spec
SRPM URL: http://sdz.fedorapeople.org/rpmbuild/python-olpcgames-1.6-1.src.rpm

Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1888451
See also: https://bugzilla.redhat.com/show_bug.cgi?id=250533

[sebast...@localhost rpmbuild]$ rpmlint
./RPMS/noarch/python-olpcgames-1.6-1.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[sebast...@localhost rpmbuild]$ rpmlint ./SRPMS/python-olpcgames-1.6-1.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550102] New: Review Request: ghc-mtl - Haskell Monad Transformer Library

2009-12-23 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: ghc-mtl - Haskell Monad Transformer Library

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

   Summary: Review Request: ghc-mtl - Haskell Monad Transformer
Library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: peter...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://petersen.fedorapeople.org/ghc-mtl/ghc-mtl.spec
SRPM URL:
http://petersen.fedorapeople.org/ghc-mtl/ghc-mtl-1.1.0.2-1.fc12.src.rpm
Description: Haskell Monad Transformer Library

A standard library part of haskell-platform.

This package is needed for ghc-6.12 to build a bunch of packages
already in fedora including ghc-gtk2hs, ghc-uniplate, happy
xmonad, and haskell-platform of course.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 546620] Review Request: git-bugzilla - Attach patches to a bugzilla bug

2009-12-23 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=546620





--- Comment #7 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
10:23:53 EDT ---
(In reply to comment #6)
  SHOULD:
  - If the source package does not include license text(s) as a separate file
  from upstream, the packager SHOULD query upstream to include it.
 
 This is basically just one file; I'm not sure if upstream would include 
 license
 separately.

Upstream don't *have* to include a license file, you just need to say hi and
ask them to do so. If they don't, it'll ok, too.


Joshua, you started good. Do you intent continuing this review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550102] Review Request: ghc-mtl - Haskell Monad Transformer Library

2009-12-23 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=550102


Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

   Priority|medium  |high
 CC||fedora-haskell-l...@redhat.
   ||com




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 548877] Review Request: python-virtualenv - Virtual Python Environment builder

2009-12-23 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=548877





--- Comment #4 from Peter Halliday phalli...@excelsiorsystems.net  2009-12-23 
10:26:53 EDT ---
I actually am not sure why the define was there at all, because I wasn't using
it.I removed the define.  I changed python-setuptools-devel just to
python-setuptools.  I changed the description as you suggested.  I also updated
both the spec file and the SRPM as per the changes to the original URL.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 548607] Review Request: pvs-sbcl - SRI's Prototype Verification System

2009-12-23 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=548607





--- Comment #15 from David A. Wheeler dwhee...@dwheeler.com  2009-12-23 
10:37:11 EDT ---
 Let's try using emacs-nox or xemacs-nox.

Agree.  That's an unfortunate solution if it's long-term, but it's a reasonable
short-term solution *AND* it would definitely help narrow down where the bug
is.

Please modify the spec so that it'll be easy to switch back to normal emacs
or xemacs.  It's my hope that using *emacs-nox is a relatively temporary
solution.

Your earlier patch modified exit-pvs-process by replacing (sit-for 1) with
(accept-process-output nil 1).  Although it's a nasty hack, you could instead
repeatedly (1) wait for 1 second using sleep, then (2) use system to invoke
ps directly and see if the process is gone.  Yeah, that's a nasty hack. 
Might work, 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550104] New: Review Request: magento - Magento is an ecommerce web application.

2009-12-23 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: magento - Magento is an ecommerce web application.

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

   Summary: Review Request: magento - Magento is an ecommerce web
application.
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: fhorn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fhornain.fedorapeople.org/pkgs/magento/magento.spec
SRPM URL:
http://fhornain.fedorapeople.org/pkgs/magento/magento-1.3.2.4-1.fc11.src.rpm
Description: Magento is an ecommerce web application.

Since it is a web application, this electronic commerce browser works on many
platforms, making your store application easily manageable from any location.

Magento is the perfect ecommerce browser for the sell professional
and novice alike.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550104] Review Request: magento - Magento is an ecommerce web application.

2009-12-23 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=550104


Frederic Hornain fhorn...@gmail.com 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550105] New: Review Request: ghc-parsec - Haskell parser library

2009-12-23 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: ghc-parsec - Haskell parser library

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

   Summary: Review Request: ghc-parsec - Haskell parser library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: peter...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://petersen.fedorapeople.org/ghc-parsec/ghc-parsec.spec
SRPM URL:
http://petersen.fedorapeople.org/ghc-parsec/ghc-parsec-2.1.0.1-1.fc12.src.rpm
Description: Haskell parser library

Part of haskell-platform, currently need for ghc-6.12.1
to build ghc-network, haskell-platform, and darcs I think.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550105] Review Request: ghc-parsec - Haskell parser library

2009-12-23 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=550105


Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

   Priority|medium  |high




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 542747] Review Request: oglappth - Libraries for the oglappth chemistry package

2009-12-23 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=542747





--- Comment #8 from Jussi Lehtola jussi.leht...@iki.fi  2009-12-23 10:49:56 
EDT ---
oglappth.src: W: name-repeated-in-summary oglappth
oglappth.x86_64: W: name-repeated-in-summary oglappth
oglappth.x86_64: W: shared-lib-calls-exit /usr/lib64/liboglappth.so.2.0.0
e...@glibc_2.2.5
oglappth-devel.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 4 warnings.

Fix the summaries. Judging by the description,
 Library for creating portable OpenGL applications with easy-to-code
 scene setup and selection operations.
the summary should be e.g.
 An OpenGL wrapper library

**

The -devel package doesn't contain static libraries, so drop their mention from
the summary.

**

The changelog does not conform to Fedora standards. Leave the Mon Jan 12 entry
(replacing the + with a -), drop the rest.

**

Source url should be
http://www.uku.fi/~thassine/projects/download/current/lib%{name}-%{version}.tar.gz

**

Change
 %{_includedir}/*
to
 %{_includedir}/oglappth/
for clarity.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 542747] Review Request: oglappth - Libraries for the oglappth chemistry package

2009-12-23 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=542747


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|jussi.leht...@iki.fi
   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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 530198] Review Request: rubygem-columnize - Sorts an array in column order

2009-12-23 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=530198


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
10:55:04 EDT ---
Closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 532309] Review Request: rubygem-ruby-net-ldap - A full-featured pure-Ruby LDAP client

2009-12-23 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=532309


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #18 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
10:56:05 EDT ---
Closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 542747] Review Request: oglappth - Libraries for the oglappth chemistry package

2009-12-23 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=542747





--- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi  2009-12-23 10:56:51 
EDT ---
MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. OK

MUST: The package must be named according to the Package Naming Guidelines.
NEEDSWORK
- According to the naming guidelines, the name of the package should be
liboglappth, not oglappth.

MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the 
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual license.
OK

MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. NEEDSWORK
- Source URL is incorrect, but source matches upstream.

MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. N/A

MUST: Optflags are used and time stamps preserved. NEEDSWORK
- Add INSTALL=install -p to make install to preserve time stamps.

MUST: Packages containing shared library files must call ldconfig. OK
MUST: A package must own all directories that it creates or require the package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. OK
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. N/A

MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. OK
- You might want to add statements
 [ -s NEWS ]  exit 1
 [ -s README ]  exit 1
in %setup to make the build fail and thus become notified if these files
acquire content.

MUST: Header files must be in a -devel package. OK
MUST: Static libraries must be in a -static package. N/A

MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'.
NEEDSWORK
- This is automatically picked up on at least Fedora 12; if you want to build
for EPEL then you need to add Requires: pkgconfig. Better safe than sorry, so
please add this.

MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. OK
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. OK
MUST: Packages does not contain any .la libtool archives. OK
MUST: Desktop files are installed properly. N/A
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The package builds in mock. OK

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 532306] Review Request: rubygem-ruby-debug - Faster implementation of the standard Debugging

2009-12-23 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=532306


Bug 532306 depends on bug 533725, which changed state.

Bug 533725 Summary: Review Request: rubygem-linecache - Caches (Ruby source) 
files
https://bugzilla.redhat.com/show_bug.cgi?id=533725

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 522820] Review Request: ghc-xmonad-contrib-bluetilebranch - bluetile branch of xmonad addon libraries

2009-12-23 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=522820


Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

   Priority|medium  |low




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 533725] Review Request: rubygem-linecache - Caches (Ruby source) files

2009-12-23 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=533725


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
10:56:49 EDT ---
Closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 522819] Review Request: ghc-xmonad-bluetilebranch - bluetile branch of xmonad libraries

2009-12-23 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=522819


Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

   Priority|medium  |low




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 543549] Review Request: rubygem-haml - XHTML/XML templating engine

2009-12-23 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=543549





--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
10:58:45 EDT ---
Well, for now I won't object to use %test_files. So
would you modify your spec file if there is something you want
to do so and reupload your spec/srpm 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


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

2009-12-23 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 peter...@redhat.com changed:

   What|Removed |Added

   Priority|medium  |low




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 547118] Review Request: crun - Lightweight, easy to use, simpler cron-like tool

2009-12-23 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=547118





--- Comment #2 from Damien Durand splinu...@gmail.com  2009-12-23 11:33:01 
EDT ---
New Package CVS Request
===
Package Name: crun
Short Description: Lightweight, easy to use, simpler cron-like tool
Owners: splinux
Branches: F-11 F-12
InitialCC: splinux

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 517743] Review Request: PyPE - Lightweight but powerful graphical editor for developers

2009-12-23 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=517743





--- Comment #10 from Josiah Carlson josiah.carl...@gmail.com  2009-12-23 
11:39:14 EDT ---
FYI, I released a new version with all of the license and code updates
yesterday.  :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 517743] Review Request: PyPE - Lightweight but powerful graphical editor for developers

2009-12-23 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=517743





--- Comment #11 from Sandro Mathys s...@sandro-mathys.ch  2009-12-23 11:44:28 
EDT ---
Just wanted to post the same thing as Josiah - there's a new release with the
fixes and I'm going to create a new srpm with it. Might very well get 2010
until I'm able to do so, tho. But maybe I'll find some time to do it earlier...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 547118] Review Request: crun - Lightweight, easy to use, simpler cron-like tool

2009-12-23 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=547118


Xavier Lamien lxt...@gmail.com changed:

   What|Removed |Added

 CC||lxt...@gmail.com
   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486757] Review Request: divine-mc - Multi-core model checking system for proving specifications

2009-12-23 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=486757


David A. Wheeler dwhee...@dwheeler.com changed:

   What|Removed |Added

   Flag|needinfo?(dwhee...@dwheeler |
   |.com)   |




--- Comment #24 from David A. Wheeler dwhee...@dwheeler.com  2009-12-23 
12:03:28 EDT ---
Petr - thanks for releasing version 2.0 of DiVinE.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550100] Review Request: python-olpcgames - Utilities for developing games on the OLPC platform

2009-12-23 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=550100


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org




--- Comment #1 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
12:14:23 EDT ---
Just a few comments for now, because I'm unsure, if reviewing this is the right
procedure...

- use %global and not %define
 
https://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define

- permissions of buildmanifest.py need to be 664, like they where.
  But this is no executable script and contains #!/usr/bin/env
  Fix this with [1] to delete the shebang and not touching the permissions.


[1]
https://fedoraproject.org/wiki/PackageMaintainers/Packaging_Tricks#Remove_shebang_from_files



About the procedure:
It seems, the original submitter does not care anymore about this package and
you want to take it.

http://fedoraproject.org/wiki/Policy_for_nonresponsive_package_maintainers

would be the best thing to do in this case and no new 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 547993] Package Name Change: jconvolver - Real-time Convolution Engine

2009-12-23 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=547993


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
12:25:09 EDT ---
Well for SOURCE1, I leave it to your choise how to package it
(however please make upstream clarify the license first)

---
  This package (jconvolver) is APPROVED by mtasaka
---

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 521993] New Package for Dogtag PKI:pki-setup

2009-12-23 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=521993


Kevin Wright kwri...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #12 from Kevin Wright kwri...@redhat.com  2009-12-23 12:31:13 EDT 
---
New Package CVS Request
===
Package Name: pki-setup
Short Description: The Dogtag PKI Setup Framework 
Owners: kwright
Branches: F-11, F-12, EL-5
InitialCC: ausil

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453422] Review Request: songbird - Mozilla based multimedia player

2009-12-23 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=453422





--- Comment #94 from David Halik auralva...@gmail.com  2009-12-23 12:36:41 
EDT ---
1.4.3 is on it's way to being released, so I'm going to wait for that.

@Scott the svn checkout isn't the same as their officially released source ball
(different paths, structure, etc), so I wait for that. Otherwise I have to
change the build process to cope and it's a bunch of extra work for nothing.

Songbird dev stevel will notify me when the ball is out, so I'll get on it soon
after.

@Andreas You should probably post your issues on the songbird help list,
because what I do here with the respin is very different than what they
provide. I use almost all system components from Fedora, except for xulrunner
and tagib, they package everything to their own patched copies. It's very
likely that gstreamer bugs will not be the same because of the different
versions involved.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550138] New: Review Request: xcftools - Command-line tools for extracting information from XCF files

2009-12-23 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: xcftools - Command-line tools for extracting 
information from XCF files

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

   Summary: Review Request: xcftools - Command-line tools for
extracting information from XCF files
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: nicoleau.fab...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://rpms.nicoleau-fabien.net/SPECS/xcftools.spec
SRPM URL:
http://rpms.nicoleau-fabien.net/srpms.fc12/xcftools-1.0.7-1.fc12.src.rpm
Description:
Xcftools is a set of fast command-line tools for extracting information from
the Gimp's native file format XCF. The tools are designed to allow efficient
use of layered XCF files as sources in a build system that use 'make' and
similar tools to manage automatic processing of the graphics.
These tools work independently of the Gimp engine and do not require
the Gimp to even be installed.

This package builds on koji :
http://koji.fedoraproject.org/koji/taskinfo?taskID=1888672

rpmlint output : 
[buil...@fedobox ~]$ rpmlint
/home/builder/rpmbuild/SRPMS/xcftools-1.0.7-1.fc12.src.rpm
/home/builder/rpmbuild/RPMS/x86_64/xcftools-1.0.7-1.fc12.x86_64.rpm
/home/builder/rpmbuild/RPMS/x86_64/xcftools-debuginfo-1.0.7-1.fc12.x86_64.rpm
3 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550139] New: Review Request: pino - A fast, easy and free Twitter client

2009-12-23 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: pino - A fast, easy and free Twitter client

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

   Summary: Review Request: pino - A fast, easy and free Twitter
client
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: allis...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://allisson.fedorapeople.org/packages/pino/pino.spec
SRPM URL:
http://allisson.fedorapeople.org/packages/pino/pino-0.1.0-0.1.rc2.fc12.src.rpm

Description: A fast, easy and free Twitter client

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550143] New: Review Request: webattery - Command line tool to display battery status

2009-12-23 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: webattery - Command line tool to display battery status

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

   Summary: Review Request: webattery - Command line tool to
display battery status
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: alagunambiwel...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://download.sf.net/webattery/webattery.spec
SRPM URL: http://download.sf.net/webattery/webattery-1.1-1.fc10.src.rpm
Description: Webattery is a command line tool to display battery status with a
cool battery like output. 
This my first package, need Sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 461119] Review Request: libtiger - Rendering library for Kate streams using Pango and Cairo

2009-12-23 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=461119


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org




--- Comment #4 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
13:10:31 EDT ---
It would be easier to remove %{buildroot}%{_docdir} and add '%doc doc/html' to
the %files devel section...

Furthermore docs need to be installed in %{_docdir}/%{name}-%{version}. The
above would fix this issue, too.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 436612] Review Request: avfs - A Virtual File System

2009-12-23 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=436612


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||toms...@fedoraproject.org
 Resolution||NOTABUG




--- Comment #6 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
12:56:33 EDT ---
Lorenzo, you should open your own review request and close this one as
dublicate of your new request.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550143] Review Request: webattery - Command line tool to display battery status

2009-12-23 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=550143


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

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




--- Comment #1 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
13:27:29 EDT ---
Just a few comments, because I'm no sponsor anyway:

- $ rpmlint webattery-1.1-1.fc12.src.rpm x86_64/webattery-*
webattery.src: W: no-cleaning-of-buildroot %install
webattery.src: W: no-buildroot-tag
webattery-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/webattery-1.1/src/webattery.c
3 packages and 0 specfiles checked; 0 errors, 3 warnings.

  * in %install needs to be rm -rf %{buildroot} as first line
  * no-buildroot-tag can be ignored, this is not needed anymore in fedora
(only if you want to import it in EL-5, too...)
  * webattery.c needs to be 644, please change this in %install

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 548607] Review Request: pvs-sbcl - SRI's Prototype Verification System

2009-12-23 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=548607





--- Comment #16 from Jerry James loganje...@gmail.com  2009-12-23 13:45:48 
EDT ---
Argh.  The elisp code calls add-submenu AT COMPILE TIME!  Since this function
doesn't exist in (x)emacs-nox, the compilation fails.

I still don't understand why the accept-process-output hack isn't working for
you, though.  It runs code that is supposed to check for process statuses, on
both emacsen.  Since it solved the problem I was having with XEmacs, I know
that code is being run.

I'm going to make a virtual 32-bit machine and see if I can debug the hang
you're seeing.  It looks like the same hang, but may not be.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 541346] Review Request: polkit-kde - PolicyKit integration for KDE Desktop

2009-12-23 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=541346





--- Comment #14 from Rex Dieter rdie...@math.unl.edu  2009-12-23 14:26:01 EDT 
---
I'll take a look,

Not a blocker, looks like we have an upstream release/tarball, woo,
ftp://ftp.kde.org/pub/kde/stable/apps/KDE4.x/admin/polkit-kde-1-0.95.1.tar.bz2

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 542747] Review Request: oglappth - Libraries for the oglappth chemistry package

2009-12-23 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=542747





--- Comment #10 from Carl Byington c...@five-ten-sg.com  2009-12-23 14:24:49 
EDT ---
done

http://www.five-ten-sg.com/liboglappth.spec
http://www.five-ten-sg.com/liboglappth-0.98-5.fc12.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=1888795

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 544964] Review Request: rubygem-mime-types - Return the MIME Content-Type for a given filename

2009-12-23 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=544964


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review?




--- Comment #1 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-23 
14:41:21 EDT ---
Well,

* License
  - I guess the license for this package should
be GPL+ or Ruby or Artistic.

* rcov for %check
  - build.log says:
---
89  + rake test
90  Hoe.new {...} deprecated. Switch to Hoe.spec.
91  (in
/builddir/build/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16)
92  which: no dot in
(/usr/sbin:/usr/bin:/sbin:/bin:/usr/X11R6/bin:/root/bin:/usr/local/sbin)
93  /usr/bin/ruby -w -Ilib:ext:bin:test -e 'require rubygems; require
test/unit; require test/test_mime_type.rb; require
test/test_mime_types.rb' 
94  RCov is not available. In order to run rcov, you must: sudo gem install
spicycode-rcov
---
I don't know well about RCov, however when I install
rubygem-rcov, rake check fails as:
---
[tasa...@localhost mime-types-1.16]$ rake test --trace
(in
/home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16)
Hoe.new {...} deprecated. Switch to Hoe.spec.
rake aborted!
undefined method `test_files' for #Hoe:0xb75d9a78
/home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:53
/usr/lib/ruby/gems/1.8/gems/rcov-0.9.6/lib/rcov/rcovtask.rb:91:in `initialize'
/home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:51:in
`new'
/home/tasaka1/rpmbuild/BUILD/rubygem-mime-types-1.16/usr/lib/ruby/gems/1.8/gems/mime-types-1.16/Rakefile:51
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2382:in `load'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2382:in `raw_load_rakefile'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2016:in `load_rakefile'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2067:in
`standard_exception_handling'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2015:in `load_rakefile'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1999:in `run'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2067:in
`standard_exception_handling'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1997:in `run'
/usr/lib/ruby/gems/1.8/gems/rake-0.8.7/bin/rake:31
/usr/bin/rake:19:in `load'
/usr/bin/rake:19
---
  ref: http://bugs.gentoo.org/288996
  Would you check 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 544581] Review Request: python-minimock - The simplest possible mock library

2009-12-23 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=544581


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #5 from Dennis Gilmore den...@ausil.us  2009-12-23 14:42:32 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510376] Review Request: bluemodem - A bluetooth modem configuration utility

2009-12-23 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=510376


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #8 from Dennis Gilmore den...@ausil.us  2009-12-23 14:41:16 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549884] Review Request: plexus-io - Plexus IO Components

2009-12-23 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=549884


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #5 from Dennis Gilmore den...@ausil.us  2009-12-23 14:40:13 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 542559] Review Request: rubygem-thor - Scripting framework that replaces rake, sake and rubigen

2009-12-23 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=542559


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Dennis Gilmore den...@ausil.us  2009-12-23 14:45:30 EDT 
---
CVS done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549863] Review Request: plexus-interpolation - Plexus Interpolation API

2009-12-23 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=549863


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #5 from Dennis Gilmore den...@ausil.us  2009-12-23 14:39:18 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 541346] Review Request: polkit-kde - PolicyKit integration for KDE Desktop

2009-12-23 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=541346


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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #15 from Rex Dieter rdie...@math.unl.edu  2009-12-23 14:43:39 EDT 
---
Simple spec.

$ rpmlint *.rpm */*.rpm
polkit-kde.src:24: W: unversioned-explicit-provides
PolicyKit-authentication-agent
polkit-kde.x86_64: W: obsolete-not-provided PolicyKit-kde
polkit-kde.x86_64: W: non-conffile-in-etc
/etc/xdg/autostart/polkit-kde-authentication-agent-1.desktop
3 packages and 0 specfiles checked; 0 errors, 3 warnings.

License: OK

Source: OK (better, when using verifiable upstream tarball)

macros: OK

Obsoletess/Provides: OK


I see now issues or blockers, looks good,

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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 529253] Review Request: dmenu - Generic menu for X

2009-12-23 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=529253


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #5 from Dennis Gilmore den...@ausil.us  2009-12-23 14:46:49 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 549809] Review Request: mingw32-libzip - mingw32 port of libzip

2009-12-23 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=549809


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Dennis Gilmore den...@ausil.us  2009-12-23 14:44:29 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 547118] Review Request: crun - Lightweight, easy to use, simpler cron-like tool

2009-12-23 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=547118


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #3 from Dennis Gilmore den...@ausil.us  2009-12-23 14:43:25 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 532813] Review Request: gummi - A simple LaTeX editor

2009-12-23 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=532813


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #8 from Dennis Gilmore den...@ausil.us  2009-12-23 14:47:59 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 519221] Review Request: rfkill - A tool to query the state of RF kill interfaces

2009-12-23 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=519221


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #13 from Dennis Gilmore den...@ausil.us  2009-12-23 14:53:36 EDT 
---
CVS done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 544628] at-spi2-core - Protocol definitions and daemon for D-Bus at-spi

2009-12-23 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=544628


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Dennis Gilmore den...@ausil.us  2009-12-23 14:51:22 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 521993] New Package for Dogtag PKI:pki-setup

2009-12-23 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=521993


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #13 from Dennis Gilmore den...@ausil.us  2009-12-23 14:50:34 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 487097] Review Request: ReviewBoard - web based code review tool

2009-12-23 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=487097


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #32 from Dennis Gilmore den...@ausil.us  2009-12-23 14:49:33 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 544581] Review Request: python-minimock - The simplest possible mock library

2009-12-23 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=544581


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #6 from Thomas Spura toms...@fedoraproject.org  2009-12-23 
14:57:13 EDT ---
Oh no,... forgot to add the branches :'(
Sorry, Dennis.


Package Change Request
==
Package Name: python-minimock
New Branches: F-12 F-11 EL-5
Owners: tomspur

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 521995] New Package for Dogtag PKI:pki-java-tools

2009-12-23 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=521995


Kevin Wright kwri...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #8 from Kevin Wright kwri...@redhat.com  2009-12-23 14:59:15 EDT 
---
New Package CVS Request
===
Package Name: pki-native-tools
Short Description: The Dogtag PKI Java Tools
Owners: kwright
Branches: F-11, F-12, EL-5
InitialCC: ausil

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 521995] New Package for Dogtag PKI:pki-java-tools

2009-12-23 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=521995





--- Comment #9 from Kevin Wright kwri...@redhat.com  2009-12-23 15:07:43 EDT 
---
(fixing the short description)

New Package CVS Request
===
Package Name: pki-native-tools
Short Description: The Dogtag PKI Native Tools
Owners: kwright
Branches: F-11, F-12, EL-5
InitialCC: ausil

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 542747] Review Request: oglappth - Libraries for the oglappth chemistry package

2009-12-23 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=542747


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #11 from Jussi Lehtola jussi.leht...@iki.fi  2009-12-23 15:08:16 
EDT ---
The Requires: pkgconfig should be for -devel which actually contains the
pkgconfig file. Otherwise the package looks very clean now.

APPROVED

Fix the require before import to CVS.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 465511] Review Request: itext - A Free Java-PDF library

2009-12-23 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=465511





--- Comment #18 from Jussi Lehtola jussi.leht...@iki.fi  2009-12-23 15:10:56 
EDT ---
Whoops, this slipped under my radar.

I had a look at the requirements: to build itext in EL-5 three more packages
would have to be built. Looks like too much work, I'll just have to deal with
an older release of jmol in EL-5 for good.

Sorry for bothering you.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 550143] Review Request: webattery - Command line tool to display battery status

2009-12-23 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=550143





--- Comment #2 from Alagunambi Welkin alagunambiwel...@gmail.com  2009-12-23 
15:08:02 EDT ---
(In reply to comment #1)
 
   * in %install needs to be rm -rf %{buildroot} as first line
   * no-buildroot-tag can be ignored, this is not needed anymore in fedora
 (only if you want to import it in EL-5, too...)
   * webattery.c needs to be 644, please change this in %install  

Cleared all, should I replace the given src.rpm and spec file with the new one
or do I need to add them separately along with the old src.rpm and sepc file?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 521995] New Package for Dogtag PKI:pki-java-tools

2009-12-23 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=521995





--- Comment #10 from Kevin Wright kwri...@redhat.com  2009-12-23 15:13:48 EDT 
---
ugh! back to the first description:

Short Description: The Dogtag PKI Java Tools

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 528150] Review Request: invulgotracker - Tasks projects tracking tool

2009-12-23 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=528150





--- Comment #16 from Tareq Al Jurf talj...@fedoraproject.org  2009-12-23 
15:15:28 EDT ---
New Package CVS Request
===
Package Name: invulgotracker
Short Description: Tasks  projects tracking tool
Owners: taljurf
Branches: F-11 F-12
InitialCC: taljurf

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 529253] Review Request: dmenu - Generic menu for X

2009-12-23 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=529253





--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-12-23 15:25:34 EDT ---
dmenu-4.0-2.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/dmenu-4.0-2.fc12

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 547993] Package Name Change: jconvolver - Real-time Convolution Engine

2009-12-23 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=547993


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #10 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-12-23 
15:24:42 EDT ---
Thanks! I won't add it until the license is cleared.

New Package CVS Request
===
Package Name: jconvolver
Short Description: Real-time Convolution Engine
Owners: oget nando
Branches: F-11 F-12
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 542747] Review Request: oglappth - Libraries for the oglappth chemistry package

2009-12-23 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=542747


Carl Byington c...@five-ten-sg.com changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #12 from Carl Byington c...@five-ten-sg.com  2009-12-23 15:46:26 
EDT ---
done

http://www.five-ten-sg.com/liboglappth.spec
http://www.five-ten-sg.com/liboglappth-0.98-6.fc12.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=186

New Package CVS Request
===
Package Name: liboglappth
Short Description: An OpenGL wrapper library
Owners: carllibpst
Branches: F-11 F-12
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


  1   2   >