[Bug 198816] Review Request: gfs-utils - global file system userland utilities

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

Summary: Review Request: gfs-utils - global file system userland utilities


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 02:53 EST ---
Mock build failed for rawhide i386
main.c:29:26: error: libvolume_id.h: No such file or directory

You Need to add libvolume_id-devel in BuildRequires



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

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


[Bug 198835] Review Request: atlascpp - WorldForge message protocol library

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

Summary: Review Request: atlascpp - WorldForge message protocol library
Alias: Atlas-C++

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: Atlas-C++ - |Review Request: atlascpp -
   |WorldForge message protocol |WorldForge message protocol
   |library |library




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

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


[Bug 198827] Review Request: perl-Statistics-Descriptive - Perl module of basic descriptive statistical functions

2006-07-15 Thread bugzilla
Please 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-Statistics-Descriptive - Perl module of basic 
descriptive statistical functions


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 04:11 EST ---
%{?dist} is missing from the release tag, perhaps by accident?  Either way, I
have nothing else to add, 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 190156] Review Request: php-pear-HTTP

2006-07-15 Thread bugzilla
Please 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-HTTP


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 04:56 EST ---
*** Bug 197561 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 197967] Review Request: gkrellm - Multiple stacked system monitors in one process

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

Summary: Review Request: gkrellm - Multiple stacked system monitors in one 
process


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 05:07 EST ---
- "standard" BuildRoot not used
- useradd/groupadd dependencies missing
- chkconfig dependencies should be context marked, and chkconfig called 
  consistently (with full path)
- using a macro for %{flags} seems a bit odd, normal shell variables should 
work 
  just fine
- %{?_smp_mflags} missing
- could use %{_initrddir} for the init script location
- could use init script directly in %preun daemon
- "SMP CPU" sounds odd in the description (gkrellm does UP CPU too), and it
  could be improved a bit otherwise too

Will attach a patch for the above.

Another random note (for upstream?):

- gkrellm.pc doesn't look very useful at the moment.  For plugin/theme packages
  it would be nice to have the lib and data dirs defined in it, for example by
  adding pkgdatadir=%{_datadir}/gkrellm2 and pkglibdir=%{_libdir}/gkrellm2 in
  it; then those could be queried like pkg-config gkrellm --variable=pkglibdir

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

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


[Bug 197967] Review Request: gkrellm - Multiple stacked system monitors in one process

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

Summary: Review Request: gkrellm - Multiple stacked system monitors in one 
process


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 05:09 EST ---
Created an attachment (id=132478)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=132478&action=view)
Suggested patch for findings in comment 5


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

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


[Bug 197967] Review Request: gkrellm - Multiple stacked system monitors in one process

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

Summary: Review Request: gkrellm - Multiple stacked system monitors in one 
process


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 05:13 EST ---
Oh, and gkrellmd should be condrestarted on -daemon upgrades.  I'd also add LSB
action aliases to the init script (try-restart -> condrestart, force-reload ->
restart).

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

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


[Bug 198827] Review Request: perl-Statistics-Descriptive - Perl module of basic descriptive statistical functions

2006-07-15 Thread bugzilla
Please 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-Statistics-Descriptive - Perl module of basic 
descriptive statistical functions


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 05:41 EST ---
Thanks, I indeed forgot the %{?dist} tag. 

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

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


[Bug 198198] Review Request: ooo2txt - Convert OpenOffice documents to simple text

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

Summary: Review Request: ooo2txt - Convert OpenOffice documents to simple text


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 05:43 EST ---
Thanks, pushed 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 198287] Review Request: python-paste - Tools for using a Web Server Gateway Interface stack

2006-07-15 Thread bugzilla
Please 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-paste - Tools for using a Web Server Gateway 
Interface stack


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 05:59 EST ---
There is an inconsistent use of RPM_BUILD_ROOT and %buildroot
(also in other Paste related python 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 198878] Review Request: mutagen - Python module to handle audio metadata

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

