[Bug 193319] New: Review Request: chestnut-dialer - A PPP dialing program

2006-05-27 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=193319

   Summary: Review Request: chestnut-dialer - A PPP dialing program
   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://lostclus.linux.kiev.ua/files/chestnut-dialer.spec
SRPM URL: http://lostclus.linux.kiev.ua/files/chestnut-dialer-0.3.2-2.src.rpm
Description:
Chestnut Dialer is PPP dialing program, written in Python. Many Internet
providers, that provide dialup service, use PPP protocol; this program helps
you to connect to Internet using a modem. Current version 0.3.2 can work
with GTK2, QT, and without GUI (command line interface). Chestnut Dialer uses
standard pppd daemon to set up network interface.

-- 
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 193187] Review Request: pcsc-lite ccid

2006-05-27 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: pcsc-lite  ccid


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 04:55 EST ---
Yes, /usr/sbin/update-reader.conf is being run on every pcscd start, see start()
in /etc/init.d/pcscd.

pcsc-lite does not depend on ccid specifically.  It depends on pcsc-ifd-handler,
which is a virtual dependency, provided by packages that contain suitable
drivers.  In addition to ccid, in Extras it is provided by pcsc-lite-openct and
(should be provided) by ctapi-cyberjack-pcsc, as well as number of other, mainly
proprietary, driver packages elsewhere.


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

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


[Bug 193216] Review Request: qcwebcam

2006-05-27 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: qcwebcam


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 05:39 EST ---
John,
 Please check SPEC file again at
http://people.redhat.com/pnemade/Qcwebcam/qcwebcam.spec. I am getting error 
error: Bad file: /usr/src/redhat/SOURCES/Webcam.desktop: No such file or 
directory
what changes should i do in SPEC file??
 Also for using dist tag i saw onse statement on
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-beca3bf84972f19a384cc2e5091ed47c2b3cebc7
that NOTE: Fedora Core packages cannot use the %{?dist} tag as the current build
system cannot handle it.
  So how to use dist in SPEC file? I tried but no value is substituted there
and got blank space in src rpm name.


-- 
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 190390] Review Request: AGReader: Console reader for viewing AmigaGuide files

2006-05-27 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: AGReader: Console reader for viewing AmigaGuide files


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 05:45 EST ---
Created an attachment (id=130064)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=130064action=view)
Patch fixing gcc41 compile and all compiler warnings

Ok,

Attached is a patch fixing compiling with gcc4.1 and all compiler warnings.
also I've removed the -s from the link command in the Makefile so that a proper
-debuginfo package can be generated by rpmbuild.

About the name, the name as is is fine, especially if you've already released
rpms with the name like this.

On to a full review:

MUST:
=
0 rpmlint output is:
W: AGReader incoherent-version-in-changelog 1.1-7.iss 1.1-7
This needs fixing.
* Package and spec file named appropriately
* Packaged according to packaging guidelines
* License (GPL) ok, license file not included because it isn't included by
upstream
* spec file is legible and in Am. English.
* Source matches upstream
* Compiles and builds on devel-x86_64
* BR: ok
* No locales
* No shared libraries
* Not relocatable
* Package owns / or requires all dirs
* No duplicate files  Permissions ok
* %clean  macro usage OK
* Contains code only
* %doc does not affect runtime, and isn't large enough to warrent a sub package

* no -devel package needed, no libs / .la files.
* no gui - no .desktop file required


MUST fix:
=
* Drop CC=gcc32 from make command params and instead use the attached patch
* Add CFLAGS=$RPM_OPT_FLAGS to make command params so that the rpmoptflags
get
  used during compilation.
* Fix incoherent Changelog version.
* Home, End, F1, F2, F3 Keys don't work as advertised in the manual, I've a fix

  and I'll attach a patch for this.

-- 
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 192867] Review Request: ctorrent - BitTorrent Client written in C

2006-05-27 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: ctorrent - BitTorrent Client written in C


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 06:12 EST ---
Done, i fixed all errors listed above.
http://www.netservers.org/packages/rpm/ctorrent.spec
http://www.netservers.org/packages/rpm/ctorrent-1.3.2-3.src.rpm
The license is GPL, I'm sure about it. The author have written a presentation of
his program on a Debian list and he writes that license is GPL. Anyway i've
found some more websites that confirm this.

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

