[Bug 190040] Review Request: hydrogen - Advanced drum machine

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

Summary: Review Request: hydrogen - Advanced drum machine


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




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

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


[Bug 189892] Review Request: dssi - Disposable Soft Synth Interface

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

Summary: Review Request: dssi - Disposable Soft Synth Interface


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




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

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


[Bug 190027] Review Request: hexter-dssi - DSSI software synthesizer plugin

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

Summary: Review Request: hexter-dssi - DSSI software synthesizer plugin


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




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

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


[Bug 192583] New: Review Request: php-pear-XML-Parser - XML parsing class based on PHP's bundled expat

2006-05-21 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=192583

   Summary: Review Request: php-pear-XML-Parser - XML parsing class
based on PHP's bundled expat
   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://remi.collet.free.fr/rpms/extras/php-pear-XML-Parser.spec
SRPM URL: 
http://remi.collet.free.fr/rpms/extras/php-pear-XML-Parser-1.2.7-1.fc5.src.rpm
Description: 
http://remi.collet.free.fr/rpms/extras/php-pear-XML-Parser-build.log

This is an XML parser based on PHPs built-in xml extension.
It supports two basic modes of operation: "func" and "event".
In "func" mode, it will look for a function named after each element
(xmltag_ELEMENT for start tags and xmltag_ELEMENT_ for end tags),
and in "event" mode it uses a set of generic callbacks.

Since version 1.2.0 there's a new XML_Parser_Simple class that makes
parsing of most XML documents easier, by automatically providing a stack
for the elements. Furthermore its now possible to split the parser from
the handler object, so you do not have to extend XML_Parser anymore in
order to parse a document with it.

