[Bug 829015] Review Request: python-nose-cover3 - Coverage 3.x support for Nose

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829015

Bohuslav Slavek Kabrda bkab...@redhat.com changed:

   What|Removed |Added

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

--- Comment #1 from Bohuslav Slavek Kabrda bkab...@redhat.com ---
I'll take this one for review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832178] Review Request: python-django-picklefield - Implementation of a pickled object field

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832178

Bohuslav Slavek Kabrda bkab...@redhat.com changed:

   What|Removed |Added

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

--- Comment #1 from Bohuslav Slavek Kabrda bkab...@redhat.com ---
I'll take this one for a review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823337] Review Request: rubygem-mixlib-shellout - mixin for running external commands

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823337

--- Comment #9 from Jonas Courteau r...@courteau.org ---
Ah, I didn't know it was okay to patch stuff in %check.  I've made the
recommended changes:
- now extracting and patching tests in %check
- tests are extracted, patched, and run from .%{gem_instdir} instead of
buildroot.
- excluding %{gem_cache}

Updated SRPM -
http://rpms.courteau.org/fedora/rubygem-mixlib-shellout-1.0.0-3.fc17.src.rpm
Updated SPEC -
https://raw.github.com/jcourteau/rubygems-rpms/master/fc17/rubygem-mixlib-shellout/rubygem-mixlib-shellout.spec

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 829015] Review Request: python-nose-cover3 - Coverage 3.x support for Nose

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829015

Bohuslav Slavek Kabrda bkab...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

--- Comment #2 from Bohuslav Slavek Kabrda bkab...@redhat.com ---
Everything works fine and the specfile is correct and clean. This package is
APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

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

--- Comment #1 from Mikolaj Izdebski mizde...@redhat.com ---
I am taking this review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832178] Review Request: python-django-picklefield - Implementation of a pickled object field

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832178

Bohuslav Slavek Kabrda bkab...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

--- Comment #2 from Bohuslav Slavek Kabrda bkab...@redhat.com ---
The only thing I would advise here is adding runtime dependency on Django
(Requires: python-django), as the package needs Django to do something.

Otherwise I see no problems = The package is APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

--- Comment #2 from Mikolaj Izdebski mizde...@redhat.com ---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated


 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[x]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[!]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm  4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[x]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[-]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[!]: MUST Rpmlint output is silent.
[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
Package has no sources or they are generated by developer
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[-]: MUST Useful -debuginfo package or justification otherwise.
[x]: SHOULD Reviewer should test that the package builds in mock.
[?]: SHOULD 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]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[!]: SHOULD Package functions as described.
[x]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD Patches link to upstream bugs/comments/lists or are otherwise
 justified.
[!]: SHOULD SourceX / PatchY prefixed with %{name}.
 Note: Source0: felix-osgi-obr-resolver-%{namedversion}.tar.xz (felix-
 osgi-obr-resolver-%{namedversion}.tar.xz) Patch0: 0001-Use-felix-
 directly.patch (0001-Use-felix-directly.patch) Patch1:
 0002-JDK7-support.patch (0002-JDK7-support.patch)
[x]: SHOULD SourceX is a working URL.
[x]: SHOULD Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[?]: SHOULD Package should compile and build into binary rpms on all supported
 architectures.
[-]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve timestamps of original installed
 files.
[x]: SHOULD Spec use %global instead of %define.


 Java 
[x]: MUST If source tarball includes bundled jar/class files these need to be
 removed prior to building
[x]: MUST Packages have proper BuildRequires/Requires on 

[Bug 826979] Review Request: pairs - Memory and logic stimulation game for pre-school children

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826979

Lukáš Tinkl lti...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

--- Comment #8 from Lukáš Tinkl lti...@redhat.com ---
Remaining problem solved, good to go

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

--- Comment #3 from Marek Goldmann mgold...@redhat.com ---
Fixed both issues:

Spec URL:
http://goldmann.fedorapeople.org/package_review/felix-osgi-obr-resolver/0.1.0-0.2.Beta1/felix-osgi-obr-resolver.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/felix-osgi-obr-resolver/0.1.0-0.2.Beta1/felix-osgi-obr-resolver-0.1.0-0.2.Beta1.fc17.src.rpm

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832178] Review Request: python-django-picklefield - Implementation of a pickled object field

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832178

Matthias Runge mru...@matthias-runge.de changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #3 from Matthias Runge mru...@matthias-runge.de ---
thank you for the review!

I'll fix the requirement before submitting to scm.

