[Bug 648100] Review Request: ghc-xml - A simple XML library

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


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

--- Comment #3 from Jens Petersen peter...@redhat.com 2011-01-06 03:27:06 EST 
---
Created attachment 472006
  -- https://bugzilla.redhat.com/attachment.cgi?id=472006
ghc-xml.spec-1.patch

Small patch to update to cabal2spec-0.22.4 (still in testing).

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


[Bug 666520] ibus-typing-booster - Indic Typing Booster Table IMEngine for ibus

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


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

--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2011-01-06 04:03:39 EST 
---
First, I have few suggestions for tarball release actually. Please take care of
them in following release.
* Contents of NEWS should be part of README.
* License written in README should say version 3, or (at your option) any
later version.

About packaging,
* License tag should be GPLv3+
* Buildroot is not needed now.
* %clean section also not needed now.
* subpackage ibus-typing-booster-marathi should 
  Requires: ibus-typing-booster and not Requires: ibus-table
* this package have empty %post which should be removed.

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

[Bug 657403] Review Request: spice-gtk - A GTK widget for SPICE clients

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


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

Hans de Goede hdego...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #8 from Hans de Goede hdego...@redhat.com 2011-01-06 04:07:58 EST 
---
Hi,

(In reply to comment #7)
 http://fedorapeople.org/gitweb?p=elmarco/public_git/spice-gtk.git;a=commitdiff;h=59051f0de1772fadf1fd5ad6661d595585148279;hp=16c5fea8350282425bfe4d36ce4d4c5a9e381304
 

Looks good now. Note though that it is custom to bump the spec file's release
field and add a changelog entry whenever you make changes, even during the
package review phase.

 The SRPM is actually made from a git snapshot of version 0.3.20, which has a
 broken python module bug.
 
 I propose we wait until 0.4 release.

Waiting with importing and building till the 0.4 release is fine with me. But
lets move forward with getting you sponsored and creating a git repo and
bugzilla component, etc. for spice-gtk. So that it can get imported and build
as soon as 0.4 is released. If you can give me your fas account name, then I'll
add you to the packagers group and sponsor you. After that you can move forward
with the SCM request:
http://fedoraproject.org/wiki/PackageMaintainers/Join#Add_Package_to_Source_Code_Management_.28SCM.29_system_and_Set_Owner

Regards,

Hans

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


[Bug 664812] Review Request: perl-HTML-Quoted - Extract structure of quoted HTML mail message

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


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

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

   What|Removed |Added

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

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


[Bug 666517] scim-typing-booster - Indic Typing Booster Table IMEngine for SCIM

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


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

--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2011-01-06 05:01:13 EST 
---
First, I have few suggestions for tarball release actually. Please take care of
them in next release.
* Contents of NEWS should be part of README.
* License written in README should say version 3, or (at your option) any
later version.

* rpmlint reported
scim-typing-booster.src:36: W: mixed-use-of-spaces-and-tabs (spaces: line 1,
tab: line 36)
scim-typing-booster-marathi.x86_64: W: no-version-in-last-changelog


* Buildroot is not needed now.
* removal of buildroot in %install is not needed now.
* %clean section is not needed now.
* All install and cp commands should preserve timestamps. Therefore, use cp
-p
* You should avoid generic files in %doc like INSTALL. I see INSTALL is
different here but still see no use of 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 667205] Review Request: radius-engine - A Lua based real-time 2D graphics game engine

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


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

Jan Kaluža jkal...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jkal...@redhat.com
 AssignedTo|nob...@fedoraproject.org|jkal...@redhat.com
   Flag||fedora-review?,
   ||needinfo?(tcall...@redhat.c
   ||om)

--- Comment #2 from Jan Kaluža jkal...@redhat.com 2011-01-06 05:24:53 EST ---
 $ rpmlint radius-engine-*
 radius-engine-devel.x86_64: W: no-documentation
 3 packages and 0 specfiles checked; 0 errors, 1 warnings.

 $ rpmlint radius-engine.spec 
 radius-engine.spec: W: no-cleaning-of-buildroot %clean
 radius-engine.spec: W: no-buildroot-tag
 radius-engine.spec: W: no-%clean-section
 0 packages and 1 specfiles checked; 0 errors, 3 warnings.

These are all okay for F13+. I presume you don't want to have this package in
EPEL, right?

Formal review according to Review Guidelines:
Explanation:
[ok]  the package meets the guideline item
[--]  the guideline item is not relevant for this package
[ERR] ... the package fails to meet the guideline and must be fixed.


[ok] rpmlint must be run on every package.
[ok] named according to the Package Naming Guidelines.
[ok] The spec file name must match the base package %{name}
[ok] License must be Fedora approved; Licensing Guidelines.
[ok] The License field in the package spec file must match the actual license.
[ok] license file (if any) must packaged in %doc.
[ok] spec file in American English.
[ok] spec legible.
[ok] sources must match the upstream source
[ok] must compile and build.
[--] ExcludeArch if it does not.
[ok] complete and sensible BuildRequires
[--] handling of locales
[ok] ldconfig for dynamic libs
[ok] Packages must NOT bundle copies of system libraries.
[--] rules for relocatable packages
[ok] directory ownership
[ok] no duplicate listing in %files
[ok] sane permissions; %defattr(...)
[ok] consistent macro usage
[ok] code or permissable content
[ok] large doc
[ok] header files
[--] static libs
[ok] .so in -devel
[ok] devel requires base package
[--] remove .la files
[--] GUI app must include a %{name}.desktop and use desktop-file-install
[ok] no owning of other packages' files/dirs
[ok] UTF-8 filenames

Formal review according to Packaging Guidelines:

[ok] naming
[ok] version and release
[ok] Licensing
[ok] no inclusion of pre-built binaries or libraries
[ok] spec legibility
[ok] arch support
[ok] filesystem layout
[ok] changelogs
[ok] tags
[ok] BuildRoot
[ok] Requires
[ok] BuildRequires
[ok] summary and description
[ok] encoding
[ok] compiler flags
[ok] debuginfo
[ok] devel packages
[ok] no duplication of system libraries
[ok] no rpath
[ok] config files
[--] initscripts
[--] desktop files
[--] Icon tag in Desktop Files
[ok] macros (inconsistent usage, as already noted)
[--] handling locale files
[ok] timestamps
[ok] parallel make
[--] scriptlets
[--] conditional deps
[--] relocatable packages
[ok] code vs content
[ok] file and dir ownership
[--] users and groups
[--] web apps
[ok] no conflicts
[ok] no kernel modules
[ok] nothing in /srv
[ok] no bundling
[ok] no fonts bundling
[--] epoch
[ok] symlinks
[--] man pages

