[Bug 205300] Review Request: gtk-sharp - a set of mono bindings for gtk1.2

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

Summary: Review Request: gtk-sharp - a set of mono bindings for gtk1.2


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 02:54 EST ---
gtk-sharp *IS* in Fedora Core 5.  Thus it makes no sense to branch it there and
rebuild it in Extras for Fedora Core 5.

It does make sense to do this for FC6.

Do you concur?


-- 
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 206487] Review Request: jd - A 2ch browser

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

Summary: Review Request: jd - A 2ch browser


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 02:49 EST ---
Thank you for reviewing and approving this package!!
Now I will try to import this to cvs.

(In reply to comment #6) 
> Mamuro can you remove AUTHORS from %doc since it's empty ?
I think AUTHORS file is not included in jd (binary) package.

> 
> The case of missing gtkmm24, you are right. There will be less breakage via 
> yum?
Yes. Actually I installed this (jd) by using yum as following.

When I want to review a package or to install some packages rebuilt by
myself,

* I usually rebuild it in mock, then the binary rpms are created.
* Then I usually move all binary rpms (which are rebuilt by me in mock) to
  some directory (for me /var/lib/mock/LOCALRPMS/i386)
* Move to /var/lib/mock/LOCALRPMS/i386, then I do:

chmod 0644 *rpm ; createrepo $(pwd) ; chmod 0755 repodata/ ; chmod 0644 
repodata/*

  then repository metadata is created (createrepo rpm is in CORE).
* I have the following repository entry in /etc/yum.repos.d/LOCAL.repo:
--
[LOCAL]
name=LOCAL - locally created rpms
baseurl=file:///var/lib/mock/LOCALRPMS/i386/
enabled=1
gpgcheck=0
--

* Then I can install rpms rebuilt locally by me with yum.

-- 
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 205300] Review Request: gtk-sharp - a set of mono bindings for gtk1.2

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

Summary: Review Request: gtk-sharp - a set of mono bindings for gtk1.2


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 02:46 EST ---
There isn't anything in FC-5 or rawhide which uses gtk-sharp anymore. Have a
word with caillon and alexl.

The removal of gtk-sharp-1.0.10 was taken about a month back and before it was
placed on the orphan list, I snapped it up. It is now an FE package, so by
default only has a rawhide entry.

-- 
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 206487] Review Request: jd - A 2ch browser

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

Summary: Review Request: jd - A 2ch browser


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  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 206487] Review Request: jd - A 2ch browser

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

Summary: Review Request: jd - A 2ch browser


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 02:29 EST ---
MUST Items:

- MUST: rpmlint's output is clean
- MUST: The package is named according to the Package Naming Guidelines.
- MUST: The spec file name matches the base package %{name}
- MUST: The package meets the Packaging Guidelines.
- MUST: The package is licensed (GPL) with an open-source compatible license and
meet other legal requirements as defined in the legal section of Packaging
Guidelines.
- MUST: The License field in the package spec file matches the actual license.
- MUST: the source package includes the text of the license(s) in its own file,
then that file, containing the text of the license(s) for the package is
included in %doc.
- MUST: The spec file must be written in American English.
- MUST: The spec file for the package is be legible. 
- MUST: The sources used to build the package must matches the upstream source,
as provided in the spec URL.
- MUST: The package successfully compiles and builds into binary rpms on at
least i386.
- MUST: All build dependencies is listed in BuildRequires.
- No locales present
- MUST: If the package does not contain shared library files located in the
dynamic linker's default paths
- MUST: the package is not designed to be relocatable
- MUST: the package owns all directories that it creates.
- MUST: the package does not contain any duplicate files in the %files listing.
- MUST: Permissions on files are set properly.
- MUST: The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
- MUST: The package consistently uses macros, as described in the macros section
of Packaging Guidelines.
- MUST: The package contains code, or permissable content. This is described in
detail in the code vs. content section of Packaging Guidelines.
- MUST: There are no Large documentation files
- MUST: %doc does not affect the runtime of the application. To summarize: If it
is in %doc, the program must run properly if it is not present.
- MUST: There are no Header files or static libraries 
- MUST: The package does not contain library files with a suffix 
- MUST: Package does NOT contain any .la libtool archives
- MUST: Package containing GUI applications includes a %{name}.desktop file, and
that file must be properly installed with desktop-file-install in the %install
section.
- MUST: Package does not own files or directories already owned by other 
packages. 

SHOULD Items:

 - SHOULD: The source package does include license text(s) as COPYING
 - SHOULD: mock builds succcessfully in i386.
 - SHOULD: The reviewer tested that the package functions as described. A
package should not segfault instead of running, for example.
 - SHOULD: No subpackages present.

Mamuro can you remove AUTHORS from %doc since it's empty ?

The case of missing gtkmm24, you are right. There will be less breakage via yum?

This is package has been 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 208034] Review Request: HippoDraw - Interactive and Python scriptable data analysis application

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

Summary: Review Request: HippoDraw - Interactive and Python scriptable data 
analysis application


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 01:12 EST ---
*** Bug 208032 has been marked as a duplicate of 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 208032] Review Request: HippoDraw - Interactive and Python scriptable data analysis application

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

Summary: Review Request: HippoDraw - Interactive and Python scriptable data 
analysis application


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 01:12 EST ---


*** This bug has been marked as a duplicate of 208034 ***

-- 
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 204140] Review Request: libmtp - MTP client library

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

Summary: Review Request: libmtp - MTP client library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 01:09 EST ---
MUST FIX:
* *-devel ships a *.pc
=> Missing "Requires: pkgconfig" in *-devel

BUG:
* The *.pc being shipped should "Require: libusb"
instead of hard-coding -lusb in Libs.

SHOULD:
* Use make DESTDIR=... install instead of %makeinstall

-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 00:26 EST ---
Tarball edited to remove goo/vms_*

New SRPM: http://www.auroralinux.org/people/spot/review/xpdf-3.01-26.fc6.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/xpdf.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 207763] Review Request: ddd - GUI for several command-line debuggers

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

Summary: Review Request: ddd - GUI for several command-line debuggers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 00:12 EST ---
Added the BR: libtool
Fixed the mixed tab spaces.
Ignored the info.gz files.

New SRPM: 
http://www.auroralinux.org/people/spot/review/ddd-3.3.11-10.fc6.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/ddd.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 177512] Review Request: mysql-connector-net

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

Summary: Review Request: mysql-connector-net


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-26 00:06 EST ---
OK - Package name
OK - Spec file matches base package name.
OK - Meets Packaging Guidelines.
OK - License (GPL)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
153f4bb405af5ae59565084a9ce6e1b1  mysql-connector-net-1.0.7-noinstall.zip
153f4bb405af5ae59565084a9ce6e1b1  mysql-connector-net-1.0.7-noinstall.zip.1
OK - Package compiles and builds on at least one arch.
OK - BuildRequires correct
OK - Package owns all the directories it creates.
OK - Package has no duplicate files in %files.
See below - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Spec has consistant macro usage.
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.
OK - Package doesn't own any directories other packages own.
See below - No rpmlint output.

SHOULD Items:

OK - Should include License or ask upstream to include it.
OK - Should build in mock.

Issues:

1. Might change defattr from:
%defattr(-,root,root)
to
%defattr(-,root,root,-)

2. rpmlint says:

E: mysql-connector-net no-binary
E: mysql-connector-net only-non-binary-in-usr-lib

I think those can be ignored, as rpmlint doesn't know mono bins.

W: mysql-connector-net wrong-file-end-of-line-encoding /usr/share/doc/mysql-
connector-net-1.0.7/COPYING
W: mysql-connector-net wrong-file-end-of-line-encoding /usr/share/doc/mysql-
connector-net-1.0.7/EXCEPTIONS
W: mysql-connector-net wrong-file-end-of-line-encoding /usr/share/doc/mysql-
connector-net-1.0.7/CHANGES
W: mysql-connector-net wrong-file-end-of-line-encoding /usr/share/doc/mysql-
connector-net-1.0.7/README

Should run a quick:
%{__sed} -i 's/\r//'
on those files?

E: mysql-connector-net hardcoded-library-path in %{_prefix}/lib

I think this is due to the core mono packaging?

W: mysql-connector-net rpm-buildroot-usage %build rm -rf %{buildroot}

That rm -rf %{buildroot} should at the top of 'install' and not 'build' ?

E: mysql-connector-net-debuginfo empty-debuginfo-package

Why no debuginfo files? Other mono packages have them.


-- 
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 205075] Review Request: fwbackups - a user backup program, with support for automated backups and on-demand backups

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

Summary: Review Request: fwbackups - a user backup program, with support for 
automated backups and on-demand backups


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 23:42 EST ---
Hmm, the tarball does not match upstream:

c987096dc11e2605f4a73c19484a4a9b  fwbackups-1.42.tar.gz
10c8fda1c5809681aa870a2c71f84ed5  ../fwbackups-1.42.tar.gz

The files are different sizes as well (31347 in the srpm, 32282 upstream).

I'm not sure why you have constructs like this in %install:
   install -p -d -m755 etc/fwbackups\
 ${RPM_BUILD_ROOT}%{_sysconfdir}/fwbackups

install -d creates each of its arguments as directories, but "etc/fwbackups"
already exists because it's in your source tarball.

You install fwbackups.conf twice.

I think you could significantly shrink your %install section with by using "cp
-rp" instead of installing each file separately, but I suppose that's up to you.

Note that there's currently some discussion about changing the recommendations
for installing desktop files, but that won't be decided for another several days
and I don't think it will effect your package.

Are you sure fwbackups.conf.default should go in /etc?  It should certainly not
be marked noreplace as you want it to change on an upgrade (since the old
version isn't the default any longer).  I would consider marking it %doc 
instead.

-- 
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 204140] Review Request: libmtp - MTP client library

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

Summary: Review Request: libmtp - MTP client library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 23:38 EST ---
OK - Package name
OK - Spec file matches base package name.
OK - Meets Packaging Guidelines.
OK - License (LGPL)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
5d22b16cb7e6a117cdf489669821df09  libmtp-0.0.15.tar.gz
5d22b16cb7e6a117cdf489669821df09  libmtp-0.0.15.tar.gz.1
OK - Package compiles and builds on at least one arch.
OK - BuildRequires correct
OK - Spec has needed ldconfig in post and postun
OK - Package owns all the directories it creates.
OK - Package has no duplicate files in %files.
See below - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Spec has consistant macro usage.
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.
OK - Headers/static libs in -devel subpackage.
OK - .pc files in -devel subpackage.
OK - .so files in -devel subpackage.
OK - -devel package Requires: %{name} = %{version}-%{release}
OK - .la files are removed.
OK - Package doesn't own any directories other packages own.
See below - No rpmlint output.

SHOULD Items:

OK - Should include License or ask upstream to include it.
OK - Should build in mock.
OK - Should have subpackages require base package with fully versioned depend.

Issues:

1. 0.0.19 is now available.

2. Might change the defattr from:
%defattr(-, root, root)
to
%defattr(-, root, root,-)

3. rpmlint says:

W: libmtp no-documentation
W: libmtp-examples no-documentation

Can be ignored.

W: libmtp mixed-use-of-spaces-and-tabs

Should clean up spec to only use tabs or spaces, not both.


-- 
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 208051] New: Review Request: dirvish - Fast, disk based, rotating network backup system

2006-09-25 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=208051

   Summary: Review Request: dirvish - Fast, disk based, rotating
network backup system
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://www.marcanoonline.com/downloads/fedora/package_submissions/dirvish/dirvish.spec
SRPM URL: 
http://www.marcanoonline.com/downloads/fedora/package_submissions/dirvish/dirvish-1.2.1-1.src.rpm
Description:
Dirvish is a fast, disk based, rotating network backup system. With dirvish you
can maintain a set of complete images of your filesystems with unattended
creation and expiration. A dirvish backup vault is like a time machine for your
data.

-- 
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 208006] Review Request: perl-Time-Period - A Perl module to deal with time periods

2006-09-25 Thread bugzilla
Please 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-Time-Period - A Perl module to deal with time 
periods


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


[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 196591] Review Request: bitlbee

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

Summary: Review Request: bitlbee


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 23:19 EST ---
I am still testing it. I ended up not trusting the old copy of the otrproxy I
used for testing, so I ended up upgrading that (and turning it into an FE 
package).
I should hopefully finish testing 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


[Bug 199108] Review Request: gutenprint: Printer Drivers Package

2006-09-25 Thread bugzilla
Please 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-09-25 23:15 EST ---
Oddly, that error only happens on devel/fc6. On fc5, the above patch gets it 
compiling on x86_64 just fine. ;( 

For folks that would like to test, I have put up some repos for fc5{i386/
x86_64} and fc6(i386 only for now): 

--cut--
[gutenprint-test]
name=gutenprint testing rpms
baseurl=http://www.scrye.com/gutenprint/$releasever/$basearch/
enabled=1
gpgcheck=0
--cut--


-- 
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 205075] Review Request: fwbackups - a user backup program, with support for automated backups and on-demand backups

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

Summary: Review Request: fwbackups - a user backup program, with support for 
automated backups and on-demand backups


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 199108] Review Request: gutenprint: Printer Drivers Package

2006-09-25 Thread bugzilla
Please 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-09-25 22:53 EST ---
ok, looking at the cups.spec from core, what they do is at the top define: 

%define cups_serverbin %{_exec_prefix}/lib/cups

I think you should do likewise and then replace the %{_libdir} in the cups 
package and the directories you remove from the cups package with the macro 
above... 

I tried something like this:

--- gutenprint.spec.1   2006-09-08 06:34:57.0 -0600
+++ gutenprint.spec 2006-09-25 20:12:30.0 -0600
@@ -1,4 +1,5 @@
 %define build_with_ijs_support 1
+%define cups_serverbin %{_exec_prefix}/lib/cups
 
 Name:   gutenprint
 Summary:Printer Drivers Package
@@ -223,9 +224,9 @@
 rm -rf %{buildroot}%{_bindir}/escputil
 rm -rf %{buildroot}%{_mandir}/man1/escputil.1*
 rm -rf %{buildroot}%{_libdir}/gimp/2.0/plug-ins/print
-rm -rf %{buildroot}%{_libdir}/cups/backend/*
-rm -rf %{buildroot}%{_libdir}/cups/filter/commandtocanon
-rm -rf %{buildroot}%{_libdir}/cups/filter/commandtoepson
+rm -rf %{buildroot}%{cups_serverbin}/backend/*
+rm -rf %{buildroot}%{cups_serverbin}/filter/commandtocanon
+rm -rf %{buildroot}%{cups_serverbin}/filter/commandtoepson
 rm -rf %{buildroot}%{_bindir}/cups-calibrate
 rm -rf %{buildroot}%{_mandir}/man8/cups-calibrate.8*
 
@@ -304,10 +305,10 @@
 %defattr(-, root, root,-)
 %config(noreplace) %{_sysconfdir}/cups/command.types
 %{_datadir}/cups/calibrate.ppm
-#%{_libdir}/cups/backend/*
-#%{_libdir}/cups/filter/*
+#%{cups_serverbin}/cups/backend/*
+#%{cups_serverbin}/cups/filter/*
 #%{_bindir}/cups-calibrate
-%{_libdir}/cups/filter/rastertogutenprint.5.0
+%{cups_serverbin}/filter/rastertogutenprint.5.0
 %{_sbindir}/cups-genppd*
 %{_datadir}/cups/model/gutenprint/5.0/C
 #%{_mandir}/man8/cups-calibrate.8*

That gets it further, but it's now dying in a very odd way making the ppd 
files. If you would like access to my x86_64 test box to do test builds, drop 
me an email and I will get you access there. 


-- 
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 208007] Review Request: perl-Alien-wxWidgets - Building, finding and using wxWidgets binaries

2006-09-25 Thread bugzilla
Please 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-Alien-wxWidgets - Building, finding and using 
wxWidgets binaries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 22:15 EST ---
The build does complain about many libraries (which seem to be wxwidgets
components) not being found but this seems to have no effect on that resulting
package.

* source files match upstream:
   6975385d6e4d4d58c5d589c2a7077d23  Alien-wxWidgets-0.21.tar.gz
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text not included upstream.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane:
   perl(Alien::wxWidgets) = 0.21
   perl(Alien::wxWidgets::Config::gtk2_2_6_3_uni_gcc_3_4)
   perl(Alien::wxWidgets::Utility)
   perl-Alien-wxWidgets = 0.21-1.fc6
  =
   perl(:MODULE_COMPAT_5.8.8)
   perl(Alien::wxWidgets::Utility)
   perl(Carp)
   perl(Config)
   perl(Module::Pluggable)
   perl(base)
   perl(strict)
* %check is present and all tests pass:
   All tests successful.
   Files=2, Tests=17,  0 wallclock secs ( 0.11 cusr +  0.06 csys =  0.17 CPU)
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

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 208007] Review Request: perl-Alien-wxWidgets - Building, finding and using wxWidgets binaries

2006-09-25 Thread bugzilla
Please 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-Alien-wxWidgets - Building, finding and using 
wxWidgets binaries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 205300] Review Request: gtk-sharp - a set of mono bindings for gtk1.2

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

Summary: Review Request: gtk-sharp - a set of mono bindings for gtk1.2


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 21:52 EST ---
FC5 has gtk-sharp.  Why did you request a branch for FE5?

-- 
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 208006] Review Request: perl-Time-Period - A Perl module to deal with time periods

2006-09-25 Thread bugzilla
Please 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-Time-Period - A Perl module to deal with time 
periods


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 21:41 EST ---
* source files match upstream:
   63b073af8ec96e7fa48801cd6fcccdd1  Period-1.20.tar.gz
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text not included upstream.
* latest version is being packaged.
* BuildRequires are proper (BR: perl is unnecessary)
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane:
   perl(Time::Period) = 1.20
   perl-Time-Period = 1.20-1.fc6
  =
   perl >= 0:5.001
   perl(:MODULE_COMPAT_5.8.8)
   perl(Exporter)
* %check is present but the package has no tests to run.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

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 205929] Review Request: libfreebob - FreeBoB firewire audio driver library

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

Summary: Review Request: libfreebob - FreeBoB firewire audio driver library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 21:26 EST ---
(In reply to comment #1)
> I'm glad this is in process of getting included in Extras finally. I also own 
> a 
> PreSonus FireBox, great card :)
> 
> I have been providing FreeBoB packages along with JACK and relevant libs for 
> FC5 for a while. They are here: http://www.ffri.hr/~vmiletic/linux/fedora/

Oh, cool.  I'll look at your package.  BTW, contributing to Extras is pretty
straight forward for anybody to do.
 
> You say you bumped libavc1394. 

Yes, I got the maintainer to push a newer version into FC.

> It also needs libiec61883 1.1.0, afaik. I also 
> packaged svn version of libraw1394, as suggested on FreeBoB page.

When I asked about this I was told that it wasn't all that important, and  I
could just tweak the configury to require the older version we have in FC.

> And last, but not the least: do you also plan to add FireWire soundcard 
> detection support to system-config-soundcard at some point?

I wasn't planning on it.  My understanding is that there's some upheaval in the
works for audio in Fedora (see PulseAudio).   Whatever is done, should probably
be done in the context of PulseAudio.



-- 
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 208006] Review Request: perl-Time-Period - A Perl module to deal with time periods

2006-09-25 Thread bugzilla
Please 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-Time-Period - A Perl module to deal with time 
periods


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 208004] Review Request: perl-Test-Perl-Critic - Use Perl::Critic in test programs

2006-09-25 Thread bugzilla
Please 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-Test-Perl-Critic - Use Perl::Critic in test 
programs


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 21:02 EST ---
* source files match upstream:
   91c1dc234b4c6ec1f06300085f0559cf  Test-Perl-Critic-0.07.tar.gz
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane:
   perl(Test::Perl::Critic) = 0.07
   perl-Test-Perl-Critic = 0.07-1.fc6
  =
   perl(:MODULE_COMPAT_5.8.8)
   perl(Carp)
   perl(English)
   perl(File::Spec)
   perl(Perl::Critic)
   perl(Perl::Critic::Utils)
   perl(Test::Builder)
   perl(strict)
   perl(warnings)
* %check is present and all tests pass:
   All tests successful.
   Files=7, Tests=14,  3 wallclock secs ( 3.09 cusr +  0.39 csys =  3.48 CPU)
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

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 208004] Review Request: perl-Test-Perl-Critic - Use Perl::Critic in test programs

2006-09-25 Thread bugzilla
Please 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-Test-Perl-Critic - Use Perl::Critic in test 
programs


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 187706] Review Request: alsa-oss - Userspace OSS emulation

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

Summary: Review Request: alsa-oss - Userspace OSS emulation


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  QAContact|fedora-extras-  |fedora-package-
   |[EMAIL PROTECTED] |[EMAIL PROTECTED]

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 20:52 EST ---
There's a bunch of applications that use only OSS (Flash and Skype come to
mind), and no amount of tinkering with asound.conf / .asoundrc would allow me to
dmix them, so no other audio stream can be played while one of those apps is
running.

AOSS, on the other hand, has no problem dmixing multiple OSS streams, and I
didn't have to tweak anything. It should surely be brought back.

See this thread on Fedora forum for more information:
http://www.fedoraforum.org/forum/showthread.php?p=615401

-- 
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 207927] Review Request: supertuxkart - Kids 3D go-kart racing game featuring Tux

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

Summary: Review Request: supertuxkart - Kids 3D go-kart racing game featuring 
Tux


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 20:39 EST ---
OK - Package name
OK - Spec file matches base package name.
OK - Meets Packaging Guidelines.
OK - License (GPL)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
No, but ok - Sources match upstream md5sum:
(images removed)
OK - Package compiles and builds on at least one arch.
OK - BuildRequires correct
OK - Package owns all the directories it creates.
OK - Package has no duplicate files in %files.
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Spec has consistant macro usage.
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.
OK - Package is a GUI app and has a .desktop file
OK - Package doesn't own any directories other packages own.
See below - No rpmlint output.

SHOULD Items:

OK - Should include License or ask upstream to include it. 
OK - Should build in mock.

Issues:

1. Perhaps 'images-leagal.txt' could be renamed 'supertuxkart-images-legal.txt'
just to make it easier to find if you unpack the src.rpm? (just a minor nit).
Has any move been made to get upstream to change images to make it more
distributable moving forward?

2. Your provides and obsoletes don't look quite right to me.
Perhaps that should be:

Provides: tuxkart = %{version}-%{release}
Obsoletes: tuxkart <= 0.4.0-6

I'm looking at:
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-
581c3fb3ff1c6ef7404e8b288b59cd5280d75ad6

3. rpmlint says:
supertuxkart incoherent-version-in-changelog 0.4.0-5 0.2-1.fc6
(the .fc6 doesn't need to be there, just the version... )


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

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


[Bug 207927] Review Request: supertuxkart - Kids 3D go-kart racing game featuring Tux

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

Summary: Review Request: supertuxkart - Kids 3D go-kart racing game featuring 
Tux


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 20:23 EST ---
ok. I would be happy to (re)review this. 

Look for a full review in a bit here... 

-- 
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 207805] Review Request: skey - An S/Key implementation

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

Summary: Review Request: skey - An S/Key implementation


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 20:18 EST ---
Given the confusion over the license, perhaps these bits of clarification could
find their way into the package so there's no question later.  (Perhaps I'm
being dense here, but comment #6 seems to incorporate the logdaemon license by
reference.  It might be good to include it as well.)

I agree that it would be good to split the package into skey-clients and
skey-server (or whatever is reasonable) to get the setuid stuff away from what
most people would need to install.  The thought of very old source and setuid
bits makes me pucker, but you're the maintainer so it's your call.

-- 
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 205300] Review Request: gtk-sharp - a set of mono bindings for gtk1.2

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

Summary: Review Request: gtk-sharp - a set of mono bindings for gtk1.2


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 19:22 EST ---
gtk-sharp has been moved from core to extras as there is nothing in core which
uses it anymore.

-- 
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 208034] Review Request: HippoDraw - Interactive and Python scriptable data analysis application

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

Summary: Review Request: HippoDraw - Interactive and Python scriptable data 
analysis application


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |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 208034] New: Review Request: HippoDraw - Interactive and Python scriptable data analysis application

2006-09-25 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=208034

   Summary: Review Request: HippoDraw - Interactive and Python
scriptable data analysis application
   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://ftp.slac.stanford.edu/users/pfkeb/hippodraw/HippoDraw.spec
SRPM URL: 
ftp://ftp.slac.stanford.edu/users/pfkeb/hippodraw/HippoDraw-1.18.5-1.src.rpm
Description: 
HippoDraw is a highly interactive data analysis
environment that can be used as a stand-a-lone application or as a
Python extension module.

This is my first package and am seeking sponsor.

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

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


[Bug 207846] Review Request: perl-Finance-YahooQuote - Perl interface to get stock quotes from Yahoo! Finance

2006-09-25 Thread bugzilla
Please 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-Finance-YahooQuote - Perl interface to get stock 
quotes from Yahoo! Finance
Alias: finance-YahooQuote

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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 18:57 EST ---
I've looked into this problem a bit more and the root.log is showing that using
perl(HTTP::Request::Common) brings in the mod_perl package instead of the
perl-libwww-perl package.

Should mod_perl be Providing perl(HTTP::Request::Common)?


-- 
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 208032] New: Review Request: HippoDraw - Interactive and Python scriptable data analysis application

2006-09-25 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=208032

   Summary: Review Request: HippoDraw - Interactive and Python
scriptable data analysis application
   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://ftp.slac.stanford.edu/users/pfkeb/hippodraw/HippoDraw.spec
SRPM URL: 
ftp://ftp.slac.stanford.edu/users/pfkeb/hippodraw/HippoDraw-1.18.5-1.src.rpm
Description: 
HippoDraw is a highly interactive data analysis
environment that can be used as a stand-a-lone application or as a
Python extension module.

-- 
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 203694] Review Request: rawstudio - digital camera raw-image converter

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

Summary: Review Request: rawstudio - digital camera raw-image converter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 18:52 EST ---
Package imported and built for FC-5 and devel branches

-- 
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 193867] Review Request: klamav - Clam Anti-Virus on the KDE Desktop

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

Summary: Review Request: klamav - Clam Anti-Virus on the KDE Desktop


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 18:46 EST ---
drop bzip2-devel as well as BuildRequires, as klamav builds successfully without
it in mock.

-- 
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 207763] Review Request: ddd - GUI for several command-line debuggers

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

Summary: Review Request: ddd - GUI for several command-line debuggers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 18:40 EST ---
Now missing a br on libtool.

Otherwise passes rpmlint other than mixed tabs/spaces and info.gz files not
being in utf8 format.

-- 
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 207846] Review Request: perl-Finance-YahooQuote - Perl interface to get stock quotes from Yahoo! Finance

2006-09-25 Thread bugzilla
Please 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-Finance-YahooQuote - Perl interface to get stock 
quotes from Yahoo! Finance
Alias: finance-YahooQuote

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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 18:12 EST ---
Warren:  I tried to build this under mock with %check enabled and a BR of
perl(HTTP::Request::Common).  This also results in a failure and warnings
mentioned in the build log of missing packages.  By using a BR of
perl(HTTP::Request), this fixes the problem.

perl(HTTP::Request) needs to be added to the BR in order for %check to function
and should be added even though it is not used for official builds.  A developer
who enables the %check section would not automatically know this otherwise.

The build log also indicates that rpm is adding a Requires of
perl(HTTP::Request::Common), however, when I tried using this for a BR, it fails
and the following warnings are shown:
Warning: prerequisite HTML::Parser 2.2 not found.
Warning: prerequisite HTTP::Request 1.23 not found.
Warning: prerequisite LWP::UserAgent 1.62 not found.

These packages do not show up as being required by rpm.  Therefore, I think a
"Requires: perl(HTTP::Request)" also needs to be added to the spec file.  Please
add this unless you can show that it is not required.


 Review Checklist 
- rpmlint output clean
- package named according to package naming guidelines
- filename matches spec's %{name}
- package meets packaging guidelines
- package licensed with opensource compatible license
- license in spec matches actual license
- license included in %doc
- spec written in American english
- spec file legible
- sources match upstream
844d26ef02f16216840aac5815a6a6a7  Finance-YahooQuote-0.21.tar.gz
- package successfully compiles and builds on FC5 x86_64
- package does not contain locales
- package does not contain shared libraries/static libraries/pkgconfig files or
any other files required for a devel subpackage
- package is not relocatable
- package does not contain duplicate files in %files
- permissions on files set properly
- package contains proper %clean section
- macro usage is consistent
- package contains code
- no large documentation files
- files in %doc do not affect runtime
- no need for .desktop files

=== MUST ===
- Please address concerns I have about BuildRequires and Requires issues I
mention above.

-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 18:12 EST ---
(In reply to comment #23)

> The goo/vms_* files aren't used at all. I fear the problematic files
> goo/vms_directory.c and goo/vms_*dirent.h should be removed.

And goo/vms_unix_time.h also should certainly be 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 207079] Review Request: perl-Cache - The Cache interface

2006-09-25 Thread bugzilla
Please 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-Cache -  The Cache interface


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 18:07 EST ---
the FC-5 package is available 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 205300] Review Request: gtk-sharp - a set of mono bindings for gtk1.2

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

Summary: Review Request: gtk-sharp - a set of mono bindings for gtk1.2


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 18:03 EST ---
Why do you want a FE5 branch when this is already in FC5?


-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 17:49 EST ---
It is right now. I looked at all the files to find other 
licence issues. 

in goo/, some vms_* files have no licence and the copyright:

* vms_directory.c:   Patrick L. Mahan
* vms_unix_times.c:  ?
* vms_unlink.c:  Thanks to Patrick Moreau ([EMAIL PROTECTED]).
* vms_dirent.h:  @(#)dirent.h 1.7 89/06/25 SMI
* vms_sys_dirent.h:  @(#)dirent.h 1.4 89/06/16 SMI

The ones that seem problematic are goo/vms_directory.c and
goo/vms_*dirent.h, since they have a copyright and no licence.

With a licence, one have:
* vms_unix_time.h: 1982, 1986 Berkeley software License Agreement
It seems also problematic to me since I believe at that time 
it was the BSD incompatible with the GPL.

The file in splash don't have any copyright nor licence. That
is strange, but they can certainly be considered public domain,
so no problem here.


The goo/vms_* files aren't used at all. I fear the problematic files
goo/vms_directory.c and goo/vms_*dirent.h should be removed.


And the upstream should certainly be contacted for those
issues (and the CMap issues), although there is a comment in 
the README, about the GPL which seems very strange.

-- 
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 206487] Review Request: jd - A 2ch browser

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

Summary: Review Request: jd - A 2ch browser


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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 193867] Review Request: klamav - Clam Anti-Virus on the KDE Desktop

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

Summary: Review Request: klamav - Clam Anti-Virus on the KDE Desktop


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 17:24 EST ---
#1: remove zlib-devel as BuildRequires
since curl-devel requires openssl-devel and the later requires zlib-devel

#2: replace "Requires: %{_bindir}/freshclam" by "Requires: clamav-update"

#3: In %post and %postun
replace

if [ -x %{_bindir}/gtk-update-icon-cache ]; then
%{_bindir}/gtk-update-icon-cache --quiet %{_datadir}/icons/hicolor || :
fi

by

%{_bindir}/gtk-update-icon-cache --quiet %{_datadir}/icons/hicolor || :

since "|| :" causes the command to exit with a successful exit status whether or
not the command worked.

#4  debug rpm:
chitlesh(SPECS)[0]$rpmlint
/home/chitlesh/rpmbuild/RPMS/i386/klamav-debuginfo-0.38-2.i386.rpm
E: klamav-debuginfo script-without-shebang
/usr/src/debug/klamav-0.38/src/klammail/client.c
E: klamav-debuginfo script-without-shebang
/usr/src/debug/klamav-0.38/src/klammail/options.c
E: klamav-debuginfo script-without-shebang
/usr/src/debug/klamav-0.38/src/klammail/options.h
E: klamav-debuginfo script-without-shebang
/usr/src/debug/klamav-0.38/src/klammail/clamdmail.c
E: klamav-debuginfo script-without-shebang
/usr/src/debug/klamav-0.38/src/klammail/treewalk.c

Solution:
in %install, add
chmod 644 src/klammail/*.{c,h}

#5: rpmlint rpm
chitlesh(SPECS)[0]$rpmlint 
/home/chitlesh/rpmbuild/RPMS/i386/klamav-0.38-2.i386.rpm
W: klamav dangling-relative-symlink /usr/share/doc/HTML/en/klamav02/common 
../common

/tmp/klamav-0.38-2.i386.rpm.20210/usr/share/applications/fedora-klamav.desktop:
warning: file contains key "DocPath", this key is currently reserved for use
within KDE, and should in the future KDE releases be prefixed by "X-"

Ignore both.

/tmp/klamav-0.38-2.i386.rpm.20210/usr/share/applications/fedora-klamav.desktop:
warning: boolean key "Terminal" has value "0", boolean values should be "false"
or "true", although "0" and "1" are allowed in this field for backwards
compatibility

Solution (one line):
%{__sed} -i.orig -e '/^Terminal/s|^.*$|Terminal=false|' \
${RPM_BUILD_ROOT}%{_datadir}/applnk/Utilities/%{name}.desktop
%{__rm} -f ${RPM_BUILD_ROOT}%{_datadir}/applnk/Utilities/%{name}.desktop.orig
before "desktop-file-install --vendor fedora \" ..


#6: in %doc, drop NEWS, because it's empty

-- 
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 205300] Review Request: gtk-sharp - a set of mono bindings for gtk1.2

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

Summary: Review Request: gtk-sharp - a set of mono bindings for gtk1.2


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 17:10 EST ---
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 16:46 EST ---
Whoops. Forgot to patch the pathing for thai/cyrillic. Fixed now.

New SRPM: http://www.auroralinux.org/people/spot/review/xpdf-3.01-25.fc6.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/xpdf.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 205300] Review Request: gtk-sharp - a set of mono bindings for gtk1.2

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

Summary: Review Request: gtk-sharp - a set of mono bindings for gtk1.2


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 16:43 EST ---
ok, all those changes look pretty good. 

I'll go ahead and APPROVE this package. Thanks for all your hard work on it. 

Please remember to close this NEXTRELEASE once it's been imported and built. 


-- 
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 205929] Review Request: libfreebob - FreeBoB firewire audio driver library

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

Summary: Review Request: libfreebob - FreeBoB firewire audio driver library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 16:42 EST ---
I'm glad this is in process of getting included in Extras finally. I also own a 
PreSonus FireBox, great card :)

I have been providing FreeBoB packages along with JACK and relevant libs for 
FC5 for a while. They are here: http://www.ffri.hr/~vmiletic/linux/fedora/

You say you bumped libavc1394. It also needs libiec61883 1.1.0, afaik. I also 
packaged svn version of libraw1394, as suggested on FreeBoB page.

And last, but not the least: do you also plan to add FireWire soundcard 
detection support to system-config-soundcard at some point?

-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 16:36 EST ---
OK. CMap files are gone from source tarballs, patched out of add-to-xpdfrc
files. Also noticed that the thai and cyrillic files weren't actually being
packaged (thankfully, they don't have CMap files).

New SRPM: http://www.auroralinux.org/people/spot/review/xpdf-3.01-24.fc6.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/xpdf.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 206266] Review Request: transmission - lightweight GTK+ BitTorrent client

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

Summary: Review Request: transmission - lightweight GTK+ BitTorrent client


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 16:11 EST ---
1. ok. Looks good. 

2. Looks fine. I can't see that it matters too much either way if the desktop 
is a source file or part of the spec. 

3. Well, %makeinstall should be avoided where possible for the reasons outlined 
above, but in this case it looks like you do need to use it. ;( 

All the blockers I see are fixed, this package is APPROVED. 
Don't forget to close this bug NEXTRELEASE once it's been imported and built. 

-- 
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 208007] Review Request: perl-Alien-wxWidgets - Building, finding and using wxWidgets binaries

2006-09-25 Thread bugzilla
Please 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-Alien-wxWidgets - Building, finding and using 
wxWidgets binaries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||208009
  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 208009] Review Request: perl-Wx - Interface to the wxWidgets cross-platform GUI toolkit

2006-09-25 Thread bugzilla
Please 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-Wx - Interface to the wxWidgets cross-platform 
GUI toolkit


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||208007




-- 
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 205300] Review Request: gtk-sharp - a set of mono bindings for gtk1.2

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

Summary: Review Request: gtk-sharp - a set of mono bindings for gtk1.2


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 15:59 EST ---
Ping!

-- 
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 208009] New: Review Request: perl-Wx - Interface to the wxWidgets cross-platform GUI toolkit

2006-09-25 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=208009

   Summary: Review Request: perl-Wx - Interface to the wxWidgets
cross-platform GUI toolkit
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-Wx.spec
SRPM URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-Wx-0.57-1.src.rpm

Description:
The Wx module is a wrapper for the wxWidgets (formerly known as
wxWindows) GUI toolkit.

This module comes with extensive documentation in HTML format;
you can download it from http://wxperl.sourceforge.net/.

Note: 
A requirement for perl-PPI-Tester
(perl-Wx -> perl-PPI-Tester)

-- 
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 206871] Review Request: ekg2 - Multi-protocol instant messaging and chat client

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

Summary: Review Request: ekg2 - Multi-protocol instant messaging and chat client


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 15:53 EST ---
I'm going to have to duck out of this one - work pressures mean I can keep up
with what I have but not much else. Reassigning this BZ to the default nobody to
pick up

-- 
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 208007] New: Review Request: perl-Alien-wxWidgets - Building, finding and using wxWidgets binaries

2006-09-25 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=208007

   Summary: Review Request: perl-Alien-wxWidgets - Building, finding
and using wxWidgets binaries
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-Alien-wxWidgets.spec
SRPM URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-Alien-wxWidgets-0.21-1.src.rpm

Description: 
"Alien::wxWidgets" can be used to detect and get configuration
settings from an installed wxWidgets.

Note: 
A requirement for perl-PPI-Tester
(perl-Alien-wxWidgets -> perl-Wx -> perl-PPI-Tester)

-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 15:48 EST ---
Linking together is not the only reason why GPL and GPL-incompatible
soft cannot cooperate. They shouldn't be in the same 'container'. 
Quoting the GPL:

  b)  You must cause any work that you distribute or publish, 
  that in whole or in part contains or is derived from the 
  Program or any part thereof, to be licensed as a whole at no 
  charge to all third parties under the terms of this License.

So, for example it could be argued that xpdf-japanese-2004-jul-27.tar.gz
contains the GPL-licenced encoding files, and therefore cause the
files in CMap, distributed alongside to be licenced under the GPL 
which is not possible.

It is also explained here:
 
  If identifiable sections of that work are not derived from the Program, 
  and can be reasonably considered independent and separate works in 
  themselves, then this License, and its terms, do not apply to those 
  sections when you distribute them as separate works. But when you 
  distribute the same sections as part of a whole which is a work based 
  on the Program, the distribution of the whole must be on the terms of 
  this License, whose permissions for other licensees extend to the 
  entire whole, and thus to each and every part regardless of who wrote 
  it.

The first sentence, in my opinion describes what are the CMap files, 
"independent and separate works in themselves", so the GPL don't apply
to them. However, as explained in the second sentence, "when you 
distribute the same sections as part of a whole which is a work based 
on the Program", which is the case for CMap files bundled together with
encoding files covered by the GPL, "the distribution of the whole must 
be on the terms of this License, whose permissions for other licensees 
extend to the entire whole, and thus to each and every part regardless 
of who wrote it." Since the CMap cannot be distributed under the GPL,
we have somewhere a licence violation.

-- 
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 208006] New: Review Request: perl-Time-Period - A Perl module to deal with time periods

2006-09-25 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=208006

   Summary: Review Request: perl-Time-Period - A Perl module to deal
with time periods
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://www.marcanoonline.com/downloads/fedora/package_submissions/perl-Time-Period/perl-Time-Period.spec
SRPM URL: 
http://www.marcanoonline.com/downloads/fedora/package_submissions/perl-Time-Period/perl-Time-Period-1.20-1.src.rpm
Description: Period.pm is a Perl module that contains code to deal with time 
periods.

-- 
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 208004] New: Review Request: perl-Test-Perl-Critic - Use Perl::Critic in test programs

2006-09-25 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=208004

   Summary: Review Request: perl-Test-Perl-Critic - Use Perl::Critic
in test programs
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-Test-Perl-Critic.spec
SRPM URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-Test-Perl-Critic-0.07-1.src.rpm

Description:
Test::Perl::Critic wraps the Perl::Critic engine in a convenient
subroutine suitable for test programs written using the Test::More
framework. This makes it easy to integrate coding-standards enforcement
into the build process. For ultimate convenience (at the expense of some
flexibility), see the criticism pragma.

-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 14:45 EST ---
Having the Free but GPL-incompatible CMap files (which are not compiled) shipped
alongside the GPL application (xpdf) is not a problem, since they are not linked
together.

The missing README fixes are now in the new patch.

Here comes -23:

New SRPM: http://www.auroralinux.org/people/spot/review/xpdf-3.01-23.fc6.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/xpdf.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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 14:31 EST ---
You have super power to do patches quicker than light...
You didn't repatch the xpdf-LANG/README files, though...
It isn't a blocker (though it should be better).

Moving to the issue of licencing I found that the 
LANG specific packages are under licences that are not GPL
compatible (distribution without modification). Since it is 
font information (data), it seems to me that it doesn't 
contradict fedora goals, however it is dubious from a legal
point of view to distribute the font information together
with GPL code. It isn't obvious since it is code and font 
information. 

If the GPL part and distributable parts have to
be split, it won't be obvious to do it since a simple split 
(for example all the xpdf-LANG*.tar.gz in a single package)
wouldn't be right: in the xpdf-LANG.tar.gz the encoding 
files are covered by the GPL, only the CMap/* files are 
covered by the GPL-incompatible licence...

-- 
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 175433] Review Request: tor - Anonymizing overlay network for TCP (The onion router)

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

Summary: Review Request: tor - Anonymizing overlay network for TCP (The onion 
router)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 14:10 EST ---
MD5Sum
2d861e91e45a709acd921f26214319c1  tor-0.1.1.23-3.fc5x.src.rpm

Both issues raised before have been fixed.
 
APPROVED

-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 14:04 EST ---
Fixed in -22:

New SRPM: http://www.auroralinux.org/people/spot/review/xpdf-3.01-22.fc6.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/xpdf.spec

Anything else? :)

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

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

Summary: Review Request: initng


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:53 EST ---
(In reply to comment #338)
> Also, I noticed that I'm suddenly not member of cvsextras anymore (which I was
> when I looked a few hours ago). I don't really know what went wrong this time,
> but I guess I need someone to sponsor me again...

Some accounts were removed from cvsextras recently, as described here:
http://fedoraproject.org/wiki/Extras/RemovedFromCvsextras

Ask on fedora-extras-list about getting cvsextras membership back.

-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:51 EST ---
Created an attachment (id=137074)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=137074&action=view)
use versionned source files in ftp


-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:50 EST ---
The sources are not right for the lang packages. It seems that
these are not the latest versions that are used. In my opinion
it would be much better to have the real files and not the 
ftp links that points to the versionned files as source files.
This would have the added benefit to have different files with
different names in the look-aside cache. I made a patch to test 
the rpm build with the language files really present, and it
seems that the patches have to be updated... I'll attach a quick 
spec file diff with the real source file names, but you'll have
to update the patches ;-)

-- 
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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:47 EST ---
Well, if desktop entry appears in "Utilities->File" on KDE as
you have expected, please go ahead (i.e. import to cvs).

I don't stop you from commiting this package to cvs.

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

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

Summary: Review Request: initng


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:46 EST ---
Also, I noticed that I'm suddenly not member of cvsextras anymore (which I was
when I looked a few hours ago). I don't really know what went wrong this time,
but I guess I need someone to sponsor me again...

-- 
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 189889] Review Request: vkeybd - Virtual MIDI Keyboard

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

Summary: Review Request: vkeybd - Virtual MIDI Keyboard


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:40 EST ---
(In reply to comment #13) 
> - please do not use a full path in the .desktop file in the Exec line because
> this make the path to the binary hardcoded.

Fixed.

> - there are some warnings in build.log in mock: sffile.c:122: warning: 
> ignoring
> return value of 'fread', declared with attribute warn_unused_result - see them
> all in the attachment.

I'll let upstream know.

> - ChangeLog is not packaged

Fixed.

> - did you ask upstream to include your icon / desktop files? There is already 
> an
> desktop file in the upstream tarball, so maybe upstream will include your
> improved desktop file.

I'll do that.
 
> - the lash patch does not patch the README properly(LADCCA is still 
> mentioned):
> 
> -  --ladcca bool
> +  --lash bool
> Specify the support of LADCCA.  Give yes or no as the

Fixed.

> - the manpage does not mention the --lash option (the upstream version not the
> --ladcca option)

I'll report upstream.

> - have you submitted lash patch to upstream? (Just out of curiosity, what are
> the advantages of lash against ladcca?

LADCCA is dead.  LASH is the new LADCCA.

> - some files have strange permissions, but I don't know whether or not this
> needs to be fixed:
> $ rpm -vql vkeybd | grep -- -r--r--r
> -r--r--r--1 rootroot 2278 Sep 19 23:41
> /usr/share/man/man1/vkeybd.1.gz

Fixed.

> -r--r--r--1 rootroot 5765 Sep 19 23:41
> /usr/share/vkeybd/vkeybd.list
> -r--r--r--1 rootroot  282 Sep 19 23:41
> /usr/share/vkeybd/vkeybdmap
> -r--r--r--1 rootroot  590 Sep 19 23:41
> /usr/share/vkeybd/vkeybdmap-german

I didn't change these.

> - changing %{_datadir}/vkeybd to %{_datadir}/vkeybd/ in %files makes it more
> obvious that an directory is meant

Done.

I also updated the .desktop file as per comment #14.

Updated bits here:

Spec URL: http://people.redhat.com/green/FE/FC5/vkeybd.spec
SRPM URL: http://people.redhat.com/green/FE/FC5/vkeybd-0.1.17-8.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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:12 EST ---
(In reply to comment #20)
> Spec URL: 
> http://linuxelectronique.free.fr/download/fedora/5/SPECS/kbackup.spec
> SRPM URL: 
> http://linuxelectronique.free.fr/download/fedora/5/SRPMS/kbackup-0.5-4.src.rpm
> 
> %changelog
> * Mon Sep 25 2006 Alain Portal  0.5-4
>   - Install only one desktop file
>   - Don't remove absolute symlinks
>   - Update patch0

This time, in GNOME desktop entry appears on "accessory".
For KDE, where does the entry for KBackup appear?

-- 
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 207763] Review Request: ddd - GUI for several command-line debuggers

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

Summary: Review Request: ddd - GUI for several command-line debuggers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:02 EST ---
Fixed up in -9:

New SRPM: http://www.auroralinux.org/people/spot/review/ddd-3.3.11-9.fc6.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/ddd.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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 13:01 EST ---
Spec URL: 
http://linuxelectronique.free.fr/download/fedora/5/SPECS/kbackup.spec
SRPM URL: 
http://linuxelectronique.free.fr/download/fedora/5/SRPMS/kbackup-0.5-4.src.rpm

%changelog
* Mon Sep 25 2006 Alain Portal  0.5-4
  - Install only one desktop file
  - Don't remove absolute symlinks
  - Update patch0

-- 
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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 12:59 EST ---
Well, if it is complicated to unify desktop files, I think
owing two desktop files is not so bad idea.

-- 
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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 12:38 EST ---
I found two desktop entries in my menu
- one in "System"
- one in "Utilities->File"

It seems to me that the better place is "Utilities->File".
To remove the one in "System", I think that I have to remove "System" from the 
categories field in desktop. But if I do that, it seems to me there won't be 
any entry in a GNOME menu.
Is there a GNOME category similar to "X-KDE-Utilities-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 184530] Review Request: perl-RPM2

2006-09-25 Thread bugzilla
Please 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-RPM2


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 12:30 EST ---
No, he isn't.  I'm maintaining most of the perl* packages for FC and RHEL, so I
could take it if no-one else wants it.  I'll send a quick mail to
fedora-perl-devel to see if anyone else wants it first, though.

-- 
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 207896] Review Request: astyle - Source code formatter

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

Summary: Review Request: astyle - Source code formatter


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 12:20 EST ---
p.s. -- Just heard back from the developer, and he's fine with this being in 
Extras.

-- 
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 207725] Review Request: sshfp - Generate SSHFP DNS records from knownhosts files or ssh-keyscan

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

Summary: Review Request: sshfp - Generate SSHFP DNS records from knownhosts 
files or ssh-keyscan


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 12:14 EST ---
Okay. I guess I agree this discussion should move to the openssh-clients 
package.

* Mon Sep 25 2006 Paul Wouters <[EMAIL PROTECTED]> - 1.0.6-2
- Don't change VerifyHostKeyDNS in /etc/ssh/ssh_config anymore


-- 
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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 12:04 EST ---
(In reply to comment #16)
> (In reply to comment #14)
> > And,
> > # Fix absolute symlink
> > %{__rm} -f %{buildroot}%{_docdir}/HTML/*/%{name}/common
> > 
> > you remove the symlink, but then don't ever actually replace/fix it.  
> > Either 
> > fix it to be a relative symlink or leave as is.  Else, you'll end up with 
> > broken help links.
> 
> Please, could you tell me how to fix?
> 

Simply not removing %{buildroot}%{_docdir}/HTML/*/%{name}/common
is the easiest idea to follow the recommendation from Rex.
I don't complain for 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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:53 EST ---
(In reply to comment #14)
> And,
> # Fix absolute symlink
> %{__rm} -f %{buildroot}%{_docdir}/HTML/*/%{name}/common
> 
> you remove the symlink, but then don't ever actually replace/fix it.  Either 
> fix it to be a relative symlink or leave as is.  Else, you'll end up with 
> broken help links.

Please, could you tell me how to fix?


-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:48 EST ---
Patrice, if you're happy with the package, feel free to take over the review and
approve it.  I ran into some time issues and couldn't get to it this weekend.

-- 
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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:37 EST ---
(In reply to comment #12)
> OK. Group is a throwaway, it matches Evince and there is nothing better to set
> it to. 

Indeed. I also use that group for example for xchm which is even
more wrong in my opinion.
Everything seems fine to me now, but I am not the reviewer ;-). 

I had a look at the gcc warnings, most of them seems not worrying,
however I spotted some warnings like:

warning: 'tpgrCXPtr1$x' may be used uninitialized in this function

certainly not a blocker for inclusion in extras, but maybe 
something to be reported upstream, if somebody feels like 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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:34 EST ---
Crap, my comment #13 got truncated, it was *supposed* to say change to

desktop-file-install --vendor="" \
  --dir %{buildroot}%{_datadir}/applications/kde \
  --add-category="Utility" \
  --add-category="X-KDE-Utilities-File" \
  --add-category="X-Fedora" \
  --remove-only-show-in "KDE" \
  %{buildroot}%{_datadir}/applications/kde/*.desktop

-- 
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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:32 EST ---
And,
# Fix absolute symlink
%{__rm} -f %{buildroot}%{_docdir}/HTML/*/%{name}/common

you remove the symlink, but then don't ever actually replace/fix it.  Either 
fix it to be a relative symlink or leave as is.  Else, you'll end up with 
broken help 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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:30 EST ---
FYI, you shouldn't have 2 .desktop files, I'd recommend changing
mkdir -p %{buildroot}%{_datadir}/applications
desktop-file-install --vendor fedora \
  --dir %{buildroot}%{_datadir}/applications \
  --add-category "Utility" \
  --add-category "X-KDE-Utilities-File" \
  --add-category "X-Fedora" \
  --remove-only-show-in "KDE" \
  src/%{name}.desktop

-- 
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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:26 EST ---
I am pleased to say that this package is now 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 207806] Review Request: evolution-bogofilter - A plugin for bogofilter support in evolution

