Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Ville Skyttä <ville.sky...@iki.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nob...@fedoraproject.org    |ville.sky...@iki.fi
               Flag|                            |fedora-review?

--- Comment #1 from Ville Skyttä <ville.sky...@iki.fi> 2011-12-20 16:15:37 EST 
---
* Why include the ucm dir in docs?  I think it's essentially 1.2M of source
code that's not useful when installed.

* Package is uninstallable after being built in Rawhide, it requires
perl(Encode::JIS2K::2022JP3) which isn't provided by the package itself nor
anything else in Fedora.  Just a hunch, maybe the package declaration at top of
lib/Encode/JIS2K/2022JP3.pm should be Encode::JIS2K::2022JP3 instead of
Encode::JIS7::2022JP3.

See the rpmdevtools perl spec template, following it would have prevented these
issues:

* perl-Encode-JIS2K.x86_64: E: zero-length
/usr/lib64/perl5/vendor_perl/auto/Encode/JIS2K/JIS2K.bs

* OPTIMIZE="$RPM_OPT_FLAGS" missing from Makefile.PL line.

* Unnecessary ownership of /usr/lib64/perl5/vendor_perl/auto dir, already owned
by perl-libs.

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

Reply via email to