Summary: Review Request: mutagen - Python module to handle audio metadata


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 06:07 EST ---
Created an attachment (id=132479)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=132479&action=view)
mutagen-1.5.1-2.spec

New SPEC file version (1.5.1-2)

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

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


[Bug 198878] Review Request: mutagen - Python module to handle audio metadata

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

Summary: Review Request: mutagen - Python module to handle audio metadata


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 06:08 EST ---
Thanks for review this package.

The line:
Requires:   python-abi = %(%{__python} -c "import sys ; print sys.version[:3]"
is from fedora-rpmdevtools template so I didn't change it and, as I see, other 
SPEC files also has this line so I don't delete it.

W: mutagen wrong-file-end-of-line-encoding /usr/share/doc/mutagen-1.5.1/TUTORIAL
I checked it and I noticed that TUTORIAL file has Windows-style line endings,
so there is \r\n instead of \n. In new SPEC this is fixed.

E: mutagen non-executable-script ...
The first line of file is
#! /usr/bin/env python
and for this reason rpmlint treat this file as executable but it shouldn't be 
executable. I think we can leave it alone.
I've done new SPEC with your and my fixes.

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

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


[Bug 198878] Review Request: mutagen - Python module to handle audio metadata

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

Summary: Review Request: mutagen - Python module to handle audio metadata


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #132479|application/octet-stream|text/plain
  mime type||




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

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


[Bug 194470] Review Request: php-magickwand

2006-07-15 Thread bugzilla
Please 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-magickwand


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 06:17 EST ---
As per current status of http://fedoraproject.org/wiki/Packaging/PHP, PHP RPM 
packages like this aren't handled, yet. And of my understanding, any package
not listed at http://pecl.php.net/, isn't a PHP PECL package.

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

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


[Bug 197967] Review Request: gkrellm - Multiple stacked system monitors in one process

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

Summary: Review Request: gkrellm - Multiple stacked system monitors in one 
process


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 07:01 EST ---
(In reply to comment #5)
Thanks for the patch, I've applied it, but I've undone this:

> - could use init script directly in %preun daemon
I agree, but for cases where a full example is given on the ScriptletSnippets
wiki page, I always use the code from the wiki in the name of consistency across
FE as a whole. And the ScriptletSnippets wiki page uses /sbin/service:
http://fedoraproject.org/wiki/ScriptletSnippets

Also the current code in the wiki doesn't use " || :", so neither does this
version of gkrellm (for the service stuff), that can be fixed if you want 
though.

> Another random note (for upstream?):
> 
> - gkrellm.pc doesn't look very useful at the moment.  For plugin/theme 
> packages
>   it would be nice to have the lib and data dirs defined in it, for example by
>   adding pkgdatadir=%{_datadir}/gkrellm2 and pkglibdir=%{_libdir}/gkrellm2 in
>   it; then those could be queried like pkg-config gkrellm --variable=pkglibdir

I'll send this upstream.

(In reply to comment #7)
> Oh, and gkrellmd should be condrestarted on -daemon upgrades.  I'd also add 
> LSB
> action aliases to the init script (try-restart -> condrestart, force-reload ->
> restart).

Done and done.

New version at:
Spec URL: http://people.atrpms.net/~hdegoede/gkrellm.spec
SRPM URL: http://people.atrpms.net/~hdegoede/gkrellm-2.2.9-6.src.rpm

Changes:
* Sat Jul 15 2006 Hans de Goede <[EMAIL PROTECTED]> 2.2.9-6
- Various specfile improvements by Ville Skyttä ([EMAIL PROTECTED])
- Make the daemon package scripts match the ScriptletSnippets wiki page
- Add LSB aliases (try-restart, force-reload) to the -daemon initscript
- Add %%{?dist} to the release for consistency with other packages I maintain



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

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


[Bug 198928] Review Request: lsscsi

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

Summary: Review Request: lsscsi


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 09:41 EST ---
(In reply to comment #1)
> * In build, why not use "%configure"?  Similarly, in install, why not
> "%makeinstall"? 

%makeinstall is deprecated. See the "Macros" section of the Packaging 
Guidelines.

http://fedoraproject.org/wiki/Packaging/Guidelines

> * There may be comments on the test before "rm -rf ..." in %clean.  I think 
> it's
> ok, as it will always evaluate to true in the buildsys, but it could be nixed
> for brevity's sake.

Not only that but the tests are completely redundant since a buildroot is
explicitly specified in the spec file.


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

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


[Bug 198878] Review Request: mutagen - Python module to handle audio metadata

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

Summary: Review Request: mutagen - Python module to handle audio metadata


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 09:50 EST ---
(In reply to comment #4)
> Thanks for review this package.
> 
> The line:
> Requires:   python-abi = %(%{__python} -c "import sys ; print sys.version[:3]"
> is from fedora-rpmdevtools template so I didn't change it and, as I see, 
> other 
> SPEC files also has this line so I don't delete it.

The python spec template will soon be fixed to get rid of this:

http://www.redhat.com/archives/fedora-extras-list/2006-July/msg00557.html

If you check the "requires" of the built package on FC4 or later, you should
find a dependency on "python(abi) = 2.4" automatically added by RPM, making the
python-abi dependency in the spec file redundant.

> E: mutagen non-executable-script ...
> The first line of file is
> #! /usr/bin/env python
> and for this reason rpmlint treat this file as executable but it shouldn't be 
> executable. I think we can leave it alone.

rpmlint knows it's not executable but thinks that it's a script (which it isn't)
because of the "#! /usr/bin/env python" as you say. You can shut rpmlint up by
editing the first line out of the affected files in the %prep stage of the 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 174021] Review Request: aplus-fsf - Advanced APL Interpreter

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

Summary: Review Request: aplus-fsf - Advanced APL Interpreter


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 11:12 EST ---
You're right about renaming the xemacs subpackage; the naming guidelines have
changed since this review started.

Unfortunately my testing infrastructure has changed as well, and I'm now doing a
proper rpmlint on the installed package, which turns up an additional category
of problems.

Firstly there's this warning about the srpm which I don't really have a problem
with:
   W: aplus-fsf setup-not-quiet
It wants you to use "-q" on the %setup line.

Then there's three no-documentation warnings, which are OK:
   W: fonts-truetype-apl no-documentation
   W: fonts-x11-apl no-documentation
   W: xemacs-aplus-fsf no-documentation

And finally there are nearly 3800 undefined-non-weak-symbol warnings, of the 
form:
   W: aplus-fsf undefined-non-weak-symbol /usr/lib64/libesf-4.20.2.so P1
   W: aplus-fsf undefined-non-weak-symbol /usr/lib64/libesf-4.20.2.so APL
I will attach a full list, or you can just run "rpmlint aplus-fsf" with the
package already installed.  It looks like the libraries themselves are merely
compiled but not linked against each other; the loading program has to be set up
to link in all of the necessary libraries in the proper order.  It seems this
not allowed in Fedora, but I'm not sure how reasonable it is to fix it.  I'm
asking around to find how strong the prohibition is.

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

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


[Bug 174021] Review Request: aplus-fsf - Advanced APL Interpreter

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

Summary: Review Request: aplus-fsf - Advanced APL Interpreter


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 11:16 EST ---
Created an attachment (id=132489)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=132489&action=view)
List of undefined-non-weak-symbol warnings


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

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


[Bug 198287] Review Request: python-paste - Tools for using a Web Server Gateway Interface stack

2006-07-15 Thread bugzilla
Please 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-paste - Tools for using a Web Server Gateway 
Interface stack


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 12:23 EST ---
http://people.redhat.com/lmacken/python-paste.spec
http://people.redhat.com/lmacken/python-paste-0.9.3-2.src.rpm

* Sat Jul 15 2006 Luke Macken <[EMAIL PROTECTED]> - 0.9.3-2
- Clean up docs directory
- Remove shebang from from non-executable scripts
- Use consistent build root variables


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

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


[Bug 198880] Review Request: perl-POE-Component-Client-DNS

2006-07-15 Thread bugzilla
Please 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-POE-Component-Client-DNS


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 12:44 EST ---
I have to tell you that the "0" release violates the naming guidelines; please
make sure to bump this to "1" before you build.

I enabled the test suite just to try it, but I don't think it works under mock
due to an undefined network configuration.  However, five of the tests did run
properly.

* source files match upstream:
   17d8823b50185e4a2536cad45410  POE-Component-Client-DNS-0.99.tar.gz
O package meets naming and packaging guidelines (0 release should be 1)
* 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.
O BuildRequires are proper (BR: perl is redundant)
O No need to pass any compiler flags.
* %clean is present.
* package builds in mock (development, x86_64).
* rpmlint is silent.
* final provides and requires are sane:
   perl(POE::Component::Client::DNS) = 0.99
   perl-POE-Component-Client-DNS = 0.99-0.fc6
  =
   perl(:MODULE_COMPAT_5.8.8)
   perl(Carp)
   perl(Net::DNS)
   perl(POE)
   perl(Socket)
   perl(constant)
   perl(strict)
   perl(vars)
* noarch package; no debuginfo.
* %check is present but necessarily disabled for mock.
* 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.
* 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.

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 198881] Review Request: perl-POE-Filter-IRCD

2006-07-15 Thread bugzilla
Please 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-POE-Filter-IRCD


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


[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 198878] Review Request: mutagen - Python module to handle audio metadata

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

Summary: Review Request: mutagen - Python module to handle audio metadata


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #132479|0   |1
is obsolete||




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 13:19 EST ---
Created an attachment (id=132496)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=132496&action=view)
mutagen-1.5.1-3.spec

Okay, I've made new SPEC file with deleted python-abi require and fix in %prep
section.

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

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


[Bug 199016] New: Review Request: mxml - Miniature XML development library

2006-07-15 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=199016

   Summary: Review Request: mxml - Miniature XML development library
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://people.redhat.com/~green/FE/FC5/mxml.spec
SRPM URL: http://people.redhat.com/~green/FE/FC5/mxml-2.2.2-2.src.rpm
Description: Mini-XML is a small XML parsing library that you can use to read 
XML
and XML-like data files in your application without requiring large
non-standard libraries.

mxml is required by zynaddsubfx, which I also plan to submit.
This spec file is based on the PlanetCCRMA 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 183438] Review Request: idioskopos (C++ Introspection Library)

2006-07-15 Thread bugzilla
Please 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

 Status|NEW |CLOSED
 Resolution||DEFERRED




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 14:25 EST ---
This library was submitted as a dependency for papyrus and conexus. Neither
depend on this library anymore, so I'm withdrawing this submission. If there is
other interest in this library, I'll resubmit.

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

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


[Bug 189400] Review Request: em8300(-kmod) - Hollywood+/DXR3 hardware MPEG decoder drivers and tools

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

Summary: Review Request: em8300(-kmod) - Hollywood+/DXR3 hardware MPEG decoder 
drivers and tools


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 14:26 EST ---
Review:

== em8300 ==
rpmlint:

SRPM:
no output

RPM:
W: em8300-devel no-documentation
-> can be ignored

* download of Source0 doesn't work -- seems to be a sourceforge problem afaics
* Open-Source-License (GPL)
* Source matches upstream
* package is named according to the guidelines
* builds in mock on ix86-FC5
 
== em8300-kmod ==

rpmlint:

SRPM:
no output

RPMS:
W: kmod-em8300 summary-not-capitalized em8300 kernel module(s)
-> This is probably the right thing in this case

W: kmod-em8300 unstripped-binary-or-object
/lib/modules/2.6.17-1.2139_FC5/extra/em8300/adv717x.ko
W: kmod-em8300 unstripped-binary-or-object
/lib/modules/2.6.17-1.2139_FC5/extra/em8300/bt865.ko
W: kmod-em8300 unstripped-binary-or-object
/lib/modules/2.6.17-1.2139_FC5/extra/em8300/em8300.ko
-> they get stripped during build, we will ignore that

W: kmod-em8300 no-documentation
-> expected behaviour for kmod-packages

Review:
* package is named according to the guidelines
* Open-Source-License (GPL)
* Source matches upstream
* builds in mock on ix86-FC5
* matches fedora kmod scheme

== Summary ==

both nearly approved -- waiting for FESCo approval of kmod (we forgot to discuss
those in the last meeting -- sorry)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are 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-07-15 Thread bugzilla
Please 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

OtherBugsDependingO|163776  |
  nThis||




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

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


[Bug 198880] Review Request: perl-POE-Component-Client-DNS

2006-07-15 Thread bugzilla
Please 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-POE-Component-Client-DNS


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 15:09 EST ---
Given the recent thread on the list, I'll start submitting future packages with
an initial release of 1.

+Import to CVS
+Add to owners.list
+Bump release, build for devel
+devel build succeeds
+Request branching (FC-4, FC-5)
+Close bug

Thanks for the review!

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

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


[Bug 198884] Review Request: perl-POE-Component-Client-Keepalive

2006-07-15 Thread bugzilla
Please 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-POE-Component-Client-Keepalive


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


Bug 198884 depends on bug 198880, which changed state.

Bug 198880 Summary: Review Request: perl-POE-Component-Client-DNS
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=198880

   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 198882] Review Request: perl-POE-Component-IRC

2006-07-15 Thread bugzilla
Please 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-POE-Component-IRC


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


Bug 198882 depends on bug 198880, which changed state.

Bug 198880 Summary: Review Request: perl-POE-Component-Client-DNS
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=198880

   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 183322] Review Request: conexus (network and serial I/O library with Gtkmm widgets)

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