I've tested also those two games you've on review with this library and it
seems they both work.
To sum it up, it's OK if it's F13+ 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 666520] ibus-typing-booster - Indic Typing Booster Table IMEngine for ibus

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


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

--- Comment #2 from Pravin Satpute psatp...@redhat.com 2011-01-06 05:34:55 
EST ---
SRPM URL:
http://pravins.fedorapeople.org/ibus-typing-booster-1.0.0-2.fc14.src.rpm

SPEC URL: http://pravins.fedorapeople.org/ibus-typing-booster.spec

I have updated contents of NEWS and README file in upstream git repo, it will
come in next release
http://git.fedorahosted.org/git/?p=indic-typing-booster.git

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


[Bug 656408] Review Request: perl-Class-Load - A working (require Class::Name) and more

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


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

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
   Fixed In Version||perl-Class-Load-0.06-3.fc15
 Resolution||NEXTRELEASE
Last Closed||2011-01-06 05:47:38

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


[Bug 666517] scim-typing-booster - Indic Typing Booster Table IMEngine for SCIM

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


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

--- Comment #2 from Pravin Satpute psatp...@redhat.com 2011-01-06 05:59:18 
EST ---
Spec URL: http://pravins.fedorapeople.org/scim-typing-booster.spec
SRPM URL:
http://pravins.fedorapeople.org/scim-typing-booster-1.0.0-2.fc14.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 666889] Review Request: perl-Package-Pkg - Handy package munging utilities

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


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

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com
   Flag||fedora-cvs+

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

rpm -qp --provides
/home/marca/rpmbuild/RPMS/noarch/perl-Package-Pkg-0.0019-1.fc14.noarch.rpm 
perl-Package-Pkg = 0.0019-1.fc14

rpm -qp --requires
/home/marca/rpmbuild/RPMS/noarch/perl-Package-Pkg-0.0019-1.fc14.noarch.rpm 
perl(:MODULE_COMPAT_5.12.2)  
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(CompressedFileNames) = 3.0.4-1

ACCEPT

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

[Bug 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com
   Flag||fedora-cvs+

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

rpm -qp --provides
../RPMS/noarch/perl-Dist-CheckConflicts-0.02-2.fc14.noarch.rpm 
perl-Dist-CheckConflicts = 0.02-2.fc14
rpm -qp --requires
../RPMS/noarch/perl-Dist-CheckConflicts-0.02-2.fc14.noarch.rpm
perl(:MODULE_COMPAT_5.12.2)  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1

ACCEPT

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

[Bug 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #2 from Paul Howarth p...@city-fan.org 2011-01-06 07:51:30 EST ---
New Package SCM Request
===
Package Name: perl-Dist-CheckConflicts
Short Description: Declare version conflicts for your dist
Owners: pghmcfc
Branches: f13 f14 el4 el5 el6
InitialCC: perl-sig

Thanks for the review Marcela; unfortunately you set the fedora-cvs flag rather
than the fedora-review flag, so could you please fix 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 523877] Review Request: CBFlib - crystallography binary format library

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


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

--- Comment #30 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 07:57:46 EST 
---
You can mail le...@lists.fedoraproject.org with details of the license.

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


[Bug 666889] Review Request: perl-Package-Pkg - Handy package munging utilities

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


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

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

   What|Removed |Added

   Flag|fedora-cvs+ |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 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

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

   What|Removed |Added

   Flag||fedora-review+

--- Comment #3 from Marcela Mašláňová mmasl...@redhat.com 2011-01-06 08:03:48 
EST ---
I'm sorry. Fixed.

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

[Bug 665190] Review Request: perl-List-UtilsBy - Higher-order list utility functions

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


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

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com
   Flag||fedora-review+

--- Comment #1 from Marcela Mašláňová mmasl...@redhat.com 2011-01-06 08:30:28 
EST ---
- rpmlint:
rpmlint perl-List-UtilsBy-0.06-1.fc15.src.rpm 
W: spelling-error %description -l en_US behaviour - behavior, behave, behalves
^ that's not American English, but seems to me rude changing description from
British author to something else. Therefore OK
W: spelling-error %description -l en_US Util - Til, U til, Utile
^ nonsense
- package must be named according to Guidelines OK
- spec file name must match the base package %{name} OK
- package must meet the Packaging Guidelines OK
- package must be licensed with Fedora approved license OK
- license field must match actual license OK
- text of the license in its own file must be included in %doc OK
- sources must match the upstream source OK
- package MUST successfully compile and build OK
^ http://koji.fedoraproject.org/koji/taskinfo?taskID=2704543
- architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK
- build dependencies must be listed in BuildRequires OK
- handle locales properly with %find_lang macro OK
- shared library files must call ldconfig in %post(un) OK
- packages must NOT bundle system libraries OK
- package must own all directories that it creates OK
- permissions on files must be set properly OK
- package must consistently use macros OK
- package must contain code, or permissable content OK
- large documentation must go in a -doc OK
- %doc must not affect the runtime of the application OK
- header files must be in a -devel package OK
- static libraries must be in a -static package OK
- library files that end in .so (without suffix) must go in a -devel OK
- devel package usually require base package OK
- packages must NOT contain any .la libtool archives OK
- GUI applications must include a %{name}.desktop file OK
- packages must not own files or directories already owned by other packages OK

rpm -qp --provides ../RPMS/noarch/perl-List-UtilsBy-0.06-1.fc14.noarch.rpm 
perl-List-UtilsBy = 0.06-1.fc14
rpm -qp --requires ../RPMS/noarch/perl-List-UtilsBy-0.06-1.fc14.noarch.rpm 
perl(:MODULE_COMPAT_5.12.2)  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1

ACCEPT

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

[Bug 667205] Review Request: radius-engine - A Lua based real-time 2D graphics game engine

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


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

Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

   Flag|needinfo?(tcall...@redhat.c |
   |om) |

--- Comment #3 from Tom spot Callaway tcall...@redhat.com 2011-01-06 
08:55:00 EST ---
If someone wants this added to EPEL, I will add back the necessary fields, but
I have no plans to include it in EPEL at this time, just F13+.

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


[Bug 640205] Review Request: visualvm - Lightweight profiler that integrates many command-line JDK tools

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


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

--- Comment #47 from jiri vanek jva...@redhat.com 2011-01-06 08:59:17 EST ---
package updated to visualvm 1.3.1, harness 1.1 and profiler 6.9.1
https://admin.fedoraproject.org/updates/visualvm-1.3.1-1.fc14

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


[Bug 664980] Review Request: perl-Test-TCP - Testing TCP program

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


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

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com
   Flag||fedora-review+

--- Comment #1 from Marcela Mašláňová mmasl...@redhat.com 2011-01-06 09:00:54 
EST ---
I used srpm from your webpage...

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

rpm -qp --provides perl-Test-TCP-1.11-1.fc15.noarch.rpm 
perl(Test::TCP) = 1.11
perl-Test-TCP = 1.11-1.fc15
rpm -qp --requires perl-Test-TCP-1.11-1.fc15.noarch.rpm 
perl = 0:5.00800
perl(:MODULE_COMPAT_5.12.2)  
perl(Carp)  
perl(Config)  
perl(IO::Socket::INET)  
perl(POSIX)  
perl(Test::More)  
perl(Test::SharedFork) = 0.12
perl(Time::HiRes)  
perl(base)  
perl(strict)  
perl(warnings)  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(FileDigests) = 4.6.0-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(VersionedDependencies) = 3.0.3-1
rpmlib(PayloadIsXz) = 5.2-1

ACCEPT

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

[Bug 664980] Review Request: perl-Test-TCP - Testing TCP program

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


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

--- Comment #2 from Marcela Mašláňová mmasl...@redhat.com 2011-01-06 09:07:51 
EST ---
On F-14 will be needed update of Test::SharedFork to 0.14 for this package.

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

[Bug 647885] Review Request: apache-rat - Apache Release Audit Tool (RAT)

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


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

--- Comment #15 from Stanislav Ochotnicky socho...@redhat.com 2011-01-06 
09:17:14 EST ---
So...Few more things:
 * rm -rf $RPM_BUILD_ROOT in %install is not needed
 * you shouldn't install BUILD.txt
 * you probably didn't see my note about useless Requires where:

apache-rat-core - apache-rat - jpackage utils
- jpackage-utils

the requires on jpackage-utils in core is not needed because it is pulled in by
main package. Similar other requires like this are in the spec. It's not
exactly a packaging problem, but makes the spec file bigger without any reason
(that I can see). If you have a reason for this, please explain. 

All in all the package is good now, but I'll wait for new version/response
before accepting...

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


[Bug 666889] Review Request: perl-Package-Pkg - Handy package munging utilities

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Iain Arnell iarn...@gmail.com 2011-01-06 09:37:07 EST ---
New Package SCM Request
===
Package Name: perl-Package-Pkg
Short Description: Handy package munging utilities
Owners: iarnell
Branches: f13 f14
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 634621] Review Request: mingw32-qpid-cpp - MinGW Windows AMQP daemons and libraries

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


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

