[Bug 245342] Review Request: perl-Gnome2 - Perl interface to the 2.x series of the GNOME libraries

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

Summary: Review Request: perl-Gnome2 - Perl interface to the 2.x series of the 
GNOME libraries
Alias: perl-Gnome2

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||om)




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 03:17 EST ---
Ping?  I'm also getting 404 errors on the above links :\

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

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


[Bug 245492] Review Request: ndesk-dbus-glib - glib mainloop integration for ndesk-dbus

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

Summary: Review Request: ndesk-dbus-glib - glib mainloop integration for 
ndesk-dbus
Alias: ndesk-dbus-glib

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||ndesk-dbus-glib-0.4.1-1.fc9
 Resolution||CURRENTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 03:58 EST ---
This is, finally, in devel - closing

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

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

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


[Bug 382371] Review Request: gnome-do - quick object search and interaction

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

Summary: Review Request: gnome-do - quick object search and interaction
Alias: gnome-do

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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 04:32 EST ---
- fixed the BuildRequires
- updated libdir patch for gnome-do-0.0.2
- nuke X-Fedora with the orbital laser

Spec URL: http://www.lovesunix.net/fedora/gnome-do.spec
SRPM URL: http://www.lovesunix.net/fedora/gnome-do-0.0.2-2.fc8.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, or are watching someone who is.

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


[Bug 355971] Review Request: poker3d - Three dimensional multi-user online poker game

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

Summary: Review Request: poker3d - Three dimensional multi-user online poker 
game
Alias: poker3d

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 04:09 EST ---
(In reply to comment #18)
 Also, do I still need to run ldconfig in the %post/un sections?

No.

Must Fix

-remove /sbin/ldconfig invocation from %post %postun

Could Fix
-
-why use a subdir under /usr/libexec for only one binary?

Approved, under the condition that you remove the 2 /sbin/ldconfig calls from
the scripts.


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

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


[Bug 373621] Review Request: qct - Multi-vcs GUI commit tool

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

Summary: Review Request: qct - Multi-vcs GUI commit tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 04:54 EST ---
== Good ==

Odd noarch vs. building arch specific stuff is gone
Permissions correct
.desktop file no present
correct cleaning on %install

== Minor ==

adding X-Fedora to the .desktop file is considered cruft
* Nuke it from orbit, the only way to be sure

rpmlint complains:
qct.src: W: mixed-use-of-spaces-and-tabs (spaces: line 39, tab: line 3)
* minor but for consistency please fix this.

qct-mercurial.x86_64: W: no-documentation
* minor, unsure, you might want to consider putting the correct README file in
this package.

qct-mercurial.x86_64: W: non-conffile-in-etc /etc/mercurial/hgrc.d/qct.rc 
* ignore

qct.x86_64: E: no-binary 
* ignore

== Bad ==

rpmlint complains:
qct-debuginfo.x86_64: E: empty-debuginfo-package
* if you are not generating debug symbols then add:

%define debug_package %{nil} 

to the top of your spec to disable building -debuginfo

So nothing major left, just one more rev for good measure to fix the last few
minor problems.

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

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


[Bug 249236] Review Request: kde-style-domino - KDE style with a soft look

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

Summary: Review Request: kde-style-domino - KDE style with a soft look


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 05:47 EST ---
OMG, next issue: doubled %changelog line.

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

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


[Bug 249236] Review Request: kde-style-domino - KDE style with a soft look

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

Summary: Review Request: kde-style-domino - KDE style with a soft look


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 05:45 EST ---
Also in line below you could change domino with %{name} amd 0.4 with %{version}.

Source0: http://www.kde-look.org/CONTENT/content-files/42804-domino-0.4.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, or are watching someone who is.

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


[Bug 371411] Review Request: alienarena - 3d Deathmatch game

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

Summary: Review Request: alienarena - 3d Deathmatch game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 06:02 EST ---
As promised some time ago, I'm doing a full review now.


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

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


[Bug 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

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

Summary: Review Request: ntop - A network traffic probe similar to the UNIX top 
command
Alias: ntop

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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 05:44 EST ---
When starting with LANG=C, I indeed don't see the previous warnings.
And I can see some graphs I believe I couldn't before. I believe this
should be set in ntop.init.

One time I got the following error, which should be corrected
(and Requires /usr/bin/dot or graphviz added):

Missing dot tool (expected /usr/local/bin/dot). Please set its path (key
dot.path) here.

Also rpmlint says (among ignorable warnings):

ntop.i386: W: service-default-enabled /etc/rc.d/init.d/ntop

As always the non-weak symbols should be reported upstream such that
they correct their link commands.


So it is basically ok for me, with the service-default-enabled
corrected, dot path correctly setup and LANG=C set before starting
ntop.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 249236] Review Request: kde-style-domino - KDE style with a soft look

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

Summary: Review Request: kde-style-domino - KDE style with a soft look


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|needinfo?([EMAIL PROTECTED]|
   |ail.com)|




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 05:43 EST ---
I think this package's name should be domino, like polyester style...

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

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


[Bug 387261] Review Request: libcmpiutil - Utility library for CIM providers

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

Summary: Review Request: libcmpiutil - Utility library for CIM providers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 365131] Review Request: ruby-imagesize - Measure image size code by Pure Ruby

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

Summary: Review Request: ruby-imagesize - Measure image size code by Pure Ruby


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 06:25 EST ---
Rebuilt on devel, F-8, F-7, requested on bodhi, closing.

Thank you for your 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, or are watching someone who is.

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


[Bug 378421] Review Request: starplot - 3-dimensional perspective star map viewer

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

Summary: Review Request: starplot - 3-dimensional perspective star map viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 06:26 EST ---
Actually all fixed.

-
 This package (starplot) is APPROVED by me
-

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

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


[Bug 376421] Review Request: tla - A version control system

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

Summary: Review Request: tla - A version control system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |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, or are watching someone who is.

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


[Bug 371421] Review Request: alienarena-data - Data files for Alien Arena 2007

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

Summary: Review Request: alienarena-data - Data files for Alien Arena 2007


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 06:57 EST ---
Doing a full review, as promised.


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

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


[Bug 378421] Review Request: starplot - 3-dimensional perspective star map viewer

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

Summary: Review Request: starplot - 3-dimensional perspective star map viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 06:52 EST ---
New Package CVS Request
===
Package Name: starplot
Short Description: 3-dimensional perspective star map viewer
Owners: rishi
Branches: F-7 F-8
InitialCC:
Cvsextras Commits: no

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

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


[Bug 373621] Review Request: qct - Multi-vcs GUI commit tool

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

Summary: Review Request: qct - Multi-vcs GUI commit tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 07:39 EST ---
https://nbecker.dyndns.org/RPM/qct.spec
https://nbecker.dyndns.org/RPM/qct-1.5-5.fc8.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, or are watching someone who is.

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


[Bug 383621] Review Request: gnofract4d - Gnofract 4D is a Gnome-based program to draw fractals

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

Summary: Review Request: gnofract4d - Gnofract 4D is a Gnome-based program to 
draw fractals


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   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, or are watching someone who is.

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