New Package SCM Request
===
Package Name: python-django-picklefield
Short Description: Implementation of a pickled object field
Owners: mrunge
Branches: devel

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 822046] Review Request: exfat - Free exFAT file system implementation

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=822046

Dan Mashal dan.mas...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
  Flags||needinfo?(tcallawa@redhat.c
   ||om)

--- Comment #6 from Dan Mashal dan.mas...@gmail.com ---
any update on this spot?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 822046] Review Request: exfat - Free exFAT file system implementation

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=822046

Dan Mashal dan.mas...@gmail.com changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|tcall...@redhat.com

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 829015] Review Request: python-nose-cover3 - Coverage 3.x support for Nose

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829015

Matthias Runge mru...@matthias-runge.de changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #3 from Matthias Runge mru...@matthias-runge.de ---
Thank you for the review!

New Package SCM Request
===
Package Name: python-nose-cover3
Short Description: Coverage 3.x support for Nose
Owners: mrunge
Branches: f17 devel

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 822046] Review Request: exfat - Free exFAT file system implementation

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=822046

Dan Mashal dan.mas...@gmail.com changed:

   What|Removed |Added

 QA Contact|extras...@fedoraproject.org |dan.mas...@gmail.com

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

--- Comment #4 from Marek Goldmann mgold...@redhat.com ---
New scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4173268

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+

--- Comment #5 from Mikolaj Izdebski mizde...@redhat.com ---
It's OK now.

**
** APPROVED **
**

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

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

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #6 from Marek Goldmann mgold...@redhat.com ---
Thanks!

New Package SCM Request
===
Package Name: felix-osgi-obr-resolver
Short Description: Apache Felix Resolver
Owners: goldmann
Branches: f17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832953] New: Review Request: Syntastic

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832953

Bug ID: 832953
QA Contact: extras...@fedoraproject.org
  Severity: unspecified
   Version: 17
  Priority: unspecified
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: Syntastic
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Unspecified
  Reporter: bioinfornat...@gmail.com
  Type: Bug
 Documentation: ---
  Hardware: Unspecified
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Dear,


I have do a package for a great vim plugin named Syntastic

SRPMS:
http://bioinfornatics.fedorapeople.org/syntastic-2.3.0-1.20120617git1e94b98.fc17.src.rpm

SPEC: http://bioinfornatics.fedorapeople.org/syntastic.spec

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830159] Review Request: ghc-smallcheck - A testing library to verify properties of test cases

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830159

Shakthi Kannan shakthim...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|RELEASE_PENDING
  Flags||fedora-cvs?

--- Comment #3 from Shakthi Kannan shakthim...@gmail.com ---
Thanks for the review.

New Package SCM Request
===
Package Name: ghc-smallcheck
Short Description: A testing library to verify properties of test cases
Owners: shakthimaan
Branches: f16 f17
InitialCC: haskell-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 829402] Review Request: jbosgi-vfs - JBoss OSGi Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829402

Patryk Obara pob...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

--- Comment #5 from Patryk Obara pob...@redhat.com ---

*** APPROVED ***


-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821712] Review Request: apache-commons-vfs - Commons Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821712

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

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

--- Comment #1 from Mikolaj Izdebski mizde...@redhat.com ---
I taking this review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 829402] Review Request: jbosgi-vfs - JBoss OSGi Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829402

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

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #6 from Marek Goldmann mgold...@redhat.com ---
New Package SCM Request
===
Package Name: jbosgi-vfs
Short Description: JBoss OSGi Virtual File System
Owners: goldmann
Branches: f17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832999] New: Review Request: colord-gtk - GTK support library for colord

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832999

Bug ID: 832999
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: colord-gtk - GTK support library for
colord
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: rhug...@redhat.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://people.freedesktop.org/~hughsient/temp/colord-gtk.spec
SRPM URL:
http://people.freedesktop.org/~hughsient/temp/colord-gtk-0.1.1-1.fc17.src.rpm
Koji URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=4173465
Description: colord is a support library for colord and provides additional
functionality that requires GTK+.
Fedora Account System Username: rhughes

$ rpmlint */colord-gtk*
4 packages and 1 specfiles checked; 0 errors, 0 warnings.

Note: colord-gtk.so used to be shipped in the main colord package, but this
made it hard to rebuild a distro (e.g. for ARM) as there was a dependency loop
of colord-GTK and GTK-colord. Now we can build in the order: colord, gtk,
colord-gtk, gnome-color-manager, etc

