[Bug 484931] Renaming Review Request: hosts3d - 3D real-time network visualiser

2009-02-11 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=484931


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 CC||jussi.leht...@iki.fi




--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi  2009-02-11 03:06:42 
EDT ---
Now the package
Provides:homestead = 0.95-1
Obsoletes:   homestead  0.95-1

but the current version in F10 is homestead-0.95-1, which won't be obsoleted. 

Change this to
Provides:homestead = %{version}-%{release}
Obsoletes:   homestead  %{version}-%{release}

or just remove the versioning from Provides  Obsoletes?

-- 
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 467420] Review Request: mingw32-gtk2 - MinGW Windows Gtk2 library

2009-02-11 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=467420





--- Comment #11 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 
04:46:38 EDT ---
Imported into CVS but not built yet because of
dependencies on mingw32-pango, mingw32-jasper.

-- 
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 467421] Review Request: mingw32-gtk-vnc - MinGW Windows port of VNC client GTK widget

2009-02-11 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=467421





--- Comment #9 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 04:46:40 
EDT ---
Imported into CVS but not built yet because of
dependencies on mingw32-gtk2 and mingw32-gnutls.

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





--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro  2009-02-11 
05:04:59 EDT ---
Quote from a testrun on F10, after I have installed like this:

yum -y install --nogpgcheck ./reviewtool-0.04-1.fc10.noarch.rpm
./perl-Fedora-App-ReviewTool-0.04-1.fc10.noarch.rpm 



[r...@pc64 ~]# reviewtool setup
Could not load class (MooseX::App::Cmd) because : Can't locate
MooseX/App/Cmd.pm in @INC (@INC contains: /usr/bin/../lib
/usr/lib/perl5/5.10.0/i386-linux-thread-multi /usr/lib/perl5/5.10.0
/usr/local/lib/perl5/site_perl/5.10.0/i386-linux-thread-multi
/usr/local/lib/perl5/site_perl/5.10.0
/usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi
/usr/lib/perl5/vendor_perl/5.10.0 /usr/lib/perl5/vendor_perl
/usr/local/lib/perl5/site_perl .) at
/usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Class/MOP.pm line
151.
 at /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Class/MOP.pm line
133
Class::MOP::load_first_existing_class('MooseX::App::Cmd') called at
/usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Class/MOP.pm line 157
Class::MOP::load_class('MooseX::App::Cmd') called at
/usr/lib/perl5/vendor_perl/5.10.0/Moose.pm line 52
Moose::extends('Fedora::App::ReviewTool', 'MooseX::App::Cmd') called at
/usr/lib/perl5/vendor_perl/5.10.0/Moose/Exporter.pm line 201
Moose::extends('MooseX::App::Cmd') called at
/usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool.pm line 34
require Fedora/App/ReviewTool.pm called at /usr/bin/reviewtool line 30
main::BEGIN() called at
/usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool.pm line 0
eval {...} called at
/usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool.pm line 0
Compilation failed in require at /usr/bin/reviewtool line 30.
BEGIN failed--compilation aborted at /usr/bin/reviewtool line 30.

-- 
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: python-Mayavi - The Mayavi scientific data 3-dimensional visualizer

2009-02-11 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 #3 from Rakesh Pandit rakesh.pan...@gmail.com  2009-02-11 
05:13:31 EDT ---
Created an attachment (id=331546)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=331546)
Just a test, please 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 451582] Review Request: kvirc - themeable qt based irc client

2009-02-11 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=451582


Michael Schwendt bugs.mich...@gmx.net changed:

   What|Removed |Added

   Flag||needinfo?(smparr...@shallow
   ||creek.net)




--- Comment #7 from Michael Schwendt bugs.mich...@gmx.net  2009-02-11 
05:29:10 EDT ---
[just setting NEEDINFO so the status becomes clear in the review queue]

-- 
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 481192] Review Request: perl-pgsql_perl5 - Pg - Perl5 extension for PostgreSQL

2009-02-11 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=481192


Itamar Reis Peixoto ita...@ispbrasil.com.br changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #9 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-02-11 
05:26:59 EDT ---
New Package CVS Request
===
Package Name: perl-pgsql_perl5
Short Description: Perl5 extension for PostgreSQL
Owners: itamarjp
Branches: 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 461736] Review Request: perl-Schedule-Cron-Events - Take a line from a crontab and find out when events will occur

2009-02-11 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=461736


Miroslav Suchy msu...@redhat.com changed:

   What|Removed |Added

 CC||redhat-bugzi...@linuxnetz.d
   ||e




--- Comment #4 from Miroslav Suchy msu...@redhat.com  2009-02-11 05:35:01 EDT 
---
*** Bug 483390 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 483390] Review Request: perl-Schedule-Cron-Events - Take a line from a crontab and find out when events will occur

2009-02-11 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=483390


Miroslav Suchy msu...@redhat.com changed:

   What|Removed |Added

 Resolution|RAWHIDE |DUPLICATE




--- Comment #16 from Miroslav Suchy msu...@redhat.com  2009-02-11 05:35:01 
EDT ---
Marking as DUPLICATE rather then CLOSED RAWHIDE to better track that this BZ is
indeed duplicate.

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

-- 
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 483390] Review Request: perl-Schedule-Cron-Events - Take a line from a crontab and find out when events will occur

2009-02-11 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=483390





--- Comment #17 from Miroslav Suchy msu...@redhat.com  2009-02-11 05:49:43 
EDT ---
I contacted upstream author to clarify this doubts and if possible release new
version with LICENSE file.
If possible track further discussion under BZ 461736.

-- 
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 483543] Review Request: systemtapguiserver

2009-02-11 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=483543


Anithra anit...@linux.vnet.ibm.com changed:

   What|Removed |Added

 Depends on|483205  |




-- 
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 483205] Review Request: eclipse-systemtapgui - GUI interface for SystemTap

2009-02-11 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=483205


Anithra anit...@linux.vnet.ibm.com changed:

   What|Removed |Added

 Blocks|483543  |177841




--- Comment #27 from Anithra anit...@linux.vnet.ibm.com  2009-02-11 05:54:31 
EDT ---
Thanks Andrew.

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





--- Comment #2 from Parag AN(पराग) panem...@gmail.com  2009-02-11 06:02:30 
EDT ---
that mean spec need
Requires: perl (MooseX::App::Cmd)

Even after adding that command failed
need
Requires: perl (RPC::XML::Client)

-- 
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 479803] Review Request: cabal2spec - generates spec files for Haskell packages

2009-02-11 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=479803





--- Comment #10 from Parag AN(पराग) panem...@gmail.com  2009-02-11 06:12:58 
EDT ---
you should have 
%config(noreplace) %{_sysconfdir}/rpmdevtools
like we have in rpmdevtools

-- 
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 457160] Review Request: Zorba - General purpose XQuery processor implemented in C++

2009-02-11 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=457160


Michael Schwendt bugs.mich...@gmx.net changed:

   What|Removed |Added

 CC||bugs.mich...@gmx.net




--- Comment #11 from Michael Schwendt bugs.mich...@gmx.net  2009-02-11 
06:22:46 EDT ---
I'd like to see some fixes prior to further reviewing. It's not trivial to
review the current packaging:


 BuildRequires: cmake = 2.4 libxml2-devel = 2.2.16 icu = 2.6 libicu-devel