[Bug 314161] Review Request: noip - Dynamic DNS client for no-ip.com

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

Summary: Review Request: noip - Dynamic DNS client for no-ip.com


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED])




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:20 EST ---
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, or are watching someone who is.

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


[Bug 330831] Review Request: gbrainy - a brain teaser game

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

Summary: Review Request: gbrainy - a brain teaser game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||)




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:26 EST ---
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, or are watching someone who is.

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


[Bug 291741] Review Request: gnome-hearts - Game of Hearts implementation for gnome

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

Summary: Review Request: gnome-hearts - Game of Hearts implementation for gnome


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||org)




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:27 EST ---
Please someone update the status of this bug.

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

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


[Bug 307821] Review Request: sooperlooper - Realtime software looping sampler

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

Summary: Review Request: sooperlooper - Realtime software looping sampler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|NEW
   Flag|needinfo?   |




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:31 EST ---
Someone, 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, or are watching someone who is.

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


[Bug 371411] Review Request: alienarena - 3d Deathmatch game

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

Summary: Review Request: alienarena - 3d Deathmatch game


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:30 EST ---
Full review results (Thanks to Mamoru for some of these):

Must Fix:
-
* rpmlint: W: unstripped-binary-or-object /usr/lib/alienarena/game.so
  (chmod +x ?)

* Make the Requires: alienarena-data versioned, so that if a newer version of
  the data is needed by a newer release, an someone does yum update alienarena,
  the data will get updated too.

  Usually I use a = version requirement in the engine and a = version req on
  the engine in the data.

* '--vendor ' argument to desktop-file-install must be '--vendor fedora'

* Add ActionGame to the .desktop file Categories, so that alienarena will not
  end up on the main games sub-menu when the games menu has been further 
  sub-menud through install of the games-menus package (as the live dvd does for
  example)

* Remove these non RPM_OPTFLAGS from the flags used for the compile:
  -O2 -fomit-frame-pointer -ftree-vectorize -fexpensive-optimizations -march=k8
  Notice that upstream also adds this to CFLAGS:
  -fno-strict-aliasing -fmerge-constants
  But those are ok.

* You install symlinks to %{_libdir}/%{name}/game.so under /usr/share, but
  %{_libdir} is arch dependent and files under /usr/share may not be arch 
  dependent.

  Suggestion, instead use this in %prep:
  sed -i 's|game.so|%{_libdir}/%{name}/game.so|g' unix/sys_unix.c

  Together with a patch commenting the stupid code where it first tries to
  fopen the .so in a number of different places totally irrelevant to dlopen's
  search path (will attach the patch next).


Should Fix:
---
* defattr
  - We now recommend %defattr(-,root,root,-)
* Timestamps
  - Please use -p option to keep timestamps when using
cp or install commands (for SOURCE3, SOURCE1)
* -n %{name}-%{version} to %setup is redundant, remove please
* Make .desktop file use aa as Icon, not /full/path/Aa.ico
* Use just alienarena as Exec in the .desktop file not a full path
* The symlinks to the binaries under /usr/share/alienarena do not appear to be
  needed
* Since crx.sdl may not be directly called, it should be moved to %_libexecdir
* Change wrapper script from:
---
#!/bin/bash

cd /usr/share/alienarena/
./crx.sdl
---
   To:
---
#!/bin/bash

cd /usr/share/alienarena/
exec /usr/libexec/crx.sdl $@
---
So that bash doesn't linger in memory and cmd line parameters can be passed.


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

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


[Bug 253549] Review Request: TzClock - GTK+ Time Zone Clock

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

Summary: Review Request: TzClock - GTK+ Time Zone Clock


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:25 EST ---
Again ping?

It may be that this bug will be closed if no response from the reporter
is received within ONE WEEK.

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

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


[Bug 383621] Review Request: gnofract4d - Gnofract 4D is a Gnome-based program to draw fractals

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

Summary: Review Request: gnofract4d - Gnofract 4D is a Gnome-based program to 
draw fractals


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:18 EST ---
By the way, I would appreciate it if you would review my review
request (bug 380081).

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

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


[Bug 383621] Review Request: gnofract4d - Gnofract 4D is a Gnome-based program to draw fractals

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

Summary: Review Request: gnofract4d - Gnofract 4D is a Gnome-based program to 
draw fractals


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:17 EST ---
For 3.6-1:

* License
  - 3 files
--
lex.py
yacc.py
fract4dgui/FCTGen.py
--
makes the license of this package LGPLv2+, not BSD.

* Redundant BuildRequires
  - Not a big issue, however some of BuildRequires are
redundant
* pkgconfig is needed by gtk2-devel
* libpng-devel is needed by gtk2-devel

* desktop-file-install
  - build.log says %{buildroot}%{_datadir}/gnofract4d/gnofract4d.desktop
does not exist.
  - Category X-Fedora is deprecated and should not be added.

* scriptlets
  http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
  - scrollkeeper-updates does not seem to be needed.
  - mime database must be updated
  - desktop database must be updated as installed desktop file
contains MimeType key.

* %check
  - If some test program can be done, create %check section and write
some test program with in the section (what is test.py for?)

* %files entry
  - By the way, the two lines