Summary: Review Request: conexus (network and serial I/O library with Gtkmm 
widgets)


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 15:10 EST ---
I've submitted another bug report for the conexusmm library which now has the
Gtkmm widgets. The spec file for the conexusmm library is identical to this one
with the exception of the dependencies and the inclusion of a conexusmm-demo
application that is based on gtkdemo.

All the issues of the previous review have been taken care of in the revised
spec along with several other issues that came up in the bit library review.

Here are the new files:
Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/conexus.spec

SRPM Name or Url:
http://miskatonic.cs.nmsu.edu/pub/fedora/5/srpms/conexus-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 199020] New: Review Request: conexusmm (Gtkmm widgets for the conexus network and serial I/O library)

2006-07-15 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=199020

   Summary: Review Request: conexusmm (Gtkmm widgets for the conexus
network and serial I/O library)
   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://miskatonic.cs.nmsu.edu/pub/conexusmm.spec

SRPM URL: 
http://miskatonic.cs.nmsu.edu/pub/fedora/5/srpms/conexusmm-0.2.0-1.src.rpm

Description: 

conexusmm provides Gtkmm widgets for the conexus library objects. These widgets 
include IPv4 and IPv6 address and host entry boxes (including validated IPv4 
dotted decimal address), serial port configuration widgets, et. al.

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

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


