[Bug 768286] New: Review Request: libskk - Library to deal with Japanese kana-to-kanji conversion method

2011-12-16 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: libskk - Library to deal with Japanese kana-to-kanji 
conversion method

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

   Summary: Review Request: libskk - Library to deal with Japanese
kana-to-kanji conversion method
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: du...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, i18n-b...@lists.fedoraproject.org,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://ueno.fedorapeople.org/libskk/libskk.spec
SRPM URL: http://ueno.fedorapeople.org/libskk/libskk-0.0.2-1.fc16.src.rpm
Description:
The libskk project aims to provide GObject-based interface of Japanese
input methods.  Currently it supports SKK (Simple Kana Kanji) with
various typing rules including romaji-to-kana, AZIK, ACT, TUT-Code,
T-Code, and NICOLA.

This package is necessary to build the next release of ibus-skk.

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

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

[Bug 739263] Review Request: sugar-bounce - Fast paced 3D action game

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


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

--- Comment #6 from Peter Robinson pbrobin...@gmail.com 2011-12-16 03:57:53 
EST ---
Yes, the BuildArch line can just be dropped.

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

[Bug 768100] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

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


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

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Pierre-YvesChibon pin...@pingoured.fr 2011-12-16 04:12:29 
EST ---
Thanks for the review, I will open a ticket to upstream for the license.

New Package SCM Request
===
Package Name: trac-mastertickets-plugin
Short Description: Add support for ticket dependencies to Trac
Owners: pingou kevin
Branches: EL6 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 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

--- Comment #3 from Pierre-YvesChibon pin...@pingoured.fr 2011-12-16 04:14:31 
EST ---
After a quick look, if you are not planning an EPEL5 branch, you can remove the
Group and %clean section.

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

[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

--- Comment #4 from adev ade...@gmail.com 2011-12-16 04:25:35 EST ---
Hi,

In fact, the finality is to apply for EPEL5  EPEL6, but I do not know if my
bugzilla ticket is correct for this.

Thank 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 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

--- Comment #5 from Pierre-YvesChibon pin...@pingoured.fr 2011-12-16 04:51:30 
EST ---
It is the right way, thanks for the precision.

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

[Bug 442714] Review Request: satsolver - Satisfiability Solver library which can be used to compute inter-package dependencies.

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


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

--- Comment #55 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 04:54:41 
EST ---
Lorenzo - are you still there?

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

[Bug 493335] Review Request: perl-RTx-Calendar - Calendar for RT due tasks

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


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

--- Comment #14 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 04:59:58 
EST ---
Is this *still* NotReady?

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

[Bug 461484] Review Request: twin - Textmode window environment for Linux

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


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

--- Comment #25 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 04:56:21 
EST ---
Ping Milos, Manuel?

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

[Bug 729199] Review Request: satsolver - Dependency solving library for libzypp

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


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

Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 CC||jussi.leht...@iki.fi
 Resolution|WONTFIX |DUPLICATE

--- Comment #16 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 04:55:20 
EST ---


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

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

[Bug 442714] Review Request: satsolver - Satisfiability Solver library which can be used to compute inter-package dependencies.

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


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

Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 CC||tchollingswo...@gmail.com

--- Comment #56 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 04:55:20 
EST ---
*** Bug 729199 has been marked as a duplicate of this bug. ***

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

[Bug 483863] Review Request: g3dviewer - A 3D file/object viewer

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


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

Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
Last Closed||2011-12-16 04:58:55

--- Comment #11 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 04:58:55 
EST ---
Well, looks like libg3d isn't going anywhere, so there's no sense in keeping
this bug open. Closing as WONTFIX.

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

[Bug 479978] Review Request: newmat - C++ matrix library

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


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

--- Comment #35 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 04:57:11 
EST ---
Ping Pascal?

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

[Bug 482757] Review Request: objcryst-fox - Viewing and solving crystal structures from powder diffraction data

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


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

--- Comment #18 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 04:57:25 
EST ---
Ping Pascal?

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

[Bug 506425] Review Request: lightsquid - a light, small and fast log analyzer for squid proxy

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


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

Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CURRENTRELEASE
Last Closed|2010-07-20 06:23:05 |2011-12-16 05:01:13

--- Comment #52 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 05:01:13 
EST ---
This bug looks closed to me. Closing.

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

[Bug 710917] Review Request: vmpk - Virtual MIDI Piano Keyboard

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


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

Robin Lee robinlee.s...@gmail.com changed:

   What|Removed |Added

   Flag||needinfo?(oget.fedora@gmail
   ||.com)

Bug 710917 depends on bug 744492, which changed state.

Bug 744492 Summary: Review Request: rtmidi - Library for realtime MIDI 
input/output
https://bugzilla.redhat.com/show_bug.cgi?id=744492

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|MODIFIED
 Status|MODIFIED|ON_QA
 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #11 from Robin Lee robinlee.s...@gmail.com 2011-12-16 05:04:28 
EST ---
Do you still work on 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 508922] Review Request: system-config-selinux - GUI Code for system-config-selinux, polgen, and lockdown

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


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

--- Comment #44 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 05:02:29 
EST ---
Ping Christopher, is this package (system-config-selinux) going anywhere..?

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

[Bug 768310] New: Review Request: main package name here - short summary here

2011-12-16 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: main package name here - short summary here

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

   Summary: Review Request: main package name here - short
summary here
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ade...@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: https://firwen.org/redmine/attachments/download/10/gridftp-ifce.spec
SRPM URL:
https://firwen.org/redmine/attachments/download/9/gridftp-ifce-2.99.99-1.src.rpm
Description: GridFTP abstraction layer using the globus toolkit for wlcg ( CERN
). This library provides an easy way to do gridFTP file transfert.

