[Bug 210790] Review Request: dar - Collection of scripts for making/restoring CD/DVD backups

2006-11-04 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: dar - Collection of scripts for making/restoring 
CD/DVD backups


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 03:28 EST ---
ok, updated files:

http://rpm.forevermore.net/dar/dar-2.3.1-2.src.rpm
http://rpm.forevermore.net/dar/dar.spec

Disabled the static build by default, esp. since the current glibc seems to be
broken and won't actually produce a real static binary, anyway.

I can't retest this in mock at the moment since I'm still in the middle of
setting up fc6, but it should still work fine.

-- 
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 172869] Review Request: nss-mdns

2006-11-04 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: nss-mdns


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 04:16 EST ---
I can't download the srpm, could you fix 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 187843] Review Request: phpMyAdmin - Web based MySQL browser written in php

2006-11-04 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: phpMyAdmin - Web based MySQL browser written in php


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 04:30 EST ---
Cross site scripting vulnerability in 2.6.4-2.9.0.2:
http://nvd.nist.gov/nvd.cfm?cvename=CVE-2006-5718


-- 
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 201636] Review Request: CSpace p2p Instant Messenger

2006-11-04 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: CSpace p2p Instant Messenger


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 04:36 EST ---
Any news on this ? Where is the spec file ?

-- 
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 213432] Review Request: kaffeine - Xine-based media player

2006-11-04 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: kaffeine - Xine-based media player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 06:14 EST ---
#001
Isn't it time to merge drop %{?fedora}  2 ?

%if %{?fedora}  2 || %{?rhel}  4
%define _with_gstreamer --with-gstreamer
%endif

I recommend to merge it with %if %{?fedora}  4 || %{?rhel}  4

#002
desktop-file-install \
  --vendor= \

use fedora as vendor :)
desktop-file-install --vendor fedora \

#003
README*
There is one README file to me.

#004
BuildRequires: cdparanoia-devel cdparanoia
cdparanoia-devel already requires cdparanoia

correct these,  I'll approve it as quick as possible

#005:
can you contact upstream to update their download page
http://kaffeine.sourceforge.net/index.php?page=download with respect to fedora.
I just want people to have a good fedora KDE image :)



-- 
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 211703] Review Request: nyquist - Sound synthesis and composition language with a Lisp syntax

2006-11-04 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: nyquist - Sound synthesis and composition language 
with a Lisp syntax


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 07:06 EST ---
Built on FC5, FC6 and FC7.
Added entries to owners and comps files.
Thanks for the review.

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

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


[Bug 184011] Review Request: nickle

2006-11-04 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: nickle


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 07:18 EST ---
I think it is enough to have
  rm -rf $RPM_BUILD_ROOT
instead of
  [ $RPM_BUILD_ROOT != / ]  rm -rf $RPM_BUILD_ROOT

The examples directory appears twice, once in the docdir and once in
/usr/share/nickle. I prefer the docdir.
The examples directory contains Makefile* files which are not functional.


-- 
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 203520] Review Request: evolution-brutus

2006-11-04 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: evolution-brutus
Alias: evolution-brutus

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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 08:21 EST ---
You are right again. I'll fix this when I get back to my office on Monday.

Sorry,
  jules


-- 
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 210790] Review Request: dar - Collection of scripts for making/restoring CD/DVD backups

2006-11-04 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: dar - Collection of scripts for making/restoring 
CD/DVD backups


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 09:19 EST ---
Okay, first full review of this package (dar).
Almost okay.

1. From http://fedoraproject.org/wiki/Packaging/Guidelines :
*  rpmlint
   rpmlint is not silent.

W: dar file-not-utf8 /usr/share/man/man1/dar.1.gz
   dar.1.gz contains ISO-8859-1 character (around the line 341).

   This allows one to merge two archive in a single one. See also -$, 
  -THIS CHARACTER and -% for other options concerning auxiliary 
   archive of reference

   Please change this character MANUALLY (iconv does not work for
   this case as this character seems to be tilde, while iconv tries
   to change this character to English pound character.

* Requires:
  - For libdar-devel
Usually the dependency for main package (for this package, it
is libdar) is release-specific. i.e.
Requires: libdar = %{version}-%{release}

  - Provides like
Provides:   libdar = %{version}-%{release}
are all unnecessary as rpm always provides these implicitly.

* BuildRequires:
  - Redundant BuildRequires is found.
* zlib-devel - this is required by openssl-devel

2. http://fedoraproject.org/wiki/Packaging/ReviewGuidelines = Nothing.

3. Other things I have noticed:
* README.Fedora
  - Okay, this document is very preferable. Also I recommend you add 
your name and the date when you wrote 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 209261] Review Request: beryl-settings - Beryl OpenGL window and compositing manager GUI config utility

