[Bug 740687] Review Request: python-pygraphviz - PyGraphviz package for Python

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


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

Scott Tsai scottt...@gmail.com changed:

   What|Removed |Added

 CC||scottt...@gmail.com
   Flag||needinfo?(riva...@gmail.com
   ||)

--- Comment #2 from Scott Tsai scottt...@gmail.com 2011-12-22 03:13:51 EST ---
The SPEC and SRPM URLs no longer work.
Vedran, are you still interested in submitting this 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 756443] Review Request: cagraph - A PyGTK widget for plotting charts and graphs

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


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

--- Comment #5 from Juan Orti Alcaine j.orti.alca...@gmail.com 2011-12-22 
04:10:25 EST ---
I have improved the SPEC file by removing the python shebang from the
libraries, the new files are here:

Spec URL:
https://raw.github.com/jorti/cagraph-fedora-package/v1.2-9/SPECS/python-cagraph.spec
SRPM URL:
https://github.com/jorti/cagraph-fedora-package/raw/v1.2-9/SRPMS/python-cagraph-1.2-9.fc16.src.rpm

Now the rpmlint output is:

$ rpmlint SPECS/python-cagraph.spec
RPMS/noarch/python-cagraph-1.2-9.fc16.noarch.rpm
SRPMS/python-cagraph-1.2-9.fc16.src.rpm
SPECS/python-cagraph.spec: W: invalid-url Source0:
http://cagraph.googlecode.com/files/cagraph-1.2.tar.gz HTTP Error 404: Not
Found
python-cagraph.src: W: invalid-url Source0:
http://cagraph.googlecode.com/files/cagraph-1.2.tar.gz HTTP Error 404: Not
Found
2 packages and 1 specfiles checked; 0 errors, 2 warnings.

The invalid-url error is a known problem with googlecode.com already reported
in bug 767739

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

[Bug 769487] Review Request: python-glumpy - Small python library for rapid visualization of numpy arrays

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


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

Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org

--- Comment #1 from Thomas Spura toms...@fedoraproject.org 2011-12-22 
05:21:13 EST ---
Nice program :)

Also run into the same problem like you:
http://code.google.com/p/glumpy/issues/detail?id=13

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

[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils

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


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

--- Comment #2 from David Howells dhowe...@redhat.com 2011-12-22 05:23:00 EST 
---
(In reply to comment #1)
 You've explained the dangling symlinks, and I think that's OK.

Possibly I should just put all the manual page symlinks into the doc rpm,
though I'd prefer them only to be installed if the things they're describing
are present.

 If you can arrange a man page for ld, that would be nice but it's not 
 mandatory.

There are manual pages for ld.  Do you mean ld.bfd?  If so, there is no manual
page specifically for that.  I'm not entirely sure what the ld.bfd is for.

 We'll have to check what the rules are for non-standard directories in /usr,

I wonder if that's going to happen if I let it create, say,
/usr/xtensa-linux-gnu/ instead of /usr/cross/xtensa-linux-gnu/.  I wonder if
these things should be in /usr/libexec.

 and the cross-directory hard links.

I missed that.  Interesting...  I wonder if the core binutils package does this
too.  Even though it's marked as a cross-dir hardlink, it isn't installed so
(even though it could be).

 Also:
 cross-binutils.src:49: W: macro-in-comment %{version}

That's a comment borrowed from the original binutils.spec.  I should probably
keep it as long as that does.

 cross-binutils.src:138: W: macro-in-comment %ifarch
 cross-binutils.src:139: W: macro-in-comment %if
 cross-binutils.src:139: W: macro-in-comment %{_lib}
 cross-binutils.src:140: W: macro-in-comment %patch03
 cross-binutils.src:141: W: macro-in-comment %endif
 cross-binutils.src:142: W: macro-in-comment %endif

I'm not sure whether I need this.  I should find an IA64 box and try it.

 cross-binutils.src:398: W: macro-in-comment %{buildroot}
 cross-binutils.src:398: W: macro-in-comment %{_prefix}
 cross-binutils.src:399: W: macro-in-comment %{buildroot}
 cross-binutils.src:399: W: macro-in-comment %{_mandir}

I should get rid of those.

 cross-binutils.src: W: patch-not-applied Patch3:
 binutils-2.20.51.0.2-ia64-lib64.patch

That's one of the original binutils.spec patches and is related to the
macro-in-comment warnings of lines 138-142.  I wonder if I should just apply
all of the original binutils.spec patches as applied by 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 759057] Review Request: libSOIL - Simple OpenGL Image Library

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


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

Mohamed El Morabity pikachu.2...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||pikachu.2...@gmail.com
 AssignedTo|nob...@fedoraproject.org|pikachu.2...@gmail.com

--- Comment #1 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-22 
05:37:43 EST ---
Since SFML requires libSOIL, I will review it.

Just one comment, by reading your spec file: is libSOIL a good name for it?
The upstream name of the library is SOIL, and Fedora packages don't have to
prefix library packages with lib, like in Debian.
   http://fedoraproject.org/wiki/Packaging:NamingGuidelines#General_Naming
Could you rename the package to SOIL ?

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

[Bug 737972] Review Request: spacewalk-admin - Various utility scripts and data files for RHN Satellite installations

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


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

Milan Zazrivec mzazri...@redhat.com changed:

   What|Removed |Added

 Blocks||765736(space17)

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

[Bug 737972] Review Request: spacewalk-admin - Various utility scripts and data files for RHN Satellite installations

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


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

Milan Zazrivec mzazri...@redhat.com changed:

   What|Removed |Added

 Blocks|765736(space17) |

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

[Bug 749752] Review Request: dmg2img - Uncompress the Apple compressed disk image files

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


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

--- Comment #2 from Scott Tsai scottt...@gmail.com 2011-12-22 06:31:33 EST ---
Created attachment 549174
  -- https://bugzilla.redhat.com/attachment.cgi?id=549174
[PATCH] don't strip binaries

This patch adds a STRIP variable to dmg2img's Makefile.
You can build with
   make STRIP=0
to produce unstripped binaries and non empty debuginfo packages

I just emailed this patch to dmg2img's author as listed in the README file,
Jean-Pierre Demailly demai...@fourier.ujf-grenoble.fr

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

[Bug 742388] Review Request: polybori - Framework for Boolean Rings

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


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

Brendan Jones brendan.jones...@gmail.com changed:

   What|Removed |Added

 CC||brendan.jones...@gmail.com
  QAContact|extras...@fedoraproject.org |brendan.jones...@gmail.com
   Flag||fedora-review?

--- Comment #6 from Brendan Jones brendan.jones...@gmail.com 2011-12-22 
06:42:32 EST ---
I will take this 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 744432] Review Request: pyenet - python bindings for ENet

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


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

Nelson Marques nmo.marq...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX
Last Closed||2011-12-22 07:14:00

--- Comment #4 from Nelson Marques nmo.marq...@gmail.com 2011-12-22 07:14:00 
EST ---
Closing bug as WONTFIX. For further information please consider viewing this
bug report[1].


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

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

[Bug 718430] Review Request: unknown-horizons - a 2D RTS game written in python which uses the fife engine

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


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

Nelson Marques nmo.marq...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
Last Closed|2011-07-05 13:06:16 |2011-12-22 07:12:23

--- Comment #24 from Nelson Marques nmo.marq...@gmail.com 2011-12-22 07:12:23 
EST ---
According to information[1] on BZ#757352 someone seems to be working already on
Unknown Horizons for Fedora. Therefore, I see no need of continuing this.

I will inform upstream of this and let all the involved parties work this issue
while I'm jumping ship with a sincere apology request, it was not my intention
to 'jack someone else's contribution.

