[Bug 563673] Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

--- Comment #10 from Mathieu Bridon boche...@fedoraproject.org 2010-02-13 
03:16:38 EST ---
Package imported into CVS and built for Rawhide.

Thanks Jens for the re-review and Kevin for the CVS creation.

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


[Bug 564557] Review Request: gfs-goschen-fonts - A 19th century Greek typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Nicolas Mailhot nicolas.mail...@laposte.net changed:

   What|Removed |Added

 CC||fonts-b...@lists.fedoraproj
   ||ect.org

--- Comment #1 from Nicolas Mailhot nicolas.mail...@laposte.net 2010-02-13 
05:30:07 EST ---
SRPM URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=1981773
Description:

Georg Joachim Göschen founded in 1782 the publishing house of G.J.
Göschensche Verlagsbuchhandlung in Leipzig and was one of the most active
publishers of the period in Germany. Göschen was very interested in
typography, influenced by the fame and quality of the editions of G. Bodoni
and F. Didot.

In 1797, he collaborated with the leading scholar of the period, Johann Jakob
Griesbach, to edit and publish the New Testament in Greek for which he formed
a committee of scholars to decide the new Greek type which were eventually
cut by Johann Prillwitz. The book appeared in 1803 and the types show many
influences from the Greek types of Bodoni. Their characteristic was the
neoclassical form of marked contrast between thick and thin strokes, the
cursive style and the large size of the font.

The design was too cumbersome to allow general use and can be considered
successful only for its indirect influence on the later cut Greek Leipzig
type. It is, however, part of the greater heritage of Greek type design and
therefore the type has been digitized by George D. Matthiopoulos in 2009 and
is part of GFS' type library under the name GFS Göschen cursive, in
commemoration of the great German publisher.

http://fedoraproject.org/wiki/GFS_G%C3%B6schen_fonts

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

[Bug 564557] Review Request: gfs-goschen-fonts - A 19th century Greek typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Nicolas Mailhot nicolas.mail...@laposte.net 2010-02-13 
05:34:38 EST ---
Created an attachment (id=390627)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=390627)
repo-font-audit output for the package

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


[Bug 564557] Review Request: gfs-goschen-fonts - A 19th century Greek typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Nicolas Mailhot nicolas.mail...@laposte.net 2010-02-13 
05:45:58 EST ---
Since koji garbage-collects after a while:

http://nim.fedorapeople.org/gfs-goschen-fonts-20100203-1.fc13.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 556308] Review Request: paratype-pt-sans-fonts - A pan-Cyrillic typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Nicolas Mailhot nicolas.mail...@laposte.net 2010-02-13 
05:50:37 EST ---
Since koji garbage-collected

http://nim.fedorapeople.org/paratype-pt-sans-fonts-20100112-1.fc13.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 564557] Review Request: gfs-goschen-fonts - A 19th century Greek typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #5 from Mathieu Bridon boche...@fedoraproject.org 2010-02-13 
06:21:08 EST ---
Oops... Forgot the review-flag

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


[Bug 564557] Review Request: gfs-goschen-fonts - A 19th century Greek typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 CC||boche...@fedoraproject.org

--- Comment #4 from Mathieu Bridon boche...@fedoraproject.org 2010-02-13 
06:19:31 EST ---
+:ok, =:needs attention, -:needs fixing

MUST Items:
[+] MUST: rpmlint must be run on every package.
$ rpmlint
/var/lib/mock/fedora-rawhide-x86_64/result/gfs-goschen-fonts-20100203-1.fc13.*
2 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint SPECS/gfs-goschen-fonts.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

[+] 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 be licensed with a Fedora approved license and meet
the Licensing Guidelines.
  = OFL

[+] 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.
$ sha1sum SOURCES/GFS_GOSCHEN.zip 
4a9f048bd0016d8b0df49c63848c14686f48fc6a  SOURCES/GFS_GOSCHEN.zip

[+] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.
[+] MUST: All build dependencies must be listed in BuildRequires
[+] 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.
  = not applicable

[+] 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. Every %files section must include a
%defattr(...) line.
[+] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot}
[+] 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: If a package includes something as %doc, it must not affect the
runtime of the application.
[+] 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.
  = not applicable

[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: At the beginning of %install, each package MUST run rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[+] MUST: All filenames in rpm packages must be valid UTF-8.

SHOULD Items:
[+] SHOULD: The reviewer should test that the package builds in mock.
[+] SHOULD: The package should compile and build into binary rpms on all
supported architectures.
  = package was built in Koji and is noarch anyway



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 560457] Review Request: pyutil - A collection of mature utilities for Python programmers

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-02-13 
06:27:37 EST ---
I think either will do, however I would rename the files rather than
deleting the files.

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


[Bug 564567] New: Review Request: gwaei - A Japanese dictionary for Gnome

2010-02-13 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: gwaei - A Japanese dictionary for Gnome

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

   Summary: Review Request: gwaei - A Japanese dictionary for
Gnome
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rene.rib...@free.fr
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://uggla.free.fr/rpmbuild/SPECS/gwaei.spec
SRPM URL: http://uggla.free.fr/rpmbuild/SRPMS/gwaei-1.2.1-4.fc12.src.rpm
Description: This is my first package, and I'm looking for a sponsor.
gWaei is an easy to use and yet powerful full-featured dictionary program
for Japanese to English translation.
It organizes results by relevance, supports regex searches, tabs,
spell checking, kanji handwriting recognition and an accompanying console
version for searches through the terminal.

The dictionary files it uses are from Jim Breen's WWWJDIC project and
are installed separately through the program.

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


