[Bug 223588] New: Review Request: rudeconfig - library for manipulating config files

2007-01-20 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/bugzilla/show_bug.cgi?id=223588

   Summary: Review Request: rudeconfig - library for manipulating
config files
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://www.rudeserver.com/config/download/rudeconfig.spec
SRPM URL: http://www.rudeserver.com/config/download/rudeconfig-5.0.4-1.src.rpm
Description: 

rudeconfig is a C++ library for reading, writing and 
manipulating configuration/.ini files.

Thanks in advance for the review.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223588] Review Request: rudeconfig - C++ library for manipulating config files

2007-01-20 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: rudeconfig - C++ library for manipulating config files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: rudeconfig -|Review Request: rudeconfig -
   |library for manipulating|C++ library for manipulating
   |config files|config files




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223588] Review Request: rudeconfig - C++ library for manipulating config files

2007-01-20 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: rudeconfig - C++ library for manipulating config files


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 04:32 EST ---
Congrats, a pretty clean package!

Disable building the static library and I'll approve it.



Minor issue: The website doesn't mention version 5.0.4 yet. ATM, the tarball
only seems to be accessible when directly accessing it (I could access and
verifiy it)

A note to upstream (seemingly you): Your configure scripts supports building
bz2-compressed tarballs - It might we worth considering to using the bz2'ed
tarball instead of the gziped versions. Spares a couple of bytes on disks/CDs 
etc.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223588] Review Request: rudeconfig - C++ library for manipulating config files

2007-01-20 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: rudeconfig - C++ library for manipulating config files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 07:27 EST ---
MUSTFIX:

- Disable building the static lib (%configure --disable-static)
  and reflect this change where needed.

- Remove commented out duplicate %configure.
  rpm expands comments which causes %configure to be invoked twice.
  If you want to use % in comments you must use %% instead of %.

COSMETICS:

- Minor typo in changelog:
  ...
  Fri Sep 2 2005 .. server.com?
  ...

  This should read ...server.com



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223591] New: Review Request: Magic - A very capable VLSI layout tool

2007-01-20 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/bugzilla/show_bug.cgi?id=223591

   Summary: Review Request: Magic - A very capable VLSI layout tool
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://tux.u-strasbg.fr/~chit/RPMS/magic.spec
SRPM URL: http://tux.u-strasbg.fr/~chit/RPMS/magic-7.4.33-1.src.rpm
Description:
Magic is a venerable VLSI layout tool, written in the 1980's at
Berkeley by John Ousterhout, now famous primarily for writing
the scripting interpreter language Tcl. Due largely in part to
its liberal Berkeley open-source license, magic has remained
popular with universities and small companies.

Magic is widely cited as being the easiest tool to use for
circuit layout, even for people who ultimately rely on commercial
tools for their product design flow.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223591] Review Request: Magic - A very capable VLSI layout tool

2007-01-20 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: Magic - A very capable VLSI layout tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 07:49 EST ---
The following rpmlints can be ignored:
E: magic script-without-shebang /usr/lib/magic/tcl/console.tcl
= Useful for launching magic /usr/lib/magic/tcl/magicexec

W: magic hidden-file-or-dir /usr/lib/magic/sys/.magicrc
= On launching magic, you can see on the tkcon main:
[..]
MOSIS Scalable CMOS Technology for Standard Rules
Processing system .magicrc file
New windows will not have a title caption.
[..]

E: magic-doc only-non-binary-in-usr-lib
W: magic-doc no-documentation
Documentation are at:
/usr/lib/magic/doc

The copyright file is /usr/lib/magic/doc/copyright.ps



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221027] Review Request: LabPlot - Data Analysis and Visualization

2007-01-20 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: LabPlot - Data Analysis and Visualization


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 08:25 EST ---
The original cephes-math library, http://www.moshier.net/cephes-math-38.tar.gz/
From readme file:

   Some software in this archive may be from the book _Methods and
Programs for Mathematical Functions_ (Prentice-Hall, 1989) or
from the Cephes Mathematical Library, a commercial product. In
either event, it is copyrighted by the author.  What you see here
may be used freely but it comes with no support or guarantee.