Closing bug as WONTFIX.


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

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

[Bug 749752] Review Request: dmg2img - Uncompress the Apple compressed disk image files

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


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

Scott Tsai scottt...@gmail.com changed:

   What|Removed |Added

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

--- Comment #3 from Scott Tsai scottt...@gmail.com 2011-12-22 07:10:48 EST ---
Some issues I see:

1. I think license should be GPLv2+ and MIT. Since the COPYING file included
in the upstream tarball contains GPLv2 I think we should use GPLv2+ instead
of GPL+.

2. As noted by Richard in comment #1, BuildRequires: openssl-devel is
required to build vfdecrypt

3. Dist tag in release field should be %{?dist} instead of %{dist}

4. Unless you're packaging for EPEL 5, I recommend you:
  Remove the BuildRoot tag
  Use make instead of %{__make}
  Remove rm -rf $RPM_BUILD_ROOT in the %install section
  Remove the %clean section
  Remove %defattr from the %files section

5. The dmg2img-1.6.2-nostrip.patch already used in the patch and the patch I
attached both work to produce non empty debuginfo packages. I think my version
has a better chance of being accepted in a future upstream version but that may
just be wishful thinking.


Package Review
==

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



 C/C++ 
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contain kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.


 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[!]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported architecture.
[!]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.

Unless BuildRequires: openssl-devel is added to the spec, it won't build in
mock.

[!]: MUST Buildroot is not present
 Note: Buildroot is not needed unless packager plans to package for EPEL5

I recommend removing the BuildRoot tag.

[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[!]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean is needed only if supporting EPEL
[x]: MUST Sources contain only permissible code or content.
[!]: MUST Each %files section contains %defattr if rpm  4.4
 Note: defattr() present in %files section. This is OK if packaging
 for EPEL5. Otherwise not needed
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[!]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf is only needed if supporting EPEL5
[x]: MUST If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %doc.
[!]: MUST License field in the package spec file matches the actual license.

As noted above I think the License should be GPLv2+ and MIT.

[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.

Package installs if the missing BuildRequires is fixed.

[x]: MUST Requires correct, justified where necessary.

Requires are correct if the missing BuildRequires is fixed.

[x]: MUST Rpmlint output is silent.
rpmlint output:
dmg2img.x86_64: W: spelling-error Summary(en_US) Uncompress - Uncompressed,
Compression, Compressor
dmg2img.x86_64: W: spelling-error %description -l en_US uncompress -
uncompressed, compression, compressor
dmg2img.x86_64: W: spelling-error %description -l en_US dmg - mg, deg, dig
dmg2img.x86_64: W: 

[Bug 756443] Review Request: python-cagraph - A PyGTK widget for plotting charts and graphs

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


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

Juan Orti Alcaine j.orti.alca...@gmail.com changed:

   What|Removed |Added

Summary|Review Request: cagraph - A |Review Request:
   |PyGTK widget for plotting   |python-cagraph - A PyGTK
   |charts and graphs   |widget for plotting charts
   ||and graphs

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

[Bug 769832] New: Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

2011-12-22 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: kmod - Linux kernel module management utilities 
(official replacement for module-init-tools)

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

   Summary: Review Request: kmod - Linux kernel module management
utilities (official replacement for module-init-tools)
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: j...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://jcm.fedorapeople.org/kmod/packages/review/initial/kmod.spec
SRPM URL:
http://jcm.fedorapeople.org/kmod/packages/review/initial/kmod-2-2.fc17.src.rpm
Description: The kmod package provides various programs needed for automatic
loading and unloading of modules under 2.6, 3.x, and later kernels, as well
as other module management programs. Device drivers and filesystems are two
examples of loaded and unloaded modules.

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #1 from Jon Masters j...@redhat.com 2011-12-22 07:51:38 EST ---
This is likely to get an update over the weekend, but it's a rough first cut.
Neil Horman has agreed to help review it for me, so he'll be grabbing this bug.

For the moment, it is intention that the new tools be parallel installed and
have a kmod- prefix. I do not think that should affect review of the package
though, since my understanding is that it is more about general guideline
conformance than whether we're yet ready to replace the binaries. However, I
will read through the material and in the worst case we can hold off a few days
or a week or so before moving forward.

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

[Bug 752089] Review Request: rubygem-wirble - Handful of common Irb features, made easy

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


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

Darryl L. Pierce dpie...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Darryl L. Pierce dpie...@redhat.com 2011-12-22 08:08:34 
EST ---
Thank you!

New Package SCM Request
===
Package Name:  rubygem-wirble
Short Description: Handful of common IRB features, made easy
Owners:mcpierce
Branches:  f15 f16 el6
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 227034] Review Request: asm2-2.1-2jpp - A code manipulation tool to implement adaptable systems

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


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

--- Comment #8 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:09:39 EST 
---
Git done (by process-git-requests).

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

[Bug 678674] Review Request: rubygem-goocanvas - Ruby binding of GooCanvas

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


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

--- Comment #7 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:14:07 EST 
---
Git done (by process-git-requests).

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

[Bug 277931] Review Request: jgoodies-looks - Java lookfeel

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


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

--- Comment #15 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:11:14 EST 
---
Git done (by process-git-requests).

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

[Bug 756445] Review Request: tor-arm - Terminal status monitor for Tor

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


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

--- Comment #3 from Juan Orti Alcaine j.orti.alca...@gmail.com 2011-12-22 
08:14:50 EST ---
A new update to the package:

Spec URL:
https://raw.github.com/jorti/arm-fedora-package/v1.4.4.1-7/SPECS/tor-arm.spec
SRPM URL:
https://github.com/jorti/arm-fedora-package/raw/v1.4.4.1-7/SRPMS/tor-arm-1.4.4.1-7.fc16.src.rpm

The rpmlint output is:

$ rpmlint RPMS/noarch/tor-arm-1.4.4.1-7.fc16.noarch.rpm
SRPMS/tor-arm-1.4.4.1-7.fc16.src.rpm SPECS/tor-arm.spec
tor-arm.noarch: W: spelling-error %description -l en_US anonymizing -
anatomizing, canonizing, anodizing
tor-arm.noarch: W: spelling-error %description -l en_US tty - try, ttys, atty
tor-arm.noarch: W: devel-file-in-non-devel-package
/usr/share/arm/resources/torrcOverride/override.c
tor-arm.noarch: W: devel-file-in-non-devel-package
/usr/share/arm/resources/torrcOverride/override.h
tor-arm.src: W: spelling-error %description -l en_US anonymizing -
anatomizing, canonizing, anodizing
tor-arm.src: W: spelling-error %description -l en_US tty - try, ttys, atty
2 packages and 1 specfiles checked; 0 errors, 6 warnings.

About the devel-file-in-non-devel-package warning, it's the source code of a 
wrapper provided as example if you want to run arm as root to be able to
reconfigure the daemon it connects to. I'm not very sure what to do about this,
I left it untouched.

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

[Bug 769029] Review Request: oxygen-gtk3 - Oxygen GTK+3 theme

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


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

--- Comment #4 from Baif bai...@hotmail.com 2011-12-22 08:12:03 EST ---
excellent, thanks to all of you.

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

[Bug 675976] Review Request: jgoodies-common - Common library shared by JGoodies libraries and applications

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


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

--- Comment #11 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:11:41 EST 
---
Git done (by process-git-requests).

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

[Bug 728826] Review Request: wadofstuff-django-serializers - Extended Django Serializer Module

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


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

--- Comment #12 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:16:46 EST 
---
Git done (by process-git-requests).

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

[Bug 753016] gcr - A library for bits of crypto UI and parsing

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


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

--- Comment #10 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:18:49 EST 
---
Git done (by process-git-requests).

Tom, please take ownership of review BZs.  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 700352] Review Request: perl-DublinCore-Record - Container for Dublin Core metadata elements

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


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