[Bug 199020] Review Request: conexusmm (Gtkmm widgets for the conexus network and serial I/O library)

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

Summary: Review Request: conexusmm (Gtkmm widgets for the conexus network and 
serial I/O library)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||183322




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

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


[Bug 183322] Review Request: conexus (network and serial I/O library with Gtkmm widgets)

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

Summary: Review Request: conexus (network and serial I/O library with Gtkmm 
widgets)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||199020
  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 183439] Review Request: papyrus (Canvas drawing library based on cairo/cairomm)

2006-07-15 Thread bugzilla
Please 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)
Alias: papyrus

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||papyrus




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

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


[Bug 192052] Review Request: bitgtkmm (Gtkmm widgets for the bit library)

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

Summary: Review Request: bitgtkmm (Gtkmm widgets for the bit library)
Alias: bitgtkmm

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||bitgtkmm




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

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


[Bug 199020] Review Request: conexusmm (Gtkmm widgets for the conexus network and serial I/O library)

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

Summary: Review Request: conexusmm (Gtkmm widgets for the conexus network and 
serial I/O library)
Alias: conexusmm

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||conexusmm




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

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


[Bug 196748] Review Request: setroubleshoot - automatic diagnosis of SELinux problems

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

Summary: Review Request: setroubleshoot - automatic diagnosis of SELinux 
problems


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 16:10 EST ---
Created an attachment (id=132498)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=132498&action=view)
Cosmetic fix for setroubleshoot usage statement

