[Bug 559552] Review Request: qca-pkcs11 - Smartcard integration for QCA

2010-05-24 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=559552

Chen Lei  changed:

   What|Removed |Added

 CC||supercyp...@gmail.com

--- Comment #8 from Chen Lei  2010-05-25 02:18:54 EDT ---
All qca plugin seems merged to qca in svn.

See http://websvn.kde.org/trunk/kdesupport/qca/plugins/

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594744] Review Request: zint - A barcode generator and library

2010-05-24 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=594744

--- Comment #6 from Martin Gieseking  2010-05-25 
01:49:28 EDT ---
(In reply to comment #5)
> Adding BR to subpackages make no sense.

To me, it does make sense to indicate where the BRs are actually needed. As the
rpm specification allows it, the question is if the Fedora guidelines want them
kept together in the base package. However, it's not that important to me. I'll
move them to the base 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 581278] Review Request: erlang-etap - Erlang testing library

2010-05-24 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=581278

Peter Lemenkov  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Peter Lemenkov  2010-05-25 01:13:04 EDT 
---
New Package CVS Request
==
Package Name: erlang-etap
Short Description: Erlang testing library
Owners: peter
Branches: F-11 F-12 F-13 EL-6 EL-5 EL-4
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 578759] Review request: django-mako - Mako Templates Plugin for Django

2010-05-24 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=578759

Rahul Sundaram  changed:

   What|Removed |Added

 CC||methe...@gmail.com
 AssignedTo|nob...@fedoraproject.org|methe...@gmail.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 591222] Review Request: gtk3 - version 3 of GTK+

2010-05-24 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=591222

--- Comment #15 from Chen Lei  2010-05-25 00:32:03 EDT 
---
Some many suggestion:

Most IM packages(scim, ibus) add %config to its conf file.


%{_sysconfdir}/X11/xinit/xinput.d/im-cedilla.conf
->%config  %{_sysconfdir}/X11/xinit/xinput.d/im-cedilla.conf


I suggest to add a gnome-filessystem to fedora, like kde-filesystem,
mozilla-filesystem to own some common directories in gnome. So that, gnome
packages can only depends on this filesystem package instead of owning the
directories.
e.g.
%{_datadir}/gtk-doc
%{_datadir}/gtk-doc/html 
%{_datadir}/gir-1.0
%{_datadir}/gnome/help

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 591222] Review Request: gtk3 - version 3 of GTK+

2010-05-24 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=591222

Chen Lei  changed:

   What|Removed |Added

 CC||supercyp...@gmail.com

--- Comment #14 from Chen Lei  2010-05-25 00:17:04 EDT 
---
I'll be much better to not require gtk-doc in gtk3-devel-docs
Simply own /usr/share/gtk-doc/html in %file will be more appropiate, actually
html documentations in /usr/share/gtk-doc/html is used by devhelp or webbrower
not by gtk-doc. gtk-doc is a tool to generate those html files.

Requires: automake is also not necessrary, it'll pull in some additional
dependency which is useless for most SRPMs.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 567678] Review Request: Cameramonitor - Notification icon when webcam is active

2010-05-24 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=567678

Andrew Dingman  changed:

   What|Removed |Added

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

--- Comment #15 from Andrew Dingman  2010-05-24 23:25:02 
EDT ---
My daughter is 3 days old. I truly do appreciate your feedback and attention,
but try again in a month?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 590522] Review Request: plexus-cipher-1.4-1 - Plexus Cipher: encryption/decryption Component

2010-05-24 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=590522

--- Comment #19 from huwang  2010-05-24 22:43:45 EDT ---
(In reply to comment #18)
> Ok the commons-logging thing has been fixed, but it still fails to build for
> me:
> 
> http://koji.fedoraproject.org/koji/taskinfo?taskID=2201405
> 
> 
> [INFO] 
> 
> [ERROR] BUILD ERROR
> [INFO] 
> 
> [INFO] Internal error in the plugin manager executing goal
> 'org.apache.maven.plugins:maven-javadoc-plugin:2.4:javadoc': Unable to find 
> the
> mojo 'org.apache.maven.plugins:maven-javadoc-plugin:2.4:javadoc' in the plugin
> 'org.apache.maven.plugins:maven-javadoc-plugin'
> org.apache.maven.doxia.logging.LogEnabled
> [INFO] 
> 
> 
> 
> I'm not sure what "Unable to find the mojo" means, is it a problem with this
> package or a problem in the maven javadoc plugin?

I think this isn't this package's problem. I built again, it isn't failed, the
link is http://koji.fedoraproject.org/koji/taskinfo?taskID=2207394

spec URL:
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-cipher.spec&can=2&q=#makechanges

SRPM URL: 
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-cipher-1.5-1.src.rpm&can=2&q=#makechanges

please review again, thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 595268] Review Request: tangerine-fonts - Tangerine is a calligraphy font inspired by many italic chancery hands

2010-05-24 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=595268

Daiki Ueno  changed:

   What|Removed |Added

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

--- Comment #1 from Daiki Ueno  2010-05-24 22:06:30 EDT ---
I'm trying to review.  The packaging looks mostly ok with me, except that:

1. packaging request

Please create a font description wiki page for this font:
http://fedoraproject.org/wiki/Font_package_lifecycle#1

2. the fontconfig.conf prefix is 65

Since this font seems to contain only LGC glyphs (and some symbols), I think it
should be 61-64:
http://fedoraproject.org/wiki/Fontconfig_packaging_tips#Choosing_a_ruleset_numeral_prefix

3. locale-specific override without "lang" test

IIUC, it works as if an inverted "Simple priority list" rule applies:
http://fedoraproject.org/wiki/Fontconfig_packaging_tips#Locale-specific_overrides
I guess 2 & 3 are complementary each other, is that intended?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594744] Review Request: zint - A barcode generator and library

2010-05-24 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=594744

--- Comment #5 from Chen Lei  2010-05-24 21:51:20 EDT ---

> > 3. I'll be better to include .desktop as Source1 in spec
> I prefer keeping the desktop file in the spec. :) It's kind of personal
> preference, I think. I'll ask upstream to add it to the tarball, though.

Historically, fedora packaging guideline only accepts including desktop file as
a source file. But, recently incluing desktop file in spec is also accepted.

> > 7.
> > BuildRequires: qt-devel >= 4.4
> > BuildRequires: desktop-file-utils
> > 
> > BR should not be added to subpackages.
> Is there a hint about this in the guidelines? Technically, there's no
> difference whether the BRs are placed in the base package or a subpackage.

Adding BR to subpackages make no sense.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594943] Review Request: libnoise - A general-purpose library that generates three-dimensional coherent noise

2010-05-24 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=594943

