[Bug 760594] Review Request: simcrs - C++ Simulated Travel-Oriented Distribution System Library

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Denis Arnaud  2012-03-02 
02:56:46 EST ---
Thanks for the review, Matthias!

Of course, the spec file name will change to be simcrs.spec (in the Fedora Git
repository). In fact, I use a symbolic link (from simcrs.spec to
simcrs-0.1.1-1.spec) and it is enough for rpmlint to be happy when hacking with
rpmbuild.
I generally prefer stating explicitly the version number in the spec file name
when located on my FedoraPeople's account, as it is easier to spot the right
file to use.
[In the future, I will use my GitHub account, and therefore use Git to version
the spec file]

Then, you are of course right, the source tar-ball mismatch is not good. I have
forgotten to update the tar-ball on FedoraPeople with a latest update. Sorry.

I will deliver a new version this week-end.

Thanks again

Denis

-- 
Configure 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 799023] Review Request: eot-utils - Tools to convert OTF/TTF to EOT font format + show its metadata

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

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

--- Comment #1 from Parag AN(पराग)  2012-03-02 03:44:13 EST 
---
Review:
- koji build failed (f17)
->http://koji.fedoraproject.org/koji/taskinfo?taskID=3845922

+ source verified with upstream as (sha1sum)
7e8a68ba1ae4b533113e7965aa2cca133367f31f  eot-utilities-1.1.tar.gz
7e8a68ba1ae4b533113e7965aa2cca133367f31f  ../SOURCES/eot-utilities-1.1.tar.gz


Suggestions:
1)  you need to remove autoreconf from %build as this will pass the scratch
build on koji.

2) install section change from
rm -rf -- $RPM_BUILD_ROOT
make install-strip DESTDIR=$RPM_BUILD_ROOT

to
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"

-- 
Configure 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 798891] Review Request: libtranslit - transliteration library with multiple backends

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

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

--- Comment #1 from Parag AN(पराग)  2012-03-02 03:58:06 EST 
---
Review:
+ koji build (f17) ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3845925

+ rpmlint on rpms gave
libtranslit.x86_64: W: spelling-error Summary(en_US) backends -> back ends,
back-ends, backhands
libtranslit.x86_64: W: spelling-error %description -l en_US backends -> back
ends, back-ends, backhands
libtranslit.x86_64: W: no-documentation
libtranslit.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/libtranslit/modules/libtranslitm17n.so
libtranslit.src: W: spelling-error Summary(en_US) backends -> back ends,
back-ends, backhands
libtranslit.src: W: spelling-error %description -l en_US backends -> back ends,
back-ends, backhands
2 packages and 0 specfiles checked; 0 errors, 6 warnings.

+ source verified with upstream as (sha1sum)
ea9dbe0b4d2bd9495394df307b385686eaffa7a7  ../SOURCES/libtranslit-0.0.1.tar.gz
ea9dbe0b4d2bd9495394df307b385686eaffa7a7  libtranslit-0.0.1.tar.gz

+ package follows packaging guidelines.

APPROVED.

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

[Bug 798891] Review Request: libtranslit - transliteration library with multiple backends

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Daiki Ueno  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Daiki Ueno  2012-03-02 04:05:23 EST ---
Thanks for the review, Parag.

New Package SCM Request
===
Package Name: libtranslit
Short Description: transliteration library with multiple backends
Owners: ueno
Branches: f17 f16 f15
InitialCC: i18n-team

-- 
Configure 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 750911] Review Request: moconti - Web Application Server for Sleep

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Tomas Radej  changed:

   What|Removed |Added

 AssignedTo|tra...@redhat.com   |nob...@fedoraproject.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 794725] Review Request: txw2 - Typed XML writer for Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Juan Hernández  2012-03-02 
04:43:02 EST ---
Fixed issue #1 and added comments for the patches.

The new spec and SRPM are available here:

http://jhernand.fedorapeople.org/rpms/txw2/20110809-3

An scratch build is available here:

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

-- 
Configure 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 796201] Review Request: glassfish-jaxb - JAXB Reference Implementation

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Juan Hernández  2012-03-02 
04:50:22 EST ---
The spec needed some minor updates after changing the names of the jar files in
txw2. The new spec and SRPM are available here:

http://jhernand.fedorapeople.org/rpms/glassfish-jaxb/2.2.4u1-3

-- 
Configure 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 790564] Review Request: glassfish-fi - Fast Infoset

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Juan Hernández  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |glassfish-fi -  |glassfish-fi - Fast Infoset
   |Development/Libraries   |

-- 
Configure 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 798657] Review Request: perl-Test-Moose-More - More tools for testing Moose packages

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-Test-Moose-More-0.005-
   ||1.fc18
 Resolution||RAWHIDE
Last Closed||2012-03-02 04:51:56

--- Comment #4 from Petr Pisar  2012-03-02 04:51:56 EST ---
Thank you for the review and the repository.

-- 
Configure 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 798741] Review Request: perl-WebService-Rajce - Perl interface for www.rajce.idnes.cz

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
   Fixed In Version||perl-WebService-Rajce-0.07-
   ||1.fc18

--- Comment #4 from Petr Pisar  2012-03-02 04:59:02 EST ---
I corrected the descriptions.

Thank you for the review and the reporitory.

-- 
Configure 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 798741] Review Request: perl-WebService-Rajce - Perl interface for www.rajce.idnes.cz

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Fedora Update System  2012-03-02 
05:05:20 EST ---
perl-WebService-Rajce-0.07-1.fc16 has been submitted as an update for Fedora
16.
https://admin.fedoraproject.org/updates/perl-WebService-Rajce-0.07-1.fc16

-- 
Configure 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 798741] Review Request: perl-WebService-Rajce - Perl interface for www.rajce.idnes.cz

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Fedora Update System  2012-03-02 
05:03:37 EST ---
perl-WebService-Rajce-0.07-1.fc17 has been submitted as an update for Fedora
17.
https://admin.fedoraproject.org/updates/perl-WebService-Rajce-0.07-1.fc17

-- 
Configure 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 510962] Review Request: pypoppler - Python bindings for the Poppler PDF rendering library

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fabian Affolter  changed:

   What|Removed |Added

 Blocks||794970
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #15 from Fabian Affolter  2012-03-02 
05:31:53 EST ---
Package Change Request
==
Package Name: pypoppler
New Branches: el6
Owners: fab