2006-11-04 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-settings -  Beryl OpenGL window and compositing 
manager GUI config utility


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 09:52 EST ---
chitlesh[0]$rpm -qf /usr/bin/beryl-settings
beryl-settings-0.1.1-2.fc6

chitlesh[0]$cat /usr/share/applications/fedora-beryl-settings.desktop
[Desktop Entry]
Encoding=UTF-8
Name=Beryl Settings Manager
Comment=Configure beryl
Icon=/usr/share/pixmaps/beryl-settings.png
Exec=beryl-settings
Terminal=false
Type=Application
Categories=Settings;X-Fedora;
X-Desktop-File-Install-Version=0.10

chitlesh[0]$ls /usr/share/pixmaps/beryl-settings.png
ls: /usr/share/pixmaps/beryl-settings.png: No such file or directory


Hence no icon on kmenu !

-- 
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 213193] Review Request: gaim-rhythmbox - Rhythmbox plugin for GAIM

2006-11-04 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: gaim-rhythmbox - Rhythmbox plugin for GAIM


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 11:08 EST ---
Bizarre, that does not appear on my system when using mock to build the package.
When /not/ using mock the dependency appears properly.. I'll wait for the FE
result to comes out and reverify.

% rpm -q gaim-rhythmbox
gaim-rhythmbox-2.0-0.4.beta4.fc6

% rpm -q --requires gaim-rhythmbox | grep gaim
%


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

2006-11-04 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: nickle


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 11:35 EST ---
http://hircus.org/fedora/nickle/nickle-2.54-2.src.rpm
http://hircus.org/fedora/nickle/nickle.spec

Fixed, thanks. Note that the two left-over COPYING files in %{_docdir}/examples
are there because they cite different authors than the main COPYING file.

-- 
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 199108] Review Request: gutenprint: Printer Drivers Package

2006-11-04 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: gutenprint: Printer Drivers Package


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 11:39 EST ---
I've installed the packages on extras and everything appears to be working fine
with the exception of replacement of the gimp-print plugin.  I tried removing
the gimp-print-plugin provided in FC6 core, but then realized their doesn't
appear to be an rpm available to replace it.  Is one in the works, or is this
deferred to FC7?  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 213432] Review Request: kaffeine - Xine-based media player

2006-11-04 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: kaffeine - Xine-based media player


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 11:43 EST ---
Chitlesh, remember to move the bug to the FE-REVIEW tracker

-- 
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 213432] Review Request: kaffeine - Xine-based media player

2006-11-04 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: kaffeine - Xine-based media player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 11:49 EST ---
 #002
 desktop-file-install \
  --vendor= \
 use fedora as vendor :)
 desktop-file-install --vendor fedora \

The packaging guidelines were recently changed to not override upstream vendor
(kde in this case).

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

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


[Bug 213432] Review Request: kaffeine - Xine-based media player

2006-11-04 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: kaffeine - Xine-based media player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 11:53 EST ---
 #004
 BuildRequires: cdparanoia-devel cdparanoia
 cdparanoia-devel already requires cdparanoia

Check again. (:  Last I checked, cdparanoia-devel only Req'd cdparanoia-libs.

-- 
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 213432] Review Request: kaffeine - Xine-based media player

2006-11-04 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: kaffeine - Xine-based media player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 11:55 EST ---
 #001
 Isn't it time to merge drop %{?fedora}  2 ?

I could, buy why?  It's not wrong.  changing to fedora  4 isn't as correct and
misleads one to think that only 4+ is supported.

-- 
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 213432] Review Request: kaffeine - Xine-based media player

2006-11-04 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: kaffeine - Xine-based media player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 11:59 EST ---
 Check again. (:  Last I checked, cdparanoia-devel only Req'd cdparanoia-libs.

Ok, you are right about it

 I could, buy why?  It's not wrong.  changing to fedora  4 isn't as correct 
 and
misleads one to think that only 4+ is supported.

Is kaffeine intended to be for fc4 and below ?

-- 
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 213906] Review Request: Pound - a reverse proxy and load balancer

2006-11-04 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: Pound - a reverse proxy and load balancer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 12:01 EST ---
Needswork :
 - Do not repeat the name in the Summary
 - Please create an init script for pound (you can base it on
/etc/init.d/transparent-proxying in the squidGuard package for example)
 - Please make a default config file for /etc/pound.cfg, with a very basic setup
(like the second example in the pound(8) man page), but with the control socket
and running as a different user
  - This implies that you create a directory for Pound's control sockets, for
example /var/lib/pound, and that you own it in the rpm
  - It also implies that you create a pound user in %pre, see the Zope package
