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

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-15 02:39 EST ---
(In reply to comment #26)
> (In reply to comment #25)
> 
> > > Before it fails, there is a big warning:
> > > *** Warning: Linking the shared library libsnmpPlugin.la against the
> > > *** static library
> > > /usr/lib/perl5/5.8.8/i386-linux-thread-multi/auto/DynaLoader/DynaLoader.a
is not
> > > portable!
> > 
> > I don't believe that this is relevent as we nuke libtool archives anyway.
> Only partially correct. 

Understood, but I'd like to point out that if you google search this error,
you'll find that this is not the only package that suffers from this.  As one
site puts it - it's a result of perl's crappy dynamic linking/loading 
facilities.

In fact, FE's own xchat suffers from it.  Here is a recent build log of xchat in
the build system:
http://buildsys.fedoraproject.org/logs/fedora-development-extras/23222-xchat-gnome-0.15-2.fc7/i386/build.log
You can find an almost identical warning there.

I'm not saying it's ok, just probably not the end of the world.

-- 
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 219581] Review Request: gourmet - Recipe organization app

2006-12-14 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: gourmet - Recipe organization app


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request:  -|




-- 
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 219581] Review Request: -

2006-12-14 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:  - 


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |
  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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-15 02:21 EST ---
(In reply to comment #25)

> > Before it fails, there is a big warning:
> > *** Warning: Linking the shared library libsnmpPlugin.la against the
> > *** static library
> > /usr/lib/perl5/5.8.8/i386-linux-thread-multi/auto/DynaLoader/DynaLoader.a 
> > is not
> > portable!
> 
> I don't believe that this is relevent as we nuke libtool archives anyway.
Only partially correct. 

This is libtool warning you about this package doing something non-portable:
Linking a shared library against a static.

This warning is largely irrelevant when building on Linux, because linking a
shared against a static library basically (i.e. in most cases) is possible on 
Linux.

Whether this warning is related to the breakdown is a different question, and
should be analyzed. It's rather unlikely, but it could be an indication of ntop
not being ready for dynamic linkage.

All removing the *.la would do, is to silence the warning, it would not change
anything about ntop potentially doing something "naughty".



-- 
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 219653] Review Request: solfege - music education software

2006-12-14 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: solfege - music education software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-15 01:35 EST ---
Got rid of all the rpmlint errors and warnings.

updated:

Spec URL: http://folk.ntnu.no/sindrb/packages/solfege.spec
SRPM URL: http://folk.ntnu.no/sindrb/packages/solfege-3.6.4-4.src.rpm

-- 
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 216130] Review Request: glest - 3D real time strategy game

2006-12-14 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: glest - 3D real time strategy game
Alias: glest

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-15 01:19 EST ---
It plays real choppy on my machine. The audio stutters and the mouse is very 
slow. 
I have FC6 installed, 32bit on an Athlon 64 3200+ with two gigs of memory. I
have an nVidia 5200 video card with 256megs of memory. Ric

-- 
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 219751] New: Review Request: python-TurboMail - Multi-threaded mail queue manager for TurboGears applications.

2006-12-14 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=219751

   Summary: Review Request: python-TurboMail - Multi-threaded mail
queue manager for TurboGears applications.
   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://people.redhat.com/lmacken/SPECS/python-TurboMail.spec
SRPM URL: 
http://people.redhat.com/lmacken/SRPMS/python-TurboMail-2.0.3-1.src.rpm
Description:
TurboMail is a TurboGears extension, meaning that it starts up and shuts down
alongside any TurboGears applications you write, in the same way that visit
tracking and identity do. TurboMail uses built-in Python modules for SMTP
communication and MIME e-mail creation, but greatly simplifies these tasks by
performing the grunt-work for you. Additionally, TurboMail is multi-threaded,
allowing for single or batch enqueueing and background delivery of mail.

-- 
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 189195] Review Request: horde - php application framework

2006-12-14 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: horde - php application framework


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 23:24 EST ---
Yes thanks Greg but if you read that paragraph in context we don't WANT them to
enable wider access until after they've been to the horde configuration web site
and set up their own Authentication backend.  Since horde comes predefined to
auto-login every web session as Administrator, we want them to go to the site,
set up something else, THEN and only THEN let other IPs hit the site.  Otherwise
they would be opening their horde site for lots of other people to play
Administrator.

If you think it's important I tell users that they need to graceful apache after
editing the conf file, that's not a big deal, but that doesn't preclude this
RPM's need to re[load|start]ing apache in %post.

Now, if you don't mind, I'd like to give Jason a chance to do a real review on
this package and give his feedback.  This review has been open for 8 months...

-- 
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 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility

2006-12-14 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: beryl-vidcap - Beryl OpenGL window and compositing 
manager video capture utility


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 23:06 EST ---
Created an attachment (id=143732)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=143732&action=view)
Mock build log of beryl-vidcap 0.1.3-1 on FC-devel i386

Well, I am working on FC-devel i386 and
on this environ rpmlint surely complains about
beryl-vidcap 0.1.3-1.

I don't know about yasm, however I suspect this does
something because mockbuild log says:


libtool --tag=CC --mode=compile yasm -f elf -m x86 -o src/arch/x86/frame.lo
-prefer-non-pic src/arch/x86/frame.asm
mkdir src/arch/x86/.libs
 yasm -f elf -m x86 src/arch/x86/frame.asm -o src/arch/x86/.libs/frame.o
 yasm -f elf -m x86 src/arch/x86/frame.asm -o src/arch/x86/frame.o >/dev/null
