[Bug 730570] Review Request: python-requests - Python HTTP library for Humans

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #21 from Fedora Update System upda...@fedoraproject.org 
2012-04-01 02:50:56 EDT ---
python-requests-0.10.8-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/python-requests-0.10.8-1.el6

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

[Bug 689112] Review Request: php-pear-File-CSV - Read and write of CSV files

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|NEW |MODIFIED

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

[Bug 689114] Review Request: php-pear-File-Util - Common file and directory utility functions

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2012-04-01 
04:06:39 EDT ---
php-pear-File-Util-1.0.0-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/php-pear-File-Util-1.0.0-2.fc17

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

[Bug 689112] Review Request: php-pear-File-CSV - Read and write of CSV files

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2012-04-01 
04:05:18 EDT ---
php-pear-File-CSV-1.0.0-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/php-pear-File-CSV-1.0.0-2.fc17

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

[Bug 689114] Review Request: php-pear-File-Util - Common file and directory utility functions

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|NEW |MODIFIED

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

[Bug 755510] Review Request: gnome-shell-extension-system-monitor-applet - Gnome shell system monitor extension

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

coll...@ieee.org changed:

   What|Removed |Added

 CC||coll...@ieee.org

--- Comment #19 from coll...@ieee.org 2012-04-01 04:31:57 EDT ---
regarding comment#18

I think it would be worth to notice that from Gnome-Shell 3.3/3.4 
according to the new README.md file and the website, 
installation could be done in One Click Install via 
https://extensions.gnome.org/extension/120/system-monitor/;. 
As my production host is F-16, I haven't tested this feature 
and I cannot say if it's working.

i had recently tried the 
  'gnome-shell-extension-system-monitor'
install through firefox from https://extensions.gnome.org 
on fc17.

(one-click install is neet but was 
kind of unnerving that you can 
install/configure/remove a shell extension   
through firefox by enabling a firefox gnome plugin 
and going to https://extensions.gnome.org .  
concerned that would make for a nasty script vector. :'\)

'system-monitor' seemed to work realy nicely in the upper bar 
(though seemed to only showed info for 1 of the cpu) 

at some point in playing with the extension it 
seemed to conflict with the lower tray system monitor 
  gnome-shell-extension-systemMonitor-3.4.0-1.fc17.noarch  
and seemed to break login during the gnome-panel bar startup 
until i uninstalled 
  gnome-shell-extension-systemMonitor-3.4.0-1.fc17.noarch  
and cleared out 
  '.local/share/gnome-shell/extensions'

looking forward to both the 'system-monitor' and 'systemMonitor' extensions

thanks for working on the system-monitor
i will be glad when my gnome panel has all those graphs again. 
in withdrawl since fc14 ;')

keep up the good work,
'system-monitor' is a great companion to 'systemMonitor' :')

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

[Bug 690728] Review Request: Nitrate - A test case management system written in Django

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Yuguang Wang yuw...@redhat.com changed:

   What|Removed |Added

 CC||ya...@redhat.com

--- Comment #61 from Yuguang Wang yuw...@redhat.com 2012-04-01 05:10:30 EDT 
---
add yawli to cc 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 730314] Review Request: jboss-invocation - JBoss Invocation API

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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

[Bug 730314] Review Request: jboss-invocation - JBoss Invocation API

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #16 from Fedora Update System upda...@fedoraproject.org 
2012-04-01 05:49:21 EDT ---
jboss-invocation-1.1.1-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/jboss-invocation-1.1.1-2.fc17

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

[Bug 804625] Review Request: perl-MooseX-Has-Options - Succinct options for Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mario.blaetterm...@gmail.co
   ||m
 AssignedTo|nob...@fedoraproject.org|mario.blaetterm...@gmail.co
   ||m
   Flag||fedora-review?

--- Comment #1 from Mario Blättermann mario.blaetterm...@gmail.com 2012-04-01 
06:24:58 EDT ---
Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3954319

$ rpmlint -i -v *
perl-MooseX-Has-Options.noarch: I: checking
perl-MooseX-Has-Options.noarch: I: checking-url
http://search.cpan.org/dist/MooseX-Has-Options/ (timeout 10 seconds)
perl-MooseX-Has-Options.noarch: E: incorrect-fsf-address
/usr/share/doc/perl-MooseX-Has-Options-0.002/LICENSE
The Free Software Foundation address in this file seems to be outdated or
misspelled.  Ask upstream to update the address, or if this is a license file,
possibly the entire file with a new copy available from the FSF.