Trivial and cosmetic, but I'm just starting to learn Python.

-- 
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 199021] New: Review Request: zynaddsubfx - Real-time software synthesizer

2006-07-15 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=199021

   Summary: Review Request: zynaddsubfx - Real-time software
synthesizer
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://people.redhat.com/~green/FE/FC5/zynaddsubfx.spec
SRPM URL: http://people.redhat.com/~green/FE/FC5/zynaddsubfx-2.2.1-5.src.rpm
Description: ZynAddSubFX is a opensource software synthesizer capable of making 
a
countless number of instruments, from some common heared from
expensive hardware to interesting sounds that you'll boost to an
amazing universe of sounds.

This package was derived from the latest planetccrma package.
It depends on mxml, which I've also submitted for review.

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

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


[Bug 199021] Review Request: zynaddsubfx - Real-time software synthesizer

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

Summary: Review Request: zynaddsubfx - Real-time software synthesizer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||199016




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

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


[Bug 199016] Review Request: mxml - Miniature XML development library

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

Summary: Review Request: mxml - Miniature XML development library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||199021
  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 198944] Review Request: compiz

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

Summary: Review Request: compiz


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 17:31 EST ---
I'd propose to close it as obsolete once compiz shows up in core

-- 
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 199028] New: Review Request: perl-eperl