Please review. I don't think I need to provide any conflicts for previous
versions of colord, as nothing requires colord-gtk that's been built for F18 or
older (colord-gtk is pretty new). If you think it's safer to just add a
Conflicts: colord = 0.1.22 then yell. Thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833004] New: Review Request: pyp2rpm - Convert Python packages to RPM SPECFILES

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833004

Bug ID: 833004
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: pyp2rpm - Convert Python packages to
RPM SPECFILES
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: bkab...@redhat.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://bkabrda.fedorapeople.org/pkgs/pyp2rpm/pyp2rpm.spec
SRPM URL:
http://bkabrda.fedorapeople.org/pkgs/pyp2rpm/pyp2rpm-0.5.1-1.fc17.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4173552
Description:
Convert Python packages to RPM SPECFILES. The packages can be downloaded from
PyPI and the produced SPEC is in line with Fedora Packaging Guidelines.

Users can provide their own templates for rendering the package metadata. Both
the package source and metadata can be extracted from PyPI or from local
filesystem (local file doesn't provide that much information though).

Fedora Account System Username: bkabrda

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833004] Review Request: pyp2rpm - Convert Python packages to RPM SPECFILES

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833004

Matthias Runge mru...@matthias-runge.de changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mru...@matthias-runge.de
   Assignee|nob...@fedoraproject.org|mru...@matthias-runge.de
  Flags||fedora-review?

--- Comment #1 from Matthias Runge mru...@matthias-runge.de ---
I'll do the review

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821712] Review Request: apache-commons-vfs - Commons Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821712

--- Comment #2 from Mikolaj Izdebski mizde...@redhat.com ---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated


 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[x]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm  4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[x]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[!]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.

rpmlint apache-commons-vfs-javadoc-2.0-1.fc18.noarch.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.


rpmlint apache-commons-vfs-2.0-1.fc18.src.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.


rpmlint apache-commons-vfs-examples-2.0-1.fc18.noarch.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.


rpmlint apache-commons-vfs-2.0-1.fc18.noarch.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.


[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[-]: MUST Useful -debuginfo package or justification otherwise.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD 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]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD Patches link to upstream bugs/comments/lists or are otherwise
 justified.
[!]: SHOULD SourceX / PatchY prefixed with %{name}.
 Note: Source0: http://www.apache.org/dist/commons/vfs/source/commons-
 vfs-2.0-src.tar.gz (commons-vfs-2.0-src.tar.gz) Patch0: apache-commons-
 vfs-2.0-fixbuild.patch (apache-commons-vfs-2.0-fixbuild.patch)
[x]: SHOULD SourceX is a working URL.
[x]: SHOULD Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[?]: SHOULD Package should compile and build into binary rpms on all supported
 architectures.
[-]: SHOULD %check is 

[Bug 821712] Review Request: apache-commons-vfs - Commons Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821712

--- Comment #3 from gil cattaneo punto...@libero.it ---
Spec URL:
http://gil.fedorapeople.org/apache-commons-vfs/1/apache-commons-vfs.spec
SRPM URL:
http://gil.fedorapeople.org/apache-commons-vfs/1/apache-commons-vfs-2.0-2.fc16.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821712] Review Request: apache-commons-vfs - Commons Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821712

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+

--- Comment #4 from Mikolaj Izdebski mizde...@redhat.com ---
It's good now. Thank you.

Tested on: http://koji.fedoraproject.org/koji/taskinfo?taskID=4173625

**
** APPROVED **
**

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821712] Review Request: apache-commons-vfs - Commons Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821712

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

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from gil cattaneo punto...@libero.it ---
New Package SCM Request
===
Package Name: apache-commons-vfs
Short Description: Commons Virtual File System
Owners: gil
Branches: f17
InitialCC: java-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 813832] Review Request: linuxdcpp - linux port of dc++

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=813832

Peter Hatina phat...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #7 from Peter Hatina phat...@redhat.com ---
Package Change Request
==
Package Name: linuxdcpp
New Branches: f16 f17
Owners: phatina

Unretire master, please.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 813832] Review Request: linuxdcpp - linux port of dc++

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=813832

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

Unretired.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821712] Review Request: apache-commons-vfs - Commons Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821712

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

[Bug 829015] Review Request: python-nose-cover3 - Coverage 3.x support for Nose

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829015

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

[Bug 829402] Review Request: jbosgi-vfs - JBoss OSGi Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829402

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

[Bug 830159] Review Request: ghc-smallcheck - A testing library to verify properties of test cases

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830159

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

[Bug 830221] Review Request: perl-Library-CallNumber-LC - Normalizes Library of Congress call numbers for sorting

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830221

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