[Bug 564567] Review Request: gwaei - A Japanese dictionary for Gnome

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

René Ribaud rene.rib...@free.fr changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

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

[Bug 560457] Review Request: pyutil - A collection of mature utilities for Python programmers

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Ruben Kerkhof ru...@rubenkerkhof.com 2010-02-13 07:42:13 
EST ---
Thanks, I've renamed them

Spec URL: http://ruben.fedorapeople.org/pyutil.spec
SRPM URL: http://ruben.fedorapeople.org/pyutil-1.6.1-3.fc13.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 564537] Review Request: grc - simple python logfile colouriser

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

 CC||terje...@phys.ntnu.no

--- Comment #1 from Terje Røsten terje...@phys.ntnu.no 2010-02-13 07:52:24 
EST ---
Some comments:


Release:0.1%{?dist}.svac
Why so uncommon release tag, what't wrong with 1%{?dist}?

%build

- Add a comment about why %build in empty.

%install
rm -rf $RPM_BUILD_ROOT
%{_builddir}/grc-1.3/install.sh %{_builddir}/grc-1.3/ $RPM_BUILD_ROOT

- %install will cd into source dir, you should use:
  ./install.sh  $RPM_BUILD_ROOT
 I don't understand why you need the build patch, (which is a install 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 563336] Review Request: libisds - Library for accessing the Czech Data Boxes

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2010-02-13 
08:11:26 EST ---
libisds-0.1-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/libisds-0.1-2.fc11

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


[Bug 563336] Review Request: libisds - Library for accessing the Czech Data Boxes

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Matej Cepl mc...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

--- Comment #7 from Matej Cepl mc...@redhat.com 2010-02-13 08:13:15 EST ---
Build in Rawhide
http://koji.fedoraproject.org/koji/buildinfo?buildID=155918

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


[Bug 551911] Review Request: monodevelop-boo - A boo plugin for monodevelop

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Christian Krause c...@plauener.de 2010-02-13 08:09:14 EST 
---
Yes, it looks like that your fix partially fixed it, but probably e.g. md
itself looks for the other location...

I have used something like this in a different package to mark the language
files properly even if there are at an unusual place:

%install
make DESTDIR=... install

find %{buildroot} -type f -o -type l|sed '
s:'%{buildroot}'::
s:\(.*/lib/python2.6/site-packages/ankiqt/locale/\)\([^/_]\+\)\(.*\.mo$\):%lang(\2)
\1\2\3:
s:\(.*/lib/python2.6/site-packages/anki/locale/\)\([^/_]\+\)\(.*\.mo$\):%lang(\2)
\1\2\3:
s:^\([^%].*\)::
s:%lang(C) ::
/^$/d'  anki.lang

%files -f anki.lang

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


[Bug 564537] Review Request: grc - simple python logfile colouriser

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Carl van Tonder c...@supervacuo.com 2010-02-13 08:45:30 
EST ---
Spec URL: http://supervacuo.com/fedora/grc.spec
SRPM URL: http://supervacuo.com/fedora/grc-1.3-0.2.fc12.src.rpm

 Why so uncommon release tag, what't wrong with 1%{?dist}?
It started life as a personal use package; fixed

 Add a comment about why %build in empty.
Done; it's because there is no compilation step — grc and grcat are python
scripts

 %install will cd into source dir, you should use:
 ./install.sh  $RPM_BUILD_ROOT
 I don't understand why you need the build patch, (which is a install patch:-)
For some reason, specifying relative paths seemed not to work yesterday, but
now it does indeed work. Patch now removed.

Thanks for taking a 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 555018] Review Request: gnac - An audio converter for GNOME (first package, seeking sponsor)

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Taylon taylonsi...@gmail.com 2010-02-13 09:43:02 EST ---
Oh sorry, I thought I should increase the release value just after the review
process.

Here is the package with the release increased:
Spec URL: http://taylon.fedorapeople.org/gnac-0.2.1-2.fc12.src.rpm
SRPM URL: http://taylon.fedorapeople.org/gnac.spec

Thank you very much by the reviews, now another reviewer will continue the
package revision?

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


[Bug 564588] Review Request: callgit - A tool for Ham Radio Operators to look up callsigns on the web

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jens Maucher je...@fedoraproject.org changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from Jens Maucher je...@fedoraproject.org 2010-02-13 10:11:36 
EST ---
This is my first self-made package.
I need a sponsor.

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


[Bug 564588] New: Review Request: callgit - A tool for Ham Radio Operators to look up callsigns on the web

2010-02-13 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: callgit -  A tool for Ham Radio Operators to look up 
callsigns on the web

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

   Summary: Review Request: callgit -  A tool for Ham Radio
Operators to look up callsigns on the web
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: je...@fedoraproject.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://jensm.fedorapeople.org/callgit.spec
SRPM URL: http://jensm.fedorapeople.org/callgit-2.0-1.fc13.src.rpm
Description: 
This progam allows you to search for another Ham's name and address
without having a web browser up. It is very simple.

rpmlint:
rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/callgit-2.0-1.fc13.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

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


[Bug 550582] Review Request: dpkg - Package maintenance system for Debian Linux

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #15 from Andrew Colin Kissa and...@topdog.za.net 2010-02-13 
11:08:13 EST ---

The requested changes have been implemented, comments are inline justifying the
various issues.