2006-09-25 Thread bugzilla
Please 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-bogofilter -  A plugin for bogofilter 
support in evolution


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:17 EST ---
Fixed in -2:

New SRPM:
http://www.auroralinux.org/people/spot/review/evolution-bogofilter-0.2.0-2.fc6.src.rpm
New SPEC:
http://www.auroralinux.org/people/spot/review/evolution-bogofilter.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 207761] Review Request: xpdf - A PDF file viewer for the X Window System

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

Summary: Review Request: xpdf - A PDF file viewer for the X Window System


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:14 EST ---
OK. Group is a throwaway, it matches Evince and there is nothing better to set
it to. Everything else is fixed in -21.

New SRPM: http://www.auroralinux.org/people/spot/review/xpdf-3.01-21.fc6.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/xpdf.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 206842] Review Request: seq24 - Real-time MIDI sequencer

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

Summary: Review Request: seq24 - Real-time MIDI sequencer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 11:14 EST ---
These updated bits take care of comment #4 and comment #5.

Spec URL: http://people.redhat.com/green/FE/devel/seq24.spec
SRPM URL: http://people.redhat.com/green/FE/devel/seq24-0.8.7-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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 10:41 EST ---
Spec URL: 
http://linuxelectronique.free.fr/download/fedora/5/SPECS/kbackup.spec
SRPM URL: 
http://linuxelectronique.free.fr/download/fedora/5/SRPMS/kbackup-0.5-3.src.rpm

