[Bug 552554] Merge Review: perl-Convert-BinHex - Macintosh BinHex extractor library for Perl

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=552554

--- Comment #2 from Paul Howarth <paul@xxxxxxxxxxxx> 2010-01-14 15:37:38 EST ---
(In reply to comment #1)
> MUST items:
> 
> [YES] rpmlint is silent:
> 
> $ rpmlint ./perl-Convert-BinHex.spec
> perl-Convert-BinHex-1.119-11.fc13.noarch.rpm
> perl-Convert-BinHex-1.119-11.fc13.src.rpm
> 2 packages and 1 specfiles checked; 0 errors, 0 warnings.
> 
> [YES] Package meets naming and packaging guidelines.
> [YES] Spec file matches the base package name.
> 
> [NOT SURE] Meets licensing guidelines.
> [NO] License specified in spec file and matches.
>  - CPAN says: Unknown license
>  - included COPYING contains text of GPLv1
>  - spec file says: GPL+ or Artistic

lib/Convert/BinHex.pm (the actual code) says (near the end):

  Copyright (c) 1997 by Eryq.  All rights reserved.  This program is free
  software; you can redistribute it and/or modify it under the same terms as
  Perl itself.

This makes it GPL+ or Artistic

> [YES] Spec file in American English.
> [YES] Spec file is legible.
> 
> [NO] Spec file uses macros consistently.
>  - %install section contains
>    /usr/bin/find %{buildroot} -type f -name .packlist -exec rm -f {} ';'
>    while few lines above macro %{__rm} is used

Fixed

>  - %{__chmod} -R u+w %{buildroot}/*
>    might be replaced with
>    %{_fixperms} %{buildroot}/*

Used %{_fixperms} %{buildroot}

Changes applied in CVS (1.119-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@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]