perl-MooseX-Has-Options.src: I: checking
perl-MooseX-Has-Options.src: I: checking-url
http://search.cpan.org/dist/MooseX-Has-Options/ (timeout 10 seconds)
perl-MooseX-Has-Options.src: I: checking-url
http://www.cpan.org/authors/id/P/PS/PSHANGOV/MooseX-Has-Options-0.002.tar.gz
(timeout 10 seconds)
perl-MooseX-Has-Options.spec: I: checking-url
http://www.cpan.org/authors/id/P/PS/PSHANGOV/MooseX-Has-Options-0.002.tar.gz
(timeout 10 seconds)
2 packages and 1 specfiles checked; 1 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 804625] Review Request: perl-MooseX-Has-Options - Succinct options for Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Mario Blättermann mario.blaetterm...@gmail.com 2012-04-01 
06:30:51 EDT ---
-
key:

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

[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license.
GPL+ or Artistic
[+] MUST: The License field in the package spec file must match the actual
license.
[.] MUST: The file containing the text of the license(s) for the package must
be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
$ md5sum *
6ea980a6321d664b9948a95d58807f4c  MooseX-Has-Options-0.002.tar.gz
6ea980a6321d664b9948a95d58807f4c  MooseX-Has-Options-0.002.tar.gz.packaged

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
- Succesful Koji build available.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[.] MUST: The spec file MUST handle locales properly.
[.] MUST: If a package installs files below %{_datadir}/icons, the icon cache
must be updated.
[.] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
[.] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, ...
[+] MUST: A package must own all directories that it creates.
[+] MUST: A Fedora package must not list a file more than once in %files.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Packages must not provide RPM dependency information when that
information is not global in nature, or are otherwise handled.
[.] MUST: When filtering automatically generated RPM dependency information,
the filtering system implemented by Fedora must be used.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
[+] MUST: Files in %doc must not affect the runtime of the application.
[.] MUST: Header files must be in a -devel package.
[.] MUST: Static libraries must be in a -static package.
[.] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), ...
[.] MUST: devel packages must require the base package using a fully versioned
dependency.
[.] MUST: Packages must NOT contain any .la libtool archives.
[.] MUST: Packages containing GUI applications must include a %{name}.desktop
file
[.] MUST: .desktop files 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.

[.] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream...
[+] SHOULD: Timestamps of files should be preserved.
[+] SHOULD: The reviewer should test that the package builds in mock.
See Koji build above (which uses mock anyway)
[+] SHOULD: The reviewer should test that the package functions as described.
I assume the packager has tested it. Don't know how to test it on my
system.
[+] SHOULD: If scriptlets are used, those scriptlets must be sane.
[.] SHOULD: Usually, subpackages other than devel should require the base
package using a fully versioned dependency.
[.] SHOULD: pkgconfig(.pc) files should be placed in a -devel pkg.
[.] SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin,
/usr/bin, or /usr/sbin ...
[+] SHOULD: Your package should contain man pages for binaries/scripts.



PACKAGE 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 800526] Review Request: seahorse-sharing - Sharing of PGP public keys via DNS-SD and HKP

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Kalev Lember kalevlem...@gmail.com changed:

   What|Removed |Added

 CC||kalevlem...@gmail.com

--- Comment #11 from Kalev Lember kalevlem...@gmail.com 2012-04-01 06:52:08 
EDT ---
I am pretty uncomfortable seeing Obsoletes: seahorse  3.1.4 in this spec file.
What this does on upgrades is removing seahorse and installing seahorse-sharing
instead. Is this behaviour intended?

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

[Bug 798738] Review request: mysqlenum - is an automatic blind SQL injection tool.

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #20 from Michael Scherer m...@zarb.org 2012-04-01 07:17:49 EDT ---
The tarball is still not matching the md5sum, the tarball upstream is using
root user, and seems that the one in the rpm use the rpm user.

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

[Bug 760154] Review Request: xcb-util-keysyms - Standard X key constants and keycodes conversion on top of libxcb

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Mohamed El Morabity pikachu.2...@gmail.com 2012-04-01 
07:41:35 EDT ---
Do you plan to push the packages in Bodhi soon?

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

[Bug 807017] Review Request: ovirt-engine - Management server for Open Virtualization

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Juan Hernández juan.hernan...@redhat.com 2012-04-01 
08:04:32 EDT ---
The spec has been updated to fix most of the errors detected during the review
that Ofer did. The updated .src.rpm is available here:

http://jhernand.fedorapeople.org/rpms/ovirt-engine-3.0.0-9.0001.fc18.src.rpm

The updated package builds correctly in Koji:

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

There are still rpmlint warnings and also some errors that can't be avoided,
but can be explained:

ovirt-engine.noarch: E: non-readable /etc/ovirt-engine/engine.conf 0640L
ovirt-engine-iso-uploader.noarch: E: non-readable
/etc/ovirt-engine/isouploader.conf 0600L
ovirt-engine-log-collector.noarch: E: non-readable
/etc/ovirt-engine/logcollector.conf 0600L
ovirt-engine-notification-service.noarch: E: non-readable
/etc/ovirt-engine/notifier/notifier.conf 0640L