With DB, HTTP (Bug #190156), Mail (Bug #190066), Net_Socket (Bug #190957), 
Net_SMTP (Bug #190958), extras will provides all the extensions which was 
provides by php until version 5.0.4 (in FC4).

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

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


[Bug 189884] Review Request: LASH Audio Session Handler

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

Summary: Review Request: LASH Audio Session Handler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




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

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


[Bug 190040] Review Request: hydrogen - Advanced drum machine

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

Summary: Review Request: hydrogen - Advanced drum machine


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


[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 190040] Review Request: hydrogen - Advanced drum machine

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

Summary: Review Request: hydrogen - Advanced drum machine


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 04:34 EST ---
Michael: You have dibs on this review, but you hadn't assigned it to yourself. I
do want to work on this tommorow. Sorry if I'm stepping on your toes. Reassign
it if you want 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 189892] Review Request: dssi - Disposable Soft Synth Interface

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

Summary: Review Request: dssi - Disposable Soft Synth Interface


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


[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 190027] Review Request: hexter-dssi - DSSI software synthesizer plugin

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

Summary: Review Request: hexter-dssi - DSSI software synthesizer plugin


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


[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 191604] Review Request: lineakd - linux easy-access-key daemon

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

Summary: Review Request: lineakd - linux easy-access-key daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 04:38 EST ---
Still not 100% happy with the ldconfig stuff, but at least this way my specs are
consistent with each other.  Shuffling files around, so new files are at:

http://rpm.forevermore.net/lineak/lineakd.spec
http://rpm.forevermore.net/lineak/lineakd-0.8.4-3.src.rpm

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

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


[Bug 189088] Review Request: knemo Network monitor applet.

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

Summary: Review Request: knemo Network monitor applet.


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 04:39 EST ---
Hugo, just to make it clear: I understand your motives and it's nice of you to
help Richard with this package. Except, you're giving him a fish, instead of
teaching him how to fish.

About the *.la files, the ones directly in /usr/lib/*.la are safe to remove, but
the others are often needed by KDE.

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

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


[Bug 191607] Review Request: lineak_xosdplugin - Onscreen display support for lineakd

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

Summary: Review Request: lineak_xosdplugin - Onscreen display support for 
lineakd


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 04:40 EST ---
Shuffling files around on my server, and making a few changes to bring things up
to par.

http://rpm.forevermore.net/lineak/lineak_xosdplugin.spec
http://rpm.forevermore.net/lineak/lineak_xosdplugin-0.8.4-2.src.rpm

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

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


[Bug 191606] Review Request: lineak_kdeplugins - KDE-based actions for lineakd

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

Summary: Review Request: lineak_kdeplugins - KDE-based actions for lineakd


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 04:41 EST ---
Shuffling files around on my server, and making a few changes to bring things up
to par.

http://rpm.forevermore.net/lineak/lineak_kdeplugins.spec
http://rpm.forevermore.net/lineak/lineak_kdeplugins-0.8.4-2.src.rpm

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

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


[Bug 191605] Review Request: lineak_defaultplugin - default actions for lineakd

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

Summary: Review Request: lineak_defaultplugin - default actions for lineakd


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 04:41 EST ---
Shuffling files around on my server, and making a few changes to bring things up
to par.

http://rpm.forevermore.net/lineak/lineak_defaultplugin.spec
http://rpm.forevermore.net/lineak/lineak_defaultplugin-0.8.4-2.src.rpm

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

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


[Bug 192540] Review Request: cal3d - A skeletal based 3-D character animation library

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

Summary: Review Request: cal3d - A skeletal based 3-D character animation 
library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 05:50 EST ---
spec file looks good to me.

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

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


[Bug 189892] Review Request: dssi - Disposable Soft Synth Interface

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

Summary: Review Request: dssi - Disposable Soft Synth Interface


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 06:24 EST ---
(In reply to comment #4)
> After poking things a bit more, I think dssi-devel should Depend: on
> alsa-lib-devel, ladspa-devel, and liblo-devel. But not jack-devel, the dssi 
> API
> itself does not use jack.

Done.

> Not so sure the examples package needs a versioned dependency on the main
> package. Any other dssi plugins wouldn't do so.

The difference is that the examples are bundled with the main package sources.
I don't think it hurts to keep them in sync.
 
> And since its just a header file, I don't think the devel package actually 
> needs
> to depend on the main package at all... (This would prevent mock from having 
> to
> d/l and install dssi, which pulls in jack...)

Done.  I've also moved COPYING to the devel package, since it only applies to
the header file.  The README file covers everything else.

> If you really wanted to reduce dependency bloat, you could split 
> jack-dssi-host
> off into its own package, isolating the jack dependency, but I really don't
> think its worth going quite that far.

Not done :-)

> Just rm-ing the .la files in %install seems marginally cleaner to me. And is
> what most other packages seem to do.

Done.

I've also fixed the .pc file problem.

Thanks for your help!

 Updated bits here

Spec URL: http://people.redhat.com/green/FE/FC5/dssi.spec
SRPM URL: http://people.redhat.com/green/FE/FC5/dssi-0.9.1-3.src.rpm

AG


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

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


[Bug 187621] Review Request: blam

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

Summary: Review Request: blam


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 08:12 EST ---
> removed the .a, .la, and .so files.

Eeek! Why also the .so files? These are plugin DSOs in %_libdir/blam/
which are local to this application.

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

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


[Bug 178900] Review Request: monodoc

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

Summary: Review Request: monodoc


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 11:04 EST ---
Couple items still need work:

* Unnecessary scriptlet for shared libraries is still in spec.
* -devel package should require on %{name} = %{version}-%{release}

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

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


[Bug 191603] Review Request: rsnapshot -- rsync-based filesystem snapshots

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

Summary: Review Request: rsnapshot -- rsync-based filesystem snapshots


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG




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

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


[Bug 183438] Review Request: idioskopos (C++ Introspection Library)

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

Summary: Review Request: idioskopos (C++ Introspection Library)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  QAContact|fedora-extras-  |fedora-package-
   |[EMAIL PROTECTED] |[EMAIL PROTECTED]
   Severity|enhancement |normal
   Keywords||FutureFeature

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Keywords||FutureFeature




--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 12:37 EST ---
New release - 0.2.0

Small changes to spec file to remove doxygen and graphiz build dependencies
since docs are now pre-built.

Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/idioskopos.spec

SRPM Name or Url:
http://miskatonic.cs.nmsu.edu/pub/fedora/5/srpms/idioskopos-0.2.0-1.src.rpm

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

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


[Bug 190956] Review Request: php-pear-Auth-SASL

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

Summary: Review Request: php-pear-Auth-SASL


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 12:55 EST ---
Update to new version 1.0.2

Spec URL: http://remi.collet.free.fr/rpms/extras/php-pear-Auth-SASL.spec
SRPM
URL:http://remi.collet.free.fr/rpms/extras/php-pear-Auth-SASL-1.0.2-1.fc5.src.rpm
 
Mock build.log : 
http://remi.collet.free.fr/rpms/extras/php-pear-Auth-SASL-build.log




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

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


[Bug 192568] Review Request: perl-File-Type

2006-05-21 Thread bugzilla
Please 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-File-Type


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 12:59 EST ---
A couple of questions:

Any idea why Module::Build shows up in the final requires list and if it really
needs to be there?
What on Earth is the NINJA file for?

Review:
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* license field matches the actual license.
* license is open source-compatible. Text not included upstream.
* source files match upstream:
   4be3b0b7000b325c60351fcc8a04815d  File-Type-0.22.tar.gz
   4be3b0b7000b325c60351fcc8a04815d  File-Type-0.22.tar.gz-srpm
* latest version is being packaged.
* BuildRequires are proper.
* package builds in mock (development, x86_64).
* rpmlint is silent.
? final provides and requires are sane:
   perl(File::Type) = 0.22
   perl(File::Type::Builder) = 0.11
   perl-File-Type = 0.22-1.fc6
  -
   perl(:MODULE_COMPAT_5.8.8)
   perl(IO::File)
   perl(strict)
   perl(warnings)
   rpmlib(CompressedFileNames) <= 3.0.4-1
   rpmlib(PayloadFilesHavePrefix) <= 4.0-1
   rpmlib(VersionedDependencies) <= 3.0.3-1
   perl(Module::Build)
   rpmlib(CompressedFileNames) <= 3.0.4-1
* no shared libraries are present.
* package is not relocatable.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* %clean is present.
* %check is present and all tests pass:
   All tests successful.
   Files=3, Tests=58,  0 wallclock secs ( 0.13 cusr +  0.06 csys =  0.19 CPU)
* 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.
* no headers.
* no pkgconfig files.
* no libtool .la droppings.
* not a GUI app.


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

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


[Bug 183439] Review Request: papyrus (Canvas drawing library based on cairo/cairomm)

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

Summary: Review Request: papyrus (Canvas drawing library based on cairo/cairomm)


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 13:41 EST ---
rpmlint of papyrus:
W: papyrus incoherent-version-in-changelog 0.1.10-1 0.1.11-1.fc6

You're one behind in the changelogs.

Sources do not match:

3d0c75f05409da34a8766e7ddbc9df0f  papyrus-0.1.11.tar.bz2 (download)
17886301eb027ff4d727c9d27b38570f  papyrus-0.1.11.tar.bz2 (srpm)

The srpm appears to have a more recent ChangeLog.

While you're at it change the Source to point to download.sourceforge.net to
automatically choose a mirror.



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

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


[Bug 192600] New: Review Request: sflowtool - Command line utility for analyzing sFlow data

2006-05-21 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=192600

   Summary: Review Request: sflowtool - Command line utility for
analyzing sFlow data
   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://home.bawue.net/~ixs/sflowtool/sflowtool.spec
SRPM URL: http://home.bawue.net/~ixs/sflowtool/sflowtool-3.10-1.src.rpm
Description:
The core component of the sFlow toolkit is the sflowtool command line 
utility. sflowtool interfaces to utilities such as tcpdump, ntop and 
Snort for detailed packet tracing and analysis, NetFlow compatible 
collectors for IP flow accounting, and provides text based output that 
can be used in scripts to provide customized analysis and reporting and 
for integrating with other tools such as MRTG or rrdtool


NB: Please take a look at the license, it looks pretty BSDish to me, but 
someone else should please make sure.

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

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


[Bug 190129] Review Request: python-krbV

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

Summary: Review Request: python-krbV


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 14:13 EST ---
New specfile and srpm posted:

http://people.redhat.com/mikeb/python-krbV/python-krbV.spec
http://people.redhat.com/mikeb/python-krbV/python-krbV-1.0.12-3.src.rpm

Fixed .spec permissions (rpmbuild -ts was doing something funny).
Removed BuildRequires: python
Replaced $RPM_BUILD_ROOT with %{buildroot}


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

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


[Bug 191208] Review Request: The Ipe extensible drawing editor

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

Summary: Review Request: The Ipe extensible drawing editor


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 14:13 EST ---
Update:
  Spec URL: http://www.di.ens.fr/~rineau/Fedora/ipe.spec
  SRPM URL: http://www.di.ens.fr/~rineau/Fedora/ipe-6.0-0.3.pre26.src.rpm

* Sun May 21 2006 Laurent Rineau 
<[EMAIL PROTECTED]> - 6.0-0.3.pre26
- Added a desktop file for Ipe.
* Sun May 21 2006 Laurent Rineau 
<[EMAIL PROTECTED]> - 6.0-0.2.pre26
- Fix directories ownership.
- ipelets/*.so are now in -devel.
- Creation of -doc subpackage.


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

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


[Bug 190175] Review Request: p0rn-comfort - Support programs for browsing image-gallery sites

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

Summary: Review Request: p0rn-comfort - Support programs for browsing 
image-gallery sites


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 14:21 EST ---
H tibbs,

Updated package at http://home.bawue.de/~ixs/p0rn-comfort/
I've incorporated your idea of moving some files out of %{_bindir} and put them
into %{_libexecdir}/%{_name} as decided on IRC.

I also fixed the issue with p0rn-proxy, so it could be checked in.
On IRC you said, you already approved the package, but reading your bugzilla
entry, that's not very clear.


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

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


[Bug 191604] Review Request: lineakd - linux easy-access-key daemon

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

Summary: Review Request: lineakd - linux easy-access-key daemon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 14:24 EST ---
Everything looks fine now.  rpmlint is fine once you ignore the bugus warning,
requires are fine now, buildrequires are fine and the scriptlets are good.  I
also note that you've added a %{?dist} tag.

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 190129] Review Request: python-krbV

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

Summary: Review Request: python-krbV


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 14:38 EST ---
Odd that the tarball is still mode 664, but this definitely won't matter after
you import.  Otherwise, everything looks good.

APPROVED

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

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


[Bug 190471] Review Request: yakuake - Quake-style terminal emulator for KDE

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

Summary: Review Request: yakuake - Quake-style terminal emulator for KDE


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


Bug 190471 depends on bug 186283, which changed state.

Bug 186283 Summary: yakuake not included in the fc5 extras repository
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=186283

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|NEW |CLOSED



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

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


[Bug 183439] Review Request: papyrus (Canvas drawing library based on cairo/cairomm)

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

Summary: Review Request: papyrus (Canvas drawing library based on cairo/cairomm)


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 14:50 EST ---
> While you're at it change the Source to point to download.sourceforge.net to
> automatically choose a mirror.

wget http://download.sourceforge.net/libpapyrus/papyrus-0.1.11.tar.bz2
just hangs.

wget http://download.sourceforge.net/libpapyrus/papyrus-0.1.11.tar.bz2?download
succeeds.

Is the latter syntax ok?


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

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


[Bug 188180] Review Request: qt4: Qt GUI toolkit

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

Summary: Review Request: qt4: Qt GUI toolkit


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 15:17 EST ---
Spec Name or Url: 
http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/qt4-4.1.3-3.spec
SRPM Name or Url:
http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.stable/qt4-4.1.3-3.src.rpm

%changelog
* Sun May 21 2006 Rex Dieter  4.1.3-3
- fix %%mysql_libs macro


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

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


[Bug 190471] Review Request: yakuake - Quake-style terminal emulator for KDE

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

Summary: Review Request: yakuake - Quake-style terminal emulator for KDE


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CURRENTRELEASE
OtherBugsDependingO|177841  |
  nThis||
   Fixed In Version||yakuake-2.7.5-2.fc5




--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 15:25 EST ---
Yakuake is now in FE5 so I'm closing 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 191604] Review Request: lineakd - linux easy-access-key daemon

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

Summary: Review Request: lineakd - linux easy-access-key daemon


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


[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 191606] Review Request: lineak_kdeplugins - KDE-based actions for lineakd

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

Summary: Review Request: lineak_kdeplugins - KDE-based actions for lineakd


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


Bug 191606 depends on bug 191604, which changed state.

Bug 191604 Summary: Review Request: lineakd - linux easy-access-key daemon
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=191604

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 191605] Review Request: lineak_defaultplugin - default actions for lineakd

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

Summary: Review Request: lineak_defaultplugin - default actions for lineakd


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


Bug 191605 depends on bug 191604, which changed state.

Bug 191604 Summary: Review Request: lineakd - linux easy-access-key daemon
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=191604

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 191607] Review Request: lineak_xosdplugin - Onscreen display support for lineakd

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

Summary: Review Request: lineak_xosdplugin - Onscreen display support for 
lineakd


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


Bug 191607 depends on bug 191604, which changed state.

Bug 191604 Summary: Review Request: lineakd - linux easy-access-key daemon
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=191604

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 192603] New: Review Request: openalpp - Object Oriented version of OpenAL

2006-05-21 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=192603

   Summary: Review Request: openalpp - Object Oriented version of
OpenAL
   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://tkmame.retrogames.com/fedora-extras/openalpp.spec
SRPM URL: http://tkmame.retrogames.com/fedora-extras/openalpp-20060405-2.src.rpm

Description:
Written in C++, makes OpenAL so much more accessible. Compare openalpp to a
scenegraph for OpenGL.

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

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


[Bug 191582] Review Request: xgalaga - Galaga clone for X11

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

Summary: Review Request: xgalaga - Galaga clone for X11


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


[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 189892] Review Request: dssi - Disposable Soft Synth Interface

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

Summary: Review Request: dssi - Disposable Soft Synth Interface


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 16:22 EST ---
Ah, licensing. According to the README, the header is LGPL, and jack-dssi-host
is "BSD-style". COPYING is included for the header, but everything else seems to
have the license in the source code. jack-dssi-host contains:

/*
 * Copyright 2004 Chris Cannam, Steve Harris and Sean Bolton.
 *
 * Permission to use, copy, modify, distribute, and sell this software
 * for any purpose is hereby granted without fee, provided that the
 * above copyright notice and this permission notice are included in
 * all copies or substantial portions of the software.
 */

Does this count as BSD?

Everything in the examples directory has something to the effect of "This
example file is in the public domain." Note that dssi_osc_send and
dssi_osc_update are in the main package, which otherwise seems like the right
place to me.

So, my guess is dssi-devel should be "License: LGPL", dssi-examples should be
"License: Public Domain", and the main package "License: BSD/Public Domain"
however mock complains about that one. Maybe it can just be "Distributable".

Someone who is knowledgeable in licensing, please take a look at 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 191582] Review Request: xgalaga - Galaga clone for X11

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