http://www.topdog-software.com/oss/SRPMS/fedora/dpkg/dpkg.spec
http://www.topdog-software.com/oss/SRPMS/fedora/dpkg/dpkg-1.15.5.6-1.fc12.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 532523] Review request: jarjar - Jar Jar Links

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steve Traylen steve.tray...@cern.ch changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #10 from Steve Traylen steve.tray...@cern.ch 2010-02-13 11:14:41 
EST ---
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 556308] Review Request: paratype-pt-sans-fonts - A pan-Cyrillic typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Nicolas Mailhot nicolas.mail...@laposte.net 2010-02-13 
11:17:10 EST ---
(In reply to comment #5)
 First, one cosmetic nitpick:
 $ rpm -qivp SRPMS/paratype-pt-sans-fonts-20100112-1.fc12.src.rpm 
 [snip]
 This package includes the four basic styles and two narrows styles for \
 economic setting.
 
 Could you make the backslash go away?

Stupid mistake, sorry about it, fixed


 +:ok, =:needs attention, -:needs fixing
 
 MUST Items:
 [+] MUST: rpmlint must be run on every package.
 $ rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/paratype-pt-sans-*
 paratype-pt-sans-caption-fonts.noarch: W: invalid-license PTFL

rpmlint is not up-to-date, PTFL is on the official Fedora license list
http://fedoraproject.org/wiki/Legal_considerations_for_fonts#Approved_font_licenses

 paratype-pt-sans-caption-fonts.noarch: W: wrong-file-end-of-line-encoding
 /usr/share/doc/paratype-pt-sans-caption-fonts-20100112/PT Free Font
 License_eng.txt
 paratype-pt-sans-caption-fonts.noarch: W: file-not-utf8
 /usr/share/doc/paratype-pt-sans-caption-fonts-20100112/PT Free Font
 License_eng.txt

Missed it with all the stupid spellchecking warnings, fixed

(sorry about it, I'm usually better than that)

 paratype-pt-sans-fonts.src: W: macro-in-%description %_font_pkg

This is a bug in rpmlint  rpmbuild


 [-] MUST: The sources used to build the package must match the upstream 
 source,
 as provided in the spec URL.
   = First one is the one I got from the URL in the spec file, second one is
 from the SRPM.
 $ sha1sum PTSans.zip 
 d812b275f2b13e59a69671785b53a4eb5805c561  PTSans.zip
 $ sha1sum SOURCES/PTSans.zip 
 f72ab3c42157ef81f7398b641213851f77e2deec  SOURCES/PTSans.zip

Upstream probably did the usual in-place archive update (very common for
fonts), I'll use the new version.

 This package is not approved as the source archives don't match.

Fixed
http://nim.fedorapeople.org/paratype-pt-sans-fonts-20100112-2.fc13.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 564557] Review Request: gfs-goschen-fonts - A 19th century Greek typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Nicolas Mailhot nicolas.mail...@laposte.net changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #6 from Nicolas Mailhot nicolas.mail...@laposte.net 2010-02-13 
11:27:37 EST ---
Thanks for the review!

New Package CVS Request
===
Package Name: gfs-goschen-fonts
Short Description: A 19th century Greek typeface
Owners: nim
Branches: devel
InitialCC: fonts-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 560457] Review Request: pyutil - A collection of mature utilities for Python programmers

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review?

--- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-02-13 
11:28:12 EST ---
Well,

* BuildRequires/Requires
  - Please check if python-zbase32 python-twisted-core
python-argparse should be listed only for BuildRequires.
! For example
  ./pyutil/iputil.py contains:
-
 9  # from Twisted
10  from twisted.internet import defer, reactor
11  from twisted.python import failure
12  from twisted.internet.protocol import DatagramProtocol
13  from twisted.internet.utils import getProcessOutput
14  from twisted.python.procutils import which
15  from twisted.python import log
-
  It seems that pyutil should also have Requires:
  python-twisted-core.

* rpmlint
-
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_iputil.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_picklesaver.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/scripts/randfile.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/scripts/randcookie.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_mathutil.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/lineutil.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_template 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_dictutil.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/benchmarks/bench_xor.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_time_format.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_strutil.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/scripts/tailx.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_odict.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_zlibutil.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/scripts/lines.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/scripts/memdump2dot.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/scripts/unsort.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_assertutil.py 0644L
/usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_xor.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_cache.py 0644L /usr/bin/env
pyutil.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/pyutil/test/test_jsonutil.py 0644L
/usr/bin/env
-
  - These scripts should not have shebang.

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


[Bug 531107] Review Request: perl-SQL-Tokenizer - A Perl package to tokenize SQL, generically

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Steve Traylen steve.tray...@cern.ch 2010-02-13 11:29:34 
EST ---
Hi,
Looks easy to approve but a couple of comments.

I see no reason for including MANIFEST.bak in the resulting package.

The license is as perl so not just Artistic 2.0 see: 

http://fedoraproject.org/wiki/Packaging/Perl#License_tag

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


[Bug 546451] Review Request: php-pear-HTML-Template-IT - Simple template API.

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Remi Collet fed...@famillecollet.com 2010-02-13 11:52:19 
EST ---
ping ?

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


[Bug 563510] Review Request: php-xcache - yet another php cacher

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 CC||fed...@famillecollet.com

--- Comment #15 from Remi Collet fed...@famillecollet.com 2010-02-13 11:49:50 
EST ---
Quick notes

- don't understand why you try to register this as a pecl extension (%post /
%postun). 
This is not a pecl extension, and you don't have the package.xml file required
for this.


- so, could also be removed
%global pecl_name xcache
Requires(post): %{__pecl}
Requires(postun): %{__pecl}
Provides:  php-pecl(%{pecl_name}) = %{version}


- xcache.ini 
zend_extension = /EXT_DIR/xcache.so
no need to give full path.