2006-05-27 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: qcwebcam


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 06:36 EST ---
(In reply to comment #2)
 John,
  Please check SPEC file again at
 http://people.redhat.com/pnemade/Qcwebcam/qcwebcam.spec. I am getting error 
 error: Bad file: /usr/src/redhat/SOURCES/Webcam.desktop: No such file or 
 directory

The file Webcam.desktop was missing from the RPM SOURCES directory. This
shouldn't happen if you're building from the SRPM, which contains this file.

 what changes should i do in SPEC file??

I'd suggest reading the packaging guidelines here:
http://fedoraproject.org/wiki/Packaging/Guidelines

It doesn't cover everything here though. In particular, RPM packages should not
install anything under /usr/local - you need to change the install step to put
files under /usr instead.

  Also for using dist tag i saw onse statement on

http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-beca3bf84972f19a384cc2e5091ed47c2b3cebc7
 that NOTE: Fedora Core packages cannot use the %{?dist} tag as the current 
 build
 system cannot handle it.

This is a Fedora Extras package submission, not a Fedora Core one. The Fedora
Extras build system (which will soon be used for Fedora Core anyway) handles the
dist tag just fine.

   So how to use dist in SPEC file? 

Exactly as it says on the wiki.

I tried but no value is substituted there
 and got blank space in src rpm name.

The macros will defined automatically in the Fedora Extras build system, and you
can try this for yourself by building your package in mock. See:
http://fedoraproject.org/wiki/Projects/Mock

You could alternatively define the macros manually when you build the package:

$ rpmbuild --define fedora 5 --define dist .fc5 ...

I see there is another project called qcwebcam:

http://www.fhttpd.org/pub/qcwebcam/README.html

This could cause confusion...




-- 
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 193154] Review Request: asymptote - Descriptive vector graphics language

2006-05-27 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: asymptote - Descriptive vector graphics language


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 06:53 EST ---
* package naming: ok
* license good and included: ok
* spec file layout: ok
* source md5: ok
* builds in mock/i386: ok
* buildreq: ok
* no locales
* owns all directories it creates: ok
* correct use of $RPM_BUILD_ROOT: ok
* documentation in the right place: ok
* no header files or shared libraries
* no desktop file
* works as expected on examples: ok
* triggers generate warnings by rpmlint: ignore

? group tag is Development/Tools
  I suggest Applications/Publishing in analog to tetex (and metapost)

APPROVED
  

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

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


[Bug 189268] Review Request: xscreensaver

2006-05-27 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: xscreensaver


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 08:30 EST ---
This time, I could upload my src.rpm. Then I tried to rebuild it but it failed
perhaps because development repomd.xml seems to be broken currently.

I re-try to rebuild when repomd problem is solved.

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

2006-05-27 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: elektra


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 08:37 EST ---
Please try the new src.rpm from 
http://prdownloads.sourceforge.net/elektra/elektra-0.6.1-31.src.rpm

(In reply to comment #25)
 blockers: 
 * changelog 
 * source not found
 * .so for libelekra should be in a devel package

All three points fixed.


 rpmlint output: 
 W: elektra incoherent-version-in-changelog 0.6.1-3 0.6.1-30
 E: elektra invalid-soname /lib/libelektra-filesys.so libelektra-filesys.so
 E: elektra invalid-soname /lib/libelektra-fstab.so libelektra-fstab.so
 W: elektra wrong-file-end-of-line-encoding
 /usr/share/doc/elektra-0.6.1/standards/signature.xml
 W: elektra devel-file-in-non-devel-package /usr/lib/libelektratools.so
 W: elektra devel-file-in-non-devel-package /lib/libelektra.so
 W: elektra devel-file-in-non-devel-package /lib/libelektra-default.so
 E: elektra script-without-shellbang
 /usr/share/doc/elektra-0.6.1/scripts/convert-hwconfKudzu

All coherent messages fixed.


 other comments:
 * --prefix=%{_prefix} and --exec-prefix=/ were unneeded in my tests

This is correct. If removed, software won't be installed in /bin and /lib

 * The redundant %doc line is not needed in devel subpackage:
 %doc AUTHORS COPYING ChangeLog README INSTALL

Removed.


The key.3.gz conflict was also removed.

-- 
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 193319] Review Request: chestnut-dialer - A PPP dialing program

