[Bug 877096] Review Request: perl-Fsdb - A set of commands for manipulating flat-text databases from the shell

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=877096

John Heidemann jo...@isi.edu changed:

   What|Removed |Added

  Flags|needinfo?(jo...@isi.edu)|

--- Comment #6 from John Heidemann jo...@isi.edu ---
Yes, I'm happy to maintain this package, in that we maintain it internally for
our research group, we run Fedora, and I am the package's lead developer.

But the guidelines for getting sponsored are a bit vague (hang out on irc). 
I think I submit reasonable bug reports if you want to assess my clue level. 
It's unclear (to me) how I should proceed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=l6n2xc6e2Da=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG), 983981

--- Comment #1 from Marek Goldmann mgold...@redhat.com ---
Rebuild: http://koji.fedoraproject.org/koji/taskinfo?taskID=5649989

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Hd4qMCKKdna=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 986172] Review Request: jackson-databind - General data-binding package for Jackson (2.x)

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986172

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks||857136, 857139, 984554,
   ||857137

--- Comment #5 from Marek Goldmann mgold...@redhat.com ---
*** Bug 857102 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=uNasvrgvVia=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857137] Review Request: jackson2-module-jaxb-annotations - JAXB annotations support for Jackson (2.x)

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857137

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends On||986172

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=MRE8XKzsCTa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857139] Review Request: jackson-module-mrbean - An extension that implements support for POJO type materialization

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857139

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends On||986172

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=IGqzgTrK8va=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857102] Review Request: jackson2-databind - General data-binding package for Jackson (2.x)

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Resolution|NOTABUG |DUPLICATE

--- Comment #12 from Marek Goldmann mgold...@redhat.com ---


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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=2rFCd5QS2ta=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857136] Review Request: jackson2-dataformat-smile - Jackson extension that adds support for Smile

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857136

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends On||986172

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=D3NgMgMWZYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857102] Review Request: jackson2-databind - General data-binding package for Jackson (2.x)

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends On||986170

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ZjUC7yN6dSa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 986170] Review Request: jackson-core - Core part of Jackson

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986170

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks||857139, 984554, 857080,
   ||857102, 857137

--- Comment #5 from Marek Goldmann mgold...@redhat.com ---
*** Bug 857077 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=IWMpxBg87la=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857077] Review Request: jackson2-core - Core part of Jackson

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857077

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Resolution|NOTABUG |DUPLICATE

--- Comment #16 from Marek Goldmann mgold...@redhat.com ---


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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=gjpRF69FMQa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857139] Review Request: jackson-module-mrbean - An extension that implements support for POJO type materialization

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857139

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends On||986170

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=UXHTBgSxksa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857080] Review Request: jackson2-annotations - Core annotations for Jackson data processor

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857080

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends On||986170

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=xQA15pwqvZa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857137] Review Request: jackson2-module-jaxb-annotations - JAXB annotations support for Jackson (2.x)

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857137

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends On||986170

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=TDiPNVubuia=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 986165] Review Request: jackson-annotations - Core annotations for Jackson data processor

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986165

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks||984554, 857102

--- Comment #5 from Marek Goldmann mgold...@redhat.com ---
*** Bug 857080 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ponM6BBtzMa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857102] Review Request: jackson2-databind - General data-binding package for Jackson (2.x)

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Depends On||986165

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=za0mnejO8Ya=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857080] Review Request: jackson2-annotations - Core annotations for Jackson data processor

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857080

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Resolution|NOTABUG |DUPLICATE

--- Comment #10 from Marek Goldmann mgold...@redhat.com ---


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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=9twKyjQK56a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857138] Review Request: jackson2-jaxrs-json-provider - JAX-RS MessageBodyReader and -Writer implementations for JSON

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857138

--- Comment #7 from Marek Goldmann mgold...@redhat.com ---
Gil, was this package renamed or the content of this package is available in
some other package? Which one?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=9FKCLwOexxa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 859114] Review Request: grizzly - Java NIO Server Framework

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=859114

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

   What|Removed |Added

 Blocks|652183 (FE-JAVASIG) |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=rjbZ0SkFQZa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987204] Review Request: libevdev - Kernel Evdev Device Wrapper Library

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987204

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

   What|Removed |Added

  Flags|needinfo?(bugs.michael@gmx. |
   |net)|

--- Comment #6 from Michael Schwendt bugs.mich...@gmx.net ---
Automatic pkg-config deps have been enabled several years ago (2008):
https://bugzilla.redhat.com/465377

For a package that includes a .pc file, you get an automatic Requires:
/usr/bin/pkg-config as well as versioned pkgconfig(…) Provides/Requires for
the .pc file inter-dependencies:

  $ repoquery --whatrequires /usr/bin/pkg-config|wc -l
  3694
  $ repoquery --whatrequires 'pkgconfig(glib-2.0)'|wc -l
  466
  $ repoquery --whatprovides 'pkgconfig(glib-2.0)'
  glib2-devel-0:2.36.3-2.fc19.i686
  glib2-devel-0:2.36.3-2.fc19.x86_64

There are packagers, who rely on these automatic pkgconfig(…) Provides even for
their BuildRequires tags (instead of -devel package names).

 I can find sources that say I'd have to change this to
 BuildRequires: pkgconfig instead

BuildRequires: pkgconfig is what you would add, if your src.rpm uses
pkg-config directly. (or if a build requirement uses it and is missing such a
dep for reasons to examine)


 the official stance

When I posted the comment, I've searched a bit, but I'm at a loss why this has
not been added to the guidelines by the FPC.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=nmqAEmyr9Xa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987619] Review Request: python-parse - Opposite of format()

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987619

--- Comment #2 from Matěj Cepl mc...@redhat.com ---
New versions at
http://mcepl.fedorapeople.org/tmp/python-parse-1.6.2-2.el7.src.rpm
http://mcepl.fedorapeople.org/tmp/python-parse.spec

Thank you for the review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=SLZFvkVlCwa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987619] Review Request: python-parse - Opposite of format()

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987619