-
%dir %{python_sitearch}/fract4d/
%{python_sitearch}/fract4d/*
-
can be replaced by one line
-
%{python_sitearch}/fract4d/
-

* About non-executable-script
  - Non executable script should not have shebangs.

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

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


[Bug 307901] Review Request: lsvpd - A utility to list device Vital Product Data (VPD) information.

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

Summary: Review Request: lsvpd - A utility to list device Vital Product Data 
(VPD) information.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:37 EST ---
MUSTFIX:

* You must not touch the running system when building a package, like you do 
here:

%install
%{__rm} -rf $RPM_BUILD_ROOT
%{__make} install DESTDIR=$RPM_BUILD_ROOT

if [ -e /etc/udev/rules.d/99-lsvpd.rules ] ; then
%{__rm} /etc/udev/rules.d/99-lsvpd.rules
fi

if [ -e /etc/udev/rules.d/99-lsvpd.disabled ] ; then
%{__rm} /etc/udev/rules.d/99-lsvpd.disabled
fi

* Missing: 
Requires(post): /usr/sbin/vpdupdate

* Replace all /etc/lsvpd with %{_sysconfdir}/lsvpd
It this package which installs it there through %configure

* Requires: /sbin/chkconfig is wrong
You want Requires(post) and Requires(preun).

* Remove stray/unused %postun



CONSIDER:
* Consider to remove inserv etc. I an not sure if we want Fedora's rpms'
scriptlets to be cluttered with insserv.

* Would you please explain:
Requires: libvpd_cxx

AFAIS, this is superflous and should be removed.



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

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


[Bug 307901] Review Request: lsvpd - A utility to list device Vital Product Data (VPD) information.

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

Summary: Review Request: lsvpd - A utility to list device Vital Product Data 
(VPD) information.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 08:07 EST ---
The shape is much better now. The only suggestion I have is to ditch the INSTALL
file from %doc because it brings no value to Fedora users. Just keep this in
mind and implement whenever you happen to modify the spec, it's not at all
critical. Make sure it occurs before the commit phase, once a sponsor approves
the 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, or are watching someone who is.

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


[Bug 371411] Review Request: alienarena - 3d Deathmatch game

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

Summary: Review Request: alienarena - 3d Deathmatch game


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 09:31 EST ---
Created an attachment (id=262271)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=262271action=view)
Patch to not look for .so files under /usr/share


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

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


[Bug 371421] Review Request: alienarena-data - Data files for Alien Arena 2007

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

Summary: Review Request: alienarena-data - Data files for Alien Arena 2007


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 09:33 EST ---
Full review done

Should fix:
-mv aa.png to /usr/share/icons/hicolor/32x32/apps
-add icon-cache scriptlets
-do not ship Aa.ico
-remove unused Tools subdir?
-make the requires on alienarena versioned?

No blockers, 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, or are watching someone who is.

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


[Bug 307821] Review Request: sooperlooper - Realtime software looping sampler

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

Summary: Review Request: sooperlooper - Realtime software looping sampler


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 09:37 EST ---
I guess that reviewers are waiting for a new version with the .desktop
Catogories  fixed and that Fernando is busy and waiting for a full review.

I'll try todo a full review as time permits.


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

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


[Bug 376421] Review Request: tla - A version control system

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

Summary: Review Request: tla - A version control system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 09:35 EST ---
Sorry for the confusion here too. ;( 

I have reset you back to owner, so you should be all set now. 
Let me know if you need any further cvs changes. 

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

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


[Bug 329291] Review Request: debootstrap - Bootstrap a basic Debian GNU/Linux system

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

Summary: Review Request: debootstrap - Bootstrap a basic Debian GNU/Linux system
Alias: debootstrap-review

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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 09:39 EST ---
You should use the latest package. The -udeb part has been
removed, it now installs in /usr/share and is much simpler.

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

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


[Bug 280751] Review Request: qmmp - Qt-based multimedia player

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

Summary: Review Request: qmmp - Qt-based multimedia player


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 09:52 EST ---
Hi Karel,

I see that you are looking for someone to sponsor you, and as it happens I'm
allowed to sponsor people. Here is what I would like to do: I see that you've
submitted 3 packages for review, I will review all 3 of them and then after one
or two iterations the can hopefully be approved, assuming that process go well
I'll sponsor you when all 3 are 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, or are watching someone who is.

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


[Bug 226415] Merge Review: sgml-common

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

Summary: Merge Review: sgml-common


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 09:49 EST ---
(In reply to comment #4)

 So the points which I want to discuss are:
 1) Do you really think that including openjade tarball into sources to digout 
 a
 few files from it is necessary? Those files didn't changed for years and I'm
 maintainer of Openjade package - and it is mentioned that the files are from
 OpenJade/pubtext dir.

Ok, but then add to the comment that you are the openjade maintainer
and verify that the timestamps are right such that it is easy to 
check the file age and compare with the files from openjade. If the 
timestamps are not right and cannot be changes (it is often the case
when the files are already in cvs), please note the original timestamps
in the spec file.

 2) About that usage of automake package: This would require to rewrite some
 things inside the tarball and without adding any value - except more clean
package.

I don't really understand why. What it the technical issue here?
In any case having clean package is an explicit goal of 
fedora packaging and package review.

 3) suggestion about with-docdir.patch for upstream - there is no upstream
 page/bugzilla afaik, last package version is made 3 years ago - so I don't 
 know
 how can I get it to upstream and remove from fedora.

You can keep it in the srpm if you like but you shouldn't apply
it nor rerun the autotools, there are less intrusive ways to
achieve the same result. I can do some spec file patch if you like.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 282521] Review Request: dblatex - DocBook to LaTeX/ConTeXt Publishing

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

Summary: Review Request: dblatex - DocBook to LaTeX/ConTeXt Publishing


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 10:01 EST ---
Neal, could you please approve all my packagedb requests
for dblatex and python-which?

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

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


[Bug 280751] Review Request: qmmp - Qt-based multimedia player

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

Summary: Review Request: qmmp - Qt-based multimedia player


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 10:20 EST ---
Here are the Must Fix and Should fix items resulting from a full review, notice
that I've also recycled some of the comments above, thanks to all involved for
those!

Must Fix:
-
* buildroot does not match the buildroot mandated by the guidelines
* remove * MPEG1 layer 1/2/3 support; from %description
* BuildRequires line longer then 80 chars, please split this up in multiple
  lines each starting with BuildRequires:
* desktop-file-install line longer then 80 chars, please split this up in
  multiple lines.
* use %defattr(-,root,root,-)
* put %doc directly under %defattr
* drop %{?_smp_mflags}, if it fails on some systems it must be dropped, the fact
  that it happens to work on others is not relevant, we don't want a lottery, we
  want reproducable builds
* add: Requires(post): /sbin/ldconfig and Requires(postun): /sbin/ldconfig
* All these lines:
  %dir %{_libdir}/qmmp
  %dir %{_libdir}/qmmp/Input
  %dir %{_libdir}/qmmp/Output
  %{_libdir}/qmmp/Input/*.so
  %{_libdir}/qmmp/Output/*.so
  Can be written as just:
  %{_libdir}/qmmp
  Notice that you will still need:
  %{_libdir}/libqmmp.so
  Listed seperately
* Drop the second %defattr line


Possible improvements:
--
* Use single quotes instead of double quotes around your sed scripts so that
  you do not have to use \ infront of  inside the scripts.



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

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


[Bug 385041] Review Request: swfdec - Flash animation rendering library

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

Summary: Review Request: swfdec - Flash animation rendering library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 10:20 EST ---
(In reply to comment #4)
 Maybe MAINTAINERS should be in %doc 

Not going to bother packaging this, since it's fairly redundant with the AUTHORS
file already being included.
 
 Same remark about source file timestamp not kept:
 -rw-rw-r-- 1 dumas dumas 6857084 nov 15 16:23 ../SOURCES/swfdec-0.5.4.tar.gz
 -rw-rw-r-- 1 dumas dumas 6857084 nov 15 11:05 swfdec-0.5.4.tar.gz

Already been fixed.

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

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


[Bug 280751] Review Request: qmmp - Qt-based multimedia player

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

Summary: Review Request: qmmp - Qt-based multimedia player


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 10:37 EST ---
Hmm,

I've just find out I've been mixing you up with someone else who also needs a
sponsor and who has 2 packages submitted for review, hence my: I see that 
you've
submitted 3 packages for review remark, my bad. Please submit some more
packages and / or do some pre reviews (some normal reviews were you clearly
indicate you cannot do official reviews as you are not sponsored yet), thanks.


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

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


[Bug 385041] Review Request: swfdec - Flash animation rendering library

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

Summary: Review Request: swfdec - Flash animation rendering library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 10:41 EST ---
I still have a different timestamp for the file I get with spectool -g
and the one in srpm. Not a blocker, though and the timestamps are
very similar.

* follow guidelines
* free software, license included
* rpmlint can be ignored:
swfdec.src: W: mixed-use-of-spaces-and-tabs (spaces: line 117, tab: line 1)
swfdec-gtk.i386: W: no-documentation
swfdec-gtk-devel.i386: W: no-documentation
* match upstream
506b032204b71ffac7d0424eaf9608c4  swfdec-0.5.4.tar.gz
* libs correctly packaged
* %files section right

I guess that you know what you do when you put the icons in the
gtk subpackage.

I haven't verified precisely the buildrequires but they look
fine.

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, or are watching someone who is.

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


[Bug 386341] Review Request: fxload - A helper program to download firmware into FX and FX2 EZ-USB devices

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

Summary: Review Request: fxload - A helper program to download firmware into FX 
and FX2 EZ-USB devices


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 10:53 EST ---
I'm not aware of any finer meaning of in conjunction with. While I do not think
it was Cypress's intent for this to be a distribution restriction (instead of
just a use restriction), the wording is poorly formulated.

a3load is not permissable for Fedora, unless someone from Cypress Semiconductor
Corp. clarifies the intent (or even better, rewords the license).

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

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


[Bug 385041] Review Request: swfdec - Flash animation rendering library

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

Summary: Review Request: swfdec - Flash animation rendering library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 10:56 EST ---
New Package CVS Request
===
Package Name: swfdec
Short Description: Flash animation rendering library
Owners: bpepple
Cvsextras Commits: yes

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

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


[Bug 178922] Review Request: asterisk - The Open Source PBX

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

Summary: Review Request: asterisk - The Open Source PBX


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 11:07 EST ---
Use mock as it comes with a centos/epel config file supplied

I found I couldn't build for el5 due to the requirement for autoconf = 2.60
when el5 comes with 2.59

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 373621] Review Request: qct - Multi-vcs GUI commit tool

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

Summary: Review Request: qct - Multi-vcs GUI commit tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 11:16 EST ---
Thank you, I'm pleased to label this, 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, or are watching someone who is.

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


[Bug 316141] Review Request: gridengine - Grid Engine - Distributed Computing Management software

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

Summary: Review Request: gridengine - Grid Engine - Distributed Computing 
Management software


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||182235
  nThis||




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

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


[Bug 316141] Review Request: gridengine - Grid Engine - Distributed Computing Management software

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

Summary: Review Request: gridengine - Grid Engine - Distributed Computing 
Management software


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 11:26 EST ---
SISSL is GPL incompatible. You cannot have any SISSL code which links to GPL
code. Looking at the above summary, the ltree code linking to the rest of qmon
is a problem, but everything else seems ok. (Note: I have not done a code 
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, or are watching someone who is.

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


[Bug 388541] New: Review Request: fedoraflyinghigh - Flying High wallpapers for GNOME (slideshow)

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




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

   Summary: Review Request: fedoraflyinghigh - Flying High
wallpapers for GNOME (slideshow)
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://liviopl.fedorapeople.org/fedoraflyinghigh-backgrounds.spec
SRPM URL: 
http://liviopl.fedorapeople.org/fedoraflyinghigh-backgrounds-7-2.fc8.src.rpm
Description: Flying High wallpapers for GNOME (slideshow)

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

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


[Bug 355971] Review Request: poker3d - Three dimensional multi-user online poker game

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

Summary: Review Request: poker3d - Three dimensional multi-user online poker 
game
Alias: poker3d

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 11:49 EST ---
Hi, che mentioned a couple problems last night, and I asked him to post to this
bug.  I put him on the CC list.  One thing he mentioned was a missing

Requires: xwnc

I think he also mentioned something else too, but I'm not sure, so I want to
wait to hear from him.

@Hans: Did xwnc get installed on your system when you tested 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, or are watching someone who is.

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


[Bug 382761] Review Request: python-cerealizer - Secure pickle-like module

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

Summary: Review Request: python-cerealizer - Secure pickle-like module


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
   Flag||fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 11:55 EST ---
CVS done, builds building. Thanks for 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, or are watching someone who is.

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


[Bug 355971] Review Request: poker3d - Three dimensional multi-user online poker game

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

Summary: Review Request: poker3d - Three dimensional multi-user online poker 
game
Alias: poker3d

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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:02 EST ---
(In reply to comment #20)
 
 @Hans: Did xwnc get installed on your system when you tested this?

I must admit I didn't test it. I'm not much into poker, I only did a technical
review, and for the record, no xwnc did not get installed.


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

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


[Bug 370751] Review Request: ggz-client-libs - Client libraries for GGZ gaming zone

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

Summary: Review Request: ggz-client-libs - Client libraries for GGZ gaming zone


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:03 EST ---
* Sat Nov 17 2007 Rex Dieter rdieter[AT]fedoraproject.org 0.0.14-4
- --disable-ggzwrap (for now, until multilib, licensing is sorted out)
- move ggz-config to main pkg (runtime management of ggz modules)
- clarify GPL vs. LGPL bits
- drop BR: automake libtool

Spec URL: 
http://kdeforge.unl.edu/apt/kde-redhat/SOURCES/ggz/ggz-client-libs.spec
SRPM URL:
http://kdeforge.unl.edu/apt/kde-redhat/SOURCES/ggz/ggz-client-libs-0.0.14-4.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, or are watching someone who is.

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


[Bug 291741] Review Request: gnome-hearts - Game of Hearts implementation for gnome

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

Summary: Review Request: gnome-hearts - Game of Hearts implementation for gnome


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|NEW
   Flag|needinfo?([EMAIL PROTECTED]|
   |org)|




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 11:36 EST ---
Will do a release this weekend. I was planning on waiting for the 0.2.1 version,
but it's not yet released (or it's not on the website, yet).

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

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


[Bug 291371] Review Request: teg - teg is a clone of a clone of Risk

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

Summary: Review Request: teg - teg is a clone of a clone of Risk


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:05 EST ---
I see that you are looking for someone to sponsor you, and as it happens I'm
allowed to sponsor people. Here is what I would like to do: I see that you've
submitted 3 packages for review, I will review all 3 of them and then after one
or two iterations the can hopefully be approved, assuming that process go well
I'll sponsor you when all 3 are approved, if you agree with this procedure,
please let me know and I'll start reviewing all 3.




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

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


[Bug 245357] Review Request: libopensync-plugin-syncml - plugin for using syncml with opensync

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

Summary: Review Request: libopensync-plugin-syncml - plugin for using syncml 
with opensync


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:04 EST ---
I see that you are looking for someone to sponsor you, and as it happens I'm
allowed to sponsor people. Here is what I would like to do: I see that you've
submitted 3 packages for review, I will review all 3 of them and then after one
or two iterations the can hopefully be approved, assuming that process go well
I'll sponsor you when all 3 are approved, if you agree with this procedure,
please let me know and I'll start reviewing all 3.




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

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


[Bug 246351] Review Request: libopensync-plugin-gnokii - plugin for using gnokii with opensync

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

Summary: Review Request: libopensync-plugin-gnokii - plugin for using gnokii 
with opensync


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:05 EST ---
I see that you are looking for someone to sponsor you, and as it happens I'm
allowed to sponsor people. Here is what I would like to do: I see that you've
submitted 3 packages for review, I will review all 3 of them and then after one
or two iterations the can hopefully be approved, assuming that process go well
I'll sponsor you when all 3 are approved, if you agree with this procedure,
please let me know and I'll start reviewing all 3.




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

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


[Bug 374531] Review Request: kdebase-runtime - K Desktop Environment - Runtime

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

Summary: Review Request: kdebase-runtime - K Desktop Environment - Runtime


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:11 EST ---
re: xine+lib+libxcb on F-7, see bug #373411

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

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


[Bug 178922] Review Request: asterisk - The Open Source PBX

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

Summary: Review Request: asterisk - The Open Source PBX


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:13 EST ---
(In reply to comment #139)
 Use mock as it comes with a centos/epel config file supplied

Yes, I have mock as well.  I just don't have any RHEL4/CentOS4 systems in use so
I didn't try building under mock.

 I found I couldn't build for el5 due to the requirement for autoconf = 2.60
 when el5 comes with 2.59

I have a newer package under development that doesn't require autoconf during
the build, so that's not a problem.  Asterisk 1.4.14 was released yesterday so I
 hope to have new packages uploaded this weekend.  There are a couple of other
problematic packages, notably speex.  speex is at version 1.0 in the base
RHEL/CentOS package set but speex 1.2beta is needed for the build.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 355971] Review Request: poker3d - Three dimensional multi-user online poker game

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

Summary: Review Request: poker3d - Three dimensional multi-user online poker 
game
Alias: poker3d

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:14 EST ---
Spec URL: http://tkmame.retrogames.com/fedora-extras/poker3d.spec
SRPM URL: 
http://tkmame.retrogames.com/fedora-extras/poker3d-1.1.36-6.fc7.src.rpm

%changelog
* Sat Nov 17 2007 Christopher Stone [EMAIL PROTECTED] 1.1.36-6
- Remove no longer needed ldconfig
- Add xwnc to Requires

This one should be good enough for updates-testing. I'll have che and a couple
others test it from the testing repo.  Thanks for the review!

New Package CVS Request
===
Package Name: poker3d
Short Description: Three dimensional multi-user online poker game
Owners: xulchris
Branches: F-7 F-8
InitialCC:
Cvsextras Commits: yes


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

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


[Bug 374531] Review Request: kdebase-runtime - K Desktop Environment - Runtime

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

Summary: Review Request: kdebase-runtime - K Desktop Environment - Runtime


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:18 EST ---
Spec URL:
http://kdeforge.unl.edu/apt/kde-redhat/SOURCES/kdebase-runtime/kdebase-runtime.spec
SRPM URL:
http://kdeforge.unl.edu/apt/kde-redhat/SOURCES/kdebase-runtime/kdebase-runtime-3.96.0-2.src.rpm

* Sat Nov 17 2007 Rex Dieter rdieter[AT]fedoraproject.org 3.96.0-2
- BR: clucene-core-devel libsmbclient-devel libXScrnSaver-devel

* Thu Nov 15 2007 Rex Dieter rdieter[AT]fedoraproject.org 3.96.0-1
- kde-3.96.0


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

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


[Bug 367871] Review Request: gnome-theme-curvylooks - A modern Clearlooks theme using a Bluecurve-like color scheme

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

Summary: Review Request: gnome-theme-curvylooks -  A modern Clearlooks theme 
using a Bluecurve-like color scheme


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


[EMAIL PROTECTED] 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, or are watching someone who is.

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


[Bug 367871] Review Request: gnome-theme-curvylooks - A modern Clearlooks theme using a Bluecurve-like color scheme

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

Summary: Review Request: gnome-theme-curvylooks -  A modern Clearlooks theme 
using a Bluecurve-like color scheme


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:25 EST ---
+ source files match upstream: c68acb3013b82b0e3852f3a5da46aa869be00832

+ package meets naming and versioning guidelines.
+ specfile is properly named, is cleanly written and uses macros consistently.
+ dist tag is present.
+ build root is correct.
+ license field matches the actual license.
+ license is open source-compatible.
+ latest version is being packaged.
+ BuildRequires are proper.
+ %clean is present.
+ package builds in mock ( ).
+ package installs properly
+ rpmlint is silent.
+ final provides and requires are sane:
  * Provides:
gnome-theme-curvylooks = 0.2-1.fc8
  * Requires:
gtk2-engines = 2.12
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
+ no shared libraries are added to the regular linker search paths.
+ owns the directories it creates.
+ doesn't own any directories it shouldn't.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets present.
+ %docs are not necessary for the proper functioning of the package.
+ no headers.
+ no pkgconfig files.
+ no libtool .la droppings.
+ not a GUI app.
ACCEPTED

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

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


[Bug 291741] Review Request: gnome-hearts - Game of Hearts implementation for gnome

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

Summary: Review Request: gnome-hearts - Game of Hearts implementation for gnome


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:24 EST ---
In the case that SVN contains specific patches that you want to use, you can
just release based on the last stable release, get the required patches from
SVN, and apply them during the build setup phase with %patch

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

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


[Bug 178922] Review Request: asterisk - The Open Source PBX

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

Summary: Review Request: asterisk - The Open Source PBX


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:23 EST ---
I'm willing to test and report problems if you build it for epel4. there are 
lot of those installations out there for long time.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 374531] Review Request: kdebase-runtime - K Desktop Environment - Runtime

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

Summary: Review Request: kdebase-runtime - K Desktop Environment - Runtime


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:38 EST ---
scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=246299


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

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


[Bug 383621] Review Request: gnofract4d - Gnofract 4D is a Gnome-based program to draw fractals

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

Summary: Review Request: gnofract4d - Gnofract 4D is a Gnome-based program to 
draw fractals


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:58 EST ---
I've looked at test.py, two tests fail every time.

One checks the .desktop file's Version= key and compares it to the program
version, but this is invalid as Version= is used to designate the freedesktop
standard version, not the program version so that test has been patched out.

The second one tests generating an AVI file from fractals frames. Since
Transcode isn't in the Fedora repositories that test has also been patched out.
Selecting Director from the GUI informs the user that transcode isn't installed,
no harm done.

SPEC: http://www.diffingo.com/downloads/diffingo-repo/review/gnofract4d.spec
SRPM:
http://www.diffingo.com/downloads/diffingo-repo/review/gnofract4d-3.6-2.fc8.src.rpm

Changes:
* Sat Nov 17 2007 Stewart Adam s.adam at diffingo.com - 3.6-2
- License is actually LGPLv2+ because of lex.py, yacc.py, FCTGen.py
- Add patch for test suite files since two tests are invalid
- Update MIME and desktop databases
- Use virtual X for GUI test suite
- Remove redundant BR
- Remove redundant entries from %%files
- Fix rpmlint's errors about executable files/shebangs

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

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


[Bug 329291] Review Request: debootstrap - Bootstrap a basic Debian GNU/Linux system

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

Summary: Review Request: debootstrap - Bootstrap a basic Debian GNU/Linux system
Alias: debootstrap-review

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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:03 EST ---
crap :)

I will look at it

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

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


[Bug 380081] Review Request: rubygem-zoom - Ruby binding to ZOOM

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

Summary: Review Request: rubygem-zoom - Ruby binding to ZOOM


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   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, or are watching someone who is.

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


[Bug 291741] Review Request: gnome-hearts - Game of Hearts implementation for gnome

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

Summary: Review Request: gnome-hearts - Game of Hearts implementation for gnome


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:43 EST ---
 I was planning on waiting for the 0.2.1 version

It should be out in a week or two. The code is basically done. All that's left
is fixing up the NEWS, README, changelog, and importing the last translations
from launchpad. That kind of stuff. The real reason I haven't released it yet is
because of a hardware upgrade on my server. I'm waiting for the last few
component to install a new SATA hardware RAID. That server runs my svn,
bugzilla, etcetera.

I don't want to do a release and then pull the plug on my bugzilla and
subversion the next day for the upgrade. I'm playing it safe. Upgrade first,
release shortly after.

If you want, you can use the code from branches/0.2.x HEAD for the redhat
package. It's just a bunch of bugfixes on top of the 0.2 release. No new 
features.

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

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


[Bug 386341] Review Request: fxload - A helper program to download firmware into FX and FX2 EZ-USB devices

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

Summary: Review Request: fxload - A helper program to download firmware into FX 
and FX2 EZ-USB devices


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|182235  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 12:44 EST ---
OK. Version 4 of the package in comment 5 above completely removes a3load.hex.

The package is certainly still useful without this file; only certain HW needs
the 2nd stage loader (don't know which). The Xilinx Spartan 3E starter board's
USB programmer, which I'm using with this S/W, certainly doesn't need a3load.

I guess we should start out packaging/distributing the SW without a3load, and
I'll see if I can get Cypress to clarify/reword the license, so we can include
a3load in a future revision.

As such, removing FE-legal blocking; hope that's 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, or are watching someone who is.

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


[Bug 380081] Review Request: rubygem-zoom - Ruby binding to ZOOM

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

Summary: Review Request: rubygem-zoom - Ruby binding to ZOOM


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:06 EST ---
+ source files match upstream: 117dd33cd73fa4fd7f32c52034142eb1394f9365

+ package meets naming and versioning guidelines.
+ specfile is properly named, is cleanly written and uses macros consistently.
+ dist tag is present.
+ build root is correct.
+ license field matches the actual license.
+ license is open source-compatible.  License text not included upstream.
+ latest version is being packaged.
+ BuildRequires are proper.
+ compiler flags are appropriate.
+ %clean is present.
+ package builds in mock
+ package installs properly
+ debuginfo package looks complete.
+ rpmlint is silent.
  * The obsoletes warning OK, explained in specfile
+ final provides and requires are sane:
$ rpm -q --provides --requires rubygem-zoom.x86_64
  rubygem(zoom) = 0.3.0-1.fc8
  zoom.so()(64bit)  
  rubygem-zoom = 0.3.0-1.fc8
  /usr/bin/ruby  
  libc.so.6()(64bit)  
  libc.so.6(GLIBC_2.2.5)(64bit)  
  libc.so.6(GLIBC_2.3)(64bit)  
  libc.so.6(GLIBC_2.3.4)(64bit)  
  libc.so.6(GLIBC_2.4)(64bit)  
  libcrypt.so.1()(64bit)  
  libcrypto.so.6()(64bit)  
  libdl.so.2()(64bit)  
  libexslt.so.0()(64bit)  
  libgcrypt.so.11()(64bit)  
  libgpg-error.so.0()(64bit)  
  libm.so.6()(64bit)  
  libpthread.so.0()(64bit)  
  libruby.so.1.8()(64bit)  
  libssl.so.6()(64bit)  
  libwrap.so.0()(64bit)  
  libxml2.so.2()(64bit)  
  libxslt.so.1()(64bit)  
  libyaz.so.3()(64bit)  
  libz.so.1()(64bit)  
  rpmlib(CompressedFileNames) = 3.0.4-1
  rpmlib(PayloadFilesHavePrefix) = 4.0-1
  rpmlib(VersionedDependencies) = 3.0.3-1
  rtld(GNU_HASH)  
  ruby(abi) = 1.8
  ruby(rubygems)
+ no shared libraries are added to the regular linker search paths.
+ owns the directories it creates.
+ doesn't own any directories it shouldn't.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets present.
+ code, not content.
+ no headers.
+ no pkgconfig files.
+ no libtool .la droppings.
+ not a GUI app.
ACCEPTED

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

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


[Bug 368161] Review Request: awn-extras-applets - extras applets of avant window navigator

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

Summary: Review Request: awn-extras-applets - extras applets of avant window 
navigator


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:25 EST ---
Now that it has been built, we can close 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, or are watching someone who is.

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


[Bug 307821] Review Request: sooperlooper - Realtime software looping sampler

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

Summary: Review Request: sooperlooper - Realtime software looping sampler


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:25 EST ---
(In reply to comment #9)
 I guess that reviewers are waiting for a new version with the .desktop
 Catogories  fixed and that Fernando is busy and waiting for a full review.
 
 I'll try todo a full review as time permits.

I'm sorry about the delay, I've been very busy (and this last week rebuilding
Planet CCRMA on f8 on top of everything, out soon!). I'll see if I can release a
new spec/srpm later today for a full 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, or are watching someone who is.

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


[Bug 291741] Review Request: gnome-hearts - Game of Hearts implementation for gnome

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

Summary: Review Request: gnome-hearts - Game of Hearts implementation for gnome


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:36 EST ---
I'm actually wondering if a versioning based on SVN might be a good thing to do
right now since, I'm assuming, these are all development releases anyway. On my
own machine, I'd probably have rpmbuild pull source directly from SVN. But as
this is going on koji, I might have to do the %patch route.

Thanks for the input.

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

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


[Bug 381241] Review Request: ncl - NCAR Command Language and NCAR Graphics

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

Summary: Review Request: ncl -  NCAR Command Language and NCAR Graphics


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:39 EST ---
The remaining rpmlint complaints are:
  ncl.x86_64: W: non-conffile-in-etc /etc/profile.d/ncarg.csh
  ncl.x86_64: W: non-conffile-in-etc /etc/profile.d/ncarg.sh
which are both OK as I understand things.

I know this isn't going to be a pretty package, what with needing both imake and
fortran to build, but the end result doesn't seem terribly bad.

Some of the binaries seem destined to conflict with something (fontc, med,
psplit, etc.) but this package has been around for so long that it would be
pointless to think of renaming any of them.

Now, on to the stuff in %_libdir:

First, why is it all in %_libdir/ncarg/ncarg?  Why not just one ncarg?

What are the .o files in %_libdir/ncarg/ncarg/robj for?

Your -devel package should provide ncl-static = %{version}-%{release}.

But really, for a crufty piece of software like this, this package isn't bad.

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

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


[Bug 367871] Review Request: gnome-theme-curvylooks - A modern Clearlooks theme using a Bluecurve-like color scheme

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

Summary: Review Request: gnome-theme-curvylooks -  A modern Clearlooks theme 
using a Bluecurve-like color scheme


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:46 EST ---
Yay! Thanks for the review, Stewart.

New Package CVS Request
===
Package Name: gnome-theme-curvylooks
Short Description: A modern Clearlooks theme using a Bluecurve-like color scheme
Owners: pgordon
Branches: F-8 devel
InitialCC: (none)
Cvsextras Commits: yes

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

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


[Bug 329291] Review Request: debootstrap - Bootstrap a basic Debian GNU/Linux system

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

Summary: Review Request: debootstrap - Bootstrap a basic Debian GNU/Linux system
Alias: debootstrap-review

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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:52 EST ---
SPEC: http://people.redhat.com/lkundrak/SPECS/debootstrap.spec
SRPM:
http://people.redhat.com/lkundrak/mock-results/debootstrap-1.0.7-1.fc9.noarch/debootstrap-1.0.7-1.fc9.src.rpm
mock: 
http://people.redhat.com/lkundrak/mock-results/debootstrap-1.0.7-1.fc9.noarch/

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

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


[Bug 307821] Review Request: sooperlooper - Realtime software looping sampler

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

Summary: Review Request: sooperlooper - Realtime software looping sampler


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 13:50 EST ---
(In reply to comment #10)
 (In reply to comment #9)
  I guess that reviewers are waiting for a new version with the .desktop
  Catogories  fixed and that Fernando is busy and waiting for a full review.
  
  I'll try todo a full review as time permits.
 
 I'm sorry about the delay, I've been very busy (and this last week rebuilding
 Planet CCRMA on f8 on top of everything, out soon!). I'll see if I can 
 release a
 new spec/srpm later today for a full review. 

Spec URL: http://ccrma.stanford.edu/planetccrma/extras/sooperlooper.spec
SRPM URL: 
http://ccrma.stanford.edu/planetccrma/extras/sooperlooper-1.2.0-2.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, or are watching someone who is.

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


[Bug 291741] Review Request: gnome-hearts - Game of Hearts implementation for gnome

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

Summary: Review Request: gnome-hearts - Game of Hearts implementation for gnome


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 14:04 EST ---
You can do an svn export and then tar up the result and include that as the
Source tarball in your package.  All you need to do is to make sure it's
reproducible (so you should check out a specific version) and you need to
document the procedure in comments in the specfile.  See
http://fedoraproject.org/wiki/Packaging/SourceURL


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

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


[Bug 376421] Review Request: tla - A version control system

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

Summary: Review Request: tla - A version control system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 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, or are watching someone who is.

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


[Bug 382371] Review Request: gnome-do - quick object search and interaction

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

Summary: Review Request: gnome-do - quick object search and interaction
Alias: gnome-do

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 14:30 EST ---
Review: gnome-do 0.0.2-2

Okay, the update successfully builds in Mock (devel, x86_64 and i386). 

rpmlint only has a few minor complaints:
 gnome-do.x86_64: E: no-binary
 gnome-do.x86_64: E: only-non-binary-in-usr-lib

These are ignorable, since it's a Mono package and it doesn't ship native code,
but still needs to use the per-arch lib(64) directories, and has only a small
wrapper script in /usr/bin.

 gnome-do.x86_64: W: no-documentation
 gnome-do-devel.x86_64: W: no-documentation

These are because your package has no %doc files. Unfortunately, there are no
reasonable files in the tarball that would be appropriate here. Thus, this is
acceptable; but *please* bug upstream about including some files here: at the
very least, a copy of the license text and perhaps an AUTHORS or MAINTAINERS
file.


=== GOOD ===
+ Builds in mock (Devel, x86_64 and i386).
+ Included source tarball matches that of upstream:
4a5287dd85d920771e9c9c086b532a51  gnome-do_0.0.2-srpm.orig.tar.gz
4a5287dd85d920771e9c9c086b532a51  gnome-do_0.0.2-upstream.orig.tar.gz
+ All necessary BuildRequires are listed, with no duplicates.
+ Final file/directory ownership is good; no duplicates are listed.
+ File/directory permissions are OK; proper %defattr is used.
+ Final requires/provides are OK. Provides:
mono(Do) = 1.0.2877.19873
mono(Do.Addins) = 1.0.2877.19872
mono(Do.DBus) = 1.0.2877.19872
gnome-do = 0.0.2-2.fc8

Requires:
/bin/sh  
mono(Do.Addins) = 1.0.2877.19872
mono(Do.DBus) = 1.0.2877.19872
mono(Mono.Cairo) = 2.0.0.0
mono(System) = 2.0.0.0
mono(gconf-sharp) = 2.16.0.0
mono(gdk-sharp) = 2.10.0.0
mono(gtk-sharp) = 2.10.0.0
mono(mscorlib) = 2.0.0.0
mono(pango-sharp) = 2.10.0.0
mono-core  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(VersionedDependencies) = 3.0.3-1

+ When built, the application runs fine with no apparent bugs or segfaults/etc.

All blockers have been fixed, so gnome-do 0.0.2-2 is APPROVED.

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

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


[Bug 382371] Review Request: gnome-do - quick object search and interaction

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

Summary: Review Request: gnome-do - quick object search and interaction
Alias: gnome-do

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 15:48 EST ---
New Package CVS Request
===
Package Name: gnome-do
Short Description: quick object search and interaction for the GNOME desktop
Owners: dnielsen
Branches: devel F-8 F-7
InitialCC: (none)
Cvsextras Commits: Yes

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

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


[Bug 386531] Review Request: kuftp - Graphical FTP Client for KDE

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

Summary: Review Request: kuftp - Graphical FTP Client for KDE


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
OtherBugsDependingO||177841
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 15:46 EST ---
(In reply to comment #2)
 New .spec and srpm files uploadet. 

Please increase the %{release} everytime you make changes to your packet. So 
the reviewer could follow. And of course the url to the new packages then must 
be posted here.

Added 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, or are watching someone who is.

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


[Bug 291741] Review Request: gnome-hearts - Game of Hearts implementation for gnome

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

Summary: Review Request: gnome-hearts - Game of Hearts implementation for gnome


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 15:53 EST ---
By the way, this is actually my first attempt at packaging and I thought I was
following the procedure for first-time packagers. I don't think I have build
system access as Jason Tibbits mentions above. I do not know why this bug isn't
blocking FE-NEEDSPONSOR. Perhaps I missed something in the process, but I tried
to follow it as best as I could and is why I'm in my current status.

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

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


[Bug 381241] Review Request: ncl - NCAR Command Language and NCAR Graphics

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

Summary: Review Request: ncl -  NCAR Command Language and NCAR Graphics


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 16:02 EST ---
There are some items I don't understand in Site.local. Why is there
#define BuildUdunits FALSE
#define BuildV5D TRUE
#define BuildNetCDF4 TRUE

Also, I haven't verified, but in the README it is said that the 
CcOptions,... can be redefined in Site.local.

Another thing is that it seems to me that it would be better
to set NCARG_ROOT and NCARG_LIB to the right default in 
config/Project or config/Site.local instead of using 
environment variables.



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

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


[Bug 381241] Review Request: ncl - NCAR Command Language and NCAR Graphics

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

Summary: Review Request: ncl -  NCAR Command Language and NCAR Graphics


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 16:18 EST ---
I can do patches to do what I propose.

Also looks like imake is not used, instead they use ymake,
though this may be the same?

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

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


[Bug 386341] Review Request: fxload - A helper program to download firmware into FX and FX2 EZ-USB devices

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

Summary: Review Request: fxload - A helper program to download firmware into FX 
and FX2 EZ-USB devices


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 16:34 EST ---
Nope, thats fine. Just let me know if Cypress changes the license.

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

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


[Bug 291741] Review Request: gnome-hearts - Game of Hearts implementation for gnome

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

Summary: Review Request: gnome-hearts - Game of Hearts implementation for gnome


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 16:29 EST ---
Almost have things ready. Just the matter of certain py modules marked
non-executable yet starting with #!. Apparently they're not meant to be
executed. What should the line be replaced with? There's nothing specific
mentioned in http://fedoraproject.org/wiki/Packaging/Python

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

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


[Bug 386341] Review Request: fxload - A helper program to download firmware into FX and FX2 EZ-USB devices

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

Summary: Review Request: fxload - A helper program to download firmware into FX 
and FX2 EZ-USB devices


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 16:45 EST ---
I have updated the package after more thoroughly reviewing the packaging
guidelines in the Wiki.

Spec URL: http://www.wwwdotorg.org/downloads/fxload/fxload.spec
SRPM URL: 
http://www.wwwdotorg.org/downloads/fxload/fxload-2002_04_11-5.fc8.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, or are watching someone who is.

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


[Bug 282521] Review Request: dblatex - DocBook to LaTeX/ConTeXt Publishing

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

Summary: Review Request: dblatex - DocBook to LaTeX/ConTeXt Publishing


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 16:49 EST ---
(In reply to comment #64)
 Neal, could you please approve all my packagedb requests
 for dblatex and python-which?

I'm sorry, I'm not sure what you're asking me to do.  What are 'packagedb 
requests'?

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

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


[Bug 373621] Review Request: qct - Multi-vcs GUI commit tool

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

Summary: Review Request: qct - Multi-vcs GUI commit tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 16:52 EST ---
New Package CVS Request
===
Package Name: qct
Short Description: Multi-vcs GUI commit tool
Owners: nbecker
Branches: F-7 F-8
InitialCC: [EMAIL PROTECTED]
Cvsextras Commits: yes


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

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


[Bug 388931] New: Review Request: harmony - A utility to program Logitech Harmony universal remote controls.

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




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

   Summary: Review Request: harmony - A utility to program Logitech
Harmony universal remote controls.
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.wwwdotorg.org/downloads/harmony/harmony.spec
SRPM URL: http://www.wwwdotorg.org/downloads/harmony/harmony-0.11-1.fc8.src.rpm

Description:
This software supports programming Logitech Harmony remote controls. While
currently in beta, this software already provides most of the functionality
of the Windows software provided by Logitech, but is much smaller and
cross-platform.

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

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


[Bug 366841] Review Request: NoGravity - 3D 'Wing Commander'-like game

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

Summary: Review Request: NoGravity - 3D 'Wing Commander'-like game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 17:02 EST ---
Interesting game, I'll do a review of this.

First of all all issues mentioned in comment 2 and 3 must be resolved.

For starters please make 2 packages (and with that I mean 2 srpms) which require
each other, one data package and one engine. This way bugfixes to the engine can
be done easily without the user needing redownload all the data. The general
rule of thumb is: If there are seperate upstream tarbals, make seperate srpms.

While making this new packages please also take the other points from comment 2
and 3 into account.

I've just done a testbuild of this on my 64 bit machine, and it is not 64 bit
clean (it crashes). I've already fixed 2 crashes, so no I can get upto the start
of the game (at first it crashed in the menus). I'll try to finish making this
64 bit clean tomorrow and attach a patch.


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

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


[Bug 388931] Review Request: harmony - A utility to program Logitech Harmony universal remote controls.

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

Summary: Review Request: harmony - A utility to program Logitech Harmony 
universal remote controls.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 17:06 EST ---
This would seem to be a duplicate of bug 328161 .

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

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


[Bug 225682] Merge Review: desktop-printing

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

Summary: Merge Review: desktop-printing


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 252133] Review Request: quicksynergy - GTK frontend for synergy

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

Summary: Review Request: quicksynergy - GTK frontend for synergy


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


[EMAIL PROTECTED] 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, or are watching someone who is.

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


[Bug 304301] Review Request: pysol - A Python Solitaire game collection

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

Summary: Review Request: pysol -  A Python Solitaire game collection


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


[EMAIL PROTECTED] 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, or are watching someone who is.

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


[Bug 253781] Review Request: outerspace - client for 4x on-line strategy game

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

Summary: Review Request: outerspace - client for 4x on-line strategy game


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


[EMAIL PROTECTED] 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, or are watching someone who is.

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


[Bug 225290] Merge Review: attr

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

Summary: Merge Review: attr


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 17:09 EST ---
Could someone set fedora-review to '+' if this was approved?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 282521] Review Request: dblatex - DocBook to LaTeX/ConTeXt Publishing

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

Summary: Review Request: dblatex - DocBook to LaTeX/ConTeXt Publishing


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 17:16 EST ---
(In reply to comment #65)
 (In reply to comment #64)
  Neal, could you please approve all my packagedb requests
  for dblatex and python-which?
 
 I'm sorry, I'm not sure what you're asking me to do.  What are 'packagedb 
 requests'?

PackageDB:

https://admin.fedoraproject.org/pkgdb/

is the way you approve co-maintainers, or orphan a package (it replaces the old
owners.list method), you login with your FAS and approve requests e.g.:

https://admin.fedoraproject.org/pkgdb/packages/name/dblatex
https://admin.fedoraproject.org/pkgdb/packages/name/python-which


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

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


[Bug 386531] Review Request: kuftp - Graphical FTP Client for KDE

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

Summary: Review Request: kuftp - Graphical FTP Client for KDE


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





--- Additional Comments From [EMAIL PROTECTED]  2007-11-17 17:22 EST ---
(In reply to comment #3)
 (In reply to comment #2)
  New .spec and srpm files uploadet. 
 
 Please increase the %{release} everytime you make changes to your packet. So 
 the reviewer could follow.
Ok i change it

And of course the url to the new packages then must be posted here.
I delete the old package and new package have the same name. but of course when
i change the release number the srpm url change it to. 

the new srpm + .spec file can be found here

Spec URL: http://yasha.alfahosting.org/kuftp.spec
SRPM URL: http://yasha.alfahosting.org/kuftp-0.9.0-3.fc8.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, or are watching someone who is.

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


  1   2   >