--- Comment #11 from Robin Norwood  2010-05-24 
21:02:51 EDT ---
I've emailed MathMap's developer and am awaiting a response.  I'll update this
bug when I hear back from him.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 595551] Review Request: ibus-table-mathwriter - ibus-table-mathwriter provides input method for writing Unicode Math symbols

2010-05-24 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=595551

Naveen Kumar  changed:

   What|Removed |Added

 CC||fedora-package-rev...@redha
   ||t.com,
   ||i18n-b...@lists.fedoraproje
   ||ct.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 578480] Review Request: spectrum - XMPP transport/gateway

2010-05-24 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=578480

--- Comment #11 from Matej Cepl  2010-05-24 18:46:40 EDT ---
Don't we need a special user "spectrum"? Any group?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 578480] Review Request: spectrum - XMPP transport/gateway

2010-05-24 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=578480

--- Comment #12 from Matej Cepl  2010-05-24 18:48:55 EDT ---
Should also emphasize somewhere (%description? README.Fedora?) that users of
RHEL-5 needs to have enabled RHEL Optional Productivity Apps channel.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 578480] Review Request: spectrum - XMPP transport/gateway

2010-05-24 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=578480

--- Comment #10 from Matej Cepl  2010-05-24 18:28:23 EDT ---
Revised RHEL-5 build (no, I haven't bumped Release, sorry)
http://koji.fedoraproject.org/koji/taskinfo?taskID=2207035

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 578480] Review Request: spectrum - XMPP transport/gateway

2010-05-24 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=578480

--- Comment #9 from Matej Cepl  2010-05-24 18:27:17 EDT ---
I guess it casues installation of at least:

X cairo   i386 1.2.4-5.el5  rhel-i386-server-5 394
k
X cups-libs   i386 1:1.3.7-18.el5   rhel-i386-server-5 197
k
X cvs i386 1.11.22-7.el5rhel-i386-server-5 725
k
X gettext-devel   i386 0.14.6-4.el5 rhel-i386-server-5 1.0
M
X gtk2i386 2.10.4-20.el5rhel-i386-server-5 6.5
M
X libgcj  i386 4.1.2-48.el5 rhel-i386-server-5  16
M
X pango   i386 1.14.9-8.el5 rhel-i386-server-5 335
k

25MB of packages.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 578480] Review Request: spectrum - XMPP transport/gateway

2010-05-24 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=578480

--- Comment #8 from Matej Cepl  2010-05-24 17:48:49 EDT ---
One more problem (but that's probably for upstream):

[r...@luther ~]# rpm -qRp /home/matej/spectrum-0.2-2.el5.i386.rpm |grep gettext
libgettextpo.so.0  
[ma...@tikanga build]$ rpm -qf /usr/lib/libgettextpo.so.0
gettext-devel-0.14.6-4.el5
[ma...@tikanga build]$ 

Why do I need gettext-devel (with all its numerous dependencies) for spectrum?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 587438] Review Request: rubygem-snmp - A Ruby implementation of SNMP (the Simple Network Management Protocol)

2010-05-24 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=587438

--- Comment #9 from Michael Stahnke  2010-05-24 17:49:16 
EDT ---
I've contacted upstream about the test failures and fixed the macros usage. 
They will be fixed upon import.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 587438] Review Request: rubygem-snmp - A Ruby implementation of SNMP (the Simple Network Management Protocol)

2010-05-24 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=587438

Michael Stahnke  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Michael Stahnke  2010-05-24 17:48:46 
EDT ---
New Package CVS Request
===
Package Name: rubygem-snmp
Short Description: SNMP bindings for Ruby 
Owners: stahnma
Branches: F-12 F-13  EL-5 EL-6
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561456] Review Request: jnr-x86asm - Pure-java port of asmjit

2010-05-24 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=561456

--- Comment #4 from Mohammed Morsi  2010-05-24 17:41:05 EDT 
---
(In reply to comment #2)
> NOTOK: rpmlint must be run on every package. The output should be posted in 
> the
> review.
> NOTOK: The package must be named according to the Package Naming 
> NOTOK: The sources used to build the package must match the upstream source, 
> as
> provided in the spec URL.
> 
> Output:
> jnr-x86asm.src: W: spelling-error Summary(en_US) asmjit -> Asmara, Asquith,
> fajitas
> jnr-x86asm.src: W: spelling-error %description -l en_US asmjit -> Asmara,
> Asquith, fajitas
> jnr-x86asm.src: W: invalid-url Source0: jnr-x86asm-0.1.tgz
> jnr-x86asm.noarch: W: spelling-error Summary(en_US) asmjit -> Asmara, Asquith,
> fajitas
> jnr-x86asm.noarch: W: spelling-error %description -l en_US asmjit -> Asmara,
> Asquith, fajitas
> jnr-x86asm-javadoc.noarch: W: spelling-error Summary(en_US) asm -> as, am, mas
> jnr-x86asm-javadoc.noarch: W: spelling-error %description -l en_US asm -> as,
> am, mas
> 3 packages and 0 specfiles checked; 0 errors, 7 warnings.

I'm ignoring all the spelling errors as it's complaining about 'asmjit' which
is a legit part of the package summary / description. (Also I'm not seeing
those warnings locally, running rpmlint against the SRPM doesn't yield those
spelling errors)


> 
> While spec file explains how to create Source0 it fails to provide
> reliable way to verify no changes happened between Source0 was taken
> and building of RPM. You need to specify git hash (part of it at
> least) of commit that should be checked out after cloning. Once some
> version is released by developer, it would be ideal if git tag hash was
> used instead.
> 

FIXED. It seems the upstream author released a project tarball not long after I
uploaded this package, so now I'm just using that.


> This also applies to naming of package. This is most certainly a
> pre-relase so naming convetions are these:
> https://fedoraproject.org/wiki/Packaging:NamingGuidelines#PreReleasePackages
> 

This is not a pre-release. This is the 0.1 release. According to that wiki link
a prerelease would be something like 0.2alpha or what not.



> 
> OK: The spec file name must match the base package %{name}, in the format
> %{name}.spec unless your package has an exemption.  .
> OK: The package must meet the Packaging Guidelines .
> OK: The package must be licensed with a Fedora approved license and meet the
> Licensing Guidelines .
> NOTOK: The License field in the package spec file must match the actual
> license. 
> 
> License of package is obviously LGPLv3 not MIT

FIXED.

> 
> OK: 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.
> 
> License is included,  but it would be good to include COPYING* files
> too especially since COPYING file has the text of license itself.

DONE


> 
> OK: The spec file must be written in American English. 
> OK: The spec file for the package MUST be legible. 
> OK: The package MUST successfully compile and build into binary rpms on at
> least one primary architecture. 
> OK: 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. Apply common sense.
> OK: A package must own all directories that it creates. If it does not create 
> a
> directory that it uses, then it should require a package which does create 
> that
> directory. 
> OK: A Fedora package must not list a file more than once in the spec file's
> %files listings. 
> OK: Permissions on files must be set properly. Executables should be set with
> executable permissions, for example. Every %files section must include a
> %defattr(...) line. 
> NOTOK: Each package must consistently use macros.
> you mix $RPM_BUILD_ROOT and %{buildroot}. Pick one and stick to it
> 

FIXED

> OK: The package must contain code, or permissable content.
> OK: Large documentation files must go in a -doc subpackage. (The definition of
> large is left up to the packager's best judgement, but is not restricted to
> size. Large can refer to either size or quantity). 
> OK: Packages must not own files or directories already owned by other 
> packages.
> The rule of thumb here is that the first package to be installed should own 
> the
> files or directories that other packages may rely upon. This means, for
> example, that no package in Fedora should ever share ownership with any of the
> files or directories owned by the filesystem or man package. If you feel that
> you have a good reason to own a file or directory that another package owns,
> then please present that at package review time. 
> OK: All filenames in rpm packages must be v

[Bug 581619] Review Request: mlterm - Multi Lingual TERMinal emulator on X

2010-05-24 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=581619

--- Comment #8 from Muayyad Alsadi  2010-05-24 17:13:55 EDT 
---
up!

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561451] Review Request: nailgun - Framework for running Java from the cli without the JVM startup overhead