These file need restrictive read permissions because they may contain
passwords.

ovirt-engine-log-collector.noarch: E: script-without-shebang
/usr/lib/python2.7/site-packages/sos/plugins/postgresql.py
ovirt-engine-log-collector.noarch: E: script-without-shebang
/usr/lib/python2.7/site-packages/sos/plugins/jboss.py
ovirt-engine-log-collector.noarch: E: script-without-shebang
/usr/lib/python2.7/site-packages/sos/plugins/engine.py

These files have no shebang and execution permissions like all the other SOS
plugins.

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

[Bug 760154] Review Request: xcb-util-keysyms - Standard X key constants and keycodes conversion on top of libxcb

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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

[Bug 760154] Review Request: xcb-util-keysyms - Standard X key constants and keycodes conversion on top of libxcb

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2012-04-01 
08:13:24 EDT ---
xcb-util-keysyms-0.3.8-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/xcb-util-keysyms-0.3.8-2.fc17

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

[Bug 760154] Review Request: xcb-util-keysyms - Standard X key constants and keycodes conversion on top of libxcb

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Thomas Moschny thomas.mosc...@gmx.de 2012-04-01 08:19:56 
EDT ---
Sorry almost forgot that. Thanks for the reminder!

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

[Bug 804625] Review Request: perl-MooseX-Has-Options - Succinct options for Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Iain Arnell iarn...@gmail.com 2012-04-01 08:41:33 EDT ---
New Package SCM Request
===
Package Name: perl-MooseX-Has-Options
Short Description: Succinct options for Moose
Owners: iarnell
Branches: f15 f16 f17
InitialCC: perl-sig

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

[Bug 804625] Review Request: perl-MooseX-Has-Options - Succinct options for Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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

[Bug 800526] Review Request: seahorse-sharing - Sharing of PGP public keys via DNS-SD and HKP

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Michael Scherer m...@zarb.org 2012-04-01 08:45:43 EDT ---
From what I understand, if I have the old seahorse, and if I install the new
seahorse-sharing , it will remove the old seahorse, that's all.

If I do a upgrade from one fedora version to another one, it should also
install a new seahorse, and so the Obsoletes should not apply on it.

But I do not know enough yum to be sure

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

[Bug 808875] New: Review Request: perl-Role-Tiny - A nouvelle cusine portion size slice of Moose

2012-04-01 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-Role-Tiny - A nouvelle cusine portion size slice 
of Moose
Alias: perl-Role-Tiny

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

   Summary: Review Request: perl-Role-Tiny - A nouvelle cusine
portion size slice of Moose
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/Role-Tiny/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-Role-Tiny.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Role-Tiny-1.00-1.fc18.src.rpm

Description:
Role::Tiny is a minimalist role composition tool.

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

*rt-0.10_02

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

[Bug 805047] Review Request: janino - An embedded Java compiler

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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 808881] New: Review Request: perl-MetaCPAN-API - Comprehensive, DWIM-featured API to MetaCPAN

2012-04-01 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-MetaCPAN-API - Comprehensive, DWIM-featured API 
to MetaCPAN
Alias: perl-MetaCPAN-API

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

   Summary: Review Request: perl-MetaCPAN-API - Comprehensive,
DWIM-featured API to MetaCPAN
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/MetaCPAN-API/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-MetaCPAN-API.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-MetaCPAN-API-0.42-1.fc18.src.rpm