Koji scratch : https://koji.fedoraproject.org/koji/taskinfo?taskID=3587051

rpmlint : gridftp-ifce.src: W: invalid-url Source0: gridftp-ifce-2.99.99.tar.gz
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

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

[Bug 523877] Review Request: CBFlib - crystallography binary format library

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


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

--- Comment #37 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 05:13:04 
EST ---
ping Dmitrij?

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

[Bug 524119] Review Request: nmon - Nigel's performance MONitor for Linux

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


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

--- Comment #24 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 05:13:21 
EST ---
Ping Dylan?

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

[Bug 553055] Review Request: gpxe - Open Source network bootloader

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


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

--- Comment #14 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 05:14:33 
EST ---
Ping again Jeroen.

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

[Bug 550277] Review Request: x2goclient-cli - A command-line client for the x2go system

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


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

--- Comment #14 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 05:14:11 
EST ---
Ping Fabian..?

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

[Bug 515164] Review Request: gnomenu - A consolidated menu for gnome that brings eye candy to the world of the Gnome menu's

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


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

Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX
Last Closed||2011-12-16 05:12:41

--- Comment #31 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 05:12:41 
EST ---
Looks like WONTFIX. Closing.

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

[Bug 768310] Review Request: gridftp-ifce - GridFTP abstraction layer for wlcg.

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


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

adev ade...@gmail.com changed:

   What|Removed |Added

URL||https://svnweb.cern.ch/trac
   ||/lcgutil/wiki
 Blocks||177841(FE-NEEDSPONSOR)
Summary|Review Request: main   |Review Request:
   |package name here - short |gridftp-ifce - GridFTP
   |summary here   |abstraction layer for wlcg.

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

[Bug 482757] Review Request: objcryst-fox - Viewing and solving crystal structures from powder diffraction data

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


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

--- Comment #19 from Pascal Parois pascal...@parois.net 2011-12-16 05:46:17 
EST ---
Still here.

Last update (from august):
http://depot.debroglie.net/fedora/RPMS/15/repoview/objcryst-fox.html
https://svn.parois.net/debroglie/specs/trunk/objcryst-fox.spec

It's outdated, the 1.9.7 is out. I put it on my todo list.

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

[Bug 718502] Review Request: Temperature.app - Window Maker applet which fetches local temperature information

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


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

--- Comment #8 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
05:56:36 EST ---
(In reply to comment #7)
 The download URL is up again! It's now a subdomain of windowmaker.org.
Great news :)

About the man page: you should provide it uncompressed: when building a RPM,
man pages are automatically compressed.

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

[Bug 524119] Review Request: nmon - Nigel's performance MONitor for Linux

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


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

--- Comment #25 from Dylan Swift dylan.sw...@gmail.com 2011-12-16 06:08:32 
EST ---
Pong, 

Hi Jussi

I have just been learning a lot about building rpms (at work) so I plan to get
this up to the latest version soon. 

I did discover someone else packaging this I thought it was dag wieers, but I
can't locate the package now so you might have better luck finding it
elsewhere.

Dylan

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on 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-16 Thread bugzilla
Please do not reply directly 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

 Status|NEW |ASSIGNED
 CC||t...@compton.nu
   Flag||fedora-review?

--- Comment #1 from Tom Hughes t...@compton.nu 2011-12-16 06:23:35 EST ---
Initial review comments:

MUST License field in the package spec file matches the actual license

  gcr/icons/render-icons.py is LGPLv3 / CC-BY-SA 3.0
  gcr/gcr-menu-button.c is GPLv2
  gcr/gcr-collection-model.c is GPLv2
  gcr/gcr-collection-model.h is GPLv2
  gck/pkcs11n.h is MPLv1.1

MUST Rpmlint output is silent

  gcr.x86_64: E: incorrect-fsf-address /usr/share/doc/gcr-3.3.1/COPYING
  gcr.x86_64: W: no-manual-page-for-binary gcr-viewer

  gcr-devel.x86_64: E: incorrect-fsf-address
/usr/include/gcr-3/gcr/gcr-key-widget.h
  [ repeated for many other header files ]

  gcr-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/gcr-3.3.1/gcr/gcr-unlock-options-widget.h
  [ repeated for many other source files ]

SHOULD Patches link to upstream bugs/comments/lists or are otherwise justified

  No explanation for gcr-fix.patch

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

[Bug 508922] Review Request: system-config-selinux - GUI Code for system-config-selinux, polgen, and lockdown

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


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

--- Comment #45 from Daniel Walsh dwa...@redhat.com 2011-12-16 06:34:51 EST 
---
No

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

[Bug 767838] Review Request: libclastfm - Unofficial C-API for the Last.fm web service

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


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

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-16 
06:36:47 EST ---
I will review your 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 767838] Review Request: libclastfm - Unofficial C-API for the Last.fm web service

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


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

Mohamed El Morabity pikachu.2...@gmail.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 767838] Review Request: libclastfm - Unofficial C-API for the Last.fm web service

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


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

--- Comment #2 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
06:51:13 EST ---
Here is the review:

 +:ok, =:needs attention, -:needs fixing

MUST Items:

[+] MUST: rpmlint must be run on every package.
libclastfm.src: W: spelling-error Summary(en_US) fm - FM, Fm, gm
libclastfm.src: W: spelling-error %description -l en_US fm - FM, Fm, gm
libclastfm.src: W: spelling-error %description -l en_US libcurl - lib curl,
lib-curl, curlicue
libclastfm.src: W: spelling-error %description -l en_US distros - bistros,
distress
libclastfm.src: W: spelling-error %description -l en_US scrobble - scribble,
scrabble, Scrabble
libclastfm.src:13: W: macro-in-comment %{name}
libclastfm.src: W: invalid-url Source0: libclastfm-20111214gitcc78dde5.tar.bz2
libclastfm.x86_64: W: spelling-error Summary(en_US) fm - FM, Fm, gm
libclastfm.x86_64: W: spelling-error %description -l en_US fm - FM, Fm, gm
libclastfm.x86_64: W: spelling-error %description -l en_US libcurl - lib curl,
lib-curl, curlicue
libclastfm.x86_64: W: spelling-error %description -l en_US distros - bistros,
distress
libclastfm.x86_64: W: spelling-error %description -l en_US scrobble -
scribble, scrabble, Scrabble
libclastfm-devel.x86_64: W: no-documentation
libclastfm-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/libclastfm/src/md5.c
4 packages and 0 specfiles checked; 1 errors, 13 warnings.
These warnings can be safely ignored.

[+] MUST: The package must be named according to the Package Naming Guidelines.

[+] MUST: The spec file name must match the base package %{name}

[=] MUST: The package must meet the Packaging Guidelines.
See below