-- 
Configure 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 606421] Review Request: quickfix - Open Source FIX Engine

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #21 from Martin Dengler  2012-03-02 
05:33:46 EST ---
I've updated the package for F16, which builds fine (
http://koji.fedoraproject.org/koji/taskinfo?taskID=3846127 ) ...I've uploaded
the SRPM and patches to
http://www.martindengler.com/proj/quickfix/build-20120210 and
http://www.martindengler.com/proj/quickfix/build-20120210 , respectively, in
case anyone wants to try it out.

I am going to rip out the included unittest code this weekend and hopefully
that will finish this off.  I will clear the whiteboard.  If there is anything
else people know they would like me to see for the review then please let me
know.

-- 
Configure 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 788630] Review Request: plexus-pom - Root Plexus Projects pom

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Stanislav Ochotnicky  2012-03-02 
05:48:34 EST ---
Please remember to close these bugs after the builds (or if you are planning on
putting it to F16+, add them as bugs to bodhi updates so they get closed
automatically)

-- 
Configure 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 799171] Review Request: openvswitch - multi-layer software switch (userspace components)

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Daniel Berrange  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||berra...@redhat.com
 AssignedTo|nob...@fedoraproject.org|berra...@redhat.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 799171] Review Request: openvswitch - multi-layer software switch (userspace components)

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Daniel Berrange  2012-03-02 06:16:22 
EST ---
PASS MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.[1]

openvswitch.src: W: spelling-error Summary(en_US) vSwitch -> v Switch, switch,
vs witch
openvswitch.src: W: spelling-error %description -l en_US vSwitch -> v Switch,
switch, vs witch
openvswitch.x86_64: W: spelling-error Summary(en_US) vSwitch -> v Switch,
switch, vs witch
openvswitch.x86_64: W: spelling-error %description -l en_US vSwitch -> v
Switch, switch, vs witch
openvswitch.x86_64: W: only-non-binary-in-usr-lib
python-openvswitch.noarch: W: spelling-error Summary(en_US) vSwitch -> v
Switch, switch, vs witch
python-openvswitch.noarch: W: spelling-error %description -l en_US vSwitch -> v
Switch, switch, vs witch
ovsdbmonitor.noarch: W: spelling-error Summary(en_US) vSwitch -> v Switch,
switch, vs witch
ovsdbmonitor.noarch: W: spelling-error %description -l en_US vSwitch -> v
Switch, switch, vs witch
ovsdbmonitor.noarch: W: spelling-error %description -l en_US ovs -> obs, ova,
vs
ovsdbmonitor.noarch: W: spelling-error %description -l en_US dpctl -> pct
ovsdbmonitor.noarch: W: spelling-error %description -l en_US vsctl -> scuttle
openvswitch-test.noarch: W: spelling-error Summary(en_US) vSwitch -> v Switch,
switch, vs witch
openvswitch-test.noarch: W: spelling-error %description -l en_US vSwitch -> v
Switch, switch, vs witch
5 packages and 0 specfiles checked; 0 errors, 14 warnings.



PASS MUST: The package must be named according to the Package Naming Guidelines
.
PASS MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption. [2] .
PASS MUST: The package must meet the Packaging Guidelines .
PASS MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .

FAIL MUST: The License field in the package spec file must match the actual
license. [3]

 The spec says

# Nearly all of openvswitch is ASL 2.0.  The bugtool is LGPLv2+, and the
# lib/sflow*.[ch] files are SISSL
License:ASL 2.0 and LGPLv2+ and SISSL

 This is not quite the whole story 

python/compat/uuid.py : PYTHON SOFTWARE FOUNDATION LICENSE VERSION 2
datapath/linux/compat/flex_array.c: GPLv2+
datapath/dp_notify.c: GPLv2-only  (and more files)

  There are a number of files which have no license header, which upstream
should be encouraged to fix

  I browsed license headers with:

$ find -name *.c | xargs head -30 | less
$ find -name *.py | xargs head -30 | less


PASS 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]

PASS MUST: The spec file must be written in American English. [5]
PASS MUST: The spec file for the package MUST be legible. [6]
PASS 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.

3847c60af329bfe81ff7220b9f489fa5  openvswitch-1.4.0.tar.gz
3847c60af329bfe81ff7220b9f489fa5  rpmbuild/SOURCES/openvswitch-1.4.0.tar.gz


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

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

N/A 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]

PASS 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.

N/A MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.[9]
N/A MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun. [10]
PASS MUST: Packages must NOT bundle copies of system libraries.[11]
N/A MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered

[Bug 799284] New: Review Request: perl-Pod-Plainer - Perl extension for converting Pod to old-style Pod

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-Pod-Plainer - Perl extension for converting Pod 
to old-style Pod

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

   Summary: Review Request: perl-Pod-Plainer - Perl extension for
converting Pod to old-style Pod
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: xn...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: https://dl.dropbox.com/u/64704553/perl-pod-plainer.spec
SRPM URL:
https://dl.dropbox.com/u/64704553/perl-Pod-Plainer-1.03-1.fc16.src.rpm
Description: 
Pod::Plainer uses Pod::Parser which takes Pod with the (new) 'C<< .. >>'
constructs and returns the old(er) style with just 'C<>'; '<' and '>' are
replaced by 'E' and 'E'.
This can be used to pre-process Pod before using tools which do not
recognise the new style Pods.
This my second package. But I also need a sponsor. Thanks a lot.
This package is required to support LSB 4.1

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

[Bug 799023] Review Request: eot-utils - Tools to convert OTF/TTF to EOT font format + show its metadata

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Jan Pokorny  2012-03-02 06:50:07 EST 
---
(In reply to comment #1)

> 1)  you need to remove autoreconf from %build as this will pass
> the scratch build on koji.

Agreed, I've just found [1] and it works well for me without it in
the mock environment.

(Otherwise, these BuildRequires would be needed: autoconf, automake, m4.)

> 2) install section change from
> rm -rf -- $RPM_BUILD_ROOT
> make install-strip DESTDIR=$RPM_BUILD_ROOT
> to
> make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"

The removal command comes from rpmdev-newspec.  If this is not needed
or it serves only to ensure everything is rebuilt properly when building
locally, bug against rpmdevtools should be filed.