Description:
This is a hopefully-complete API-compliant interface to MetaCPAN
(https://metacpan.org) with DWIM capabilities, to make your life easier.

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

*rt-0.10_02

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

[Bug 808884] New: Review Request: perl-Data-Clone - Polymorphic data cloning

2012-04-01 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-Data-Clone - Polymorphic data cloning
Alias: perl-Data-Clone

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

   Summary: Review Request: perl-Data-Clone - Polymorphic data
cloning
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/Data-Clone/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-Data-Clone.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Data-Clone-0.003-1.fc18.src.rpm

Description:
Data::Clone does data cloning, i.e. copies things recursively. This is
smart so that it works with not only non-blessed references, but also
with blessed references (i.e. objects). When clone() finds an object, it
calls a clone method of the object if the object has a clone, otherwise
it makes a surface copy of the object. That is, this module does
polymorphic data cloning.

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

*rt-0.10_02

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

[Bug 808886] New: Review Request: perl-CPAN-Meta-Requirements - Set of version requirements for a CPAN dist

2012-04-01 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-CPAN-Meta-Requirements - Set of version 
requirements for a CPAN dist

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

   Summary: Review Request: perl-CPAN-Meta-Requirements - Set of
version requirements for a CPAN dist
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/CPAN-Meta-Requirements/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL:
http://fedorapeople.org/~iarnell/review/perl-CPAN-Meta-Requirements.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-CPAN-Meta-Requirements-2.121-1.fc18.src.rpm

Description:
A CPAN::Meta::Requirements object models a set of version constraints like
those specified in the META.yml or META.json files in CPAN distributions.
It can be built up by adding more and more constraints, and it will reduce
them to the simplest representation.

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

*rt-0.10_02

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

[Bug 808886] Review Request: perl-CPAN-Meta-Requirements - Set of version requirements for a CPAN dist

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Iain Arnell iarn...@gmail.com 2012-04-01 10:16:56 EDT ---
Upstream is messing with me. CPAN-Meta-Requirements used to be
Version-Requirements. It was merged into CPAN-Meta and has now been split back
out again with the new name. There's an explicit Conflicts in the spec to
ensure that CPAN-Meta-Requirements doesn't implicitly conflict with CPAN-Meta 
2.120921.

This will also need to be sub-packaged in perl.spec for 5.16 and possibly
5.14.3.

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

[Bug 798738] Review request: mysqlenum - is an automatic blind SQL injection tool.

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #21 from pjp pj.pan...@yahoo.co.in 2012-04-01 11:18:07 EDT ---
I replaced the source tarball with the upstream one. Please see

SPEC: http://pjp.dgplug.org/tools/mysqlenum.spec
SRPM: http://pjp.dgplug.org/tools/mysqlenum-0.3-8.fc16.src.rpm

$ rpmlint SPECS/mysqlenum.spec SPECS/mysqlenum.spec: W: invalid-url Source0:
mysqlenum-0.3.tar.gz
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

$ rpmlint RPMS/x86_64/mysqlenum-0.3-8.fc16.x86_64.rpm 
mysqlenum.x86_64: W: no-manual-page-for-binary mysqlenum
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint SRPMS/mysqlenum-0.3-8.fc16.src.rpm 
mysqlenum.src: W: invalid-url Source0: mysqlenum-0.3.tar.gz
1 packages and 0 specfiles checked; 0 errors, 1 warnings.


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 808518] Review Request: perl-Time-OlsonTZ-Download - Olson timezone database from source

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mario.blaetterm...@gmail.co
   ||m
 AssignedTo|nob...@fedoraproject.org|mario.blaetterm...@gmail.co
   ||m
   Flag||fedora-review?

--- Comment #1 from Mario Blättermann mario.blaetterm...@gmail.com 2012-04-01 
11:42:36 EDT ---
$ rpmlint -i -v *perl-Time-OlsonTZ-Download.noarch: I: checking
perl-Time-OlsonTZ-Download.noarch: W: spelling-error %description -l en_US
tzfiles - files
The value of this tag appears to be misspelled. Please double-check.

perl-Time-OlsonTZ-Download.noarch: I: checking-url
http://search.cpan.org/dist/Time-OlsonTZ-Download/ (timeout 10 seconds)
perl-Time-OlsonTZ-Download.src: I: checking
perl-Time-OlsonTZ-Download.src: W: spelling-error %description -l en_US tzfiles
- files
The value of this tag appears to be misspelled. Please double-check.

perl-Time-OlsonTZ-Download.src: I: checking-url
http://search.cpan.org/dist/Time-OlsonTZ-Download/ (timeout 10 seconds)
perl-Time-OlsonTZ-Download.src: I: checking-url
http://www.cpan.org/authors/id/Z/ZE/ZEFRAM/Time-OlsonTZ-Download-0.003.tar.gz
(timeout 10 seconds)
perl-Time-OlsonTZ-Download.spec: I: checking-url
http://www.cpan.org/authors/id/Z/ZE/ZEFRAM/Time-OlsonTZ-Download-0.003.tar.gz
(timeout 10 seconds)
2 packages and 1 specfiles checked; 0 errors, 2 warnings.

Just a few spelling errors which can be ignored.

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

[Bug 808908] New: Review Request: boost-gdb-printers - GDB pretty printers for boost

2012-04-01 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: boost-gdb-printers - GDB pretty printers for boost

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

   Summary: Review Request: boost-gdb-printers - GDB pretty
printers for boost
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dtar...@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://dtardon.fedorapeople.org/boost-gdb-printers.spec
SRPM URL: http://dtardon.fedorapeople.org/boost-gdb-printers-1-1.fc16.src.rpm
Description:

A collection of pretty printers for GDB 7.x for various boost types.
Currently it handles classes from the following boost modules:
* Optional
* Pointer Container
* Smart Pointer
* Unordered

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

[Bug 808518] Review Request: perl-Time-OlsonTZ-Download - Olson timezone database from source

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Mario Blättermann mario.blaetterm...@gmail.com 2012-04-01 
11:49:21 EDT ---
-
key:

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