- %defattr(-, root, root, 0755)
(-,root,root,-) must be enough. php script should be 644 
This will remove a lot of rpmlint message
php-xcache.x86_64: E: script-without-shebang /usr/share/php-xcache/admin/...


- admin / coverager
You need to create a alias (httpd/conf.d) to give access to admin URL.
probably also need a writable dir for xcache.coveragedump_directory

If you don't want to make this available, don't install it, just add it in %doc

- INSTALL
don't need this file in RPM

- Buildroot
is acceptable, but read
http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot

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


[Bug 543940] Review Request: libmpdclient - Library for interfacing Music Player Daemon

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Adrian Reber adr...@lisas.de changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #15 from Adrian Reber adr...@lisas.de 2010-02-13 12:01:23 EST ---
Package Change Request
==
Package Name: libmpdclient
New Branches: F-11 F-12
Owners: adrian

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


[Bug 543940] Review Request: libmpdclient - Library for interfacing Music Player Daemon

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Adrian Reber adr...@lisas.de changed:

   What|Removed |Added

 Blocks||562918

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


[Bug 532523] Review request: jarjar - Jar Jar Links

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mary Ellen Foster mefos...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from Mary Ellen Foster mefos...@gmail.com 2010-02-13 12:03:38 
EST ---
New Package CVS Request
===
Package Name: jarjar
Short Description: Jar Jar Links
Owners: mef
Branches: F-11 F-12

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


[Bug 556308] Review Request: paratype-pt-sans-fonts - A pan-Cyrillic typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Mathieu Bridon boche...@fedoraproject.org 2010-02-13 
12:28:46 EST ---
[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
$ sha1sum PTSans.zip 
d812b275f2b13e59a69671785b53a4eb5805c561  PTSans.zip

I diffed the new spec file with the previous one, no changes but what was asked
here.

This package is now 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 556308] Review Request: paratype-pt-sans-fonts - A pan-Cyrillic typeface

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Nicolas Mailhot nicolas.mail...@laposte.net changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Nicolas Mailhot nicolas.mail...@laposte.net 2010-02-13 
12:41:17 EST ---
Thanks a lot for the review! (esp. since I had missed a few problems)

New Package CVS Request
===
Package Name: paratype-pt-sans-fonts
Short Description: A pan-Cyrillic typeface
Owners: nim
Branches: devel, F11, F12
InitialCC: fonts-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 550668] Review Request: mysql-administrator - A graphical tool for handling MySQL queries, construct and analize it

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 CC||fed...@famillecollet.com

--- Comment #1 from Remi Collet fed...@famillecollet.com 2010-02-13 12:37:55 
EST ---
MySQL Adminsitrator (and MySQL Query Browser) goes EOL and will be replace by
MySQL Workbench. See http://wb.mysql.com/?p=470

Don't you think it will be greater to work on this new software ?

I already have a working RPM, but it need more work (to split out embeded libs)

+

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


[Bug 530473] Review Request: lessfs - Lessfs is an inline data deduplicating filesystem.

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #30 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 
2010-02-13 12:38:32 EST ---
5) Ok, an this note about upstream state is higly appreciated. I saw it in
Patch0 (link to public bugtracker is even good, if it mailed - word about it).

Patch1 have not such note at all. If it is completely Fedora-related Patch - no
problem, just mention it in two words too.

6) No, then no. Ok.


(In reply to comment #28)
 Looks like qlz is available under the GPL-3, although the upstream packaging
 is, um, minimal in its license statements. I think a _better_ patch would make
 it a ./configure option, but I don't think the current state should be a
 blocker on getting the package in Fedora.
In most cases such modifications of course good idea make more common. But in
our case it is Fedora-specific short cut-off patch and If such configuration
switch is not interesting for upstream developer I think it absolutely have no
worth for us in packaging.


 On another note, though: the /etc/lessfs.cfg file is really a _per filesystem_
 configuration, not a global one. This is somewhat strange. This is probably a
 discussion for upstream, but it's important enough that it should get resolved
 before stuff starts going into production. Instead of /etc/lessfs.cfg, I think
 there should be /etc/lessfs/sample.cfg, and that should be configured with
 sensible defaults with the backing storage in (in /var/lib/lessfs/sample,
 maybe?) instead of the current default /data.
Yes. This is good mention. /data is not fit in any hierarhial standards.
Package also do not create and do not own it! If package try use it package it
provided must be required. I think there one more patch and defaults tuning
according our hier policy required.

Matthew Miller, thanks for note.

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


[Bug 560457] Review Request: pyutil - A collection of mature utilities for Python programmers

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Ruben Kerkhof ru...@rubenkerkhof.com 2010-02-13 12:41:44 
EST ---
Thanks, it should be fixed:

Spec URL: http://ruben.fedorapeople.org/pyutil.spec
SRPM URL: http://ruben.fedorapeople.org/pyutil-1.6.1-4.fc13.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 531107] Review Request: perl-SQL-Tokenizer - A Perl package to tokenize SQL, generically

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from David Fetter da...@fetter.org 2010-02-13 12:46:36 EST ---
Fixed :)

http://fetter.org/~shackle/perl-SQL-Tokenizer.spec
http://fetter.org/~shackle/perl-SQL-Tokenizer-0.19-2.fc12.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 550667] Review Request: mysql-querybrowser - A graphical tool for handling MySQL queries, construct and analize it

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 550667 depends on bug 550664, which changed state.