for an example)

Possible improvements:
 - Generate a dummy SSL cert in /etc/pki/tls/certs/pound.pem, you can look at
the %post script in the openldap-servers package for a way to do it.
 - Add an SELinux policy, but that's a bit more complicated

-- 
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 199108] Review Request: gutenprint: Printer Drivers Package

2006-11-04 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: gutenprint: Printer Drivers Package


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 12:09 EST ---
(In reply to comment #90)
 I've installed the packages on extras and everything appears to be working 
 fine
 with the exception of replacement of the gimp-print plugin.  I tried removing
 the gimp-print-plugin provided in FC6 core, but then realized their doesn't
 appear to be an rpm available to replace it.  Is one in the works, or is this
 deferred to FC7?  Thanks!
yes extras version of gutenprint is meant for parallel installation of old
gimp-print package along with new gutenprint package. Check more in gutenprint's
SPEC file about what is set for FC7 and what excluded in extras.
http://cvs.fedora.redhat.com/viewcvs/rpms/gutenprint/devel/gutenprint.spec?root=extrasview=markup



-- 
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 209311] Review Request: espeak - Software speech synthesizer (text-to-speech)

2006-11-04 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: espeak - Software speech synthesizer (text-to-speech)


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 12:21 EST ---
New build:
Spec URL: http://dialogpalette.sourceforge.net/extras/fedora/espeak.spec
SRPM URL: 
http://dialogpalette.sourceforge.net/extras/fedora/espeak-1.16-3.src.rpm

Changes:
- Fixed source file permissions for -debuginfo package in %prep
- Added RPM_OPT_FLAGS to make command in %build; removed RPM_OPT_FLAGS
makefile patch
- Modified makefile install target patch to include general support for setting
compiler optimization flags via CXXFLAGS
- Removed creation of .orig backup files during patching
- Modified patch files to have different suffixes


This time, rpmlint is truly silent ;-) I'm not too sure what you mean by
different suffixes for different patch files; I've resorted to using a suffix
corresponding to the patch ID number in the spec (e.g. somefile.patch0).
Thanks for the feedback!

-- 
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 209311] Review Request: espeak - Software speech synthesizer (text-to-speech)

2006-11-04 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: espeak - Software speech synthesizer (text-to-speech)


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 12:29 EST ---
Well, what I meant was adding some different suffixes each time you apply 
patches
are preferred and I didn't meant you have to change the name of the patches.

Like:

%patch0 -p1 -b .pthread
%patch1 -p1 -b .install_target
%patch2 -p1 -b .sizeof


to make it easy that you can check how the patches were applied afterwards.

(I have not yet rebuilt the srpm you re-uploaded

-- 
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 213662] Review Request: openmpi - Upstream MPI package with native InfiniBand support

2006-11-04 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: openmpi - Upstream MPI package with native InfiniBand 
support


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 13:46 EST ---
Oops, sorry I don't understand this; last time I checked openmpi was bundled
with FC6 and has been in devel (rawhide) for a little while.

-- 
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 214024] New: Review Request: perl-Net-DBus - Net::DBus Perl module

2006-11-04 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=214024

   Summary: Review Request: perl-Net-DBus - Net::DBus Perl module
   Product: Fedora Extras
   Version: devel
  Platform: All
   URL: http://search.cpan.org/dist/Net-DBus/
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


SRPM URL: http://home.comcast.net/~ckweyl/perl-Net-DBus-0.33.3-1.fc6.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-Net-DBus.spec

Description:
Net::DBus provides a Perl API for the DBus message system. The DBus Perl
interface is currently operating against the 0.32 development version of
DBus, but should work with later versions too, providing the API changes
have not been too drastic.

-- 
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 214024] Review Request: perl-Net-DBus - Net::DBus Perl module

2006-11-04 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-Net-DBus - Net::DBus Perl module


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


[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 214025] New: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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=214025

   Summary: Review Request: Tracker -  An object database,
tag/metadata database, search tool and indexer
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.0-1.src.rpm
Description: Tracker is a powerful desktop-neutral first class object database, 
tag/metadata database, search tool and indexer. 

Tracker is also extremely fast and super efficient with your systems memory when
compared with some other competing frameworks and is by far the fastest and most
memory efficient Nautilus search and Deskbar backends currently availble.

It consists of a common object database that allows entities to have an almost
infinte number of properties, metadata (both embedded/harvested as well as user
definable), a comprehensive database of keywords/tags and links to other 
entities.

It provides additional features for file based objects including context linking
and audit trails for a file object.

It has the ability to index, store, harvest metadata. retrieve and search all 
types of files and other first class objects.