[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license.
GPL+ or Artistic
[+] MUST: The License field in the package spec file must match the actual
license.
[.] MUST: The file containing the text of the license(s) for the package must
be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
$ md5sum *
eff33875b0c5147df65604ff2b0ba85d  Time-OlsonTZ-Download-0.003.tar.gz
eff33875b0c5147df65604ff2b0ba85d 
Time-OlsonTZ-Download-0.003.tar.gz.packaged

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
- Succesful Koji build available.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[.] MUST: The spec file MUST handle locales properly.
[.] MUST: If a package installs files below %{_datadir}/icons, the icon cache
must be updated.
[.] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
[.] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, ...
[+] MUST: A package must own all directories that it creates.
[+] MUST: A Fedora package must not list a file more than once in %files.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Packages must not provide RPM dependency information when that
information is not global in nature, or are otherwise handled.
[.] MUST: When filtering automatically generated RPM dependency information,
the filtering system implemented by Fedora must be used.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
[+] MUST: Files in %doc must not affect the runtime of the application.
[.] MUST: Header files must be in a -devel package.
[.] MUST: Static libraries must be in a -static package.
[.] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), ...
[.] MUST: devel packages must require the base package using a fully versioned
dependency.
[.] MUST: Packages must NOT contain any .la libtool archives.
[.] MUST: Packages containing GUI applications must include a %{name}.desktop
file
[.] MUST: .desktop files 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.

[.] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream...
[+] SHOULD: Timestamps of files should be preserved.
[+] SHOULD: The reviewer should test that the package builds in mock.
See Koji build above (which uses mock anyway)
[+] SHOULD: The reviewer should test that the package functions as described.
I assume the packager has tested it. Don't know how to test it on my
system.
[+] SHOULD: If scriptlets are used, those scriptlets must be sane.
[.] SHOULD: Usually, subpackages other than devel should require the base
package using a fully versioned dependency.
[.] SHOULD: pkgconfig(.pc) files should be placed in a -devel pkg.
[.] SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin,
/usr/bin, or /usr/sbin ...
[+] SHOULD: Your package should contain man pages for binaries/scripts.



PACKAGE 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 808518] Review Request: perl-Time-OlsonTZ-Download - Olson timezone database from source

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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

[Bug 808258] Review Request: python-sh - Python module to simplify calling shell commands

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

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

--- Comment #3 from Terje Røsten terje...@phys.ntnu.no 2012-04-01 12:11:37 
EDT ---
 Description: A Python module to simplify calling shell commands.
 Also known as PBS, or Python Bash Scripting.
 
 Upstream project is currently called pbs. Due to python-pbs being taken 
 for
 Fedora, upstream has agreed to python-sh

Why not call the rpm package python-bash-scripting then?

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

[Bug 808875] Review Request: perl-Role-Tiny - A nouvelle cusine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mario.blaetterm...@gmail.co
   ||m
 AssignedTo|nob...@fedoraproject.org|mario.blaetterm...@gmail.co
   ||m
   Flag||fedora-review?

--- Comment #1 from Mario Blättermann mario.blaetterm...@gmail.com 2012-04-01 
12:07:31 EDT ---
$ rpmlint -i -v *perl-Role-Tiny.noarch: I: checking
perl-Role-Tiny.noarch: W: spelling-error Summary(en_US) nouvelle - Noelle
The value of this tag appears to be misspelled. Please double-check.

perl-Role-Tiny.noarch: W: spelling-error Summary(en_US) cusine - cuisine,
cosine, cu sine
The value of this tag appears to be misspelled. Please double-check.

perl-Role-Tiny.noarch: I: checking-url http://search.cpan.org/dist/Role-Tiny/
(timeout 10 seconds)
perl-Role-Tiny.src: I: checking
perl-Role-Tiny.src: W: spelling-error Summary(en_US) nouvelle - Noelle
The value of this tag appears to be misspelled. Please double-check.

perl-Role-Tiny.src: W: spelling-error Summary(en_US) cusine - cuisine, cosine,
cu sine
The value of this tag appears to be misspelled. Please double-check.

perl-Role-Tiny.src: I: checking-url http://search.cpan.org/dist/Role-Tiny/
(timeout 10 seconds)
perl-Role-Tiny.src: I: checking-url
http://www.cpan.org/authors/id/M/MS/MSTROUT/Role-Tiny-1.00.tar.gz (timeout
10 seconds)
perl-Role-Tiny.spec: I: checking-url
http://www.cpan.org/authors/id/M/MS/MSTROUT/Role-Tiny-1.00.tar.gz (timeout
10 seconds)
2 packages and 1 specfiles checked; 0 errors, 4 warnings.

Some ignorable spelling errors, as usual.

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