--- Comment #10 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:15:48 EST 
---
Git done (by process-git-requests).

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

[Bug 753016] gcr - A library for bits of crypto UI and parsing

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


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

Tom Hughes t...@compton.nu changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|t...@compton.nu

--- Comment #11 from Tom Hughes t...@compton.nu 2011-12-22 08:25:06 EST ---
I thought I had... To be more precise I thought setting the state to ASSIGNED
assigned it to the person making that change, at least by default. Apparently I
was wrong, at least in this particular implementation of bugzilla.

Done now anyway.

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

[Bug 759760] Review Request: perl-Encode-JIS2K - JIS X 0212 (aka JIS 2000) Encodings

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


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

--- Comment #5 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:21:33 EST 
---
Git done (by process-git-requests).

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

[Bug 768800] Review Request: libwacom - Tablet Information Client Library

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


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

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:23:02 EST 
---
Git done (by process-git-requests).

Matthias, please take ownership of review BZs.  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 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

Neil Horman nhor...@redhat.com changed:

   What|Removed |Added

 CC||nhor...@redhat.com
 AssignedTo|nob...@fedoraproject.org|nhor...@redhat.com
   Flag||fedora-review?

--- Comment #2 from Neil Horman nhor...@redhat.com 2011-12-22 08:29:08 EST ---
I'll start reviewing based on the rpm you have above.

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

[Bug 742388] Review Request: polybori - Framework for Boolean Rings

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


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

--- Comment #7 from Brendan Jones brendan.jones...@gmail.com 2011-12-22 
08:36:53 EST ---
Hi Jerry,

I get rpmlint errors in polybori.x86_64: E: no-binary

Ordinarily this means that this package should be BuildArch: noarch, but after
looking at the script you can see its a wrapper for the ipython script.

This highlights something that needs changing I think, namely that we have an
architecture specific script in /usr/share (which is not permitted)

I haven't tried it but I'd suggest you move the script to /usr/bin where it
belongs and provide a simple patch such that it looks for what it needs in
/usr/share/polybori 


rpm -qlpv /home/bsjones/rpmbuild/RPMS/x86_64/polybori-0.8.0-1.fc16.x86_64.rpm
lrwxrwxrwx1 rootroot   31 Dec 22 14:09
/usr/bin/ipbori - ../share/polybori/ipbori/ipbori
-rw-r--r--1 rootroot 1337 Sep  7 22:25
/usr/share/man/man1/ipbori.1.gz
drwxr-xr-x2 rootroot0 Dec 22 14:09
/usr/share/polybori
drwxr-xr-x2 rootroot0 Dec 22 14:09
/usr/share/polybori/ipbori
-rwxr-xr-x1 rootroot 2003 Dec 22 14:09
/usr/share/polybori/ipbori/ipbori
-rw-r--r--1 rootroot 1162 Dec 22 14:09
/usr/share/polybori/ipbori/ipythonrc-polybori

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

[Bug 769609] Review Request: device-mapper-persistent-data - device-mapper thin provisioning tools

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


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

--- Comment #14 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:38:23 EST 
---
Fixed.

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #3 from Neil Horman nhor...@redhat.com 2011-12-22 08:39:27 EST ---
rpmlint output on source package:
[nhorman@hmsreliant jcm]$ rpmlint ./kmod-2-2.fc17.src.rpm 
kmod.src: W: spelling-error %description -l en_US filesystems - file systems,
file-systems, systematizes
kmod.src:20: W: unversioned-explicit-obsoletes modutils-devel
kmod.src:20: W: unversioned-explicit-obsoletes modutils
kmod.src:78: E: hardcoded-library-path in $RPM_BUILD_ROOT/lib
kmod.src:79: E: hardcoded-library-path in $RPM_BUILD_ROOT/lib/modprobe.d
kmod.src:106: E: hardcoded-library-path in /lib/modprobe.d
kmod.src: W: invalid-url Source0: kmod-2.tar.xz
1 packages and 0 specfiles checked; 3 errors, 4 warnings.

I'd ignore the filesystem warning, I think the spell checker  is overly
sensitive there anyway.

The unversioned Obsoletes is certainly a problem, as It would seem to prevent
the side-by-side installation. I'd suggest removing it given that you plan to
do a kmod-prefix on your tools to make it parallel install.  You can add it
back later.

I expect you want to replace the lib in the mkdirs on line 78/79/106 with
%{_libdir}

The url you want on line 106 I think is
http://packages.profusion.mobi/kmod/%{name}-%{version}.tar.xz

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #4 from Neil Horman nhor...@redhat.com 2011-12-22 08:44:07 EST ---
[nhorman@hmsreliant x86_64]$ rpmlint ./kmod-2-2.fc15.x86_64.rpm 
kmod.x86_64: W: spelling-error %description -l en_US filesystems - file
systems, file-systems, systematizes
kmod.x86_64: W: incoherent-version-in-changelog 2-1 ['2-2.fc15', '2-2']
kmod.x86_64: W: obsolete-not-provided modutils-devel
kmod.x86_64: W: obsolete-not-provided modutils
kmod.x86_64: E: binary-or-shlib-defines-rpath /sbin/kmod-rmmod ['/usr/lib64']
kmod.x86_64: E: binary-or-shlib-defines-rpath /sbin/kmod-modprobe
['/usr/lib64']
kmod.x86_64: E: binary-or-shlib-defines-rpath /sbin/kmod-insmod ['/usr/lib64']
kmod.x86_64: E: binary-or-shlib-defines-rpath /sbin/kmod-modinfo ['/usr/lib64']
kmod.x86_64: E: binary-or-shlib-defines-rpath /sbin/kmod-lsmod ['/usr/lib64']
kmod.x86_64: W: no-documentation
kmod.x86_64: W: no-manual-page-for-binary kmod-lsmod
kmod.x86_64: W: no-manual-page-for-binary kmod-rmmod
kmod.x86_64: W: no-manual-page-for-binary kmod-modinfo
kmod.x86_64: W: no-manual-page-for-binary kmod-insmod
kmod.x86_64: W: no-manual-page-for-binary kmod-modprobe
kmod.x86_64: W: dangerous-command-in-%pre mv
1 packages and 0 specfiles checked; 5 errors, 11 warnings.

Nothing really here that the source file didn't catch.  T

The changelog is an easy fix

The obsoletes have already been discussed

he binary-or-shlib errors are the result of the hardcoded /lib in the spec file
and will go away when thats fixed.

The docs should be included (at least NEWS README TODO and COPYING).  The last
one will help re-enforce the license selected in the spec file.

The man page stuff can be ignored, I'm sure it will get added as the package
continues to mature.

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #7 from Jon Masters j...@redhat.com 2011-12-22 08:48:52 EST ---
Thanks for the initial review (comment 6 was written before seeing yours). I'm
going to crash out at some point soon and get back to this later.

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #6 from Jon Masters j...@redhat.com 2011-12-22 08:48:01 EST ---
NOTE: The use of /lib(64) is obviously required at this point. We might get a
unified filesystem in the future, but for now it needs to be done this way. I
have been told it is best to have the generic devel .so in the -devel package
and under /usr. I'll look into correct practice there - have to admit, I'm not
so used to splitting libraries up between two lib locations for the same thing.

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #5 from Neil Horman nhor...@redhat.com 2011-12-22 08:47:50 EST ---
[nhorman@hmsreliant x86_64]$ rpmlint kmod-debuginfo-2-2.fc15.x86_64.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Good.