Summary: Review Request: xgalaga - Galaga clone for X11


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 16:28 EST ---
I'm getting a build error in mock during configure:

checking for gcc... gcc
checking whether the C compiler (gcc -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2
-fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic
-fsigned-char -DXF86VIDMODE -lXxf86vm) works... no
configure: error: installation or configuration problem: C compiler cannot
create executables.
error: Bad exit status from /var/tmp/rpm-tmp.65804 (%build)


RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.65804 (%build)


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

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


[Bug 189892] Review Request: dssi - Disposable Soft Synth Interface

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

Summary: Review Request: dssi - Disposable Soft Synth Interface


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||182235
  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 192363] Review Request: GTS - Gnu Triangulated Surface Library

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

Summary: Review Request: GTS - Gnu Triangulated Surface Library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 16:35 EST ---
Hi Ralf, I've included your patch (comment #4), added one for gts-config,
and imported it into CVS.  Please let me know if you see any problems.  
I'll request builds on FC-4/5 and devel in day or two.

And co-maintainership is fine with me -- please feel free to make whatever 
changes you want to it in CVS at any time.

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

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


[Bug 191582] Review Request: xgalaga - Galaga clone for X11

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

Summary: Review Request: xgalaga - Galaga clone for X11


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 17:01 EST ---
Well it works fine for me can you lift the actual gcc error from config.log that
might help.


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

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