%changelog
* Mon Sep 25 2006 Alain Portal  0.5-3
  - desktop-file-install don't work as I expected, so update patch0


-- 
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 207839] Review Request: lush - An object-oriented Lisp interpreter and compiler

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

Summary: Review Request: lush - An object-oriented Lisp interpreter and compiler


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 10:33 EST ---
Now builds in FC6 mock (this SRPM will not work on FC5).
Spaces/Tab fixed.

http://math.ifi.unizh.ch/fedora/5/i386/SRPMS.gemi/lush-1.2-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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 10:10 EST ---
* Well, currently in GNOME only one menu appears for kbackup,
however, my opinion is "OnlyShowIn=KDE;" should be in
/usr/share/applications/kde/kbackup.desktop, not in
/usr/share/applications/fedora-kbackup.desktop.

I can see that /usr/share/applications/kde/kresources.desktop
(in kdelibs) has the entry "OnlyShowIn=KDE;".


* Another thing: /usr/share/applications/kde/kbackup.desktop has
X-SuSE-Backup in Category, this should be changed.

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

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

Summary: Review Request: initng


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 09:48 EST ---
So this means I'm free to commit initng against FE svn and request a build?
That's good news indeed!

Another completely different questions to the folks here: I just noticed that I
had a process nash-hotplug running 100% all the time. Is this something init(ng)
is supposed to kill? I find some rows in rc.sysinit which seems to do 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 186452] Review Request: kdebluetooth: The KDE Bluetooth Framework