[nhorman@hmsreliant x86_64]$ rpmlint kmod-devel-2-2.fc15.x86_64.rpm
kmod-devel.x86_64: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
Good.  The warning can be ignored, that belongs in the base package.

[nhorman@hmsreliant x86_64]$ rpmlint kmod-libs-2-2.fc15.x86_64.rpm
kmod-libs.x86_64: W: spelling-error %description -l en_US runtime - run time,
run-time, rudiment
kmod-libs.x86_64: W: no-documentation
kmod-libs.x86_64: W: devel-file-in-non-devel-package /lib64/libkmod.so

Documentation and spelling errors can be ignored.  The library file needs
fixing.  IIRC the right thing to do is version that library file, which will
squash the error, and then add a %post script to the -devel package to symlink
libkmod.so to its corresponding versioned library.

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

[Bug 752089] Review Request: rubygem-wirble - Handful of common Irb features, made easy

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


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

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2011-12-22 08:55:47 EST 
---
Git done (by process-git-requests).

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

[Bug 752993] gnome-dictionary - A dictionary application for GNOME

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


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

Matthias Clasen mcla...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #12 from Matthias Clasen mcla...@redhat.com 2011-12-22 09:01:20 
EST ---
New Package SCM Request
===
Package Name: gnome-dictionary
Short Description: A dictionary application for GNOME 
Owners: mclasen cosimoc
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 753016] gcr - A library for bits of crypto UI and parsing

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


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

Matthias Clasen mcla...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-12-22 09:13:09

--- Comment #12 from Matthias Clasen mcla...@redhat.com 2011-12-22 09:13:09 
EST ---
build underway

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #9 from Neil Horman nhor...@redhat.com 2011-12-22 09:19:55 EST ---
Jon, FWIW a good example usage of -devel and base library usage is zlib.  the
base package has the versioned library, while the devel package includes a
symlink file in the manifest that is unversioned and points to the versioned
file from the base package.  by doing that the, along with the Requires line
that you already include in the -devel part of the spec, the devel library
always points to the correct version of the base library as per the version of
the rpms you have installed.

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #8 from Neil Horman nhor...@redhat.com 2011-12-22 09:15:35 EST ---
The package must be named according to the Package Naming Guidelines . - Check

The spec file name must match the base package %{name}, in the format
%{name}.spec - Check

The package must meet the Packaging Guidelines - Check, nothing obviously
wrong.  The COPYINg file needs to be included to support the license tag inthe
spec file, but thats already been mentioned in the rpmlint output.

The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines . - check

The License field in the package spec file must match the actual license. [3] -
Check

If (and only if) the source package includes the text of the license(s) in its
own file, then that file, containing the text of the license(s) for the package
must be included in %doc. - This will be fixed when the warning about the lack
of docs is fixed from the rpmlint output

The spec file must be written in American English. - Check

The spec file for the package MUST be legible. - Check

The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this. - Check, although the url needs updating
in the spec file:
[nhorman@hmsreliant jcm]$ rpm -ivh kmod-2-2.fc17.src.rpm 
   1:kmod   warning: user jcm does not exist - using root
warning: group jcm does not exist - using root
warning: user jcm does not exist - using root 98%)
warning: group jcm does not exist - using root
### [100%]
[nhorman@hmsreliant jcm]$ cd ~/rpmbuild/SOURCES/^C
[nhorman@hmsreliant jcm]$ md5sum ~/rpmbuild/SOURCES/kmod-2.tar.xz 
6017364434377f6724f749d7a28c5d7a  /home/nhorman/rpmbuild/SOURCES/kmod-2.tar.xz
wget: /home/nhorman/.netrc:1: unknown token “sZ3MK4yz5UH6”
Resolving packages.profusion.mobi... 74.207.229.112
Connecting to packages.profusion.mobi|74.207.229.112|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 262484 (256K) [application/x-tar]
Saving to: “kmod-2.tar.xz”

100%[==]
262,484  824K/s   in 0.3s

2011-12-22 09:02:27 (824 KB/s) - “kmod-2.tar.xz” saved [262484/262484]

[nhorman@hmsreliant jcm]$ md5sum ./kmod-2.tar.xz 
6017364434377f6724f749d7a28c5d7a  ./kmod-2.tar.xz



The package MUST successfully compile and build into binary rpms on at least
one primary architecture - Check, see
http://koji.fedoraproject.org/koji/taskinfo?taskID=3601228

If the package does not successfully compile, build or work on an architecture,
then those architectures should be listed in the spec in ExcludeArch. Each
architecture listed in ExcludeArch MUST have a bug filed in bugzilla,
describing the reason that the package does not compile/build/work on that
architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line - Not applicable, see previous


All build dependencies must be listed in BuildRequires, except for any that are
listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense. - check, although, why
is glibc-static required.  The initramfs has glibc available doesn't it?  We
should be able to build dynamically here.


The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.- Not
applicable, no human readable output provided from the pkg.

Every binary RPM package (or subpackage) which stores shared library files (not
just symlinks) in any of the dynamic linker's default paths, must call ldconfig
in %post and %postun. - check

Packages must NOT bundle copies of system libraries.- Check

If the package is designed to be relocatable, the packager must state this fact
in the request for review, along with the rationalization for relocation of
that specific package. Without this, use of Prefix: /usr is considered a
blocker. - Not applicable, not relocatable.

 A package must own all directories that it creates. If it does not create a
directory that it uses, then it should require a package which does create that
directory. - Check

 A Fedora package must not list a file more than once in the spec file's %files
listings. (Notable exception: license texts in specific situations)- Check

Permissions on files must be set properly. Executables should be set with
executable permissions, for example. - Check

Each package must consistently use macros. - check

The package must contain code, or permissable content. - Check

 Large documentation files must go in a -doc subpackage. (The 

[Bug 767583] Review Request: python-poppler-qt4

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


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

Brendan Jones brendan.jones...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Brendan Jones brendan.jones...@gmail.com 2011-12-22 
09:22:54 EST ---
New Package SCM Request
===
Package Name: python-poppler-qt4
Short Description: python bindings to poppler-qt4
Owners: bsjones
Branches: f15 f16
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 752993] gnome-dictionary - A dictionary application for GNOME

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


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

--- Comment #13 from Jon Ciesla limburg...@gmail.com 2011-12-22 09:22:20 EST 
---
Git done (by process-git-requests).

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

[Bug 752089] Review Request: rubygem-wirble - Handful of common Irb features, made easy

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


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

Darryl L. Pierce dpie...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-12-22 09:26:37

--- Comment #7 from Darryl L. Pierce dpie...@redhat.com 2011-12-22 09:26:37 
EST ---
Thank you, Jon and Mamoru!

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

[Bug 767583] Review Request: python-poppler-qt4

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


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

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2011-12-22 09:37:07 EST 
---
Git done (by process-git-requests).

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

[Bug 618354] Review Request: cwallpaper - A front end for fbsetbg, Esetroot, feh, and other wallpaper changers

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


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG
 Blocks||201449(FE-DEADREVIEW)
  Status Whiteboard|StalledSubmitter|
   Flag|fedora-review?  |
Last Closed||2011-12-22 10:00:19

--- Comment #7 from Martin Gieseking martin.giesek...@uos.de 2011-12-22 
10:00:19 EST ---
Closed according to
http://fedoraproject.org/wiki/Policy_for_stalled_package_reviews#Submitter_not_responding

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

[Bug 760033] Review Request: kde-plasma-publictransport - Public Transport plasma applet

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


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