2006-05-27 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: chestnut-dialer - A PPP dialing program


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  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 192867] Review Request: ctorrent - BitTorrent Client written in C

2006-05-27 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: ctorrent - BitTorrent Client written in C


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 08:48 EST ---
You've still got a problem with the %file section.  Looks like you changed
'/usr/bin/ctorrent' to just '%{_bindir}'.  It needs to be '%{_bindir}/ctorrent'
or '%{_bindir}/%{name}'

-- 
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 190390] Review Request: AGReader: Console reader for viewing AmigaGuide files

2006-05-27 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: AGReader: Console reader for viewing AmigaGuide files


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 09:26 EST ---
Many thanks for the patches Hans. Here's the latest spec and srpm which should
fix those issues.

http://www.dribble.org.uk/AGReader.spec
http://www.dribble.org.uk/AGReader-1.1-8.src.rpm

I'll also try and submit those patches upstream. I'm working this weekend, but
will get on with the other two rpms asap.

-- 
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 193110] Review Request: python-sexy

2006-05-27 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-sexy


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 09:37 EST ---
Thanks for your pieces of advice.
I uploaded new spec and srpm:
http://darkenphoenix.free.fr/RPMS/RPMS/Extras/SRPMS/python-sexy-0.1.8-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 193281] Review Request: python-xmpp - Python library for easy scripting with Jabber

2006-05-27 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-xmpp - Python library for easy scripting with 
Jabber


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 09:39 EST ---
I get a 404 error on both links...

-- 
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 191538] Review Request: qsynth - Qt based Fluidsynth GUI front end

2006-05-27 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: qsynth - Qt based Fluidsynth GUI front end


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||189886




-- 
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 189886] Review Request: FluidSynth - Real-time software synthesizer

2006-05-27 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: FluidSynth - Real-time software synthesizer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||191538
  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 193154] Review Request: asymptote - Descriptive vector graphics language

2006-05-27 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: asymptote - Descriptive vector graphics language


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 09:54 EST ---
(In reply to comment #12)
 ...
 ? group tag is Development/Tools
   I suggest Applications/Publishing in analog to tetex (and metapost)

Done.

 APPROVED

Thanks for the review.

Do you mind if I send the emacs init file upstream?

jpo

-- 
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 193210] Review Request: htmldoc - Convert HTML source files into HTML, PostScript, or PDF

2006-05-27 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: htmldoc - Convert HTML source files into HTML, 
PostScript, or PDF


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 10:04 EST ---
* No downloadable source. Please give the full URL in the Source tag, which
should be http://ftp.easysw.com/pub/htmldoc/1.8.26/htmldoc-1.8.26-source.tar.bz2
(ok, that one was kind of hard to find...)

-- 
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 193210] Review Request: htmldoc - Convert HTML source files into HTML, PostScript, or PDF

2006-05-27 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: htmldoc - Convert HTML source files into HTML, 
PostScript, or PDF


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 193281] Review Request: python-xmpp - Python library for easy scripting with Jabber

2006-05-27 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-xmpp - Python library for easy scripting with 
Jabber


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 10:32 EST ---
(In reply to comment #1)
 I get a 404 error on both links...

Oops... forgot that i had reorganized the directories on my web server and I
uploaded them to the wrong spot...

-- 
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 189884] Review Request: LASH Audio Session Handler

2006-05-27 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: LASH Audio Session Handler


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 10:33 EST ---
* Build failure:

make[2]: Entering directory `/home/qa/tmp/rpm/BUILD/lash-0.5.1/lashd'
test -z /usr/bin || mkdir -p --
/home/qa/tmp/rpm/tmp/lash-0.5.1-1-root-qa/usr/bin
  /bin/sh ../libtool --mode=install /usr/bin/install -c 'lashd'
'/home/qa/tmp/rpm/tmp/lash-0.5.1-1-root-qa/usr/bin/lashd'
libtool: install: warning: `../liblash/liblash.la' has not been installed in
`/usr/lib'
/usr/bin/install -c .libs/lashd
/home/qa/tmp/rpm/tmp/lash-0.5.1-1-root-qa/usr/bin/lashd
if ! grep -q ^lash /etc/services; then \
  echo -e \nlash\t\t14541/tcp\t\t\t# LASH client/server protocol 
/etc/services; \
fi
/bin/sh: /etc/services: Permission denied


* Configure warning: (would be BuildRequires: texi2html)

checking for texi2html... no
configure: WARNING: texi2html not found, manual will not be built

* Wrong BuildRequires: tetex

* pkgconfig file contains hardcoded rpath linker options

* Missing -devel package Requires for at least jack and alsa
(see pkgconfig file) -devel packages, and e2fsprogs-devel
(see installed C headers)

* Don't install the .la libtool archive file unless you like
libtool dependency hell

* Use %_infodir instead of %_datadir/info/

* Add rm -f %{buildroot}%{_infodir}/dir at end of %install section

* Install info files with install-info:
  http://fedoraproject.org/wiki/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 189886] Review Request: FluidSynth - Real-time software synthesizer

2006-05-27 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: FluidSynth - Real-time software synthesizer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 10:54 EST ---
Unnecessary build dependence e2fsprogs-devel.

-- 
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 193210] Review Request: htmldoc - Convert HTML source files into HTML, PostScript, or PDF

2006-05-27 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: htmldoc - Convert HTML source files into HTML, 
PostScript, or PDF


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 11:20 EST ---
Thank you for your comment.

I have created a new package here:
http://www.spicenitz.org/fedora/htmldoc-1.8.26-2.src.rpm
http://www.spicenitz.org/fedora/htmldoc.spec


I used %{name} and %{version} in the Source tag. Is that ok, or should I have
put the URL in directly?

-- 
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 193240] Review Request: XaraLX - Vector/general purpose graphics utility

2006-05-27 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: XaraLX - Vector/general purpose graphics utility


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 11:34 EST ---
Does not build in mock:
  autoreconf: failed to run autopoint: No such file or directory
Needs BuildReq: gettext-devel

-- 
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 192867] Review Request: ctorrent - BitTorrent Client written in C

2006-05-27 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: ctorrent - BitTorrent Client written in C


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 11:48 EST ---
ops yes, my mistake sorry.
Anyway fixed 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 192867] Review Request: ctorrent - BitTorrent Client written in C

2006-05-27 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: ctorrent - BitTorrent Client written in C


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 12:10 EST ---
+1 APPROVE.

Fixed errors noted in comment #7.

I'll also be your sponser.  It looks like your next step is:
http://fedoraproject.org/wiki/Extras/Contributors#head-a89c07b5b8abe7748b6b39f0f89768d595234907

-- 
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 193187] Review Request: pcsc-lite ccid

2006-05-27 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: pcsc-lite  ccid


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 12:56 EST ---
A conf file in /usr/sbin just seems a bit um... not cool.  Is there any way to
put this somewhere else?  /etc/sysconfig/ ? /etc/ ?  /usr/share/ ?  Somewhere
that isn't /usr/sbin...

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

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


[Bug 193312] Review Request: DevIL: A cross-platform image library

2006-05-27 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: DevIL: A cross-platform image library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 13:22 EST ---
Here's the new spec and srpm which includes a patch to fix the pic problem.

http://www.dribble.org.uk/DevIL.spec
http://www.dribble.org.uk/DevIL-1.6.8-0.5.rc1.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 193342] New: Review Request: cegui: Free library providing windowing and widgets for graphics APIs / engines

2006-05-27 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=193342

   Summary: Review Request: cegui: Free library providing windowing
and widgets for graphics APIs / engines
   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://dribble.org.uk/cegui.spec
SRPM URL: http://dribble.org.uk/cegui-0.4.1-2.src.rpm
Description:

Package uploaded for review by Hans de Goede

Crazy Eddie's GUI System is a free library providing windowing and widgets for
graphics APIs / engines where such functionality is not natively available, or
severely lacking. The library is object orientated, written in C++, and
targeted at games developers who should be spending their time creating great
games, not building GUI sub-systems!

-- 
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 193342] Review Request: cegui: Free library providing windowing and widgets for graphics APIs / engines

2006-05-27 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: cegui: Free library providing windowing and widgets 
for graphics APIs / engines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: cegui: Free|Review Request: cegui: Free
   |library providing windowing |library providing windowing
   |and widgets for graphics|and widgets for graphics
   |APIs / engines |APIs / engines
 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




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

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


[Bug 193187] Review Request: pcsc-lite ccid

2006-05-27 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: pcsc-lite  ccid


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 14:15 EST ---
/usr/sbin/update-reader.conf is not a config file, it's a shell script which
creates /etc/reader.conf from  the contents of /etc/reader.conf.d/*.conf.  Yes,
I did nag to upstream about the potentially confusing name of the script...

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

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


[Bug 181599] Review Request: gallery2: web based photo album software

2006-05-27 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: gallery2: web based photo album software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 16:13 EST ---
I have no idea. I've tried the -17 packages on FC5 and FC3 and both have worked;
I've pasted the error to upstream and asked if they have any ideas.

-- 
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 192577] Review Request: perl-OpenFrame

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

Summary: Review Request: perl-OpenFrame


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 16:28 EST ---
(In reply to comment #1)
 Only one of the files (lib/OpenFrame/Argument/Blob.pm) seems to have a 
 statement
 of the license.  I don't think that's enough to suggest the license for the
 entire package.

I've emailed the author (the one listed in that file anyway) for clarification.

 I wonder about the need for these:
 
 Requires:   perl(File::Type) = 0.01
 Requires:   perl(HTTP::Request) = 0.01
 Requires:   perl(IO::Null) = 0.01
 
 The versions are so low that they seem to have been put in as placeholders.  
 RPM
 should figure out all of these on its own.  (It doesn't find HTTP::Request but
 it does find other modules, all provided by perl-libwww-perl.)

Fixed in -2.


t/02http_request[OpenFrame::Segment::HTTP::Response::OpenFrame::Segment::HTTP::Response::dispatch]
 no response available
 at /usr/lib/perl5/vendor_perl/5.8.8/Pipeline/Dispatch.pm line 74
 ok
[...]
 I'm not sure if the first is a mock artifact or a problem with the test suite.

It happens when I'm not running under mock either.

 t/98compile.skipped
 all skipped: - do not have File::Find::Rule installed
[...]
 The second is fixed up with the obvious BR:.

Also fixed in -2.

http://ftp.kspei.com/pub/steve/rpms/perl-OpenFrame-3.05-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 193380] Review Request: hardinfo

2006-05-27 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: hardinfo


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: hardinfo|Review Request: hardinfo
 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 18:19 EST ---
* Needed BuildRequires: which, pciutils
* There is no documentation in the source file, not even a license file.
  Upstream should be notified about this.
* Shared object files must not reside in %{_datadir}/hardinfo, they
  must go to %{_libdir}/hardinfo. If necessary you need to patch
  the source 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 193342] Review Request: cegui: Free library providing windowing and widgets for graphics APIs / engines

2006-05-27 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: cegui: Free library providing windowing and widgets 
for graphics APIs / engines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 18:26 EST ---
Created an attachment (id=130112)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=130112action=view)
Patch making cegui use the system pcre

cegui comes with its own copy of pcre. Which is a library which is also
available seperatly in FE. Using private copies of libraries is concidered a
bad practice (it removes / reverts most advantages of using shared libs if
there are many copies of each lib in different packages).

Please do a rm -fr src/pcre in %prep, add pcre-devel to the BR and use the
attached patch to get cegui to compile with the system pcre.


-- 
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 181599] Review Request: gallery2: web based photo album software

2006-05-27 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: gallery2: web based photo album software


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-27 20:37 EST ---
I ran into this problem configuring for multisite when I tried out the earlier
RPMS. The problem for me seemed to be that no matter what I put in for the data
directory (the non web accessible data area), after the config file was written,
it had a different (i.e., not the one I specified) /srv based data directory.
After the config file writing step I had to edit the just written config file to
point to the correct data directory or else I would get the same error on the
next step when the install procedure copied over the files to the new multisite.
Hopefully not to obtuse...

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