Bug 550664 Summary: Review Request: mysql-gui-common-static - Common data 
shared among the MySQL GUI Suites
https://bugzilla.redhat.com/show_bug.cgi?id=550664

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||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 550668] Review Request: mysql-administrator - A graphical tool for handling MySQL queries, construct and analize it

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 550668 depends on bug 550664, which changed state.

Bug 550664 Summary: Review Request: mysql-gui-common-static - Common data 
shared among the MySQL GUI Suites
https://bugzilla.redhat.com/show_bug.cgi?id=550664

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||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 550664] Review Request: mysql-gui-common-static - Common data shared among the MySQL GUI Suites

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX

--- Comment #1 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 
2010-02-13 12:53:36 EST ---
Work on its release starts many month ago... It died before the birth.
MySQL Adminsitrator (and MySQL Query Browser) goes EOL and will be replace by
MySQL Workbench. See http://wb.mysql.com/?p=470

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


[Bug 564567] Review Request: gwaei - A Japanese dictionary for Gnome

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from René Ribaud rene.rib...@free.fr 2010-02-13 13:05:44 EST 
---
Sorry,

Just I just forget to mention the rpmlint output :

[...@uggla SRPMS]$ rpmlint gwaei-1.2.1-4.fc12.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.


[...@uggla x86_64]$ rpmlint gwaei-1.2.1-4.fc12.x86_64.rpm 
gwaei.x86_64: W: conffile-without-noreplace-flag
/etc/gconf/schemas/gwaei.schemas
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Mock is fine for i386 as well as x86_64.
I don't put the output on this thread feel free to tell if if required.

I made a build test with Koji, it looks fine for all architecture.
Output here :
https://koji.fedoraproject.org/koji/taskinfo?taskID=1983466

Best regards.
René

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

[Bug 563480] Review Request: aduna-commons-concurrent - Extensions to the Java Concurrency package

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|fed...@matbooth.co.uk
   Flag||fedora-review?

--- Comment #2 from Mat Booth fed...@matbooth.co.uk 2010-02-13 13:21:38 EST 
---
I will take this one.

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


[Bug 563479] Review Request: aduna-commons-text - Manipulate/transform/parse text in various ways

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|fed...@matbooth.co.uk
   Flag||fedora-review?

--- Comment #2 from Mat Booth fed...@matbooth.co.uk 2010-02-13 13:19:59 EST 
---
I will take this one.

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


[Bug 564567] Review Request: gwaei - A Japanese dictionary for Gnome

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-02-13 
13:38:09 EST ---
Well, I want to go to bed for now, however some brief notes:

- On rawhide your package does not build as below.
  http://koji.fedoraproject.org/koji/taskinfo?taskID=1983436
  This is because Fedora 13 changed the behavior of linker:
  http://fedoraproject.org/wiki/UnderstandingDSOLinkChange

  To check F-13 linker behavior on F-12/11, you can do this by adding
-
export LDFLAGS=-Wl,--no-add-needed
-
  before %configure line.

Then some other notes
- Requires: cjkuni-ukai-fonts is not desired. On Fedora cjkuni- is
  the default font for Chinese but not for Japanese. On Fedora Japanese
  people use vlgothic-fonts rpm for fonts by default.

- Your spec file contains:
-
#%configure --disable-schemas
%configure --disable-schemas-install
-
  With this build log will show (actually
  https://koji.fedoraproject.org/koji/taskinfo?taskID=1983466
  shows) that configure is executed twice, not once.
  This is because rpmbuild will expand macros (in this case %configure)
  anyway even if the line is marked as a comment.

  To prevent this, you should use #%%configure so that macros
  won't be expanded in comment lines.

- Please consider to use
-
make install DESTDIR=$RPM_BUILD_ROOT INSTALL=install -p
-
  to keep timestamps on installed files as much as possible.
  This method usually works for Makefiles generated by recent autotools.

- Currently documents are installed both under
  - /usr/share/doc/gwaei
  - /usr/shaer/doc/gwaei-1.2.1
  Please unify these directories.

- I have not checked this package in details, however usually the
  file ABOUT-NLS is unuseful.

- Files/directories under %{_datadir}/doc/ are automatically marked
  as %doc.

- rpmlint may complain, however we usually don't regard gconf schemas
  files as config files so please remove %config from gconf schemas
  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 550582] Review Request: dpkg - Package maintenance system for Debian Linux

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #16 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-02-13 
13:47:04 EST ---
Thanks, we are almost there. I have a few questions, comments and a few minor
issues:


? The only rpmlint is:
dpkg.src:1: E: hardcoded-library-path in /usr/lib
which is ironically a lot better than our own rpm package. Anyway, rpmlint is
an rpmlint. I fail to understand that why we hardcode the libdir to /usr/lib. I
read your comment, but is it not possible to patch those perl scripts to look
at /usr/share instead of /usr/lib? 

On the other hand our rpm package also puts files in /usr/lib. I wonder if this
is really a must.

! The devel package looks like it can be made noarch. I don't know if this
would screw up our build system though. (Meanwhile dpkg and dselect packages
cannot be noarch)

* The  following directories and possibly their subdirectories are not owned:
   /usr/lib/perl5/vendor_perl/5.10.0/Debian/
   %dpkgdir/dpkg/methods/

* Build process log does not show us what flags are used during compilation.
Please make it verbose.

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


[Bug 550582] Review Request: dpkg - Package maintenance system for Debian Linux

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #17 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-02-13 
13:48:52 EST ---
koji seems to be happy with this package too:
   http://koji.fedoraproject.org/koji/taskinfo?taskID=1983856

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


[Bug 550582] Review Request: dpkg - Package maintenance system for Debian Linux

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #18 from Andrew Colin Kissa and...@topdog.za.net 2010-02-13 
13:57:17 EST ---