--- Comment #9 from Gregor Tätzner gre...@freenet.de 2011-12-22 10:23:06 EST 
---
(In reply to comment #8)
  The problem is, in libs subpackage are also icons for timetablemate 
  included.
  This means I can't move them all together into the main package because
  timetable doesn't need the rest of the main package.
 
 Well, right now it does because you have a Requires…
 
 To achieve what you want, please:
 * remove the Requires: %{name} = %{version}-%{release} from -libs and
check

 * make the icon scriptlets apply to the -libs subpackage where the icons
 actually are. (You'll have to remove the -p /sbin/ldconfig and write
 /sbin/ldconfig as a scriptlet line instead, so that the other stuff can be
 added.)
What do you mean exactly by make the icon scriptlets apply to the -libs
subpackage I'm a little bit confused.

Another point: Atm my package is missing some doc files (changelogs from
various plasmoids in the main package) How can I include them? They have all
the same file name.

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

[Bug 766932] Review Request: icfg - utiltiy for scriptable editing of network interface files

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


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

Neil Horman nhor...@redhat.com changed:

   What|Removed |Added

 AssignedTo|ita...@ispbrasil.com.br |j...@redhat.com

--- Comment #4 from Neil Horman nhor...@redhat.com 2011-12-22 10:24:51 EST ---
Thank you for your offer, but I've already made a deal to have jcm review this,
and I've started reviewing his bug, so I'd just as soon he did the official
review on this.  You're of course welcome to review/comment as well.

To answer your question, the man page should have examples of this, if it
doesn't I'll add it.  But the command would be something like:

icfg -c -i vlan10 -t vlan -s PHYSDEV=eth0

Replace vlan10 and eth0 of course with the appropriate corresponding vlan and
physical device names that you want.

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

[Bug 766932] Review Request: icfg - utiltiy for scriptable editing of network interface files

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


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

--- Comment #5 from Neil Horman nhor...@redhat.com 2011-12-22 10:25:51 EST ---
Terje, I'll fix those, along with whatever else jcm finds, 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 767583] Review Request: python-poppler-qt4

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


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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

[Bug 767583] Review Request: python-poppler-qt4

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


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

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-12-22 
10:31:24 EST ---
python-poppler-qt4-0.16.2-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/python-poppler-qt4-0.16.2-1.fc16

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

[Bug 767583] Review Request: python-poppler-qt4

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


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

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-12-22 
10:44:58 EST ---
python-poppler-qt4-0.16.2-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/python-poppler-qt4-0.16.2-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 767042] Review Request: js-of-ocaml - OCaml to Javascript compiler

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


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

--- Comment #6 from Jerry James loganje...@gmail.com 2011-12-22 10:54:13 EST 
---
(In reply to comment #5)
 Indeed. I meant to say this in my mysterious This is way I didn't split
 produce a separate -debug package sentence in comment 0 as well. (Apparently
 my command of the English language comes and goes ..)

Heh.  I'd say your command of English is quite good.  It's certainly far better
than my command of Mandarin Chinese, which I used to speak (but not read or
write) semi-fluently 20-some years ago.  Today I have trouble putting a
coherent sentence together.

 I used hyphens because that's what Debian did:

Good enough.  Thanks for the explanation.

And I screwed up on some of the pluses and minuses.  Sorry about that.  This is
what happens when I cut  paste from a template.  I'm going to make the
template have nothing in between the brackets so I'm forced to think about the
contents of every single one in the future.  Let me draw your attention to 2
more items, with the correct character between the brackets this time:

MUST:
[=] package meets the packaging guidelines: but consider adding -p to the
cp invocations, as per
https://fedoraproject.org/wiki/Packaging:Guidelines#Timestamps.
[-] license field matches the actual license: not quite.  The LICENSE file
mentions exceptions, so the license should be LGPLv2+ with exceptions.  Also,
some of the examples are GPLv2+ (e.g., examples/graph_viewer/scene.mli), and
one is WTFPL (examples/boulderdash/boulderdash.ml).  I think the -doc
subpackage will need to have a license along the lines of (LGPLv2+ with
exceptions) and GPLv2+ and WTFPL.

Everything else looks fine.

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

[Bug 755104] Review Request: condor-wallaby - condor related tools for interacting with wallaby

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


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

Nuno Santos nsan...@redhat.com changed:

   What|Removed |Added

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

--- Comment #1 from Nuno Santos nsan...@redhat.com 2011-12-22 10:58:19 EST ---
I've reviewed the package, comments below in the guidelines template:


See notes - MUST: rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.
$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/condor-wallaby*.rpm
condor-wallaby.src:36: W: unversioned-explicit-obsoletes
condor-remote-configuration
condor-wallaby.src:54: W: unversioned-explicit-obsoletes
condor-remote-configuration-server
condor-wallaby.src: W: no-%build-section
condor-wallaby.src: W: invalid-url Source0: condor-wallaby-4.1.2.tar.gz
condor-wallaby-client.noarch: W: obsolete-not-provided
condor-remote-configuration
condor-wallaby-client.noarch: W: conffile-without-noreplace-flag
/etc/condor/config.d/99configd.config
condor-wallaby-client.noarch: W: no-manual-page-for-binary condor_configd
2 packages and 0 specfiles checked; 0 errors, 7 warnings.

NOTES: no build section, invalid url for source, obsolete not provided, and
config file without noreplace flag warnings should be fixed

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

OK - MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption.

OK - MUST: The package must meet the Packaging Guidelines .

OK (ASL 2.0) - MUST: The package must be licensed with a Fedora approved
license and meet the Licensing Guidelines .

OK - MUST: The License field in the package spec file must match the actual
license.

OK - MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.

OK - MUST: The spec file must be written in American English.

OK - MUST: The spec file for the package MUST be legible.

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

NOTES: unable to check md5sums, is it possible to get the tarfile upstream?

OK - MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.

$ mock -r fedora-rawhide-i386 condor-wallaby-4.1.2-1.fc15.src.rpm 
INFO: mock.py version 1.1.18 starting...
State Changed: init plugins
INFO: selinux enabled
State Changed: start
INFO: Start(condor-wallaby-4.1.2-1.fc15.src.rpm)  Config(fedora-rawhide-i386)
State Changed: lock buildroot
State Changed: clean
INFO: chroot (/var/lib/mock/fedora-rawhide-i386) unlocked and deleted
State Changed: unlock buildroot
State Changed: init
State Changed: lock buildroot
Mock Version: 1.1.18
INFO: Mock Version: 1.1.18
INFO: calling preinit hooks
INFO: enabled root cache
State Changed: unpacking root cache
INFO: enabled yum cache
State Changed: cleaning yum metadata
INFO: enabled ccache
State Changed: running yum
State Changed: unlock buildroot
INFO: Installed packages:
State Changed: setup
State Changed: build
INFO: Done(condor-wallaby-4.1.2-1.fc15.src.rpm) Config(fedora-rawhide-i386) 2
minutes 17 seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-i386/result
State Changed: end


NA - MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.

OK - MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.

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

NA - MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun.

OK - MUST: Packages must NOT bundle copies of system libraries.

NA - MUST: 

[Bug 702143] Review Request: wallaby - configuration service for Condor pools

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


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

Will Benton wi...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #12 from Will Benton wi...@redhat.com 2011-12-22 11:11:14 EST ---
New Package SCM Request
===
Package Name: wallaby
Short Description: configuration service for Condor pools
Owners: willb
Branches: f15 f16
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 755079] Review Request: condor-wallaby-base-db - A condor database for wallaby

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


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

Nuno Santos nsan...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+

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

[Bug 755079] Review Request: condor-wallaby-base-db - A condor database for wallaby

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


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

Nuno Santos nsan...@redhat.com changed:

   What|Removed |Added

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

--- Comment #3 from Nuno Santos nsan...@redhat.com 2011-12-22 11:19:36 EST ---
I've reviewed the package as per the packaging guidelines template:


OK - MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.

$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/condor-wallaby-base-db*.rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.


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

OK - MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption.

OK - MUST: The package must meet the Packaging Guidelines .

OK (ASL 2.0) - MUST: The package must be licensed with a Fedora approved
license and meet the Licensing Guidelines .

OK - MUST: The License field in the package spec file must match the actual
license.

OK - MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.

OK - MUST: The spec file must be written in American English.

OK - MUST: The spec file for the package MUST be legible.

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

$ rpm2cpio condor-wallaby-base-db-1.18-1.fc15.src.rpm | cpio -ivd ; md5sum
condor-wallaby-base-db-1.18.tar.gz
0001-Build-fixes.patch
0002-Packaging-fixes.patch
condor-wallaby-base-db-1.18.tar.gz
condor-wallaby-base-db.spec
112 blocks
55af66139852f6a0cb97041ea26402b8  condor-wallaby-base-db-1.18.tar.gz

$ wget
https://fedorahosted.org/releases/g/r/grid/condor-wallaby-base-db-1.18.tar.gz ;
md5sum condor-wallaby-base-db-1.18.tar.gz
--2011-12-22 11:14:32-- 
https://fedorahosted.org/releases/g/r/grid/condor-wallaby-base-db-1.18.tar.gz
Resolving fedorahosted.org... 66.135.62.191
Connecting to fedorahosted.org|66.135.62.191|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 44076 (43K) [application/x-gzip]
Saving to: “condor-wallaby-base-db-1.18.tar.gz”

100%[==] 44,076 
--.-K/s   in 0.1s

2011-12-22 11:14:33 (308 KB/s) - “condor-wallaby-base-db-1.18.tar.gz” saved
[44076/44076]

55af66139852f6a0cb97041ea26402b8  condor-wallaby-base-db-1.18.tar.gz


OK - MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.

$ mock -r fedora-rawhide-i386 condor-wallaby-base-db-1.18-1.fc15.src.rpm 
INFO: mock.py version 1.1.18 starting...
State Changed: init plugins
INFO: selinux enabled
State Changed: start
INFO: Start(condor-wallaby-base-db-1.18-1.fc15.src.rpm) 
Config(fedora-rawhide-i386)
State Changed: lock buildroot
State Changed: clean
INFO: chroot (/var/lib/mock/fedora-rawhide-i386) unlocked and deleted
State Changed: unlock buildroot
State Changed: init
State Changed: lock buildroot
Mock Version: 1.1.18
INFO: Mock Version: 1.1.18
INFO: calling preinit hooks
INFO: enabled root cache
State Changed: unpacking root cache
INFO: enabled yum cache
State Changed: cleaning yum metadata
INFO: enabled ccache
State Changed: running yum
State Changed: unlock buildroot
INFO: Installed packages:
State Changed: setup
State Changed: build
INFO: Done(condor-wallaby-base-db-1.18-1.fc15.src.rpm)
Config(fedora-rawhide-i386) 1 minutes 44 seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-i386/result
State Changed: end


NA - MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.

OK - MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.

