[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rakesh Pandit rpan...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #25 from Rakesh Pandit rpan...@redhat.com  2009-06-26 02:25:27 
EDT ---
Thanks .. fixed all

http://rakesh.fedorapeople.org/spec/Mayavi.spec
http://rakesh.fedorapeople.org/srpm/Mayavi-3.2.0-6.fc11.src.rpm

New Package CVS Request
===
Package Name: Mayavi
Short Description: The Mayavi scientific data 3-dimensional visualizer
Owners: oget
Branches: F-10 F-11
InitialCC:

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

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


[Bug 503490] Review Request: ayttm - Universal Instant Messaging Client

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Parag AN(पराग) panem...@gmail.com  2009-06-26 03:01:28 
EDT ---
1) You should not use following
%ifnarch x86_64 # x86_64 does not have brp-compress
%define __spec_install_post %{_libdir}/rpm/brp-compress
%endif

remove this.

2) also I have fixed few more issues as per given in Changelog at
http://paragn.fedorapeople.org/ayttm1.spec
Use this spec and resubmit SPEC and SRPM for review.

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

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


[Bug 506939] Review Request: dracut - a generic initramfs generator

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Harald Hoyer har...@redhat.com  2009-06-26 04:48:51 EDT 
---
The kernel should BuildRequire the dracut-generic package and deliver a generic
initramfs image, which can be chosen as an alternative image to the special
host generated image, which can/should be triggered with a kernel install.

I think the kernel only Requires /sbin/new-kernel-pkg, which is part of grubby.
new-kernel-pkg will be modified to use dracut to build an image.

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

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


[Bug 507501] Review Request: inkboy-fonts - a simple clean latin font

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-06-26 
04:50:46 EDT ---
BTW, upstream would probably be happy to learn of the Fedora packaging

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

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


[Bug 506721] Package Review: nbtk - A toolkit for moblin NetBooks

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #10 from Peter Robinson pbrobin...@gmail.com  2009-06-26 05:19:22 
EDT ---
New Package CVS Request
===
Package Name: nbtk
Short Description: A toolkit for moblin NetBooks
Owners: pbrobinson
Branches: F-11 F-10
InitialCC:

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

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


[Bug 507660] Review Request: xylib - Library for reading x-y data from several file formats

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Jussi Lehtola jussi.leht...@iki.fi  2009-06-26 06:07:14 
EDT ---
- Don't call libtool manually. Remove the last three lines in %install, add
instead BuildRequires: automake and 
 sed -i s|noinst_|bin_|g Makefile.am
 automake
to %setup, this will make the build process install xyconv to %{_bindir}.

- Use SMP make flags. Change 
 make
to
 make %{?_smp_mflags}