[Bug 832178] Review Request: python-django-picklefield - Implementation of a pickled object field

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832178

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

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

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

[Bug 830361] Review Request: juk - Music player

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830361

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2012-06-18 09:23:28

--- Comment #10 from Rex Dieter rdie...@math.unl.edu ---
imported

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 829402] Review Request: jbosgi-vfs - JBoss OSGi Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829402

--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
jbosgi-vfs-1.1.2-0.3.CR1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/jbosgi-vfs-1.1.2-0.3.CR1.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 829402] Review Request: jbosgi-vfs - JBoss OSGi Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829402

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

[Bug 830125] Review Request: jbosgi-spi - JBossOSGi SPI

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830125

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mizde...@redhat.com
   Assignee|pob...@redhat.com   |mizde...@redhat.com

--- Comment #1 from Mikolaj Izdebski mizde...@redhat.com ---
I am reassigning this bug to myself.
This is quire urgent rewiew so Marek asked me to take.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821712] Review Request: apache-commons-vfs - Commons Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821712

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

   What|Removed |Added

 Status|POST|MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821712] Review Request: apache-commons-vfs - Commons Virtual File System

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821712

--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
apache-commons-vfs-2.0-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/apache-commons-vfs-2.0-2.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

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

   What|Removed |Added

 Status|POST|MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832420] Review Request: felix-osgi-obr-resolver - Apache Felix Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832420

--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
felix-osgi-obr-resolver-0.1.0-0.2.Beta1.fc17 has been submitted as an update
for Fedora 17.
https://admin.fedoraproject.org/updates/felix-osgi-obr-resolver-0.1.0-0.2.Beta1.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826979] Review Request: pairs - Memory and logic stimulation game for preschool children

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826979

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

   What|Removed |Added

Summary|Review Request: pairs - |Review Request: pairs -
   |Memory and logic|Memory and logic
   |stimulation game for|stimulation game for
   |pre-school children |preschool children
  Flags||fedora-cvs?

--- Comment #9 from Jaroslav Reznik jrez...@redhat.com ---
New Package SCM Request
===
Package Name: pairs
Short Description: Memory and logic stimulation game for preschool children
Owners: jreznik than ltinkl rdieter kkofler rnovacek
Branches: f17
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 829116] Review Request: ninja-build - A small build system with a focus on speed

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=829116

--- Comment #6 from Matthew Woehlke mw_tr...@users.sourceforge.net ---
Created attachment 592653
  -- https://bugzilla.redhat.com/attachment.cgi?id=592653action=edit
patch to fix vim auto-syntax

So, it's not too hard to fix the previous point; necessary vim commands
attached. I put this in vimfiles/ftdetect, but not sure if that's the
appropriate place (maybe it should be part of syntax/ninja.vim?).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826979] Review Request: pairs - Memory and logic stimulation game for preschool children

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826979

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

[Bug 832953] Review Request: Syntastic

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832953

--- Comment #1 from MERCIER Jonathan bioinfornat...@gmail.com ---
update spec for easy subpackage maintenance:

http://bioinfornatics.fedorapeople.org/syntastic-2.3.0-2.20120617git1e94b98.fc17.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830125] Review Request: jbosgi-spi - JBossOSGi SPI

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830125

--- Comment #2 from Mikolaj Izdebski mizde...@redhat.com ---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated


 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[x]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm  4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[!]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[x]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[-]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[!]: MUST Rpmlint output is silent.

rpmlint jbosgi-spi-javadoc-3.0.1-1.fc18.noarch.rpm

jbosgi-spi-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs - Java
docs, Java-docs, Avocados
jbosgi-spi-javadoc.noarch: W: invalid-url URL: http://www.jboss.org/ HTTP Error
403: Forbidden
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


rpmlint jbosgi-spi-3.0.1-1.fc18.src.rpm

jbosgi-spi.src: W: spelling-error %description -l en_US pakcage - package
jbosgi-spi.src: W: invalid-url URL: http://www.jboss.org/ HTTP Error 403:
Forbidden
jbosgi-spi.src: W: invalid-url Source0: jbosgi-spi-3.0.1.Final.tar.xz
1 packages and 0 specfiles checked; 0 errors, 3 warnings.


rpmlint jbosgi-spi-3.0.1-1.fc18.noarch.rpm

jbosgi-spi.noarch: W: spelling-error %description -l en_US pakcage - package
jbosgi-spi.noarch: W: invalid-url URL: http://www.jboss.org/ HTTP Error 403:
Forbidden
jbosgi-spi.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

These warnings can be ignored.