--- Comment #3 from Matěj Cepl mc...@redhat.com ---
Scratch build http://koji.fedoraproject.org/koji/taskinfo?taskID=5650117

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=MsuRJHRHAea=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987153] Review Request: python-modernize - Modernizes Python code for eventual Python 3 migration

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987153

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=QIJP7r9BUYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 CC||punto...@libero.it

--- Comment #2 from gil cattaneo punto...@libero.it ---
hi
is available a new version: 1.0.0.Beta5.
please, consider upgrading.
regards

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=BQ6GLL6v4ea=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 962839] Review Request: hibernate-validator5 - Bean Validation 1.1 (JSR 349) Reference Implementation

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=962839

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks|652183 (FE-JAVASIG) |
 Resolution|--- |DEFERRED
Last Closed||2013-07-24 04:02:38

--- Comment #3 from Marek Goldmann mgold...@redhat.com ---
It seems that hibernate-validator 5 is required now for WildFly. We need to
upgrade the hibernate-validator package to version 5. I think we can skip for
now creating the compat package (hibernate-validator4) until something will
strictly require it.

I'm closing this bug.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=f4RjZpbIWEa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

--- Comment #3 from Marek Goldmann mgold...@redhat.com ---
Skipping Beta5, WildFly Alpha3 requires Beta3. This will be upgraded to newer
version once WildFly will require it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=yaUCAfsqT8a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|punto...@libero.it
  Flags||fedora-review?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=8eestqArFla=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=SGhGULufWya=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

--- Comment #4 from gil cattaneo punto...@libero.it ---

Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- Package installs properly.
  Note: Installation errors (see attachment)
  See: https://fedoraproject.org/wiki/Packaging:Guidelines
  IGNORE: manual review.
  In rawhide build fails wit the following errors:
[ERROR]
/builddir/build/BUILD/undertow-1.0.0.Beta3/parser-generator/src/main/java/io/undertow/annotationprocessor/AbstractParserGenerator.java:[40,38]
cannot find symbol
  symbol:   class TableSwitchBuilder
  location: package org.jboss.classfilewriter.code
[ERROR]
/builddir/build/BUILD/undertow-1.0.0.Beta3/parser-generator/src/main/java/io/undertow/annotationprocessor/AbstractParserGenerator.java:[635,68]
cannot find symbol
  symbol:   class TableSwitchBuilder
  location: class io.undertow.annotationprocessor.AbstractParserGenerator
[ERROR]
/builddir/build/BUILD/undertow-1.0.0.Beta3/parser-generator/src/main/java/io/undertow/annotationprocessor/AbstractParserGenerator.java:[253,9]
cannot find symbol
  symbol:   class TableSwitchBuilder
  location: class io.undertow.annotationprocessor.AbstractParserGenerator
[ERROR]
/builddir/build/BUILD/undertow-1.0.0.Beta3/parser-generator/src/main/java/io/undertow/annotationprocessor/AbstractParserGenerator.java:[253,42]
cannot find symbol
  symbol:   class TableSwitchBuilder
  location: class io.undertow.annotationprocessor.AbstractParserGenerator
[ERROR]
/builddir/build/BUILD/undertow-1.0.0.Beta3/parser-generator/src/main/java/io/undertow/annotationprocessor/AbstractParserGenerator.java:[485,10]
cannot find symbol
  symbol:   method
setupFrame(java.lang.String,java.lang.String,java.lang.String,java.lang.String,java.lang.String,java.lang.String,java.lang.String,java.lang.String,java.lang.String)
  location: variable c of type org.jboss.classfilewriter.code.CodeAttribute
[INFO] 5 errors

- Bundled jar/class files should be removed before build
  Note: Jar files in source (see attachment)
  See:
http://fedoraproject.org/wiki/Packaging:Java#Pre-built_JAR_files_.2F_Other_bundled_software'
Please, remove these classes:
./undertow-1.0.0.Beta3/mac-jdk-fix/jdk6/sun/nio/ch/KQueueSelectorImpl$MapEntry.class
./undertow-1.0.0.Beta3/mac-jdk-fix/jdk6/sun/nio/ch/KQueueArrayWrapper$Update.class
./undertow-1.0.0.Beta3/mac-jdk-fix/jdk6/sun/nio/ch/KQueueArrayWrapper.class
./undertow-1.0.0.Beta3/mac-jdk-fix/jdk6/sun/nio/ch/KQueueSelectorImpl.class
./undertow-1.0.0.Beta3/mac-jdk-fix/jdk7/sun/nio/ch/KQueueSelectorImpl$MapEntry.class
./undertow-1.0.0.Beta3/mac-jdk-fix/jdk7/sun/nio/ch/KQueueArrayWrapper$Update.class
./undertow-1.0.0.Beta3/mac-jdk-fix/jdk7/sun/nio/ch/KQueueArrayWrapper.class
./undertow-1.0.0.Beta3/mac-jdk-fix/jdk7/sun/nio/ch/KQueueSelectorImpl.class

= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
 Note: Using prebuilt rpms.
[!]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Fully versioned dependency in subpackages, if present.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in undertow-
 javadoc
[x]: Package complies to the Packaging Guidelines
[!]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 Apache (v2.0), LGPL (v2.1 or later), GPL (v2), Unknown or
 generated. 261 files have unknown license.
 Please, FIXME
GPL (v2)  GPLv2 with exceptions

/mac-jdk-fix/jdk6/KQueueArrayWrapper.java
/mac-jdk-fix/jdk6/KQueueSelectorImpl.java
/mac-jdk-fix/jdk7/KQueueArrayWrapper.java
/mac-jdk-fix/jdk7/KQueueSelectorImpl.java

LGPL (v2.1 or later)

/core/src/main/java/io/undertow/client/HttpClient.java
/core/src/main/java/io/undertow/client/HttpClientConnection.java
/core/src/main/java/io/undertow/client/HttpClientRequest.java
/core/src/main/java/io/undertow/client/HttpClientResponse.java
/core/src/main/java/io/undertow/server/TruncatedResponseException.java