It's highly recommended to put on BuildRequires per line and either sort them
alphabetically or group them appropriately.

icu is  2.6 for several years. Even in RHEL 5.

Please reconsider the decision to list the versions. Often packagers forget to
keep them up-to-date.


 sh: ruby: command not found

BuildRequires: ruby is missing.


 -- Warning: GNU Bison not available -- the parser will not be regenerated
 -- Warning: GNU Flex not available -- the lexer will not be regenerated
 -- PHP5 binding not generated because library and include file not installed.
 -- Looking for doxygen... - NOT found

Just quoted for completeness.


 -- Can't build Zorba with TIDY support because TIDY is not found.

BuildRequires: libtidy-devel makes it happy. If you don't want that, adding a
comment would be good.


 -- Found PythonInterp: /usr/bin/python2.5
 -- Could NOT find PythonLibs  (missing:  PYTHON_LIBRARIES PYTHON_INCLUDE_PATH)
 -- Python binding not generated because library and include file not 
 installed.

BuildRequires: python-devel is missing.


 %description devel
 The %{name}-devel package contains headers for building applications
 that use %{zorba}.

%{zorba} is undefined.


 cmake

There are guidelines for proper cmake usage:
https://fedoraproject.org/wiki/Packaging:Cmake

In particular, execute make VERBOSE=1 ... for increased verbosity in the
build log.