[Bug 808518] Review Request: perl-Time-OlsonTZ-Download - Olson timezone database from source

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Iain Arnell iarn...@gmail.com 2012-04-01 12:10:00 EDT ---
New Package SCM Request
===
Package Name: perl-Time-OlsonTZ-Download
Short Description: Olson timezone database from source
Owners: iarnell
Branches: f15 f16 f17
InitialCC: perl-sig

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

[Bug 808875] Review Request: perl-Role-Tiny - A nouvelle cusine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Mario Blättermann mario.blaetterm...@gmail.com 2012-04-01 
12:13:25 EDT ---
-
key:

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

[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license.
GPL+ or Artistic
[+] MUST: The License field in the package spec file must match the actual
license.
[.] MUST: The file containing the text of the license(s) for the package must
be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
$ md5sum *
d80f53a69e418adea9a91f67af0f7b36  Role-Tiny-1.00.tar.gz
d80f53a69e418adea9a91f67af0f7b36  Role-Tiny-1.00.tar.gz.packaged

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
- Succesful Koji build available.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[.] MUST: The spec file MUST handle locales properly.
[.] MUST: If a package installs files below %{_datadir}/icons, the icon cache
must be updated.
[.] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
[.] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, ...
[+] MUST: A package must own all directories that it creates.
[+] MUST: A Fedora package must not list a file more than once in %files.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Packages must not provide RPM dependency information when that
information is not global in nature, or are otherwise handled.
[.] MUST: When filtering automatically generated RPM dependency information,
the filtering system implemented by Fedora must be used.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
[+] MUST: Files in %doc must not affect the runtime of the application.
[.] MUST: Header files must be in a -devel package.
[.] MUST: Static libraries must be in a -static package.
[.] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), ...
[.] MUST: devel packages must require the base package using a fully versioned
dependency.
[.] MUST: Packages must NOT contain any .la libtool archives.
[.] MUST: Packages containing GUI applications must include a %{name}.desktop
file
[.] MUST: .desktop files 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.

[.] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream...
[+] SHOULD: Timestamps of files should be preserved.
[+] SHOULD: The reviewer should test that the package builds in mock.
See Koji build above (which uses mock anyway)
[+] SHOULD: The reviewer should test that the package functions as described.
I assume the packager has tested it. Don't know how to test it on my
system.
[+] SHOULD: If scriptlets are used, those scriptlets must be sane.
[.] SHOULD: Usually, subpackages other than devel should require the base
package using a fully versioned dependency.
[.] SHOULD: pkgconfig(.pc) files should be placed in a -devel pkg.
[.] SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin,
/usr/bin, or /usr/sbin ...
[+] SHOULD: Your package should contain man pages for binaries/scripts.



PACKAGE 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 808875] Review Request: perl-Role-Tiny - A nouvelle cusine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Iain Arnell iarn...@gmail.com 2012-04-01 12:18:56 EDT ---
New Package SCM Request
===
Package Name: perl-Role-Tiny
Short Description: A nouvelle cusine portion size slice of Moose
Owners: iarnell
Branches: f15 f16 f17
InitialCC: perl-sig

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

[Bug 808875] Review Request: perl-Role-Tiny - A nouvelle cusine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

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