[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
Package has no sources or they are generated by developer
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[-]: MUST Useful -debuginfo package or justification otherwise.
[x]: SHOULD Reviewer should test that the package builds in mock.
[?]: SHOULD 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]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[!]: SHOULD Package functions as described.
[!]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD Patches link to upstream bugs/comments/lists or are otherwise
 justified.

[Bug 830125] Review Request: jbosgi-spi - JBossOSGi SPI

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830125

--- Comment #3 from Marek Goldmann mgold...@redhat.com ---
I incorporated the change, without bumping the release and overriding the
previous packages.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830125] Review Request: jbosgi-spi - JBossOSGi SPI

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830125

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+

--- Comment #4 from Mikolaj Izdebski mizde...@redhat.com ---
It's OK now.

**
** APPROVED **
**

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830125] Review Request: jbosgi-spi - JBossOSGi SPI

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830125

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

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from Marek Goldmann mgold...@redhat.com ---
Thanks!

New Package SCM Request
===
Package Name: jbosgi-spi
Short Description: JBossOSGi SPI
Owners: goldmann
Branches: f17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830125] Review Request: jbosgi-spi - JBossOSGi SPI

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830125

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

[Bug 830277] Review Request: wmcoincoin - Funny dock-app for browsing and interact to XML board sites

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830277

--- Comment #10 from pierrejourda...@gmail.com ---
I 'm changing some lines in my  .spec file to follows the fedora package
guidlines I'm removed the useless %post %postun %posttrans lines . 
I'am also changed the .desktop file removing the encoding line (because it's
useless) .
Section %changlog is upated . 

http://pierre80.fedorapeople.org/SPECS/wmcoincoin.spec SPEC file 
http://pierre80.fedorapeople.org/SRPMS/wmcoincoin-2.5.1-5.f.fc17.src.rpm SRPM
file

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833115] New: Review Request: rawdog - RSS Aggregator Without Delusions of Grandeur

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833115

Bug ID: 833115
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: rawdog - RSS Aggregator Without
Delusions of Grandeur
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: agro...@redhat.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://fedorapeople.org/~grover/new/rawdog.spec
SRPM URL: http://fedorapeople.org/~grover/new/rawdog-2.13-1.fc17.src.rpm
Description: Rawdog aggregates RSS feeds to generate an HTML page, commonly
published as planets on the internet. An example (my personal planet) is
http://planet.groveronline.com/ .
Fedora Account System Username: grover

Note on packaging: Rawdog includes feedparser
(http://code.google.com/p/feedparser/) which is packaged in python-feedparser,
unfortunately Rawdog uses a significantly modified version so I thought it best
not to try to make it work with the upstream version.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832074] Review Request: python-tahrir-api - python module for commuicating with a tahrir database

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832074

Ross Delinger rossdy...@csh.rit.edu changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
  Flags||fedora-cvs?

--- Comment #6 from Ross Delinger rossdy...@csh.rit.edu ---
New Package SCM Request
===
Package Name: python-tahrir-api
Short Description: API for interacting with tahrir databases
Owners: rossdylan
Branches: f17 el6
InitialCC: rossdylan

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832074] Review Request: python-tahrir-api - python module for commuicating with a tahrir database

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832074

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

Luke, please take ownership of review BZs, thanks!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826955] Review Request: apache-commons-chain - An implementation of the GoF Chain of Responsibility pattern

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826955

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

   What|Removed |Added

 Status|POST|MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826955] Review Request: apache-commons-chain - An implementation of the GoF Chain of Responsibility pattern

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826955

--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
apache-commons-chain-1.2-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/apache-commons-chain-1.2-2.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 820344] Review Request: apache-commons-javaflow - Commons Javaflow

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=820344

--- Comment #2 from gil cattaneo punto...@libero.it ---
tested on: http://koji.fedoraproject.org/koji/taskinfo?taskID=4174731

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830125] Review Request: jbosgi-spi - JBossOSGi SPI

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830125

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

   What|Removed |Added

 Status|POST|MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830125] Review Request: jbosgi-spi - JBossOSGi SPI

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830125

--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
jbosgi-spi-3.0.1-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/jbosgi-spi-3.0.1-1.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821284] Review Request: dbunit - DbUnit Framework - extension for JUnit

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821284

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

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826955] Review Request: apache-commons-chain - An implementation of the GoF Chain of Responsibility pattern

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826955

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

   What|Removed |Added

 Blocks|652183 (FE-JAVASIG) |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 827761] vdr-skinenigmang - A skin for VDR based on the Enigma text2skin add on

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=827761