[Bug 190129] Review Request: python-krbV

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

Summary: Review Request: python-krbV


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 17:05 EST ---
Thanks, built in devel.


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

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


[Bug 192606] New: Review Request: yafc: yet another ftp client

2006-05-21 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=192606

   Summary: Review Request: yafc: yet another ftp client
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://rpm.forevermore.net/yafc/yafc.spec
SRPM URL: http://rpm.forevermore.net/yafc/yafc-1.1.1-1.src.rpm
Description:

yafc is Yet Another FTP Client.  It is an interactive interface to the FTP
and SFTP protocols.

FEATURES
  - cached directory listings
  - uses readline (tab completion, emacs/vi editing keys, history file, etc.)
  - extensive tab completion
  - multiple connections open
  - aliases
  - colored ls (i.e., ls --color, uses $LS_COLORS like GNU ls)
  - autologin and bookmarks
  - recursive get/put/rm/ls
  - nohup mode get and put
  - tagging (queueing) of files for later transferring
  - automagically enters nohup-mode when SIGHUP received (in get and put)
  - redirection to local command or file ('>', '>>' and '|')
  - proxy support
  - more...

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

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


[Bug 192546] Review Request: gnubiff

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

Summary: Review Request: gnubiff


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 17:31 EST ---
Ok, i've made the changes.
Source : http://glive.tuxfamily.org/fedora/gnubiff/


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

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