2>&1
libtool --tag=CC --mode=link gcc -Wl,--as-needed -rpath /usr/lib -o libseom.la
src/buffer.lo src/client.lo src/codec.lo src/frame.lo src/server.lo
src/stream.lo src/arch/x86/frame.lo -ldl -lpthread
---


-- 
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 219164] Review Request: fontypython - TTF font manager

2006-12-14 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: fontypython - TTF font manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 22:56 EST ---
Well, okay.

-
  This package (fontypython) is APPROVED by me.
-

Please go forward according to 

http://fedoraproject.org/wiki/Extras/Contributors

to import this package to Fedora Extras. I will sponsor
you when you have taken steps partway written in the page above
(then I should receive the mail that you need a sponsor)

--
Note:
This package cannot be rebuilt on FE-devel 
until wxPython is rebuilt against python 2.5.
It seems that the maintainer of wxPython (on FE) wants
to upgrade it to 2.8

-- 
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 189188] Review Request: sqlgrey - postfix grey-listing policy service

2006-12-14 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: sqlgrey - postfix grey-listing policy service


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 22:27 EST ---
Looks all good to me. All the blockers I saw have been fixed... so this package
is APPROVED. 

Don't forget to close this package NEXTRELEASE once it's been imported and 
built. 

Do consider reviewing another waiting package to help spread out the reviewing
load. 

-- 
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 219653] Review Request: solfege - music education software

2006-12-14 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: solfege - music education software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 22:15 EST ---
Again just a comment:

W: solfege strange-permission solfege.sh.in 0755
   - permission should be 0644
W: solfege mixed-use-of-spaces-and-tabs (spaces: line 6, tab: line 4)
   - Indentation in spec file should be spaces _or_ tabs and using
 _both_ is not considered good for cosmetic annoyance.

W: solfege unstripped-binary-or-object
/usr/lib/debug/usr/lib/solfege/_solfege_c_midi.so.debug
E: solfege shared-lib-without-dependency-information
/usr/lib/debug/usr/lib/solfege/_solfege_c_midi.so.debug
   - This is because normal binary rpm contain debug information
 binary.
 You should not write:
-
%files
...   
%{_libdir}/*
--
 because this includes %{_libdir}/debug, this is wrong.

-- 
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 219164] Review Request: fontypython - TTF font manager

2006-12-14 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: fontypython - TTF font manager


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 21:44 EST ---
I've pre-reviewed #218678.

-- 
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 217497] Review Request: dbmail - The DBMail mail storage system

2006-12-14 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: dbmail - The DBMail mail storage system


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 21:18 EST ---
Spec URL: http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/dbmail.spec
SRPM URL:
http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/dbmail-2.2.1-5.fc6.src.rpm

* Thu Dec 14 2006 Bernard Johnson <[EMAIL PROTECTED]> 2.2.1-5
- fix my local svn that caused x bit on init files to sneak in


My local svn caused the init files to be executable, in the srpm, which they
shouldn't have been.  Other than that, everything is the same as the previous
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 218678] Review Request: pybluez - python API for the bluez bluetooth stack

2006-12-14 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: pybluez - python API for the bluez bluetooth stack


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 21:12 EST ---
This is my first review.  I cannot officially review the package - this is just
a "pre-review".

The SRPM passes rpmlint.

It builds in mock fc6-i386, which is a good sign.

I notice a couple of issues with the spec file, however:
* You need to remove the appropriate line (and the comment) from the 
  beginning of the spec file.  At appears that this is an arch-specific(?)
  package, so you need to remove the '%{!?python_sitelib' line.

* I think you should remove the comment from the Build section, and
  also the one from above '%{python_sitearch}/*'

According to
http://fedoraproject.org/wiki/Packaging/Guidelines#head-b7d622f4bb245300199c6a33128acce5fb453213:
* The changelog items should begin with a dash.





-- 
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 209112] Review Request: gspca - v4l2 kernel module driver for webcams

2006-12-14 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: gspca - v4l2 kernel module driver for webcams


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 21:02 EST ---
Thanks very much to FESCO members for approving this. Before proceeding further
on this review i think i should wait for FE-KMOD-APPROVED set on this bug.

-- 
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 219164] Review Request: fontypython - TTF font manager

2006-12-14 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: fontypython - TTF font manager


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 20:14 EST ---
Spec URL: http://cr33.is-a-geek.com/RPMS/fontypython.spec
SRPM URL: http://cr33.is-a-geek.com/RPMS/fontypython-0.2.0-4.src.rpm

%changelog
- Corrected Source0 to include URL
- removed python >= 2.4 from Requires

OK, hopefully this latest package is now up to par.

I will try (pre)reviewing some other requests.  In the meanwhile, if someone
else wants to adopt this package I will not be offended.

-- 
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 219653] Review Request: solfege - music education software

2006-12-14 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: solfege - music education software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 19:58 EST ---
Tried to fix the permissions. There are still problems, solutions welcome.

SRPM: 
W: solfege strange-permission solfege.sh.in 0755
W: solfege mixed-use-of-spaces-and-tabs (spaces: line 6, tab: line 4)

RPM:
W: solfege unstripped-binary-or-object
/usr/lib/debug/usr/lib/solfege/_solfege_c_midi.so.debug
E: solfege shared-lib-without-dependency-information
/usr/lib/debug/usr/lib/solfege/_solfege_c_midi.so.debug

Updated: 

Spec URL: http://folk.ntnu.no/sindrb/packages/solfege.spec
SRPM URL: http://folk.ntnu.no/sindrb/packages/solfege-3.6.4-3.src.rpm

-- 
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 217806] Review Request: tktreectrl - flexible listbox widget for Tk

2006-12-14 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: tktreectrl - flexible listbox widget for Tk


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 19:32 EST ---
Update to 2.2.1:

http://www.kobold.org/~wart/fedora/tktreectrl.spec
http://www.kobold.org/~wart/fedora/tktreectrl-2.2.1-1.src.rpm

-- 
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 219732] New: Review Request: ruby-fam - Gamin/FAM bindings for Ruby

2006-12-14 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=219732

   Summary: Review Request: ruby-fam - Gamin/FAM bindings for Ruby
   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://people.redhat.com/dlutter/yum/spec/ruby-fam.spec
SRPM URL: http://people.redhat.com/dlutter/yum/SRPMS/ruby-fam-0.2.0-1.src.rpm
Description: 
FAM-Ruby is a Ruby interface to SGI's File Alteration Monitor
(http://oss.sgi.com/projects/fam/).  FAM allows you to monitor files and
directories for changes (file modification, creation, and removal) -- in
an event-driven manner.

-- 
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 207473] Review Request: ruby-activerecord - Implements the ActiveRecord pattern for ORM

2006-12-14 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-activerecord - Implements the ActiveRecord 
pattern for ORM


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 18:48 EST ---
Imported and successfully built as plague job 23782

-- 
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 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-14 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: conky - A system monitor for X originally based on the 
torsmo code


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|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 219610] Review Request: uncrustify-0.30 - code beautifier

2006-12-14 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: uncrustify-0.30 - code beautifier


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 17:23 EST ---
Hi Neil, here are some observations and I'll post a more thorough review 
as soon as I get a chance:

good:
 + source matches upstream
 + builds in mock on FC6 i386

needswork:
 - rpmlint reports:
 W: uncrustify summary-not-capitalized reformat source
 W: uncrustify invalid-license gpl

-- 
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 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility

2006-12-14 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: beryl-vidcap - Beryl OpenGL window and compositing 
manager video capture utility


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 17:22 EST ---
It could be.. the .rodata section on top of frame.asm could cause this, but I'm
not an assembler expert so I can't tell for sure. but I could convert the
[yuv]Mul variables to 'immediates' (or how they are called), simply use the
values directly in the code instead of referencing the variables. Just need to
figure out how to do that correctly WRT endianess :)

-- 
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 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility

2006-12-14 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: beryl-vidcap - Beryl OpenGL window and compositing 
manager video capture utility


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 17:08 EST ---
Ugh. I get no such errors (no rpmlint error and no ldconfig spew) at all on my
system, but its x86_64. I have a feeling this is an issue with the x86 asm...

-- 
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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 16:51 EST ---
(In reply to comment #19)
> There is a missing BR on net-snmp-devel. After net-snmp-devel is 
> installed, build fails with:
> /usr/bin/ld: cannot find -lsensors

You'll need to put lm_sensors-devel in your BR as well.  I'll fix this in the
next version.

> Before it fails, there is a big warning:
> *** Warning: Linking the shared library libsnmpPlugin.la against the
> *** static library
> /usr/lib/perl5/5.8.8/i386-linux-thread-multi/auto/DynaLoader/DynaLoader.a is 
> not
> portable!

I don't believe that this is relevent as we nuke libtool archives anyway.

> Some warnings that may be worrisome:
I'll try to look into some of these, but don't have time for a full code 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 189195] Review Request: horde - php application framework

2006-12-14 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: horde - php application framework


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 16:30 EST ---
or better:
"By default, Horde is only configured to be accessible only from localhost.  It 
is unlikely that this is sufficient.  To enable wider access and apply the 
configuration, edit /etc/httpd/conf.d/%{name}.conf and then issue the 
command 'service httpd grafeful'"


-- 
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 189195] Review Request: horde - php application framework

2006-12-14 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: horde - php application framework


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 16:15 EST ---
> at least HUPing apache is required in order for it to read the new conf
> entries 

Yes, but people are going to manually edit that to remove the 127.0.0.1 line 
and switch the deny/allow, etc, so really it only needs to be done after you 
edit that file, not on every minor update from 3.1.3 to 3.1.4, etc (that file 
is marked noreplace).  

How about just changeing the README.fedora bit to say:
"By default, Horde is only accessible from localhost. To enable wider access, 
edit:
/etc/httpd/conf.d/%{name}.conf and then issue the command 'service httpd 
grafeful'"



-- 
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 216103] Review Request: python-twisted-names - A Twisted DNS implementation

2006-12-14 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-twisted-names - A Twisted DNS implementation


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 16:09 EST ---
http://thomas.apestaart.org/download/pkg/fedora-6-x86_64-extras/python-twisted-names-0.3.0-3.fc6/

incorporated suggested changes


-- 
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 195645] Review Request: rasqal - RDF query library

2006-12-14 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: rasqal - RDF query library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 15:55 EST ---
http://thomas.apestaart.org/download/pkg/fedora-6-x86_64-extras/rasqal-0.9.12-4.fc6/

-- 
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 189195] Review Request: horde - php application framework

2006-12-14 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: horde - php application framework


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 15:41 EST ---
Perhaps "service httpd status |grep running && service httpd graceful"?

-- 
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 189195] Review Request: horde - php application framework

2006-12-14 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: horde - php application framework


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 15:35 EST ---
While I agree that a full restart is unnecessary, at least HUPing apache is
required in order for it to read the new conf entries (Aliases, VirtualHosts,
whatever the case may be).  Since we only want to signal httpd if it's already
running, but can't guarantee it's running on every end user's box, we have to
use condrestart.  An ideal solution would be a 'condreload' command in apache's
init script.  Would the benefits of reload vs. restart warrant filing an RFE
against httpd in Core?  What other concerns would need to be considered?

-- 
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 189195] Review Request: horde - php application framework

2006-12-14 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: horde - php application framework


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 15:06 EST ---
> for example the moin rpm in Extras doesn't.

Sorry, that was a dumb example as it doesn't put anything in conf.d.  
phpMyAdmin is a better example and it does do condrestart so I guess that's 
right for Fedora Extras (although not my preference).

-- 
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 189188] Review Request: sqlgrey - postfix grey-listing policy service

2006-12-14 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: sqlgrey - postfix grey-listing policy service


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 15:06 EST ---
That was easy...

http://ftp.kspei.com/pub/steve/rpms/sqlgrey-1.7.4-5.src.rpm

-- 
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 189188] Review Request: sqlgrey - postfix grey-listing policy service

2006-12-14 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: sqlgrey - postfix grey-listing policy service


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 14:26 EST ---
After using this for an entire day (finally), the only issue that I've noticed 
is the init script clobbering its own output.

I'll try to get -5 together later today with a fix for that.

-- 
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 209112] Review Request: gspca - v4l2 kernel module driver for webcams

2006-12-14 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: gspca - v4l2 kernel module driver for webcams


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 14:09 EST ---
This was discussed in the FESCo meeting today.  The largest concern is the need
to get kmods in the upstream kernel (all kmods, not just this one).  After
further review and a few testimonials from some developers, we have decided to
approve this kmod for the period of one year.  We'll revisit this after that and
see what the status is.

Thank you for your submission, and good luck with your packaging!

-- 
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 189195] Review Request: horde - php application framework

2006-12-14 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: horde - php application framework


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 14:01 EST ---
Should you be restarting httpd in post and postun?  I can't find a guideline on 
that on the Fedora wiki, but for example the moin rpm in Extras doesn't.

Also http://wiki.horde.org/SecurityTips has a good example of apache 
configuration.  I think some improvements can be made with the addition of the 
two sections:
# Deny access to files that are not served directly by the webserver
# Deny access to the test.php files except from localhost

-- 
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 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-14 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 13:44 EST ---
Created an attachment (id=143674)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=143674&action=view)
conky-1.4.5 patch for fedora (signature)

My signature for the patch.  My public key:

http://home.comcast.net/~147F2970/pubkey.asc

Philip

-- 
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 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-14 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 13:36 EST ---
Created an attachment (id=143672)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=143672&action=view)
conky-1.4.5 patch for fedora

Attached is a patch that will work against the conky 1.4.5 tarball.

The svn keyword $Id$ was the problem.  Working copies have expanded 
keywords, but the svn repository does not.

tar -xjf conky-1.4.5.tar.bz2
cd conky-1.4.5
patch -p0 < patchfile

That should work cleanly.

-- 
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 219653] Review Request: solfege - music education software

2006-12-14 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: solfege - music education software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 13:25 EST ---
(In reply to comment #3)

> * python module (_solfege_c_midi.so) should have the permission 0644.
Oops, I wanted to say this should be 0755, not 0644.


-- 
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 219653] Review Request: solfege - music education software

2006-12-14 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: solfege - music education software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 13:23 EST ---
Just a comment:

W: solfege unstripped-binary-or-object /usr/lib/solfege/_solfege_c_midi.so
E: solfege non-executable-script /usr/share/solfege/mpd/testsetup.py 0644
E: solfege non-executable-script /usr/share/solfege/src/testsetup.py 0644
E: solfege-debuginfo empty-debuginfo-package

Perhaps all permission issues.
* *.py python script should have the permission 0755
* python module (_solfege_c_midi.so) should have the permission 0644.
  This is also related to debuginfo issue. Binaries are not stripped
  unless they have executable permission flags.

-- 
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 219653] Review Request: solfege - music education software

2006-12-14 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: solfege - music education software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 12:52 EST ---
Not sure I should be doing the review, as you're implementing some of my ideas
already, but I'll clear up a few simple points.

rpmlint produces output for the RPMs.

SRPM:
W: solfege strange-permission solfege.sh.in 0755
W: solfege mixed-use-of-spaces-and-tabs (spaces: line 6, tab: line 4)

 For the first, `chmod -x olfege.sh.in` -- the script doesn't need to be
executable in the SRPM.
 The second isn't particularly critical, just a matter of presentation. Pretty
easy to fix so you don't have any warnings, though.

the main RPM:
W: solfege unstripped-binary-or-object /usr/lib/solfege/_solfege_c_midi.so
E: solfege non-executable-script /usr/share/solfege/mpd/testsetup.py 0644
E: solfege non-executable-script /usr/share/solfege/src/testsetup.py 0644

 No idea what's going on there.

debuginfo RPM:
E: solfege-debuginfo empty-debuginfo-package

 I'm a bit confused by this; it looks like the binaries may have gotten stripped
by the `make install` or something.

-- 
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 196393] Review Request: svrcore

2006-12-14 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: svrcore
Alias: svrcore

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 12:52 EST ---
Thank you very much, Toshio-san.  I have taken your changes and added some code
to make finding nspr and nss flexible depending on the build environment.  I
unfortunately could not make configure/libtool work on Windows with the MSVC
compiler, so the Windows builds will have to build the old way. No big deal. 
But the new configure way to build should be much better on the linux/unix
platforms we have to support.  Since these changes are pretty significant, I
went ahead and changed the version number.  The new files are here:
SRPM:
ftp://ftp.mozilla.org/pub/mozilla.org/directory/svrcore/releases/4.0.3/svrcore-4.0.3.01-0.src.rpm
SOURCE:
ftp://ftp.mozilla.org/pub/mozilla.org/directory/svrcore/releases/4.0.3/svrcore-4.0.3.01.tar.gz
SPEC:
ftp://ftp.mozilla.org/pub/mozilla.org/directory/svrcore/releases/4.0.3/svrcore.spec

-- 
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 219653] Review Request: solfege - music education software

2006-12-14 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: solfege - music education software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 12:44 EST ---
Changes:

- Use install to install wrapper script
- Improvements to wrapper script

Updated: 

Spec URL: http://folk.ntnu.no/sindrb/packages/solfege.spec
SRPM URL: http://folk.ntnu.no/sindrb/packages/solfege-3.6.4-2.src.rpm

-- 
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 218230] Review Request: php-pear-Image-Canvas - Common interface to image drawing

2006-12-14 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: php-pear-Image-Canvas - Common interface to image 
drawing
Alias: pear-Image-Canvas

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 12:31 EST ---
> I think we should keep the files there anyway
Ok for me

> However I agree that adding a README.Fedora file explaining this should be
required
Of course, i also think it is a MUST.



-- 
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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 12:10 EST ---
Created an attachment (id=143662)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=143662&action=view)
backtrace from the segfault


-- 
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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 12:09 EST ---
Thanks, I got a backtrace. It isn't very reproducible. Sometimes
it segfaults, and sometimes it shutdowns gracefully. It is not only
during the first run but anytime.

-- 
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 219653] New: Review Request: solfege - music education software

2006-12-14 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=219653

   Summary: Review Request: solfege - music education software
   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://folk.ntnu.no/sindrb/packages/solfege.spec
SRPM URL: http://folk.ntnu.no/sindrb/packages/solfege-3.6.4-1.src.rpm

Description:

Solfege is free music education software. Use it to train your rhythm, 
interval, scale and chord skills. Solfege - Smarten your ears!

Note: Solfege relies on esound for large parts of it's functionality. To ensure 
that esd is running I've created a simple wrapper script. The wrapper script 
uses a pid file instead of the -terminate option of esd due to Solfege breaking 
the connection to esd and as such, esd would stop. The goal of the wrapper 
script is to ensure that Solfege "just works". This is a hack and if someone 
has a better solution to make Solfege "just work" please provide it.

-- 
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 218230] Review Request: php-pear-Image-Canvas - Common interface to image drawing

2006-12-14 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: php-pear-Image-Canvas - Common interface to image 
drawing
Alias: pear-Image-Canvas

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:51 EST ---
I think perhaps it would be better to leave the PDF files in the RPM.  Even
though they require a package which cannot be included in Extras, I think we
should keep the files there anyway incase someone wants to install the packages
from another repository such as Remi's repo.  However I agree that adding a
README.Fedora file explaining this should be required.  Please let me know if
you think this is not a good idea.  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 218230] Review Request: php-pear-Image-Canvas - Common interface to image drawing

2006-12-14 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: php-pear-Image-Canvas - Common interface to image 
drawing
Alias: pear-Image-Canvas

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:47 EST ---
Okay, looks like the pdflib stuff can't go into Extras, I'll go ahead and start
working on the items you mentioned in comment #3.

-- 
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 219020] Review Request: php-pecl-pdflib - Package for generating PDF files

2006-12-14 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: php-pecl-pdflib - Package for generating PDF files
Alias: php-pecl-pdflib

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
OtherBugsDependingO|163778  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:45 EST ---
Since pdflib-lite cannot be included in extras, Im closing this bug also as 
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 219013] Review Request: pdflib-lite - Portable C library for dynamically generating PDF files

2006-12-14 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: pdflib-lite - Portable C library for dynamically 
generating PDF files
Alias: pdflib-lite

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


[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 219020] Review Request: php-pecl-pdflib - Package for generating PDF files

2006-12-14 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: php-pecl-pdflib - Package for generating PDF files
Alias: php-pecl-pdflib

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


Bug 219020 depends on bug 219013, which changed state.

Bug 219013 Summary: Review Request: pdflib-lite - Portable C library for 
dynamically generating PDF files
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219013

   What|Old Value   |New Value

 Resolution||WONTFIX
 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 219013] Review Request: pdflib-lite - Portable C library for dynamically generating PDF files

2006-12-14 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: pdflib-lite - Portable C library for dynamically 
generating PDF files
Alias: pdflib-lite

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX




--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:42 EST ---
okay. closing as wontfix.  Thanks for the clarification.

-- 
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 219013] Review Request: pdflib-lite - Portable C library for dynamically generating PDF files

2006-12-14 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: pdflib-lite - Portable C library for dynamically 
generating PDF files
Alias: pdflib-lite

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:37 EST ---
Doesn't matter.  OSI requires that the license cannot restrict use of the 
software, see items 5,6 on:
http://www.opensource.org/docs/definition.php
which is exactly what pdflib-lite's license (apparently) does: limits use to 
only OSS developers, private non-profit users, or edu/research.

-- 
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 190189] Review Request: PyQt4: Python bindings for Qt4

2006-12-14 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: PyQt4: Python bindings for Qt4


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:32 EST ---
PyQt4-4.1.1 hit the streets, (haven't had a chance to track down the rpath
issue(s) yet):

%changelog
* Mon Dec 11 2006 Rex Dieter  4.1.1-1
- PyQt4-4.1.1
- BR: sip-devel >= 4.5.1

Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/PyQt4.spec
SRPM URL:
http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.stable/PyQt4-4.1.1-1.src.rpm

Bad news, is that this *again* needs a sip upgrade.  *sigh*.  Maybe we'll have
to resign ourselves to target this only for F*7.


-- 
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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:31 EST ---
Sorry: the correct one is:

(gdb) handle SIGINT nostop print pass
not "nonstop".


-- 
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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:29 EST ---
(In reply to comment #20)
> (In reply to comment #17)
> 
> > I'm unable to reproduce this.  Try loading the debug symbols and getting a
> > backtrace under gdb.
> 
> I don't know how to do that. I start with gdb, enters the pasword,
> and then hit ctrl-C, but then gdb catch the signal:
> 
> jeu 14 déc 2006 17:17:06 CET  THREADMGMT[t3018537872]: NPS(1,eth0): 
> pcapDispatch
> thread running [p12392]
> 
> Program received signal SIGINT, Interrupt.
> [Switching to Thread -1209092416 (LWP 12392)]
> 0x001d6402 in __kernel_vsyscall ()
> (gdb)
> 
> What should I do now?
> 

I don't checked 3.2-6, however, what happens if you try
(gdb) handle SIGINT nonstop print pass
?



-- 
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 219013] Review Request: pdflib-lite - Portable C library for dynamically generating PDF files

2006-12-14 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: pdflib-lite - Portable C library for dynamically 
generating PDF files
Alias: pdflib-lite

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:27 EST ---
Rex:  You are quoting something that is from a "summary" and not the actual
license.  The actual license is here:
http://www.pdflib.com/fileadmin/pdflib/pdf/license/PDFlib-Lite-license.pdf

In which clause 2.1 excepts open source uses from needed a commercial license.

Please refer to the actual license text if you find something not OSI
compatible.  I apologize for not linking directly to it before.

-- 
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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:19 EST ---
(In reply to comment #17)

> I'm unable to reproduce this.  Try loading the debug symbols and getting a
> backtrace under gdb.

I don't know how to do that. I start with gdb, enters the pasword,
and then hit ctrl-C, but then gdb catch the signal:

jeu 14 déc 2006 17:17:06 CET  THREADMGMT[t3018537872]: NPS(1,eth0): pcapDispatch
thread running [p12392]

Program received signal SIGINT, Interrupt.
[Switching to Thread -1209092416 (LWP 12392)]
0x001d6402 in __kernel_vsyscall ()
(gdb)

What should I do now?


-- 
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 219013] Review Request: pdflib-lite - Portable C library for dynamically generating PDF files

2006-12-14 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: pdflib-lite - Portable C library for dynamically 
generating PDF files
Alias: pdflib-lite

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:18 EST ---
NOT OSI, this in particular:
"All other uses not mentioned here require a commercial license."



-- 
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

2006-12-14 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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 11:06 EST ---
Well, three questions/issues.

* Is /usr/bin/extract work with no plugins? I think libextractor should
  depend at least on libextractor-plugins-base.

* Please use the more specific home URL. I suggest 
  http://gnunet.org/libextractor/

* I think README.debian is not needed.

-- 
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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 09:59 EST ---
There is a missing BR on net-snmp-devel. After net-snmp-devel is 
installed, build fails with:

gcc -shared  .libs/libsnmpPlugin_la-snmpPlugin.o  -L/lib -lgdome -lglib-2.0
-lxml2 -L/usr/local/lib -L/usr/lib -lne
tsnmpmibs -lnetsnmpagent -lnetsnmphelpers -lnetsnmp -lsensors -L/usr/lib/lib
/usr/lib/perl5/5.8.8/i386-linux-thread
-multi/auto/DynaLoader/DynaLoader.a
-L/usr/lib/perl5/5.8.8/i386-linux-thread-multi/CORE -lperl -lresolv -lnsl -ldl 
-lm -lutil -lpthread -lcrypt -lc -lssl -lcrypto -lpcap -lgdbm -lgd -lpng -lz
-lwrap  -m32 -march=i386 -mtune=generi
c -Wl,-E -Wl,-rpath -Wl,/usr/lib/perl5/5.8.8/i386-linux-thread-multi/CORE
-Wl,-soname -Wl,libsnmpPlugin-3.2.so -o .
libs/libsnmpPlugin-3.2.so
/usr/bin/ld: cannot find -lsensors
collect2: ld returned 1 exit status
make[3]: *** [libsnmpPlugin.la] Error 1

Before it fails, there is a big warning:
*** Warning: Linking the shared library libsnmpPlugin.la against the
*** static library
/usr/lib/perl5/5.8.8/i386-linux-thread-multi/auto/DynaLoader/DynaLoader.a is not
portable!




Some warnings that may be worrisome:

address.c: In function 'ipaddr2str':
address.c:516: warning: 'key_data.dptr' may be used uninitialized in this 
function
address.c:516: warning: 'key_data.dsize' may be used uninitialized in this 
function

fcUtils.c: In function 'processFcNSCacheFile':
fcUtils.c:565: warning: format '%02hx' expects type 'short unsigned int *', but
argument 3 has type 'u_int32_t *'
fcUtils.c:565: warning: format '%02hx' expects type 'short unsigned int *', but
argument 4 has type 'u_int32_t *'
fcUtils.c:565: warning: format '%02hx' expects type 'short unsigned int *', but
argument 5 has type 'u_int32_t *'

plugin.c: In function 'loadPlugins':
plugin.c:54: warning: 'prev' may be used uninitialized in this function

pbuf.c: In function 'processPacket':
pbuf.c:3153: warning: pointer targets in passing argument 2 of
'__builtin___strncpy_chk' differ in signedness
pbuf.c:3153: warning: pointer targets in passing argument 2 of '__strncpy_ichk'
differ in signedness

pbuf.c: In function 'processIpPkt':
pbuf.c:951: warning: 'off' may be used uninitialized in this function
pbuf.c:943: warning: 'snapend' may be used uninitialized in this function
pbuf.c:942: warning: 'cp' may be used uninitialized in this function
pbuf.c:944: warning: 'icmp6len' may be used uninitialized in this function

sessions.c: In function 'handleSession':
sessions.c:2550: warning: suggest parentheses around && within ||
sessions.c:2552: warning: suggest parentheses around && within ||

util.c: In function 'createThread':
util.c:1670: warning: format '%lu' expects type 'long unsigned int', but
argument 5 has type 'pthread_t *'
util.c: In function '_killThread':
util.c:1688: warning: format '%lu' expects type 'long unsigned int', but
argument 5 has type 'pthread_t *'
util.c: In function '_joinThread':
util.c:1708: warning: format '%lu' expects type 'long unsigned int', but
argument 5 has type 'pthread_t *'
util.c: In function '_accessMutex':
util.c:1796: warning: format '%s' expects type 'char *', but argument 7 has type
'void *'
util.c:1796: warning: format '%d' expects type 'int', but argument 8 has type
'char *'
util.c:1796: warning: too many arguments for format
util.c: In function '_tryLockMutex':
util.c:1870: warning: format '%s' expects type 'char *', but argument 7 has type
'void *'
util.c:1870: warning: format '%d' expects type 'int', but argument 8 has type
'char *'
util.c:1870: warning: too many arguments for format
util.c: In function '_releaseMutex':
util.c:1957: warning: format '%p' expects type 'void *', but argument 10 has
type 'pid_t'

util.c: In function 'pathSanityCheck':
util.c:3020: warning: array subscript has type 'char'
util.c: In function 'fileSanityCheck':
util.c:3093: warning: array subscript has type 'char'
util.c: In function 'ipSanityCheck':
util.c:3149: warning: array subscript has type 'char'

util.c: In function '_ntopSleepMSWhileSameState':
util.c:3950: warning: format '%u' expects type 'unsigned int', but argument 5
has type 'long unsigned int'
util.c:3968: warning: format '%d' expects type 'int', but argument 5 has type
'__time_t'
util.c:3968: warning: format '%d' expects type 'int', but argument 6 has type
'long int'
util.c: In function 'i18n_xvert_locale2common':
util.c:4252: warning: passing argument 1 of 'ntop_safestrdup' discards
qualifiers from pointer target type
util.c: In function 'i18n_xvert_acceptlanguage2common':
util.c:4277: warning: passing argument 1 of 'ntop_safestrdup' discards
qualifiers from pointer target type

util.c: In function 'convertNtopVersionToNumber':
util.c:5050: warning: format '%1[a-z' e

[Bug 186452] Review Request: kdebluetooth: The KDE Bluetooth Framework

2006-12-14 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: kdebluetooth: The KDE Bluetooth Framework


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 09:43 EST ---
Yeah, been swamped with other stuff, and I was dreading treading into the
selinux ickiness.

I'll see about addressing at least the other issues, and update this soon.

-- 
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 219164] Review Request: fontypython - TTF font manager

2006-12-14 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: fontypython - TTF font manager


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 09:33 EST ---
Well, now I do a full review.

A. From http://fedoraproject.org/wiki/Packaging/Guidelines :
* Requires
  - python >= 2.4
  Current rpm automatically checks python abi requirement,
  so this requirement is not needed.
--
[EMAIL PROTECTED] things]$ rpm -q --requires fontypython
/usr/bin/python  
python >= 2.4
python(abi) = 2.5 <- this is automatically by rpmbuild process
python-imaging  
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
wxPython  
--

B. From http://fedoraproject.org/wiki/Packaging/ReviewGuidelines :
* The sources used to build the package must.
  - Please specify the URL of Source0.

Other things are okay.

= For command name:
  - Well, in fact I maintain a rpm package named "jd", which
contains a binary named "/usr/bin/jd"!! So I cannot reject
it if you don't want to rename /usr/bin/fp .

Then:
-
NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to formally review other 
submitters' review request and approve the packages. 
For this reason, the person who want to be sponsored (like you) 
are required to "show that you have an understanding 
of the process and of the packaging guidelines" as is descriped
on :
http://fedoraproject.org/wiki/Extras/HowToGetSponsored


Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a "pre-review" (at the time you are not sponsored, you cannot do
   a formal review) of other person's review request.

When you submitted a new review request or have pre-reviewed other person's
review request, please write the bug number on this bug report so that I
can check your comments or review request.

Fedora Extras package review requests which are waiting for someone to review
can be checked on:
https://bugzilla.redhat.com/bugzilla/showdependencytree.cgi?id=FE-NEW&hide_resolved=1

Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets


-- 
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 190189] Review Request: PyQt4: Python bindings for Qt4

2006-12-14 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: PyQt4: Python bindings for Qt4


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


Bug 190189 depends on bug 214193, which changed state.

Bug 214193 Summary: sip: upgrade to 4.5.1+
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214193

   What|Old Value   |New Value

 Resolution|RAWHIDE |
 Status|CLOSED  |NEW



-- 
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 216103] Review Request: python-twisted-names - A Twisted DNS implementation

2006-12-14 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-twisted-names - A Twisted DNS implementation


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 08:57 EST ---
(In reply to comment #2)
> (In reply to comment #1)
> >
> > W: python-twisted-names doc-file-dependency
> > /usr/share/doc/python-twisted-names-0.3.0/examples/dns-service.py
/usr/bin/python
> > W: python-twisted-names doc-file-dependency
> > /usr/share/doc/python-twisted-names-0.3.0/examples/gethostbyname.py 
> > /usr/bin/env
> > 
> > Perhaps make those mode 644 ? 444?
> 
> They aren't really adding any extra deps; the first is a dep on python, which 
> is
> already required by this package, and the second is a dep on coreutils, which
> every system is going to have anyway.

Still, they are annoyances that are easily fixed in %prep.


-- 
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 219610] New: Review Request: uncrustify-0.30 - code beautifier

2006-12-14 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=219610

   Summary: Review Request: uncrustify-0.30 - code beautifier
   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://nbecker.dyndns.org:8080/RPM/uncrustify.spec
SRPM URL: http://nbecker.dyndns.org:8080/RPM/uncrustify-0.30-1.fc6.src.rpm
Description: Source Code Beautifier for C, C++, C#, D, Java, and Pawn

-- 
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 217497] Review Request: dbmail - The DBMail mail storage system

2006-12-14 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: dbmail - The DBMail mail storage system


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 06:35 EST ---
Spec URL: http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/dbmail.spec
SRPM URL:
http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/dbmail-2.2.1-4.fc6.src.rpm

* Thu Dec 14 2006 Bernard Johnson <[EMAIL PROTECTED]> 2.2.1-4
- cleanup of spec file
- use fedora-usermgmt hooks
- split and build all database libraries
- kill modules/.libs from the module load path


-- 
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 219582] Review Request: gourmet - personal recipe database

2006-12-14 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: gourmet - personal recipe database


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 06:08 EST ---
# 001 : Shouldn't this be removed from the spec file now ?
Vendor: Thomas Mills Hinkle [...]
Packager: John (J5) Palmieri [...]

# 002 :
Source0: %{name}-%{version}.tar.gz
should have a complete url

# 003 :
The buildroot tag should comply to FE packaging guidelines

#004 :
These should be a changelog as well.

Perhaps these should be among the changes to be made to the already approuved FE
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 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility

2006-12-14 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: beryl-vidcap - Beryl OpenGL window and compositing 
manager video capture utility


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 06:03 EST ---
(In reply to comment #15)

> Then please check this rpmlint error.
> ---
> E: beryl-vidcap shlib-with-non-pic-code /usr/lib/libseom.so.0.0.0
> ---
> 

If beryl-vidcap is installed, each time yum performs a ldconfig, it spits an
error dur to usr/lib/libseom.so.0.0.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 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-14 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 04:57 EST ---
(In reply to comment #39)
> My pleasure. Thank you for including conky in your distribution.

I think that you should consider adding the GPL and LGPL license
texts in your package, for example with names like COPYING.LGPL, 
COPYING.GPL. Otherwise it may be possible that one (a judge) consider 
the GPL/LGPL to be invalid for your code.

-- 
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 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

2006-12-14 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


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 04:48 EST ---
Spec URL: http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/ntop.spec
SRPM URL:
http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/ntop-3.2-6.fc6.src.rpm

* Thu Dec 14 2006 Bernard Johnson <[EMAIL PROTECTED]> - 3.2-6
- configure --disable-static
- configure --enable-snmp
- patch to fix permissions of created gdbm databases
- no more ntop-passwd
- fix OK printing in init file, redirect stdout of ntop command to null
- fix permissions on LsWatch.db database creation
- only listen on 127.0.0.1:3000 by default


(In reply to comment #17)
> > * tcp_wrappers-devel is out in devel
> 
> I will fix that in the next update.

I'm building for FC6, so I can't do it at this time.
 
> > * ntop is still uninterruptible by Ctrl-C when the password is 
> >   entered, but I guess it is because Ctrl-C is interpreted as a 
> >   character of the password.
> I believe that is true, but I'll look into it.

ntop calls the GETPASS(3) routine.  From the man page:
While reading the password, signal generation (SIGINT, SIGQUIT, SIGSTOP,
SIGTSTOP) is disabled  and the  corresponding  characters (usually control-C,
control-\, control-Z and control-Y) are transmitted as part of  the  password.

So "as designed".


-- 
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 219539] Review Request: qsa - QT Script for Applications

2006-12-14 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: qsa - QT Script for Applications


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 04:17 EST ---
Well, how about qt-qsa?

-- 
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 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-14 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 04:04 EST ---
(In reply to comment #40)
> Patrice, that patch wont apply cleanly.. I just made a diff -Naur of the
> timed_thread.c and .h files and of the News file, will that be fine??

Yes, it's fine, you can import it.

-- 
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 217311] Review Request: xarchiver - Archive manager for Xfce

2006-12-14 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: xarchiver - Archive manager for Xfce


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-14 04:00 EST ---
(In reply to comment #20)

> Should be gettext for %find_lang. Because this package is a SVN snapshot it 
> also
> BuildRequires xfce-dev-tools, which have a dependency on gettext-devel (I 
> think
> you know, because you reviewed it)

Right.

-- 
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