--- Comment #12 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 09:40:29 EST 
---
Git done (by process-git-requests).

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


[Bug 609295] Review Request: python-cement - CLI Application Framework for Python

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


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

--- Comment #15 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 09:39:57 EST 
---
Git done (by process-git-requests).

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


[Bug 666889] Review Request: perl-Package-Pkg - Handy package munging utilities

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


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

--- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 09:44:15 EST 
---
Git done (by process-git-requests).

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


[Bug 665395] Review Request: firewalld - A firewall daemon with D-BUS interface providing a dynamic firewall

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


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

--- Comment #7 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 09:43:58 EST 
---
Git done (by process-git-requests).

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


[Bug 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

--- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 09:44:42 EST 
---
Git done (by process-git-requests).

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


[Bug 646663] Rename review: drupal - drupal6, F15+

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


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

Jon Ciesla l...@jcomserv.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #6 from Jon Ciesla l...@jcomserv.net 2011-01-06 10:02:03 EST ---
Ok.  Thanks!

I think what I'll do is hold off on pushing this until all the module renames
are ready, and then send out an announcement and update the release notes. 
We're not doing any data migration, and all the important paths in etc,
usr/share and /var/lib are changing.

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


[Bug 655954] Review Request: drupal6-token - Tokens are small bits of text that can be placed into larger documents

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


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

Paul W. Frields sticks...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #4 from Paul W. Frields sticks...@gmail.com 2011-01-06 10:17:45 
EST ---
[ O K ] MUST: rpmlint must be run on every package. The output should be posted
in the review.

$ rpmlint rpmbuild/SPECS/drupal6-token.spec
rpmbuild/SRPMS/drupal6-token-1.15-1.fc14.src.rpm
rpmbuild/RPMS/noarch/drupal6-token-1.15-1.fc14.noarch.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

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

[ O K ] MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption.

[ O K ] MUST: The package must meet the Packaging Guidelines.

* To work in EPEL, the definition of %drupaldir in the spec should be
  changed to read:

  %if %{?fedora}
  %define drupaldir %{_datadir}/drupal
  %endif
  %if %{?rhel}
  %define drupaldir %{_datadir}/drupal6
  %endif

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

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

[ O K ] 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.

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

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

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

$ md5sum rpmbuild/SOURCES/drupal6-token-1.15/token-6.x-1.15.tar.gz ; curl -s -o
- http://ftp.drupal.org/files/projects/token-6.x-1.15.tar.gz | md5sum -
65749e2a67c3a8725be611ea75244873 
rpmbuild/SOURCES/drupal6-token-1.15/token-6.x-1.15.tar.gz
65749e2a67c3a8725be611ea75244873  -

[ O K ] MUST: The package MUST successfully compile and build into binary rpms
on at least one primary architecture. 

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

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

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

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

[ O K ] MUST: Packages must NOT bundle copies of system libraries.

[ O K ] MUST: If the package is designed to be relocatable, the packager must
state this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. 

[ O K ] 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. 

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

[ O K ] 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. 

[ O K ] MUST: Each package must consistently use macros. 

[ O K ] MUST: The package must contain code, or permissable content. 

[ O K ] MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity). 

[ O K ] MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program must
run properly if it is not present. 

[ N/A ] MUST: Header files must be in a -devel package. 

[ N/A ] MUST: Static libraries must be 