NA - MUST: The spec file MUST handle locales properly. This is done by using
the %find_lang macro. Using %{_datadir}/locale/* is 

[Bug 711230] Review Request: ruby-rhubarb - simple versioned object-graph persistence for ruby

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


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

--- Comment #5 from Will Benton wi...@redhat.com 2011-12-22 11:21:02 EST ---
Vit, it is packaged that way so it will work on EL5 (which doesn't have
RubyGems).

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

[Bug 711229] Review Request: ruby-spqr - easy QMF agent framework for Ruby

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


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

Will Benton wi...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Will Benton wi...@redhat.com 2011-12-22 11:24:16 EST ---
New Package SCM Request
===
Package Name: ruby-spqr
Short Description: easy QMF agent framework for Ruby
Owners: willb
Branches: f15 f16
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 711230] Review Request: ruby-rhubarb - simple versioned object-graph persistence for ruby

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


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

Will Benton wi...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #6 from Will Benton wi...@redhat.com 2011-12-22 11:22:17 EST ---
New Package SCM Request
===
Package Name: ruby-rhubarb
Short Description: simple object-graph persistence library for Ruby
Owners: willb
Branches: f15 f16
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 769002] Review Request: sugar-toolkit-gtk3 - the Sugar toolkit for gtk3

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


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

--- Comment #4 from Simon Schampijer si...@schampijer.de 2011-12-22 11:30:03 
EST ---
Thanks Peter for the review, I updated the spec file to follow your
suggestions.

The sfs-address-errors I did not fix yet, as I first would like to hear
clarification on https://bugzilla.redhat.com/show_bug.cgi?id=769002#c3, happy
to do it then. The  'invalid-lc-messages-dir' can not be fixed as we support
languages that are not as popular.

SPEC: http://dev.laptop.org/~erikos/rpms/sugar-toolkit-gtk3.spec

SOURCES:
http://download.sugarlabs.org/sources/sucrose/glucose/sugar-toolkit-gtk3/sugar-toolkit-gtk3-0.95.2.tar.bz2

SRPM:
http://dev.laptop.org/~erikos/rpms/sugar-toolkit-gtk3-0.95.2-1.fc16.src.rpm

I did as well a scratch build in koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3601450

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

[Bug 711230] Review Request: ruby-rhubarb - simple versioned object-graph persistence for ruby

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


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

--- Comment #7 from Jon Ciesla limburg...@gmail.com 2011-12-22 11:39:35 EST 
---
Git done (by process-git-requests).

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

[Bug 702143] Review Request: wallaby - configuration service for Condor pools

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


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

--- Comment #13 from Jon Ciesla limburg...@gmail.com 2011-12-22 11:38:15 EST 
---
Git done (by process-git-requests).

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

[Bug 759057] Review Request: libSOIL - Simple OpenGL Image Library

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


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

--- Comment #2 from Julian Sikorski beleg...@gmail.com 2011-12-22 11:37:02 
EST ---
Sure:
Spec URL: http://belegdol.fedorapeople.org/SOIL.spec
SRPM URL: http://belegdol.fedorapeople.org/SOIL-1.07-2.20080706.fc16.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 711229] Review Request: ruby-spqr - easy QMF agent framework for Ruby

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


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

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2011-12-22 11:39:03 EST 
---
Git done (by process-git-requests).

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

[Bug 769002] Review Request: sugar-toolkit-gtk3 - the Sugar toolkit for gtk3

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


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

Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Peter Robinson pbrobin...@gmail.com 2011-12-22 11:50:54 
EST ---
Looks fine to me, core issues addressed. 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 709328] Review Request: psi-plus - Jabber client based on Qt

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


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

Raphael Groner raph...@web.de changed:

   What|Removed |Added

   Flag||needinfo?(dr...@land.ru)

--- Comment #25 from Raphael Groner raph...@web.de 2011-12-22 12:08:37 EST ---
Tried to build rev 20111221git5158 with files from
github.com/RussianFedora/psi-plus ...

g++ -Wl,-O1 -shared -o libtranslateplugin.so translateplugin.o   -L/usr/lib64
-lQtXml -lQtGui -lQtCore -lpthread  
translateplugin.o: file not recognized: File truncated
collect2: ld gab 1 als Ende-Status zur?ck
make: *** [libtranslateplugin.so] Fehler 1
Fehler: Fehler-Status beim Beenden von /var/tmp/rpm-tmp.hNl26N (%build)


Fehler beim Bauen des RPM:
Fehler-Status beim Beenden von /var/tmp/rpm-tmp.hNl26N (%build)

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

Neil Horman nhor...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(j...@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 767042] Review Request: js-of-ocaml - OCaml to Javascript compiler

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


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

--- Comment #7 from Scott Tsai scottt...@gmail.com 2011-12-22 13:16:50 EST ---
(In reply to comment #6)
Updated spec at same URL:
1. Preserve timestamps with cp -p
2. Update license fields

diff --git a/js-of-ocaml.spec b/js-of-ocaml.spec
index 16c048d..7ca3b82 100644
--- a/js-of-ocaml.spec
+++ b/js-of-ocaml.spec
@@ -7,7 +7,7 @@ Version:1.0.9
 Release:1%{?dist}
 Summary:An OCaml to Javascript compiler

-License:LGPLv2+
+License:LGPLv2+ with exceptions
 URL:http://ocsigen.org/js_of_ocaml
 Source0:http://ocsigen.org/download/js_of_ocaml-%{version}.tar.gz

@@ -38,6 +38,7 @@ Its key features are the following:
 Summary:User manual and other documentation for %{name}
 Group:  Development/Libraries
 BuildArch:  noarch
+License:(LGPLv2+ with exceptions) and GPLv2+ and WTFPL

 %descriptiondoc
 The %{name}-doc package contains the user manual, API
@@ -56,14 +57,14 @@ export OCAMLFIND_DESTDIR=$RPM_BUILD_ROOT%{_libdir}/ocaml
 mkdir -p $OCAMLFIND_DESTDIR $OCAMLFIND_DESTDIR/stublibs
 D=$RPM_BUILD_ROOT%{_libdir}/ocaml/js_of_ocaml
 mkdir -p $D
-cp lib/META \
+cp -p lib/META \
 lib/*.{cmi,cma,a,mli} \
 lib/syntax/pa_js.cmo \
 runtime/runtime.js \
 $D
 mkdir -p $RPM_BUILD_ROOT%{_bindir}
-cp compiler/js_of_ocaml $RPM_BUILD_ROOT%{_bindir}/
-cp lib/dlljs_of_ocaml.so $OCAMLFIND_DESTDIR/stublibs/
+cp -p compiler/js_of_ocaml $RPM_BUILD_ROOT%{_bindir}/
+cp -p lib/dlljs_of_ocaml.so $OCAMLFIND_DESTDIR/stublibs/

 strip $OCAMLFIND_DESTDIR/stublibs/dll*.so
 chrpath --delete $OCAMLFIND_DESTDIR/stublibs/dll*.so
@@ -76,7 +77,7 @@ echo '-b /usr/bin/js_of_ocaml' 
$RPM_BUILD_ROOT/etc/prelink.conf.d/js-of-ocaml.
 # docs
 find examples '(' -name '*.cmi' -or -name '*.cmo' -or -name '*.byte' -or -name
'.depend' ')' -exec rm '{}' ';'
 # Replacing Makefile.common allows the user to build the examples
-cp %{SOURCE1} examples/Makefile.common
+cp -p %{SOURCE1} examples/Makefile.common

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

[Bug 461407] Review Request: laf-plugin - Generic plugin framework for Java look-and-feels

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


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

Orion Poplawski or...@cora.nwra.com changed:

   What|Removed |Added

 CC||or...@cora.nwra.com

--- Comment #29 from Orion Poplawski or...@cora.nwra.com 2011-12-22 13:20:28 
EST ---
I'd like to see this in epel 6.  mycae - are you willing to maintain it or
shall I?

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

[Bug 563471] Tracker: Review Requests for Fedora Security Lab related packages

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


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

Athmane Madjoudj athma...@gmail.com changed:

   What|Removed |Added

 Depends on||769919

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

[Bug 769919] Review Request: hydra - Very fast network log-on cracker

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


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

Athmane Madjoudj athma...@gmail.com changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR),
   ||563471(FE-SECLAB)

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

[Bug 769919] New: Review Request: hydra - Very fast network log-on cracker

2011-12-22 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: hydra - Very fast network log-on cracker

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

   Summary: Review Request: hydra - Very fast network log-on
cracker
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: athma...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://athmane.fedorapeople.org/pkgs/hydra.spec
SRPM URL: http://athmane.fedorapeople.org/pkgs/hydra-7.1-1.fc16.src.rpm

Description:
Hydra is a parallelized log-in cracker which supports numerous protocols to 
attack. New modules are easy to add, beside that, it is flexible and very fast.

This tool gives researchers and security consultants the possibility to show 
how easy it would be to gain unauthorized access from remote to a system.

Currently this tool supports: TELNET, FTP, HTTP, HTTPS, HTTP-PROXY, LDAP, SMB,
SMBNT, MS-SQL, MySQL, REXEC, SOCKS5, VNC, POP3, IMAP, NNTP, PCNFS, ICQ, 
Cisco auth, Cisco enable, Cisco AAA (incorporated in telnet module). 

Notes:
This is my first package and I need a sponsor.

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

[Bug 769919] Review Request: hydra - Very fast network log-on cracker

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


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

--- Comment #1 from Athmane Madjoudj athma...@gmail.com 2011-12-22 13:30:35 
EST ---
rpmlint output:

$ rpmlint hydra-7.1-1.fc16.src.rpm
hydra.src: W: spelling-error %description -l en_US parallelized - paralleled,
palatalized, pluralized
hydra.src: W: spelling-error %description -l en_US auth - auto, Ruth, author
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint hydra-7.1-1.fc16.x86_64.rpm
hydra.x86_64: W: spelling-error %description -l en_US parallelized -
paralleled, palatalized, pluralized
hydra.x86_64: W: spelling-error %description -l en_US auth - auto, Ruth,
author
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint hydra-frontend-7.1-1.fc16.x86_64.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint hydra.spec
0 packages and 1 specfiles checked; 0 errors, 0 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 769922] New: Review Request: trac-navadd-plugin - Allows the addition of custom items

2011-12-22 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: trac-navadd-plugin - Allows the addition of custom 
items

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

   Summary: Review Request: trac-navadd-plugin - Allows the
addition of custom items
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: andrea.v...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL:
http://averi.fedorapeople.org/RPMs/trac-navadd-plugin/trac-navadd-plugin.spec
SRPM URL:
http://averi.fedorapeople.org/RPMs/trac-navadd-plugin/trac-navadd-plugin-0.9-1.fc16.src.rpm
Description: A trac plugin that should allow you to add custom items
(buttons) 
to both the main and the meta navigation bar.

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

[Bug 769919] Review Request: hydra - Very fast network log-on cracker

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


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

--- Comment #2 from Athmane Madjoudj athma...@gmail.com 2011-12-22 14:07:53 
EST ---
Updated license to 'GPLv3 with exceptions' since it seems to be GPLv3 with
OpenSSL exception

Spec URL: http://athmane.fedorapeople.org/pkgs/hydra.spec
SRPM URL: http://athmane.fedorapeople.org/pkgs/hydra-7.1-2.fc16.src.rpm

rpmlint output:

$ rpmtlint hydra-7.1-2.fc16.src.rpm
hydra.src: W: spelling-error %description -l en_US parallelized - paralleled,
palatalized, pluralized
hydra.src: W: spelling-error %description -l en_US auth - auto, Ruth, author
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmtlint hydra-7.1-2.fc16.x86_64.rpm
hydra.x86_64: W: spelling-error %description -l en_US parallelized -
paralleled, palatalized, pluralized
hydra.x86_64: W: spelling-error %description -l en_US auth - auto, Ruth,
author
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmtlint hydra-frontend-7.1-2.fc16.x86_64.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmtlint hydra.spec
0 packages and 1 specfiles checked; 0 errors, 0 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 759758] Review Request: perl-Encode-HanExtra - Extra sets of Chinese encodings

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


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

Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-12-22 14:28:05

--- Comment #8 from Xavier Bachelot xav...@bachelot.org 2011-12-22 14:28:05 
EST ---
Imported and build for Rawhide, F16, F15, EL6 and EL5.

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

[Bug 759760] Review Request: perl-Encode-JIS2K - JIS X 0212 (aka JIS 2000) Encodings

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


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

Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-12-22 14:28:52

--- Comment #6 from Xavier Bachelot xav...@bachelot.org 2011-12-22 14:28:52 
EST ---
Imported and built for Rawhide, F16, F15, EL6 and EL5.

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

[Bug 767042] Review Request: js-of-ocaml - OCaml to Javascript compiler

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


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

Jerry James loganje...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #8 from Jerry James loganje...@gmail.com 2011-12-22 14:27:58 EST 
---
Looks good.  This package is 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 767042] Review Request: js-of-ocaml - OCaml to Javascript compiler

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


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

Scott Tsai scottt...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #9 from Scott Tsai scottt...@gmail.com 2011-12-22 14:35:23 EST ---
New Package SCM Request
===
Package Name: js-of-ocaml
Short Description: An OCaml to Javascript compiler
Owners: scottt
Branches: f16
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 767042] Review Request: js-of-ocaml - OCaml to Javascript compiler

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


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

--- Comment #10 from Jon Ciesla limburg...@gmail.com 2011-12-22 14:58:50 EST 
---
Git done (by process-git-requests).

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

[Bug 769919] Review Request: hydra - Very fast network log-on cracker

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


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

--- Comment #3 from Athmane Madjoudj athma...@gmail.com 2011-12-22 15:16:46 
EST ---
Koji scratch build (x86_64 and i686):
http://koji.fedoraproject.org/koji/taskinfo?taskID=3601861

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

[Bug 769056] Review Request: wbox - HTTP testing tool and configuration-less HTTP server

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


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

--- Comment #4 from Athmane Madjoudj athma...@gmail.com 2011-12-22 15:18:11 
EST ---
Koji scratch build (for x86_64 and i686):
http://koji.fedoraproject.org/koji/taskinfo?taskID=3601866

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

[Bug 767042] Review Request: js-of-ocaml - OCaml to Javascript compiler

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


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

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-12-22 15:29:47 EST ---
js-of-ocaml-1.0.9-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/js-of-ocaml-1.0.9-1.fc16

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

[Bug 767042] Review Request: js-of-ocaml - OCaml to Javascript compiler

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


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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

[Bug 709328] Review Request: psi-plus - Jabber client based on Qt

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


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

--- Comment #26 from Raphael Groner raph...@web.de 2011-12-22 16:09:36 EST ---
Scratch build for rev 20111221git5158 with files from
github.com/RussianFedora/psi-plus

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

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

[Bug 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

Jon Masters j...@redhat.com changed:

   What|Removed |Added

   Flag|needinfo?(j...@redhat.com)   |

--- Comment #10 from Jon Masters j...@redhat.com 2011-12-22 16:12:30 EST ---
Taking care of these now. Just discussed all of the existing issues with Neil.

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

[Bug 709328] Review Request: psi-plus - Jabber client based on Qt

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


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

--- Comment #27 from Raphael Groner raph...@web.de 2011-12-22 16:12:10 EST ---
Add BuildRequires libicu-devel to psi-plus.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 769832] Review Request: kmod - Linux kernel module management utilities (official replacement for module-init-tools)

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


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

--- Comment #11 from Neil Horman nhor...@redhat.com 2011-12-22 16:14:12 EST 
---
Btw, jon, you were just asking on the phone about a macro to generically encode
/lib and /lib64 in an arch independent way.  %{_lib} will do that for you.  Its
defined in /usr/lib/rpm/macros in fedora.  Hope that helps!

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

[Bug 666311] Review Request: fritzing - Intuitive EDA platform from prototype to product

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


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

--- Comment #29 from Ed Marshall e...@logic.net 2011-12-22 17:33:45 EST ---
The build problem cleared up with today's rawhide (changes to libX11, it
appears). :)

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

[Bug 666311] Review Request: fritzing - Intuitive EDA platform from prototype to product

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


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

--- Comment #29 from Ed Marshall e...@logic.net 2011-12-22 17:33:45 EST ---
The build problem cleared up with today's rawhide (changes to libX11, it
appears). :)

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

[Bug 734014] Review Request: nafees-pakistani-web-naskh-fonts nafees-pakistani-web-naskh-fonts for writing Urdu

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-12-22 17:33:29 EST ---
Package nafees-pakistani-web-naskh-fonts-2.0-2.fc16:
* should fix your issue,
* was pushed to the Fedora 16 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing
nafees-pakistani-web-naskh-fonts-2.0-2.fc16'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2011-17261/nafees-pakistani-web-naskh-fonts-2.0-2.fc16
then log in and leave karma (feedback).

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

[Bug 757484] Review Request: perl-Router-Simple - Simple HTTP router

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Router-Simple-0.09-1.f
   ||c15
 Resolution||ERRATA
Last Closed||2011-12-22 17:33:35

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-12-22 
17:33:35 EST ---
perl-Router-Simple-0.09-1.fc15 has been pushed to the Fedora 15 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 757484] Review Request: perl-Router-Simple - Simple HTTP router

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


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

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

   What|Removed |Added

   Fixed In Version|perl-Router-Simple-0.09-1.f |perl-Router-Simple-0.09-1.f
   |c15 |c16

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-12-22 
17:36:20 EST ---
perl-Router-Simple-0.09-1.fc16 has been pushed to the Fedora 16 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 741494] Review Request: sugar-pukllanapac - A sliding puzzle game

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2011-12-22 17:41:05 EST ---
sugar-pukllanapac-8-2.fc15 has been pushed to the Fedora 15 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 769124] Update Request: emesene2.11.11 - instant messenger

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #4 from Fedora Update System upda...@fedoraproject.org 2011-12-22 
17:40:29 EST ---
Package emesene-2.11.11-1.fc15:
* should fix your issue,
* was pushed to the Fedora 15 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing emesene-2.11.11-1.fc15'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2011-17297/emesene-2.11.11-1.fc15
then log in and leave karma (feedback).

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

[Bug 754137] Review Request: nagios-plugins-bdii - Nagios Probe for the BDII

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||nagios-plugins-bdii-1.0.14-
   ||1.fc16
 Resolution||ERRATA
Last Closed||2011-12-22 17:45:01

--- Comment #29 from Fedora Update System upda...@fedoraproject.org 
2011-12-22 17:45:01 EST ---
nagios-plugins-bdii-1.0.14-1.fc16 has been pushed to the Fedora 16 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 624182] Review Request: stardict-xmllittre - Authoritative 19th century French dictionary

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-12-22 17:47:50 EST ---
stardict-xmllittre-1.0-3.fc16 has been pushed to the Fedora 16 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 765625] Review Request: pymodbus - A Modbus Protocol Stack in Python

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


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

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

   What|Removed |Added

   Fixed In Version|pymodbus-0.9.0-1.fc16   |pymodbus-0.9.0-1.fc15

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-12-22 17:46:38 EST ---
pymodbus-0.9.0-1.fc15 has been pushed to the Fedora 15 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 765625] Review Request: pymodbus - A Modbus Protocol Stack in Python

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


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

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

   What|Removed |Added

 Status|MODIFIED|CLOSED
   Fixed In Version||pymodbus-0.9.0-1.fc16
 Resolution||ERRATA
Last Closed||2011-12-22 17:45:10

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2011-12-22 17:45:10 EST ---
pymodbus-0.9.0-1.fc16 has been pushed to the Fedora 16 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

  1   2   >