[Bug 482880] Review Request: perl-Sane - Access SANE-compatible scanners with perl

[bugzilla at redhat.com]

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





--- Comment #3 from Bernard Johnson <bjohnson at symetrix.com>  2009-01-29 13:53:02 EDT ---
(In reply to comment #1)
> Perl package source generally includes license test in its perl module source
> text files(.pm here say) only.

The author thinks that the information in the README is sufficient.

> Suggestions:
> 1) SRPM when extracted got SPEC with License as  "Unknown" whereas SPEC file
> link shows correct license as "GPL+ or Artistic"

Someone I think I uploaded a file in progress - this has been fixed.

> 2) Summary should be 
> "Perl extension for the SANE (Scanner Access Now Easy) Project"

fixed

> 3) %files should be
> %files
> %defattr(-,root,root,-)
> %doc Changes README examples
> %{perl_vendorarch}/auto/*
> %{perl_vendorarch}/Sane*
> %{_mandir}/man3/*

fixed

Spec URL: http://fedorapeople.org/~bjohnson/perl-Sane.spec
SRPM URL: http://fedorapeople.org/~bjohnson/perl-Sane-0.02-2.fc10.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.