Actually i put the files in /usr/lib based on our own rpm package, let me see
if i can patch it to move the files into /usr/share.

Will update and post.

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


[Bug 561086] Review Request: perl-String-Escape - String::Escape Perl module

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Chris Weyl cw...@alumni.drew.edu 2010-02-13 14:17:03 EST 
---
Koji (success) http://koji.fedoraproject.org/koji/taskinfo?taskID=1983996

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


[Bug 511144] Review Request: openstreetmap - A JavaScript library for displaying the OpenStreetMap.org map data

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX

Bug 511144 depends on bug 508533, which changed state.

Bug 508533 Summary: Please, update to OL 2.8
https://bugzilla.redhat.com/show_bug.cgi?id=508533

   What|Old Value   |New Value

 Status|ON_QA   |CLOSED
 Resolution||ERRATA

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


[Bug 538172] Review Request: pyactivemq - Python wrapper around activemq-cpp for messaging

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 538172 depends on bug 533899, which changed state.

Bug 533899 Summary: Review Request: activemq-cpp - C++ implementation of 
JMS-like messaging client
https://bugzilla.redhat.com/show_bug.cgi?id=533899

   What|Old Value   |New Value

 Status|ASSIGNED|ON_QA
 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #4 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-02-13 
14:37:06 EST ---
Any progress?

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


[Bug 561086] Review Request: perl-String-Escape - String::Escape Perl module

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Chris Weyl cw...@alumni.drew.edu 2010-02-13 14:44:51 EST 
---
Hmmm...  I'm seeing a 404 error when trying to fetch the upstream tarball.

[cw...@athena bin]$ wget
http://www.cpan.org/modules/by-module/String/String-Escape-2010.002.tar.gz
--2010-02-13 11:41:37-- 
http://www.cpan.org/modules/by-module/String/String-Escape-2010.002.tar.gz
Resolving www.cpan.org... 207.171.7.177
Connecting to www.cpan.org|207.171.7.177|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2010-02-13 11:41:37 ERROR 404: Not Found.

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


[Bug 560894] Review Request: perl-Pod-Coverage-TrustPod - Allow a module's pod to contain Pod::Coverage hints

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Chris Weyl cw...@alumni.drew.edu 2010-02-13 14:55:56 EST 
---
Koji (success) http://koji.fedoraproject.org/koji/taskinfo?taskID=1984102

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


[Bug 561761] Review Request: perl-HTML-Entities-Numbered - Conversion of numbered HTML entities

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Chris Weyl cw...@alumni.drew.edu 2010-02-13 14:57:15 EST 
---
Koji (success) http://koji.fedoraproject.org/koji/taskinfo?taskID=1984104

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


[Bug 560894] Review Request: perl-Pod-Coverage-TrustPod - Allow a module's pod to contain Pod::Coverage hints

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Weyl cw...@alumni.drew.edu changed:

   What|Removed |Added

   Flag|fedora-review?  |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 561761] Review Request: perl-HTML-Entities-Numbered - Conversion of numbered HTML entities

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Chris Weyl cw...@alumni.drew.edu 2010-02-13 15:07:16 EST 
---
Not blockers, but:

Replacing PERL_INSTALL_ROOT with DESTDIR would seem to be a good idea, given
the recent fun along those lines :)

Think about including %{?perl_default_filter} by default, even if just pro
forma.  It won't hurt anything and might save some pain down the line.

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

Spec looks sane, clean and consistent; license is correct (GPL+ or Artistic);
make test passes cleanly.

Source tarballs match upstream (sha1sum):
894094d26046006d7afd332ad3860d761ed16405 HTML-Entities-Numbered-0.04.tar.gz
894094d26046006d7afd332ad3860d761ed16405
HTML-Entities-Numbered-0.04.tar.gz.srpm

Final provides / requires are sane:

0 packages and 1 specfiles checked; 0 errors, 0 warnings.
= perl-HTML-Entities-Numbered-0.04-2.fc13.noarch.rpm =
 rpmlint
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
 provides for perl-HTML-Entities-Numbered-0.04-2.fc13.noarch.rpm
perl(HTML::Entities::Numbered) = 0.04
perl-HTML-Entities-Numbered = 0.04-2.fc13
perl(HTML::Entities::Numbered::Table) = 0.01
 requires for perl-HTML-Entities-Numbered-0.04-2.fc13.noarch.rpm
perl(base)  
perl(HTML::Entities::Numbered::Table)  
perl(:MODULE_COMPAT_5.10.1)  
perl(strict)  
perl(vars)  

= perl-HTML-Entities-Numbered-0.04-2.fc13.src.rpm =
 rpmlint
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
 provides for perl-HTML-Entities-Numbered-0.04-2.fc13.src.rpm
 requires for perl-HTML-Entities-Numbered-0.04-2.fc13.src.rpm
perl(ExtUtils::MakeMaker)  
perl(Test::More) = 0.32

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 515247] Review Request: tint2 - a lightweight X11 desktop panel and task manager

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Chess Griffin ch...@chessgriffin.com 2010-02-13 15:07:52 
EST ---
Thanks for the ping.  I am going to refocus on this one.

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


[Bug 561761] Review Request: perl-HTML-Entities-Numbered - Conversion of numbered HTML entities

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Weyl cw...@alumni.drew.edu 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 561761] Review Request: perl-HTML-Entities-Numbered - Conversion of numbered HTML entities

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Weyl cw...@alumni.drew.edu changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|cw...@alumni.drew.edu

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