From the INSTALL file:
Precompiled libraries and programs are supplied here for ix86-linux.

I don't think that the original cephes-math library is compatible with Fedora
policies

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222372] Review Request: tilda - a quake like drop down terminal for GNOME

2007-01-20 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: tilda - a quake like drop down terminal for GNOME


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 08:33 EST ---
Review for
dd8afaafb2657d6cabc43b93a7d408c8  tilda-0.9.4-3.src.rpm

FIX - rpmlint not silent on the SRPM:
W: tilda macro-in-%changelog doc
fix this by adding a second '%' to %doc: %%doc
OK - package and spec named according to the package naming guidelines
OK - package meets packaging guidelines
OK - license is open-source compatible (GPLv2)
OK - license field in spec matches actual license
OK - COPYING included in source and correctly installed in %doc
OK - spec is in American English
OK - spec is legible
OK - source in srpm matches upstream by md5
773d47e3985f7e778b662a38b053c1df
OK - package compiles and build into binaries on Core 6 i386
OK - no known ExcludeArchs
FIX - BuildRequires not correct: gettext is not required. 
FIX - ./configure checks for libXt (see config.log ~182), so you should
BuildRequire libXT-devel
Note - two redundant BRs that could be removed: glib2-devel is required by
pango-devel, gtk2-devel is pulled in by vte-devel
OK - no locales to worry about
OK - no shared libs to worry about
OK - package is not relocatable
OK - package owns all directories it creates (it doesn't create any)
OK - no duplicates in %files
NOTE - I guess tilda.png should also be installed to
%{_datadir}/icons/hicolor/48x48/apps/ for it is size specific
OK - file permissions and %defattr correct
OK - valid clean section
OK - macro usage consistent
OK - code, not content
OK - no large docs
OK - docs don't affect runtime
NOTE - You can remove the NEWS from %doc as long as it only tells you to look at
the README
OK - no header files, static libs or *.pc files
OK - no libtool archives
OK - desktop file included and correctly installed with desktop-file-install
FIX - add '--remove-category=Application' to desktop-file-install as
Application no longer is a valid category according to 
http://standards.freedesktop.org/menu-spec/latest/apa.html
NOTE - you could also add '--copy-name-to-generic-name' so we get a generic name
NOTE - consider adding a comment to the desktop file. As a quick fix use
something like 'echo Comment=A quake like terminal for GNOME  tilda.desktop'
during %prep
OK - package doesn't own directories already owned by other files
OK - package builds in mock (devel)
OK - basically package works as described but there is no drop down or roll up.
Also I see a critical gtk error ( Gtk-CRITICAL **: gtk_window_resize: assertion
`height  0' failed ).

NEEDSWORK

Please fix at least the FIX-issues, before I can approve the package.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222372] Review Request: tilda - a quake like drop down terminal for GNOME

2007-01-20 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: tilda - a quake like drop down terminal for GNOME


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 08:36 EST ---
(In reply to comment #6)
 BuildRequire libXT-devel

Should be libXt-devel, sorry for the typo.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222521] Review Request: IceWM - Lightweight Window Manager.

2007-01-20 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: IceWM -  Lightweight Window Manager.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 08:58 EST ---
(In reply to comment #27)

 Either way, I'm pointing the menu and toolbar files to BlueCurve to reduce the
 dep-chain to redhat-artwork. (Which, AFAIK is installed by default)

That seems right to me.

 * Auto-generate menu:
 
 A simple solution will be to add an entry within menu that points to another
 file say, auto-generated-menu-entry-do-not-edit.
 If the user wants to disable this script he can either:
 A. Delete the entry from his menu file.
 B. Create his own startup script that will over-ride the default one.

That's more or less how it is done for fluxbox. It seems to be right
to me. (in fluxbox the conf is directly copied into the user home
directory, I think that it is preferable to have the default shipped by
fedora in %_datadir/icewm. Also the whole menu is generated, not 
only an entry, but I modified it to have only an entry generated...).

 * A couple of simple question: (More of a user-opinion)
 - I'm thinking about adding xscreensaver to the dep chain.
 Autostart it using startup and add Ctrl+Alt+L to activate it.
(xscreensaver-command)
 Do you see any problem with auto-starting xscreensaver by default?

I don't think this should be done unless it is done upstream.
There could be a comment in startup file, however to indicate 
something along

# to have a screensaver started automatically, install the
# xscreensaver package and uncomment the following line
# xscreensaver 


 - Second, I'm thinking about doing adding menu support to a couple of key
 system-config-xxx applets. (desktop, screensaver, mouse, sound, etc)

No, they should be brought in by pyxdg or similar, if it is not the
case it should be handled at that level, such that it is also right
in fluxbox, pekwm...

 - Third. How about adding xrdb -load ~/.Xdefaults by default?

Isn't that done by session startup scripts?
I have checked, it is done in 
/etc/X11/xinit/xinitrc-common

 * Sat Jan 20 2007 gilboad AT gmail DOT com - 1.2.30-7
 - Fix source1 URL.

It is not completely right, since the file isn't there, if I recall well
there is no .py in the original url



Please remove the dot at the end of the -gnome Summary.

Seems like we're not far from being done. 

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222998] Review Request: iDesk - Desktop icons and background for minimal WMs

2007-01-20 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: iDesk - Desktop icons and background for minimal WMs


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223592] Review Request: wuja - Gnome desktop applet for integration with Google calendar

2007-01-20 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: wuja - Gnome desktop applet for integration with 
Google calendar


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 09:34 EST ---
Also, this is my first contribution for Fedora extras and as such I am in need
of a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219112] Review Request: mmsrip - mms stream recorder

2007-01-20 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: mmsrip - mms stream recorder


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 10:32 EST ---
According to comment #3 MMS support is not acceptable. MMSRip without MMS 
support looks very 
useless to me, so should we close this review with WONTFIX?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219112] Review Request: mmsrip - mms stream recorder

2007-01-20 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: mmsrip - mms stream recorder


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223598] New: Review Request: liborigin - Library for reading OriginLab OPJ project files

2007-01-20 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/bugzilla/show_bug.cgi?id=223598

   Summary: Review Request: liborigin - Library for reading
OriginLab OPJ project files
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://tux.u-strasbg.fr/~chit/RPMS/liborigin.spec
SRPM URL: http://tux.u-strasbg.fr/~chit/RPMS/liborigin-20070115-1.src.rpm
Description:
A library for reading OriginLab OPJ project files.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221027] Review Request: LabPlot - Data Analysis and Visualization

2007-01-20 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: LabPlot - Data Analysis and Visualization


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn|221163  |223598




--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 11:23 EST ---
Looking forward to use liborigin system wide.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221163] Review Request: qwtplot3d - Qt/OpenGL-based C++ library providing a bunch of 3D-widgets for programmers

2007-01-20 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: qwtplot3d - Qt/OpenGL-based C++ library providing a 
bunch of 3D-widgets for programmers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|221027  |
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223598] Review Request: liborigin - Library for reading OriginLab OPJ project files

2007-01-20 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: liborigin - Library for reading OriginLab OPJ project 
files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||221027
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219869] Review Request: perl-Mozilla-LDAP

2007-01-20 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-Mozilla-LDAP
Alias: perldap

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223601] New: Review Request: perl-B-Keywords - Lists of reserved barewords and symbol names

2007-01-20 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/bugzilla/show_bug.cgi?id=223601

   Summary: Review Request: perl-B-Keywords - Lists of reserved
barewords and symbol names
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-B-Keywords.spec

SRPM URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-B-Keywords-1.05-1.src.rpm

Description:
Lists of reserved barewords and symbol names.

Note:
New requirement of Perl::Critic 0.23.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219869] Review Request: perl-Mozilla-LDAP

2007-01-20 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-Mozilla-LDAP
Alias: perldap

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||163779




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 190144] Review Request: hevea - LaTeX to HTML translator

2007-01-20 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: hevea - LaTeX to HTML translator


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 214087] Review Request: libextractor -- Simple library for keyword extraction

2007-01-20 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: libextractor -- Simple library for keyword extraction


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 12:24 EST ---
Closing this bug as NEXTRELEASE

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221349] Review Request: doodle - tool to quickly search the documents on a computer

2007-01-20 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: doodle - tool to quickly search the documents on a 
computer


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


Bug 221349 depends on bug 214087, which changed state.

Bug 214087 Summary: Review Request: libextractor -- Simple library for keyword 
extraction
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214087

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222998] Review Request: iDesk - Desktop icons and background for minimal WMs

2007-01-20 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: iDesk - Desktop icons and background for minimal WMs


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 12:32 EST ---
(In reply to comment #3)
 idesk would only work if the user puts an .idesktoprc in his home. But I 
 don't 
 know how to put this in the spec.

This is something we can not do with rpm. Look at your devilspie package:
Without a custom config the program is pretty much useless, too. 

All we can do is include as much information in %doc as possible. Sample files,
possibly a README.Fedora, but I don't think it's really necessary in this case,
the message  Error: you have to create the .idesktop dir on your HOME!! is
clear enough.


(In reply to comment #1)
 ... The description is taken from the upstream owner. 
 I'll re-phrase it in -2.

Yeah. I suggest removing the note regarding svg support, since it was removed in
0.6.

Review for
e958b9f882c088c15ad7ce5390f54664  idesk-0.7.5-1.src.rpm

OK - rpmlint silent
OK - package and spec named according to the package naming guidelines
FIX - package meets packaging guidelines, but specfile needs some tweaking:
- Group: tag Development/Debuggers is definitely wrong (this is what Sebastian
meant in with strange group in comment #3). I suggest User
Interface/Desktops or User Interface/X. BTW: For comps.xml I suggest X
Window System or Window Managers.
- Change URL: to http://idesk.sourceforge.net, this is more save if upstream
will change their website.
- Change Source0 to
http://dl.sourceforge.net/sourceforge/%{name}/%{name}-%{version}.tar.bz2
instead of heanet mirror.
- insert line wraps after max 79 characters when you rewrite %description
OK - license is open-source compatible (BSD)
FIX - license field in spec file doesn't match actual license: the included
COPYING is a BSD license, but spec file says it's GPL.
OK - COPYING included in source and correctly installed in %doc
OK - spec is in American English
OK - spec is legible
OK - source in srpm matches upstream by md5
beb48c97815c7b085e3b3d601297fbb8
OK - package compiles and build into binaries on Core 6 i386
OK - no known ExcludeArchs
FIX - BuildRequires: should have libSM-devel (./configure checks libICE and 
libSM)
OK - no locales to worry about
OK - no shared libs to worry about
OK - package is not relocatable
OK - package owns all directories it creates
OK - no duplicates in %files
OK - file permissions and %defattr correct
OK - valid clean section
OK - macro usage consistent
OK - code, not content
OK - no large docs
OK - docs don't affect runtime
OK - no header files, static libs or *.pc files
OK - no libtool archives
OK - no desktop file needed
OK - package doesn't own directories already owned by other files
OK - builds in mock (devel)
OK - works as described

NEEDSWORK

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222521] Review Request: IceWM - Lightweight Window Manager.

2007-01-20 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: IceWM -  Lightweight Window Manager.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 12:39 EST ---
OK.
All the bugs fixed.

Time to introduce new ones.
I broke the icewm-xdg-menu into a new sub-package and created a bash script that
uses its output to generate static menu files. (And locate the best matching 
icons)
The script itself seems to work just fine on two of my machines (Minus some wine
problems) - it should be stable enough for wide-testing.
For now, it requires manual activation. (icewm-generate-menu
_default_icon_theme_name_)

* Sat Jan 20 2007 gilboad AT gmail DOT com - 1.2.30-8
- Fix source0 URL. (2nd is a winner)
- Fix -gnome summery.
- New sub-package: icewm-xdg-menu
- ALPHA: icewm-generate-menu script added to use icewm-xdg-menu to generate
static menus.

Spec URL: http://gilboadavara.thecodergeek.com/icewm.spec
SRPM URL: http://gilboadavara.thecodergeek.com/icewm-1.2.30-8.src.rpm
Script URL: http://gilboadavara.thecodergeek.com/icewm-generate-menu


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222521] Review Request: IceWM - Lightweight Window Manager.

2007-01-20 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: IceWM -  Lightweight Window Manager.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 12:42 EST ---
Oh... I didn't test it with xinit, but when I start icewm from gdm, .Xdefaults
doesn't get loaded automatically.

- Gilboa

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 218256] Review Request: audacious-itouch - iTouch keyboard control plugin for the Audacious media player

2007-01-20 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: audacious-itouch - iTouch keyboard control plugin for 
the Audacious media player


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 12:58 EST ---
I still think that audacious-itouch is a valid name for this package, since this
is the upstream name. Nevertheless - after reading the review request for
audacious-plugin-fc (bug #222648) and thinking about this more deeply, I like
the idea of getting this straight. This would also include renaming
audacious-docklet.

Yu, what you think?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222998] Review Request: iDesk - Desktop icons and background for minimal WMs

2007-01-20 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: iDesk - Desktop icons and background for minimal WMs


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 13:09 EST ---
- Group:
My bad. I used an old spec file and forgot to clean it up.

- License:
Weird! Owner [1] claims that iDesk is GPL.
He must have switched license and forgot to update the COPYING file.
Any ideas what to do?

[1] http://idesk.sourceforge.net/wiki/index.php/Idesk-License

Either way:

* Sat Jan 20 2007 gilboad AT gmail DOT com - 0.7.5-2
- Fix description.
- Fix group.
- Missing BR: libSM-devel.

Spec URL: http://gilboadavara.thecodergeek.com/idesk.spec
SRPM URL: http://gilboadavara.thecodergeek.com/idesk-0.7.5-2.src.rpm

- Gilboa

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222998] Review Request: iDesk - Desktop icons and background for minimal WMs

2007-01-20 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: iDesk - Desktop icons and background for minimal WMs


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 13:25 EST ---
(In reply to comment #7)
 - License:
 Weird! Owner [1] claims that iDesk is GPL.
 He must have switched license and forgot to update the COPYING file.
 Any ideas what to do?
 
 [1] http://idesk.sourceforge.net/wiki/index.php/Idesk-License

COPYING is definitely BSD, so are the headers. Take a look at Main.cpp for 
example.

If this is really GPL, you should include a copy of it as Source1 and install it
instead of COPYING. Please ask upstream first.

 Either way:
 
 * Sat Jan 20 2007 gilboad AT gmail DOT com - 0.7.5-2
 - Fix description.
 - Fix group.
 - Missing BR: libSM-devel.

Looks good so far, if the license issue is clear, I  will approve the package.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223592] Review Request: wuja - Gnome desktop applet for integration with Google calendar

2007-01-20 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: wuja - Gnome desktop applet for integration with 
Google calendar


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 13:33 EST ---
% rpmlint wuja-0.0.5-1.src.rpm 
% rpmlint wuja-0.0.5-1.noarch.rpm 
W: wuja non-conffile-in-etc /etc/gconf/schemas/wuja.schema
% 

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222998] Review Request: iDesk - Desktop icons and background for minimal WMs

2007-01-20 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: iDesk - Desktop icons and background for minimal WMs


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 14:11 EST ---
OK. Thanks. I'll contact the author and report back.
Other then that, iDesk/x86_64 dies a horrible death during startup - making it
i386 only for now.

- Gilboa


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222998] Review Request: iDesk - Desktop icons and background for minimal WMs

2007-01-20 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: iDesk - Desktop icons and background for minimal WMs


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 14:17 EST ---
Forgot to add:

Source0:
At least in my case
http://dl.sourceforge.net/sourceforge/%{name}/%{name}-%{version}.tar.bz2 doesn't
work but the heatnet mirror (or any other mirror for that mater) does work.
I can change it back to dl.sourceforge, but IMHO, it is less useful.

- Gilboa


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221906] Review Request: gmediaserver - UPnP compatible media server for the GNU system

2007-01-20 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: gmediaserver - UPnP compatible media server for the 
GNU system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 15:02 EST ---
I fixed it, but I must fix problems with info (%post and %preun sections) before
I publish second release. I think tommorow I will upload new release.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 204495] Review Request: perl-GStreamer - GStreamer Perl module

2007-01-20 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-GStreamer - GStreamer Perl module


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 15:08 EST ---
(In reply to comment #10)
 Trying to rebuild on devel the tests failed:
 
 t/GstRegistry...ok 1/21undef is not of type GStreamer::PluginFeature 
 at
 t/GstRegistry.t line 22.

This test the presence of the alsasink from the alsa plugin

 gstreamer-0.10.11-1.fc7
 gstreamer-devel-0.10.11-1.fc7

and the alsa plugin is in gstreamer-plugins-base...

Maybe the tests shouldn't test specific plugins, or maybe GStreamer should
depend on gstreamer-plugins-base, as I'm not sure the gstreamer package by
itself is very useful.



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2007-01-20 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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 15:52 EST ---
Spec URL: http://tkmame.retrogames.com/fedora-extras/poker-network.spec
SRPM URL: 
http://tkmame.retrogames.com/fedora-extras/poker-network-1.0.33-8.src.rpm

%changelog
* Sat Jan 20 2007 Christopher Stone [EMAIL PROTECTED] 1.0.33-8
- Readd poker-client-lib package to this spec file
- Readd %%{release} to packages that require poker-client-lib
- Remove manual removal of files since they are all in poker-client-lib
- Update comments on rpmlint errors

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222612] Review Request: poker2d - GTK poker client to play on a poker-network server

2007-01-20 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: poker2d - GTK poker client to play on a poker-network 
server
Alias: poker2d

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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 15:53 EST ---
Spec URL: http://tkmame.retrogames.com/fedora-extras/poker2d.spec
SRPM URL: http://tkmame.retrogames.com/fedora-extras/poker2d-1.0.33-4.src.rpm

%changelog
* Sat Jan 20 2007 Christopher Stone [EMAIL PROTECTED] 1.0.33-4
- Move %%{_libexecdir}/poker-interface into a common subpackage
- Move locales to poker2d-common
- Move poker-client-lib back into poker-network package
- Remove Version from desktop file
- Fix up Requires and BuildRequires
- Fix descriptions for sub packages


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223618] New: Review Request: obexftp - Tool to access devices via the OBEX protocol

2007-01-20 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/bugzilla/show_bug.cgi?id=223618

   Summary: Review Request: obexftp -  Tool to access devices via
the OBEX protocol
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://rpm.greysector.net/extras/obexftp.spec
SRPM URL: http://rpm.greysector.net/extras/obexftp-0.20-1.src.rpm
Description:
The overall goal of this project is to make mobile devices featuring the OBEX
protocol and adhering to the OBEX FTP standard accessible by an open source
implementation. The common usage for ObexFTP is to access your mobile phones
memory to store and retrieve e.g. your phonebook, logos, ringtones, music,
pictures and alike.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 218256] Review Request: audacious-itouch - iTouch keyboard control plugin for the Audacious media player

2007-01-20 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: audacious-itouch - iTouch keyboard control plugin for 
the Audacious media player


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 16:33 EST ---
To me it seems best to ask the packaging committee for a recomandation. Right
now it seems that package-pluginname, package-plugin-pluginname,
package-plugins-pluginname and package-pluginname-plugin for
different packages. I would prefer package-plugins-pluginname or
package-plugin-pluginname, this would make it easier to install all
plugins, e.g. with yum install audacious-plugins-\* while audacious-\* also
matches the devel package.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223618] Review Request: obexftp - Tool to access devices via the OBEX protocol

2007-01-20 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: obexftp -  Tool to access devices via the OBEX protocol


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
OtherBugsDependingO|163776  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 16:42 EST ---
Good:

- rpmlint checks return:
W: obexftp-python no-documentation
W: obexftp-perl no-documentation
W: obexftp-devel no-documentation

All safe to ignore.

- package meets naming guidelines
- package meets packaging guidelines
- license (GPL) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file
- devel package ok
- no .la files
- post/postun ldconfig ok
- devel requires base package n-v-r 

Looks good, APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223618] Review Request: obexftp - Tool to access devices via the OBEX protocol

2007-01-20 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: obexftp -  Tool to access devices via the OBEX protocol


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223618] Review Request: obexftp - Tool to access devices via the OBEX protocol

2007-01-20 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: obexftp -  Tool to access devices via the OBEX protocol


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 17:06 EST ---
%defattr missing in -perl package

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221405] Review Request: libdirac - Dirac is an open source video codec

2007-01-20 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: libdirac - Dirac is an open source video codec


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 17:23 EST ---
Per
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-cfd71146dbb6f00cec9fe3623ea619f843394837
the release field should be:
Release:5.%{cvs}cvs%{?dist}

Why is the testsuite disabled?

I don't like /usr/bin/ binaries in a package called libwhatever. Moreover, if
the source tarball is called dirac, why is the main package called libdirac?
Please put the binaries in a package named dirac

There are two redundant BuildRequires: libtool requires automake and automake
requires autoconf.

Builds fine in mock/x86_64/devel.

Shouldn't package docs be in %{_datadir}/doc/dirac-docs-%{version} ?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223618] Review Request: obexftp - Tool to access devices via the OBEX protocol

2007-01-20 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: obexftp -  Tool to access devices via the OBEX protocol


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 17:41 EST ---
Requires: perl(:MODULE_COMPAT_...) missing as well.

Subpackages should probably be called python-obexftp and perl-obexftp instead of
obexftp-* according to package naming guidelines.

_obexftp.so.X and _obexftp.so.X.X for a Python extension are probably just
cruft, and could be replaced with a plain _obexftp.so.

%configure --disable-dependency-tracking would result in cleaner build output
and possibly speed up the build.

The minimum openobex version this works with is 1.2, making the build dep on
openobex-devel versioned (= 1.2) would save some trouble from people using
older distros.

Requires: openobex-devel = 1.2 missing from -devel, see #includes in installed
header files.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223627] New: Review Request: system-switch-java - Java toolset switcher

2007-01-20 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/bugzilla/show_bug.cgi?id=223627

   Summary: Review Request: system-switch-java - Java toolset
switcher
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://fitzsim.org/packages/system-switch-java.spec
SRPM URL: http://fitzsim.org/packages/system-switch-java-1.0.0-1.src.rpm
Description: system-switch-java provides an easy way to select the default Java
toolset for the system.  It shells out to /usr/sbin/alternatives to
set the java, javac and related alternatives.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222742] Review Request: fuse-smb - FUSE-Filesystem to fast and easy access remote resources via SMB

2007-01-20 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: fuse-smb - FUSE-Filesystem to fast and easy access 
remote resources via SMB


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 18:15 EST ---
Sorry that you had to wait.
Rpmlint now shows:
W: fuse-smb mixed-use-of-spaces-and-tabs (spaces: line 16, tab: line 1)
but it's not a blocker.
However, I'll make it FE-ACCEPTed once you put isomaster into repo.

But it's now APPORVED in any case.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223557] Review Request: mdbtools - tools for extracting things from Access databases

2007-01-20 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: mdbtools - tools for extracting things from Access 
databases


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223598] Review Request: liborigin - Library for reading OriginLab OPJ project files

2007-01-20 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: liborigin - Library for reading OriginLab OPJ project 
files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 18:53 EST ---
Hi,

This package (its Makefile actually) needs to be patched up for rpmbuild to work
properly on x86_64. It currently hard-codes the library path to /usr/lib.
Also, can you make it use the rpm's %optflags for the CFLAGS?

An Aside: What a version number ;), can't upstream use sth simple as 1.0  

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 177828] Review Request: python-libtidy - Python bindings for libtidy

2007-01-20 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-libtidy - Python bindings for libtidy


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 19:12 EST ---
It has been more than two months with no comments or updates on this bug. 

Konstantin: Are you still interested in submitting this package? Please let us
know. One more week of silence on the bug and we'll have to mark it as a dead
review.

Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223633] New: Review Request: csync2 - A cluster synchronization tool

2007-01-20 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/bugzilla/show_bug.cgi?id=223633

   Summary: Review Request: csync2 - A cluster synchronization tool
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://rubenkerkhof.com/packages/csync2.spec
SRPM URL: http://rubenkerkhof.com/packages/csync2-1.33-1.src.rpm
Description:
Csync2 is a cluster synchronization tool. It can be used to keep files on 
multiple hosts in a cluster in sync. Csync2 can handle complex setups with 
much more than just 2 hosts, handle file deletions and can detect conflicts.
It is expedient for HA-clusters, HPC-clusters, COWs and server farms.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 174289] LibHdate is a small C, C++ library for Hebrew calendar and dates, holidays, and reading sequence.

2007-01-20 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: LibHdate is a small C,C++ library for Hebrew calendar and dates, 
holidays, and reading sequence.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 167147] Review Request: aqsis - 3D Rendering system

2007-01-20 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: aqsis - 3D Rendering system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 20:38 EST ---
Umm...
Any need of using svn version tarball? If there is any strong
reason to use svn version, please tell me the reason and
write where I can get svn source.
Otherwise please use the tarball which are available from
some URL until this review is completed.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 167147] Review Request: aqsis - 3D Rendering system

2007-01-20 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: aqsis - 3D Rendering system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 20:46 EST ---
A seriouse bug was fixed, so I choose to use a SVN checkout instead of the 
tarball. As I said a new tarball 
will soon be released. Of course you can get the SVN from 
sourceforge.net/projects/aqsis .

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2007-01-20 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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 21:19 EST ---
I'm having trouble running the poker-server due to a bad mysql password.  Here's
what I've done:

 1. On a fresh box, install Rawhide with the latest updates
 2. Install mysql-server, as well as the poker-server dependencies
 3. Leave the settings in /etc/poker-network/poker.server.xml as the defaults
 3. 'service mysqld start'
 5. 'service poker-server start'
 6. Look in /var/log/poker-server.log

Expected results:

Some sort of 'poker server started' success message

Observed results:

The attached stack trace

I tried using a shorter pokernetwork password in the mysql server, as well as an
empty password, but I get the same results.  I've also tried connecting to the
mysql server as the pokernetwork user, using the password in poker.server.xml,
but that fails as well.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2007-01-20 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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 21:20 EST ---
Created an attachment (id=146068)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=146068action=view)
service startup failure log


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221405] Review Request: libdirac - Dirac is an open source video codec

2007-01-20 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: libdirac - Dirac is an open source video codec


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-20 21:35 EST ---
SRPM:
http://kwizart.free.fr/fedora/6/testing/dirac/dirac-0.6.0-6.20070108cvs.kwizart.fc6.src.rpm
SPEC:
http://kwizart.free.fr/fedora/6/testing/dirac/dirac.spec
Description: Dirac is an open source video codec 

Test-suite is not provided with this snapshoot scheme (done from remark in #6 )
same problem with qt4-encodeur gui (anyway i'm not sure we need an encoder gui
for each codec !?). if encoder_gui and test-suite are really needed the best way
to do this would be to make another snaphoot and a tarball with only theses two
wanted parts!

Encoding seems to work fine but isn't useable since playbak need to be tested...



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 219972] Review Request: poker-network - A poker server, client and abstract user interface library

2007-01-20 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: poker-network - A poker server, client and abstract 
user interface library
Alias: poker-network

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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-21 01:24 EST ---
Two other issues that I've discovered while trying to run the server:

- Attempting to create a new user account through the poker-web interface fails.
 The web interface returns a cryptic string type = 121, code = 1, message =,
and attempts to log in using the new account fail.

- If selinux is enabled, then the web interface can't communicate with the
poker-server backend.  From the avc denial message, this appears to be because
the web interface wants to establish a TCP connection to the poker server (port
19382), which is disallowed by the standard httpd policy.

Jan 20 22:20:52 localhost kernel: audit(1169360452.415:4): avc:  denied  {
name_connect } for  pid=2460 comm=httpd dest=19382
scontext=user_u:system_r:httpd_t:s0 tcontext=system_u:object_r:port_t:s0
tclass=tcp_socket


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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