2006-09-25 Thread bugzilla
Please 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-09-25 09:46 EST ---
I dug into this a little with rawhide selinux execution of kbluepin is  
blocked.  I didnt try messing around setting up a selinux policy to make it 
work

however i did discover that KDE is not running the things 
in /etc/xdg/autostart/ that its supposed to 
in /etc/xdg/autostart/bluez-pin.desktop it executes bluez-pin --dbus  which  
if i ran that  i got the default pin helper to 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 207607] Review Request: telepathy-butterfly - MSN connection manager for Telepathy

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

Summary: Review Request: telepathy-butterfly - MSN connection manager for 
Telepathy


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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 207173] Review Request: pymsn - Python libraries for MSN Messenger network

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

Summary: Review Request: pymsn - Python libraries for MSN Messenger network


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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 207927] New: Review Request: supertuxkart - Kids 3D go-kart racing game featuring Tux

2006-09-25 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=207927

   Summary: Review Request: supertuxkart - Kids 3D go-kart racing
game featuring Tux
   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.atrpms.net/~hdegoede/supertuxkart.spec
SRPM URL: http://people.atrpms.net/~hdegoede/supertuxkart-0.2-1.src.rpm
Description:
3D go-kart racing game for kids with several famous OpenSource mascots
participating. Race as Tux against 3 computer players in many different fun
race courses (Standard race track, Dessert, Mathclass, etc). Full information
on how to add your own race courses is included. During the race you can pick
up powerups such as: (homing) missiles, magnets and portable zippers.

---

Notice that this really is nothing more then a new version of tuxkart, but 
since tuxkart upstream is sorta dead, a group of developers has brought out 
their new version under a new name called supertuxkart. After some discussion 
on f-e-l it was decided that it was best to create a new CVS branch for this 
new named version and thus a (Re-)Review

Since this thus essentially is not a new package (mearly a rename) I would like 
to request a "quick" review whatever that may mean :)

-- 
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 207532] Review Request: kbackup - Back up your data in a simple, user friendly way

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

Summary: Review Request: kbackup - Back up your data in a simple, user friendly 
way


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-25 09:25 EST ---
Spec URL: 
http://linuxelectronique.free.fr/download/fedora/5/SPECS/kbackup.spec
SRPM URL: 
http://linuxelectronique.free.fr/download/fedora/5/SRPMS/kbackup-0.5-2.src.rpm

%changelog
* Mon Sep 25 2006 Alain Portal  0.5-2
  - Use macro for make
  - Don't own some directories
  - Update patch0 and patch1
  - Improve desktop-file installation


-- 
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 207173] Review Request: pymsn - Python libraries for MSN Messenger network

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

Summary: Review Request: pymsn - Python libraries for MSN Messenger network


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 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


  1   2   >