[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.

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

[+] 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.

[+] MUST: The spec file must be written in American English.

[+] MUST: The spec file for the package MUST be legible.

[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
MD5 sum=9d6ba4b9f5209b42ab2170d37928a830 for both the embedded source archive
in the SRPM and an archive geerated from upstream's VCS

[+] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.

[+] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch.

[+] MUST: All build dependencies must be listed in BuildRequires

[+] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro.

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

[+] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review

[+] MUST: 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.

[+] MUST: A package must not contain any duplicate files in the %files listing.

[+] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example.

[+] MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines.

[+] MUST: The package must contain code, or permissible content. This is
described in detail in the code vs. content section of Packaging Guidelines.

[+] MUST: Large documentation files should go in a doc subpackage.

[+] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.

[+] MUST: Header files must be in a -devel package.

[+] MUST: Static libraries must be in a -static package.

[+] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in
a -devel package.

[+] MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name} =
%{version}-%{release} 

[+] MUST: Packages must NOT contain any .la libtool archives, these should be
removed in the spec.

[+] MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section.

[+] MUST: Packages must not own files or directories already owned by other
packages.

[+] MUST: All filenames in rpm packages must be valid UTF-8.

Juste two suggestions, 

[Bug 508922] Review Request: system-config-selinux - GUI Code for system-config-selinux, polgen, and lockdown

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


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

--- Comment #46 from Jussi Lehtola jussi.leht...@iki.fi 2011-12-16 07:00:19 
EST ---
So.. can we close this bug?

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

[Bug 479978] Review Request: newmat - C++ matrix library

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


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

--- Comment #36 from Pascal Parois pascal...@parois.net 2011-12-16 06:56:57 
EST ---
Permissions on *.h have been fixed:

http://depot.debroglie.net/fedora/RPMS/15/repoview/newmat.html
https://svn.parois.net/debroglie/specs/trunk/newmat.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 765652] Review Request: bvi - Display-oriented editor for binary files

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


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

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

   What|Removed |Added

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

--- Comment #7 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
07:07:17 EST ---
Don't forget to comment the patches. This will help you foremost to maintain
this package.

(In reply to comment #6)
 bvi.i686: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 2: warning: 
 macro
 `''' not defined
 bvi.i686: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 36: warning:
 macro `--' not defined
 bvi.i686: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 48: warning:
 macro `M' not defined
 bvi.i686: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 48: warning:
 macro `S' not defined
 bvi.i686: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 48: warning:
 macro `N' not defined
 bvi.i686: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 48: warning:
 macro `T' not defined
 bvi.i686: W: manual-page-warning /usr/share/man/man1/bmore.1.gz 56: warning:
 macro `L'' not defined
 bvi.i686: W: manual-page-warning /usr/share/man/man1/bmore.1.gz 56: warning:
 macro `R'' not defined
I don't know if it is really relevant. The impacted man pages seems to be
correctly displayed. I think you could ignore these warnings finally.
Anyway, don't forget to fix the bmore.1 file: the path of the bmore.help file
specified in this man page is bad (/usr/local/share/bmore.help).

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

[Bug 752311] Review Request: gitso - A front-end to reverse VNC connections

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


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

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
   Flag||fedora-review?

--- Comment #1 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
07:39:07 EST ---
Some comments:

* You don't need to define the %python_sitelib macro, it is already defined by
default (see the /usr/lib/rpm/macros file):
   http://fedoraproject.org/wiki/Packaging:Python#Macros
* I don't really think python-devel is needed to build your package.
* Since python and wxGTK are already required by wxPython, you can remove these
two packages from the Requires.
* Do you really think gitso modules need to be installed in
%{python_sitelib}/%{name} ? These files are not inteded to be visible from
Python. You should install them in %{_datadir}/%{name}, as done by upstream.
* Why did you remove the MimeType key in the .desktop file?

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

[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

--- Comment #6 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
07:45:41 EST ---
*** Bug 768171 has been marked as a duplicate of this bug. ***

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

[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

--- Comment #7 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
07:45:56 EST ---
*** Bug 768173 has been marked as a duplicate of this bug. ***

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

[Bug 768171] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

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

   What|Removed |Added

 Status|NEW |CLOSED
 CC||pikachu.2...@gmail.com
 Resolution||DUPLICATE
Last Closed||2011-12-16 07:45:41

--- Comment #1 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
07:45:41 EST ---


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

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

[Bug 768173] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

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

   What|Removed |Added

 Status|NEW |CLOSED
 CC||pikachu.2...@gmail.com
 Resolution||DUPLICATE
Last Closed||2011-12-16 07:45:55

--- Comment #1 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
07:45:55 EST ---


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

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

[Bug 751792] Review Request: felix-gogo-runtime - Community OSGi R4 Service Platform Implementation - Basic Commands

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


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

Tomas Radej tra...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Tomas Radej tra...@redhat.com 2011-12-16 07:52:11 EST ---
Thanks.

New Package SCM Request
===
Package Name: felix-gogo-runtime
Short Description: Community OSGi R4 Service Platform Implementation - Basic
Commands
Owners: tradej
InitialCC: java-sig

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

[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

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

   What|Removed |Added

 CC||pikachu.2...@gmail.com

--- Comment #8 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
07:58:19 EST ---
The %{_isa} suffix on BuildRequires packages is useless, please remove it on
each BuildRequires.

globus-gssapi-gsi-devel is already required by globus-gss-assist-devel. As
well, gsoap-devel is required by CGSI-gSOAP-devel. So you can remove
globus-gssapi-gsi-devel and  gsoap-devel from the BuildRequires.

On your devel subpackage:
   Requires: srm-ifce = %{version}-%{release}
No. You can't require a superior version of srm-ifce for your devel package,
especially in case of API breaks in the next versions.
See http://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package
to fix it.

%{_datadir}/doc/srm-access-library-for-lcg_util/ doesn't seem to me a canonic
place to install documentation files. If you don't want/can't fix the code to
put them elsewhere, you should remove them in %install and use the %doc macro
to install them in %{_docdir}/%{name}-%{version}/ automatically.

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

[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

--- Comment #9 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
08:01:10 EST ---
I forgot this point: we have guidelines for buiding packages with cmake, please
consider theme
   http://fedoraproject.org/wiki/Packaging:Cmake

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

[Bug 768100] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

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


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

--- Comment #5 from Jon Ciesla limburg...@gmail.com 2011-12-16 08:09: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 751792] Review Request: felix-gogo-runtime - Community OSGi R4 Service Platform Implementation - Basic Commands

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


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

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

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

[Bug 710917] Review Request: vmpk - Virtual MIDI Piano Keyboard

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


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

Orcan Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

   Flag|needinfo?(oget.fedora@gmail |
   |.com)   |

--- Comment #12 from Orcan Ogetbil oget.fed...@gmail.com 2011-12-16 08:22:27 
EST ---
Sorry, I forgot about it. I'll get to this this weekend.

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

[Bug 768310] Review Request: gridftp-ifce - GridFTP abstraction layer for wlcg.

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


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

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

   What|Removed |Added

 CC||pikachu.2...@gmail.com

--- Comment #1 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
08:31:32 EST ---
What is the real version of your sources? I can't find in the SVN repo a tag
corresponding to the 2.99.99. Nothing in the commit logs too.
I suppose this package is a version 3 prerelease of gridftp-ice. So please
consider the guidelines concerning prerelease versionning:
  
http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Non-Numeric_Version_in_Release

automake is already required by libtool, and globus-ftp-client-devel by
globus-gass-copy-devel. So you can remove automake and globus-ftp-client-devel
from the BuildRequires.

About the %{_isa} macro and the documentation installation, same remarks than
in https://bugzilla.redhat.com/show_bug.cgi?id=768174

You'd better use the %{configure} macro in %build instead of calling
explicitely the script ./configure. The macro already sets the «
--prefix=%{_prefix} » and « --libdir=%{_libdir} » options. By the way, you can
also remove the « --enable-debug » and « --enable-wall » options, since the
macro already sets the CFLAGS to the default ones expected by the Fedora
Project: these flags already contains the -g and -Wall options.
The --disable-static option doesn't seem to work since a static library is
build anyway. Unless you can fix it, you should remove this option to
configure.

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

[Bug 738079] Review Request: quartz - Enterprise Job Scheduler for Java

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


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

--- Comment #16 from Miroslav Suchý msu...@redhat.com 2011-12-16 08:36:48 EST 
---
==

Key:
 - = N/A
 x = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [x] Package meets the Packaging Guidelines  including the Java specific items
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 tested in: f16/koji
 http://koji.fedoraproject.org/koji/taskinfo?taskID=3587138
 [x] Rpmlint output:
quartz.src: W: invalid-url Source0: quartz-2.1.2.tar.xz
3 packages and 0 specfiles checked; 0 errors, 1 warnings.
Waiving per comment #13
 [x] Package is not relocatable.
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: ASL 2.0
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
I get tar.gz according the comment in spec and compared it to tar.gz inside of
src.rpm and did
diff -Naur src.rpm upstream
and there was no difference
 [x] Package is not known to require ExcludeArch
 [x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -fR $RPM_BUILD_ROOT.
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [x] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [-] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Latest version is packaged.
 2.1.2
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
 Tested on: koji scratch build
 [x] Package should compile and build into binary rpms on all supported
architectures.
 Tested on:koji scratch build
 [?] Package functions as described.
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files is correct.
 [-] File based requires are sane.
 [-] %check is present and the tests pass



*** 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 738079] Review Request: quartz - Enterprise Job Scheduler for Java

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


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

Miroslav Suchý msu...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   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 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

--- Comment #10 from adev ade...@gmail.com 2011-12-16 08:36:40 EST ---
Ok, thank you for this fast response.

I updated the spec files and the SRPMS considering your comments :

Spec URL: https://firwen.org/redmine/attachments/download/11/srm-ifce.spec
SRPM URL:
https://firwen.org/redmine/attachments/download/12/srm-ifce-1.12-2.src.rpm

rpmlint : RPMS check ...
srm-ifce.src: W: invalid-url Source0: srm-ifce-1.12.tar.gz
4 packages and 0 specfiles checked; 0 errors, 1 warnings.
end RPMS check

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

[Bug 767838] Review Request: libclastfm - Unofficial C-API for the Last.fm web service

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


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

--- Comment #3 from Christoph Wickert cwick...@fedoraproject.org 2011-12-16 
08:47:44 EST ---
Thanks for the review.

(In reply to comment #2)

 Juste two suggestions, before approving this package:
 - since autoconf is already required by libtool, you could remove autoconf 
 from
 the BuildRequires

Ok, do you want me to change this now or can I do this after you approved the
package?

 - unless you intend to maintain libclastfm for EPEL 5, you should remove the
 %clean section in you .spec file, as well as all the BuildRoot cleans and the
 BuildRoot tag:
http://fedoraproject.org/wiki/Packaging:Guidelines#.25clean
http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag

The guidelines don't say I should remove it bug I could. I like keeping these
things around for backwards compatibility.

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

[Bug 768183] Review Request: is-interface - library for the information system in wlcg

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


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

--- Comment #1 from adev ade...@gmail.com 2011-12-16 09:03:48 EST ---
Update of the files considering comments from  768174 :

Spec URL :  https://firwen.org/redmine/data/docs/specs/is-interface.spec
SRPM URL:
https://firwen.org/redmine/data/docs/specs/is-interface-1.12.1-2_epel.src.rpm

rpmlint :
is-interface.src: W: invalid-url Source0: is-interface-1.12.1.src.tar.gz
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

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

[Bug 766916] Review Request: grinder - tool for synchronizing yum repositories

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


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

--- Comment #3 from Jeff Ortel jor...@redhat.com 2011-12-16 09:05:11 EST ---
I think you still have a problem with the grinder Source0: URL.  The URL is
looking for grinder-0.136.tar.gz and
https://fedorahosted.org/releases/g/r/grinder/ contains only:
grinder-0.0.136.tar.gz which does not match because of the leading 0 (epoch).

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

[Bug 766916] Review Request: grinder - tool for synchronizing yum repositories

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


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

--- Comment #4 from Jeff Ortel jor...@redhat.com 2011-12-16 09:43:10 EST ---
Created attachment 547827
  -- https://bugzilla.redhat.com/attachment.cgi?id=547827
Review Checklist

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

[Bug 235189] Review Request: nautilus-python - Python bindings for Nautilus

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


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

Caghan Demirci cagh...@gmail.com changed:

   What|Removed |Added

 CC||cagh...@gmail.com
   Flag||needinfo?

--- Comment #7 from Caghan Demirci cagh...@gmail.com 2011-12-16 09:46:07 EST 
---
Package Change Request
==
Package Name: nautilus-python
New Branches: el5 el6
Owners: dignan
InitialCC: jkeating

dignan is already listed as the owner of the EPEL package under
https://admin.fedoraproject.org/pkgdb/acls/name/nautilus-python.
However, the package can still not be found in the EPEL repository.  I imagine
that a branch request is necessary.
Note that versions higher than 0.7.0-3 will not work on RHEL because they
require Nautilus 3.

I am not able to set the fedora-cvs flag to ?, as recommended in the
instructions.  It is locked at +.  So I set the needinfo flag to ?.

Thanks!

-- 
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 542715] Review Request: rabbitvcs - Easy version control

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


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

Caghan Demirci cagh...@gmail.com changed:

   What|Removed |Added

 CC||cagh...@gmail.com
   Flag||needinfo?

Bug 542715 depends on bug 474428, which changed state.

Bug 474428 Summary: not loading 64 bit libpython on 64 bit platform
https://bugzilla.redhat.com/show_bug.cgi?id=474428

   What|Old Value   |New Value

 Resolution||WONTFIX
 Status|ON_QA   |CLOSED

--- Comment #45 from Caghan Demirci cagh...@gmail.com 2011-12-16 09:53:45 EST 
---

  I was wondering if the package owner (nushio = Juan) could maintain this
package in EPEL as well.  Only EPEL 6 would be necessary, since rabbitvcs
requires python2.6, which RHEL 5 does not have.

  I am setting the needinfo flag to ? in hopes that someone will see this bug.
:)

  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 766916] Review Request: grinder - tool for synchronizing yum repositories

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


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

Jeff Ortel jor...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Jeff Ortel jor...@redhat.com 2011-12-16 10:25:04 EST ---
Just noticed the version is 0.0.136 so your Source0 is fine.  All issues
addressed.

Package Approved.

Updated checklist included here (as well) per review guidelines.

== MUST ==
[x]: rpmlint must be run on the source rpm and all binary rpms the build
produces. 
 ** Output from rpmlint

rpmlint grinder-0.0.132-1.fc14.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

rpmlint grinder-0.0.132-1.fc14.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.


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

GPLv2

[x]: The License field in the package spec file must match the actual license. 
[x]: If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.

LICENSE

[x]: The spec file must be written in American English. 
[x]: The spec file for the package MUST be legible.

[x]: 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.

[x]: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. 
[x]: 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. 

[x]: 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.

[x]: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[x]: 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. 
[x]: Packages must NOT bundle copies of system libraries.
[x]: 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. 
[x]: 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. 
[x]: 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)
[x]: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. 
[x]: Each package must consistently use macros. 
[x]: The package must contain code, or permissable content. 
[x]: Large documentation files must go in a -doc subpackage. (The definition of
large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity). 
[x]: If a package includes something as %doc, it must not affect the runtime of
the application. To summarize: If it is in %doc, the program must run properly
if it is not present. 
[x]: Header files must be in a -devel package. 
[x]: Static libraries must be in a -static package. 
[x]: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel
package. 
[x]: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name}%{?_isa} =
%{version}-%{release} 
[x]: Packages must NOT contain any .la libtool archives, these must be removed
in the spec if they are built.
[x]: Packages containing GUI applications must include a %{name}.desktop file,
and that file must be properly installed with desktop-file-install in the

[Bug 698405] Review Request: mcollective - A framework to build server orchestration or parallel job execution systems

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


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

--- Comment #25 from Fedora Update System upda...@fedoraproject.org 
2011-12-16 11:04:43 EST ---
mcollective-1.3.1-6.fc16,rubygem-systemu-2.4.1-2.fc16 has been submitted as an
update for Fedora 16.
https://admin.fedoraproject.org/updates/mcollective-1.3.1-6.fc16,rubygem-systemu-2.4.1-2.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 751114] Review Request: sgabios - bios option rom for serial port display

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


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

Justin M. Forbes jfor...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Justin M. Forbes jfor...@redhat.com 2011-12-16 11:04:31 
EST ---
New Package SCM Request
===
Package Name: sgabuis
Short Description: Open-source serial graphics BIOS option rom
Owners: jforbes
Branches: f16 
InitialCC: fedora-virt-maint

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

[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation

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


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

--- Comment #11 from adev ade...@gmail.com 2011-12-16 11:10:23 EST ---
I did a new update of the spec file with the usage of %{_docdir} macro :

Spec URL : https://firwen.org/redmine/data/docs/specs/srm-ifce.spec
SRPM URL : https://firwen.org/redmine/data/docs/specs/srm-ifce-1.12-2.src.rpm

rpmlint : srm-ifce.src: W: invalid-url Source0: srm-ifce-1.12.tar.gz
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

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

[Bug 768310] Review Request: gridftp-ifce - GridFTP abstraction layer for wlcg.

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


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

--- Comment #2 from adev ade...@gmail.com 2011-12-16 11:25:57 EST ---
I have updated the spec/SRPMS with following your recommendations :

- version number is now 2.1.3, the old one was a mistake
- the useless dependencies are drop, and the isa macro on buildreq too.
- %configure is now used
- the documentation is now in %_docdir
- disable-static is remove, the operation is done manually.
- internal dependencies are corrected.

Spec URL : https://firwen.org/redmine/data/docs/specs/gridftp-ifce.spec
SRPM URL:
https://firwen.org/redmine/data/docs/specs/gridftp-ifce-2.1.3-1.src.rpm

rpmlint : gridftp-ifce.src: W: invalid-url Source0: gridftp-ifce-2.1.3.tar.gz
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

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

[Bug 529441] Review Request: pdfbox - Java PDF library

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


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

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

   What|Removed |Added

   Flag|fedora-review?, |fedora-review+
   |needinfo?(akurtako@redhat.c |
   |om) |

--- Comment #24 from Alexander Kurtakov akurt...@redhat.com 2011-12-16 
11:31:32 EST ---

Package Review
==

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



 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[!]: 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.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[x]: MUST If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package 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.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.

rpmlint pdfbox-javadoc-1.6.0-2.fc17.noarch.rpm

pdfbox-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs - Java docs,
Java-docs, Avocados
1 packages and 0 specfiles checked; 0 errors, 1 warnings.


rpmlint jempbox-1.6.0-2.fc17.noarch.rpm

jempbox.noarch: W: name-repeated-in-summary C JempBox
jempbox.noarch: W: spelling-error %description -l en_US subproject - sub
project, sub-project, projector
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


rpmlint fontbox-1.6.0-2.fc17.noarch.rpm

fontbox.noarch: W: name-repeated-in-summary C FontBox
fontbox.noarch: W: spelling-error %description -l en_US subproject - sub
project, sub-project, projector
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


rpmlint pdfbox-ant-1.6.0-2.fc17.noarch.rpm

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


rpmlint pdfbox-1.6.0-2.fc17.noarch.rpm

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


rpmlint pdfbox-app-1.6.0-2.fc17.noarch.rpm

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


rpmlint pdfbox-1.6.0-2.fc17.src.rpm

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


rpmlint pdfbox-lucene-1.6.0-2.fc17.noarch.rpm

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


[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
/home/akurtakov/tmp/529441/pdfbox-1.6.0-src.zip :
  MD5SUM this package : 103c620cd58d69f107c9a63f16ba7da7
  MD5SUM upstream package : 103c620cd58d69f107c9a63f16ba7da7

[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.

[Bug 738079] Review Request: quartz - Enterprise Job Scheduler for Java

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


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

Andy Grimm agr...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #17 from Andy Grimm agr...@gmail.com 2011-12-16 11:40:01 EST ---
New Package SCM Request
===
Package Name:  quartz
Short Description: Enterprise Job Scheduler for Java
Owners:arg

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

[Bug 529441] Review Request: pdfbox - Java PDF library

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #25 from Orion Poplawski or...@cora.nwra.com 2011-12-16 11:40:54 
EST ---
New Package SCM Request
===
Package Name: pdfbox
Short Description: Java PDF library
Owners: orion
Branches: 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 738079] Review Request: quartz - Enterprise Job Scheduler for Java

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


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

--- Comment #18 from Jon Ciesla limburg...@gmail.com 2011-12-16 12:04:56 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 529441] Review Request: pdfbox - Java PDF library

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


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

--- Comment #26 from Jon Ciesla limburg...@gmail.com 2011-12-16 12:04:31 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 751114] Review Request: sgabios - bios option rom for serial port display

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


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

--- Comment #8 from Jon Ciesla limburg...@gmail.com 2011-12-16 12:06:19 EST 
---
Pacakge name does not match summary, and fedora-virt-maint isn't a valid FAS
account.

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

[Bug 742166] Review Request: wmctrl - X Window Manager command-line tool

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


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

--- Comment #15 from Fedora Update System upda...@fedoraproject.org 
2011-12-16 12:41:40 EST ---
wmctrl-1.07-9.el6 has been pushed to the Fedora EPEL 6 stable repository.

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

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

2011-12-16 Thread bugzilla
Please do not reply directly 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 #2 from Matthias Clasen mcla...@redhat.com 2011-12-16 12:40:51 
EST ---
I've filed an upstream bug about the minor license inconsistencies:
https://bugzilla.gnome.org/show_bug.cgi?id=666378

Here is an updated srpm for gcr-3.3.2.1. Also added a comment about the patch.

http://mclasen.fedorapeople.org/gcr.spec
http://mclasen.fedorapeople.org/gcr-3.3.2.1-1.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 710904] Review Request: octave-communications - Communications for Octave

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


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

--- Comment #6 from Thomas Sailer t.sai...@alumni.ethz.ch 2011-12-16 12:40:17 
EST ---
Spec URL: http://sailer.fedorapeople.org/octave-communications.spec
SRPM URL:
http://sailer.fedorapeople.org/octave-communications-1.1.0-1.fc16.src.rpm

Pong...

I've updated the package to the current version.

I can't use %{octave_pkg_build}, because dependency checking does not work for
me.

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

[Bug 739798] Review Request: pcp-gui - Visualization tools for the Performance Co-Pilot toolkit

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2011-12-16 12:41:17 EST ---
pcp-gui-1.5.2-2.el5 has been pushed to the Fedora EPEL 5 testing repository.

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

[Bug 755602] Review Request: json_diff - Generates diff between two JSON files

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


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

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

   What|Removed |Added

   Fixed In Version|json_diff-1.1.0-1.el6   |json_diff-1.1.0-1.el5

--- Comment #15 from Fedora Update System upda...@fedoraproject.org 
2011-12-16 12:42:27 EST ---
json_diff-1.1.0-1.el5 has been pushed to the Fedora EPEL 5 stable repository. 
If problems still persist, please make note of it in this bug report.

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

[Bug 757290] Review Request: unicode-ucd - Unicode Character Database

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


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

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

   What|Removed |Added

   Fixed In Version|unicode-ucd-6.0.0-2.fc16|unicode-ucd-6.0.0-2.el6

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-12-16 12:41:25 EST ---
unicode-ucd-6.0.0-2.el6 has been pushed to the Fedora EPEL 6 stable repository.
 If problems still persist, please make note of it in this bug report.

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

[Bug 755602] Review Request: json_diff - Generates diff between two JSON files

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


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

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

   What|Removed |Added

   Fixed In Version|json_diff-0.9.2-2.fc16  |json_diff-1.1.0-1.el6

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-12-16 12:42:19 EST ---
json_diff-1.1.0-1.el6 has been pushed to the Fedora EPEL 6 stable repository. 
If problems still persist, please make note of it in this bug report.

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

[Bug 766882] Review Request: htmlunit - A headless web browser for automated testing

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


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

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

   What|Removed |Added

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

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

[Bug 767838] Review Request: libclastfm - Unofficial C-API for the Last.fm web service

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


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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Mohamed El Morabity pikachu.2...@gmail.com 2011-12-16 
14:09:21 EST ---
 Ok, do you want me to change this now or can I do this after you approved the
 package?
I trust you to fix it at the package import :)

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 442714] Review Request: satsolver - Satisfiability Solver library which can be used to compute inter-package dependencies.

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


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

--- Comment #57 from Lorenzo Villani lore...@villani.me 2011-12-16 14:21:53 
EST ---
Jussi, I am no longer involved in Fedora for the time being. I relinquished
ownership of all my packages about one year ago. I thought I closed or
reassigned all of my bugs but, apparently, I was wrong.

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

[Bug 766882] Review Request: htmlunit - A headless web browser for automated testing

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


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

Alexander Kurtakov akurt...@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 747092] Review Request: lzma-sdk457 - SDK for lzma compression

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


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

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-16 14:48:16 EST ---
lzma-sdk457-4.57-1.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 728815] Review Request: trademgen - C++ Simulated Travel Demand Generation Library

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-12-16 14:48:59 EST ---
trademgen-0.2.2-1.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 759045] Review Request: ghc-base16-bytestring - ByteString hex encoding and decoding

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-12-16 
14:49:11 EST ---
ghc-base16-bytestring-0.1.1.3-1.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 755176] Review Request: ghc-mwc-random - Pseudo random number generation library for Haskell

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


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

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

   What|Removed |Added

   Fixed In Version|ghc-mwc-random-0.10.0.1-1.f |ghc-mwc-random-0.10.0.1-1.f
   |c15 |c16

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-12-16 
14:52:06 EST ---
ghc-mwc-random-0.10.0.1-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 755176] Review Request: ghc-mwc-random - Pseudo random number generation library for Haskell

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||ghc-mwc-random-0.10.0.1-1.f
   ||c15
 Resolution||ERRATA