MartinKG mgans...@alice.de changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|ville.sky...@iki.fi |nob...@fedoraproject.org
  Flags|fedora-review+  |fedora-review?

--- Comment #8 from MartinKG mgans...@alice.de ---
New Package SCM Request
===
Package Name: vdr-skinenigmang
Short Description: a skin for VDR based on the Enigma text2skin add on.
Owners: martinkg
Branches: f16 f17 f18
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833154] New: Review Request: eclipse-wtp-jeetools - Frameworks and tools for Eclipse, focused on the development of J2EE artifacts

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833154

Bug ID: 833154
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: eclipse-wtp-jeetools - Frameworks and
tools for Eclipse, focused on the development of J2EE
artifacts
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: ger...@ryan.lt
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL:
http://galileo.fedorapeople.org/eclipse-wtp-jeetools/3.4.0/0.1.20120618cvs/eclipse-wtp-jeetools.spec

SRPM URL:
http://galileo.fedorapeople.org/eclipse-wtp-jeetools/3.4.0/0.1.20120618cvs/eclipse-wtp-jeetools-3.4.0-0.1.20120618cvs.fc17.src.rpm

Description: The Java EE Tools Project is a set of Eclipse plugins that
provides frameworks and tools focused on the development of J2EE artifacts.

Fedora Account System Username: galileo

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 827761] vdr-skinenigmang - A skin for VDR based on the Enigma text2skin add on

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=827761

MartinKG mgans...@alice.de changed:

   What|Removed |Added

  Flags||fedora-cvs?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833164] New: Review Request: python-pyudev-rpm - udev bindings for python

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833164

Bug ID: 833164
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: python-pyudev-rpm - udev bindings for
python
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: clock...@redhat.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL:
http://fedorapeople.org/~clockfort/python-pyudev-rpm/SPECS/python-pyudev.spec
SRPM URL:
http://fedorapeople.org/~clockfort/python-pyudev-rpm/SRPMS/python-pyudev-0.15-1.fc17.src.rpm
Description: udev bindings for python
Fedora Account System Username: clockfort

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830221] Review Request: perl-Library-CallNumber-LC - Normalizes Library of Congress call numbers for sorting

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830221

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

[Bug 830221] Review Request: perl-Library-CallNumber-LC - Normalizes Library of Congress call numbers for sorting

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830221

--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
perl-Library-CallNumber-LC-0.22-1.fc17 has been submitted as an update for
Fedora 17.
https://admin.fedoraproject.org/updates/perl-Library-CallNumber-LC-0.22-1.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 831871] Review Request: openstack-swift-plugin-swift3 - The swift3 plugin for Openstack Swift

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=831871

Pete Zaitcev zait...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2012-06-18 15:15:34

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 827761] vdr-skinenigmang - A skin for VDR based on the Enigma text2skin add on

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=827761

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|ville.sky...@iki.fi
  Flags|fedora-review?  |fedora-review+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 827761] Review Request: vdr-skinenigmang - A skin for VDR based on the Enigma text2skin add on

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=827761

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

   What|Removed |Added

Summary|vdr-skinenigmang - A skin   |Review Request:
   |for VDR based on the Enigma |vdr-skinenigmang - A skin
   |text2skin add on|for VDR based on the Enigma
   ||text2skin add on

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 717748] Review Request: UST - LTTng Userspace Tracer

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=717748

--- Comment #8 from Yannick Brosseau yannick.bross...@gmail.com ---
Updating this request with the latest LTTng-UST package.

SPEC: http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SPECS/lttng-ust.spec
SRPMS:
http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SRPMS/lttng-ust-2.0.3-1.fc17.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 717748] Review Request: LTTng-UST - LTTng Userspace Tracer

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=717748

Yannick Brosseau yannick.bross...@gmail.com changed:

   What|Removed |Added

Summary|Review Request: UST - LTTng |Review Request: LTTng-UST -
   |Userspace Tracer|LTTng Userspace Tracer

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832074] Review Request: python-tahrir-api - python module for commuicating with a tahrir database

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832074

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

   What|Removed |Added

 Status|NEW |MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832074] Review Request: python-tahrir-api - python module for commuicating with a tahrir database

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832074

--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
python-tahrir-api-0.1.3.1-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/python-tahrir-api-0.1.3.1-1.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832074] Review Request: python-tahrir-api - python module for commuicating with a tahrir database

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832074