2010-05-24 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=561451

--- Comment #12 from Fedora Update System  
2010-05-24 16:18:46 EDT ---
nailgun-0.7.1-2.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/nailgun-0.7.1-2.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561451] Review Request: nailgun - Framework for running Java from the cli without the JVM startup overhead

2010-05-24 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=561451

--- Comment #11 from Fedora Update System  
2010-05-24 16:09:41 EDT ---
nailgun-0.7.1-2.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/nailgun-0.7.1-2.fc13

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579376] Review Request: perl-CGI-Application-PSGI - PSGI Adapter for CGI::Application

2010-05-24 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=579376

--- Comment #26 from Fedora Update System  
2010-05-24 15:46:11 EDT ---
perl-CGI-Application-PSGI-1.00-1.fc11 has been pushed to the Fedora 11 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-24 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=588122

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|perl-Symbol-Util-0.0202-3.f |perl-Symbol-Util-0.0202-3.f
   |c12 |c11

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579376] Review Request: perl-CGI-Application-PSGI - PSGI Adapter for CGI::Application

2010-05-24 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=579376

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|perl-CGI-Application-PSGI-1 |perl-CGI-Application-PSGI-1
   |.00-1.fc12  |.00-1.fc11

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

2010-05-24 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=593879

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|libvpx-0.9.0-5.fc11 |libvpx-0.9.0-5.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

2010-05-24 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=593879

--- Comment #22 from Fedora Update System  
2010-05-24 15:45:37 EDT ---
libvpx-0.9.0-5.fc12 has been pushed to the Fedora 12 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-24 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=588122

--- Comment #14 from Fedora Update System  
2010-05-24 15:49:13 EDT ---
perl-Symbol-Util-0.0202-3.fc11 has been pushed to the Fedora 11 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 566560] Review Request: libaesgm - Library implementation of AES (Rijndael) cryptographic methods

2010-05-24 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=566560

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|libaesgm-20090429-2.fc13|libaesgm-20090429-2.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 566560] Review Request: libaesgm - Library implementation of AES (Rijndael) cryptographic methods

2010-05-24 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=566560

--- Comment #12 from Fedora Update System  
2010-05-24 15:47:09 EDT ---
libaesgm-20090429-2.fc12 has been pushed to the Fedora 12 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579369] Review Request: kanjistrokeorders-fonts - Font to view stroke order diagrams for Kanji, Kana and etc...

2010-05-24 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=579369

--- Comment #14 from Fedora Update System  
2010-05-24 15:44:17 EDT ---
kanjistrokeorders-fonts-2.014-4.fc12 has been pushed to the Fedora 12 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579376] Review Request: perl-CGI-Application-PSGI - PSGI Adapter for CGI::Application

2010-05-24 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=579376

--- Comment #24 from Fedora Update System  
2010-05-24 15:40:48 EDT ---
perl-CGI-Application-PSGI-1.00-1.fc13 has been pushed to the Fedora 13 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594943] Review Request: libnoise - A general-purpose library that generates three-dimensional coherent noise

2010-05-24 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=594943

--- Comment #10 from Robin Norwood  2010-05-24 
15:43:48 EDT ---
Ok, thanks.  I'll work on contacting the upstream developers for MathMap and
libnoise and see what we can work out.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579376] Review Request: perl-CGI-Application-PSGI - PSGI Adapter for CGI::Application

2010-05-24 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=579376

--- Comment #25 from Fedora Update System  
2010-05-24 15:43:08 EDT ---
perl-CGI-Application-PSGI-1.00-1.fc12 has been pushed to the Fedora 12 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

2010-05-24 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=593879

--- Comment #21 from Fedora Update System  
2010-05-24 15:44:06 EDT ---
libvpx-0.9.0-5.fc11 has been pushed to the Fedora 11 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579376] Review Request: perl-CGI-Application-PSGI - PSGI Adapter for CGI::Application

2010-05-24 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=579376

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|perl-CGI-Application-PSGI-1 |perl-CGI-Application-PSGI-1
   |.00-1.fc13  |.00-1.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 528288] Review Request: python-blist - A faster list implementation for Python

2010-05-24 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=528288

--- Comment #9 from Fedora Update System  2010-05-24 
15:41:17 EDT ---
python-blist-1.1.1-1.fc12 has been pushed to the Fedora 12 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update python-blist'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/python-blist-1.1.1-1.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579376] Review Request: perl-CGI-Application-PSGI - PSGI Adapter for CGI::Application

2010-05-24 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=579376

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version|perl-CGI-PSGI-0.10-1.fc13   |perl-CGI-Application-PSGI-1
   ||.00-1.fc13
 Resolution||ERRATA

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579369] Review Request: kanjistrokeorders-fonts - Font to view stroke order diagrams for Kanji, Kana and etc...

2010-05-24 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=579369

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|kanjistrokeorders-fonts-2.0 |kanjistrokeorders-fonts-2.0
   |14-4.fc13   |14-4.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 566560] Review Request: libaesgm - Library implementation of AES (Rijndael) cryptographic methods

2010-05-24 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=566560

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|libaesgm-20090429-2.fc11|libaesgm-20090429-2.fc13

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 566560] Review Request: libaesgm - Library implementation of AES (Rijndael) cryptographic methods