Last Closed||2011-12-16 14:51:06

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-12-16 
14:51:06 EST ---
ghc-mwc-random-0.10.0.1-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 766882] Review Request: htmlunit - A headless web browser for automated testing

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


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

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

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2011-12-16 
14:52:54 EST ---

Package Review
==

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



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

rpmlint htmlunit-2.9-1.fc17.noarch.rpm

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


rpmlint htmlunit-2.9-1.fc17.src.rpm

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


rpmlint htmlunit-javadoc-2.9-1.fc17.noarch.rpm

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


[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
/home/akurtakov/tmp/766882/htmlunit-2.9-src.zip :
  MD5SUM this package : 52604527e9aaab2183dfd0be43400faa
  MD5SUM upstream package : 52604527e9aaab2183dfd0be43400faa

[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
 separate file from upstream, the packager SHOULD query upstream to
 include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD Patches link to upstream bugs/comments/lists or are otherwise
 justified.
[x]: SHOULD SourceX / PatchY prefixed with %{name}.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: SHOULD Package should compile and build into binary rpms on all supported
 architectures.
[-]: SHOULD %check is present and all 

[Bug 767047] Review Request: ViTables - Viewer for Hierarical Datafiles (HDF5)

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-12-16 
14:59:43 EST ---
ViTables-2.1-2.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 758958] Review Request: ghc-simple-sendfile - Cross-platform sendfile system call

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-12-16 
14:59:00 EST ---
ghc-simple-sendfile-0.1.3-1.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 556128] Review Request: ff-utils - Utilities to test force feedback of input device

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


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

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

   What|Removed |Added

   Fixed In Version|ff-utils-2.4.21-4.fc16  |ff-utils-2.4.21-4.fc15

