[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #23 from Vít Ondruch vondr...@redhat.com 2011-07-20 02:39:12 EDT 
---
Mamoru, what about using rubygem-rdoc as default for rubygems instead of rdoc?
We could also think about marking ruby-rdoc and rubygem-rdoc as conflicting and
rubygems then pulls in rubygem-rdoc instead of ruby-rdoc.

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

[Bug 723427] New: Review Request: jinput - Java Game Controller API

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: jinput - Java Game Controller API

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

   Summary: Review Request: jinput - Java Game Controller API
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: guido.grazi...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---


Spec URL: 
http://guidograzioli.fedorapeople.org/packages/jinput/jinput.spec
SRPM URL: 
http://guidograzioli.fedorapeople.org/packages/jinput/jinput-2.0.6-1.20110719svn.fc16.src.rpm
Description: 
jinput is an implementation of an API for game controller discovery and polled
input. It is part of a suite of open-source technologies initiated by the
Game Technology Group at Sun Microsystems with intention of making the
development of high performance games in Java a reality.
The API itself is pure Java and presents a platform-neutral completely
portable model of controller discovery and polling. It can handle arbitrary
controllers and returns both human and machine understandable descriptions of
the inputs available.

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


[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #24 from Vít Ondruch vondr...@redhat.com 2011-07-20 02:46:16 EDT 
---
May be the gem could also provide ruby(rdoc).

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

[Bug 723125] Review Request: jutils - Common utilities for the Java Gaming Interface

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Guido Grazioli guido.grazi...@gmail.com changed:

   What|Removed |Added

 Blocks||723427

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


[Bug 723427] Review Request: jinput - Java Game Controller API

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Guido Grazioli guido.grazi...@gmail.com changed:

   What|Removed |Added

 Blocks||652183(FE-JAVASIG)
 Depends on||723125

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


[Bug 719469] Review Request: kbsh - command-line shell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Zack Parsons k3ba...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks|177841(FE-NEEDSPONSOR)  |
 Resolution||DEFERRED
Last Closed||2011-07-20 03:50:47

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


[Bug 690038] Review Request: ompl - The Open Motion Planning Library

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de
   Flag||fedora-review?

--- Comment #7 from Martin Gieseking martin.giesek...@uos.de 2011-07-20 
03:49:34 EDT ---
Sorry for the delay. I've been rather busy in the last few month.
You're right, I somehow mixed up the definitions of soname and soversion. I'm
sorry for that too.

Your latest package currently fails building because of missing BR: python:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3213719

Everything else looks fine.


$ rpmlint ompl-*.rpm
4 packages and 0 specfiles checked; 0 errors, 0 warnings.
-
key:

[+] OK
[.] OK, not applicable
[X] needs work
-

[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license.
- BSD (3-clause variant)

[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: The file containing the text of the license(s) for the package must
be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
$ md5sum ompl-0.9.3-Source.tar.gz*
b95b4b3a6eebc15a668a52a133384cc1  ompl-0.9.3-Source.tar.gz
b95b4b3a6eebc15a668a52a133384cc1  ompl-0.9.3-Source.tar.gz.1

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[X] MUST: All build dependencies must be listed in BuildRequires.
BR: python is missing

[+] MUST: When compiling C, C++, or Fortran files, %{optflags} must be applied.
[.] MUST: The spec file MUST handle locales properly.
[+] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
[+] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, ...
[+] MUST: A package must own all directories that it creates. 
[+] MUST: A Fedora package must not list a file more than once in %files.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
[+] MUST: Files in %doc must not affect the runtime of the application.
[+] MUST: Header files must be in a -devel package.
[.] MUST: Static libraries must be in a -static package.
[+] MUST: library files that end in .so (without suffix) must go in a -devel
package.
[+] MUST: devel packages must require the base package using a fully versioned
dependency.
[+] MUST: Packages must NOT contain any .la libtool archives.
[.] MUST: Packages containing GUI applications must include a %{name}.desktop
file.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: All filenames in rpm packages must be valid UTF-8.

[.] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
[+] SHOULD: Patch files should be prefixed with %{name}-
[+] SHOULD: All patches should be commented in the spec file
[+] SHOULD: The reviewer should test that the package builds in mock.
[+] SHOULD: The reviewer should test that the package functions as described.
[+] SHOULD: If scriptlets are used, those scriptlets must be sane.
[.] SHOULD: Usually, subpackages other than devel should require the base
package using a fully versioned dependency.
[.] SHOULD: pkgconfig(.pc) files should be placed in a -devel pkg.
[.] SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin,
/usr/bin, or /usr/sbin ...
[.] SHOULD: Your package should contain man pages for binaries/scripts.

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #17 from Jens Petersen peter...@redhat.com 2011-07-20 03:58:32 
EDT ---
Created attachment 513944
  -- https://bugzilla.redhat.com/attachment.cgi?id=513944
leksah.spec-2.patch

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


[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #25 from Mamoru Tasaka mtas...@fedoraproject.org 2011-07-20 
04:31:28 EDT ---
(In reply to comment #23)
 Mamoru, what about using rubygem-rdoc as default for rubygems instead of rdoc?

This sounds reasonable, but:

 We could also think about marking ruby-rdoc and rubygem-rdoc as conflicting 
 and
 rubygems then pulls in rubygem-rdoc instead of ruby-rdoc.

This conflict usage is _forbidden_ on Fedora's packaging policy.
https://fedoraproject.org/wiki/Packaging/Conflicts
But for some exceptions, using conflicts needs FPC's approval and I don't think
FPC would approve this case.

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


[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #26 from Vít Ondruch vondr...@redhat.com 2011-07-20 04:44:05 EDT 
---
(In reply to comment #25)
  We could also think about marking ruby-rdoc and rubygem-rdoc as conflicting 
  and
  rubygems then pulls in rubygem-rdoc instead of ruby-rdoc.
 
 This conflict usage is _forbidden_ on Fedora's packaging policy.
 https://fedoraproject.org/wiki/Packaging/Conflicts
 But for some exceptions, using conflicts needs FPC's approval and I don't 
 think
 FPC would approve this case.

I am aware of this condition. I'll be thinking about it. May be somebody will
help to crystallize this idea ;)

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

[Bug 721144] Review Request: imagefactory - System image generation tool

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Mark McLoughlin mar...@redhat.com 2011-07-20 04:46:51 EDT 
---
(In reply to comment #2)
 (In reply to comment #1)
   - I don't think euca2ools is actually required; we run all the euca-
 commands inside EC2 instances not locally
 
 When we are doing upload or local style builds, the euca2ools are actually
 required to do the bundling.  See imagefactory/builders/FedoraBuilder.py,
 around lines 1383.

Ah, of course - right

I was just about to approve this, but then I remembered to run rpmlint on the
actual RPM itself:

$ rpmlint imagefactory-0.3.1-2.fc14.noarch.rpm
imagefactory.noarch: E: explicit-lib-dependency python-httplib2
imagefactory.noarch: W: conffile-without-noreplace-flag
/etc/pki/imagefactory/cert-ec2.pem
imagefactory.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/imagefactory/ApplicationConfiguration.py 0644L
/usr/bin/env
imagefactory.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/imagefactory/Template.py 0644L /usr/bin/env
imagefactory.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/imagefactory/ImageWarehouse.py 0644L
/usr/bin/env
imagefactory.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/imagefactory/VMWare.py 0644L /usr/bin/env
imagefactory.noarch: W: no-manual-page-for-binary imgfac.py
1 packages and 0 specfiles checked; 5 errors, 2 warnings.


Re: explicit-lib-dependency - this looks like a false positive to me. See
http://lists.fedoraproject.org/pipermail/devel/2010-March/133955.html

Re: conffile-without-noreplace-flag - do we allow users to replace this with
their own cert? If so, the error is valid.

Re: non-executable-script - do sed -i '/\/usr\/bin\/env python/d'
imagefactory/*.py

Re: no-manual-page-for-binary - there actually is a man page, but not with the
same name as the binary. Do we really want the binary to be installed as
imgfac.py ?

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


[Bug 677496] Review Request: nbc - Next Byte Codes (NBC) and Not Exactly C (NXC)

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jon Ciesla l...@jcomserv.net 2011-07-20 06:03:25 EDT ---
Git done (by process-git-requests).

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


[Bug 720085] Review Request: perl-Net-FTP-AutoReconnect - FTP client class with automatic reconnect on failure

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Jon Ciesla l...@jcomserv.net 2011-07-20 06:03:54 EDT ---
Git done (by process-git-requests).

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


[Bug 720086] Review Request: perl-Net-FTP-RetrHandle - Provides a file reading interface for reading files on a remote FTP server

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Jon Ciesla l...@jcomserv.net 2011-07-20 06:04:17 EDT ---
Git done (by process-git-requests).

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


[Bug 723472] New: Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-aws-sdk - AWS SDK for Ruby

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

   Summary: Review Request: rubygem-aws-sdk - AWS SDK for Ruby
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mgold...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---


Spec URL:
http://goldmann.fedorapeople.org/package_review/rubygem-aws-sdk/1/rubygem-aws-sdk.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/rubygem-aws-sdk/1/rubygem-aws-sdk-1.0.1-1.fc15.src.rpm

Description:

Build Ruby applications with the aws-sdk gem that takes the complexity
out of coding directly against the web service interfaces. The gem handles
common tasks, such as authentication, request retries, XML processing,
error handling, and more.

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

$ rpmlint ~/rpmbuild/SRPMS/rubygem-aws-sdk-1.0.1-1.fc15.src.rpm 
rubygem-aws-sdk.src: I: enchant-dictionary-not-found en_US
rubygem-aws-sdk.src: W: no-%build-section
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint ./rubygem-aws-sdk.spec 
./rubygem-aws-sdk.spec: W: no-%build-section
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

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


[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 CC||mfoj...@redhat.com
   Flag||fedora-review?

--- Comment #1 from Michal Fojtik mfoj...@redhat.com 2011-07-20 06:13:30 EDT 
---
Taking this for a review.

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


[Bug 706832] Review Request: hibernate-commons-annotations - Hibernate Annotations

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-07-20 06:19:09

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


[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Michal Fojtik mfoj...@redhat.com 2011-07-20 06:23:37 EDT 
---
[ OK ] Packages that contain Ruby Gems must be called rubygem-%{gemname} where
gemname

[ OK ] The Source of the package must be the full URL to the released Gem
archive; the version of the package must be the Gem's version

[ OK ] Package has proper license (ASL 2.0)

[ OK ] The package must have a Requires and a BuildRequires on rubygems

[ OK? ] The %prep and %build sections of the specfile should be empty.

%build section is missing, but I though it's OK, since it's not required.

[ OK ] The Gem must be installed into %{gemdir} defined as %global gemdir
%(ruby -rubygems -e 'puts Gem::dir' 2/dev/null)

[ OK ] The package must own the following files and directories:

%{gemdir}/gems/%{gemname}-%{version}/
%{gemdir}/cache/%{gemname}-%{version}.gem
%{gemdir}/specifications/%{gemname}-%{version}.gemspec

[ OK] Architecture-specific content must not be installed into %{gemdir}

There is no architecture specific content.

[ OK] If the Gem only contains pure Ruby code, it must be marked as BuildArch:
noarch.

Marked properly.

[ OK ] If the Gem contains  binary content (e.g., for a database driver), it
must be marked as architecture specific, and all architecture specific content
must be moved from the %{gemdir} to the [#ruby_sitearch %{ruby_sitearch}
directory] during %install

This gem include only SSL certificate, which is not architecture specific.


All clear to go so APPROVED. Nice work Marek!

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


[Bug 722914] Review Request: volumeicon - Lightweight volume control for the system tray

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Veeti Paananen veeti.paana...@rojekti.fi 2011-07-20 
06:21:18 EDT ---
 Oh? For me it gave GPLv3 straight away.

If licensecheck (and the headers) say that the license is GPLv3, isn't the
GPLv3+ in the spec incorrect?

 This is nitpicking. But in principle you may be correct.

While the spelling error definitely wouldn't block a review, I thought I'd
point it out.

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


[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

  QAContact|extras...@fedoraproject.org |mfoj...@redhat.com

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


[Bug 642592] Review Request: rubygem-macaddr - Cross platform mac address determination for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #15 from Vít Ondruch vondr...@redhat.com 2011-07-20 06:45:31 EDT 
---
Ping?

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

[Bug 642583] Review Request: rubygem-rr - RR is a framework that features a rich selection of double techniques

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Vít Ondruch vondr...@redhat.com 2011-07-20 06:45:07 EDT 
---
Ping?

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

[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #3 from Marek Goldmann mgold...@redhat.com 2011-07-20 06:48:22 
EDT ---
Michal, thank you for this review!

New Package SCM Request
===
Package Name: rubygem-aws-sdk
Short Description: AWS SDK for Ruby
Owners: goldmann
Branches: f14 f15 el5 el6

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


[Bug 708670] Review Request: hawtjni - Code generator that produces the JNI code

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 708670 depends on bug 708645, which changed state.

Bug 708645 Summary: Review Request: fusesource-pom - Parent POM for FuseSource 
Maven projects
https://bugzilla.redhat.com/show_bug.cgi?id=708645

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|ASSIGNED|CLOSED

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


[Bug 708645] Review Request: fusesource-pom - Parent POM for FuseSource Maven projects

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-07-20 07:02:16

--- Comment #5 from Stanislav Ochotnicky socho...@redhat.com 2011-07-20 
07:02:16 EDT ---
Package already built: 
http://koji.fedoraproject.org/koji/buildinfo?buildID=245728

Please close review bugs after building the package.

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


[Bug 722914] Review Request: volumeicon - Lightweight volume control for the system tray

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Christoph Wickert cwick...@fedoraproject.org 2011-07-20 
07:06:29 EDT ---
Thanks Veeti, both are valid points. GPLv3+ is definitely wrong, GPLv3 is
correct.

As for the spelling error: Capitalization can/should be used in the summary but
not in the description.

(In reply to comment #2)
 - As a courtesy to people reading your spec files, please don't use macros in
 the URL if they're not absolutely necessary.

Why not? we have spectool...

 - Patch0 is missing a description in the spec file. Please document what it
 does. Has it been sent upstream?

It fixes linking problems and I sent it upstream by mail. I'll add a comment on
this.

 - Where is Source1 from? Has it been sent upstream?

It's from gnome-media and has not been sent upstream (yet).

(In reply to comment #1)
 - Wouldn't it make sense to place another desktop file in the actual menu for
 this instead of just having an autostart entry? (?)

I don't think so. Icons in the systray are supposed to be persistent throughout
the whole session. They should be started on login and not from the menu.

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


[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Jon Ciesla l...@jcomserv.net 2011-07-20 07:55:00 EDT ---
Git done (by process-git-requests).

Michal, in the future, please take ownership of review bugs you're working
on.  Thanks!

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


[Bug 720323] Review Request: perl-Module-Runtime - Runtime module handling

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

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

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

[Bug 720323] Review Request: perl-Module-Runtime - Runtime module handling

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #1 from Marcela Mašláňová mmasl...@redhat.com 2011-07-20 08:01:57 
EDT ---
- rpmlint OK
- package must be named according to Guidelines OK
- spec file name must match the base package %{name} OK
- package must meet the Packaging Guidelines OK
- package must be licensed with Fedora approved license OK
- license field must match actual license OK
- text of the license in its own file must be included in %doc OK
- sources must match the upstream source OK
- package MUST successfully compile and build OK
 http://koji.fedoraproject.org/koji/taskinfo?taskID=3214430
- architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK
- build dependencies must be listed in BuildRequires OK
- handle locales properly with %find_lang macro OK
- shared library files must call ldconfig in %post(un) OK
- packages must NOT bundle system libraries OK
- package must own all directories that it creates OK
- permissions on files must be set properly OK
- package must consistently use macros OK
- package must contain code, or permissable content OK
- large documentation must go in a -doc OK
- %doc must not affect the runtime of the application OK
- header files must be in a -devel package OK
- static libraries must be in a -static package OK
- library files that end in .so (without suffix) must go in a -devel OK
- devel package usually require base package OK
- packages must NOT contain any .la libtool archives OK
- GUI applications must include a %{name}.desktop file OK
- packages must not own files or directories already owned by other packages OK

rpm -qp --provides
rpmbuild/RPMS/noarch/perl-Module-Runtime-0.008-1.fc16.noarch.rpm 
perl(Module::Runtime) = 0.008
perl-Module-Runtime = 0.008-1.fc16

rpm -qp --requires
rpmbuild/RPMS/noarch/perl-Module-Runtime-0.008-1.fc16.noarch.rpm 
perl(:MODULE_COMPAT_5.12.3)  
perl(Exporter)  
perl(Params::Classify) = 0.000
perl(parent)  
perl(strict)  
perl(warnings)  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1

APPROVED

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

[Bug 675050] Review Request: cloudfs - Cloud Filesystem

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Kaleb KEITHLEY kkeit...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #25 from Kaleb KEITHLEY kkeit...@redhat.com 2011-07-20 08:11:33 
EDT ---
New Package SCM Request
===
Package Name: cloudfs
Short Description: CloudFS is a cloud-capable file system based on GlusterFS
(http://gluster.org)
Owners: kkeithle
Branches: f16
InitialCC: jdarcy

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


[Bug 722914] Review Request: volumeicon - Lightweight volume control for the system tray

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Christoph Wickert cwick...@fedoraproject.org changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #6 from Christoph Wickert cwick...@fedoraproject.org 2011-07-20 
08:16:34 EDT ---
New Package SCM Request
===
Package Name: volumeicon
Short Description: Lightweight volume control for the system tray
Owners: cwickert
Branches: f14 f15
InitialCC:

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


[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||RAWHIDE
   Flag|fedora-cvs+ |fedora-cvs?
Last Closed||2011-07-20 08:23:53

--- Comment #5 from Marek Goldmann mgold...@redhat.com 2011-07-20 08:23:53 
EDT ---
Thank you for review and git!

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


[Bug 722914] Review Request: volumeicon - Lightweight volume control for the system tray

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Jon Ciesla l...@jcomserv.net 2011-07-20 08:35:38 EDT ---
Git done (by process-git-requests).

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


[Bug 675050] Review Request: cloudfs - Cloud Filesystem

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #26 from Jon Ciesla l...@jcomserv.net 2011-07-20 08:35:13 EDT ---
Git done (by process-git-requests).

Don't request f16, devel is created by default.

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


[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

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

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

[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jon Ciesla l...@jcomserv.net 2011-07-20 08:38:11 EDT ---
Why was the cvs flag set again?

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


[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #1 from Marcela Mašláňová mmasl...@redhat.com 2011-07-20 08:40:52 
EDT ---
- rpmlint OK
- package must be named according to Guidelines OK
- spec file name must match the base package %{name} OK
- package must meet the Packaging Guidelines OK
- package must be licensed with Fedora approved license OK
- license field must match actual license OK
- text of the license in its own file must be included in %doc OK
- sources must match the upstream source OK
- package MUST successfully compile and build OK
- architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK
- build dependencies must be listed in BuildRequires OK
- handle locales properly with %find_lang macro OK
- shared library files must call ldconfig in %post(un) OK
- packages must NOT bundle system libraries OK
- package must own all directories that it creates OK
- permissions on files must be set properly OK
- package must consistently use macros OK
- package must contain code, or permissable content OK
- large documentation must go in a -doc OK
- %doc must not affect the runtime of the application OK
- header files must be in a -devel package OK
- static libraries must be in a -static package OK
- library files that end in .so (without suffix) must go in a -devel OK
- devel package usually require base package OK
- packages must NOT contain any .la libtool archives OK
- GUI applications must include a %{name}.desktop file OK
- packages must not own files or directories already owned by other packages OK

rpm -qp --provides
rpmbuild/RPMS/noarch/perl-DynaLoader-Functions-0.000-1.fc16.noarch.rpm 
perl(DynaLoader::Functions) = 0.000
perl-DynaLoader-Functions = 0.000-1.fc16

rpm -qp --requires
rpmbuild/RPMS/noarch/perl-DynaLoader-Functions-0.000-1.fc16.noarch.rpm 
perl(:MODULE_COMPAT_5.12.3)  
perl(Carp)  
perl(DynaLoader)  
perl(Exporter)  
perl(Module::Runtime) = 0.007
perl(Params::Classify) = 0.008
perl(parent)  
perl(strict)  
perl(warnings)  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1

This package is conditionally approved, because build requires is not in
buildroot yet. Check if it builds in koji first. It does build for me in local
rawhide.
APPROVED

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

[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Marek Goldmann mgold...@redhat.com 2011-07-20 08:44:35 
EDT ---
TBH, I don't know - maybe it was in browser cache.

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


[Bug 680174] Review Request: herqq - A software library for building UPnP devices and control points

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jaroslav Reznik jrez...@redhat.com 2011-07-20 08:45:39 
EDT ---
Thanks Rex, fixed.

Spec URL: http://rezza.hofyland.cz/fedora/packages/herqq/herqq.spec
SRPM URL:
http://rezza.hofyland.cz/fedora/packages/herqq/herqq-0.9.1-3.fc14.src.rpm

* Wed Jul 20 2011 Jaroslav Reznik jrez...@redhat.com - 0.9.1-3
- fix license to LGPLv3+
- qt4 header dir for consistency
- shlib soname tracked in %files
- -devel should not duplicate COPYING

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


[Bug 680174] Review Request: herqq - A software library for building UPnP devices and control points

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Jaroslav Reznik jrez...@redhat.com 2011-07-20 08:47:32 
EDT ---
New Package SCM Request
===
Package Name: herqq
Short Description: A software library for building UPnP devices and control
points
Owners: jreznik
Branches: f14 f15
InitialCC:

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


[Bug 723523] Review Request: apiviz - APIviz is a JavaDoc doclet to generate class and package diagrams

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Blocks||652183(FE-JAVASIG)

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


[Bug 723523] New: Review Request: apiviz - APIviz is a JavaDoc doclet to generate class and package diagrams

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: apiviz - APIviz is a JavaDoc doclet to generate class 
and package diagrams

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

   Summary: Review Request: apiviz - APIviz is a JavaDoc doclet to
generate class and package diagrams
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mgold...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---


Spec URL: http://goldmann.fedorapeople.org/package_review/apiviz/1/apiviz.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/apiviz/1/apiviz-1.3.1-1.fc15.src.rpm
Description:

APIviz is a JavaDoc doclet which extends the Java standard doclet.
It generates comprehensive UML-like class and package diagrams for
quick understanding of the overall API structure.

$ rpmlint ./apiviz-1.3.1-1.fc15.src.rpm 
apiviz.src: I: enchant-dictionary-not-found en_US
apiviz.src: W: name-repeated-in-summary C APIviz
apiviz.src: W: invalid-url Source0: apiviz-1.3.1.GA.tar.xz
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint ./apiviz.spec 
./apiviz.spec: W: invalid-url Source0: apiviz-1.3.1.GA.tar.xz
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

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

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #18 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-07-20 
09:17:12 EDT ---
Thanks for the review.

Yes. I remember that ghc-leksah, although needed (IMO), was not being picked up
automatically. I will check that and  update the spec as appropriate. 

For the icon part, I will go with your suggestion and not resize.

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Lakshmi Narasimhan lakshminaras2...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #19 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-07-20 
09:18:07 EDT ---
New Package SCM Request
===
Package Name: leksah
Short Description:  An IDE for Haskell
Owners: narasim
Branches: f14 f15
InitialCC: haskell-sig

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


[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #27 from Mo Morsi mmo...@redhat.com 2011-07-20 09:25:45 EDT ---
OK, thanks for the ACK. Patch updated to address all outstanding feedback

Updated SPEC: http://mo.morsi.org/files/rpms/rubygem-rdoc.spec
Updated SRPM: http://mo.morsi.org/files/rpms/rubygem-rdoc-3.8-2.fc15.src.rpm
Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3214663


New Package SCM Request
===
Package Name: rubygem-rdoc
Short Description: Produces HTML and command-line documentation for Ruby
projects
Owners: mmorsi
Branches:
InitialCC:

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


[Bug 719067] Review Request: rubygem-hmac - This module provides common interface to HMAC functionality

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Mo Morsi mmo...@redhat.com 2011-07-20 09:26:46 EDT ---
Looks good. APPROVED.

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #20 from Jon Ciesla l...@jcomserv.net 2011-07-20 09:28:21 EDT ---
Git done (by process-git-requests).

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


[Bug 680174] Review Request: herqq - A software library for building UPnP devices and control points

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Jon Ciesla l...@jcomserv.net 2011-07-20 09:30:08 EDT ---
Git done (by process-git-requests).

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


[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #28 from Jon Ciesla l...@jcomserv.net 2011-07-20 09:32:05 EDT ---
Git done (by process-git-requests).

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


[Bug 719067] Review Request: rubygem-hmac - This module provides common interface to HMAC functionality

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Lalancette clala...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Chris Lalancette clala...@redhat.com 2011-07-20 09:52:46 
EDT ---
New Package SCM Request
===
Package Name: rubygem-hmac
Short Description: This module provides common interface to HMAC functionality
Owners: clalance mmorsi

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


[Bug 719975] Review Request: rubygem-chunky_png - Pure ruby library for read/write, chunk-level access to PNG files

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Mo Morsi mmo...@redhat.com 2011-07-20 09:59:53 EDT ---
Looks good. APPROVED.

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


[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed|2011-03-08 16:08:23 |2011-07-20 09:59:12

--- Comment #29 from Mo Morsi mmo...@redhat.com 2011-07-20 09:59:12 EDT ---
Pushed to rawhide

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


[Bug 719975] Review Request: rubygem-chunky_png - Pure ruby library for read/write, chunk-level access to PNG files

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Lalancette clala...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Chris Lalancette clala...@redhat.com 2011-07-20 10:04:40 
EDT ---
New Package SCM Request
===
Package Name: rubygem-chunky_png
Short Description: Pure ruby library for read/write, chunk-level access to PNG
files
Owners: clalance mmorsi

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


[Bug 706934] Review Request: rubygem-rspec-rails - RSpec-2 for Rails-3

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

 CC||mmo...@redhat.com

--- Comment #10 from Mo Morsi mmo...@redhat.com 2011-07-20 10:13:29 EDT ---
Looks like the push to git has been done but the koji build failed and thus
this is still not available in rawhide. Any thoughts as to the problem?

http://koji.fedoraproject.org/koji/buildinfo?buildID=251113

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


[Bug 721057] Review Request: rubygem-sass - A powerful but elegant CSS compiler that makes CSS fun again

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Mo Morsi mmo...@redhat.com 2011-07-20 10:15:56 EDT ---
Thank you

New Package SCM Request
===
Package Name: rubygem-sass
Short Description: A powerful but elegant CSS compiler 
Owners: mmorsi
Branches: 
InitialCC:

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


[Bug 721063] Review Request: rubygem-factory_girl_rails - factory_girl_rails provides integration between factory_girl and rails 3

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Mo Morsi mmo...@redhat.com 2011-07-20 10:20:54 EDT ---
New Package SCM Request
===
Package Name: rubygem-factory_girl_rails
Short Description: provides integration between factory_girl and rails 3
Owners: mmorsi
Branches: 
InitialCC:

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


[Bug 721059] Review Request: rubygem-dynamic_form - DynamicForm holds a few helper methods to help you deal with your Rails3 models

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Mo Morsi mmo...@redhat.com 2011-07-20 10:18:15 EDT ---
Thanks

New Package SCM Request
===
Package Name: rubygem-dynamic_form
Short Description: DynamicForm holds a few helper methods to help you deal with
your Rails3 models
Owners: mmorsi
Branches: 
InitialCC:

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


[Bug 721066] Review Request: rubygem-image_factory_console - QMF Console for Aeolus Image Factory

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Mo Morsi mmo...@redhat.com 2011-07-20 10:27:09 EDT ---
Updated SPEC:
http://mo.morsi.org/files/aeolus/rubygem-image_factory_console.spec
Updated SRPM:
http://mo.morsi.org/files/aeolus/rubygem-image_factory_console-0.4.0-3.fc15.src.rpm

(In reply to comment #5)
 Also, there's a thread on aeolus-devel:
 
   https://fedorahosted.org/pipermail/aeolus-devel/2011-July/003247.html
 
 Looks like we're going to change the name of gem to 'imagefactory-console'

Will update this submission when the patches doing so have been ack'd/pushed


(In reply to comment #4)
 FWIW, I just noticed ruby_sitelib is unused

Ah good catch, removed

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #21 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-07-20 
10:31:38 EDT ---
ghc-leksah is not required and I removed the Requires.

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Lakshmi Narasimhan lakshminaras2...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

--- Comment #22 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-07-20 
10:45:25 EDT ---
Built for rawhide 

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

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


[Bug 721112] Review Request: vmtk - The Vascular Modeling Toolkit

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Ankur Sinha sanjay.an...@gmail.com 2011-07-20 10:43:12 
EDT ---
Hi Richard,

(In reply to comment #4)
 I've made some progress trying to get it to build against the OpenNL package
 but I can't really test it because of:
 
 rpmbuild -bc rpmbuild/vmtk/SPECS/vmtk.spec
 error: Failed build dependencies:
 InsightToolkit-devel is needed by vmtk-0.9.0-2.fc14.x86_64
 vxl-devel is needed by vmtk-0.9.0-2.fc14.x86_64
 
 Because of that I've added the review request of vxl and InsightToolkit as a
 blocker for this Review Request.

I'm sorry, I forgot to mention that you need to build ITK and vxl for your
fedora system before you can test it. I for example, have a local mock repo
with these packages. 

[root@ankur ~]# ls /tmp/mock-repo/InsightToolkit-* /tmp/mock-repo/vxl-*
/tmp/mock-repo/InsightToolkit-3.20.0-5.fc16.i686.rpm   
/tmp/mock-repo/vxl-1.14.0-1.fc16.i686.rpm
/tmp/mock-repo/InsightToolkit-3.20.0-5.fc16.src.rpm
/tmp/mock-repo/vxl-1.14.0-1.fc16.src.rpm
/tmp/mock-repo/InsightToolkit-debuginfo-3.20.0-5.fc16.i686.rpm 
/tmp/mock-repo/vxl-debuginfo-1.14.0-1.fc16.i686.rpm
/tmp/mock-repo/InsightToolkit-devel-3.20.0-5.fc16.i686.rpm 
/tmp/mock-repo/vxl-devel-1.14.0-1.fc16.i686.rpm
/tmp/mock-repo/InsightToolkit-doc-3.20.0-5.fc16.i686.rpm   
/tmp/mock-repo/vxl-doc-1.14.0-1.fc16.i686.rpm
/tmp/mock-repo/InsightToolkit-examples-3.20.0-5.fc16.i686.rpm



 
 Back to this review :)
 
 The only part of the project that appears to use OpenNL is the
 DifferentialGeometry sub-project. 
 
 I've added a cmake option, USE_EXTERN_OPENNL to the main CMakeLists.txt and 
 I'm
 working on getting the headers and linking working in the CMakeLists.txt for
 DifferentialGeometry.


The package works with the fedora OpenNL which I packaged recently. You don't
need to make any tweaks. The changes I've made in the prep and build sections
are enough. 

At the moment, gdcm is broken in rawhide,(after the latest libpoppler soname
bump) so I can't build it as well :/

Thanks,
Ankur

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


[Bug 720323] Review Request: perl-Module-Runtime - Runtime module handling

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Petr Pisar ppi...@redhat.com 2011-07-20 10:49:50 EDT ---
New Package SCM Request
===
Package Name: perl-Module-Runtime 
Short Description: Runtime module handling
Owners: ppisar mmaslano psabata
Branches: f14 f15
InitialCC: perl-sig

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


[Bug 720323] Review Request: perl-Module-Runtime - Runtime module handling

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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


[Bug 721144] Review Request: imagefactory - System image generation tool

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Chris Lalancette clala...@redhat.com 2011-07-20 10:49:51 
EDT ---
(In reply to comment #3)
 (In reply to comment #2)
  (In reply to comment #1)
- I don't think euca2ools is actually required; we run all the euca-
  commands inside EC2 instances not locally
  
  When we are doing upload or local style builds, the euca2ools are 
  actually
  required to do the bundling.  See imagefactory/builders/FedoraBuilder.py,
  around lines 1383.
 
 Ah, of course - right
 
 I was just about to approve this, but then I remembered to run rpmlint on the
 actual RPM itself:
 
 $ rpmlint imagefactory-0.3.1-2.fc14.noarch.rpm
 imagefactory.noarch: E: explicit-lib-dependency python-httplib2
 imagefactory.noarch: W: conffile-without-noreplace-flag
 /etc/pki/imagefactory/cert-ec2.pem
 imagefactory.noarch: E: non-executable-script
 /usr/lib/python2.7/site-packages/imagefactory/ApplicationConfiguration.py 
 0644L
 /usr/bin/env
 imagefactory.noarch: E: non-executable-script
 /usr/lib/python2.7/site-packages/imagefactory/Template.py 0644L /usr/bin/env
 imagefactory.noarch: E: non-executable-script
 /usr/lib/python2.7/site-packages/imagefactory/ImageWarehouse.py 0644L
 /usr/bin/env
 imagefactory.noarch: E: non-executable-script
 /usr/lib/python2.7/site-packages/imagefactory/VMWare.py 0644L /usr/bin/env
 imagefactory.noarch: W: no-manual-page-for-binary imgfac.py
 1 packages and 0 specfiles checked; 5 errors, 2 warnings.
 
 
 Re: explicit-lib-dependency - this looks like a false positive to me. See
 http://lists.fedoraproject.org/pipermail/devel/2010-March/133955.html

Yeah, I figured it was just looking for the lib string, and that post confirms
it :).

 
 Re: conffile-without-noreplace-flag - do we allow users to replace this with
 their own cert? If so, the error is valid.

No, the user can't replace it.  It is the global ec2 certificate, comes from
amazon, and is used (in some way) to sign the bundled images.  As such, I
believe it is correct as-is.

 
 Re: non-executable-script - do sed -i '/\/usr\/bin\/env python/d'
 imagefactory/*.py

Fixed.

 
 Re: no-manual-page-for-binary - there actually is a man page, but not with the
 same name as the binary. Do we really want the binary to be installed as
 imgfac.py ?

This has been a source of annoyance to me for some time.  Upstream, I think we
should eventually rename the binary to imagefactory and fix all of the
references to it, but I don't think I want to make that kind of change right at
the moment.

I've uploaded revision 3:

SRPM:
http://people.redhat.com/clalance/imagefactory/imagefactory-0.3.1-3.fc14.src.rpm
SPEC: http://people.redhat.com/clalance/imagefactory/imagefactory.spec

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


[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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


[Bug 711462] Review Request: perl-Fedora-Rebuild - Rebuilds Fedora packages from scratch

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2011-07-20 10:55:24 EDT ---
perl-Fedora-Rebuild-0.8.0-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/perl-Fedora-Rebuild-0.8.0-1.fc15

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


[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Petr Pisar ppi...@redhat.com 2011-07-20 10:52:36 EDT ---
New Package SCM Request
===
Package Name: perl-DynaLoader-Functions 
Short Description: Deconstructed dynamic C library loading
Owners: ppisar mmaslano psabata
Branches: f14 f15
InitialCC: perl-sig

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #24 from Fedora Update System upda...@fedoraproject.org 
2011-07-20 10:57:21 EDT ---
leksah-0.10.0.4-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/leksah-0.10.0.4-3.fc15

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #25 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-07-20 
10:58:24 EDT ---
Wrong build posted on comment 22 .

Rawhide build @ http://koji.fedoraproject.org/koji/buildinfo?buildID=254319

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


[Bug 706934] Review Request: rubygem-rspec-rails - RSpec-2 for Rails-3

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Chris Lalancette clala...@redhat.com 2011-07-20 10:58:08 
EDT ---
I think the problem had something to do with an outdated railties.  I'll give
it another whirl and see what the errors are.

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


[Bug 673776] Review Request: leksah - An IDE for Haskell

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #23 from Fedora Update System upda...@fedoraproject.org 
2011-07-20 10:57:12 EDT ---
leksah-0.10.0.4-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/leksah-0.10.0.4-3.fc14

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


[Bug 721144] Review Request: imagefactory - System image generation tool

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mark McLoughlin mar...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Mark McLoughlin mar...@redhat.com 2011-07-20 10:56:31 EDT 
---
Okay, looks good to me

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


[Bug 711462] Review Request: perl-Fedora-Rebuild - Rebuilds Fedora packages from scratch

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #18 from Fedora Update System upda...@fedoraproject.org 
2011-07-20 10:59:55 EDT ---
perl-Fedora-Rebuild-0.8.0-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/perl-Fedora-Rebuild-0.8.0-1.fc14

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


[Bug 706934] Review Request: rubygem-rspec-rails - RSpec-2 for Rails-3

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Chris Lalancette clala...@redhat.com 2011-07-20 11:07:22 
EDT ---
Right, here's the problem (from root.log):

DEBUG util.py:250:   -- rubygems-1.8.5-2.fc16.noarch
DEBUG util.py:250:   -- ruby-libs-1.8.7.352-1.fc16.x86_64
DEBUG util.py:250:   -- rubygem-rspec-core-2.6.4-1.fc16.noarch
DEBUG util.py:250:   -- rubygem-rspec-core-2.6.4-1.fc16.noarch
DEBUG util.py:250:   -- 1:rubygem-rails-3.0.5-2.fc16.noarch
DEBUG util.py:250:   -- 1:rubygem-rails-3.0.5-2.fc16.noarch
DEBUG util.py:250:   -- rubygem-sqlite3-1.3.3-5.fc16.x86_64
DEBUG util.py:250:  Error: Package: 1:rubygem-rails-3.0.5-2.fc16.noarch
(fedora)
DEBUG util.py:250: Requires: rubygem(railties) = 3.0.5
DEBUG util.py:250: Available: rubygem-railties-3.0.9-1.fc16.noarch
(fedora)
DEBUG util.py:250: rubygem(railties) = 3.0.9
DEBUG util.py:250:   You could try using --skip-broken to work around the
problem
DEBUG util.py:250:   You could try running: rpm -Va --nofiles --nodigest

The problem is that the updated railties is built, but it needs the updated
rails which is not.  Now that the new rubygem-rdoc is in, hopefully we can get
rails 3.0.9 built, and then we can build this.

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


[Bug 721144] Review Request: imagefactory - System image generation tool

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Chris Lalancette clala...@redhat.com 2011-07-20 11:13:02 
EDT ---
Thanks!

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


[Bug 721066] Review Request: rubygem-image_factory_console - QMF Console for Aeolus Image Factory

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Chris Lalancette clala...@redhat.com 2011-07-20 11:12:23 
EDT ---
These patches are now in the repository, so we should be good to rename the RPM
and get this package in.

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


[Bug 721144] Review Request: imagefactory - System image generation tool

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Lalancette clala...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Chris Lalancette clala...@redhat.com 2011-07-20 11:13:57 
EDT ---
New Package SCM Request
===
Package Name: imagefactory
Short Description: System image generation tool
Owners: clalance

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


[Bug 721066] Review Request: rubygem-image_factory_console - QMF Console for Aeolus Image Factory

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Mark McLoughlin mar...@redhat.com 2011-07-20 11:17:18 EDT 
---
Well, there hasn't been an official upstream release to package yet - I'd wait
until tomorrow for that

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


[Bug 722427] Review Request: python-importlib - Backport of importlib.import_module() from Python 2.7

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-07-20 
11:30:49 EDT ---
python-importlib-1.0.2-1.el5 has been pushed to the Fedora EPEL 5 testing
repository.

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


[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #18 from Fedora Update System upda...@fedoraproject.org 
2011-07-20 11:31:17 EDT ---
python26-argparse-1.2.1-3.el5, packagedb-cli-1.0.0-6.el5 has been pushed to the
Fedora EPEL 5 stable repository.  If problems still persist, please make note
of it in this bug report.

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


[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Fixed In Version|python26-argparse-1.2.1-3.e |packagedb-cli-1.0.0-6.el6
   |l5  |

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


[Bug 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||python26-argparse-1.2.1-3.e
   ||l5
 Resolution||ERRATA
Last Closed||2011-07-20 11:31:24

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


[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #19 from Fedora Update System upda...@fedoraproject.org 
2011-07-20 11:32:13 EDT ---
packagedb-cli-1.0.0-6.el6 has been pushed to the Fedora EPEL 6 stable
repository.

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


[Bug 617340] Review Request: throttle - copy stdin to stdout at the specified speed (or lower)

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #26 from Fedora Update System upda...@fedoraproject.org 
2011-07-20 11:32:57 EDT ---
throttle-1.2-4.el6 has been pushed to the Fedora EPEL 6 stable repository.

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


[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

   Fixed In Version|packagedb-cli-1.0.0-6.fc14  |python26-argparse-1.2.1-3.e
   ||l5

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


[Bug 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-07-20 
11:31:12 EDT ---
python26-argparse-1.2.1-3.el5, packagedb-cli-1.0.0-6.el5 has been pushed to the
Fedora EPEL 5 stable repository.  If problems still persist, please make note
of it in this bug report.

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


[Bug 617340] Review Request: throttle - copy stdin to stdout at the specified speed (or lower)

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #27 from Fedora Update System upda...@fedoraproject.org 
2011-07-20 11:34:10 EDT ---
throttle-1.2-4.el5 has been pushed to the Fedora EPEL 5 stable repository.

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


[Bug 719854] Review Request: rubygem-xmlparser-0.6.81-1 - Ruby bindings to the Expat XML parsing library

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #25 from Shawn Starr shawn.st...@rogers.com 2011-07-20 11:34:43 
EDT ---
Additional comments:

the DSO library should be in %{ruby_sitearch}/%{gemname}/*.so Same as
rubygem-cairo, although rubygem-json uses %{ruby_sitearch}/%{gemname}/etc/ for
the DSO location.

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


[Bug 721078] Review Request: aeolus-conductor - A web UI for managing cloud instances.

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Lalancette clala...@redhat.com changed:

   What|Removed |Added

 CC||clala...@redhat.com

--- Comment #3 from Chris Lalancette clala...@redhat.com 2011-07-20 11:35:57 
EDT ---
I've fixed most of the issues you've identified here.  I still have to figure
out what to do about the Source URL; I'll post again once I have that licked.

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


[Bug 723575] New: Review Request: libpki - Easy-to-use PKI library

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: libpki - Easy-to-use PKI library

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

   Summary: Review Request: libpki - Easy-to-use PKI library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: p...@datasphere.ch
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---


Spec URL: http://monnerat.fedorapeople.org/libpki.spec
SRPM URL: http://monnerat.fedorapeople.org/libpki-0.6.5-1.fc14.src.rpm
Description:
  This library provides the developer with all the needed features to
manage certificates, from generation to validation. The LibPKI Project
enables developers with the possibility to implement complex cryptographic
operations with a few simple function calls by implementing an high-level
cryptographic API.

The last version of the OCSP responder (ocspd) uses this library: thus I need
libpki to upgrade ocspd.

rpmlint output:
$ rpmlint SPECS/libpki.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
$ rpmlint SRPMS/libpki-0.6.5-1.fc14.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint RPMS/x86_64/libpki-0.6.5-1.fc14.x86_64.rpm 
libpki.x86_64: E: incorrect-fsf-address /usr/share/doc/libpki-0.6.5/COPYING
libpki.x86_64: W: no-manual-page-for-binary pki-xpair
libpki.x86_64: W: no-manual-page-for-binary pki-crl
libpki.x86_64: W: no-manual-page-for-binary pki-request
libpki.x86_64: W: no-manual-page-for-binary pki-query
libpki.x86_64: W: no-manual-page-for-binary pki-cert
libpki.x86_64: W: no-manual-page-for-binary pki-derenc
libpki.x86_64: W: no-manual-page-for-binary url-tool
libpki.x86_64: W: no-manual-page-for-binary pki-tool
libpki.x86_64: W: no-manual-page-for-binary pki-siginfo
1 packages and 0 specfiles checked; 1 errors, 9 warnings.
$ rpmlint RPMS/x86_64/libpki-devel-0.6.5-1.fc14.x86_64.rpm 
libpki-devel.x86_64: W: no-documentation
libpki-devel.x86_64: W: no-manual-page-for-binary libpki-config
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
$ rpmlint RPMS/x86_64/libpki-static-0.6.5-1.fc14.x86_64.rpm 
libpki-static.x86_64: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
$ rpmlint RPMS/x86_64/libpki-doc-0.6.5-1.fc14.x86_64.rpm 
libpki-doc.x86_64: E: incorrect-fsf-address
/usr/share/doc/libpki-doc-0.6.5/COPYING
1 packages and 0 specfiles checked; 1 errors, 0 warnings.
$ rpmlint RPMS/x86_64/libpki-debuginfo-0.6.5-1.fc14.x86_64.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

- Upstream has been requested by e-mail to change the license file and to
provide man1 pages.
- Documentation that can be provided in the -devel and -static subpackages is
already present in the (required) main package.
- Although make test is recognized, no %check spec section is provided: the
upstream-provided tests need a tty input and can only be executed by themselves
without failing :-(

Thanks in advance for reviewing

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


[Bug 719067] Review Request: rubygem-hmac - This module provides common interface to HMAC functionality

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jon Ciesla l...@jcomserv.net 2011-07-20 11:46:28 EDT ---
Git done (by process-git-requests).

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


[Bug 721112] Review Request: vmtk - The Vascular Modeling Toolkit

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Richard Shaw hobbes1...@gmail.com 2011-07-20 11:43:55 EDT 
---
(In reply to comment #5)
 (In reply to comment #4)
  I've made some progress trying to get it to build against the OpenNL package
  but I can't really test it because of:
  
  rpmbuild -bc rpmbuild/vmtk/SPECS/vmtk.spec
  error: Failed build dependencies:
  InsightToolkit-devel is needed by vmtk-0.9.0-2.fc14.x86_64
  vxl-devel is needed by vmtk-0.9.0-2.fc14.x86_64
  
  Because of that I've added the review request of vxl and InsightToolkit as a
  blocker for this Review Request.
 
 I'm sorry, I forgot to mention that you need to build ITK and vxl for your
 fedora system before you can test it. I for example, have a local mock repo
 with these packages. 

Yup, I run a local repo as well. I'll try that.


  The only part of the project that appears to use OpenNL is the
  DifferentialGeometry sub-project. 
  
  I've added a cmake option, USE_EXTERN_OPENNL to the main CMakeLists.txt and 
  I'm
  working on getting the headers and linking working in the CMakeLists.txt for
  DifferentialGeometry.
 
 
 The package works with the fedora OpenNL which I packaged recently. You don't
 need to make any tweaks. The changes I've made in the prep and build sections
 are enough. 

Well, I was trying for a few reasons. One, I'm trying to improve my cmake-fu :)
Also, hacking the link.txt is not ideal. If I can come up with a good cmake
patch then maybe upstream will accept it (which is the best case). 

Also, in retrospect, I'm wondering if we should have renamed the header file
for OpenNL to be consistent with the library name, i.e.: nl.h - opennl.h

What do you think?

Richard

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


[Bug 720323] Review Request: perl-Module-Runtime - Runtime module handling

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Jon Ciesla l...@jcomserv.net 2011-07-20 11:47:15 EDT ---
Git done (by process-git-requests).

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


[Bug 721057] Review Request: rubygem-sass - A powerful but elegant CSS compiler that makes CSS fun again

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jon Ciesla l...@jcomserv.net 2011-07-20 11:48:13 EDT ---
Git done (by process-git-requests).

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


[Bug 719975] Review Request: rubygem-chunky_png - Pure ruby library for read/write, chunk-level access to PNG files

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Jon Ciesla l...@jcomserv.net 2011-07-20 11:46:52 EDT ---
Git done (by process-git-requests).

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


[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Jon Ciesla l...@jcomserv.net 2011-07-20 11:47:45 EDT ---
Git done (by process-git-requests).

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


[Bug 721059] Review Request: rubygem-dynamic_form - DynamicForm holds a few helper methods to help you deal with your Rails3 models

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Jon Ciesla l...@jcomserv.net 2011-07-20 11:48:44 EDT ---
Git done (by process-git-requests).

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


[Bug 721063] Review Request: rubygem-factory_girl_rails - factory_girl_rails provides integration between factory_girl and rails 3

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Jon Ciesla l...@jcomserv.net 2011-07-20 11:49:14 EDT ---
Git done (by process-git-requests).

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


[Bug 721066] Review Request: rubygem-imagefactory-console - QMF Console for Aeolus Image Factory

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mo Morsi mmo...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |rubygem-image_factory_conso |rubygem-imagefactory-consol
   |le - QMF Console for Aeolus |e - QMF Console for Aeolus
   |Image Factory   |Image Factory

--- Comment #9 from Mo Morsi mmo...@redhat.com 2011-07-20 11:52:51 EDT ---
Renamed the package

Updated SPEC:
http://mo.morsi.org/files/aeolus/rubygem-imagefactory-console.spec
Updated SRPM:
http://mo.morsi.org/files/aeolus/rubygem-imagefactory-console-0.4.0-4.fc15.src.rpm

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


[Bug 723581] New: aeolus-configure - Aeolus Configure Puppet Recipe

2011-07-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: aeolus-configure - Aeolus Configure Puppet Recipe

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

   Summary: aeolus-configure - Aeolus Configure Puppet Recipe
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmo...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---


Spec: http://mo.morsi.org/files/aeolus/aeolus-configure.spec
SRPM: http://mo.morsi.org/files/aeolus/aeolus-configure-2.0.1-2.src.rpm
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3215056

Description:
Aeolus Configure Puppet Recipe

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


  1   2   >