Re: INSTALL="install -p" ... agreed.

Re: install-strip -> install:
I got it, it is a matter of postprocessing that will create separate
debuginfo package and actually strip the binaries.  (Interestingly,
"install-strip" will not do the stripping in this case either.)

[1] http://stackoverflow.com/a/7754416

-- 
Configure 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 784591] Review Request: glazedlists - A Java toolkit for transformations in Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mary Ellen Foster  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Mary Ellen Foster  2012-03-02 07:16:31 
EST ---
New Package SCM Request
===
Package Name:  glazedlists
Short Description: A toolkit for transformations in Java
Owners:mef
Branches:  f17

-- 
Configure 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 510962] Review Request: pypoppler - Python bindings for the Poppler PDF rendering library

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #16 from Jon Ciesla  2012-03-02 07:56:03 EST 
---
Git done (by process-git-requests).

-- 
Configure 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 797694] Review Request: Farstream - Libraries for videoconferencing

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Jon Ciesla  2012-03-02 07:59:13 EST 
---
That was part of it.  It still thinks you want 'InitialCC' as a branch, but I
can work around
that.  My larger concern is that the name in the SCM request and summary
don't match, should it be capitalized or not?

-- 
Configure 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 798891] Review Request: libtranslit - transliteration library with multiple backends

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Jon Ciesla  2012-03-02 07:59:51 EST 
---
Git done (by process-git-requests).

-- 
Configure 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 784591] Review Request: glazedlists - A Java toolkit for transformations in Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Jon Ciesla  2012-03-02 07:56:44 EST 
---
Git done (by process-git-requests).

-- 
Configure 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 551765] Review Request: prosody - Flexible communications server for Jabber/XMPP

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 551765 depends on bug 551763, which changed state.

Bug 551763 Summary: Review Request: lua-sec - Lua binding for OpenSSL library
https://bugzilla.redhat.com/show_bug.cgi?id=551763

   What|Old Value   |New Value

 Resolution|WONTFIX |
 Status|CLOSED  |ASSIGNED

-- 
Configure 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 551763] Review Request: lua-sec - Lua binding for OpenSSL library

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matěj Cepl  changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|WONTFIX |
 AssignedTo|a...@spicenitz.org  |mc...@redhat.com
   Flag||fedora-review?
   Keywords||Reopened

--- Comment #19 from Matěj Cepl  2012-03-02 08:01:06 EST ---
In view of comment 18, reopening this bug and taking over 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 738744] Review Request: rubygem-execjs - A package for the execjs Ruby gem

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Vít Ondruch  2012-03-02 08:01:57 EST 
---
* F17/Rawhide
  - I see that you are already using ruby(abi) = 1.9.1, i.e. you are building
for Ruby 1.9.3 in F17/Rawhide, in that case, you should follow the new
guidelines [1]. BTW I placed .spec file updated according to the new
guidelines here [2].

* Tests do not run
  - There is reported "test/test_execjs.rb: cannot load such file --
multi_json"
error. It should be fixed by adding:

  BuildRequires: rubygem(multi_json) => 1.0
  BuildRequires: rubygem(multi_json) < 2
  - After adding the BR, it fails anyway with message: "Could not find a
JavaScript runtime. See https://github.com/sstephenson/execjs for a list
of available runtimes." So there should be BR some supported JS engine
(spidermonkey?) or better all engines available in Fedora.





[1] https://fedoraproject.org/wiki/PackagingDrafts/Ruby
[2] http://people.redhat.com/vondruch/rubygem-execjs.spec

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

[Bug 799023] Review Request: eot-utils - Tools to convert OTF/TTF to EOT font format + show its metadata

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Jan Pokorny  2012-03-02 08:12:16 EST 
---
Spec and SRPM available at the same URLs (the old version is in "old-v1").
I've also returned "meta-data" back to "metadata" (as in Summary)
and use the form of RPM macros consistently.

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3846459

-- 
Configure 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 608852] Review Request: epris - a dbus service to listen to music

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #36 from Jon Ciesla  2012-03-02 09:08:43 EST 
---
Login to bodhi, https://admin.fedoraproject.org/updates/, and mark each update
as stable.

-- 
Configure 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 784591] Review Request: glazedlists - A Java toolkit for transformations in Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure 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 784591] Review Request: glazedlists - A Java toolkit for transformations in Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Fedora Update System  
2012-03-02 09:10:43 EST ---
glazedlists-1.8.0-3.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/glazedlists-1.8.0-3.fc17

-- 
Configure 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 551763] Review Request: lua-sec - Lua binding for OpenSSL library

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #20 from Matěj Cepl  2012-03-02 09:46:51 EST ---
Legend: + = PASSED, - = FAILED, 0 = Not Applicable

+ MUST: rpmlint must be run on every package. The output should be posted in
the review

$ rpmlint -i *.rpm
3 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ 

+ MUST: package named according to the Package Naming Guidelines
changed from luasec to lua-sec to follow
https://fedoraproject.org/wiki/PackagingDrafts/Lua
+ MUST: The spec file name must match the base package %{name}
- MUST: The package must meet the Packaging Guidelines .
Per above mentioned Lua Packaging Guidelines spec file should contain

%if 0%{?fedora} >= 16 || 0%{?rhel} >= 7
Requires: lua(abi) = %{luaver}
%else
Requires: lua >= %{luaver}
%endif