Then you will find that Fedora's global optimisation flags are not used. This
needs another look after the cmake related fixes.


 %files

 %{_libdir}/*.so.%{version}

This means that any version upgrade will break ABI compatibility with any
programs linked against this library. That's an indication of an unstable API,
ongoing heavy development, or developers who haven't got the versioning scheme
right.


 %dir %{_datadir}/doc/%{name}-%{version}
 %{_datadir}/doc/%{name}-%{version}

The %dir statement here is superfluous, because the second line already
includes the %name-%version directory and all its contents recursively. You're
advised to make them more explicit with a trailing slash. Like this:

  %{_datadir}/doc/%{name}-%{version}/


 %files devel

Here several directories are not included.
In particular:

%dir %{_includedir}/%{name}
%dir %{_includedir}/%{name}/%{name}
%dir %{_includedir}/%{name}/%{name}/util
%dir %{_includedir}/%{name}/simplestore
%dir %{_includedir}/%{name}/simplestore/msdom
%dir %{_datadir}/doc/%{name}-%{version}/python
%dir %{_datadir}/doc/%{name}-%{version}/python/examples
%dir %{_datadir}/doc/%{name}-%{version}/python/html
%dir %{_datadir}/doc/%{name}-%{version}/ruby
%dir %{_datadir}/doc/%{name}-%{version}/ruby/examples
%dir %{_datadir}/doc/%{name}-%{version}/ruby/html

https://fedoraproject.org/wiki/Packaging:UnownedDirectories

You could shrink your %files list much by including entire trees recursively or
by increased usage of '*' wildcards where appropriate. Else the only option is
to add as many of the missing %dir statements as necessary.

-- 
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 478300] Review Request: python-wifi - Python binding for the wireless extensions

2009-02-11 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=478300





--- Comment #2 from Debarshi Ray debarshi@gmail.com  2009-02-11 06:34:08 
EDT ---
(In reply to comment #1)

 xx - contents of %doc does not affect the runtime
 + python_wifi.egg-info/SOURCES.txt contains docs/*, examples/* and 
 tests/*.
   I do not know enough about Python Eggs to say whether that is a problem
   or not. Got to find out.

According to
http://peak.telecommunity.com/DevCenter/EggFormats#sources-txt-source-files-manifest:

In other words, SOURCES.txt has little or no runtime value for being included
in distributed eggs...

So I would say that this not a problem for us.

-- 
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 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

2009-02-11 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=475065





--- Comment #3 from Michael Schwendt bugs.mich...@gmx.net  2009-02-11 
06:44:16 EDT ---
Just a comment, not a review: In your %files sections, you're advised to mark
recursively included paths with a trailing slash. That makes it trivial to
distinguish files and directories.

-- 
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 485052] Review Request: mingw32-crossreport - Analysis tool to help cross-compilation to Windows

2009-02-11 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=485052


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

 CC||berra...@redhat.com
  Alias||mingw32-crossreport




-- 
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 485052] New: Review Request: mingw32-crossreport - Analysis tool to help cross-compilation to Windows

2009-02-11 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: mingw32-crossreport - Analysis tool to help 
cross-compilation to Windows

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

   Summary: Review Request: mingw32-crossreport - Analysis tool to
help cross-compilation to Windows
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rjo...@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://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/file/tip/crossreport
SRPM URL: http://www.annexia.org/tmp/mingw32-crossreport-1-1.fc10.src.rpm
Description:
Analysis tool to help cross-compilation to Windows

CrossReport is a tool to help you analyze the APIs used by a compiled
Linux program, in order to work out the effort required to
cross-compile that program for Windows, using the Fedora MinGW
cross-compiler.

The simplest way to use it is to point it at an existing Linux binary,
and then read the generated report.

What it does in more detail: It looks at the libraries and API calls
used by the Linux binary, and compares them to the libraries and API
calls that we currently support under the Fedora MinGW cross-compiler.
It then works out what is missing, and produces a report suggesting
the amount of work that needs to be done to port the program.  For
example, whether whole libraries need to be ported first, and/or how
to substitute individual calls to work on Windows.

Some example reports can be found here:
http://lists.fedoraproject.org/pipermail/fedora-mingw/2009-February/000457.html

-- 
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 485052] Review Request: mingw32-crossreport - Analysis tool to help cross-compilation to Windows

2009-02-11 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=485052


Levente Farkas lfar...@lfarkas.org changed:

   What|Removed |Added

 CC||lfar...@lfarkas.org
 AssignedTo|nob...@fedoraproject.org|lfar...@lfarkas.org
   Flag||fedora-review+




--- Comment #1 from Levente Farkas lfar...@lfarkas.org  2009-02-11 07:22:40 
EDT ---
imho a version number 1.0 is better then 1, since it's very strange looking...
anyway the same apply to filesystem too:-(

rpmlint mingw32-crossreport-1-1.fc10.src.rpm 
mingw32-crossreport.src: W: strange-permission crossreport.pl 0775
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

mode 0755 would be better here.

koji build ok:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1119335

+ OK
- Needs to be looked into
/ Not applicable
* Overridden by MinGW guidelines

[+] rpmlint must be run on every package. The output should be posted in the
review.
[+] MUST: The package must be named according to the Package Naming Guidelines
[+] MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption on Package Naming Guidelines
[+] MUST: The package must meet the Packaging Guidelines
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[/] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
[+] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.
[/] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch.
[+] MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines;
inclusion of those as BuildRequires is optional.
[/] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[/] MUST: Every binary RPM package which stores shared library files (not just
symlinks) in any of the dynamic linker's default paths, must call ldconfig in
%post and %postun. 
[/] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package.
[+] MUST: A package must own all directories that it creates. 
[+] MUST: A package must not contain any duplicate files in the %files listing.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} ( or $RPM_BUILD_ROOT ).
[+] MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines .
[+] MUST: The package must contain code, or permissable content.
[/] MUST: Large documentation files should go in a -doc subpackage.
[/] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.
[*] MUST: Header files must be in a -devel package.
[/] MUST: Static libraries must be in a -static package.
[/] MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
(for directory ownership and usability).
[/] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in
a -devel package.
[/] MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name} =
%{version}-%{release}
[/] MUST: Packages must NOT contain any .la libtool archives, these should be
removed in the spec.
[/] MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with
desktop-file-install in the %install section.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: At the beginning of %install, each package MUST run rm -rf
%{buildroot} ( or $RPM_BUILD_ROOT ). See Prepping BuildRoot For %i install for
details.
[+] MUST: All filenames in rpm packages must be valid UTF-8.
[+] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, 

[Bug 457213] Review Request: html2text - HTML-to-text converter

2009-02-11 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=457213


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #33 from leigh scott leigh123li...@googlemail.com  2009-02-11 
07:29:34 EDT ---
New Package CVS Request
===
Package Name: html2text
Short Description: HTML-to-text converter
Owners: leigh123linux
Branches: F-9 F-10
InitialCC: leigh123linux

-- 
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 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||rjo...@redhat.com
 AssignedTo|nob...@fedoraproject.org|rjo...@redhat.com




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

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


[Bug 226055] Merge Review: libvirt

2009-02-11 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=226055


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review?




--- Comment #1 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 07:36:23 
EDT ---
Taking for review.

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

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


[Bug 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review?




--- Comment #1 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 07:36:36 
EDT ---
Taking for review.

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

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


[Bug 226055] Merge Review: libvirt

2009-02-11 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=226055


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||rjo...@redhat.com
 AssignedTo|nob...@fedoraproject.org|rjo...@redhat.com




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

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


[Bug 484486] Review Request: perl-DateTime-Format-Excel - Convert between DateTime and Excel dates

2009-02-11 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=484486


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 484486] Review Request: perl-DateTime-Format-Excel - Convert between DateTime and Excel dates

2009-02-11 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=484486


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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #1 from Parag AN(पराग) panem...@gmail.com  2009-02-11 07:56:41 
EDT ---
Review:
+ package builds in mock (rawhide i386).
+ rpmlint is silent for SRPM and for RPM.
- source files match different upstream url
d2f1934833c247b50088fd7999efee04  DateTime-Format-Excel-0.2901.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.
+ file permissions are appropriate.
+ make test outputs
All tests successful.
Files=3, Tests=77,  0 wallclock secs ( 0.02 usr  0.00 sys +  0.34 cusr  0.02
csys =  0.38 CPU)
+ Package  perl-DateTime-Format-Excel-0.2901-1.fc11-
Provides: perl(DateTime::Format::Excel) = 0.2901
Requires: perl = 0:5.005 perl(Carp) perl(DateTime) = 0.1705 perl(strict)
perl(vars)

Suggestion:
1) fix source url to 
http://cpan.perl.org/authors/id/D/DR/DROLSKY/DateTime-Format-Excel-0.2901.tar.gz
 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 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811





--- Comment #2 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 08:04:03 
EDT ---
gnome-applet-vm.x86_64: W: non-conffile-in-etc
/etc/gconf/schemas/vm-applet.schemas
gnome-applet-vm.x86_64: E: explicit-lib-dependency libvirt
gnome-applet-vm.x86_64: W: conffile-without-noreplace-flag
/etc/pam.d/vm_applet_wrapper
gnome-applet-vm.x86_64: W: conffile-without-noreplace-flag
/etc/security/console.apps/vm_applet_wrapper
gnome-applet-vm.x86_64: W: one-line-command-in-%postun /sbin/ldconfig
3 packages and 0 specfiles checked; 1 errors, 4 warnings.

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

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


[Bug 226055] Merge Review: libvirt

2009-02-11 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=226055





--- Comment #2 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 08:04:05 
EDT ---
libvirt.src:143: W: unversioned-explicit-obsoletes libvir
libvirt.src:160: W: unversioned-explicit-obsoletes libvir-devel
libvirt.src:171: W: unversioned-explicit-obsoletes libvir-python
libvirt.src:534: W: macro-in-%changelog fedora
libvirt.src:719: W: macro-in-%changelog dist
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/images 0700
libvirt.x86_64: E: non-standard-dir-perm /var/log/libvirt 0700
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/qemu 0700
libvirt.x86_64: E: non-standard-dir-perm /etc/libvirt 0700
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/uml 0700
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/lxc 0700
libvirt.x86_64: E: non-standard-dir-perm /etc/libvirt/qemu 0700
libvirt.x86_64: W: file-not-utf8 /usr/share/doc/libvirt-0.6.0/NEWS
libvirt.x86_64: E: incoherent-logrotate-file /etc/logrotate.d/libvirtd
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/iptables/filter 0700
libvirt.x86_64: E: non-standard-dir-perm /etc/libvirt/qemu/networks 0700
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/boot 0700
libvirt.x86_64: E: executable-marked-as-config-file /etc/sysconfig/libvirtd
libvirt.x86_64: E: script-without-shebang /etc/sysconfig/libvirtd
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/network 0700
libvirt.x86_64: E: non-standard-dir-perm /var/log/libvirt/qemu 0700
libvirt.x86_64: E: non-standard-dir-perm /etc/libvirt/qemu/networks/autostart
0700
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/iptables/nat 0700
libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/iptables 0700
libvirt.x86_64: E: explicit-lib-dependency libxml2
libvirt.x86_64: W: obsolete-not-provided libvir
libvirt.x86_64: W: dangerous-command-in-%post ln
libvirt.x86_64: W: one-line-command-in-%postun /sbin/ldconfig
libvirt.x86_64: W: service-default-enabled /etc/rc.d/init.d/libvirtd
libvirt.x86_64: W: service-default-enabled /etc/rc.d/init.d/libvirtd
libvirt-devel.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-devel-0.6.0/examples/python/domstart.py
libvirt-devel.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-devel-0.6.0/examples/index.py
libvirt-devel.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-devel-0.6.0/examples/python/domrestore.py
libvirt-devel.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-devel-0.6.0/examples/python/domsave.py
libvirt-devel.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-devel-0.6.0/examples/python/dominfo.py
libvirt-devel.x86_64: W: obsolete-not-provided libvir-devel
libvirt-devel.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-devel-0.6.0/examples/index.py /usr/bin/python
libvirt-devel.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-devel-0.6.0/examples/python/dominfo.py /usr/bin/python
libvirt-devel.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-devel-0.6.0/examples/python/domrestore.py
/usr/bin/python
libvirt-devel.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-devel-0.6.0/examples/python/domsave.py /usr/bin/python
libvirt-devel.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-devel-0.6.0/examples/python/domstart.py /usr/bin/python
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/python/dominfo.py
libvirt-python.x86_64: W: file-not-utf8
/usr/share/doc/libvirt-python-0.6.0/NEWS
libvirt-python.x86_64: E: non-executable-script
/usr/lib64/python2.5/site-packages/libvirt.py 0644
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/python/domsave.py
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/error.py
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/uuid.py
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/basic.py
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/python/domstart.py
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/node.py
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/python/domrestore.py
libvirt-python.x86_64: W: spurious-executable-perm
/usr/share/doc/libvirt-python-0.6.0/create.py
libvirt-python.x86_64: W: obsolete-not-provided libvir-python
libvirt-python.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-python-0.6.0/basic.py /usr/bin/python
libvirt-python.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-python-0.6.0/create.py /usr/bin/python
libvirt-python.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-python-0.6.0/error.py /usr/bin/python
libvirt-python.x86_64: W: doc-file-dependency
/usr/share/doc/libvirt-python-0.6.0/node.py 

[Bug 485052] Review Request: mingw32-crossreport - Analysis tool to help cross-compilation to Windows

2009-02-11 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=485052


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 08:08:41 
EDT ---
New Package CVS Request
===
Package Name: mingw32-crossreport
Short Description: Analysis tool to help cross-compilation to Windows
Owners: rjones berrange lfarkas
Branches: F-10 EL-5
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 484735] Review Request: fipscheck - A library for integrity verification of FIPS validated modules

2009-02-11 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=484735


Tomas Mraz tm...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
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 484725] Review Request: eclipse-moreunit - Assisting in writing more unit tests plugin

2009-02-11 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=484725


Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #6 from Alexander Kurtakov akurt...@redhat.com  2009-02-11 
08:49:32 EDT ---
Builded 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 473037] Review Request: tinycc - Tiny C Compiler

2009-02-11 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=473037





--- Comment #9 from Thomas Moschny thomas.mosc...@gmx.de  2009-02-11 09:42:43 
EDT ---
Ping? Brennan, are you still interested in maintaining this package?

-- 
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 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

2009-02-11 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=483730


Jaroslav Reznik jrez...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #31 from Jaroslav Reznik jrez...@redhat.com  2009-02-11 09:46:10 
EDT ---
(In reply to comment #30)
 Jaroslav
 
 Just in case you missed this. Looks like Rex has given his OK.

So then it's OK for me too. If you are willing to maintain it this patch or
better should be ask upstream about flags issue.

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 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811





--- Comment #3 from Karel Zak k...@redhat.com  2009-02-11 09:56:25 EDT ---
Frankly, I want to orphan this package... it means that we need upstream
maintainer too. I have already asked on many places, but without any result ;-(

Daniel, any idea? Maybe someone around virtualization in RH...

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

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


[Bug 485015] Review Request: perl-Class-C3-Adopt-NEXT - Reduce one's dependency on NEXT

2009-02-11 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=485015


Chris Weyl cw...@alumni.drew.edu 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 484486] Review Request: perl-DateTime-Format-Excel - Convert between DateTime and Excel dates

2009-02-11 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=484486


Chris Weyl cw...@alumni.drew.edu 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 485015] Review Request: perl-Class-C3-Adopt-NEXT - Reduce one's dependency on NEXT

2009-02-11 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=485015





--- Comment #2 from Chris Weyl cw...@alumni.drew.edu  2009-02-11 10:04:26 EDT 
---
New Package CVS Request
===
Package Name: perl-Class-C3-Adopt-NEXT
Short Description: Reduce one's dependency on NEXT
Owners: cweyl
Branches: F-9 F-10 devel
InitialCC: perl-sig

-- 
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 484486] Review Request: perl-DateTime-Format-Excel - Convert between DateTime and Excel dates

2009-02-11 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=484486





--- Comment #2 from Chris Weyl cw...@alumni.drew.edu  2009-02-11 10:04:02 EDT 
---
New Package CVS Request
===
Package Name: perl-DateTime-Format-Excel
Short Description: Convert between DateTime and Excel dates
Owners: cweyl
Branches: F-9 F-10 devel
InitialCC: perl-sig

-- 
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 480538] Review Request: iptux -- a tool for sharing and transporting files and directories in Lan

2009-02-11 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=480538





--- Comment #18 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-02-11 
10:05:46 EDT ---
Created an attachment (id=331571)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=331571)
Fix build error with g++44

(In reply to comment #14)
 Created an attachment (id=331461)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=331461) [details]
 build.log of mock rebuild, on 'fedora-devel-i386'
 
 I ain't sure if my F10 compiler has anything effects on this, but mock has
 following compilation error:
 
 utils.cpp:95: error: invalid conversion from 'const char*' to 'char*'

(In reply to comment #16)
 gcc-4.4.0 is developing and is not released official by FSF. There are maybe
 some bugs that need to be fixed.

I just watched these comments on fedora-package-review mailing list,
so I have not checked the srpm of this package at all. However
I guess this is a bug in source code, see:

https://www.redhat.com/archives/fedora-devel-list/2009-January/msg02248.html

A suggestion patch attached (only tested on i386, note that I just
checked if the compilation passes)

-- 
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 476471] Review Request: fedora-security-guide - A security guide for Linux

2009-02-11 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=476471





--- Comment #46 from Eric Christensen e...@christensenplace.us  2009-02-11 
10:07:43 EDT ---
SPEC: http://sparks.fedorapeople.org/fedora-security-guide-11-en-US.spec

SRPM:
http://sparks.fedorapeople.org/fedora-security-guide-11-en-US-1.0-12.fc10.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 462521] Review Request: simplyhtml - Application and a java component for rich text processing

2009-02-11 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=462521





--- Comment #9 from Mary Ellen Foster mefos...@gmail.com  2009-02-11 10:40:32 
EDT ---
Good news -- looks like there's a newer version of ElementTreePanel included in
the java-1.6.0-openjdk-demo package inside
/usr/lib/jvm/java-1.6.0-openjdk-1.6.0.0/demo/jfc/Notepad/src.zip. According to
the fedora-devel-java mailing list (same thread as before), you should just be
able to delete the included version and copy this one in instead. The license
will have to become something like GPLv2+ and BSD
(http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Multiple_Licensing_Scenarios).

-- 
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 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811





--- Comment #4 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 11:42:41 
EDT ---
kzak: If you want, orphan this  I will take it.

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

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


[Bug 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

2009-02-11 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=483730





--- Comment #32 from Eli Wapniarski e...@orbsky.homelinux.org  2009-02-11 
11:54:30 EDT ---
Thanks for that... 

I really do not know what I need to ask regarding the patch. Regarding
maintaining it, like I indicated, I am not a programmer. Will kompare handle
the creation of patches.

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





--- Comment #4 from Chris Weyl cw...@alumni.drew.edu  2009-02-11 12:12:27 EDT 
---
Spec URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool.spec
SRPM URL:
http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool-0.04-2.fc10.src.rpm

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

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





--- Comment #3 from Chris Weyl cw...@alumni.drew.edu  2009-02-11 12:12:21 EDT 
---
RPM doesn't yet pick up Moose syntax, sadly :\

-- 
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 226055] Merge Review: libvirt

2009-02-11 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=226055





--- Comment #3 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 12:23:11 
EDT ---
I've posted a patch upstream[1] which reduces the number
of rpmlint problems.  Now we are left with:

 libvirt.x86_64: E: non-standard-dir-perm /var/lib/libvirt/images 0700
 (for various directories)

I think rpmlint is wrong about these errors.  The permissions
on these directories is intended to be 0700.

 libvirt.x86_64: E: incoherent-logrotate-file /etc/logrotate.d/libvirtd

Rpmlint's complaint is the logrotate file should have the same
name as the package.  In this case it has the same name as the
daemon, which seems acceptable to me.

 libvirt.x86_64: W: dangerous-command-in-%post ln

It's a fair warning, but in this case the use of 'ln' in %post
is justified.

 libvirt-devel.x86_64: W: spurious-executable-perm 
 /usr/share/doc/libvirt-devel-0.6.0/examples/python/domstart.py
 (and various other *.py files)

For the examples, this seems OK, since people may wish to run
the examples directly.

For the python libraries, there is a mix of +x/-x permissions, and
I don't know what is correct.

 libvirt-devel.x86_64: W: doc-file-dependency 
 /usr/share/doc/libvirt-devel-0.6.0/examples/index.py /usr/bin/python
 (etc)

This seems OK to me.

[1] https://www.redhat.com/archives/libvir-list/2009-February/msg00211.html

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

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


[Bug 226055] Merge Review: libvirt

2009-02-11 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=226055





--- Comment #4 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 12:30:52 
EDT ---
+ rpmlint output

See above.  Depends on the patch being acceptable upstream.

+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
+ license matches the actual package license
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
+ package successfully builds on at least one architecture
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
+ %find_lang instead of %{_datadir}/locale/*
+ binary RPM with shared library files must call ldconfig in %post and %postun
+ does not use Prefix: /usr
+ package owns all directories it creates
+ no duplicate files in %files
+ %defattr line
+ %clean contains rm -rf $RPM_BUILD_ROOT
+ consistent use of macros
+ package must contain code or permissible content
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
+ header files should be in -devel
n/a static libraries should be in -static
+ packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
+ libfoo.so must go in -devel
+ -devel must require the fully versioned base

  I changed: Requires: libvirt = %{version}-%{release}

+ packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file
+ packages must not own files or directories owned by other packages
+ %install must start with rm -rf %{buildroot} etc.
+ filenames must be valid UTF-8

Optional:

n/a if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if
available
+ reviewer should build the package in mock
+ the package should build into binary RPMs on all supported architectures
+ review should test the package functions as described
+ scriptlets should be sane
+ pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
/usr/sbin

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

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


[Bug 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811





--- Comment #5 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 12:39:49 
EDT ---
Created an attachment (id=331595)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=331595)
Proposed patch to gnome-applet-vm.spec

With the attached patch, we are left with this single
rpmlint warning:

 gnome-applet-vm.x86_64: W: non-conffile-in-etc 
 /etc/gconf/schemas/vm-applet.schemas

I think this is one we can ignore, since this looks like
the place where gconf expects these schemata to be installed.

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

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


[Bug 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811





--- Comment #6 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 12:51:33 
EDT ---
+ rpmlint output

See comment 5.

+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora

GPL+ is an acceptable license, and was set by spot, so
assume this is correct.

+ license matches the actual package license
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
+ package successfully builds on at least one architecture
+ ExcludeArch bugs filed

Originally had ExclusiveArch i386, x86-64, but should build
everywhere so I removed this.

+ BuildRequires list all build dependencies
+ %find_lang instead of %{_datadir}/locale/*
+ binary RPM with shared library files must call ldconfig in %post and %postun

Package called ldconfig, but contains no libraries, so I
removed that.

+ does not use Prefix: /usr
+ package owns all directories it creates

Originally tried to own /usr/share/pixmaps, but I fixed that.

+ no duplicate files in %files
+ %defattr line
+ %clean contains rm -rf $RPM_BUILD_ROOT
+ consistent use of macros
+ package must contain code or permissible content
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
n/a header files should be in -devel
n/a static libraries should be in -static
n/a packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
n/a libfoo.so must go in -devel
n/a -devel must require the fully versioned base
n/a packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file

I think this is not applicable because although this is a GUI
app, it is a Gnome panel plugin so doesn't need a desktop file.

+ packages must not own files or directories owned by other packages
+ %install must start with rm -rf %{buildroot} etc.
+ filenames must be valid UTF-8

Optional:

+ if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if
available
+ reviewer should build the package in mock
+ the package should build into binary RPMs on all supported architectures
- review should test the package functions as described
+ scriptlets should be sane
n/a pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
/usr/sbin

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

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


[Bug 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

 Attachment #331595|0   |1
is obsolete||




--- Comment #7 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 12:52:37 
EDT ---
Created an attachment (id=331596)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=331596)
Proposed patch to gnome-applet-vm.spec

Updated specfile patch.

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

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


[Bug 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

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




--- Comment #8 from Richard W.M. Jones rjo...@redhat.com  2009-02-11 12:53:56 
EDT ---
Cole, can you take a quick look at the proposed
changes to the specfile in comment 7, and if you
approve, say yea.

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

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


[Bug 480887] Review Request: kguitar - Guitar Tabulature Music Editor

2009-02-11 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=480887





--- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-02-11 
13:13:48 EDT ---
( Sorry it may take a little more time for me to check your
  latest srpm. But I will review your latest srpm by Sunday )

-- 
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 193210] Review Request: htmldoc - Convert HTML source files into HTML, PostScript, or PDF

2009-02-11 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=193210


Adam Goode a...@spicenitz.org changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #11 from Adam Goode a...@spicenitz.org  2009-02-11 13:23:22 EDT 
---
Package Change Request
==
Package Name: htmldoc
New Branches: EL-4 EL-5
Owners: agoode pertusus

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

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


[Bug 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

2009-02-11 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=483730





--- Comment #33 from Jaroslav Reznik jrez...@redhat.com  2009-02-11 13:22:08 
EDT ---
(In reply to comment #32)
 Thanks for that... 
 
 I really do not know what I need to ask regarding the patch. Regarding
 maintaining it, like I indicated, I am not a programmer. Will kompare handle
 the creation of patches.

Ask them to replace flags with country codes or let user choose. The patch is
really easy, so you don't have to be programmer to take care about it.

-- 
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 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

2009-02-11 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=483730





--- Comment #34 from Jaroslav Reznik jrez...@redhat.com  2009-02-11 13:40:37 
EDT ---
And of course I can help you as comaintainer - just add me to CVS request.

-- 
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 484598] Review Request: grin - Grep-like tool for source code

2009-02-11 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=484598


Bug 484598 depends on bug 484597, which changed state.

Bug 484597 Summary: Review Request:  python-argparse - Optparse inspired 
command line parser for Python
https://bugzilla.redhat.com/show_bug.cgi?id=484597

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||NEXTRELEASE
 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 484597] Review Request: python-argparse - Optparse inspired command line parser for Python

2009-02-11 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=484597


Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #5 from Terje Røsten terje...@phys.ntnu.no  2009-02-11 13:38:35 
EDT ---
Imported, built and and pending in bodhi.

-- 
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 484597] Review Request: python-argparse - Optparse inspired command line parser for Python

2009-02-11 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=484597


Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

 CC||ru...@rubenkerkhof.com




--- Comment #6 from Terje Røsten terje...@phys.ntnu.no  2009-02-11 13:44:24 
EDT ---
*** Bug 462505 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 462505] Review Request: python-argparse - An optparse-inspired command-line parsing library

2009-02-11 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=462505


Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 CC||terje...@phys.ntnu.no
 Resolution||DUPLICATE




--- Comment #1 from Terje Røsten terje...@phys.ntnu.no  2009-02-11 13:44:24 
EDT ---
Hi Ruben, 

sorry, I did a mistake here, I need python-argparse for grin ( #484598 ) and
did a review request without searching bugzilla. Now the package is approved
and imported. 

If you want I can transfer the package to you, if not I will maintain it.

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

-- 
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 471003] Review Request: cabal-install - Haskell package utility

2009-02-11 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=471003


Bryan O'Sullivan b...@serpentine.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|b...@serpentine.com




-- 
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 471003] Review Request: cabal-install - Haskell package utility

2009-02-11 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=471003


Bryan O'Sullivan b...@serpentine.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||b...@serpentine.com




-- 
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 471003] Review Request: cabal-install - Haskell package utility

2009-02-11 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=471003


Bryan O'Sullivan b...@serpentine.com changed:

   What|Removed |Added

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




--- Comment #7 from Bryan O'Sullivan b...@serpentine.com  2009-02-11 14:08:56 
EDT ---
I tried building the above SRPM on F10 using the following packages:

ghc-6.10.1-8.fc11.i386
ghc-zlib-0.5.0.0-5.fc11.i386
ghc-HTTP-3001.1.5-1.fc11.i386

The build failed per the attachment.

-- 
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 471003] Review Request: cabal-install - Haskell package utility

2009-02-11 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=471003





--- Comment #8 from Bryan O'Sullivan b...@serpentine.com  2009-02-11 14:09:35 
EDT ---
Created an attachment (id=331608)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=331608)
Log of failed build attempt

Failed build.

-- 
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 431250] Review Request: librep - An embeddable LISP environment

2009-02-11 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=431250


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

   What|Removed |Added

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




--- Comment #51 from Jason Tibbitts ti...@math.uh.edu  2009-02-11 14:19:05 
EDT ---
It's been many months since the last comment from the submitter of this ticket;
I will close this out in a week if progress isn't made.

-- 
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 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

2009-02-11 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=483730





--- Comment #35 from Eli Wapniarski e...@orbsky.homelinux.org  2009-02-11 
14:38:57 EDT ---
OK. What I'm going to ask is that the developer add a flag for cmake so that a
package maintainer can throw the --without-countryflags. equivalent in cmake.

Do you concur?

-- 
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 481507] Review Request: clanship - Battleship clone

2009-02-11 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=481507


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

   What|Removed |Added

 Blocks||182235
Summary|Clanship 1.0.5f rpm |Review Request: clanship -
   ||Battleship clone




--- Comment #5 from Jason Tibbitts ti...@math.uh.edu  2009-02-11 14:37:14 EDT 
---
I believe Battleship is a registered mark belonging to Hasbro.  TESS shows
several marks; I am not really sure if any of them relate to generic,
unstylized usage involving board games, since the game has existed long before
it was published in any commercial form.

Blocking FE-Legal for an opinion.

-- 
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 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

2009-02-11 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=483730


Eli Wapniarski e...@orbsky.homelinux.org changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #36 from Eli Wapniarski e...@orbsky.homelinux.org  2009-02-11 
14:44:16 EDT ---
New Package CVS Request
===
Package Name: kde-plasma-translatoid
Short Description: Translator plasmoid using Google Translator
Owners: eliwap jreznik
Branches: f-9 f-10
InitialCC: rdieter

-- 
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 225811] Merge Review: gnome-applet-vm

2009-02-11 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=225811


Cole Robinson crobi...@redhat.com changed:

   What|Removed |Added

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




--- Comment #9 from Cole Robinson crobi...@redhat.com  2009-02-11 14:49:27 
EDT ---
All the changes look good.

What is the remaining process here? Do I need to apply the patch for all
supported branches or is rawhide sufficient? Can the bug be closed after the
patch is applied?

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

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


[Bug 480538] Review Request: iptux -- a tool for sharing and transporting files and directories in Lan

2009-02-11 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=480538





--- Comment #19 from Liang Suilong liangsuil...@gmail.com  2009-02-11 
14:53:33 EDT ---
Thank you Mamoru Tasaka

Kaio and Mamoru Tasaka

I have built a SPRM with iptux-0.4.4-g++44.patch.

Here is the URL:
SPEC file:http://liangsuilong.fedorapeople.org/iptux/iptux.spec
SRPM:http://liangsuilong.fedorapeople.org/iptux/iptux-0.4.4-2.fc10.src.rpm

To Mamoru Tasaka

May I add your e-mail address into CC list? If you don't want, I will remove
your 
e-mail address in CC list.

Could you do the primary review for me? I hope iptux can be put into Fedora
repository
to help more people.

-- 
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 462505] Review Request: python-argparse - An optparse-inspired command-line parsing library

2009-02-11 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=462505





--- Comment #2 from Ruben Kerkhof ru...@rubenkerkhof.com  2009-02-11 15:51:40 
EDT ---
Hi Terje,

No problem, you can keep it ;-)
I needed it for allmydata-tahoe, but it's one of the many dependencies needed,
so thanks for getting this one in.

-- 
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 484676] Review Request: eclipse-dtp - Eclipse Data Tools Platform

2009-02-11 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=484676





--- Comment #3 from Alexander Kurtakov akurt...@redhat.com  2009-02-11 
15:47:28 EDT ---
Spec URL:
http://akurtakov.fedorapeople.org/eclipse-dtp.spec
SRPM URL:
http://akurtakov.fedorapeople.org/eclipse-dtp-1.6.1-2.fc10.src.rpm



(In reply to comment #1)
 A few minor things:
 
 - please set the fedora-review flag to ?
 - change the Requires: on java to be = (or maybe '='?) 1.5.0
Fixed.

 - I prefer to add a short name after dropins:
   %files
   %{eclipse_dropin} = %{eclipse_dropin}/dtp
Fixed.

 - please add a comment above the sed line getting rid of the sun.misc.Compare
Fixed.

 - should we add some comment(s) stating why we're only building the features 
 we
 are?
Fixed.

 
 And the rest of the review (lines beginning with X need attention; those
 beginning with * are okay):
 
 X verify the final provides and requires of the binary RPMs
   - other than the java one, things look good
Fixed.

 X make sure lines are = 80 characters
   - could you add some line continuations to fix this?
Fixed wherever possible some paths are just too long.

 X package successfully compiles and builds
   - is this expected?
 
 [javac] 4. ERROR in
 /home/overholt/rpmbuild/BUILD/dtp-1.6.1/build/plugins/org.eclipse.datatools.connectivity.oda.design/src/org/eclipse/datatools/connectivity/oda/design/impl/InputElementUIHintsImpl.java
 [javac]  (at line 112)
 [javac]  assert (eContainer() instanceof InputElementAttributes);
 [javac]  ^^
 [javac] The method assert(boolean) is undefined for the type
 InputElementUIHintsImpl
 
Fixed.

-- 
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 480859] Review Request: diffuse - graphical diff tool

2009-02-11 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=480859





--- Comment #10 from Jon Levell fed...@coralbark.net  2009-02-11 15:54:38 EDT 
---
Another update:

Spec URL: http://coralbark.net/fedora/diffuse/diffuse.spec
SRPM URL: http://coralbark.net/fedora/diffuse/diffuse-0.2.15-4.fc10.src.rpm
RPM URL: http://coralbark.net/fedora/diffuse/diffuse-0.2.15-4.fc10.noarch.rpm

I've addressed the points from comment #9. I can see that I was very naive when
I initially submitted the spec file on the grounds that it seemed to work fine
on the systems I installed it on.

Thanks Michael (and everyone else) for the helpful comments

-- 
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 483865] Review Request: bpg-fonts - Georgian Unicode fonts

2009-02-11 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=483865





--- Comment #5 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-02-11 
16:10:19 EDT ---
Ok, some feedback (awesome work BTW)

1. minor: since consensus seems to be that multiple directory ownership is ok
in some conditions, %_font_pkg now does it automatically, so you don't need the
%dir %{_fontdir} in common anymore (though it's probably harmless). Will need
to update the wiki templates to reflect this

2. it does not build in mock, but does build in koji, so your package is
probably fine and that's a mock bug (bug #485146)

3. major: the font files declare font family names like BPG Courier S
GPLGNU. Since fontconfig works with whatever the font files declare, you need
to use those names in your fontconfig files in you want them to work (how to
pass the  safely is an interesting problem to discuss with Behdad)

4. I'd have used a fontconfig priority of 63-64 to make sure the latin glyphs
in those fonts do not override the ones in more mainstream fonts

-- 
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 484931] Renaming Review Request: hosts3d - 3D real-time network visualiser

2009-02-11 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=484931


Jochen Schmitt joc...@herr-schmitt.de changed:

   What|Removed |Added

 CC||joc...@herr-schmitt.de




--- Comment #2 from Jochen Schmitt joc...@herr-schmitt.de  2009-02-11 
16:20:17 EDT ---
I think, you should wrote

Provides:homestead = %{version}-%{release}
Obsoletes:   homestead  0.95-1

-- 
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 484931] Renaming Review Request: hosts3d - 3D real-time network visualiser

2009-02-11 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=484931





--- Comment #3 from Simon Wesp cassmod...@fedoraproject.org  2009-02-11 
16:29:30 EDT ---
-
Provides:homestead = %{version}-%{release}
Obsoletes:   homestead  0.95-1
-

Perhaps THIS way?! This looks correct and make sense (to me)!

-- 
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 484931] Renaming Review Request: hosts3d - 3D real-time network visualiser

2009-02-11 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=484931


Jochen Schmitt joc...@herr-schmitt.de changed:

   What|Removed |Added

   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 484931] Renaming Review Request: hosts3d - 3D real-time network visualiser

2009-02-11 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=484931


Jochen Schmitt joc...@herr-schmitt.de changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|joc...@herr-schmitt.de




--- Comment #4 from Jochen Schmitt joc...@herr-schmitt.de  2009-02-11 
16:41:20 EDT ---
Review will be done tomorrow.

-- 
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 484379] Review Request: hartke-aurulent-sans-fonts - A sans-serif font

2009-02-11 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=484379


Nicolas Mailhot nicolas.mail...@laposte.net changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|pala...@gmx.de
   Flag||fedora-review+




--- Comment #1 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-02-11 
16:45:18 EDT ---
1. This bit is not strictly necessary
# Repeat for every font family -- Sans Mono

2. This one isn't either now that multiple ownership of font dirs was approved;
%_font_pkg will now auto-own this dir
%dir %{_fontdir}

3. having upstream publish an official sfd version and build from it would be
preferred (not a blocker)

4. You need to add a
Requires:   %{name}-common = %{version}-%{release} to the main package or
it won't pull common in (and it's necessary for fontconfig-filesystem)

The last one is the only real problem and it's a minor one

✎✎✎ APPROVED ✎✎✎

Please do not forget to fix those little problems, and continue from
http://fedoraproject.org/wiki/Font_package_lifecycle#3.a

-- 
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 485154] New: Review Request: perl-MooseX-Role-Cmd - Wrap system command binaries the Moose way

2009-02-11 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-MooseX-Role-Cmd - Wrap system command binaries 
the Moose way

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

   Summary: Review Request: perl-MooseX-Role-Cmd - Wrap system
command binaries the Moose way
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: allis...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://allisson.fedorapeople.org/packages/perl-MooseX-Role-Cmd/perl-MooseX-Role-Cmd.spec

SRPM URL:
http://allisson.fedorapeople.org/packages/perl-MooseX-Role-Cmd/perl-MooseX-Role-Cmd-0.04-1.fc10.src.rpm

Description: MooseX::Role::Cmd is a Moose role intended to ease the task of
building 
command-line wrapper modules. It automatically maps Moose objects into 
command strings which are passed to IPC::Cmd.

-- 
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 484676] Review Request: eclipse-dtp - Eclipse Data Tools Platform

2009-02-11 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=484676


Andrew Overholt overh...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #4 from Andrew Overholt overh...@redhat.com  2009-02-11 17:10:44 
EDT ---
Thanks for the updates.  The only thing I'd like to see is a comment at the
beginning of the specfile stating which features we're building and which ones
we're not and reasons for that.  Please add that, but since it's not a huge
issue, I'll approve this now.

I've started a wiki page that we can use to track features/plugins we'd like
and their dependencies.  We can add bug links to the wiki page as we file bugs.

https://fedoraproject.org/wiki/Eclipse#Plug-ins_We.27d_Like_To_Ship

-- 
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 485159] New: Review Request: anki - Flashcard program for using space repetition learning

2009-02-11 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: anki - Flashcard program for using space repetition 
learning

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

   Summary: Review Request: anki - Flashcard program for using
space repetition learning
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: c...@plauener.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


anki - Flashcard program for using space repetition learning
Spec URL: http://www-user.tu-chemnitz.de/~tiwi/anki.spec
SRPM URL: http://www-user.tu-chemnitz.de/~tiwi/anki-0.9.9.6-1.fc10.src.rpm
Description: Anki is a program designed to help you remember facts (such as
words
and phrases in a foreign language) as easily, quickly and efficiently
as possible. Anki is based on a theory called spaced repetition.

This is my first package and so I'm seeking a sponsor.

-- 
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 485159] Review Request: anki - Flashcard program for using space repetition learning

2009-02-11 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=485159


Christian Krause c...@plauener.de changed:

   What|Removed |Added

 Blocks||177841




-- 
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 485000] Review Request: libbsr - Barrier Synchronization Register access library

2009-02-11 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=485000





--- Comment #5 from Tony Breeds t...@bakeyournoodle.com  2009-02-11 17:57:32 
EDT ---
(In reply to comment #4)

 Hmm okay.  I can see how to fix that.  I  clearly need to read the rpmbuild
 output more clearly.

Fixed, new scratch build here:
  https://koji.fedoraproject.or/koji/taskinfo?taskID=1120493

new SRPM and SPEC file at:
   http://bakeyournoodle.com/~tony/fedora/libbsr/

rpmlint is still clear:
[t...@mango fedora]$ rpmlint -vi SRPMS/libbsr*3.fc10.src.rpm
RPMS/ppc64/libbsr*3.fc10.ppc64.rpm SPECS/libbsr.spec 
libbsr.src: I: checking
libbsr.ppc64: I: checking
libbsr-debuginfo.ppc64: I: checking
libbsr-devel.ppc64: I: checking
4 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
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 483301] Review Request: muse - Midi/Audio Music Sequencer

2009-02-11 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=483301


Robert Scheck redhat-bugzi...@linuxnetz.de changed:

   What|Removed |Added

 CC||redhat-bugzi...@linuxnetz.d
   ||e




--- Comment #14 from Robert Scheck redhat-bugzi...@linuxnetz.de  2009-02-11 
18:37:56 EDT ---
Orcan, it would be useful, if you would accept input from packagers, even
if you don't like them e.g. on personal base or caused by their input. From
my point of view of being a provenpackager (ueberpackager), comment #2 is 
is useful and should be followed (the Epoch thing can be discussed maybe).

With respect to all what happened unil now: Valid input needs to be taken,
handled and/or discussed. If such things are refused without a valid reason,
rejecting a package review request seems to be pretty okay for me. Packaging
is not that easy as it maybe seems or some people want to have it.

-- 
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 485164] New: Review Request: perl-XML-Generator - Perl module for generating XML

2009-02-11 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-XML-Generator - Perl module for generating XML

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

   Summary: Review Request: perl-XML-Generator - Perl module for
generating XML
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ianburr...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://znark.com/fedora/perl-XML-Generator.spec
SRPM URL: http://znark.com/fedora/perl-XML-Generator-1.01-1.fc10.src.rpm
Description: Perl module for generating XML by calling methods named for each
XML 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 484323] Review Request: perl-KinoSearch - Search engine library

2009-02-11 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=484323


Ian Burrell ianburr...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #6 from Ian Burrell ianburr...@gmail.com  2009-02-11 19:55:19 EDT 
---
New Package CVS Request
===
Package Name: perl-KinoSearch
Short Description: Search engine library
Owners: iburrell
Branches: F-9 F-10 EL-5
InitialCC: perl-sig

-- 
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 471003] Review Request: cabal-install - Haskell package utility

2009-02-11 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=471003


Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

  Alias||cabal-install
   Flag|needinfo?(peter...@redhat.c |
   |om) |




--- Comment #9 from Jens Petersen peter...@redhat.com  2009-02-11 20:39:31 
EDT ---
The .spec file in comment 6 does not use %cabal_build: you trying another older
package perhaps?

-- 
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 485154] Review Request: perl-MooseX-Role-Cmd - Wrap system command binaries the Moose way

2009-02-11 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=485154


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 483331] Review Request: asterisk-sounds-core - Core sounds for Asterisk

2009-02-11 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=483331


W. Michael Petullo m...@flyn.org changed:

   What|Removed |Added

   Flag||fedora-review+




--- Comment #3 from W. Michael Petullo m...@flyn.org  2009-02-11 20:54:24 EDT 
---
Package looks good.

-- 
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 483331] Review Request: asterisk-sounds-core - Core sounds for Asterisk

2009-02-11 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=483331


W. Michael Petullo m...@flyn.org changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|m...@flyn.org




-- 
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 483331] Review Request: asterisk-sounds-core - Core sounds for Asterisk

2009-02-11 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=483331





--- Comment #4 from W. Michael Petullo m...@flyn.org  2009-02-11 20:57:35 EDT 
---
 rpmlint  asterisk-sounds-core.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
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 483301] Review Request: muse - Midi/Audio Music Sequencer

2009-02-11 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=483301





--- Comment #15 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-02-11 
21:08:45 EDT ---
Dear Robert,
I would like you to analyze the progression of this situation rather than
looking at it as a whole.

Please go ahead and have a look at my SPEC file, and comment #1. I properly
stated why I used Epoch:1 and AutoReqProv:no. I accept that I may have made a
wrong choice to handle them. Then, please read comment #2. I do NOT consider it
useful to explain what I did wrong. Saying this and this is not useful and
disregarding my comments about both issues do not help me correct myself. If
you want a useful comment, have a look at comment #11 and bug #484591's comment
#3.

Then, please read my answer, comment #3 on this bug. Do you think that I am
refusing things without a valid reason? Again, please read comment #3 once
more and explain me how you arrive at this conclusion. This is the point where
the disordered person closed my bug.



Once, I nicely told this person to not follow me around and post on my bugs. He
does nothing but gets in my nerves. 

If you want to know where this happened, please have a look at this link
(comments #2 and #3):
https://bugzilla.rpmfusion.org/show_bug.cgi?id=195
Otherwise, please proceed to the next paragraph. I apologize to bring over a
conversation that has nothing to do with Fedora, to Fedora.

Now, knowing that I do NOT want him around me (since the information he
provides has been wrong in certain cases if you saw that in the above link,
plus his repulsive attitude that has been confirmed by many people in Fedora,
is offensive), he picks my bug among 750+ other open review requests and posts
here. This has no other purpose than blowing my fuse.


I'm cutting this here. If majority of our community thinks that my behavior is
unacceptable while his is appropriate (including closing my bug), then perhaps
Fedora is not the place where I belong.

Regards,
Orcan

-- 
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 482965] Review Request: perl-XML-Generator - perl XML generator

2009-02-11 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=482965


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

   What|Removed |Added

 CC||ianburr...@gmail.com




--- Comment #7 from Parag AN(पराग) panem...@gmail.com  2009-02-11 21:14:05 
EDT ---
*** Bug 485164 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 485164] Review Request: perl-XML-Generator - Perl module for generating XML

2009-02-11 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=485164


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

   What|Removed |Added

 Status|NEW |CLOSED
 CC||panem...@gmail.com
 Resolution||DUPLICATE




--- Comment #1 from Parag AN(पराग) panem...@gmail.com  2009-02-11 21:14:05 
EDT ---


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

-- 
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 474044] Review Request: libzdb - A small, fast, and easy to use database API

2009-02-11 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=474044





--- Comment #9 from Bernard Johnson bjohn...@symetrix.com  2009-02-11 
21:19:00 EDT ---
(In reply to comment #8)
 Accordingly, we should treat this code as GPLv3+ only, and ignore the
 EXCEPTIONS.

Should the EXCEPTIONS file be left in the %doc section or left out completely?

-- 
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 482965] Review Request: perl-XML-Generator - perl XML generator

2009-02-11 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=482965


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #8 from Dennis Gilmore den...@ausil.us  2009-02-11 21:26:21 EDT 
---
New Package CVS Request
===
Package Name: perl-XML-Generator
Short Description: perl module for generating XML documents
Owners: ausil
Branches: EL-5 F-9 F-10
InitialCC: perl-sig

-- 
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 482965] Review Request: perl-XML-Generator - perl XML generator

2009-02-11 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=482965





--- Comment #9 from Dennis Gilmore den...@ausil.us  2009-02-11 21:26:39 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 485000] Review Request: libbsr - Barrier Synchronization Register access library

2009-02-11 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=485000





--- Comment #6 from Tony Breeds t...@bakeyournoodle.com  2009-02-11 21:49:01 
EDT ---
Upstream have released v0.2 including the patch from the RPM.  I've respun with
0.2

New scratch build here:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=1120621

new SRPM and SPEC file at:
   http://bakeyournoodle.com/~tony/fedora/libbsr/

rpmlint is still clear:
[t...@mango fedora]$ rpmlint -vi SRPMS/libbsr*0.2*rpm RPMS/ppc64/libbsr*0.2*rpm
SPECS/libbsr.spec
libbsr.src: I: checking
libbsr.ppc64: I: checking
libbsr-debuginfo.ppc64: I: checking
libbsr-devel.ppc64: I: checking
4 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
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 445261] Review Request: icelandic-fonts - Icelandic Magical Staves

2009-02-11 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=445261


Jon Stanley jonstan...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




--- Comment #12 from Jon Stanley jonstan...@gmail.com  2009-02-11 22:49:41 
EDT ---
This has been in for a long time, 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


  1   2   >