[Bug 666633] Review Request: lastfmlib - library providing implementation of LastFm Submission Protocol

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


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

Dan Vratil vra...@progdansoft.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from Dan Vratil vra...@progdansoft.com 2011-01-06 10:17:54 
EST ---
New Package SCM Request
===
Package Name: lastfmlib
Short Description: a library providing implementation of LastFm Submission
Protocol v1.2. It allows you to scrobble your tracks on Last.fm.
Owners: progdan
Branches: f14
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 664303] Review Request: drupal6-auto_nodetitle - auto_nodetitle is a small and efficient module that allows hiding of the content title

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


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

Paul W. Frields sticks...@gmail.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|sticks...@gmail.com
   Flag||needinfo?(peter.bo...@gmail
   ||.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 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
10:28:16 EST ---
perl-Dist-CheckConflicts-0.02-2.fc14 has been submitted as an update for Fedora
14.
https://admin.fedoraproject.org/updates/perl-Dist-CheckConflicts-0.02-2.fc14

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


[Bug 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
10:28:29 EST ---
perl-Dist-CheckConflicts-0.02-2.el5 has been submitted as an update for Fedora
EPEL 5.
https://admin.fedoraproject.org/updates/perl-Dist-CheckConflicts-0.02-2.el5

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


[Bug 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

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 667403] Review Request: perl-Dist-CheckConflicts - Declare version conflicts for your dist

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


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

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
10:28:23 EST ---
perl-Dist-CheckConflicts-0.02-2.fc13 has been submitted as an update for Fedora
13.
https://admin.fedoraproject.org/updates/perl-Dist-CheckConflicts-0.02-2.fc13

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


[Bug 664303] Review Request: drupal6-auto_nodetitle - auto_nodetitle is a small and efficient module that allows hiding of the content title

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


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

Paul W. Frields sticks...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||needinfo?(peter.bo...@gmail
   ||.com)

--- Comment #4 from Paul W. Frields sticks...@gmail.com 2011-01-06 10:31:59 
EST ---
$ rpmlint rpmbuild/SPECS/drupal6-auto_nodetitle.spec
rpmbuild/SRPMS/drupal6-auto_nodetitle-1.2-1.fc14.src.rpm
rpmbuild/RPMS/noarch/drupal6-auto_nodetitle-1.2-1.fc14.noarch.rpm 
drupal6-auto_nodetitle.noarch: E: script-without-shebang
/usr/share/drupal6/modules/auto_nodetitle/auto_nodetitle.install
drupal6-auto_nodetitle.noarch: W: spurious-executable-perm
/usr/share/doc/drupal6-auto_nodetitle-1.2/README.txt
2 packages and 1 specfiles checked; 1 errors, 1 warnings.

These two problems can be fixed by setting these files' permissions correctly
to 644 instead of 755.

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


[Bug 666763] Review Request: ax_emergency_listen - monitors APRS emergency packets

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


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

--- Comment #20 from Guido Trentalancia gu...@trentalancia.com 2011-01-06 
10:48:10 EST ---
(In reply to comment #19)
 (In reply to comment #18)
 Yes, sure. I'm also an upstream developer and a Fedora packager. All this work
 is done in my spare time and offered for free too. This is probably true for
 most of the Fedora people. So you're in good company.

Oh yes, good company for sure !

 So, one aspect that prevents your latest release from being added to Fedora
 is that you're not sponsored yet.

I suppose it's just a matter of waiting a sponsor... Let's wait.

The idea is that the utility could be useful to newbies ham and it can do
something quite useful such as picking emergency packets on the air and
reacting upon receiving them. Of course, it can also be used by non-newbies and
by non-hams (as it is not transmitting on the air, then usually no license is
required). In fact, there are similar pieces of code but they are intended to
integrate with GUI applications and pop-up windows. I believe this utility is
original in the sense that it is intended to automate the task of handling
emergency packets without requiring an operator to sit in front of a GUI. Also,
where and when legally possible alerts of any kind (email, text messages,
facsimiles or whatever else) can be forwarded directly to public emergency
services. And everything is done with just a few lines of very simple code.

Thanks again very much for your support Martin !

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


[Bug 666889] Review Request: perl-Package-Pkg - Handy package munging utilities

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


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

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 666889] Review Request: perl-Package-Pkg - Handy package munging utilities

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


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

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
10:55:32 EST ---
perl-Package-Pkg-0.0019-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/perl-Package-Pkg-0.0019-1.fc13

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


[Bug 666889] Review Request: perl-Package-Pkg - Handy package munging utilities

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


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

--- Comment #4 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
10:55:25 EST ---
perl-Package-Pkg-0.0019-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/perl-Package-Pkg-0.0019-1.fc14

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


[Bug 546872] Review Request: qcairo - A 2D graphics library

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


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

Jeffrey Ness jn...@flip-edesign.com changed:

   What|Removed |Added

 CC||jn...@flip-edesign.com

--- Comment #1 from Jeffrey Ness jn...@flip-edesign.com 2011-01-06 11:02:06 
EST ---
Hello,

It appears the URLs above are invalid:

$ curl -I http://www.kanarip.com/custom/SPECS/qcairo.spec
HTTP/1.1 404 Not Found
Date: Thu, 06 Jan 2011 16:01:07 GMT
Server: Apache/2.2.3 (CentOS)
Connection: close
Content-Type: text/html; charset=iso-8859-1
--

$ curl -I http://www.kanarip.com/custom/f12/SRPMS/qcairo-1.8.7.1-3.fc12.src.rpm
HTTP/1.1 404 Not Found
Date: Thu, 06 Jan 2011 16:01:30 GMT
Server: Apache/2.2.3 (CentOS)
Connection: close
Content-Type: text/html; charset=iso-8859-1
--

Please provide working links to both SPEC and SRPM

Jeffrey-

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


[Bug 664515] Review Request: iwl6000g2b-firmware - Firmware for Intel(R) Wireless WiFi Link 6030 Series Adapters

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


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

John W. Linville linvi...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-cvs+ |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 664515] Review Request: iwl6000g2b-firmware - Firmware for Intel(R) Wireless WiFi Link 6030 Series Adapters

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


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

--- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 11:24:30 EST 
---
Git done (by process-git-requests).

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


[Bug 656363] Review Request: perl-Package-Stash-XS - Faster and more correct implementation of the Package::Stash API

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


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

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-01-06 12:01:54 EST ---
perl-Package-Stash-XS-0.17-2.el5 has been pushed to the Fedora EPEL 5 stable
repository.  If problems still persist, please make note of it in this bug
report.

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


[Bug 656363] Review Request: perl-Package-Stash-XS - Faster and more correct implementation of the Package::Stash API

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Package-Stash-XS-0.17-
   ||2.el5
 Resolution||ERRATA
Last Closed||2011-01-06 12:01:59

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


[Bug 667226] Review Request: tepsonic - A simple, fast and lightweight Qt audio player

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


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

--- Comment #2 from Dan Vratil vra...@progdansoft.com 2011-01-06 12:43:50 EST 
---
The lastfmlib package (https://bugzilla.redhat.com/show_bug.cgi?id=33) was
just approved and I'm working on it. liblastfm is an another package.

The Qxt source is needed to provide cross-platform global shortcuts. There is
not whole Qxt source included, only the few files needed to provide the
required functionality.

Could you please provide an explanation about the replacement of
%{_buildir}/%{name}-%{version} in cmake cmd? I'm not sure I understand it.

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 646611] Rename review: drupal-cck - drupal6-cck

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


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

--- Comment #4 from Jon Ciesla l...@jcomserv.net 2011-01-06 13:02:48 EST ---
SRPMS:
http://zanoni.jcomserv.net/fedora/drupal6-cck/drupal6-cck-6.x.2.8-3.fc14.src.rpm
SPEC: http://zanoni.jcomserv.net/fedora/drupal6-cck/drupal6-cck.spec

All addressed, except I moved the cp into setup, because prep doesn't work.

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


[Bug 646610] Rename review: drupal-views - drupal6-views

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


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

--- Comment #5 from Jon Ciesla l...@jcomserv.net 2011-01-06 13:07:35 EST ---
SRPM:
http://zanoni.jcomserv.net/fedora/drupal6-views/drupal6-views-6.x.2.12-2.fc14.src.rpm
SPEC: http://zanoni.jcomserv.net/fedora/drupal6-views/drupal6-views.spec

Updated.

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


[Bug 657403] Review Request: spice-gtk - A GTK widget for SPICE clients

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


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

--- Comment #9 from Marc-Andre Lureau marcandre.lur...@gmail.com 2011-01-06 
12:38:06 EST ---
(In reply to comment #8)

 as soon as 0.4 is released. If you can give me your fas account name, then 
 I'll
 add you to the packagers group and sponsor you. After that you can move 
 forward
 with the SCM request:

My FAS account is elmarco.

cheers

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


[Bug 664515] Review Request: iwl6000g2b-firmware - Firmware for Intel(R) Wireless WiFi Link 6030 Series Adapters

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


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

John W. Linville linvi...@redhat.com changed:

   What|Removed |Added

 Blocks||667781

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


[Bug 667781] Add iwl6000g2b-firmware - Firmware for Intel(R) Wireless WiFi Link 6030 Series Adapters

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


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

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 667781] New: Add iwl6000g2b-firmware - Firmware for Intel(R) Wireless WiFi Link 6030 Series Adapters

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

Summary: Add iwl6000g2b-firmware - Firmware for Intel(R) Wireless WiFi Link 
6030 Series Adapters

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

   Summary: Add iwl6000g2b-firmware - Firmware for Intel(R)
Wireless WiFi Link 6030 Series Adapters
   Product: Fedora
   Version: 14
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: iwl6000g2b-firmware
AssignedTo: linvi...@redhat.com
ReportedBy: linvi...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, linvi...@redhat.com,
lemen...@gmail.com, fedora-package-rev...@redhat.com
Depends on: 664515
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---
  Clone Of: 664515


Add this package to f14...

+++ This bug was initially created as a clone of Bug #664515 +++

Spec URL: http://linville.fedorapeople.org/iwl6000g2b-firmware.spec
SRPM URL:
http://linville.fedorapeople.org/iwl6000g2b-firmware-17.168.5.1-1.fc14.src.rpm
Description:
This package contains the firmware required by the iwlagn driver
for Linux to support the iwl6000g2b hardware.  Usage of the firmware
is subject to the terms and conditions contained inside the provided
LICENSE file. Please read it carefully.

--- Additional comment from lemen...@gmail.com on 2010-12-21 08:50:01 EST ---

I'll review it.

--- Additional comment from lemen...@gmail.com on 2010-12-21 08:54:02 EST ---

REVIEW:

Legend: + = PASSED, - = FAILED, 0 = Not Applicable

+ rpmlint is almost silent (except one false positive)

sulaco ~/rpmbuild/SPECS: rpmlint
~/rpmbuild/RPMS/noarch/iwl6000g2b-firmware-17.168.5.1-1.fc12.noarch.rpm 
iwl6000g2b-firmware.noarch: W: spelling-error %description -l en_US iwlagn -
lagniappe, lasagna, lagging
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
sulaco ~/rpmbuild/SPECS:

+ The package is named according to the  Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format
%{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines.
+ The License field in the package spec file matches the actual license
(redistributable w/o modifications).
+ The file, containing the text of the license(s) for the package, is included
in %doc.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package, match the upstream source, as provided
in the spec URL.

sulaco ~/rpmbuild/SOURCES: sha256sum iwlwifi-6000g2b-ucode-17.168.5.1.tgz*
ae1a0f3f378975b61e99a22c3296c50857f79d52a7cf7303f0a8a6f810fc855c 
iwlwifi-6000g2b-ucode-17.168.5.1.tgz
ae1a0f3f378975b61e99a22c3296c50857f79d52a7cf7303f0a8a6f810fc855c 
iwlwifi-6000g2b-ucode-17.168.5.1.tgz.1
sulaco ~/rpmbuild/SOURCES: 

+ The package successfully compiles and builds into binary rpms on at least one
primary architecture.
+ All build dependencies are listed in BuildRequires.
0 No need to handle locales.
0 No shared library files.
+ The package does NOT bundle copies of system libraries.
+ The package is not designed to be relocatable.
+ The package does not list a file more than once in the spec file's %files
listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissible content.
0 No extremely large documentation files.
+ Anything, the package includes as %doc, does not affect the runtime of the
application.
0 No header files.
0 No static libraries.
0 No pkgconfig(.pc) files.
0 The package doesn't contain library files with a suffix (e.g. libfoo.so.1.1).
0 No devel sub-package.
+ The package does NOT contain any .la libtool archives.
0 Not a GUI application.
+ The package does not own files or directories already owned by other
packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.

APPROVED.

--- Additional comment from linvi...@redhat.com on 2010-12-21 09:26:56 EST ---

New Package SCM Request
===
Package Name: iwl6000g2b-firmware
Short Description: Firmware for Intel(R) Wireless WiFi Link 6030 Series
Adapters
Owners: linville
Branches: f14
InitialCC: linville

--- Additional comment from ti...@math.uh.edu on 2011-01-06 11:24:30 EST ---

Git done (by process-git-requests).

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

[Bug 667781] Add iwl6000g2b-firmware - Firmware for Intel(R) Wireless WiFi Link 6030 Series Adapters

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


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

--- Comment #1 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
13:46:38 EST ---
iwl6000g2b-firmware-17.168.5.1-1.fc14 has been submitted as an update for
Fedora 14.
https://admin.fedoraproject.org/updates/iwl6000g2b-firmware-17.168.5.1-1.fc14

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


[Bug 667226] Review Request: tepsonic - A simple, fast and lightweight Qt audio player

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


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

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at

--- Comment #1 from Volker Fröhlich volke...@gmx.at 2011-01-06 12:26:27 EST 
---
Some initial comments:

The license seems to be GPLv3+:

 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 3
 * of the License, or (at your option) any later version.

Please don't put all the Build Requires in one line, since it's not very clear.

There is no package called lastfmlib-devel in Fedora. The name should be
liblastfm-devel and the package doesn't build for me.

The source also contains a copy of qtx. Please make sure it isn't used. Maybe
delete it.

I noticed, the player also works with MySQL. Does that need any preparation?

Please don't mix $RPMBUILDROOT and %{buildroot} -- simply opt for one.

I guess, a full stop can replace %{_builddir}/%{name}-%{version} from the cmake
call. /usr should be %{_prefix}, but first check, if they are all necessary.

Please use %{_datadir}, to substitute for /usr/share -- also in the files
section. The category list in the desktop file should terminate in a semicolon
(desktop-file-validate).

Probably put the %doc first in the files section (after %defattr).

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

[Bug 646611] Rename review: drupal-cck - drupal6-cck

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


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

Paul W. Frields sticks...@gmail.com changed:

   What|Removed |Added

 CC||sticks...@gmail.com

--- Comment #5 from Paul W. Frields sticks...@gmail.com 2011-01-06 13:59:38 
EST ---
The Requires: drupal6 will make this work for older Fedora branches, where the
drupal package has a virtual Provides: drupal6, as well as the newer Fedora
branches where the package is actually named drupal6.

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


[Bug 667226] Review Request: tepsonic - A simple, fast and lightweight Qt audio player

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


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

--- Comment #4 from Dan Vratil vra...@progdansoft.com 2011-01-06 14:22:29 EST 
---
Thanks for the cmake. 
About the Qxt - the libqxt (or just the one class to be exact) is statically
linked into the executable binary, so no duplicate libqxt.so will be produced.

I can fix the CMakeLists.txt to try look after libqxt.so first and then
eventually use the included sources, but it will appear first in 0.98.2.

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


[Bug 645277] Review Request: puddletag - Feature rich, easy to use tag editor

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


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2011-01-06 14:22:12

--- Comment #9 from Terje Røsten terje...@phys.ntnu.no 2011-01-06 14:22:12 
EST ---
Imported, built and pushed to testing.

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

[Bug 646611] Rename review: drupal-cck - drupal6-cck

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


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

Paul W. Frields sticks...@gmail.com 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 650744] Review Request: spor - Store file modes (permission/ownership) recursively

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


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

--- Comment #16 from Fedora Update System upda...@fedoraproject.org 
2011-01-06 14:27:38 EST ---
spor-1.0-3.fc14 has been pushed to the Fedora 14 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 664815] Review Request: perl-CGI-Emulate-PSGI - PSGI adapter for CGI

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


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

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-01-06 14:23:46 EST ---
perl-CGI-Emulate-PSGI-0.10-1.fc14 has been pushed to the Fedora 14 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 650744] Review Request: spor - Store file modes (permission/ownership) recursively

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


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

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

   What|Removed |Added

   Fixed In Version|spor-1.0-3.fc13 |spor-1.0-3.fc14

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