[Bug 191582] Review Request: xgalaga - Galaga clone for X11

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

Summary: Review Request: xgalaga - Galaga clone for X11


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 17:55 EST ---
$ cat
/var/lib/mock/fedora-5-x86_64-core/root/builddir/build/BUILD/xgalaga-2.0.34/config.log
This file contains any messages produced by compilers while
running configure, to aid debugging if configure makes a mistake.

configure:564: checking host system type
configure:588: checking for gcc
configure:701: checking whether the C compiler (gcc -O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4
-m64 -mtune=generic -fsigned-char -DXF86VIDMODE -lXxf86vm) works
configure:717: gcc -o conftest -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2
-fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic
-fsigned-char -DXF86VIDMODE  -lXxf86vm conftest.c  1>&5
configure:714: warning: return type defaults to 'int'
/usr/bin/ld: cannot find -lXxf86vm
collect2: ld returned 1 exit status
configure: failed program was:

#line 712 "configure"
#include "confdefs.h"

main(){return(0);}


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

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


[Bug 192546] Review Request: gnubiff

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

Summary: Review Request: gnubiff


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 18:19 EST ---
Created an attachment (id=129786)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=129786&action=view)
Mock build log failure

Your package fails to build in Mock.  Also, you can drop the BR on
gettext-devel, since the default build environment in Mock installs gettext.

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

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