2006-07-15 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=199028

   Summary: Review Request: perl-eperl
   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://ftp.kspei.com/pub/steve/rpms/perl-eperl/perl-eperl.spec
SRPM URL: http://ftp.kspei.com/pub/steve/rpms/perl-eperl-2.2.14-2.src.rpm
Description:
ePerl interprets an ASCII file bristled with Perl 5 program statements
by evaluating the Perl 5 code while passing through the plain ASCII
data. It can operate in various ways: As a stand-alone Unix filter or
integrated Perl 5 module for general file generation tasks and as a
powerful Webserver scripting language for dynamic HTML page
programming.

The documentation and latest release can be found on
http://www.ossp.org/pkg/tool/eperl/

Note that this package does not include the Apache::ePerl module,
which is designed for mod_perl 1.x.

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

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


[Bug 199029] New: Review Request: jokosher

2006-07-15 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=199029

   Summary: Review Request: jokosher
   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.iammetal.co.uk/jokosher.spec
SRPM URL: http://www.iammetal.co.uk/jokosher-0.1-1.src.rpm
Description: Jokosher is a multi-track non-linear audio editor with a prime 
focus on usability. It is developed in Python and is Open Source.

The main goals of Jokosher functionality are:

* The ability to record from any ALSA sound card, including multi-input 
cards such as the M-Audio Delta 44.
* Non-destructive editing.
* Undo/Redo (at least to a reasonable limit, but preferably unlimited).
* Be able to edit the volume curve in different parts of the track.
* Resizable track views to easily zoom in and out of a waveform.
* Support for effects plug-ins, most notably LADSPA.
* Be able to apply effects to an entire track or a selected portion of a 
track. When applying effects, there should be the ability to preview the sound 
with the effect before it is applied.
* It should be able to master to OGG, MP3 and WAV.
* It should be able to import OGG, MP3 and WAV.
* Most important, it should be easy and intuitive to use.

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

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