[Bug 664904] Review Request: perl-Authen-Simple-Passwd - Simple Passwd authentication

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
14:25:47 EST ---
perl-Authen-Simple-Passwd-0.6-1.fc13 has been pushed to the Fedora 13 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 646610] Rename review: drupal-views - drupal6-views

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


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

Paul W. Frields sticks...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||sticks...@gmail.com
 AssignedTo|nob...@fedoraproject.org|sticks...@gmail.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 575497] Review Request: perl-CGI-Emulate-PSGI - PSGI adapter for CGI applications

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


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

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

   What|Removed |Added

   Fixed In Version||perl-CGI-Emulate-PSGI-0.10-
   ||1.fc14
 Resolution|NEXTRELEASE |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 664904] Review Request: perl-Authen-Simple-Passwd - Simple Passwd authentication

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Authen-Simple-Passwd-0
   ||.6-1.fc14
 Resolution||ERRATA
Last Closed||2011-01-06 14:24:32

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


[Bug 575497] Review Request: perl-CGI-Emulate-PSGI - PSGI adapter for CGI applications

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


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

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

   What|Removed |Added

   Fixed In Version|perl-CGI-Emulate-PSGI-0.10- |perl-CGI-Emulate-PSGI-0.10-
   |1.fc14  |1.fc13

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