[Bug 538172] Review Request: pyactivemq - Python wrapper around activemq-cpp for messaging

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Steve Traylen steve.tray...@cern.ch 2010-02-13 15:09:00 
EST ---
Hi,
I kind of made a mistake in upgrading activemq-cpp to 3.1 which then
meant that the released pyactivemq is incompatible.

The SVN head is reported as building but the tests fail, so waiting really.

Steve

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


[Bug 237336] Review Request: perl-Crypt-OpenSSL-X509 -- Perl OpenSSL bindings for X509 support

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Steve Traylen steve.tray...@cern.ch 2010-02-13 15:13:10 
EST ---
Hi Wes,

 Can you make the package request and maintain the package? Else I will :-)

Steve

-- 
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 564520] Review Request: frama-c - Framework for source code analysis of C software

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

David A. Wheeler dwhee...@dwheeler.com changed:

   What|Removed |Added

 CC||dwhee...@dwheeler.com
 AssignedTo|nob...@fedoraproject.org|dwhee...@dwheeler.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 561761] Review Request: perl-HTML-Entities-Numbered - Conversion of numbered HTML entities

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Weyl cw...@alumni.drew.edu changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

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


[Bug 564520] Review Request: frama-c - Framework for source code analysis of C software

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from David A. Wheeler dwhee...@dwheeler.com 2010-02-13 
15:16:09 EST ---
Thanks for creating this package!!  I know this was rough to package.  I'll
post comments later.

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


[Bug 562785] Package Review: perl-Debug-Client - client side code for perl debugger

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Weyl cw...@alumni.drew.edu changed:

   What|Removed |Added

 CC||cw...@alumni.drew.edu

--- Comment #1 from Chris Weyl cw...@alumni.drew.edu 2010-02-13 15:15:51 EST 
---
%description might need a bit of work :)  And using DESTDIR rather than
PERL_INSTALL_ROOT is probably a good idea, given the recent issues around it.

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


[Bug 563820] Review Request: globus-libxml2 - Globus Toolkit - LibXML2 Library

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Blocks||516536

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


[Bug 516524] Review Request: globus-duct-control - Globus Toolkit - Globus Duct Control

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Blocks|516536  |

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


[Bug 516539] Review Request: globus-gram-job-manager-setup-lsf - Globus Toolkit - LSF Job Manager Setup

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 516539 depends on bug 516531, which changed state.

Bug 516531 Summary: Review Request: globus-gass-cache-program - Globus Toolkit 
- Tools to manipulate local and remote GASS caches
https://bugzilla.redhat.com/show_bug.cgi?id=516531

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516539 depends on bug 516535, which changed state.

Bug 516535 Summary: Review Request: globus-gram-job-manager-scripts - Globus 
Toolkit - GRAM Job ManagerScripts
https://bugzilla.redhat.com/show_bug.cgi?id=516535

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

--- Comment #2 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-02-13 
16:21:18 EST ---
Review updated for Globus Toolkit 5:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager-setup-lsf.spec
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-setup-lsf-2.3-1.fc12.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 516537] Review Request: globus-gram-job-manager-setup-fork - Globus Toolkit - Fork Job Manager Setup

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 516537 depends on bug 516531, which changed state.

Bug 516531 Summary: Review Request: globus-gass-cache-program - Globus Toolkit 
- Tools to manipulate local and remote GASS caches
https://bugzilla.redhat.com/show_bug.cgi?id=516531

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516537 depends on bug 516535, which changed state.

Bug 516535 Summary: Review Request: globus-gram-job-manager-scripts - Globus 
Toolkit - GRAM Job ManagerScripts
https://bugzilla.redhat.com/show_bug.cgi?id=516535

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-02-13 
16:21:06 EST ---
Review updated for Globus Toolkit 5:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager-setup-fork.spec
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-setup-fork-4.0-1.fc12.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 516522] Review Request: globus-nexus - Globus Toolkit - Nexus Library

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Blocks|516528  |

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


[Bug 516522] Review Request: globus-nexus - Globus Toolkit - Nexus Library

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Blocks|516536  |

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


[Bug 516540] Review Request: globus-gram-job-manager-setup-pbs - Globus Toolkit - PBS Job Manager Setup

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 516540 depends on bug 516531, which changed state.

Bug 516531 Summary: Review Request: globus-gass-cache-program - Globus Toolkit 
- Tools to manipulate local and remote GASS caches
https://bugzilla.redhat.com/show_bug.cgi?id=516531

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516540 depends on bug 516535, which changed state.

Bug 516535 Summary: Review Request: globus-gram-job-manager-scripts - Globus 
Toolkit - GRAM Job ManagerScripts
https://bugzilla.redhat.com/show_bug.cgi?id=516535

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-02-13 
16:21:25 EST ---
Review updated for Globus Toolkit 5:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager-setup-pbs.spec
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-setup-pbs-4.0-1.fc12.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 516538] Review Request: globus-gram-job-manager-setup-condor - Globus Toolkit - Condor Job Manager Setup

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 516538 depends on bug 516531, which changed state.

Bug 516531 Summary: Review Request: globus-gass-cache-program - Globus Toolkit 
- Tools to manipulate local and remote GASS caches
https://bugzilla.redhat.com/show_bug.cgi?id=516531

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516538 depends on bug 516535, which changed state.

Bug 516535 Summary: Review Request: globus-gram-job-manager-scripts - Globus 
Toolkit - GRAM Job ManagerScripts
https://bugzilla.redhat.com/show_bug.cgi?id=516535

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-02-13 
16:21:11 EST ---
Review updated for Globus Toolkit 5:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager-setup-condor.spec
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-setup-condor-4.2-1.fc12.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 516528] Review Request: globus-gram-client-tools - Globus Toolkit - Job Management Tools (globusrun)

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Depends on|516526, 516522  |