+ MUST: The package licensed with a Fedora approved license and meets the
Licensing Guidelines
+ MUST: The License field in the package spec file matches the actual
license
MIT
+ 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.
LICENSE is included.
+ MUST: The spec file must be written in American English.
+ MUST: The spec file for the package MUST be legible.
+ MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task
MD5: 712158d60207bdbb6215fc7e07d8db24
+ MUST: The package successfully compiles and builds into binary rpms on at
least one primary architecture - build in koji, no problems
0 MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch
+ MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines
Builds in koji (http://koji.fedoraproject.org/koji/taskinfo?taskID=3846510)
0 MUST: The spec file handles locales properly. This is done by using the
%find_lang macro
No locales are present.
0 MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun.
No libraries provided.
+ MUST: Packages must NOT bundle copies of system libraries
0 MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker
- MUST: 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

Missing explicit requirement of lua package (which owns %{luapkgdir} used by
package).

+ MUST: Package must not list a file more than once in the spec file's %files
listings
+ MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
+ MUST: Each package must consistently use macros
+ MUST: The package must contain code, or permissible content
0 MUST: Large documentation files must go in a -doc subpackage
+ MUST: If a package includes something as %doc, it must not affect the runtime
of the application
0 MUST: Header files must be in a -devel package
0 MUST: Static libraries must be in a -static package
0 MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
0 MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel package
0 MUST: devel packages must require the base package using a fully versioned
dependency: Requires: %{name} = %{version}-%{release}
+ MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built
0 MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section
+ MUST: Packages must not own files or directories already owned by other
packages
- MUST: At the beginning of %install, each package MUST run rm -rf %{buildroot}
(or $RPM_BUILD_ROOT)
+ MUST: All filenames in rpm packages must be valid UTF-8

Just a nitpicks:
- please fix lua requirement as shown above.

-- 
Configure 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 794725] Review Request: txw2 - Typed XML writer for Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Andy Grimm  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Andy Grimm  2012-03-02 10:06:26 EST ---
Looks good.

APPROVED.

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

[Bug 798654] Review Request: cmpi-bindings - CMPI-compliant provider interface for various languages via SWIG

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jan Safranek  changed:

   What|Removed |Added

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

--- Comment #1 from Jan Safranek  2012-03-02 10:24:46 EST 
---
Rpmlint output:
cmpi-bindings.src: W: invalid-url Source0: cmpi-bindings-0.4.17.tar.bz2
This looks fine, upstream does not have a website.

cmpi-bindings-pywbem.x86_64: W: no-documentation
At least upstream README and LICENSE should be there. I'd appreciate also
some README.Fedora which would specify, where python providers are expected.

The package should create and own /usr/lib/pythonX.Y/site-packages/pycim
directory.

Otherwise, all MUST review items are OK.

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

[Bug 784592] Review Request: jpf - Java Plug-in Framework

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Alexander Kurtakov  changed:

   What|Removed |Added

 Blocks|652183(FE-JAVASIG)  |

-- 
Configure 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 799171] Review Request: openvswitch - multi-layer software switch (userspace components)

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Daniel Berrange  changed:

   What|Removed |Added

   Flag||fedora-review?

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

[Bug 794725] Review Request: txw2 - Typed XML writer for Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Juan Hernández  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Juan Hernández  2012-03-02 
10:46:02 EST ---
New Package SCM Request
===
Package Name: txw2
Short Description: Typed XML writer for Java
Owners: jhernand
Branches: f17
InitialCC: goldmann

-- 
Configure 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 799171] Review Request: openvswitch - multi-layer software switch (userspace components)

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Chris Wright  2012-03-02 10:49:33 EST ---
(In reply to comment #1)
> PASS MUST: The package must be licensed with a Fedora approved license and 
> meet
> the Licensing Guidelines .
> 
> FAIL MUST: The License field in the package spec file must match the actual
> license. [3]
> 
>  The spec says
> 
> # Nearly all of openvswitch is ASL 2.0.  The bugtool is LGPLv2+, and the
> # lib/sflow*.[ch] files are SISSL
> License:ASL 2.0 and LGPLv2+ and SISSL
> 
>  This is not quite the whole story 
> 
> python/compat/uuid.py : PYTHON SOFTWARE FOUNDATION LICENSE VERSION 2
> datapath/linux/compat/flex_array.c: GPLv2+
> datapath/dp_notify.c: GPLv2-only  (and more files)

Yes, thanks.  I had completely put those out of my mind as they are not shpped
w/ the binary package.  Only in the source.

>   There are a number of files which have no license header, which upstream
> should be encouraged to fix
> 
>   I browsed license headers with:
> 
> $ find -name *.c | xargs head -30 | less
> $ find -name *.py | xargs head -30 | less

I'll push some patches upstream.


> FAIL MUST: Each package must consistently use macros. [16]
> 
> In the %files section there are alot of hardcoded paths, eg /usr/bin, /etc,
> /usr/share, etc. Replace these with %{bindir}, %{datadir}, etc
> 
> Also avoid hardcoding  '.gz' on the end of man page paths, eg replace
> 
>%{_datadir}/man/man1/ovs-benchmark.1.gz
> 
> with
> 
>%{_datadir}/man/man1/ovs-benchmark.1*

For some reason I took that to simply mean consistent use of RPM_BUILD_ROOT vs
%{buildroot}.  Easy to fix, thanks again.

I'll respin and update shortly.

-- 
Configure 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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Dennis Gilmore  changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |

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

[Bug 794725] Review Request: txw2 - Typed XML writer for Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jon Ciesla  2012-03-02 11:16:45 EST 
---
Git done (by process-git-requests).

-- 
Configure 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 790172] Review Request: jinput - Java Game Controller API

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure 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 790172] Review Request: jinput - Java Game Controller API

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Fedora Update System  2012-03-02 
11:20:42 EST ---
jinput-2.0.6-1.20110801svn.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/jinput-2.0.6-1.20110801svn.fc16

-- 
Configure 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 790172] Review Request: jinput - Java Game Controller API

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Fedora Update System  2012-03-02 
11:20:32 EST ---
jinput-2.0.6-1.20110801svn.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/jinput-2.0.6-1.20110801svn.fc17

-- 
Configure 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 799392] New: Review Request: geronimo-commonj - CommonJ Specification

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: geronimo-commonj - CommonJ Specification

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

   Summary: Review Request: geronimo-commonj - CommonJ
Specification
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: juan.hernan...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, agr...@gmail.com,
package-review@lists.fedoraproject.org
Blocks: 652183
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL:

http://jhernand.fedorapeople.org/rpms/geronimo-commonj/1.1.0-2/geronimo-commonj.spec

SRPM URL:

http://jhernand.fedorapeople.org/rpms/geronimo-commonj/1.1.0-2/geronimo-commonj-1.1.0-2.fc17.src.rpm

Koji build:

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

Description:

Geronimo CommonJ Specification.

-- 
Configure 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 796201] Review Request: glassfish-jaxb - JAXB Reference Implementation

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 796201 depends on bug 794725, which changed state.

Bug 794725 Summary: Review Request: txw2 - Typed XML writer for Java
https://bugzilla.redhat.com/show_bug.cgi?id=794725

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|ASSIGNED|CLOSED

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