[Bug 664904] Review Request: perl-Authen-Simple-Passwd - Simple Passwd authentication

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


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

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

   What|Removed |Added

   Fixed In Version|perl-Authen-Simple-Passwd-0 |perl-Authen-Simple-Passwd-0
   |.6-1.fc14   |.6-1.fc13

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


[Bug 664815] Review Request: perl-CGI-Emulate-PSGI - PSGI adapter for CGI

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


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

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

   What|Removed |Added

   Fixed In Version|perl-CGI-Emulate-PSGI-0.10- |perl-CGI-Emulate-PSGI-0.10-
   |1.fc14  |1.fc13

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


[Bug 650744] Review Request: spor - Store file modes (permission/ownership) recursively

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||spor-1.0-3.fc13
 Resolution||ERRATA
Last Closed||2011-01-06 14:27: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 575497] Review Request: perl-CGI-Emulate-PSGI - PSGI adapter for CGI applications

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


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

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
14:24:52 EST ---
perl-CGI-Emulate-PSGI-0.10-1.fc13 has been pushed to the Fedora 13 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 650744] Review Request: spor - Store file modes (permission/ownership) recursively

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


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

--- Comment #15 from Fedora Update System upda...@fedoraproject.org 
2011-01-06 14:27:07 EST ---
spor-1.0-3.fc13 has been pushed to the Fedora 13 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 664815] Review Request: perl-CGI-Emulate-PSGI - PSGI adapter for CGI

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


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

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-01-06 14:24:48 EST ---
perl-CGI-Emulate-PSGI-0.10-1.fc13 has been pushed to the Fedora 13 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 664815] Review Request: perl-CGI-Emulate-PSGI - PSGI adapter for CGI

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


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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-CGI-Emulate-PSGI-0.10-
   ||1.fc14
 Resolution||ERRATA