[Bug 808875] Review Request: perl-Role-Tiny - A nouvelle cusine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Iain Arnell iarn...@gmail.com 2012-04-01 12:18:13 EDT ---
(In reply to comment #1)
 
 perl-Role-Tiny.noarch: W: spelling-error Summary(en_US) cusine - cuisine,
 cosine, cu sine
 The value of this tag appears to be misspelled. Please double-check.
[snip] 
 
 Some ignorable spelling errors, as usual.

Yes, but cusine really should be cuisine - will fix for submission. Thanks for
the recent reviews.

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

[Bug 808875] Review Request: perl-Role-Tiny - A nouvelle cusine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Mario Blättermann mario.blaetterm...@gmail.com 2012-04-01 
12:26:23 EDT ---
(In reply to comment #3)
 (In reply to comment #1)
  
  perl-Role-Tiny.noarch: W: spelling-error Summary(en_US) cusine - cuisine,
  cosine, cu sine
  The value of this tag appears to be misspelled. Please double-check.
 [snip] 
  
  Some ignorable spelling errors, as usual.
 
 Yes, but cusine really should be cuisine - will fix for submission. Thanks for
 the recent reviews.

Yes, indeed, But you've got the description from the upstream people, and it
should be fixed upstream, too.

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

[Bug 808875] Review Request: perl-Role-Tiny - A nouvelle cusine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Iain Arnell iarn...@gmail.com 2012-04-01 13:06:48 EDT ---
(In reply to comment #5)
 (In reply to comment #3)
  Yes, but cusine really should be cuisine - will fix for submission. Thanks 
  for
  the recent reviews.
 
 Yes, indeed, But you've got the description from the upstream people, and it
 should be fixed upstream, too.

https://rt.cpan.org/Ticket/Display.html?id=76203

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

[Bug 808875] Review Request: perl-Role-Tiny - A nouvelle cuisine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Iain Arnell iarn...@gmail.com 2012-04-01 13:07:59 EDT ---
New Package SCM Request
===
Package Name: perl-Role-Tiny
Short Description: A nouvelle cuisine portion size slice of Moose
Owners: iarnell
Branches: f15 f16 f17
InitialCC: perl-sig

(fixed spelling of cuisine in summary)

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

[Bug 808875] Review Request: perl-Role-Tiny - A nouvelle cuisine portion size slice of Moose

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |perl-Role-Tiny - A nouvelle |perl-Role-Tiny - A nouvelle
   |cusine portion size slice   |cuisine portion size slice
   |of Moose|of Moose

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

[Bug 808886] Review Request: perl-CPAN-Meta-Requirements - Set of version requirements for a CPAN dist

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

 CC||fedora-perl-devel-list@redh
   ||at.com

--- Comment #2 from Iain Arnell iarn...@gmail.com 2012-04-01 13:12:27 EDT ---
Let's add perl-sig to the CC as this affects perl.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 730314] Review Request: jboss-invocation - JBoss Invocation API

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2012-04-01 13:35:04 EDT ---
jboss-invocation-1.1.1-2.fc17 has been pushed to the Fedora 17 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 760154] Review Request: xcb-util-keysyms - Standard X key constants and keycodes conversion on top of libxcb

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2012-04-01 13:35:42 EDT ---
xcb-util-keysyms-0.3.8-2.fc17 has been pushed to the Fedora 17 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 800526] Review Request: seahorse-sharing - Sharing of PGP public keys via DNS-SD and HKP

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Kalev Lember kalevlem...@gmail.com 2012-04-01 13:59:25 
EDT ---
(In reply to comment #12)
 From what I understand, if I have the old seahorse, and if I install the new
 seahorse-sharing , it will remove the old seahorse, that's all.

Exactly, and this will also happen on package update: seahorse gets removed and
replaced with seahorse-sharing.

Yum has a nice wiki page that describes behaviour with Obsoletes:
http://yum.baseurl.org/wiki/YumPackageUpdates


If we consider a system with seahorse-3.0.0-1.fc15 installed and a repo with
seahorse-3.4.0-1.fc17 and seahorse-sharing-3.4.0-1.fc17 available, there are 3
different ways to do the upgrade:

a)
Installed:
seahorse-3.0.0-1.fc15
After distro update:
seahorse-3.4.0-1.fc17

b)
Installed:
seahorse-3.0.0-1.fc15
After distro update:
seahorse-3.4.0-1.fc17
seahorse-sharing-3.4.0-1.fc17

c)
Installed:
seahorse-3.0.0-1.fc15
After distro update:
seahorse-sharing-3.4.0-1.fc17


Which one of these is desired here? What currently happens with the way
Obsoletes are used in this package is option (c).

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

[Bug 689114] Review Request: php-pear-File-Util - Common file and directory utility functions

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2012-04-01 
13:35:12 EDT ---
php-pear-File-Util-1.0.0-2.fc17 has been pushed to the Fedora 17 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 689112] Review Request: php-pear-File-CSV - Read and write of CSV files

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2012-04-01 
13:35:49 EDT ---
php-pear-File-CSV-1.0.0-2.fc17 has been pushed to the Fedora 17 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 800526] Review Request: seahorse-sharing - Sharing of PGP public keys via DNS-SD and HKP

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #14 from Michael Scherer m...@zarb.org 2012-04-01 15:32:28 EDT ---
If we want A, that's indeed incorrect. 

But since the feature was present in seahorse before, I assume that's B.
According to the page you gave, this should requires adding Obsoletes to
seahorse and seahorse-sharing is correct.

So i think a new bug should be opened against seahorse for that.

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

[Bug 808929] New: Review Request: netbeans-platform - NetBeans Platform

2012-04-01 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: netbeans-platform - NetBeans Platform

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

   Summary: Review Request: netbeans-platform - NetBeans Platform
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: punto...@libero.it
 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://gil.fedorapeople.org/netbeans-platform.spec
SRPM URL: http://gil.fedorapeople.org/netbeans-platform-7.1.1-1.fc16.src.rpm
Description: The NetBeans Platform is a generic framework for Swing
applications.
It provides the services common to almost all large desktop
applications: window management, menus, settings and storage, update
management, file access, etc.

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

[Bug 808930] New: Review Request: netbeans-platform - NetBeans Platform

2012-04-01 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: netbeans-platform - NetBeans Platform

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

   Summary: Review Request: netbeans-platform - NetBeans Platform
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: punto...@libero.it
 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://gil.fedorapeople.org/netbeans-platform.spec