-- 
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 214025] Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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: Tracker -  An object database, tag/metadata database, 
search tool and indexer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 17:17 EST ---
Where did you get tracker 0.5.0 from? Upstream has only 0.0.4, and the URL
listed in the spec file does not work.

-- 
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 214025] Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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: Tracker -  An object database, tag/metadata database, 
search tool and indexer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 17:20 EST ---
Uh, nevermind, I typed in 0.0.5 by mistake. Jamie really should update the
Tracker website as well.

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

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


[Bug 214025] Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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: Tracker -  An object database, tag/metadata database, 
search tool and indexer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 17:58 EST ---
I had started to review it, but if you want to do it, here's what I found out so
far:

Needs work:
* Missing SMP flags. If it doesn't build with it, please add a comment
  (wiki: PackagingGuidelines#parallelmake)
* Build failed in mock. Missing BR: pango-devel, dbus-devel, dbus-glib
* Missing BR for metadata extractors : libgnomeui-devel, gnome-vfs2-devel,
gnome-desktop-devel, poppler, libvorbis-devel, libpng-devel, libexif-devel,
libgsf-devel

Possible improvements :
* Have trackerd start on login with a desktop file in
/usr/share/gnome/autostart/ (GNOME) and in /usr/share/autostart/ (KDE)
Is there a freedesktop spec for this by the way ?


-- 
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 214025] Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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: Tracker -  An object database, tag/metadata database, 
search tool and indexer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 18:14 EST ---
Created an attachment (id=140370)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=140370action=view)
Diff to tracker.spec

Weird, the package I downloaded has libexif-devel, libgsf-devel etc. as
BuildRequires. pango-devel is not required as it's pulled in by
gnome-desktop-devel.

Missing BuildRequires:
dbus-devel dbus-glib-devel gnome-utils-devel libpng-devel openssl-devel

Description has some misspelled words, and summary should not end in '.'; also,
since libraries are included you need to call ldconfig on %post and %postun

Do we want to split the libraries into separate packages?

Actually, Aurelien, you can take over the review if you want. I'm less familiar
with autostarting.


-- 
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 214025] Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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: Tracker -  An object database, tag/metadata database, 
search tool and indexer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 18:15 EST ---
Created an attachment (id=140371)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=140371action=view)
New tracker.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 214025] Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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: Tracker -  An object database, tag/metadata database, 
search tool and indexer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 18:25 EST ---
(In reply to comment #3)
 I had started to review it, but if you want to do it, here's what I found out 
 so
 far:
 
 Needs work:
 * Missing SMP flags. If it doesn't build with it, please add a comment
   (wiki: PackagingGuidelines#parallelmake)
 * Build failed in mock. Missing BR: pango-devel, dbus-devel, dbus-glib
 * Missing BR for metadata extractors : libgnomeui-devel, gnome-vfs2-devel,
 gnome-desktop-devel, poppler, libvorbis-devel, libpng-devel, libexif-devel,
 libgsf-devel

Really me culpa. After I initially uploaded the src.rpm, I realized I had not
tested it in mock, so I quickly updated the files (without bumping the version)
thinking no one is reviewing yet.
Updated files are here;
  Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.0-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 214025] Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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: Tracker -  An object database, tag/metadata database, 
search tool and indexer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 19:04 EST ---
(In reply to comment #4)
 Created an attachment (id=140370)
 -- 
(https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=140370action=view) 
[edit]
 Diff to tracker.spec
 
 Weird, the package I downloaded has libexif-devel, libgsf-devel etc. as
 BuildRequires. pango-devel is not required as it's pulled in by
 gnome-desktop-devel.
 
 Missing BuildRequires:
 dbus-devel dbus-glib-devel gnome-utils-devel libpng-devel openssl-devel


Most of these are already pulled in by other dependencies. But why is
gnome-utils-devel and openssl-devel needed, I can't find anything in the
configure log that that suggest they are 
 
 Description has some misspelled words, and summary should not end in '.'; 
 also,

Gone thru and corrected a bunch (I had copied it directly from the README).

 since libraries are included you need to call ldconfig on %post and %postun


Done. 

 Do we want to split the libraries into separate packages?
 
 Actually, Aurelien, you can take over the review if you want. I'm less 
 familiar
 with autostarting.

I've also commented the build fails with the smp flags.
I'll work on the autostarting stuff tomorrow.

Thanks for reviewing. 



-- 
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 214025] Review Request: Tracker - An object database, tag/metadata database, search tool and indexer

2006-11-04 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: Tracker -  An object database, tag/metadata database, 
search tool and indexer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-04 20:57 EST ---
I found them by calling ./configure --help and looking at the options that can
be specified. It's probably not accurate, since it missed certain things that
configure actually checks for. Will look at it again tomorrow.

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