[Bug 191582] Review Request: xgalaga - Galaga clone for X11

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

Summary: Review Request: xgalaga - Galaga clone for X11


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 19:32 EST ---
* rpmlint output clean
* Package meets Package Naming Guidelines
* Spec filename matches base package %{name}
* Package meets Packaging Guidelines
* Package licensed with open source compatible license
* License in spec matches actual license
* License text included in %doc
* Spec file written in American English
* Spec file is legible
* Sources match upstream
9f7ee685e9c4741b5f0edc3f91df9510  xgalaga_2.0.34.orig.tar.gz
9f7ee685e9c4741b5f0edc3f91df9510  xgalaga_2.0.34.orig.tar.gz
* Package successfully compiles and builds on FC5 x86_64

O Package has all BR except libXxf86vm-devel which I needed to add for it to 
compile

* Package does not have any locales
* Package does not contain any shared library files
* Package is not relocatable
* Package owns all directories it creates
* Package does not contain any duplicate files in %files
* File permissions are set properly
* Package contains proper %clean section
* Macro usage consistant enough
- I notice you use %{__sed}, but don't bother using %{__make} or %{__rm} etc..
* Package contains permissble content
* Package does not contain large documentation to warrent a seperate package
* Package does not contain header files, libraries or .pc files
* Package does not contain any .so files
* Package does not require or use a -devel package
* Package does not contain any .la files
* Package adds an appropriate .desktop entry
* Package does not own any files or directories owned by other packages


*** MUST ***

- You MUST figure out why FC5 needs to add a BuildRequires of libXxf86vm-devel
and why this is not needed for your build (presumably FC6)


Non-blocking SHOULDs:

- Be more consistant with macro usage, for example %{__sed}, but no %{__rm} etc.
- I also prefer %{buildroot} instead of $RPM_BUILD_ROOT, but that is a matter of
preference.  I just think spec files look cleaner when everything consistantly
uses %{} format.  So basically I'm saying you should use a clean more legible
consistant style in your spec files, but I'm not going to say this is a blocker
or should be fixed, just a suggestion.
- Let me know that the name xgalaga isn't going to be a problem with Namco. 
I've heard the Lgames are not allowed because the names are too close to the
original, is this going to be a problem?
- Return the favor by reviewing some of my packages ;-)


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

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


[Bug 191582] Review Request: xgalaga - Galaga clone for X11

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

Summary: Review Request: xgalaga - Galaga clone for X11


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 20:00 EST ---
One other minor thing I noticed:

cat > README.fedora << EOF

The latest Fedora xgalaga package also includes fullscreen support, start
xgalaga with -window to get the old windowed behaviour. You can switch on the
fly between window and fullscreen mode with alt+enter .
EOF

The word "behaviour" is not American English.  It should be "behavior".  In
addition there should not be a space before the final period.

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

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


[Bug 187621] Review Request: blam

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

Summary: Review Request: blam


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





--- Additional Comments From [EMAIL PROTECTED]  2006-05-21 21:03 EST ---
(In reply to comment #3) 
> Eeek! Why also the .so files? These are plugin DSOs in %_libdir/blam/
> which are local to this application.

Oops. Silly me, that isn't a devel file.
http://fedorared.org/~john/review/blam-1.8.2-4.src.rpm

This version incorporates reenables %find_lang, but it doesn't work, no
translations found.

Sindre, you're still maintainer when this gets through, srpms are just easy to
distribute.

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

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