Bug 516528 depends on bug 516519, which changed state.

Bug 516519 Summary: Review Request: globus-gram-client - Globus Toolkit - GRAM 
Client Library
https://bugzilla.redhat.com/show_bug.cgi?id=516519

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516528 depends on bug 516527, which changed state.

Bug 516527 Summary: Review Request: globus-gass-server-ez - Globus Toolkit - 
Globus Gass Server_ez
https://bugzilla.redhat.com/show_bug.cgi?id=516527

   What|Old Value   |New Value

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

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-02-13 
16:20:54 EST ---
Review updated for Globus Toolkit 5:

http://www.grid.tsl.uu.se/review/globus-gram-client-tools.spec
http://www.grid.tsl.uu.se/review/globus-gram-client-tools-7.3-1.fc12.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 516536] Review Request: globus-gram-job-manager - Globus Toolkit - GRAM Jobmanager

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Depends on|516522, 516524  |563820

Bug 516536 depends on bug 516518, which changed state.

Bug 516518 Summary: Review Request: globus-gram-protocol - Globus Toolkit - 
GRAM Protocol Library
https://bugzilla.redhat.com/show_bug.cgi?id=516518

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516536 depends on bug 516529, which changed state.

Bug 516529 Summary: Review Request: globus-gass-cache - Globus Toolkit - Globus 
Gass Cache
https://bugzilla.redhat.com/show_bug.cgi?id=516529

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516536 depends on bug 516533, which changed state.

Bug 516533 Summary: Review Request: globus-gatekeeper - Globus Toolkit - Globus 
Gatekeeper
https://bugzilla.redhat.com/show_bug.cgi?id=516533

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|CLOSED
 Resolution||ERRATA

Bug 516536 depends on bug 516532, which changed state.

Bug 516532 Summary: Review Request: globus-scheduler-event-generator - Globus 
Toolkit - Scheduler Event Generator
https://bugzilla.redhat.com/show_bug.cgi?id=516532

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516536 depends on bug 516534, which changed state.

Bug 516534 Summary: Review Request: globus-gram-job-manager-callout-error - 
Globus Toolkit - Globus GRAM Jobmanager Callout Errors
https://bugzilla.redhat.com/show_bug.cgi?id=516534

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516536 depends on bug 516531, which changed state.

Bug 516531 Summary: Review Request: globus-gass-cache-program - Globus Toolkit 
- Tools to manipulate local and remote GASS caches
https://bugzilla.redhat.com/show_bug.cgi?id=516531

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||ERRATA
 Status|ASSIGNED|CLOSED

Bug 516536 depends on bug 516535, which changed state.

Bug 516535 Summary: Review Request: globus-gram-job-manager-scripts - Globus 
Toolkit - GRAM Job ManagerScripts
https://bugzilla.redhat.com/show_bug.cgi?id=516535

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||ERRATA

--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-02-13 
16:21:00 EST ---
Review updated for Globus Toolkit 5:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager.spec
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-10.17-1.fc12.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 565232] New: Review Request: perl-Log-Any - Bringing loggers and listeners together

2010-02-13 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: perl-Log-Any - Bringing loggers and listeners together

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

   Summary: Review Request: perl-Log-Any - Bringing loggers and
listeners together
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: st...@silug.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ftp.kspei.com/pub/steve/rpms/perl-Log-Any/perl-Log-Any.spec
SRPM URL: http://ftp.kspei.com/pub/steve/rpms/perl-Log-Any-0.11-1.src.rpm
Description:
Log::Any allows CPAN modules to safely and efficiently log messages, while
letting the application choose (or decline to choose) a logging mechanism
such as Log::Dispatch or Log::Log4perl.

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


[Bug 565232] Review Request: perl-Log-Any - Bringing loggers and listeners together

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Steven Pritchard st...@silug.org 2010-02-13 17:05:01 EST 
---
This is required by the latest HTML::Mason.

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


[Bug 553705] Review Request: rinputd - A server for receiving input events over the network

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #20 from Stefan Riemens fgfs.ste...@gmail.com 2010-02-13 18:15:11 
EST ---
Ok great!

That means 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 543608] Review Request: udisks - Storage Management Service

2010-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

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


[Bug 565251] New: Review Request: coan - A commandline tool for simplifying the preprocessor conditionals in source code

2010-02-13 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: coan - A commandline tool for simplifying the 
preprocessor conditionals in source code

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

   Summary: Review Request: coan - A commandline tool for
simplifying the preprocessor conditionals in source
code
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: e...@brouhaha.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~brouhaha/coan/coan.spec
SRPM URL: http://fedorapeople.org/~brouhaha/coan/coan-4.0-1.fc12.src.rpm
Koji scratch build for F-12:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1984936
Description:

Coan is a software engineering tool for analysing preprocessor-based
configurations of C or C++ source code.  Its principle use is to
simplify a body of source code by eliminating any parts that are
redundant with respect to a specified configuration.  It is a more
powerful successor to the FreeBSD 'unifdef' tool and the 'sunifdef'
tool.

Coan is most useful to developers of constantly evolving products
with large code bases, where preprocessor conditionals and #if-directives
are used to differentiate progressive releases or parallel variants of
the product.  In these settings the upkeep of the product's configuration
tree can become difficult and the incidence of configuration-related
defects can become costly.  Coan can largely automate the maintenance of
preprocessor-based configurations and the investigation of configuration-
related questions.

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