--- Comment #24 from Fedora Update System upda...@fedoraproject.org 
2011-12-16 14:58:46 EST ---
ff-utils-2.4.21-4.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 756386] Review Request: ghc-unordered-containers - Efficient hashing-based container types

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-12-16 
15:01:54 EST ---
ghc-unordered-containers-0.1.4.3-1.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 768500] New: Review Request: subscription-manager - Tools and libraries for subscription and repository management.

2011-12-16 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: subscription-manager - Tools and libraries for 
subscription and repository management.

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

   Summary: Review Request: subscription-manager - Tools and
libraries for subscription and repository management.
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dgood...@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://fedorapeople.org/~dgoodwin/subscription-manager/subscription-manager.spec
SRPM URL:
http://fedorapeople.org/~dgoodwin/subscription-manager/subscription-manager-0.99.2-1.fc16.src.rpm
Description: Subscription manager offers tools for registration, consumption of
subscriptions, and ultimately access to content in protected yum repositories.
It is a core component for systems to register and interact with the Katello
systems management platform. (http://katello.org/)

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

[Bug 759712] Review Request: dragonegg - GCC plugin to use LLVM optimizers and code generators

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


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

--- Comment #5 from Jerry James loganje...@gmail.com 2011-12-16 16:02:36 EST 
---
(In reply to comment #4)
 I respectfully disagree with changing the macro-in-comment warnings as they 
 are
 in comments copied directly from the existing gcc-python-plugin spec, and
 doubled percent signs might be confusing to anyone trying to actually
 understand the comments.  I'd rather have the rpmlint warnings.  However, if
 you really feel strongly that those warnings have to be fixed, let me know, 
 and
 I'll do it.

What I feel strongly about is that the rpm developers should fix their code so
that rpm doesn't expand macros in comments. :-)  As long as those macros don't
cause any actual problems (and I did not notice any), then I am fine with
leaving the comments as they are.

 LLVM is supported on i386 and x86_64, which are the current Fedora Primary
 Architectures.  The Fedora llvm package spec excludes use of ocaml on s390,
 s390x, and sparc64, but does not exclude those or any other architectures. 
 While I have a hard time believing that LLVM will work on all of the Fedora
 Secondary Architectures, if the LLVM spec doesn't exclude them I don't think
 the DragonEgg spec needs to either.

That's fine.  The DragonEgg spec should definitely follow the llvm spec in this
regard.  I just wasn't sure if the llvm spec had any ExcludeArch tags, and was
too lazy to go look.

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

[Bug 766882] Review Request: htmlunit - A headless web browser for automated testing

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


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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Alexander Kurtakov akurt...@redhat.com 2011-12-16 
16:06:23 EST ---
Sorry the empty doc confused me. Please drop it, not a blocker.

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 766943] Review Request: jsilver - A pure-Java implementation of Clearsilver

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


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

Andy Grimm agr...@gmail.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 767050] Review Request: hibernate-validator - Bean Validation (JSR 303) Reference Implementation

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


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

