[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |CLOSED
 Resolution|NEXTRELEASE |CURRENTRELEASE
   Fixed In Version||1.4.1-1.fc7




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-07 15:57 EST ---
perl-ccom-1.4.1-1.fc7 has been pushed to the Fedora 7 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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-07 16:00 EST ---
perl-ccom-1.4.1-1.fc8 has been pushed to the Fedora 8 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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-03 12:04 EST ---
Jason, I'll have a look to that before importing it to CVS.

New Package CVS Request
===
Package Name: perl-ccom
Short Description: Perl module for context-sensitive phonetic string 
replacement
Owners: robert
Branches: F-7 F-8 EL-4 EL-5
InitialCC: 
Cvsextras Commits: no

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-03 15:21 EST ---
cvs done.

I might have asked before, but will again: Any reason for the cvsextras commits
no? This does mean that other maintainers can't help with this package if you
are away or unavailable. 

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-03 15:28 EST ---
I know, what this means. This package has nearly no upstream activity and I 
made the worse experience, that packagers are more likely to break somebody's 
package rather helping - sorry (add feature A, but A is know to break B or 
similar cases).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-02-03 17:19 EST ---
38201 (perl-ccom): Build on target fedora-4-epel succeeded.
38199 (perl-ccom): Build on target fedora-5-epel succeeded.

Package: perl-ccom-1.4.1-1.fc7 Tag: dist-fc7-updates-candidate Status: complete
Package: perl-ccom-1.4.1-1.fc8 Tag: dist-f8-updates-candidate Status: complete
Package: perl-ccom-1.4.1-1.fc9 Tag: dist-f9 Status: complete

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-03 17:27 EST ---
perl-ccom-1.4.1-1.fc8 has been submitted as an update for Fedora 8

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-03 17:29 EST ---
perl-ccom-1.4.1-1.fc7 has been submitted as an update for Fedora 7

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431044] Review Request: perl-ccom - Perl module for context-sensitive phonetic string replacement

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

Summary: Review Request: perl-ccom - Perl module for context-sensitive phonetic 
string replacement


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-02 12:28 EST ---
Builds OK and rpmlint is silent.

The naming is a bit tough; this isn't a CPAN module and the tarball name
(phonet.tgz) doesn't match the module name, but it does unpack into a directory
which matches the module name.  Odd.  CPAN has no hits for either phonet or
ccom.  I'd tend to agree with you that perl-ccom is a better name than
perl-phonet; this wouldn't be the first time upstream has chosen a tarball name
that has nothing to do with the actual software.

There's a language oddity in the description; I assume it comes from a
mistranslation.  The words tail and tale do not have the same spelling, but
they have the same pronunciation.  I think everything is correct if you just
s/spelling/pronunciation/.

* source files match upstream:
   05f04c7f0d4a4294d3af3df8ee07fdfe397ae1b30ac36d01ec1a381399e0830b
   phonet.tgz
* package meets naming and versioning guidelines as far as I can tell.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
? description needs a 
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
   ccom.so()(64bit)
   perl(ccom) = 1.4
   perl-ccom = 1.4.1-1.fc9
  =
   perl(:MODULE_COMPAT_5.8.8)
   perl(AutoLoader)
   perl(Carp)
   perl(DynaLoader)
   perl(Exporter)
   perl(strict)
   perl(vars)

* %check is present and passes (not much of a test suite, but it runs):
   1..1
   ok 1

* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

APPROVED, just fix up the description.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review