[!]: License file installed when any subpackage combination is installed.
[x]: Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not 

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=kRP8jMrdi9a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 962836] Review Request: beanvalidation-tck - Bean Validation (JSR 349) TCK

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=962836

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Blocks||871014

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=SBeKafBDEOa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from Marek Goldmann mgold...@redhat.com ---
New jboss-classfilewriter is available in Rawhide that fixes the problem.

I'll fix the licensing issues and remove the .class files at the import time.
Thanks!

New Package SCM Request
===
Package Name: undertow
Short Description: Java web server using non-blocking IO
Owners: goldmann

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Ae9Q3w6tgIa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks|652183 (FE-JAVASIG) |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=VGgYJ56ZJ2a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 986567] Review Request:cinnamon-control-center - Utilities to configure the Cinnamon desktop

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986567

--- Comment #3 from leigh scott leigh123li...@googlemail.com ---
Changes made


Spec URL:
http://leigh123linux.fedorapeople.org/pub/review/cinnamon-control-center/3/cinnamon-control-center.spec

SRPM URL:
http://leigh123linux.fedorapeople.org/pub/review/cinnamon-control-center/3/cinnamon-control-center-1.9.0-0.3.git430927a.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Vs9lIZmyO3a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 658754] Review Request: cubrid - a very fast and reliable open source SQL database server

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=658754

--- Comment #39 from Esen Sagynov esen.sagy...@nhn.com ---
Response to Michael's comments. The spec file has been updated with the
following changes.

  %global file_version   1
 Version:   %{cubrid_version}.%{build_version}
 %global cubrid_vendor  Search Solution Corporation
 BuildRequires: glibc-devel
 Group:  Development/Libraries
 %post devel -p /sbin/ldconfig
 %postun devel -p /sbin/ldconfig
 %preun devel

Removed/updated.

 %package devel 
 Requires:   %{name} = %{cubrid_version}.%{build_version}

 That isn't strict enough for how Fedora does it:
https://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package

 Imagine you release a fix where you want the -devel package (or other 
 subpackages) to be in sync with the base package.

Updated.

 %global debug_package %{nil}

 Why that?

We had to add this line because it was gaving an RPM building error no build
id note found in for cub_cmhttpd.

 %build
 %ifarch x86_64
 CFLAGS= -m64  
 CUBRID_COMMON_CONFIGURE=${CUBRID_COMMON_CONFIGURE} --enable-64bit
 %endif

 What about other 64-bit targets?

Which are the other 64-bit targets you are referring to?

 ./autogen.sh
 ./configure ${CUBRID_COMMON_CONFIGURE}

 autogen.sh but no %configure macro usage? See rpm --eval %configure. If 
 %configure cannot be used with this configure script, please add a comment in 
 the spec file.

Added the following comment:

# %configure macro is disabled/missing because not all options are supported in
CUBRID.
# For example, when passing bindir or libdir parameter will lead to an
error
# (not supported feature).

 %install

 It's odd that this section is so long. Why is it necessary to install/adjust 
 so many things here after make install? Why doesn't make install do all 
 that?

Currently, in the original CUBRID spec make install installs all files in one
directory. To submit to Fedora/Ubuntu we've created new specs (this spec for
Fedora). In the new spec we've tried to place all files in their proper
locations: (conf files in /etc/conf/cubrid, the variable files to /var/cubrid,
the executables to /usr/bin, the libraries to /usr/lib or /usr/lib64 depending
on architecture and so forth). This is why the install section is long.

 %post demodb 
 /sbin/ldconfig

 It contains only files below /usr/share.

 %postun demodb 
 /sbin/ldconfig

 Same here.

Updated.

 find %{buildroot} -size 0 -delete

 Hopefully you'll never need to include an empty file or touch one to include 
 it as %ghost.

The source may have empty files (for example some optional configuration
files). This was to remove warnings and possible empty files from the source.

 The %files sections list many files with very generic file names (even 
 headers such as %{_includedir}/dbi.h). It'll be a lot of fun to avoid 
 conflicts with other packages.

Yes, there are files which we have found to have similar names, but at this
moment it's not possible to change them. If any serious conflict is found, we
will put resources to fix them.

However, if you have any suggestion to avoid name conflicts, please let us
know.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=dO4wPpHceZa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 658754] Review Request: cubrid - a very fast and reliable open source SQL database server

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=658754

--- Comment #40 from Esen Sagynov esen.sagy...@nhn.com ---
Cristian, we've added the libtool to the dependency list. It should work now.
Please try and let us know if you encounter any issue.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=aUbcQGjlERa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 658754] Review Request: cubrid - a very fast and reliable open source SQL database server

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=658754

--- Comment #41 from Peter Lemenkov lemen...@gmail.com ---
(In reply to Esen Sagynov from comment #37)
 Peter, it's kadishmal. Cristian and Michael, we will go over your comments
 one by one, and update the spec.

Please make sure you're using the same email address for Bugzilla account and
for FAS. It caused strange issues earlier. I advise you to change Bugzilla
email to match FAS one.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=TuXM3ErsCXa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987619] Review Request: python-parse - Opposite of format()

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987619

--- Comment #4 from Matěj Cepl mc...@redhat.com ---
Auch, make it

http://mcepl.fedorapeople.org/tmp/python-parse-1.6.2-3.el7.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=DuEDO5K3GEa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985396] Review Request: cdi-api1 - CDI API 1.0

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985396

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Blocks|652183 (FE-JAVASIG) |
  Flags||fedora-cvs?

--- Comment #2 from Marek Goldmann mgold...@redhat.com ---
Thanks for the review!

New Package SCM Request
===
Package Name: cdi-api1
Short Description: CDI API 1.0
Owners: goldmann

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=oSLWYkZos9a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 658754] Review Request: cubrid - a very fast and reliable open source SQL database server

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=658754

--- Comment #42 from Esen Sagynov kadish...@gmail.com ---
Ok, I've just changed Bugzilla email to my FAS email.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=jsQ8n2hcjXa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987843] New: Review Request: python-bloom - A ROS release automation tool for catkin packages

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987843