[Bug 767050] Review Request: hibernate-validator - Bean Validation (JSR 303) Reference Implementation

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 767050 depends on bug 794725, which changed state.

Bug 794725 Summary: Review Request: txw2 - Typed XML writer for Java
https://bugzilla.redhat.com/show_bug.cgi?id=794725

   What|Old Value   |New Value

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

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

[Bug 794725] Review Request: txw2 - Typed XML writer for Java

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Juan Hernández  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Blocks|652183(FE-JAVASIG)  |
 Resolution||RAWHIDE
Last Closed||2012-03-02 11:39:22

-- 
Configure 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 723752] Review Request: lrslib - Reverse search for vertex enumeration/convex hull problems

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure 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 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Dan Williams  changed:

   What|Removed |Added

   Flag||needinfo?(jamie@tomoyolinux
   ||.co.uk)

-- 
Configure 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 723752] Review Request: lrslib - Reverse search for vertex enumeration/convex hull problems

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #18 from Fedora Update System  
2012-03-02 11:46:43 EST ---
lrslib-4.2c-3.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/lrslib-4.2c-3.fc16

-- 
Configure 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 723752] Review Request: lrslib - Reverse search for vertex enumeration/convex hull problems

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #17 from Fedora Update System  
2012-03-02 11:46:35 EST ---
lrslib-4.2c-3.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/lrslib-4.2c-3.fc17

-- 
Configure 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 733603] Review Request: sugar-ruler - Ruler is a simple collection of measurement tools that are displayed on the screen.

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Kalpa Welivitigoda  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #24 from Kalpa Welivitigoda  2012-03-02 
11:47:08 EST ---
Package Change Request
==
Package Name: sugar-ruler
New Branches: f17
Owners: callkalpa
InitialCC: pbrobinson

-- 
Configure 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 733603] Review Request: sugar-ruler - Ruler is a simple collection of measurement tools that are displayed on the screen.

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Robinson  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #25 from Peter Robinson  2012-03-02 11:53:56 
EST ---
(In reply to comment #24)
> Package Change Request
> ==
> Package Name: sugar-ruler
> New Branches: f17
> Owners: callkalpa
> InitialCC: pbrobinson

The branch is there, you just need to do a "git pull" to ensure it's pulled
down locally. A "fedpkg switch-branch f17" should then work

-- 
Configure 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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Larry Letelier  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|g...@xofica.cl
   Flag||fedora-cvs?

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

[Bug 750898] Review Request: icaro - A robot automation language

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Larry Letelier  changed:

   What|Removed |Added

   Flag||fedora-cvs?

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

[Bug 750898] Review Request: icaro - A robot automation language

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Larry Letelier  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|g...@xofica.cl

-- 
Configure 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 750898] Review Request: icaro - A robot automation language

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Larry Letelier  changed:

   What|Removed |Added

   Flag|fedora-cvs? |

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

[Bug 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #31 from Jon Ciesla  2012-03-02 12:21:46 EST 
---
Please include only valid FAS account names in InitialCC.  gomix, please set
fedora-review flag to +.

-- 
Configure 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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #32 from Larry Letelier  2012-03-02 12:31:02 EST ---
New Package SCM Request
===
Package Name: i4uc
Short Description: Multiple platform IDE for developing micro-controller
firmware
Owners: lletelier
Branches: f15 f16 f17
InitialCC: Lletelier

-- 
Configure 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 786602] Review Request: picketbox-commons - PicketBox Commons

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Ricardo Arguello  2012-03-02 
12:30:31 EST ---
The picketbox-commons package was updated:

Spec URL:
http://ricardo.fedorapeople.org/package_review/picketbox-commons/3/picketbox-commons.spec

SRPM URL:
http://ricardo.fedorapeople.org/package_review/picketbox-commons/3/picketbox-commons-1.0.0-2.fc17.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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Larry Letelier  changed:

   What|Removed |Added

   Flag||fedora-cvs?

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

[Bug 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #33 from Jon Ciesla  2012-03-02 12:37:43 EST 
---
FAS usernames are case-sensitive, wait for review to be + before resetting
cvs flag.

-- 
Configure 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 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jamie Nguyen  changed:

   What|Removed |Added

   Flag|needinfo?(jamie@tomoyolinux |
   |.co.uk) |

--- Comment #4 from Jamie Nguyen  2012-03-02 12:43:07 
EST ---
Hmm I couldn't build:

$ mock --rebuild -r fedora-17-x86_64
network-manager-applet-0.9.2-2.fc17.src.rpm

Config(fedora-17-x86_64) 0 minutes 17 seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-17-x86_64/result
ERROR: Command failed: 
 # ['/usr/bin/yum-builddep', '--installroot',
'/var/lib/mock/fedora-17-x86_64/root/',
'/var/lib/mock/fedora-17-x86_64/root///builddir/build/SRPMS/network-manager-applet-0.9.2-2.fc17.src.rpm']
Getting requirements for network-manager-applet-0.9.2-2.fc17.src
Error: No Package found for NetworkManager-devel = 1:0.9.2-2



Strangely, bumping nm_version to "1:0.9.3-0.2.git20120215" still causes the
same No Package error and I'm not sure why.

-- 
Configure 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 750898] Review Request: icaro - A robot automation language

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|g...@xofica.cl  |lemen...@gmail.com
   Flag|fedora-review?  |

--- Comment #2 from Peter Lemenkov  2012-03-02 12:45:55 EST 
---
I'll review it (and will sponsor you).

-- 
Configure 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 750898] Review Request: icaro - A robot automation language

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

   Flag||fedora-review?

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

[Bug 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #34 from Larry Letelier  2012-03-02 12:47:23 EST ---
New Package SCM Request
===
Package Name: i4uc
Short Description: Multiple platform IDE for developing micro-controller
firmware
Owners: lletelier
Branches: f15 f16 f17
InitialCC: lletelier

-- 
Configure 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 750898] Review Request: icaro - A robot automation language

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Larry Letelier  2012-03-02 12:54:25 EST ---
Thanks Peter,

-- LL

-- 
Configure 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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Dennis Gilmore  changed:

   What|Removed |Added

   Flag||fedora-cvs?

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