Last Closed||2011-01-06 14:23:57

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


[Bug 664904] Review Request: perl-Authen-Simple-Passwd - Simple Passwd authentication

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


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

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
14:24:27 EST ---
perl-Authen-Simple-Passwd-0.6-1.fc14 has been pushed to the Fedora 14 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 666889] Review Request: perl-Package-Pkg - Handy package munging utilities

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
14:24:09 EST ---
perl-Package-Pkg-0.0019-1.fc13 has been pushed to the Fedora 13 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update perl-Package-Pkg'.  You can
provide feedback for this update here:
https://admin.fedoraproject.org/updates/perl-Package-Pkg-0.0019-1.fc13

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


[Bug 661832] Review Request: kdevelop-pg-qt - A parser generator

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #16 from Fedora Update System upda...@fedoraproject.org 
2011-01-06 14:23:35 EST ---
kdevelop-php-1.1.1-3.fc14, kdevelop-pg-qt-0.9.0-3.fc14 has been pushed to the
Fedora 14 testing repository.  If problems still persist, please make note of
it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update kdevelop-php kdevelop-pg-qt'. 
You can provide feedback for this update here:
https://admin.fedoraproject.org/updates/kdevelop-php-1.1.1-3.fc14,kdevelop-pg-qt-0.9.0-3.fc14

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


[Bug 661833] Review Request: kdevelop-php - Php language plugin for KDevelop

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


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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-01-06 14:23:41 EST ---
kdevelop-php-1.1.1-3.fc14, kdevelop-pg-qt-0.9.0-3.fc14 has been pushed to the
Fedora 14 testing repository.  If problems still persist, please make note of
it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update kdevelop-php kdevelop-pg-qt'. 
You can provide feedback for this update here:
https://admin.fedoraproject.org/updates/kdevelop-php-1.1.1-3.fc14,kdevelop-pg-qt-0.9.0-3.fc14

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


[Bug 638647] Review Request: mom - Dynamically manage system resources on virtualization hosts

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


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

--- Comment #16 from BJ Dierkes wdier...@rackspace.com 2011-01-06 14:30:14 
EST ---
It is too early to submit for a SCM module.  You must first find a sponsor (as
this is your first package) and the sponsor will perform a formal review before
you can make an SCM request.

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


[Bug 638647] Review Request: mom - Dynamically manage system resources on virtualization hosts

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


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

BJ Dierkes wdier...@rackspace.com changed:

   What|Removed |Added

 CC||wdier...@rackspace.com
   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 667790] Review Request: python-audioprocessing - Python audio processing suite

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


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

Stefan Parviainen pa...@iki.fi 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 667790] New: Review Request: python-audioprocessing - Python audio processing suite

2011-01-06 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: python-audioprocessing - Python audio processing suite

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

   Summary: Review Request: python-audioprocessing - Python audio
processing suite
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pa...@iki.fi
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://pafcu.fi/python-audioprocessing.spec
SRPM URL: http://pafcu.fi/python-audioprocessing-0.0.5-1.fc14.src.rpm
Description: The Python audio processing suite contains Python modules and
command-line tools with a variety of convenience functions to process and
visualize audio signals, including audio fingerprinting and plotting the
spectrum of a song.

This is my first package and I need a sponsor.

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


[Bug 579925] Review Request: tcl-tclreadline - GNU Readline extension for Tcl/Tk

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


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

--- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 14:41:53 EST 
---
I guess the primary question is why you would want to push unmaintained
software into the distribution when by your own admission you aren't
sufficiently familiar with the software to maintain it if problems arise.  For
example, what happens if Fedora updates to a readline version which is no
longer ABI compatible?

As to the package itself, I think it looks fine assuming you intend to push
this to EL4 or EL5; otherwise you can drop several bits.

I'm not sure why you apply memuse.patch; did our tcl become threaded at some
point?  (I guess it doesn't really hurt anything, but generally you patch to
fix something that's actually broken.)

prompt2.patch seems too actually change the API of the library.  Are you really
sure you didn't intend to fork this software?  Debian policies may permit this
kind of thing but it's really a bad idea.

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


[Bug 638647] Review Request: mom - Dynamically manage system resources on virtualization hosts

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


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

--- Comment #17 from Adam Litke a...@us.ibm.com 2011-01-06 14:47:19 EST ---
BJ, I have already been sponsored by Richard Jones (rjones).  Is there a step
that he omitted in approving this package?

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


[Bug 560787] Review Request: python-mtTkinter - A thread-safe version of Tkinter

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


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

Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

  Status Whiteboard||NotReady

--- Comment #6 from Jason Tibbitts ti...@math.uh.edu 2011-01-06 15:06:56 EST 
---
Unfortunately the srpm link above is invalid.  The directory is browsable but I
do not see anything applicable.

Please clear the whiteboard if providing a package for 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 638647] Review Request: mom - Dynamically manage system resources on virtualization hosts

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


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

--- Comment #18 from BJ Dierkes wdier...@rackspace.com 2011-01-06 15:14:08 
EST ---
I'm sorry, but I do not see any reference to 'rjones' in this tracker.  If he
agreed to be your sponsor, he needs to formally take this review (set the
fedora-review flag to '?'), and perform the review until it passes.  At that
time he would set the fedora-review flag to '+' meaning it has passed review.

Please ask Richard Jones to visit this ticket, and he can take over from there.

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