[Bug 189322] Review Request: rosegarden4

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

Summary: Review Request: rosegarden4


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||158728
  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 189322] Review Request: rosegarden4

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

Summary: Review Request: rosegarden4


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 22:47 EST ---
Created an attachment (id=132518)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=132518&action=view)
Patch to look for DSSI plugins in lib64 directories

Please apply this patch when you check rosegarden4 in.  It tells rosegarden
where to find DSSI plugins on x86-64 systems.  

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

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


[Bug 198922] Review Request: dejavu-lgc-fonts

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

Summary: Review Request: dejavu-lgc-fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-15 23:32 EST ---
(In reply to comment #2)
> Tech ack. Are we not obsoleting Vera?

Not yet, as we are still evaluating it.  Withdrawal is not quite out of question
yet.

 
> Out of curiousity, what locales does this give us *new* coverage with, outside
> of Greek?

I don't know.  Most people just report whether DejaVu works for them, not LGC. 
So it's a bit hard to know.  The langcover.txt in the tarball is empty.

-- 
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 198834] Review Request: sage - OpenGL extensions library using SDL

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

Summary: Review Request: sage -  OpenGL extensions library using SDL
Alias: sage

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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-16 00:36 EST ---
Updated package that includes --disable-static and removes -O3 from CFLAGS:

http://www.kobold.org/~wart/fedora/sage-0.1.2-3.src.rpm
http://www.kobold.org/~wart/fedora/sage.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 198836] Review Request: freetype1 compatibility FreeType 1.x

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

Summary: Review Request: freetype1 compatibility FreeType 1.x


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-16 00:48 EST ---
(In reply to comment #1)
> Where does freetype-1.4pre come from?  The latest I can find is 1.3.1:
> http://sourceforge.net/project/showfiles.php?group_id=3157&package_id=3068

Don't know.  It probably is rolled by someone specifically for Red Hat Linux /
Fedora.  I just got it from freetype-2.1.10 SRPM.


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

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


[Bug 198836] Review Request: freetype1 compatibility FreeType 1.x

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

Summary: Review Request: freetype1 compatibility FreeType 1.x


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-16 00:53 EST ---
(In reply to comment #3)
> > %package utils
> > Summary: A collection of FreeType 1.x utilities
> > Group: System Environment/Libraries
> > Requires: %{name} = %{version}-%{release}
> > # Upgrade path
> > Provides: freetype-utils = 2.2.0-1
> > Obsoletes: freetype-utils < 2.2.0-1
> 
> Very ugly. At the time this freetype-utils sub-package was
> introduced, it should have started at 1.x, not 2.x.
> 
> You could still do that with a proper Epoch.

Right.  So, should I use Epoch instead?  Like:

Provides: freetype-utils = 1:1.4-0.1.pre


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

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


[Bug 198836] Review Request: freetype1 compatibility FreeType 1.x

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

Summary: Review Request: freetype1 compatibility FreeType 1.x


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-16 00:54 EST ---
(In reply to comment #2)
> And you should omit static libs (%_libdir/lib*.a) and %{_libdir}/lib*.la 
> files.

Correct.  The freetype package in Core is not dropping those.  Will fix 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 199016] Review Request: mxml - Miniature XML development library

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

Summary: Review Request: mxml - Miniature XML development library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-07-16 01:18 EST ---
Created an attachment (id=132521)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=132521&action=view)
Updated mxml spec file

This updated spec file replaces a /usr/lib reference with %{_libdir} for
x86-64.


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

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