[Bug 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Dennis Gilmore  changed:

   What|Removed |Added

   Flag||fedora-review+

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

[Bug 717502] Review Request: i4uc - IDE for developing micro-controllers firmware

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #35 from Jon Ciesla  2012-03-02 13:08:28 EST 
---
Git done (by process-git-requests).

-- 
Configure 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 797694] Review Request: Farstream - Libraries for videoconferencing

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Brian Pepple  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from Brian Pepple  2012-03-02 13:13:29 EST 
---
Lowercase as the tarball & specfile is. Upstream refers to it capitalized in
the titles of e-mails (1). Hence, I did the same in the summary. Anyway, the
information in the request is how it should be set-up.

New Package SCM Request
===
Package Name: farstream
Short Description: Libraries for videoconferencing
Owners: bpepple
Branches: f17
InitialCC:


(1)
http://lists.freedesktop.org/archives/farstream-devel/2012-February/02.html

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

[Bug 797694] Review Request: Farstream - Libraries for videoconferencing

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Jon Ciesla  2012-03-02 13:28:56 EST 
---
Git done (by process-git-requests).

Gotcha, 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 783837] Review Request: php-pecl-http - Extended HTTP support

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure 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 783837] Review Request: php-pecl-http - Extended HTTP support

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Fedora Update System  
2012-03-02 13:27:35 EST ---
php-pecl-http-2.0.0-0.5.dev6.fc17 has been submitted as an update for Fedora
17.
https://admin.fedoraproject.org/updates/php-pecl-http-2.0.0-0.5.dev6.fc17

-- 
Configure 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 786676] Review Request: picketbox-xacml - PicketBox XACML

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Ricardo Arguello  2012-03-02 
13:39:08 EST ---
Updated picketbox-xacml:

Spec URL:
http://ricardo.fedorapeople.org/package_review/picketbox-xacml/3/picketbox-xacml.spec

SRPM URL:
http://ricardo.fedorapeople.org/package_review/picketbox-xacml/3/picketbox-xacml-2.0.7-1.fc17.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 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Clasen  changed:

   What|Removed |Added

 CC||mcla...@redhat.com

--- Comment #5 from Matthias Clasen  2012-03-02 14:39:38 
EST ---
To get the package to build in mock, I had to change the gnome-keyring-devel BR
to libgnome-keyring-devel, and add a BR for polkit-devel. I also had to add a
patch for missing libraries in the gconf test utilities. 


$ rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/*.rpm
libnm-gtk.x86_64: W: spelling-error %description -l en_US nm -> NM, mm, n
libnm-gtk.x86_64: W: obsolete-not-provided NetworkManager-gtk
libnm-gtk.x86_64: W: no-documentation
libnm-gtk-devel.x86_64: W: no-dependency-on
libnm-gtk/libnm-gtk-libs/liblibnm-gtk
libnm-gtk-devel.x86_64: W: spelling-error %description -l en_US config -> con
fig, con-fig, configure
libnm-gtk-devel.x86_64: W: spelling-error %description -l en_US nm -> NM, mm, n
libnm-gtk-devel.x86_64: W: no-documentation
network-manager-applet.src: W: strange-permission
network-manager-applet-0.9.2.0.tar.bz2 0444L
network-manager-applet.x86_64: W: obsolete-not-provided NetworkManager-gnome
network-manager-applet.x86_64: E: zero-length
/usr/share/doc/network-manager-applet-0.9.2/AUTHORS
network-manager-applet.x86_64: W: non-conffile-in-etc
/etc/gconf/schemas/nm-applet.schemas
network-manager-applet.x86_64: E: zero-length
/usr/share/doc/network-manager-applet-0.9.2/README
network-manager-applet.x86_64: W: non-conffile-in-etc
/etc/xdg/autostart/nm-applet.desktop
network-manager-applet.x86_64: W: no-manual-page-for-binary nm-applet
nm-connection-editor.x86_64: W: no-documentation
nm-connection-editor.x86_64: W: no-manual-page-for-binary nm-connection-editor
6 packages and 0 specfiles checked; 2 errors, 14 warnings.


Looks all ignorable, except for the 0length files, which you should just drop
from the file list.

-- 
Configure 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 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Matthias Clasen  2012-03-02 14:40:47 
EST ---
Created attachment 567150
  --> https://bugzilla.redhat.com/attachment.cgi?id=567150
lib patch

-- 
Configure 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 672205] Review Request: pynag - Python Nagios plugin and configuration environment

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@matthias-runge.de

--- Comment #8 from Matthias Runge  2012-03-02 
15:08:24 EST ---
Hi Tomas,

to get sponsored into the packager group, you must convince a sponsor to
support you (or have somebody else to convince a sponsor)

please read
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group
and do some informal reviews.

You should note the bugzilla numbers here (as a reference for your sponsor).

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

[Bug 676879] Review Request: mpiexec - MPI job launcher that uses the PBS task interface directly

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@matthias-runge.de

--- Comment #14 from Matthias Runge  2012-03-02 
15:20:06 EST ---
Christos, are you still interested?

If yes, you should do some informal reviews and list the bugzilla-numbers here
(as a reference for your potential sponsor).
You need to do something to prove a sponsor your packaging-knowledge.
Corresponding links are in comment 2 and still valid.

-- 
Configure 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 683638] Review Request: pyes- Python library for connecting to and managing Elasticsearch

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@matthias-runge.de

--- Comment #5 from Matthias Runge  2012-03-02 
15:39:26 EST ---
Tavis,

are you still interested?

If yes, you should refer to
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group 
especial the section
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group#Convincing_someone_to_sponsor_you

You should do some informal reviews to convince a sponsor of your packaging
qualities. You should note the bugzilla bug numbers here (as a reference for
your sponsor)

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 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Dan Williams  changed:

   What|Removed |Added

   Flag||needinfo?(jamie@tomoyolinux
   ||.co.uk)

--- Comment #7 from Dan Williams  2012-03-02 15:41:14 EST ---
Updated specs and RPMs here, using latest upstream nm-applet:

Spec URL: http://people.redhat.com/dcbw/network-manager-applet.spec
SRPM URL:
http://people.redhat.com/dcbw/network-manager-applet-0.9.3.995-1.git20120302.fc17.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 688183] Review Request: iceplayer - A simple music player for Linux

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@matthias-runge.de

--- Comment #16 from Matthias Runge  2012-03-02 
15:42:36 EST ---
Mike, are you still interested? 

The spec file isn't accessible any more.

-- 
Configure 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 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Dan Williams  2012-03-02 15:41:34 EST ---
(In reply to comment #7)
> Updated specs and RPMs here, using latest upstream nm-applet:
> 
> Spec URL: http://people.redhat.com/dcbw/network-manager-applet.spec
> SRPM URL:
> http://people.redhat.com/dcbw/network-manager-applet-0.9.3.995-1.git20120302.fc17.src.rpm

This one passes a fedpkg mockbuild for me.

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

[Bug 667790] Review Request: python-audioprocessing - Python audio processing suite

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@matthias-runge.de
  Status Whiteboard|NotReady|Stalled Submitter

--- Comment #6 from Matthias Runge  2012-03-02 
15:47:12 EST ---
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 755291] Review Request: mod_pagespeed - Page Speed is an open-source project started at Google to help developers optimize their web pages by applying web performance best practices

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||mru...@matthias-runge.de
 Resolution||NOTABUG
Last Closed||2012-03-02 15:50:21

--- Comment #2 from Matthias Runge  2012-03-02 
15:50:21 EST ---
Christopher,

if you're going to submit mod_pagespeed as a new package for Fedora, please
open a new Review Request. 
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 675402] Review Request: qpass - password manager

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||mru...@matthias-runge.de
 Resolution||WONTFIX
   Flag|needinfo?(mateuszpiekos@gma |
   |il.com) |
Last Closed||2012-03-02 15:15:38

-- 
Configure 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 767985] Review Request: man2html - Convert man pages to HTML

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@matthias-runge.de

--- Comment #2 from Matthias Runge  2012-03-02 
15:55:48 EST ---
In order to get sponsored into the packager group, please follow 
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group#Convincing_someone_to_sponsor_you

You should do some informal reviews (and note the bug numbers e.g. here, as a
reference for a potential sponsor).

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

[Bug 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Matthias Clasen  2012-03-02 15:59:05 
EST ---
Yes, builds fine now. rpmlint:

$ rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/*.rpm
libnm-gtk.x86_64: W: spelling-error %description -l en_US nm -> NM, mm, n
libnm-gtk.x86_64: W: obsolete-not-provided NetworkManager-gtk
libnm-gtk.x86_64: W: no-documentation
libnm-gtk-devel.x86_64: W: no-dependency-on
libnm-gtk/libnm-gtk-libs/liblibnm-gtk
libnm-gtk-devel.x86_64: W: spelling-error %description -l en_US config -> con
fig, con-fig, configure
libnm-gtk-devel.x86_64: W: spelling-error %description -l en_US nm -> NM, mm, n
libnm-gtk-devel.x86_64: W: no-documentation
network-manager-applet.src: W: invalid-url Source0:
http://ftp.gnome.org/pub/GNOME/sources/network-manager-applet/0.9/network-manager-applet-0.9.3.995.git20120302.tar.bz2
HTTP Error 404: Not Found
network-manager-applet.x86_64: W: incoherent-version-in-changelog 0.9.3.995-1
['0.9.3.995-1.git20120302.fc18', '0.9.3.995-1.git20120302']
network-manager-applet.x86_64: W: obsolete-not-provided NetworkManager-gnome
network-manager-applet.x86_64: W: non-conffile-in-etc
/etc/gconf/schemas/nm-applet.schemas
network-manager-applet.x86_64: E: zero-length
/usr/share/doc/network-manager-applet-0.9.3.995/AUTHORS
network-manager-applet.x86_64: E: zero-length
/usr/share/doc/network-manager-applet-0.9.3.995/README
network-manager-applet.x86_64: W: non-conffile-in-etc
/etc/xdg/autostart/nm-applet.desktop
network-manager-applet.x86_64: W: no-manual-page-for-binary nm-applet
nm-connection-editor.x86_64: W: no-documentation
nm-connection-editor.x86_64: W: no-manual-page-for-binary nm-connection-editor
6 packages and 0 specfiles checked; 2 errors, 15 warnings.

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

[Bug 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Clasen  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|mcla...@redhat.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 676879] Review Request: mpiexec - MPI job launcher that uses the PBS task interface directly

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #15 from Christos Triantafyllidis 
 2012-03-02 16:04:27 EST ---
Hi Matthias,
   i'm still interested although i'm not sure if latest version here is still
working given the major changes in torque side (upgrade from 2.3 to 2.5). Given
that my setup is still at 2.3 i don't see any urgent reason to put effort in
updating this if there is no reviewer for it. If you are willing to review it
and sponsor me i would happily do the needed (if any) changes and re-build it.

   I have done a few reviews or simply commented on other package requests in
past:
https://bugzilla.redhat.com/show_bug.cgi?id=682553
https://bugzilla.redhat.com/show_bug.cgi?id=683587
https://bugzilla.redhat.com/show_bug.cgi?id=772485

   And i have submitted another package too:
https://bugzilla.redhat.com/show_bug.cgi?id=772406

   Let me know if you think that i have to do more reviews.

Regards,
Christos

-- 
Configure 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 799521] New: Review Request: azove - Another Zero-One Vertex Enumeration tool

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: azove - Another Zero-One Vertex Enumeration tool

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

   Summary: Review Request: azove - Another Zero-One Vertex
Enumeration tool
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: loganje...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://jjames.fedorapeople.org/azove/azove.spec
SRPM URL: http://jjames.fedorapeople.org/azove/azove-2.0-1.fc16.src.rpm
Description: Azove is a tool designed for counting (without explicit
enumeration) and enumeration of 0/1 vertices.  Given a polytope by a linear
relaxation or facet description P = {x | Ax <= b}, all 0/1 points lying in P
can be counted or enumerated.  This is done by intersecting the polytope P with
the unit-hypercube [0,1] d.  The integral vertices (no fractional ones) of this
intersection will be enumerated.  If P is a 0/1 polytope, azove solves the
vertex enumeration problem.  In fact it can also solve the 0/1 knapsack problem
and the 0/1 subset sum problem.

-- 
Configure 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 673630] Review Request: habari - Lite blogging software

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@matthias-runge.de

--- Comment #6 from Matthias Runge  2012-03-02 
15:14:23 EST ---
just two small comments:
ln -sf ../../../etc/habari/config.php
${RPM_BUILD_ROOT}%{_datadir}/habari/config.php

this will probably fail!

In order to get someone sponsor you into the packager group, you must convince
a potential sponsor of your packaging knowledge. You could prove this by doing
'informal reviews'.

Please refer to:
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group

You should note the bugzilla numbers of your reviews here (as a reference for
your sponsor).

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

[Bug 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Matthias Clasen  2012-03-02 16:20:57 
EST ---
package name: ok
spec file name: ok
packaging guidelines: generally ok; but it would be nice to use
  the standard gconf macros
license: ok
license field: ok
language: ok
readable: ok
upstream sources: ok, but the url should be fixed
buildable: ok
excludearch: ok
buildrequires: ok
locale handling: ok
ldconfig: ok
system libs: ok
relocatable: ok
directory ownership: not ok; you must own /usr/share/nm-applet and
/usr/include/libnm-gtk
duplicate files: ok
file permissions: ok
macro use: ok
content: ok
large docs: ok
%doc content: ok
static libs: ok
devel files: ok
devel deps: not ok; libnm-gtk-devel should require libnm-gtk
libtool archives: ok
gui apps: not ok; currently only the autostart file is validated, the others
should be too
duplicate files: ok
utf8 filenames: ok

Summary: a few fixes required

-- 
Configure 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 797370] Review Request: python-ssh - A Python SSH2 library

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Thomas Spura  changed:

   What|Removed |Added

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

--- Comment #1 from Thomas Spura  2012-03-02 
16:28:36 EST ---
Review:
- name ok
- license ok
- URL ok
- source ok
  26800ef2c1ee3f185f48fd05258302f4  ssh-1.7.13.tar.gz
- noarch ok
- files ok
- rpmlint contains the incorrect-fsf-addresses from above nothing else, ok


NEEDSWORK:
- please add a %check section:
%check
./test.py --verbose

- There is a python3-crypto package in fc17 onwards. Could you please add a
python3 subpackage here too?
  Might be that ipython could use this too, and I'm trying to get all the
python3 dependencies fulfilled... (When you need a co-maintainer, let me know.)
  It seems to work after 2to3 and replacing tabs with 8 spaces.

- Please delete rm -rf ssh.egg-info/ in %prep just to be sure, that it's not
used laterly.

-- 
Configure 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 790564] Review Request: glassfish-fi - Fast Infoset

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Andy Grimm  changed:

   What|Removed |Added

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

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

[Bug 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Dan Williams  2012-03-02 16:34:38 EST ---
git snapshots don't pass the URL check, I'm afraid...

libnm-gtk-devel already own /usr/include/libnm-gtk:

%files -n libnm-gtk-devel
%defattr(-,root,root,0755)
%dir %{_includedir}/libnm-gtk

I'll update it to own /usr/share/nm-applet too.

-- 
Configure 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 786587] Review Request: network-manager-applet - applet, editor, and private libs for NetworkManager GUI

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Dan Williams  2012-03-02 16:39:02 EST ---
Updated specfile and srpm at same URLs as comment 7.

-- 
Configure 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 799558] New: Review Request: xfce4-soundmenu-plugin - MPRIS2 control plugin for the Xfce panel

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: xfce4-soundmenu-plugin - MPRIS2 control plugin for the 
Xfce panel

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

   Summary: Review Request: xfce4-soundmenu-plugin - MPRIS2
control plugin for the Xfce panel
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cwick...@fedoraproject.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://cwickert.fedorapeople.org/review/xfce4-soundmenu-plugin.spec
SRPM URL:
http://cwickert.fedorapeople.org/review/xfce4-soundmenu-plugin-0.4.0-1.fc18.src.rpm
Description: A very basic xfce4-panel plugin to control MPRIS2 compatible media
players.

Features:
* Play/Pause, Stop, Previous and Next track
* Display album cover in the panel
* Last.fm support

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3848311

-- 
Configure 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 630221] Review Request: ghc-vty - A simple terminal access library

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 630221 depends on bug 736586, which changed state.

Bug 736586 Summary: Review Request: ghc-vector - Efficient Arrays for Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=736586

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #5 from Ben Boeckel  2012-03-02 21:25:43 EST ---
Spec URL: http://benboeckel.net/packaging/ghc-vty/ghc-vty.spec
SRPM URL:
http://benboeckel.net/packaging/ghc-vty/ghc-vty-4.7.0.10-1.fc18.src.rpm

ghc-vty.src: W: spelling-error %description -l en_US ncurses -> nurses, curses,
n curses
ghc-vty.src: W: strange-permission ghc-vty.spec 0640L
ghc-vty.src: W: strange-permission vty-4.7.0.10.tar.gz 0640L
ghc-vty.x86_64: W: spelling-error %description -l en_US ncurses -> nurses,
curses, n curses
ghc-vty-devel.x86_64: W: spelling-error %description -l en_US ncurses ->
nurses, curses, n curses
3 packages and 0 specfiles checked; 0 errors, 5 warnings.

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

[Bug 630228] Review Request: ghc-io-storage - A key-value store in the IO monad

2012-03-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Ben Boeckel  2012-03-02 21:30:36 EST ---
Spec URL: http://benboeckel.net/packaging/ghc-io-storage/ghc-io-storage.spec
SRPM URL:
http://benboeckel.net/packaging/ghc-io-storage/ghc-io-storage-0.3-2.fc18.src.rpm

ghc-io-storage.src: W: spelling-error Summary(en_US) monad -> nomad, gonad,
Mona
ghc-io-storage.src: W: spelling-error %description -l en_US monad -> nomad,
gonad, Mona
ghc-io-storage.src: W: strange-permission io-storage-0.3.tar.gz 0640L
ghc-io-storage.src: W: strange-permission ghc-io-storage.spec 0640L
ghc-io-storage.x86_64: W: spelling-error Summary(en_US) monad -> nomad, gonad,
Mona
ghc-io-storage.x86_64: W: spelling-error %description -l en_US monad -> nomad,
gonad, Mona
ghc-io-storage-devel.x86_64: W: spelling-error Summary(en_US) monad -> nomad,
gonad, Mona
ghc-io-storage-devel.x86_64: W: spelling-error %description -l en_US monad ->
nomad, gonad, Mona
3 packages and 0 specfiles checked; 0 errors, 8 warnings.

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

  1   2   >