[Bug 638647] Review Request: mom - Dynamically manage system resources on virtualization hosts

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


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

Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

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

--- Comment #19 from Richard W.M. Jones rjo...@redhat.com 2011-01-06 15:18:01 
EST ---
I'm sponsoring Adam, but I wasn't aware I had to review the package
as well.

However this package is still missing a review, and consequently
there is no fedora-review+ flag.  Xavier can you complete this
review?

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


[Bug 638647] Review Request: mom - Dynamically manage system resources on virtualization hosts

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


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

--- Comment #20 from BJ Dierkes wdier...@rackspace.com 2011-01-06 15:47:02 
EST ---
Ah, oops.  My mistake, I was under the impression that the sponsor did the
review as well ... I guess I just assumed that because my sponsor did.  I
apologize for the bad information.

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


[Bug 666954] Review Request: cherrytree - Hierarchical note taking application

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


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

--- Comment #10 from Christoph Wickert cwick...@fedoraproject.org 2011-01-06 
16:55:49 EST ---
(In reply to comment #7)
 * TODO: python2-devel is now preferred to python-devel.
 https://fedoraproject.org/wiki/Packaging:Python#BuildRequires

AFAIK python-devel is a virtual provides that always defaults to the perferred
version of python. IMHO we should not hardcode a specific version.

 * I hope you can convince upstream to rid of gtk requirement from setup.py. It
 should be an easy job.
 
 
 This package will be approved when the two last issues fixed.

Are you blocking this review for an issue with the upstream code that can be
easy worked around? I have contacted upstream and sent him the patch, but if he
doesn't accept it I will not use it because of our commitment to upstream.
Nevertheless this should not block a review. It's a package review and not a
code review and as long as there are no security issues or severe bugs, this
should not be considered a blocker I think.

(In reply to comment #9)
 This package uses glade mechanism, you'd better add pygtk2-libglade to
 requirement.

You don't need to have pygtk2-libglade installed to run cherrytree.

(In reply to comment #8)
 But I would suggest to not install into %{python_sitelib}, because upstream
 (and so othere distributions don't do that too, but your choice ;-)

Fine with me.

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


[Bug 560787] Review Request: python-mtTkinter - A thread-safe version of Tkinter

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


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

Paulo Roma Cavalcanti pro...@gmail.com changed:

   What|Removed |Added

  Status Whiteboard|NotReady|

--- Comment #7 from Paulo Roma Cavalcanti pro...@gmail.com 2011-01-06 
17:35:16 EST ---

Sorry ...

http://roma.fedorapeople.org/srpms/python-mtTkinter-0.3-2.fc12.src.rpm

http://roma.fedorapeople.org/specs/python-mtTkinter.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 637212] Review Request: yubikey-ksm - The YubiKey Key Storage Module

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


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

--- Comment #2 from Dennis Gilmore den...@ausil.us 2011-01-06 21:41:42 EST ---
Spec URL: http://ausil.us/packages/yubikey-ksm.spec
SRPM URL: http://ausil.us/packages/yubikey-ksm-1.5-3.fc15.src.rpm

added requires on php-pdo
moved the htaccess file to docs

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


[Bug 637213] Review Request: yubikey-val - The YubiKey Validation Server

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


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

--- Comment #2 from Dennis Gilmore den...@ausil.us 2011-01-06 21:44:05 EST ---
Spec URL: http://ausil.us/packages/yubikey-val.spec
SRPM URL: http://ausil.us/packages/yubikey-val-2.7-2.fc15.src.rpm

* Thu Jan 06 2011 Dennis Gilmore den...@ausil.us - 2.7-2
- Requires php-pdo
- remove -template from config file name

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


[Bug 664810] Review Request: perl-Regexp-IPv6 - Regular expression for IPv6 addresses

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


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

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
22:22:15 EST ---
perl-Regexp-IPv6-0.03-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/perl-Regexp-IPv6-0.03-1.fc13

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


[Bug 664810] Review Request: perl-Regexp-IPv6 - Regular expression for IPv6 addresses

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


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

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 664810] Review Request: perl-Regexp-IPv6 - Regular expression for IPv6 addresses

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


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

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-01-06 
22:22:22 EST ---
perl-Regexp-IPv6-0.03-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/perl-Regexp-IPv6-0.03-1.fc14

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


[Bug 664980] Review Request: perl-Test-TCP - Testing TCP program

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


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

--- Comment #3 from Ralf Corsepius rc040...@freenet.de 2011-01-06 22:51:10 
EST ---
(In reply to comment #1)
 ACCEPT

Thanks.

(In reply to comment #2)
 On F-14 will be needed update of Test::SharedFork to 0.14 for this package.

Yep. Another de-facto orphaned cweyl perl module. I've just added myself as
(co-) maintainer and plan to look into upgrading it on Fedora  15.

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


[Bug 664980] Review Request: perl-Test-TCP - Testing TCP program

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


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

--- Comment #4 from Ralf Corsepius rc040...@freenet.de 2011-01-06 22:57:16 
EST ---
New Package SCM Request
===
Package Name: perl-Test-TCP
Short Description: Testing TCP program
Owners: corsepiu
Branches: f13 f14
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 664980] Review Request: perl-Test-TCP - Testing TCP program

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


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

Ralf Corsepius rc040...@freenet.de changed:

   What|Removed |Added

 Depends on||667873

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


[Bug 664204] Review Request: perl-BackPAN-Index - Interface to the BackPAN index

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


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

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

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|ppi...@redhat.com

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


[Bug 664980] Review Request: perl-Test-TCP - Testing TCP program

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


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

Ralf Corsepius rc040...@freenet.de 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 667879] New: Review Request: perl-Getopt-Usaginator - Conjure up a usage function for your applications

2011-01-06 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-Getopt-Usaginator - Conjure up a usage function 
for your applications

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

   Summary: Review Request: perl-Getopt-Usaginator - Conjure up a
usage function for your applications
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/Getopt-Usaginator/
OS/Version: Unspecified
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.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/~iarnell/review/perl-Getopt-Usaginator.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Getopt-Usaginator-0.0012-1.fc15.src.rpm

Description:
Getopt::Usaginator is a tool for creating a handy usage subroutine for
commandline applications. It does not do any option parsing, but is best paired
with Getopt::Long or any of the other myriad of option parsers.

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

*rt-0.10_01

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


  1   2   >