Bug ID: 987843
   Summary: Review Request: python-bloom - A ROS release
automation tool for catkin packages
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sanjay.an...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://ankursinha.fedorapeople.org/python-bloom/python-bloom.spec
SRPM URL:
http://ankursinha.fedorapeople.org/python-bloom/python-bloom-0.4.4-1.fc20.src.rpm

Description: 
Fedora Account System Username: ankursinha

rpmlint output:

[asinha@localhost  SRPMS]$ rpmlint ../SPECS/python-bloom.spec
/var/lib/mock/fedora-rawhide-x86_64/result/*.rpm
./python-bloom-0.4.4-1.fc19.src.rpm
../SPECS/python-bloom.spec:43: W: macro-in-comment %check
../SPECS/python-bloom.spec:44: W: macro-in-comment %{__python}
python-bloom.noarch: W: spelling-error %description -l en_US gitbuildpackage -
prepackage
python-bloom.noarch: W: devel-file-in-non-devel-package
/usr/bin/git-bloom-config
python-bloom.noarch: W: no-manual-page-for-binary git-bloom-generate
python-bloom.noarch: W: no-manual-page-for-binary git-bloom-import-upstream
python-bloom.noarch: W: no-manual-page-for-binary bloom-generate
python-bloom.noarch: W: no-manual-page-for-binary git-bloom-branch
python-bloom.noarch: W: no-manual-page-for-binary git-bloom-release
python-bloom.noarch: W: no-manual-page-for-binary git-bloom-patch
python-bloom.noarch: W: no-manual-page-for-binary bloom-update
python-bloom.noarch: W: no-manual-page-for-binary git-bloom-config
python-bloom.noarch: W: no-manual-page-for-binary bloom-export-upstream
python-bloom.noarch: W: no-manual-page-for-binary bloom-release
python-bloom.src: W: spelling-error %description -l en_US gitbuildpackage -
prepackage
python-bloom.src:43: W: macro-in-comment %check
python-bloom.src:44: W: macro-in-comment %{__python}
python-bloom.src: W: spelling-error %description -l en_US gitbuildpackage -
prepackage
python-bloom.src:43: W: macro-in-comment %check
python-bloom.src:44: W: macro-in-comment %{__python}
3 packages and 1 specfiles checked; 0 errors, 20 warnings.
[asinha@localhost  SRPMS]$


Haven't been able to build the docs yet.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=TMTnNKwII8a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985753] Review Request: dfuzzer - Fuzzer for processes connected to D-Bus

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985753

--- Comment #5 from Matus Marhefka mmarh...@redhat.com ---
Eduardo: Thank you for help.
I added docs, man page and corrected gcc flags and source.

Spec URL: http://www.stud.fit.vutbr.cz/~xmarhe00/dfuzzer.spec
SRPM URL: www.stud.fit.vutbr.cz/~xmarhe00/dfuzzer-1.0-1.fc18.src.rpm

As GitHub generates different links from names of downloaded packages, you
must use spectool to download source. I included comment in spec file.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=IwIVgv0Qxga=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 956304] Review Request: fts - File Transfer Service V3

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=956304

--- Comment #20 from Fedora Update System upda...@fedoraproject.org ---
fts-3.1.0-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/fts-3.1.0-1.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=pDfMsYcMusa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 956304] Review Request: fts - File Transfer Service V3

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=956304

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=JNtW2eUELQa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 927462] Review Request: roscpp_core - The ROS C++ API

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927462

Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||sanjay.an...@gmail.com
   Assignee|nob...@fedoraproject.org|sanjay.an...@gmail.com
  Alias||roscpp_core
  Flags||fedora-review?

--- Comment #2 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
[+] OK
[-] NA
[?] Issue

** Mandatory review guidelines: **
 [?] rpmlint output:
[asinha@localhost  SRPMS]$ rpmlint ../SPECS/roscpp_core.spec
./roscpp_core-0.2.6-1.20130605gitd0b5ce1.fc19.src.rpm
/var/lib/mock/fedora-rawhide-x86_64/result/*.rpm
roscpp_core.src: W: spelling-error %description -l en_US roscpp - Roscoe
roscpp_core.src: W: spelling-error %description -l en_US roscpp - Roscoe
roscpp_core.x86_64: W: spelling-error %description -l en_US roscpp - Roscoe
roscpp_core.x86_64: E: invalid-soname /usr/lib64/libroscpp_serialization.so
libroscpp_serialization.so
roscpp_core.x86_64: E: invalid-soname /usr/lib64/libcpp_common.so
libcpp_common.so
roscpp_core.x86_64: E: invalid-soname /usr/lib64/librostime.so librostime.so
roscpp_core.x86_64: W: no-documentation
roscpp_core-debuginfo.x86_64: W: spelling-error Summary(en_US) roscpp - Roscoe
roscpp_core-debuginfo.x86_64: W: spelling-error %description -l en_US roscpp -
Roscoe
roscpp_core-devel.x86_64: W: spelling-error Summary(en_US) roscpp - Roscoe
roscpp_core-devel.x86_64: W: spelling-error %description -l en_US roscpp -
Roscoe
5 packages and 1 specfiles checked; 3 errors, 8 warnings.
[asinha@localhost  SRPMS]$

^^ The invalid soname errors need to be corrected. Generally, the versioned
soname should be present in the package, and the unversioned soname in the
devel package as a symlink of the versioned soname. At the present time, the
devel package here doesn't even have a shared object in it to link against.


 [+] License is acceptable (...)
 [+] License field in spec is correct
 [+] License files included in package %docs if included in source package
 [+] License files installed when any subpackage combination is installed
 [+] Spec written in American English
 [+] Spec is legible
 [+] Sources match upstream unless altered to fix permissibility issues
[asinha@localhost  SPECS]$ review-md5check.sh roscpp_core.spec
mkdir: cannot create directory ‘/tmp/review’: File exists
Getting
https://github.com/ros/roscpp_core/archive/d0b5ce1d8f42050a5674875b866a7dda6383a75b/roscpp_core-0.2.6-d0b5ce1.tar.gz
to /tmp/review/roscpp_core-0.2.6-d0b5ce1.tar.gz % Total% Received % Xferd 
Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
100   157  100   1570 0119  0  0:00:01  0:00:01 --:--:--   119
  0 00 468450 0  14953  0 --:--:--  0:00:03 --:--:-- 4
58af78843ead517ef8be46df06f90c0b  /tmp/review/roscpp_core-0.2.6-d0b5ce1.tar.gz
58af78843ead517ef8be46df06f90c0b
/home/asinha/rpmbuild/SOURCES/roscpp_core-0.2.6-d0b5ce1.tar.gz
removed ‘/tmp/review/roscpp_core-0.2.6-d0b5ce1.tar.gz’
removed directory: ‘/tmp/review’
[asinha@localhost  SPECS]$
 [+] Build succeeds on at least one primary arch
 [+] Build succeeds on all primary arches or has ExcludeArch + bugs filed
 [+] BuildRequires correct, justified where necessary
 [-] Locales handled with %find_lang, not %_datadir/locale/*
 [?] %post, %postun call ldconfig if package contains shared .so files
^ 
Please use the correct scriptlets

 [+] No bundled libs
 [-] Relocatability is justified
 [+] Package owns all directories it creates
 [-] Package requires others for directories it uses but does not own
 [+] No duplication in %files unless necessary for license files
 [+] File permissions are sane
 [+] Package contains permissible code or content
 [-] Large docs go in -doc subpackage
 [+] %doc files not required at runtime
 [-] Static libs go in -static package/virtual Provides
 [+] Development files go in -devel package
 [+] -devel packages Require base with fully-versioned dependency, %_isa
 [+] No .la files
 [-] GUI app uses .desktop file, installs it with desktop-file-install
 [-] File list does not conflict with other packages' without justification
 [+] File names are valid UTF-8

** Optional review guidelines: **
 [-] Query upstream about including license files
 [-] Translations of description, summary
 [+] Builds in mock
 [+] Builds on all arches
 [-] Functions as described (e.g. no crashes)
 [?] Scriptlets are sane
^
Missing ldconfig scriptlets

 [+] Subpackages require base with fully-versioned dependency if sensible
 [+] .pc file subpackage placement is sensible
 [+] No file deps outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin
 [-] Include man pages if available

Naming 

[Bug 927458] Review Request: rospack - ROS package and stack tools

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927458

Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed:

   What|Removed |Added

 CC||sanjay.an...@gmail.com

--- Comment #5 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
Rich, please do the SCM stuff so this can be pushed to testing. (Looks like
there are other packages that depend on this one)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=QToGsRxhTRa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987843] Review Request: python-bloom - A ROS release automation tool for catkin packages

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987843

Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed:

   What|Removed |Added

  Alias||python-bloom

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=TFSvRgzotDa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 928584] Review Request: ros-std_msgs - Standard ROS Messages

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=928584

--- Comment #3 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
The other packages seem to be in testing. Rich, can you see if they can be
pushed to stable so I can use mock/koji to test builds without using a local
repo please? I'll review this package later today.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ZuYOtvgxkda=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 986567] Review Request:cinnamon-control-center - Utilities to configure the Cinnamon desktop

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986567

Wolfgang Ulbrich chat-to...@raveit.de changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

--- Comment #4 from Wolfgang Ulbrich chat-to...@raveit.de ---
APPROVED !

Please remove needless update-mime-database and update-desktop-database
scriptlets, there are no mime types in desktop files and no file is installed
in /usr/share/mime.

And notice upstream about incorrect FSF addresses!

Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- update-desktop-database is invoked when required
  Note: desktop file(s) in cinnamon-control-center
  See: http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache

Fault positive, the desktop file doesn't contain any mime type.


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
 Note: Unversioned so-files in private %_libdir subdirectory (see
 attachment). Verify they are not in ld path.
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: %config files are marked noreplace or the reason is justified.
 Note: No (noreplace) in %config /etc/xdg/autostart/cinnamon-sound-
 applet.desktop %config /etc/xdg/menus/cinnamoncc.menu
[x]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[-]: Fully versioned dependency in subpackages, if present.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in cinnamon-
 control-center-filesystem
[x]: Package complies to the Packaging Guidelines
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 LGPL (v2.1 or later) (with incorrect FSF address), LGPL (v2 or
 later), GPL (v2 or later), Unknown or generated, MIT/X11 (BSD
 like), LGPL (v2 or later) (with incorrect FSF address), ISC, GPL
 (v2) (with incorrect FSF address), GPL (v2 or later) (with incorrect
 FSF address), GPL (v2.1 or later). 22 files have unknown license.
 Detailed output of licensecheck in /home/rave/986567-cinnamon-control-
 center/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: The spec file handles locales properly.
[x]: Package consistently uses macro is (instead of hard-coded directory
 names).
[-]: If the package is under multiple licenses, the licensing breakdown must
 be documented in the spec.
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: gtk-update-icon-cache is invoked when required
 Note: icons in cinnamon-control-center
[x]: Useful -debuginfo package or justification otherwise.
[x]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 30720 bytes in 3 files.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Each %files section contains %defattr if rpm  4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install if there is
 such a file.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: If (and only if) the source package includes the text of the license(s)
 

[Bug 658754] Review Request: cubrid - a very fast and reliable open source SQL database server

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=658754

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |

--- Comment #43 from Peter Lemenkov lemen...@gmail.com ---
Unblocking FE-NEEDSPONSOR - I've just sponsored Esen.

Esen, could you please provide a successful build in Koji? That's just for
starters.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=S5tuTtoePYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 658754] Review Request: cubrid - a very fast and reliable open source SQL database server

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=658754

--- Comment #44 from Esen Sagynov kadish...@gmail.com ---
Thank you Peter. I'm out of office right now. I will provide the build in Koji
tomorrow morning when I get to my office. Is that ok?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=lNj1cwCDwBa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 658754] Review Request: cubrid - a very fast and reliable open source SQL database server

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=658754

--- Comment #45 from Peter Lemenkov lemen...@gmail.com ---
(In reply to Esen Sagynov from comment #44)
 Thank you Peter. I'm out of office right now. I will provide the build in
 Koji tomorrow morning when I get to my office. Is that ok?

Yes, sure.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=B91ns4EfE6a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 986567] Review Request:cinnamon-control-center - Utilities to configure the Cinnamon desktop

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986567

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

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from leigh scott leigh123li...@googlemail.com ---
New Package SCM Request
===
Package Name: cinnamon-control-center
Short Description: Utilities to configure the Cinnamon desktop
Owners: leigh123linux
Branches: f19
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=gFJvphG5sXa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987868] New: Review Request: bean-validation-api1 - Bean Validation API (JSR 303)

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987868

Bug ID: 987868
   Summary: Review Request: bean-validation-api1 - Bean Validation
API (JSR 303)
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: punto...@libero.it
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://gil.fedorapeople.org/bean-validation-api1.spec
SRPM URL:
http://gil.fedorapeople.org/bean-validation-api1-1.0.0-10.fc19.src.rpm
Description: This package contains Bean Validation (JSR-303) API.
Fedora Account System Username: gil

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=5650687

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=jJ2YynCPgta=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987881] New: Review Request: sisu-xmlrpc - A Java implementation of XML-RPC with Async Http Client support

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987881

Bug ID: 987881
   Summary: Review Request: sisu-xmlrpc - A Java implementation of
XML-RPC with Async Http Client support
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: punto...@libero.it
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://gil.fedorapeople.org/sisu-xmlrpc.spec
SRPM URL: http://gil.fedorapeople.org/sisu-xmlrpc-1.2.0-1.fc19.src.rpm
Description:
Sisu XML-RPC is a Java implementation of XML-RPC,
a popular protocol that uses XML over HTTP to
implement remote procedure calls. Compared to SOAP,
or JAX-RPC, it is stable, much simpler and easier to
handle.
Sisu XML-RPC provides HTTP transport pluggability
via Async HTTP Client.

Fedora Account System Username: gil

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=5650858

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Nvf4exLKpPa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987884] New: Review Request: swizzle - Stream-based parsing code

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987884

Bug ID: 987884
   Summary: Review Request: swizzle - Stream-based parsing code
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: punto...@libero.it
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://gil.fedorapeople.org/swizzle.spec
SRPM URL: http://gil.fedorapeople.org/swizzle-1.6.2-1.fc19.src.rpm
Description: Swizzle started out as some stream-based parsing code.
Fedora Account System Username: gil

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=8uR00cyuTra=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987884] Review Request: swizzle - Stream-based parsing code

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987884

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Depends On||987881

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=5eHgdG097Ra=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987881] Review Request: sisu-xmlrpc - A Java implementation of XML-RPC with Async Http Client support

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987881

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Blocks||987884

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=bKsKdjhPyTa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987868] Review Request: bean-validation-api1 - Bean Validation API (JSR 303)

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987868

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 CC||mgold...@redhat.com
   Assignee|nob...@fedoraproject.org|mgold...@redhat.com

--- Comment #1 from Marek Goldmann mgold...@redhat.com ---
Assigning it to me to block reviewing this bug. We'll see if it's strictly
required.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=7GtYA7OUdba=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 975309] Review Request: libcutl - C++ utility library from Code Synthesis

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=975309

--- Comment #13 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=hf58zDgRm3a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 975309] Review Request: libcutl - C++ utility library from Code Synthesis

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=975309

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=wmdUrRTENta=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985396] Review Request: cdi-api1 - CDI API 1.0

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985396

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=m3I38RgFVua=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985396] Review Request: cdi-api1 - CDI API 1.0

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985396

--- Comment #3 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ZVkZCRUrqQa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985640] Review Request: 2ping - Bi-directional ping utility

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985640

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=dtSJksqL3Ga=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985640] Review Request: 2ping - Bi-directional ping utility

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985640

--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=uEt1FeFvwWa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 986567] Review Request:cinnamon-control-center - Utilities to configure the Cinnamon desktop

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986567

--- Comment #6 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=uhi4AMSgDla=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 986567] Review Request:cinnamon-control-center - Utilities to configure the Cinnamon desktop

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986567

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=R3qqa2149ea=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

--- Comment #6 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=u7TB3cPmyMa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987472] Review Request: undertow - Java web server using non-blocking IO

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987472

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=q8jgIfsbM3a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 879740] Review Request: python-evdev - bindings for the linux input handling subsystem

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=879740

--- Comment #1 from gvalkov georgi.t.val...@gmail.com ---
I'm renewing my effort to get this into Fedora. Please consider this comment as
the official Review Request, instead of the one above.

Spec URL: http://gvalkov.fedorapeople.org/python-evdev-0.4.1/python-evdev.spec
SRPM URL:
http://gvalkov.fedorapeople.org/python-evdev-0.4.1/python-evdev-0.4.1-1.fc19.src.rpm

Description:
 Python-evdev provides bindings to the generic input event interface in Linux.
 The evdev interface serves the purpose of passing events generated in the
 kernel directly to userspace through character devices that are typically
 located in /dev/input/.

 Python-evdev also comes with bindings to uinput, the userspace input 
 subsystem.
 Uinput allows userspace programs to create and handle input devices from which
 input events can be directly injected into the input subsystem.

 $ rpmlint python-evdev-0.4.1-1.fc19.src.rpm
 python-evdev.src: W: spelling-error %description -l en_US userspace - user 
 space, user-space, users pace
 python-evdev.src: W: spelling-error %description -l en_US dev - deb, derv, 
 div
 python-evdev.src: W: spelling-error %description -l en_US uinput - input, u 
 input, Putin
 python-evdev.src: W: spelling-error %description -l en_US Uinput - Input, U 
 input, Putin
 python-evdev.src: W: no-version-in-last-changelog
 1 packages and 0 specfiles checked; 0 errors, 5 warnings.

I am the upstream maintainer of python-evdev. This is also one of my first
package review requests. I would be thankful to anyone willing to sponsor me.  

Fedora Account System Username: gvalkov
Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5651095

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=2Xn5scbWLOa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 927458] Review Request: rospack - ROS package and stack tools

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927458

Rich Mattes richmat...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #6 from Rich Mattes richmat...@gmail.com ---
Thanks for the review!

New Package SCM Request
===
Package Name: rospack 
Short Description: ROS package and stack tools
Owners: rmattes
Branches: f18 f19 el6
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=XZBi894bPOa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 927458] Review Request: rospack - ROS package and stack tools

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927458

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=uxEcrFXyPfa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 927458] Review Request: rospack - ROS package and stack tools

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927458

--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=sWa6LYv0Ewa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 928584] Review Request: ros-std_msgs - Standard ROS Messages

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=928584

--- Comment #4 from Rich Mattes richmat...@gmail.com ---
So I removed the ros-* virtual provides from most of the packages; I'll update
this spec later tonight with the correct provides.

The gen{cpp,lisp,py} builds were only pushed a day or two ago, so they probably
won't make it into stable until next week.  You should be able to do rawhide
mock builds now (once I fix the BuildRequires)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=k1cwCR6xmoa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 972346] Review Request: ros-common_msgs - Common ROS Messages

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972346

--- Comment #2 from Rich Mattes richmat...@gmail.com ---
Right, I need to fix the BuildRequires.  I will do that later today.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=RXvhnEPzhJa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985818] Review Request: guacamole-server - Server-side native components that form the Guacamole proxy

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985818

--- Comment #4 from Simone Caronni negativ...@gmail.com ---
Hello, any news on the review?

The client package [1] has already finished the review and is pending on this.

[1] https://bugzilla.redhat.com/show_bug.cgi?id=985814

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=EB1euM2e1Ca=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987843] Review Request: python-bloom - A ROS release automation tool for catkin packages

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987843

Rich Mattes richmat...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||richmat...@gmail.com
   Assignee|nob...@fedoraproject.org|richmat...@gmail.com
  Flags||fedora-review?

--- Comment #1 from Rich Mattes richmat...@gmail.com ---
I'll take this for review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ii2im79BpEa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 984846] Review Request: jipijapa - Improve application platform integration with JPA persistence providers

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=984846

Marek Goldmann mgold...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 Blocks|652183 (FE-JAVASIG) |
   Assignee|nob...@fedoraproject.org|mgold...@redhat.com
  Flags||fedora-review?

--- Comment #2 from Marek Goldmann mgold...@redhat.com ---
I'll take this for a review, tomorrow.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=734ay3P1koa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 825347] Review Request: jersey - JAX-RS (JSR 311) production quality Reference Implementation

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=825347

--- Comment #21 from Fedora Update System upda...@fedoraproject.org ---
jersey-1.17.1-4.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/jersey-1.17.1-4.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=4yabBtz2uJa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985247] Review Request: jackson-jaxrs-providers - Jackson JAX-RS providers

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985247

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Blocks||984554

--- Comment #6 from gil cattaneo punto...@libero.it ---
*** Bug 857138 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=EVqp816lpWa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857138] Review Request: jackson2-jaxrs-json-provider - JAX-RS MessageBodyReader and -Writer implementations for JSON

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857138

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE
  Flags|fedora-cvs+ |fedora-cvs?
Last Closed||2013-07-24 11:14:44

--- Comment #8 from gil cattaneo punto...@libero.it ---
i apologize for the delay in answering.
yes, this package was includes in 
https://github.com/FasterXML/jackson-jaxrs-providers/
available at 
https://bugzilla.redhat.com/show_bug.cgi?id=985247

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=U6KPQswMVIa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857138] Review Request: jackson2-jaxrs-json-provider - JAX-RS MessageBodyReader and -Writer implementations for JSON

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857138

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=qjIVe1c6q0a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857138] Review Request: jackson2-jaxrs-json-provider - JAX-RS MessageBodyReader and -Writer implementations for JSON

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857138

--- Comment #9 from Jon Ciesla limburg...@gmail.com ---
Already exists, clearing flag.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=3BszgiScH5a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 827167] Review Request: bumblebee - Bumblebee daemon

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=827167

--- Comment #26 from Gary Gatling gsgat...@ncsu.edu ---
Christopher,

Thanks so much for taking the time to review my package. Here is another
attempt at fixing the problems with it:


http://install.linux.ncsu.edu/pub/yum/itecs/public/bumblebee/fedora19/spec/1/bumblebee.spec

http://install.linux.ncsu.edu/pub/yum/itecs/public/bumblebee/fedora19/SRPMS/bumblebee-3.2.1-2.fc19.src.rpm

Cheers,

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=FNVZrhEa62a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] New: Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

Bug ID: 988102
   Summary: Review Request: perl-DBD-Sybase - this package
provides an interface for perl to connect with Sybase
and MSSql databases
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@asshaueronline.de
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://mcas.fedorapeople.org/review/perl-DBD-Sybase.spec
SRPM URL: http://mcas.fedorapeople.org/review/perl-DBD-Sybase-1.15-1.src.rpm
Description: DBD::Sybase is a Perl module which works with the DBI module
to provide access to Sybase or MS-SQL databases.
Fedora Account System Username: mcas

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=uEPwYZYiTpa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

Björn Esser bjoern.es...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bjoern.es...@gmail.com
 Blocks||177841 (FE-NEEDSPONSOR)
   Assignee|nob...@fedoraproject.org|bjoern.es...@gmail.com
  Flags||fedora-review?

--- Comment #1 from Björn Esser bjoern.es...@gmail.com ---
I'll take this one.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=p6xQuYVaZJa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988105] New: Review Request: check-mssql-health - nagios check for mssql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988105

Bug ID: 988105
   Summary: Review Request: check-mssql-health - nagios check for
mssql databases
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@asshaueronline.de
QA Contact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org

Spec URL: http://mcas.fedorapeople.org/review/check-mssql-health.spec
SRPM URL:
http://mcas.fedorapeople.org/review/check-mssql-health-1.5.19.1-1.fc19.src.rpm
Description: check-mssql-health is a plugin, which is used to monitor different
parameters of a MS SQL server.
Fedora Account System Username: mcas

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=atGOueL0iLa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

Marcus Asshauer fed...@asshaueronline.de changed:

   What|Removed |Added

 Blocks||988105

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=OFy35tjVSpa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988105] Review Request: check-mssql-health - nagios check for mssql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988105

Marcus Asshauer fed...@asshaueronline.de changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Depends On||988102

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=SLw9uAGmMRa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988105] Review Request: check-mssql-health - nagios check for mssql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988105

Björn Esser bjoern.es...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bjoern.es...@gmail.com
   Assignee|nob...@fedoraproject.org|bjoern.es...@gmail.com
  Alias||check-mssql-health
  Flags||fedora-review?

--- Comment #1 from Björn Esser bjoern.es...@gmail.com ---
I'll take this one.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=UjEVcvvgKka=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

Björn Esser bjoern.es...@gmail.com changed:

   What|Removed |Added

  Alias||perl-DBD-Sybase

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=2IzdRJf9Aya=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

--- Comment #2 from Björn Esser bjoern.es...@gmail.com ---
spec inside srpm differs from the one provided by link.  Rebuild your srpm,
please, and re-upload.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=vr2j4cJMDca=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

--- Comment #3 from Marcus Asshauer fed...@asshaueronline.de ---
rebuild and re-upload done

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=cfDGYVgBDha=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 974145] Review Request: bluez5 - Bluetooth v5 utilities

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=974145

Don Zickus dzic...@redhat.com changed:

   What|Removed |Added

  Flags||needinfo?(kevin@tigcc.tical
   ||c.org)

--- Comment #15 from Don Zickus dzic...@redhat.com ---
(In reply to Kevin Kofler from comment #14)
 Use Obsoletes for that.

How would you obsolete packages that use bluez4?  Isn't that a rough guess at
best?

My understanding is that the API change will break all the bluez4 apps.  How do
you manage that from a packaging perspective?  You need to block the upgrade if
any package depends on bluez4 and does _not_ have a bluez5 component to migrate
too.

Having a separate bluez5 package makes that simpler to handle.  I am not sure
what rpm magic we can add to keep it inside one package.  But I defer to those
who have been here before.  Perhaps obsoletes can work, I just can't see how.

Cheers,
Don

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=0plngVwhtKa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

--- Comment #4 from Marcus Asshauer fed...@asshaueronline.de ---
http://mcas.fedorapeople.org/review/perl-DBD-Sybase-1.15-1.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=8DDl7UIQwKa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

--- Comment #5 from Marcus Asshauer fed...@asshaueronline.de ---
Spec URL: http://mcas.fedorapeople.org/review/perl-DBD-Sybase.spec
SRPM URL:
http://mcas.fedorapeople.org/review/perl-DBD-Sybase-1.15-1.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ZlOQ7leHBMa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 985087] Review Request: hadoop - A software platform for processing vast amounts of data

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985087

--- Comment #8 from Robert Rati rr...@redhat.com ---
Spec URL: http://rrati.fedorapeople.org/hadoop.spec
SRPM URL: http://rrati.fedorapeople.org/hadoop-2.0.5-2.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=GKIKKW2eaPa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 980960] Review Request: rpmgrill - utility for catching problems in koji builds

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=980960

Ed Santiago santi...@redhat.com changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=6rDZlOBRBKa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 658754] Review Request: cubrid - a very fast and reliable open source SQL database server

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=658754

--- Comment #46 from Michael Schwendt bugs.mich...@gmx.net ---
 an RPM building error no build id note found in for cub_cmhttpd.

That ought to be explained in the spec file, at least:

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

Try to find out why you get that error? Does the build run ld instead of
gcc/g++ when linking? Then try adding --build-id to the linker flags or run the
compiler instead of ld.


 Which are the other 64-bit targets you are referring to?

Secondary 64-bit archs, such as ppc64, AArch64.


 The source may have empty files (for example some optional
 configuration files). This was to remove warnings and possible
 empty files from the source.

In the source or in the %buildroot? What empty optional configuration files
get installed into the %buildroot only to delete them afterwards? It would be
safer to delete them explicitly.


 if you have any suggestion to avoid name conflicts, please let us know.

On rather simple way is to put headers into a subdirectory of %includedir.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=lrI52hvNMwa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=988102

--- Comment #6 from Björn Esser bjoern.es...@gmail.com ---
No issues found on this. :)

#

Package Review
==

Legend:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed



= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
 Note: Unversioned so-files in private %_libdir subdirectory (see
 attachment). Verify they are not in ld path.

 --- This is intentional on c-compiled perl-extensions

[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Package complies to the Packaging Guidelines
[-]: 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 is included in %doc.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 Unknown or generated. 9 files have unknown license. Detailed output of
 licensecheck in /home/besser82/shared/fedora/review/988102-perl-DBD-
 Sybase/licensecheck.txt

 --- License is derived from perl.  License-tag is fine

[x]: Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: If the package is under multiple licenses, the licensing breakdown must
 be documented in the spec.
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[-]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 102400 bytes in 10 files.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Each %files section contains %defattr if rpm  4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Fully versioned dependency in subpackages, if present.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.

Perl:
[x]: Package contains the mandatory BuildRequires and Requires:.
[x]: CPAN urls should be non-versioned.

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[-]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 

[Bug 985087] Review Request: hadoop - A software platform for processing vast amounts of data

2013-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=985087

--- Comment #9 from Robert Rati rr...@redhat.com ---
Spec URL: http://rrati.fedorapeople.org/hadoop.spec
SRPM URL: http://rrati.fedorapeople.org/hadoop-2.0.5-3.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=MzFVDUnVgQa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

  1   2   >