2010-05-24 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=566560

--- Comment #11 from Fedora Update System  
2010-05-24 15:42:22 EDT ---
libaesgm-20090429-2.fc13 has been pushed to the Fedora 13 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

2010-05-24 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=593879

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|libvpx-0.9.0-5.fc13 |libvpx-0.9.0-5.fc11

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 528288] Review Request: python-blist - A faster list implementation for Python

2010-05-24 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=528288

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA

--- Comment #8 from Fedora Update System  2010-05-24 
15:36:51 EDT ---
python-blist-1.1.1-1.fc13 has been pushed to the Fedora 13 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update python-blist'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/python-blist-1.1.1-1.fc13

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 586777] Review Request: glyphtracer - Program for creating fonts from images

2010-05-24 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=586777

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||glyphtracer-1.1-1.20100510b
   ||zr76.fc13
 Resolution||ERRATA

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579369] Review Request: kanjistrokeorders-fonts - Font to view stroke order diagrams for Kanji, Kana and etc...

2010-05-24 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=579369

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||kanjistrokeorders-fonts-2.0
   ||14-4.fc13
 Resolution||ERRATA

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 579369] Review Request: kanjistrokeorders-fonts - Font to view stroke order diagrams for Kanji, Kana and etc...

2010-05-24 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=579369

--- Comment #13 from Fedora Update System  
2010-05-24 15:38:33 EDT ---
kanjistrokeorders-fonts-2.014-4.fc13 has been pushed to the Fedora 13 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 586777] Review Request: glyphtracer - Program for creating fonts from images

2010-05-24 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=586777

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|glyphtracer-1.1-1.20100510b |glyphtracer-1.1-1.20100510b
   |zr76.fc13   |zr76.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-24 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=588122

--- Comment #13 from Fedora Update System  
2010-05-24 15:36:56 EDT ---
perl-Symbol-Util-0.0202-3.fc12 has been pushed to the Fedora 12 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 537363] Review Request: lv2-c++-tools - Support library for writing LV2 plugins in C++

2010-05-24 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=537363

--- Comment #23 from Fedora Update System  
2010-05-24 15:37:42 EDT ---
lv2-c++-tools-1.0.3-3.fc12 has been pushed to the Fedora 12 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 586777] Review Request: glyphtracer - Program for creating fonts from images

2010-05-24 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=586777

--- Comment #30 from Fedora Update System  
2010-05-24 15:39:01 EDT ---
glyphtracer-1.1-1.20100510bzr76.fc12 has been pushed to the Fedora 12 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 586777] Review Request: glyphtracer - Program for creating fonts from images

2010-05-24 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=586777

--- Comment #29 from Fedora Update System  
2010-05-24 15:35:52 EDT ---
glyphtracer-1.1-1.20100510bzr76.fc13 has been pushed to the Fedora 13 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 566560] Review Request: libaesgm - Library implementation of AES (Rijndael) cryptographic methods

2010-05-24 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=566560

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||libaesgm-20090429-2.fc11
 Resolution||ERRATA

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-24 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=588122

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|perl-Symbol-Util-0.0202-3.f |perl-Symbol-Util-0.0202-3.f
   |c13 |c12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 537363] Review Request: lv2-c++-tools - Support library for writing LV2 plugins in C++

2010-05-24 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=537363

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|lv2-c++-tools-1.0.3-3.fc13  |lv2-c++-tools-1.0.3-3.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 537363] Review Request: lv2-c++-tools - Support library for writing LV2 plugins in C++

2010-05-24 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=537363

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|lv2-c++-tools-1.0.3-3.fc11  |lv2-c++-tools-1.0.3-3.fc13

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 537363] Review Request: lv2-c++-tools - Support library for writing LV2 plugins in C++

2010-05-24 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=537363

--- Comment #22 from Fedora Update System  
2010-05-24 15:36:32 EDT ---
lv2-c++-tools-1.0.3-3.fc13 has been pushed to the Fedora 13 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-24 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=588122

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Symbol-Util-0.0202-3.f
   ||c13
 Resolution||ERRATA

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 537363] Review Request: lv2-c++-tools - Support library for writing LV2 plugins in C++

2010-05-24 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=537363

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||lv2-c++-tools-1.0.3-3.fc11
 Resolution||ERRATA

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 588122] Review Request: perl-Symbol-Util - Additional utilities for Perl symbols manipulation

2010-05-24 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=588122

--- Comment #12 from Fedora Update System  
2010-05-24 15:32:34 EDT ---
perl-Symbol-Util-0.0202-3.fc13 has been pushed to the Fedora 13 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 537363] Review Request: lv2-c++-tools - Support library for writing LV2 plugins in C++

2010-05-24 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=537363

--- Comment #21 from Fedora Update System  
2010-05-24 15:35:08 EDT ---
lv2-c++-tools-1.0.3-3.fc11 has been pushed to the Fedora 11 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 566560] Review Request: libaesgm - Library implementation of AES (Rijndael) cryptographic methods

2010-05-24 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=566560

--- Comment #10 from Fedora Update System  
2010-05-24 15:35:31 EDT ---
libaesgm-20090429-2.fc11 has been pushed to the Fedora 11 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

2010-05-24 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=593879

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||libvpx-0.9.0-5.fc13
 Resolution||ERRATA

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593879] Review Request: libvpx - VP8 Video Codec SDK

2010-05-24 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=593879

--- Comment #20 from Fedora Update System  
2010-05-24 15:32:09 EDT ---
libvpx-0.9.0-5.fc13 has been pushed to the Fedora 13 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593125] Review Request: gridsite - Grid Security for the Web, Web platforms for Grids

2010-05-24 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=593125

--- Comment #15 from Fedora Update System  
2010-05-24 15:16:02 EDT ---
gridsite-1.5.18-4.el5 has been pushed to the Fedora EPEL 5 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update gridsite'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/gridsite-1.5.18-4.el5

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593125] Review Request: gridsite - Grid Security for the Web, Web platforms for Grids

2010-05-24 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=593125

--- Comment #16 from Fedora Update System  
2010-05-24 15:17:21 EDT ---
gridsite-1.5.18-4.el4 has been pushed to the Fedora EPEL 4 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update gridsite'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/gridsite-1.5.18-4.el4

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583251] Review Request: dcfldd - Improved dd, useful for forensics and security

2010-05-24 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=583251

--- Comment #21 from Fedora Update System  
2010-05-24 15:16:51 EDT ---
dcfldd-1.3.4.1-4.el4 has been pushed to the Fedora EPEL 4 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583251] Review Request: dcfldd - Improved dd, useful for forensics and security