SRPM URL: http://gil.fedorapeople.org/netbeans-platform-7.1.1-1.fc16.src.rpm
Description: The NetBeans Platform is a generic framework for Swing
applications.
It provides the services common to almost all large desktop
applications: window management, menus, settings and storage, update
management, file access, etc.

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

[Bug 803381] Review Request: weld-api - Weld API

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Marek Goldmann mgold...@redhat.com 2012-04-01 16:22:19 
EDT ---
Fixed #1.

Spec URL:
http://goldmann.fedorapeople.org/package_review/weld-api/3/weld-api.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/weld-api/3/weld-api-1.1-3.fc17.src.rpm

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

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

[Bug 781260] Review Request: leechcraft - A Free Open Source Cross-Platform Modular Internet-Client

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||leechcraft-0.5.60-5.fc15
 Resolution||ERRATA
Last Closed||2012-04-01 18:58:03

--- Comment #33 from Fedora Update System upda...@fedoraproject.org 
2012-04-01 18:58:03 EDT ---
leechcraft-0.5.60-5.fc15 has been pushed to the Fedora 15 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 800797] Review Request: ghc-vault - Persistent store for typed values

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

  Status Whiteboard|Ready   |
   Flag||fedora-cvs?

--- Comment #2 from Jens Petersen peter...@redhat.com 2012-04-01 20:03:48 EDT 
---
Thank you for reviewing! :)


New Package SCM Request
===
Package Name: ghc-vault
Short Description: Persistent store for typed values
Owners: petersen
Branches: f17 f16 f15 el6
InitialCC: haskell-sig

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

[Bug 807623] Review Request: lohit-marathi-fonts - Open Type Font for Marathi Language

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Pravin Satpute psatp...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Pravin Satpute psatp...@redhat.com 2012-04-01 23:59:33 
EDT ---
Package Change Request
==
Package Name: lohit-marathi-fonts
New Branches: f17
Owners: pravins
InitialCC: i18n-team

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

[Bug 807625] Review Request: lohit-nepali-fonts - Open Type Font for Nepali Language

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Pravin Satpute psatp...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Pravin Satpute psatp...@redhat.com 2012-04-02 00:00:53 
EDT ---
Package Change Request
==
Package Name: lohit-nepali-fonts
New Branches: f17
Owners: pravins
InitialCC: i18n-team

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

[Bug 805416] Review Request: google-croscore-fonts - The width-compatible fonts for improved on-screen readability

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag pnem...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #15 from Parag pnem...@redhat.com 2012-04-02 01:01:13 EDT ---
Package Change Request
==
Package Name: google-croscore-fonts
New Branches: f15 f16
Owners: pnemade

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

[Bug 808336] Review Request: dia-gnomeDIAicons - Beautiful icon set for dia diagram editor

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Arthur Buliva arthurbul...@fedoraproject.org 2012-04-02 
01:08:39 EDT ---
Change applied.

(In reply to comment #9)
 Arthur, are you already sponsored?

I dont understand. Do you mean if I have a fas account? If so, yes, .fas
arthurbuliva

New Package SCM Request
===
Package Name: dia-gnomeDIAicons
Short Description: Beautiful icon set for dia diagram editor
Owners: arthurbuliva
Branches: f16 f17
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 807113] Review Request: zanata-util - Helper scripts for using Zanata

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Ding-Yi Chen dc...@redhat.com 2012-04-02 01:46:26 EDT ---
Spec URL: http://dchen.fedorapeople.org/files/rpms/zanata-util.spec
SRPM URL:
http://dchen.fedorapeople.org/files/rpms/zanata-util-0.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 807113] Review Request: zanata-util - Helper scripts for using Zanata

2012-04-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Ding-Yi Chen dc...@redhat.com 2012-04-02 01:45:12 EDT ---
(In reply to comment #1)
 1) unable to find source tarball. See
 $ wget
 https://fedorahosted.org/releases/z/a/zanata-util/zanata-util-0.1.0.tar.gz
 --2012-03-27 10:49:13-- 
 https://fedorahosted.org/releases/z/a/zanata-util/zanata-util-0.1.0.tar.gz
 Resolving fedorahosted.org... 66.135.62.191
 Connecting to fedorahosted.org|66.135.62.191|:443... connected.
 HTTP request sent, awaiting response... 404 Not Found
 2012-03-27 10:49:14 ERROR 404: Not Found.

Modified accordingly.

 2) you don't need to write each perl package Requires: explicitly. They will
 get created automatically.

Modified accordingly.

 3) Is this package for RHEL also? I see %clean section and removal of 
 buildroot
 in %install written in spec which is not needed in Fedora now

Yes, I do intent to run it in EL5 and EL6.

 4) Remove the unnecessary comments.
Modified accordingly.

I have been adding some extra functionality and man pages, thus it took a bit
longer.

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