--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
python-tahrir-api-0.1.3.1-1.el6 has been submitted as an update for Fedora EPEL
6.
https://admin.fedoraproject.org/updates/python-tahrir-api-0.1.3.1-1.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833204] New: Review Request: python-pyramid-tm - Allows pyramid requests to join the active transaction

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833204

Bug ID: 833204
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: python-pyramid-tm - Allows pyramid
requests to join the active transaction
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: rossdy...@csh.rit.edu
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL:
http://www.csh.rit.edu/~rossdylan/rpms/python-pyramid-tm/python-pyramid-tm.spec
SRPM URL:
http://www.csh.rit.edu/~rossdylan/rpms/python-pyramid-tm/python-pyramid-tm-0.4-1.fc17.src.rpm
Description: A package which allows Pyramid requests to join the active
transaction as provided by the python-transaction package.
Fedora Account System Username: rossdylan

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833204] Review Request: python-pyramid-tm - Allows pyramid requests to join the active transaction

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833204

Ross Delinger rossdy...@csh.rit.edu changed:

   What|Removed |Added

Version|rawhide |17

--- Comment #1 from Ross Delinger rossdy...@csh.rit.edu ---
Successful Koji build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4175173

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 822182] Review Request: python-websockify - Python proxy for the websockets protocol

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=822182

Pádraig Brady pbr...@redhat.com changed:

   What|Removed |Added

 Blocks||801685

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 822187] Review Request: novnc - websockets based VNC client and simple server

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=822187

Pádraig Brady pbr...@redhat.com changed:

   What|Removed |Added

 Blocks||801685

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833154] Review Request: eclipse-wtp-jeetools - Frameworks and tools for Eclipse, focused on the development of J2EE artifacts

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833154

--- Comment #1 from Gerard Ryan ger...@ryan.lt ---
Note: the following additional license files exist:

./plugins/org.eclipse.jst.j2ee/license/berkeley_license.html
./plugins/org.eclipse.jst.jee/license/berkeley_license.html

but I haven't been able to find any code licensed under anything besides the
EPL. I don't know why these files are here, and I haven't included them.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832074] Review Request: python-tahrir-api - python module for commuicating with a tahrir database

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832074

--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
python-tahrir-api-0.1.3.1-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/python-tahrir-api-0.1.3.1-1.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832074] Review Request: python-tahrir-api - python module for commuicating with a tahrir database

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832074

--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
python-tahrir-api-0.1.3.1-1.el6 has been submitted as an update for Fedora EPEL
6.
https://admin.fedoraproject.org/updates/python-tahrir-api-0.1.3.1-1.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 833226] New: Review Request: python-pycparser - C parser and AST generator written in Python

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=833226

Bug ID: 833226
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: python-pycparser - C parser and AST
generator written in Python
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: scottt...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://scottt.tw/fedora/python-pycparser/python-pycparser.spec
SRPM URL:
http://scottt.tw/fedora/python-pycparser/python-pycparser-2.07-1.fc17.src.rpm
Description:
pycparser is a complete parser for the C language, written in pure Python.
It is a module designed to be easily integrated into applications that
need to parse C source code.

Fedora Account System Username: scottt

rpmlint output:
rpmlint python-pycparser-2.07-1.fc18.src.rpm

python-pycparser.src: W: strange-permission remove-relative-sys-path 0755L
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

rpmlint python-pycparser-2.07-1.fc18.noarch.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Regarding the strange-permission warning, I think rpmlint is complaining
about  remove-relative-sys-path being listed a an source in the SRPM and
having executable permissions. I wrote that script to cleanup the sample code
in examples/ and the SPEC files runs it during %prep so I think this particular
rpmlint warning should be ignored.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 820115] Review Request: leptonica - C library for efficient image processing and image analysis operations

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=820115

Kevin Kofler ke...@tigcc.ticalc.org changed:

   What|Removed |Added

 CC||ke...@tigcc.ticalc.org

--- Comment #4 from Kevin Kofler ke...@tigcc.ticalc.org ---
I think those binaries really can't be packaged that way, without some kind of
namespacing (name prefix or whatever). Some of the names are extremely generic
and likely to cause conflicts. Others at least follow some naming scheme, but
there I'm not sure those are useful to package at all, e.g. tests probably
don't make sense to install.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 717748] Review Request: LTTng-UST - LTTng Userspace Tracer

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=717748