- Change
 %{_bindir}/*
to
 %{_bindir}/xyconv
and
 %{_includedir}/*
to
 %{_includedir}/xylib/
(no sense in using wildcards in these cases)

- The attributes
 %defattr(-,root,root)
should be
 %defattr(-,root,root,-)


SHOULD: Use a more recommended version of the BuildRoot tag
http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

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

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


[Bug 508239] New: Review Request: perl-Hash-Merge - Merges arbitrary deep hashes into a single hash

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-Hash-Merge - Merges arbitrary deep hashes into a 
single hash

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

   Summary: Review Request: perl-Hash-Merge - Merges arbitrary
deep hashes into a single hash
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: tcall...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://www.auroralinux.org/people/spot/review/new/perl-Hash-Merge.spec
SRPM URL:
http://www.auroralinux.org/people/spot/review/new/perl-Hash-Merge-0.11-1.fc12.src.rpm
Description: Merges arbitrary deep hashes into a single hash
Koji Rawhide Scratch Build: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=1436825

Note: This is needed to update perl-DBIx-Class in rawhide.

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

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


[Bug 508239] Review Request: perl-Hash-Merge - Merges arbitrary deep hashes into a single hash

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review?




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

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


[Bug 508239] Review Request: perl-Hash-Merge - Merges arbitrary deep hashes into a single hash

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #1 from Parag AN(पराग) panem...@gmail.com  2009-06-26 06:40:08 
EDT ---
+ is ok
- is Need work

Review:
+ package builds in mock (rawhide i586).
koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=1436825

- rpmlint is NOT silent for RPM.
perl-Hash-Merge.noarch: E: non-standard-executable-perm
/usr/lib/perl5/vendor_perl/5.10.0/Hash/Merge.pm 0555
perl-Hash-Merge.noarch: E: script-without-shebang
/usr/lib/perl5/vendor_perl/5.10.0/Hash/Merge.pm
== This need to be fixed.

+ source files match upstream url
23c6a4c3b1f620e47c34c9c9afb962e71195f955  Hash-Merge-0.11.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ make test gave
All tests successful.
Files=1, Tests=45,  0 wallclock secs ( 0.03 usr  0.01 sys +  0.04 cusr  0.00
csys =  0.08 CPU)
+ Package perl-Hash-Merge-0.11-1.fc12.noarch =
Provides: perl(Hash::Merge) = 0.11
Requires: perl(Carp) perl(base) perl(strict) perl(vars) perl(warnings)
+ Not a GUI application

Suggestions:
1) This package should own /usr/lib/perl5/vendor_perl/5.10.0/Hash
please correct this before cvs import.

2) you missed to add to spec
chmod -R u+w $RPM_BUILD_ROOT/*

3) fix rpmlint output.

APPROVED.

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

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


[Bug 508239] Review Request: perl-Hash-Merge - Merges arbitrary deep hashes into a single hash

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Ralf Corsepius rc040...@freenet.de changed:

   What|Removed |Added

 CC||rc040...@freenet.de
   Flag|fedora-review+  |fedora-review?




--- Comment #2 from Ralf Corsepius rc040...@freenet.de  2009-06-26 06:45:59 
EDT ---
Package must own %{perl_vendordir}/Hash

Revoking APPROVAL

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

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


[Bug 508254] New: Review Request: perl-namespace-clean - Keeps imports and functions out of your namespace

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-namespace-clean - Keeps imports and functions out 
of your namespace

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

   Summary: Review Request: perl-namespace-clean - Keeps imports
and functions out of your namespace
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: tcall...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://www.auroralinux.org/people/spot/review/new/perl-namespace-clean.spec
SRPM URL:
http://www.auroralinux.org/people/spot/review/new/perl-namespace-clean-0.11-1.fc12.src.rpm
Description: 
When you define a function, or import one, into a Perl package, it will
naturally also be available as a method. This does not per se cause
problems, but it can complicate subclassing and, for example, plugin
classes that are included via multiple inheritance by loading them as
base classes.

The namespace::clean pragma will remove all previously declared or
imported symbols at the end of the current package's compile cycle.
Functions called in the package itself will still be bound by their
name, but they won't show up as methods on your class or instances.

Koji Rawhide Scratch Build: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=1436845

Note: This is needed to update perl-DBIx-Class in rawhide.

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

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


[Bug 508239] Review Request: perl-Hash-Merge - Merges arbitrary deep hashes into a single hash

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Tom spot Callaway tcall...@redhat.com  2009-06-26 
06:53:59 EDT ---
Yeah, he pointed that out. Fixed along with the other issues in -2:

New SRPM:
http://www.auroralinux.org/people/spot/review/new/perl-Hash-Merge-0.11-2.fc12.src.rpm
New SPEC:
http://www.auroralinux.org/people/spot/review/new/perl-Hash-Merge.spec

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

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


[Bug 508254] Review Request: perl-namespace-clean - Keeps imports and functions out of your namespace

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review?




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

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


[Bug 508239] Review Request: perl-Hash-Merge - Merges arbitrary deep hashes into a single hash

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Parag AN(पराग) panem...@gmail.com  2009-06-26 07:05:26 
EDT ---
Ralf,
 I thought I have already pointed that and I see there is no need to block
this review just for that change.

Anyway as you are now reviewer for this package, feel free to check -2 release.

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

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


[Bug 508254] Review Request: perl-namespace-clean - Keeps imports and functions out of your namespace

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Parag AN(पराग) panem...@gmail.com  2009-06-26 07:15:09 
EDT ---
This package is already in Fedora. See
http://koji.fedoraproject.org/koji/packageinfo?packageID=6752

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

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


[Bug 461390] Review Request: perl-namespace-clean - Keep your namespace tidy

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 CC||tcall...@redhat.com




--- Comment #6 from Parag AN(पराग) panem...@gmail.com  2009-06-26 07:18:28 
EDT ---
*** Bug 508254 has been marked as a duplicate of this bug. ***

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

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


[Bug 508254] Review Request: perl-namespace-clean - Keeps imports and functions out of your namespace

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DUPLICATE
 AssignedTo|panem...@gmail.com  |nob...@fedoraproject.org
   Flag|fedora-review?  |




--- Comment #2 from Parag AN(पराग) panem...@gmail.com  2009-06-26 07:18:28 
EDT ---
hmm. I forgot that it was me who reviewed this package.

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

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

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


[Bug 506720] Review Request: uClibc - C library for embedded Linux

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Ivana Varekova varek...@redhat.com changed:

   What|Removed |Added

   Flag|needinfo?(varek...@redhat.c |
   |om) |




--- Comment #9 from Ivana Varekova varek...@redhat.com  2009-06-26 07:49:58 
EDT ---
Thanks fixed:
Spec URL: http://people.redhat.com/varekova/new/uClibc.spec
SRPM URL: http://people.redhat.com/varekova/new/uClibc-0.9.30.1-1.fc12.src.rpm

add notes 
- Please consider setting PREFIX instead of DEVEL_PREFIX
for now I leave DEVEL_PREFIX, but if you want it then I will do the change
- CFLAGS do not use %{optflags}, it probably doesn't make sense in this case
I don't think tit has a sense
-  The ExcludeArch: comment will have to be filed as a bug component creation
in
bugzilla
I plan to create the bz, thanks

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

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


[Bug 507801] Review Request: perl-XML-Parser-Lite-Tree - Lightweight XML tree builder

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Lubomir Rintel lkund...@v3.sk  2009-06-26 08:58:26 EDT ---
1.) Licensing

README states License: Perl Artistic License (which sounds bad as it's not
clear if GPL is permitted as well)m while XML::Parser::LiteCopy manual reads
This library is free software; you can redistribute it and/or modify it under
the same terms as Perl itself. (which is good). Please ask upstream for
clarification.

2.) Please remove Require: perl(Test::More) dependency, the runtime does not
depend on the testing framework.

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

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


[Bug 507801] Review Request: perl-XML-Parser-Lite-Tree - Lightweight XML tree builder

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Lubomir Rintel lkund...@v3.sk  2009-06-26 09:07:54 EDT ---
* rpmlint is quiet

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

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


[Bug 507801] Review Request: perl-XML-Parser-Lite-Tree - Lightweight XML tree builder

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Lubomir Rintel lkund...@v3.sk  2009-06-26 09:07:34 EDT ---
* name is well according to the guidelines
* source matches upstream
* clean and legible SPEC file, American English
* runs testsuite
* builds file in mock
* filelists, documentation ok

Needs work:
- sane provides/requires
- license ok

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

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


[Bug 507804] Review Request: perl-Flickr-API - Flickr API interface

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Lubomir Rintel lkund...@v3.sk  2009-06-26 09:03:58 EDT ---
1.) Licensing

README states License: Perl Artistic License (which sounds bad as it's not
clear if GPL is permitted as well)m while XML::Parser::LiteCopy manual reads
This library is free software; you can redistribute it and/or modify it under
the same terms as Perl itself. (which is good). Please ask upstream for
clarification.

2.) Please remove useless requires. RPM generates the list for you and the
runtime does not depend on the testing framework:

Requires:   perl(HTTP::Request)
Requires:   perl(HTTP::Response)
Requires:   perl(LWP::UserAgent)
Requires:   perl(Test::More)

(Explicitly versioned ones are ok)

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

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


[Bug 507804] Review Request: perl-Flickr-API - Flickr API interface

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Lubomir Rintel lkund...@v3.sk  2009-06-26 09:08:06 EDT ---
* rpmlint is quiet
* name is well according to the guidelines
* source matches upstream
* clean and legible SPEC file, American English
* runs testsuite
* builds file in mock
* filelists, documentation ok

Needs work:
- sane provides/requires
- license ok

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

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


[Bug 507805] Review Request: perl-Flickr-Upload - Flickr upload tool

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #1 from Lubomir Rintel lkund...@v3.sk  2009-06-26 09:19:18 EDT ---
* rpmlint is quiet
* name is well according to the guidelines
* source matches upstream
* clean and legible SPEC file, American English
* runs testsuite
* builds file in mock
* filelists, documentation ok
* sane provides/requires
* license ok

APPROVED

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

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


[Bug 507804] Review Request: perl-Flickr-API - Flickr API interface

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 Blocks||507805




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

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


[Bug 507805] Review Request: perl-Flickr-Upload - Flickr upload tool

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 Depends on||507804, 507801




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

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


[Bug 507801] Review Request: perl-XML-Parser-Lite-Tree - Lightweight XML tree builder

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 Blocks||507805




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

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


[Bug 504469] Review Request: rubygem-term-ansicolor - Ruby library that colors strings using ANSI escape sequences

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-06-26 
09:21:05 EDT ---
ping again?

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

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


[Bug 489564] Review Request: Blueman - Bluetooth Manager

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |
   Flag|fedora-review?  |fedora-review+




--- Comment #47 from Lubomir Rintel lkund...@v3.sk  2009-06-26 09:26:41 EDT 
---
'nushio' has been sponsored
removing NEEDSPONSOR now, the package can be

APPROVED

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

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


[Bug 489014] Review Request: gnome-do-plugins - Plugins for Gnome Do

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #31 from Juan Manuel Rodriguez nus...@fedoraproject.org  
2009-06-26 09:33:36 EDT ---
No problem with that RPM, but like I said, Toshio's the one that provided the
fix, I simply pointed out the error. 

This fix should let me compile gnome-do-plugins for x86_64, unless there's
another similar bug on another of the packages.

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

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


[Bug 507703] Review Request: perl-Proc-Simple - Launch and control background processes

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Lubomir Rintel lkund...@v3.sk  2009-06-26 09:34:20 EDT ---
1.) Please remove Require: perl(Test::More) dependency, the runtime does not
depend on the testing framework.  

2.) Licensing

Proc/Simple.pm contains this:
# This program is free software, you can redistribute it and/or 
# modify it under the same terms as Perl itself.

Neither README nor POD documentation contains it -- please ask upstream to
include the licensing conditions there.

* rpmlint is quiet
* name is well according to the guidelines
* source matches upstream
* clean and legible SPEC file, American English
* runs testsuite
* builds file in mock
* filelists, documentation ok

Needs work:
- sane provides/requires
- license

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

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


[Bug 489564] Review Request: Blueman - Bluetooth Manager

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #48 from Juan Manuel Rodriguez nus...@fedoraproject.org  
2009-06-26 09:39:51 EDT ---
Thanks Christian, Rajeesh and Lubomir for helping me get approved!

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

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


[Bug 489014] Review Request: gnome-do-plugins - Plugins for Gnome Do

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #32 from Juan Manuel Rodriguez nus...@fedoraproject.org  
2009-06-26 09:42:07 EDT ---
Paul, could you do a build-override with that package, so I can try to build
gnome-do-plugins on x86_64 on koji?

Thanks!

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

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


[Bug 489564] Review Request: Blueman - Bluetooth Manager

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #49 from Juan Manuel Rodriguez nus...@fedoraproject.org  
2009-06-26 09:54:46 EDT ---
New Package CVS Request
===
Package Name: blueman
Short Description: GTK+ Bluetooth Manager
Owners: nushio
Branches: devel, F-10, F-11
InitialCC:

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

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


[Bug 504469] Review Request: rubygem-term-ansicolor - Ruby library that colors strings using ANSI escape sequences

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

   Flag|needinfo?(lkund...@v3.sk)   |




--- Comment #10 from Lubomir Rintel lkund...@v3.sk  2009-06-26 09:56:37 EDT 
---
Sorry, Mamoru; I've been quite busy with rest of $daywork these day (which is
quite a bad excuse for not responding). I've integrated your suggestions:

SPEC: http://v3.sk/~lkundrak/gdc-ruby-stack/SPECS/rubygem-term-ansicolor.spec
SRPM:
http://v3.sk/~lkundrak/gdc-ruby-stack/SRPMS/rubygem-term-ansicolor-1.0.3-3.fc11.src.rpm

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

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


[Bug 489564] Review Request: Blueman - Bluetooth Manager

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Juan Manuel Rodriguez nus...@fedoraproject.org changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 507703] Review Request: perl-Proc-Simple - Launch and control background processes

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Liang Suilong liangsuil...@gmail.com  2009-06-26 10:08:04 
EDT ---
Jan Klepek

1) In my opinion, you maybe need to correct %file section. Because
%{perl_vendorlib}/Proc would be owned by another packages like perl-Proc-*.

I think these are better.

%files
%defattr(-,root,root,-)
%doc Changes README
%{perl_vendorlib}/Proc/*
%{_mandir}/man3/*

2) That BuildArch is noarch seems not to be suitable for it. As I see, building
package for various architectures may be better than noarch. So I advise that
BuildArch should be removed.

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

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


[Bug 459892] Review Request: rubygem-mocha - Mocking and stubbing library

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #26 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-06-26 
10:22:13 EDT ---
So, Jeroen, if you want to be the maintainer of this package,
I will be happy with it. Please set cvs flag again.

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

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


[Bug 507703] Review Request: perl-Proc-Simple - Launch and control background processes

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Lubomir Rintel lkund...@v3.sk  2009-06-26 10:31:16 EDT ---
(In reply to comment #3)
 Jan Klepek
 
 1) In my opinion, you maybe need to correct %file section. Because
 %{perl_vendorlib}/Proc would be owned by another packages like perl-Proc-*.
 
 I think these are better.
 
 %files
 %defattr(-,root,root,-)
 %doc Changes README
 %{perl_vendorlib}/Proc/*
 %{_mandir}/man3/*

This is not right -- that would leave an unowned %{perl_vendorlib}/Proc upon
removal of the package, which violates the guidelines. Sharing directories is
completely fine. Jan, please don't change that.

 2) That BuildArch is noarch seems not to be suitable for it. As I see, 
 building
 package for various architectures may be better than noarch. So I advise that
 BuildArch should be removed.  

Wrong. There's nothing arch-specific in the package; it's definitely noarch.

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

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


[Bug 508316] New: Review Request: emacs-jabber

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: emacs-jabber

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

   Summary: Review Request: emacs-jabber
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rpan...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


SPEC: http://rakesh.fedorapeople.org/spec/emacs-jabber.spec 
SRPM: http://rakesh.fedorapeople.org/srpm/emacs-jabber-0.7.93-1.fc11.src.rpm

Description:
Emacs plugin for jabber protocol.

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

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


[Bug 508316] Review Request: emacs-jabber - Jabber plugin for emacs

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rakesh Pandit rpan...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |emacs-jabber|emacs-jabber - Jabber
   ||plugin for emacs




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

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


[Bug 508318] New: Review Request: mutter - A window manager based on metacity and clutter

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: mutter - A window manager based on metacity and clutter

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

   Summary: Review Request: mutter - A window manager based on
metacity and clutter
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pbrobin...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
Blocks: 506446
   Estimated Hours: 0.0
Classification: Fedora


SPEC: http://pbrobinson.fedorapeople.org/mutter.spec
SRPM:
http://pbrobinson.fedorapeople.org/mutter-2.27.0-0.1.20090626gita13dec3.fc11.src.rpm

A window manager based on metacity and clutter

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

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


[Bug 484547] Review Request: pyfacebook - Python wrapper for Facebook's API

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Ian Weller i...@ianweller.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 484548] Review Request: facebook-notify - Facebook status notifier for GNOME

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 484548 depends on bug 484547, which changed state.

Bug 484547 Summary: Review Request: pyfacebook - Python wrapper for Facebook's 
API
https://bugzilla.redhat.com/show_bug.cgi?id=484547

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||RAWHIDE
 Status|ASSIGNED|CLOSED



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

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


[Bug 506939] Review Request: dracut - a generic initramfs generator

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bill Nottingham nott...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #9 from Bill Nottingham nott...@redhat.com  2009-06-26 10:46:32 
EDT ---
OK. That makes the transition a little messy, but doesn't actually affect the
current packaging.

APPROVED.

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

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


[Bug 504469] Review Request: rubygem-term-ansicolor - Ruby library that colors strings using ANSI escape sequences

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #11 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-06-26 
10:48:51 EDT ---
-
  This package (rubygem-term-ansicolor) is APPROVED by mtasaka
-

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

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


[Bug 508318] Review Request: mutter - A window manager based on metacity and clutter

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Bill Nottingham nott...@redhat.com  2009-06-26 10:53:48 
EDT ---
Just as a drive-by, that URL in the spec isn't right.

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

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


[Bug 506198] Review Request: mediawiki-CategoryTree - Provides a dynamic view of the wiki's category structure as a tree

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Ian Weller i...@ianweller.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 504469] Review Request: rubygem-term-ansicolor - Ruby library that colors strings using ANSI escape sequences

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #12 from Lubomir Rintel lkund...@v3.sk  2009-06-26 11:28:03 EDT 
---
Much thanks! I'm currently in the process of fixing the duplicate file listing
in other rubygem packages as well.

New Package CVS Request
===
Package Name: rubygem-term-ansicolor
Short Description: Ruby library that colors strings using ANSI escape sequences
Owners: hpejakle lkundrak
Branches: F-10 F-11 EL-5

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

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


[Bug 508318] Review Request: mutter - A window manager based on metacity and clutter

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bastien Nocera bnoc...@redhat.com changed:

   What|Removed |Added

 CC||otay...@redhat.com




--- Comment #2 from Bastien Nocera bnoc...@redhat.com  2009-06-26 11:32:50 
EDT ---
Owen, what do you reckon with that?

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

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


[Bug 470293] Review Request: sblim-cmpi-network - SBLIM Network Instrumentation

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Praveen K Paladugu praveen_palad...@dell.com changed:

   What|Removed |Added

 CC||praveen_palad...@dell.com

Bug 470293 depends on bug 501812, which changed state.

Bug 501812 Summary: sblim-cmpi-base file permissions incorrect
https://bugzilla.redhat.com/show_bug.cgi?id=501812

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|ON_QA
 Resolution||NEXTRELEASE
 Status|ON_QA   |CLOSED



--- Comment #4 from Praveen K Paladugu praveen_palad...@dell.com  2009-06-26 
11:33:43 EDT ---
I couldn't download the source rpm. Could you please check the link or provide
with the above mentioned changes?

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

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


[Bug 508318] Review Request: mutter - A window manager based on metacity and clutter

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Peter Robinson pbrobin...@gmail.com  2009-06-26 11:36:24 
EDT ---
(In reply to comment #1)
 Just as a drive-by, that URL in the spec isn't right.  

How right you are. I've set it to moblin.org at the moment as I can't find a
page on gnome.org or in google. Other suggestions welcome.

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

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


[Bug 489014] Review Request: gnome-do-plugins - Plugins for Gnome Do

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #33 from Paul Lange pala...@gmx.de  2009-06-26 11:49:13 EDT ---
Juan,

package is built and update submitted:
https://admin.fedoraproject.org/updates/flickrnet-2.1.5-3.fc11

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

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


[Bug 506581] Review Request: xscope - X Window Protocol Viewer

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Christian Krause c...@plauener.de  2009-06-26 12:07:50 
EDT ---
Hi,

I've reviewed the package and it looks quite good. There are only minor TODOs:
- clarification about the license (I've sent a mail to fedora-legal.)
- description formatting
- functional test
- if possible it would be great if the package would compile in F10, too...

Here is the detailed review:

* rpmlint:  OK
rpmlint SPECS/xscope.spec RPMS/i586/xscope-*
SRPMS/xscope-1.1-1.gitfccbbd6.fc11.src.rpm
3 packages and 1 specfiles checked; 0 errors, 0 warnings.

* naming: OK
- name matches upstream
- spec file name matches package name
- snapshot release tag OK according:
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Snapshot_packages

* License: TODO
- Although the meaning of the license seems to match the BSD license, I'm not
100% sure its a real BSD license since the wording is different. Just to
confirm I've asked on the fedora-legal list for confirmation.
- COPYING file packaged

* specfile in American English and legible: OK

* %description: TODO (minor)
- I would reformat the last sentence in the %description section to use up the
available space of 80 charaters.

* Sources: OK
- Source0 URL ok
- spectool -g xscope.spec works
- sources matches upstream - md5sum:
c37ec177b56d5909584c1672b6beabd5  xscope-1.1.tar.bz2

* Patch0: OK
- patch file can be regenerated by the supplied git diff command
- upstream status of the patch is obvious
fce6e9df7881061013f4acd43742585b  xscope-1.1-diff_to_git.patch

* Compilation: OK
- mock build works
- package builds correctly in koji for F12 and F11, but failed for F10
- RPMOPTFLAGS used
- parallel build supported via _smp_mflags

* debuginfo sub-package: OK
- non-empty
- debuginfo file works together with gdb

* BuildRequires: OK

* Locales handling: OK (n/a)

* shared/static libs, pkgconfig/header/*.la files: OK (n/a)

* packages must own all directories: OK

* files not listed twice: OK

* permissions of files: OK
- %defattr used
- final file permissions OK

* %clean section: OK

* macro usage: OK

* code vs. content: OK (only code)

* large documentation into subpackage: OK (n/a)

* GUI application needs %{name}.desktop: OK (n/a)

* no directories owned which are already owned by other packages: OK

* rm -rf %{buildroot} at the beginning of %{install}: OK

* all filenames UTF8: OK

* functional test: TODO
- running it on the same machine as the X server did not work well:
  - started xscope -v1
  - started DISPLAY=:1 gedit in another console
  - xscope displays some X11 protocol communication
  - but gedit doesn't start up completly, X server + xscope + gedit hangs, it
is necessary to kill xscope by logging in via ssh...
- probably it is just wrong usage - any hints welcome ;-)


Best regards,
Christian

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

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


[Bug 508318] Review Request: mutter - A window manager based on metacity and clutter

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Owen Taylor otay...@redhat.com  2009-06-26 12:53:50 EDT 
---
For the URL, I'd suggest just using the cgit URL for now:

 http://git.gnome.org/cgit/mutter

(I don't think the moblin URL is appropriate.)

Some more drive-by comments:

BuildRequires: gobject-introspection-devel
[...]
  --without-introspection

Those two conflict. We'll definitely need an introspection-enabled package of
Mutter for F12, so we can package gnome-shell. That will mean making sure that
the Clutter package is introspection enabled as well, and that we have
up-to-date versions of gobject-introspection and gir-repository packaged.

BuildRequires: fontconfig-devel
BuildRequires: libglade2-devel
BuildRequires: mesa-libGL-devel
BuildRequires: xorg-x11-proto-devel

These seem extraneous.

BuildRequires: zenity

This should be a Requires: along with a BuildRequires.

  --disable-xinerama

Why?

At a high level, getting to packaging the gnome-shell stack for Fedora:

 gobject-introspection [already old version packaged, I think]
 gir-repository [there's a review request]
 clutter-0.9
 gjs
 mutter
 gnome-shell

Is something that I want the gnome-shell team to get to soon (though probably
not until after GUADEC); and to do that, we'll definitely push out tarball
releases rather than packaging git snapshots. We're happy to have
co-maintainers and people helping out with that.

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

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


[Bug 508318] Review Request: mutter - A window manager based on metacity and clutter

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rudolf Kastl che...@gmail.com changed:

   What|Removed |Added

 CC||che...@gmail.com




--- Comment #5 from Rudolf Kastl che...@gmail.com  2009-06-26 12:56:50 EDT ---
src rpm is broken (check size). probably half upload.

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

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


[Bug 508349] New: Review Request: kdepim-runtime - KDE PIM Runtime Environment

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: kdepim-runtime - KDE PIM Runtime Environment

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

   Summary: Review Request: kdepim-runtime - KDE PIM Runtime
Environment
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: t...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://than.fedorapeople.org/rawhide/kdepim-runtime.spec
SRPM URL
http://than.fedorapeople.org/rawhide/http://than.fedorapeople.org/rawhide/kdepim-runtime-4.2.95-1.fc11.src.rpm
Description: KDE PIM Runtime Environment

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

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


[Bug 484485] Review Request: perl-Fedora-App-ReviewTool - Application classes for reviewtool

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl cw...@alumni.drew.edu changed:

   What|Removed |Added

  Status Whiteboard||NotReady




--- Comment #27 from Chris Weyl cw...@alumni.drew.edu  2009-06-26 13:07:42 
EDT ---
No problem, I've hardly been handling this at the speed I'd like either.

Work/Life are going to continue to essentially dominate my time for the next
couple weeks, so I'm going to mark this on the whiteboard as NotReady and
come back to it in a little bit.

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

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


[Bug 507501] Review Request: inkboy-fonts - a simple clean latin font

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Jim Radford radf...@blackbean.org  2009-06-26 13:08:22 
EDT ---
(In reply to comment #7)
 BTW, upstream would probably be happy to learn of the Fedora packaging  

Thanks for the reminder.  I was waiting for the package it be added to F11 or
at least to hit the rawhide mirrors.

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

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


[Bug 508318] Review Request: mutter - A window manager based on metacity and clutter

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Owen Taylor otay...@redhat.com  2009-06-26 13:12:03 EDT 
---
One more comment - the Summary should be :

 Window and compositing manager based on Clutter

(That's what I used in mutter.doap and GNOME bugzilla) I don't have a
%description handy.

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

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


[Bug 492996] Review Request: mediawiki-Renameuser - An extension that provides a special page for renaming user accounts

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


John Guthrie guth...@counterexample.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #18 from John Guthrie guth...@counterexample.org  2009-06-26 
13:16:27 EDT ---
The build has completed in all requested releases.  Closing this review request
out.

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

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


[Bug 508351] New: Review Request: josm - java openstreetmap editor

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: josm - java openstreetmap editor

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

   Summary: Review Request: josm - java openstreetmap editor
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cedric.oliv...@free.fr
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://cedric.olivier.free.fr/rpms/josm.spec
SRPM URL: http://cedric.olivier.free.fr/rpms/josm-1607-1.fc11.src.rpm
Description: 

Hi, I just finished packaging josm, and I would appreciate a review.
It's my first package, and I am seeking a sponsor.

JOSM is an editor for  OpenStreetMap (OSM) written in  Java 1.5. 
Currently it supports loading stand alone GPX track data from the OSM database,
loading and editing existing nodes, ways, metadata tags and relations.

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

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


[Bug 508352] Review Request: pxe-kexec - kexec boot from a PXE server

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Ed Swierk eswi...@aristanetworks.com changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)




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

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


[Bug 508352] New: Review Request: pxe-kexec - kexec boot from a PXE server

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: pxe-kexec - kexec boot from a PXE server

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

   Summary: Review Request: pxe-kexec - kexec boot from a PXE
server
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: eswi...@aristanetworks.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://www-cs-students.stanford.edu/~eswierk/misc/pxe-kexec/pxe-kexec.spec
SRPM URL:
http://www-cs-students.stanford.edu/~eswierk/misc/pxe-kexec/pxe-kexec-0.1.7-1.fc11.src.rpm
Description: pxe-kexec reads a PXELINUX configuration file, prompts the user
for an entry like the PXELINUX program would do and finally boots that entry
using Kexec.

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

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


[Bug 507082] Review Request: mediawiki-HTTP302Found - Forces an external HTTP 302 redirect instead of internal redirects

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Fedora Update System upda...@fedoraproject.org  
2009-06-26 13:40:01 EDT ---
mediawiki-HTTP302Found-1.0-1.fc10 has been submitted as an update for Fedora
10.
http://admin.fedoraproject.org/updates/mediawiki-HTTP302Found-1.0-1.fc10

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

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


[Bug 507082] Review Request: mediawiki-HTTP302Found - Forces an external HTTP 302 redirect instead of internal redirects

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-06-26 13:40:06 EDT ---
mediawiki-HTTP302Found-1.0-1.fc11 has been submitted as an update for Fedora
11.
http://admin.fedoraproject.org/updates/mediawiki-HTTP302Found-1.0-1.fc11

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

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


[Bug 508352] Review Request: pxe-kexec - kexec boot from a PXE server

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Ed Swierk eswi...@aristanetworks.com changed:

   What|Removed |Added

  Alias||pxe-kexec




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

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


[Bug 507082] Review Request: mediawiki-HTTP302Found - Forces an external HTTP 302 redirect instead of internal redirects

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Ian Weller i...@ianweller.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 507660] Review Request: xylib - Library for reading x-y data from several file formats

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Marcin Wojdyr woj...@gmail.com  2009-06-26 14:05:47 EDT 
---

I made all the changes you suggested above, with one exception: I kept gcc-g++
in BuildRequires, I hope it's not a problem.

Thank you both.

Spec URL: http://www.unipress.waw.pl/~wojdyr/spec/xylib.spec
SRPM URL: http://www.unipress.waw.pl/~wojdyr/spec/xylib-0.4-1.fc11.src.rpm

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

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


[Bug 507660] Review Request: xylib - Library for reading x-y data from several file formats

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Jussi Lehtola jussi.leht...@iki.fi  2009-06-26 14:19:10 
EDT ---
As you're a new packager you should learn to run rpmlint on your packages:
https://fedoraproject.org/wiki/Packaging/Guidelines#Use_rpmlint

You've written %defattr in the changelog, which may be expanded. rpmlint
complains from this among other things. In the changelog you should use %% to
prevent the macro from being expanded.

rpmlint does sometimes generate noise, for instance it will comlain about the
-devel package having no documentation.

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

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


[Bug 479978] Review Request: newmat - C++ matrix library

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Pascal pascal...@parois.net  2009-06-26 14:27:07 EDT ---
Upstream answer:
The conditions of use are at the beginning of the main documentation file.

You can make that change, if you like. Put a note in the code that you
have made this change as I want to look a little more closely as to what
should be done in my version before I make any change.

Effectively, in nm11.htm:
1.1 Conditions of use
I place no restrictions on the use of newmat except that I take no liability
for any problems that may arise from its use, distribution or other dealings
with it.
You can use it in your commercial projects (as well as your non-commercial
projects).
You can make and distribute modified or merged versions. You can include parts
of it in your own software.
If you distribute modified or merged versions, please make it clear which parts
are mine and which parts are modified.
[...]

Is it not enough ?

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

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


[Bug 507660] Review Request: xylib - Library for reading x-y data from several file formats

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Marcin Wojdyr woj...@gmail.com  2009-06-26 14:41:48 EDT 
---
Yes, I know about rpmlint, but the last time I checked only RPMs, not the spec.
Updated spec and SRPM are in the same place.

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

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


[Bug 508373] New: Review Request: perl-DateTime-Format-SQLite - Parse and format SQLite dates and times

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-DateTime-Format-SQLite - Parse and format SQLite 
dates and times

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

   Summary: Review Request: perl-DateTime-Format-SQLite - Parse
and format SQLite dates and times
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/DateTime-Format-SQLite
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cw...@alumni.drew.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://fedorapeople.org/~cweyl/review/perl-DateTime-Format-SQLite.spec
SRPM URL:
http://fedorapeople.org/~cweyl/review/perl-DateTime-Format-SQLite-0.10-1.fc11.src.rpm

Description:
This module understands the formats used by SQLite for its 'date',
'datetime' and 'time' functions. It can be used to parse these formats
in order to create the DateTime manpage objects, and it can take a
DateTime object and produce a timestring accepted by SQLite.*NOTE:*
SQLite does not have real date/time types but stores everything as
strings. This module deals with the date/time strings as
understood/returned by SQLite's 'date', 'time', 'datetime', 'julianday'
and 'strftime' SQL functions. You will usually want to store your dates
in one of these formats.

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1437534

Additional Comment:

This is an additional optional testing dep for the latest GA DBIx::Class.

*rt-0.10

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

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


[Bug 489014] Review Request: gnome-do-plugins - Plugins for Gnome Do

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #34 from David Nielsen gnomeu...@gmail.com  2009-06-26 15:25:08 
EDT ---
Created an attachment (id=349593)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=349593)
SourceFucker

Please note that that type of issue is exactly why SourceFucker was developed.
It can create patches for these type of issues for your Mono packages. I would
recommend running it against any current packages.

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

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


[Bug 507110] Review Request: openal-soft - OpenAL-Soft lib

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |




--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-06-26 
15:36:40 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 507972] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics Library

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-06-26 
15:37:19 EDT ---
(In reply to comment #3)
 If you want to import this package into Fedora 9/10, you also have
should be read as Fedora 10/11.

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

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


[Bug 507972] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics Library

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |
   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-06-26 
15:36:16 EDT ---
Well,

* This package itself can be approved (actually most issues are
  already discussed in the previous review request)
* As mentioned in
  http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored
  We request those who want to get sponsored to either submit another
  review request or do pre-review of other person's review request
  to show that you have an understanding of the process and of 
  the packaging guidelines.
  Now you have another review request bug 507110 and from a very
  quick glance at the spec file it is well-packaged
  ! Note:
For openal-soft review request, please change the release number
every time you modify your spec file/srpm to avoid confusion (even
during review request).


   This package (bullet) is APPROVED by mtasaka


Please follow the procedure written on:
http://fedoraproject.org/wiki/PackageMaintainers/Join
from Install the Client Tools (Koji).

Now I am sponsoring you.

If you want to import this package into Fedora 9/10, you also have
to look at
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT
(after once you rebuilt this package on koji Fedora rebuilding system).

If you have questions, please ask me.

Removing NEEDSPONSOR.

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

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


[Bug 507675] Review Request: poppler-data - Encoding files

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

  Alias||poppler-data




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

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


[Bug 479978] Review Request: newmat - C++ matrix library

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Denis Arnaud denis.arnaud_fed...@m4x.org  2009-06-26 
16:04:54 EDT ---
I am not a legal expert, but as stated here:
http://fedoraproject.org/wiki/Licensing#SoftwareLicenses , all software in
Fedora MUST be under licenses in the  Fedora licensing list. This list is based
on the licenses approved by the Free Software Foundation , OSI and consultation
with Red Hat Legal.

Debian packagers raised the same type of concerns about licensing of that
package: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=335185 , but they
became eventually fine with the re-worded version of the license.

It is a custom license, and the author does not appear to be ready to switch to
a standard, OSI-approved, license. So, Fedora legal people may not accept that
license. I prefer to let them have a look at it and decide.

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

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


[Bug 507110] Review Request: openal-soft - OpenAL-Soft lib

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Thomas Kowaliczek linuxdon...@linuxdonald.de  2009-06-26 
17:18:44 EDT ---
Fixed spec file because i had forget to add informations about my changes in
the spec file

http://www.linuxdonald.de/linuxdonald/openal-soft.spec
http://www.linuxdonald.de/linuxdonald/openal-soft-1.8.466-3.fc11.src.rpm

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

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


[Bug 507972] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics Library

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Thomas Kowaliczek linuxdon...@linuxdonald.de changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 507972] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics Library

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Thomas Kowaliczek linuxdon...@linuxdonald.de  2009-06-26 
18:32:41 EDT ---
New Package CVS Request
===
Package Name: bullet
Short Description: 3D Collision Detection and Rigid Body Dynamics Library
Owners: linuxdonald
Branches: devel, F-10, F11
InitialCC:

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

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


[Bug 508416] Review Request: rubygem-mkrf - Making C extensions for Ruby a bit easier

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks||508417




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

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


[Bug 508417] New: Review Request: rubygem-ruby-opengl - OpenGL Interface for Ruby

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-ruby-opengl - OpenGL Interface for Ruby

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

   Summary: Review Request: rubygem-ruby-opengl - OpenGL Interface
for Ruby
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mtas...@ioa.s.u-tokyo.ac.jp
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
Depends on: 508416
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://mtasaka.fedorapeople.org/Review_request/rubygem-ruby-oepngl/rubygem-ruby-opengl.spec
SRPM URL:
http://mtasaka.fedorapeople.org/Review_request/rubygem-ruby-oepngl/rubygem-ruby-opengl-0.60.1-2.fc.src.rpm
Description:
ruby-opengl consists of Ruby extension modules that are bindings 
for the OpenGL, GLU, and GLUT libraries. It is intended to be 
a replacement for -- and uses the code from -- Yoshi's ruby-opengl.

Koji sratch build:
- Not available yet because this depends on bug 508416
- However mock build results for dist-f12 i586/x86_64 are available on:
  http://mtasaka.fedorapeople.org/Review_request/rubygem-ruby-oepngl/Results/

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

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


[Bug 508416] New: Review Request: rubygem-mkrf - Making C extensions for Ruby a bit easier

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-mkrf - Making C extensions for Ruby a bit 
easier

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

   Summary: Review Request: rubygem-mkrf - Making C extensions for
Ruby a bit easier
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mtas...@ioa.s.u-tokyo.ac.jp
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://mtasaka.fedorapeople.org/Review_request/rubygem-mkrf/rubygem-mkrf.spec
SRPM URL:
http://mtasaka.fedorapeople.org/Review_request/rubygem-mkrf/rubygem-mkrf-0.2.3-1.fc.src.rpm
Description: 
mkrf is a library for generating Rakefiles to build Ruby
extension modules written in C. It is intended as a replacement for
mkmf. The major difference between the two is that mkrf
builds you a Rakefile instead of a Makefile.

koji scratch build:
For F-12: http://koji.fedoraproject.org/koji/taskinfo?taskID=1437438
For F-11: http://koji.fedoraproject.org/koji/taskinfo?taskID=1437830

There are many rpmlint warining/errors on -doc subpackage, however
these are on examples and I think they are safe to ignore.

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

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


[Bug 508417] Review Request: rubygem-ruby-opengl - OpenGL Interface for Ruby

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-06-26 
18:50:43 EDT ---
By the way the naming seems a bit strange, however gem name is
actually called as ruby-opengl so I think rubygem-ruby-opengl is
proper for srpm name.

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

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


[Bug 508318] Review Request: mutter - A window manager based on metacity and clutter

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Bastien Nocera bnoc...@redhat.com  2009-06-26 19:31:45 
EDT ---
FYI, I already pushed clutter 0.9 packages for F12.

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

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


[Bug 476435] Review Request: sugar-record - Recording tool for Sugar

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Steven M. Parrish smparr...@gmail.com changed:

   What|Removed |Added

 CC||smparr...@gmail.com
   Flag||needinfo?




--- Comment #5 from Steven M. Parrish smparr...@gmail.com  2009-06-26 
20:24:25 EDT ---
Fabian,

where do we stand on this?  Per the above ticket it looks like they spun up a
tarball.  Do you have a version of this that will work on F11 or F12?

Steven

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

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


[Bug 467641] Review Request: sugar-maze - Maze for Sugar

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Steven M. Parrish smparr...@gmail.com changed:

   What|Removed |Added

 CC||smparr...@gmail.com
   Flag||needinfo?




--- Comment #10 from Steven M. Parrish smparr...@gmail.com  2009-06-26 
20:28:00 EDT ---
Fabian,

Where do we stand on this one?

Steven

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

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


[Bug 481516] Review Request: sugar-stopwatch - Simple stopwatch for Sugar

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Steven M. Parrish smparr...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 CC||smparr...@gmail.com
 Resolution||CURRENTRELEASE




--- Comment #8 from Steven M. Parrish smparr...@gmail.com  2009-06-26 
20:25:29 EDT ---
Package currently in Fedora closing

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

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


[Bug 472071] Review Request: sugar-paint : Paint (Draw) Activity for Sugar

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Steven M. Parrish smparr...@gmail.com changed:

   What|Removed |Added

 CC||smparr...@gmail.com
   Flag||needinfo?




--- Comment #6 from Steven M. Parrish smparr...@gmail.com  2009-06-26 
20:26:58 EDT ---
Bryan where do we stand on this.  Do you have a working version for F11?  Do
you need help or someone to take over?

Steven

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

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


[Bug 506844] Review Request: audex - kde4 audio ripper

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #16 from Wesley S. Hearn gr33nn1...@gr33nn1nj4.com  2009-06-26 
20:34:11 EDT ---
Upstream bug, about dialog says beta3 and the sources still have the name for
beta3. So the package needs to be beta3.

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

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


[Bug 504521] Review Request: SevenZip - Java SDK for LZMA

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #10 from Jason Tibbitts ti...@math.uh.edu  2009-06-26 20:34:30 
EDT ---
CVS done.

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

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


[Bug 504710] Review Request: rubygem-gettext_rails - Localization support for Ruby on Rails(=2.3) by Ruby-GetText-Package

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-06-26 20:34:12 EDT 
---
CVS done.

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

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


[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #26 from Jason Tibbitts ti...@math.uh.edu  2009-06-26 20:38:49 
EDT ---
CVS done, except that I think you intended to make yourself the owner, not
Orcan, so that's what I set up.  Otherwise, Orcan approved his own package,
which is not appropriate.

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

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


[Bug 489564] Review Request: Blueman - Bluetooth Manager

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #50 from Jason Tibbitts ti...@math.uh.edu  2009-06-26 20:40:07 
EDT ---
CVS done.

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

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


[Bug 506721] Package Review: nbtk - A toolkit for moblin NetBooks

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #11 from Jason Tibbitts ti...@math.uh.edu  2009-06-26 20:39:34 
EDT ---
CVS done.

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

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


[Bug 489564] Review Request: Blueman - Bluetooth Manager

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #51 from Jason Tibbitts ti...@math.uh.edu  2009-06-26 20:42:03 
EDT ---
Oops, unfortunately I can't do CVS because of:

blueman: PackageDB returned an error creating blueman: Email address
nus...@gmail.com is not a valid bugzilla email address.  Either make a bugzilla
account with that email address or change your email address in the Fedora
Account System https://admin.fedoraproject.org/accounts/ to a valid bugzilla
email address and try again.

Could you fix that?

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

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


[Bug 507972] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics Library

2009-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-06-26 20:44:22 EDT 
---
CVS done.

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

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


  1   2   >