2010-05-24 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=583251

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|dcfldd-1.3.4.1-4.el5|dcfldd-1.3.4.1-4.el4

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583251] Review Request: dcfldd - Improved dd, useful for forensics and security

2010-05-24 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=583251

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|dcfldd-1.3.4.1-4.fc13   |dcfldd-1.3.4.1-4.el5

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583251] Review Request: dcfldd - Improved dd, useful for forensics and security

2010-05-24 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=583251

--- Comment #20 from Fedora Update System  
2010-05-24 15:16:15 EDT ---
dcfldd-1.3.4.1-4.el5 has been pushed to the Fedora EPEL 5 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 584111] Review Request: cmatrix - Simulate the display from "The Matrix"

2010-05-24 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=584111

--- Comment #13 from Ayush Mah'ri  2010-05-24 15:13:09 
EDT ---
SPEC URL : http://hakmn.fedorapeople.org/cmatrix.spec
SRPM URL : http://hakmn.fedorapeople.org/cmatrix-1.2a-4.fc12.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

2010-05-24 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=592579

--- Comment #16 from David A. Wheeler  2010-05-24 
14:39:51 EDT ---
Here are the guidelines from:
 http://fedoraproject.org/wiki/Packaging:ReviewGuidelines
It's hard for me to check on some of these until it can once again build binary
packages on all expected architectures, but I thought it'd be helpful to do as
much as I can now.  In that spirit, here are my comments:

*  MUST: rpmlint must be run on every package. The output should be posted
in the review.[1]

IN PROGRESS.  rpmlint output shown earlier, and issues being worked on.

* MUST: The package must be named according to the Package Naming
Guidelines .

OK

* MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption. [2] .

OK

* MUST: The package must meet the Packaging Guidelines .

OK generally, but see other comments (including this one).

* MUST: The package must be licensed with a Fedora approved license and
meet the Licensing Guidelines .

IN PROGRESS.  I suspect the license is fine, but it *must* be specifically
approved.

* MUST: The License field in the package spec file must match the actual
license. [3]
Spec claims:
License:LGPLv2 and GPL+ and GPLv2 and GPLv2+ and BSD and (QPL with
modifications)

See the previous threads about the licensing of this.
I did some additional checking.

In particular: I didn't see any examples of "GPL+", only GPLv2 and GPLv2+.  If
there are no source programs with GPL+, then "GPL+" should be removed from the
license list.  Did I miss it somehow?  If so, what file is "GPL+"?

Here's what I found when I looked:

Under the "src" directory I spot-checked the ".ml" and ".c" files, and they
were
all LGPL v2.1.  Same for "share" and "ptests".

The "cil" subdirectory has "LICENSE", which is (3-clause) BSD, and that matches
the .ml files under there.

external/ptmap.ml has the "Q modified license", so it *IS* used.

./tests/spec/purse.c: GPLv2 (*NOT* GPLv2+, but strictly GPLv2)
./tests/idct/idct.c: GPLv2+
./tests/idct/ieee_1180_1990.c: GPLv2+

The "licenses" directory contains this:
 GPLv3  LGPLv2.1  LGPLv3  Q_MODIFIED_LICENSE
The Q_MODIFIED_LICENSE includes the "Q modified license" and
a reference to "LGPLv2".
This *DIRECTORY* includes GPLv3, but I did not find ANY source file
that actually referred to GPLv3.  A few referred to GPLv2+,
but the license text should say "GPLv2+" and not "GPLv3".
So, even though there's a "GPLv3" in the "licenses" directory,
you should not add "GPLv3" to the Licenses entry.


* 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.[4]

FAIL.  Under %files, need to add entries for the license files that are there.

* MUST: The spec file must be written in American English. [5]

OK

* MUST: The spec file for the package MUST be legible. [6]

OK

* MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task.
If no upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.

OK - hashes match.

wget http://frama-c.com/download/frama-c-Beryllium-20090902.tar.gz
 sha512sum frama-c-Beryllium-20090902.tar.gz
~/rpmbuild/SOURCES/frama-c-Beryllium-20090902.tar.gz 
1f78aaaf99f62a5746a4e673a722d9384156d29027d3ca079c4f0d0de9afa4c4aacd6b65721375f094698453ed339b9de9f91a2ca2d41d4c5784390b0eccb0e3
 frama-c-Beryllium-20090902.tar.gz
1f78aaaf99f62a5746a4e673a722d9384156d29027d3ca079c4f0d0de9afa4c4aacd6b65721375f094698453ed339b9de9f91a2ca2d41d4c5784390b0eccb0e3
 /home/dwheeler/rpmbuild/SOURCES/frama-c-Beryllium-20090902.tar.gz


* MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture. [7]

IN PROGRESS.

* 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. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. [8]

FAIL - need to add those.

* 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. Apply common sense.

Nothing obviously wrong, but need to check with Mock.

* MUST: The spec file MUST handle locales properly. This is done by using
the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.[9]

NA; no locale info.

* MUST

[Bug 594943] Review Request: libnoise - A general-purpose library that generates three-dimensional coherent noise

2010-05-24 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=594943

--- Comment #9 from Martin Gieseking  2010-05-24 
14:36:54 EDT ---
I agree, 1) is not an option here. The best variant would be a corporative
upstream developer. If he can't be convinced to add the patch, the cleanest way
would probably be a fork of the library. This leads to additional work, though.
I think, Fedora shouldn't provide a shared library with an updated interface
that differs from upstream. However, I'm not sure about this. Maybe you can ask
on the devel or packaging list for further suggestions.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

2010-05-24 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=592579

--- Comment #15 from Mark Rader  2010-05-24 14:36:38 EDT ---
I should have this one fixed soon, I will probably get it fixed tonight.  I
will also be able to address some of the ones listed in the previous error
report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594844] Review Request: iec16022 - Generate ISO/IEC 16022 2D barcodes

2010-05-24 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=594844

Ville Skyttä  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|ville.sky...@iki.fi
   Flag||fedora-review?

--- Comment #3 from Ville Skyttä  2010-05-24 14:37:10 EDT 
---
1) If no previous version of -libs is installed (for example in a clean mock
build), the test suite throws errors but doesn't fail the build:

make: Entering directory `/builddir/build/BUILD/iec16022-0.2.4/test'
make  check-TESTS
make[1]: Entering directory `/builddir/build/BUILD/iec16022-0.2.4/test'
/builddir/build/BUILD/iec16022-0.2.4/.libs/lt-iec16022: error while loading
shared libraries: libiec16022.so.0: cannot open shared object file: No such
file or directory
[...]
diff: testsuite-test.text: No such file or directory
Text test FAILED
diff: testsuite-test.eps: No such file or directory
EPS test FAILED
diff: testsuite-test.png: No such file or directory
PNG test FAILED
diff: testsuite-test.bin: No such file or directory
Bin test FAILED
diff: testsuite-test.hex: No such file or directory
Hex test FAILED
rm: cannot remove `testsuite-test.*': No such file or directory
PASS: ./testsuite.sh
=
1 test passed
=

Some LD_LIBRARY_PATH settings should be added to make the test suite do the
right thing (use the just built libraries, this should be reported upstream),
for example this at start of %check:
export LD_LIBRARY_PATH=$RPM_BUILD_ROOT%{_libdir}

And patching of upstream build process/test scripts is needed; if tests fail,
the test suite and build should fail too.

2) (Cosmetic) --disable-dependency-tracking is the default nowadays, it can be
dropped from %configure arguments.

3) Would not hurt to make the dependency on -libs in -devel arch qualified,
ditto perhaps add one from main package to -libs:
Requires: %{name}-libs%{?_isa} = %{version}-%{release}
http://fedoraproject.org/wiki/PackagingDrafts/ArchSpecificRequires

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 587438] Review Request: rubygem-snmp - A Ruby implementation of SNMP (the Simple Network Management Protocol)

2010-05-24 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=587438

Mamoru Tasaka  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Mamoru Tasaka  2010-05-24 
14:23:14 EDT ---
Well, I think that if the provided tests fail, at least
- the cause should be examined (especially the cause is because
  the failed test is badly written or the main program itself
  is broken)
- the fact should be noticed to the upstream that the provided test
  fails

And if patch is available, submitting the patch to the upstream will
be appreciated (by the upstream).

By the way one other thing.
* Consistent macros usage
  - If you want to use %{__sed} style, please use this style everywhere
if this is available for consistency (like %{__mkdir}, %{__rm},
%{__mv}).

-
 This package (rubygem-snmp) is APPROVED by mtasaka
-

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 478613] Review Request: ledger - A powerful command-line double-entry accounting system

2010-05-24 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=478613

Michel Alexandre Salim  changed:

   What|Removed |Added

   Flag||needinfo?(radf...@blackbean
   ||.org)

--- Comment #6 from Michel Alexandre Salim  
2010-05-24 14:21:56 EDT ---
Until RPM supports soft dependencies of the recommends/suggests sort, there is
no good way of fixing the issue. I'm personally happy with the current
guidelines -- though unhappy with the lack of soft dependency -- but given that
your current package violates two compulsory points:

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

http://fedoraproject.org/wiki/Packaging:ReviewGuidelines#Things_To_Check_On_Review
MUST: Packages must not own files or directories already owned by other
packages

I'd find it hard to go ahead with this review. I suggest contacting the
fedora-packaging list for a clarification, or I can step back and you can try
convincing another maintainer that dangling links are the best way out of this.

(when RPM gets the ability to recommends packages, of course, you would be able
to just make the main ledger app recommends emacs-ledger, and keep Vim users
happy)

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594744] Review Request: zint - A barcode generator and library

2010-05-24 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=594744

--- Comment #4 from Martin Gieseking  2010-05-24 
14:24:12 EDT ---
Thanks for the comments, Chen Lei. I'll update the package in about a week when
version 2.3.2 is out. It incorporates the locale patch and fixes the
shared-lib-calls-exit issue.

(In reply to comment #3)
> 1.I suggest you to split shlibs into subpackages, e.g. zint-libs zint-qt-libs.
> Fedora ships multilibs in x86_64 and ppc64.

Yes, that's a good idea. However, it leads to 6 pretty small packages. If this
is not a problem, I'll go with it.

> 3. I'll be better to include .desktop as Source1 in spec

I prefer keeping the desktop file in the spec. :) It's kind of personal
preference, I think. I'll ask upstream to add it to the tarball, though.

> 7.
> BuildRequires: qt-devel >= 4.4
> BuildRequires: desktop-file-utils
> 
> BR should not be added to subpackages.

Is there a hint about this in the guidelines? Technically, there's no
difference whether the BRs are placed in the base package or a subpackage.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 595011] Review Request: sshdfilter - Filter for SSH ports

2010-05-24 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=595011

Jason Tibbitts  changed:

   What|Removed |Added

Summary|Review Request: |Review Request: sshdfilter
   ||

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 572989] Review Request: sugar-paint - Paint activity for Sugar

2010-05-24 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=572989

Sebastian Dziallas  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

--- Comment #5 from Sebastian Dziallas  2010-05-24 14:09:50 
EDT ---
Built in Rawhide and others, update being on its way soon. Thanks for the
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 555990] Review Request: rubygems-acts-as-list - Sorting and reordering capabilities for ActiveRecord model

2010-05-24 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=555990

Mamoru Tasaka  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Mamoru Tasaka  2010-05-24 
13:53:38 EDT ---
Well,

* Please remove backup .requirear file from %buildroot (this file
  is also copyed into %buildroot by "cp -a" as it is).

Please fix the above issue when importing this package into
Fedora CVS.

--
This package (rubygem-acts-as-tree) is APPROVED by mtasaka
--

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

2010-05-24 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=592579

--- Comment #14 from David A. Wheeler  2010-05-24 
13:50:49 EDT ---
The updated rpm fails on 32-bit systems.  You have a built-in assumption that
there is a "lib64", but on 32-bit system it's just "lib".  Here's the error:
cp -f man/frama-c.1
/home/dwheeler/rpmbuild/BUILDROOT/frama-c-1.4-1.fc11.i386/usr/share/man/man1/frama-c-gui.1
for plugin in ; do \
   MAKEFLAGS="" make
FRAMAC_LIBDIR=/home/dwheeler/rpmbuild/BUILDROOT/frama-c-1.4-1.fc11.i386/usr/lib/frama-c
FRAMAC_SHARE=/home/dwheeler/rpmbuild/BUILD/frama-c-Beryllium-20090902/share -C
$plugin install; \
 done
+ strip
/home/dwheeler/rpmbuild/BUILDROOT/frama-c-1.4-1.fc11.i386/usr/bin/frama-c
+ strip
/home/dwheeler/rpmbuild/BUILDROOT/frama-c-1.4-1.fc11.i386/usr/bin/frama-c-gui
+ strip
/home/dwheeler/rpmbuild/BUILDROOT/frama-c-1.4-1.fc11.i386/usr/bin/frama-c.byte
+ strip
/home/dwheeler/rpmbuild/BUILDROOT/frama-c-1.4-1.fc11.i386/usr/bin/frama-c-gui.byte
+ strip
/home/dwheeler/rpmbuild/BUILDROOT/frama-c-1.4-1.fc11.i386/usr/lib64/frama-c/plugins/Ltl_to_acsl.cmxs
strip:
'/home/dwheeler/rpmbuild/BUILDROOT/frama-c-1.4-1.fc11.i386/usr/lib64/frama-c/plugins/Ltl_to_acsl.cmxs':
No such file
error: Bad exit status from /var/tmp/rpm-tmp.TXN0rD (%install)

Can you fix this soon?

Although I haven't looked in detail at the packaging of "why", I suspect it'd
be fairly easy to create a new subpackage "why-jessie"; the new subpackage
would include Jessie and the Jessie plug-in, and depend on frama-c.  I think
that would accurately reflect the situation.  Jessie is delivered as part of
the source code of "why", you can use other parts of "why" without using
Jessie, but Jessie requires the use of frama-c (to create its data format and
the drop-in location for its plug-in).  That shouldn't be hard to do once the
"frama-c" package is in the repository.

Go ahead and package the Frama-C version you're using.  I hope that in the
not-to-distant future you'll update this package to the just-released new
version, but I understand that it's easier to not change versions midstream. 
And I don't know who's had experience with the newly-released version; it might
be best to wait a little bit, until any problems have been shaken out, before
updating the 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 478613] Review Request: ledger - A powerful command-line double-entry accounting system

2010-05-24 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=478613

--- Comment #5 from Michel Alexandre Salim  
2010-05-24 12:35:48 EDT ---
Sorry for the delay, I'll try and post an updated response later today.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

2010-05-24 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=592579

--- Comment #13 from David A. Wheeler  2010-05-24 
12:08:08 EDT ---
I'm looking over the .spec file for overall comments, before I delve into a
point-by-point comparison with the guidelines.

* Every time you make a change you should bump the "Revision" number,
  and post the new URL.  That'll be *critical* when it's submitted to the
  repository, but it'd reduce confusion now too.
* Patch0 comment says:
  "The configure file uses graph.cmo in compiling a test program to
  check for ocamlgraph, but Fedora doesn't include this file in its
  distributions."
  I'm confused, there *is* an ocaml-ocamlgraph package.
  Is the Fedora respository's ocamlgraph the wrong (too old) version?
  Or, is this just an old comment based on an older version of this spec,
  now OBE?
* Patch1 isn't prefixed with "frama-c-", and that can cause trouble.
  In general, if someone looks in their rpmbuild/SOURCES, they can get
  confused (and namespace issues could result) if they aren't prefixed.
  You should  prefix the patch filenames with NAME- (e.g., "frama-c-").
* Rejected Debian Patch2 (0002-Make-Jessie-plugin-use-Jc-from-Why-2.19.patch)
  comment says, "I don't know why we would want to use that
  Jessie -> not including this patch".  You're not including that patch
  because we want to use a later version of Why, correct?  If so, just say:
  "We use a later version of Why, so this patch is irrelevant".
  Although I applaud "I don't know..." comments for their honesty, I think
  we should track them down so you can simply say why something was done.
  In this case, there's a good reason: The patch is for an obsolete
  version of "why".
* Rejected Debian patch "0003-Add-dGraphView.cmo-when-linking.patch"
  says "I'm not sure quite what this patch does, but I'm going to
  leave it out for now".  Need to track this down a little of what this is.
  My *guess* from the text is that it merges graphview into another
  another library, and we *don't* want to do that (because then upgrading
  graphview won't updated the merged file).  If that's the
  situation, please say so directly.
* Rejected Debian patch 0004-Use-GSourceView2.patch comment says
  "I didn't see any Fedora package providing the files that this patch
  references -> dropping for now".  
  Did you check out "gtksourceview2-devel" and "gtksourceview2"?
  It's possible that the patch includes the *OCaml* bindings for these
  and that Fedora doesn't include those bindings.  If so, just say that.
* In Patch "frama-c-1.4-ptests-fix-for-ocaml-3.11.2.patch" there
  needs to be a comment explaining (1) what it's for, and (2)
  when you submitted it upstream (and if not, why not).  Shouldn't be *long*,
  but it should say *something*.
* Nit: Your "Requires:" statements have inconsistent indenting.
  Please fix while you're at it.
* Nit: "plug in's" is not a word.  An "'s" is either a possessive or
  an appreviation of "is", and this is neither (it's just a plural).
  I'd suggest "plug-ins" instead.  If the spellchecker whines, too bad.
* I notice that the entire "%check" section is commented out.
  A %check section is NOT required, and I'd prefer the comments to nothing,
  but it'd be nice if it worked :-).
* As noted earlier, the .byte files don't work, and probably shouldn't
  be packaged at all.
* I'll check the %files list separately.
* In %post, just remove the comment:
  #chcon -t textrel_shlib_t '%{_libdir}/frama-c/plugins/Ltl_to_acsl.cmxs'
  I don't see that the commend adds anything.
* The %changelog doesn't list "Mark Rader"; it should!

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594744] Review Request: zint - A barcode generator and library

2010-05-24 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=594744

--- Comment #3 from Chen Lei  2010-05-24 13:09:02 EDT ---
1.I suggest you to split shlibs into subpackages, e.g. zint-libs zint-qt-libs.
Fedora ships multilibs in x86_64 and ppc64.

2.%package -n zint-devel -> %package devel

3. I'll be better to include .desktop as Source1 in spec


4.%cmake CMakeLists.txt -> %cmake .

5.Group: Applications/Engineering is not appropriate

6.For Source field, see 
http://fedoraproject.org/wiki/Packaging:SourceURL#Sourceforge.net

7.
BuildRequires: qt-devel >= 4.4
BuildRequires: desktop-file-utils

BR should not be added to subpackages.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594943] Review Request: libnoise - A general-purpose library that generates three-dimensional coherent noise

2010-05-24 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=594943

--- Comment #8 from Robin Norwood  2010-05-24 13:06:35 
EDT ---
To be clear, the options I see are:

1) Just let MathMap build and link statically to their patched libnoise - the
packaging guidelines strongly discourage this:
http://fedoraproject.org/wiki/PackagingGuidelines#Packaging_Static_Libraries
2) Package libtool separately with the patch, the route I chose here.  Nasty
because it does change the API.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594943] Review Request: libnoise - A general-purpose library that generates three-dimensional coherent noise

2010-05-24 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=594943

--- Comment #7 from Robin Norwood  2010-05-24 12:56:29 
EDT ---
Hi Martin,

The reason I'm packaging libnoise in the first place is that the new version of
MathMap (1.3.5) requires it.  The upstream version of MathMap ships with a
tarball of libnoise, patches it, builds it, and links against it statically. 
The QUALITY_BESTCASE patch is from MathMap.  I can check to see if MathMap has
submitted the patch upstream or not...otherwise, I'm not really sure how to
proceed if libnoise upstream is not interested in the patch...none of the
options seem very appealing.  Any suggestions?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594943] Review Request: libnoise - A general-purpose library that generates three-dimensional coherent noise

2010-05-24 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=594943

--- Comment #6 from Martin Gieseking  2010-05-24 
12:49:37 EDT ---
Hi Robin,

is there a special reason why you patch in an additional noise quality variant
(QUALITY_BESTCASE)? This extends the interface and would require a bump of the
library's release number. BTW, the version number should consist of 3 integers
(libnoise.so.0.3.0 instead of libnoise.so.0.3 -- the latter can be added as a
symlink, though).
If possible, please ask upstream to fix this. Maybe he can also incorporate
your 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593069] Review Request: ** experimental ** pynie - Implementation of Python 3 on top of the Parrot virtual machine

2010-05-24 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=593069

Dave Malcolm  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DEFERRED

--- Comment #9 from Dave Malcolm  2010-05-24 12:17:04 EDT 
---
Tim: thanks for reviewing this.

Gerd: thanks for your input.

Based on private email and comment #8, it appears that upstream would prefer it
if we held off from shipping this within Fedora until Pynie does an official
release.

So we're going to hold off on this for now.  I'm going to close this review
with the resolution "DEFERRED" until there's an official upstream release of
Pynie.

At that point we can reopen this review.  It may be that we need to update the
specfile somewhat based on upstream changes to the code, so the packaging may
need doublechecking at that point.

Hope this sounds sane.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594844] Review Request: iec16022 - Generate ISO/IEC 16022 2D barcodes

2010-05-24 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=594844

Tom "spot" Callaway  changed:

   What|Removed |Added

 Blocks|182235(FE-Legal)|
   Flag|needinfo?   |

--- Comment #2 from Tom "spot" Callaway  2010-05-24 
12:10:30 EDT ---
I continue to be unaware of any valid patents which would prevent this package
from being included in Fedora. Lifting FE-Legal.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 587438] Review Request: rubygem-snmp - A Ruby implementation of SNMP (the Simple Network Management Protocol)

2010-05-24 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=587438

--- Comment #6 from Michael Stahnke  2010-05-24 11:57:30 
EDT ---
I have fixed the comments unrelated to the test.  I am trying decide if
patching simply for tests is a good idea.  In general, I'd rather not carry
patches against upstream if I can help it.  I can contact upstream about the
tests and see if they can either fix them or explain to me the proper method
for running them.  

Very thorough and good review thus far.  I do appreciate it. 

http://stahnma.fedorapeople.org/reviews/rubygem-snmp-1.0.2-3.fc12.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 594844] Review Request: iec16022 - Generate ISO/IEC 16022 2D barcodes

2010-05-24 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=594844

Ville Skyttä  changed:

   What|Removed |Added

 CC||tcall...@redhat.com,
   ||ville.sky...@iki.fi
 Blocks||182235(FE-Legal)
   Flag||needinfo?

--- Comment #1 from Ville Skyttä  2010-05-24 11:47:22 EDT 
---
spot, do your comments about another Data Matrix related package in bug 561286
also apply to this one?  Adding blocker on FE-legal until you've had a chance
to take a look.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 594943] Review Request: libnoise - A general-purpose library that generates three-dimensional coherent noise

2010-05-24 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=594943

--- Comment #5 from Robin Norwood  2010-05-24 11:25:44 
EDT ---
Hi Martin,

I've implemented all of those suggestions:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2205980

http://www.norwoodfamily.info/rpms/libnoise.spec
http://www.norwoodfamily.info/rpms/libnoise-1.0.0-3.fc12.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 586433] Review Request: Xnoise, a media player for gtk+ written in vala

2010-05-24 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=586433

Chen Lei  changed:

   What|Removed |Added

 CC||supercyp...@gmail.com

--- Comment #9 from Chen Lei  2010-05-24 11:07:27 EDT ---
Some more suggestions:

The tarball bundles two seperate projects - xnoise and xnoise-plugins-core.
It may not be appropriate.

It's seems 0.1.2 is still a prerelease, you should modify you release field to
match naming guideline.
See
http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre-Release_packages

I highly suggest you to preprocess vala code to pure C in spec and not ship
preprocessed code in you source tarball.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 593069] Review Request: ** experimental ** pynie - Implementation of Python 3 on top of the Parrot virtual machine

2010-05-24 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=593069

Gerd Pokorra  changed:

   What|Removed |Added

Summary|Review Request: pynie - |Review Request: **
   |Implementation of Python 3  |experimental ** pynie -
   |on top of the Parrot|Implementation of Python 3
   |virtual machine |on top of the Parrot
   ||virtual machine

--- Comment #8 from Gerd Pokorra  2010-05-24 10:22:53 
EDT ---
An official release of Pynie will appear in a few months. It is better to wait
for it to not having needless trouble with broken dependences. It is
explained in comment 2.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 595011] Review Request:

2010-05-24 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=595011

--- Comment #8 from Rahul Sundaram  2010-05-24 09:35:48 EDT 
---

Either you got to modify whatever is needed to the best extend possible in the
package itself or document it in something like README.Fedora.  Post-install
echo is not sufficient for anyone not using yum on the command line.  This
include users of PackageKit, Anaconda or kickstart

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 595011] Review Request:

2010-05-24 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=595011

--- Comment #7 from David Highley  2010-05-24 
09:29:21 EDT ---
I agree with most of the changes, but still think the post install should let
the user know that they need to modify the iptables and rsyslog.conf files. If
they do not get some kind of prompt then they maybe wondering how to get the
package working. I can see not modifying them, some thing we do at work as we
have to fix things so they install and uninstall cleanly. With most packages
not requiring any user changes they are caught off guard when one is not able
to automatically make all the changes needed.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 582106] Review Request: lifeograph - Private digital diary.

2010-05-24 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=582106

Gowrishankar Rajaiyan  changed:

   What|Removed |Added

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

--- Comment #5 from Gowrishankar Rajaiyan  2010-05-24 
08:49:18 EDT ---
Hi Mamoru,

I am very sorry for the delay.
New spec and srpm has been uploaded to the following location:

Spec URL: http://gowrishankar.fedorapeople.org/spec/lifeograph.spec
SRPM URL:
http://gowrishankar.fedorapeople.org/srpms/lifeograph-0.6.2-1.fc12.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 590244] Review Request: pinta - Simple Paint Application

2010-05-24 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=590244

--- Comment #3 from Andrea Musuruane  2010-05-24 08:22:47 
EDT ---
Ping.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 458402] Review Request: griffith - Media collection manager

2010-05-24 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=458402

--- Comment #11 from Fedora Update System  
2010-05-24 08:09:19 EDT ---
uncrustify-0.56-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/uncrustify-0.56-2.fc11

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


  1   2   >