--- Comment #9 from Scott Tsai scottt...@gmail.com ---
(In reply to comment #8)

Building on F17 fails for me with:

/bin/sh ../../libtool  --tag=CC   --mode=link gcc  -Wall -O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector
--param=ssp-buffer-size=4  -m64 -mtune=generic  -Wl,-z,relro  -o hello hello.o
tp.o ../../liblttng-ust/liblttng-ust.la -lurcu-bp -lurcu-bp -luuid -lpthread
-ldl
libtool: link: DIE_RPATH_DIE=/usr/lib64: gcc -Wall -O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector
--param=ssp-buffer-size=4 -m64 -mtune=generic -Wl,-z -Wl,relro -o .libs/hello
hello.o tp.o  ../../liblttng-ust/.libs/liblttng-ust.so -lurcu-bp -luuid
-lpthread -ldl
/usr/bin/ld: warning: liblttng-ust-tracepoint.so.0, needed by
../../liblttng-ust/.libs/liblttng-ust.so, not found (try using -rpath or
-rpath-link)
../../liblttng-ust/.libs/liblttng-ust.so: undefined reference to
`init_tracepoint'
../../liblttng-ust/.libs/liblttng-ust.so: undefined reference to
`__tracepoint_probe_register'
../../liblttng-ust/.libs/liblttng-ust.so: undefined reference to
`exit_tracepoint'
../../liblttng-ust/.libs/liblttng-ust.so: undefined reference to
`__tracepoint_probe_unregister'
collect2: error: ld returned 1 exit status
make[3]: *** [hello] Error 1
make[3]: Leaving directory
`/home/scottt/work/lttng/fedora/lttng-ust-2.0.3/tests/hello'

Since:
1. ld is performing a non-shared, non relocatable link for .libs/hello 
2. We patched libtool to get rid of RPATH
3. hello.o requires liblttng-ust.so which in turn requires
liblttng-ust-tracepoint.so.0
We need to either pass the directory containing liblttng-ust-tracepoint.so.0 as
-rpath-link or specify liblttng-ust-tracepoint.so as an input to ld.

The following snippet in lttnt-ust.spec would accomplish the later:

+# link all dependent libs
+#sed -i 's|^link_all_deplibs=no|link_all_deplibs=unknown|g' libtool
+
+make %{?_smp_mflags} V=1

 %install
 make DESTDIR=%buildroot install

Yannick, could you build actually build without a patch to this effect?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 826563] Review Request: Fuel Manager - keep track of your fuel mileage and consumption

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=826563

--- Comment #2 from Nick Bebout n...@fedoraproject.org ---
Spec URL: http://fedorapeople.org/~kc8hfi/fuelmanager.spec
SRPM URL: http://fedorapeople.org/~kc8hfi/fuelmanager-0.3.6-5.fc15.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 810033] Review Request: python-virtualenvwrapper - Enhancements to virtualenv

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=810033

Ralph Bean rb...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?

--- Comment #16 from Ralph Bean rb...@redhat.com ---
Package Change Request
==
Package Name: python-virtualenvwrapper
New Branches: f16
Owners: ralph
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 717748] Review Request: LTTng-UST - LTTng Userspace Tracer

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=717748

--- Comment #10 from Yannick Brosseau yannick.bross...@gmail.com ---
I was able to build it without the patch, but now I'm trying with koji and
targeting F17 and it fail there. I'll investigate and post a new version.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830750] Review Request: jbosgi-resolver1 - Standalone OSGi Resolver

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830750

Satya Komaragiri skoma...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||skoma...@redhat.com
   Assignee|nob...@fedoraproject.org|skoma...@redhat.com

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 717748] Review Request: LTTng-UST - LTTng Userspace Tracer

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=717748

--- Comment #11 from Yannick Brosseau yannick.bross...@gmail.com ---
(In reply to comment #9)
 (In reply to comment #8)
 
 Building on F17 fails for me with:
 

 +# link all dependent libs
 +#sed -i 's|^link_all_deplibs=no|link_all_deplibs=unknown|g' libtool
 +
 +make %{?_smp_mflags} V=1
 
  %install
  make DESTDIR=%buildroot install
 
 Yannick, could you build actually build without a patch to this effect?

I've updated the files, can you try again?
Now the package reinitialize the libtool script instead of doing a series of
sed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821267] Review Request: R-BiocGenerics - Generic functions for Bioconductor

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821267

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

[Bug 821267] Review Request: R-BiocGenerics - Generic functions for Bioconductor

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821267

--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
R-BiocGenerics-0.2.0-2.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/R-BiocGenerics-0.2.0-2.fc16

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 821267] Review Request: R-BiocGenerics - Generic functions for Bioconductor

2012-06-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821267

--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
R-BiocGenerics-0.2.0-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/R-BiocGenerics-0.2.0-2.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review