Andy Grimm agr...@gmail.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 766882] Review Request: htmlunit - A headless web browser for automated testing

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


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

Andy Grimm agr...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #3 from Andy Grimm agr...@gmail.com 2011-12-16 16:18:07 EST ---
New Package SCM Request
===
Package Name:  htmlunit
Short Description: A headless web browser for automated testing
Owners:arg

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

[Bug 766882] Review Request: htmlunit - A headless web browser for automated testing

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


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

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

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|akurt...@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 747659] Review Request: proxool - Java connection pool library

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


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

Andy Grimm agr...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-12-16 16:18:24

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

[Bug 738079] Review Request: quartz - Enterprise Job Scheduler for Java

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


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

Andy Grimm agr...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-12-16 16:19:05

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

[Bug 738079] Review Request: quartz - Enterprise Job Scheduler for Java

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


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

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

   What|Removed |Added

 CC||akurt...@redhat.com
 Blocks|652183(FE-JAVASIG)  |

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

[Bug 747659] Review Request: proxool - Java connection pool library

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


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

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

   What|Removed |Added

 CC||akurt...@redhat.com
 Blocks|652183(FE-JAVASIG)  |

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

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

2011-12-16 Thread bugzilla
Please do not reply directly 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

Ed Marshall esm+red...@logic.net changed:

   What|Removed |Added

   Flag|needinfo?(esm+redhat@logic. |
   |net)|

--- Comment #14 from Ed Marshall esm+red...@logic.net 2011-12-16 16:45:34 EST 
---
Hi Chitlesh,

Apologies, I was in transit recently (Martin was exactly right), and am just
getting back to this stuff. I'm working on a Fritzing build today (for today's
release, 0.6.4b), I'll try and get updated versions uploaded shortly.

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

  1   2   >