Fixing the License tag

[Stepan Kasal]

Hello,
   I set up to fix the License: tag.  The tag "GPL" has to be
replaced with something more specific, see the discussion on
fedora-maintainers-list.  In this case "GPL+" is to be used, meaning
"version 1 or any higher."

But I noticed that modules Tie::File and Getopt::Long require GPL
version 2 or higher.
After a quick discussion with Tom "spot" Callaway, I learnt that the
rpm containing these modules has to have
	License:        (GPL+ or Artistic) and (GPLv2+ or Artistic)

Another problem is Digest::MD5, which IMVHO has contradicting
licenses:
It says "under the same terms as Perl itself" in
perl-5.8.8/ext/Digest/MD5/README and, in other words, again in
perl-5.8.8/ext/Digest/MD5/MD5.pm .
But the license inside MD5.pm also says:
- every material mentioning this sw has to contain string1
- every derivative work has to be labeled with string2
It seems that these extra conditions has to be fulfilled unless the
code is distributed as part of Perl.  Confusing.
spot has asked the lawyers about this module.

Then there is perl-core rpm, which does not contain any file, just a
few requires.  I suspect the correct tag would say "no copyrightable
material", but for now, I used the usual perl license tag.

I committed this to devel (patch attached) and I'm going to commit it
to F-7 and F-6, too.  (The policy is "commit to cvs, but do not issue
an update until there is a real fix.")

Have a nice day,
	Stepan
-------------- next part --------------
Index: perl.spec
===================================================================
RCS file: /cvs/extras/rpms/perl/devel/perl.spec,v
retrieving revision 1.126
diff -u -r1.126 perl.spec
--- perl.spec	17 Aug 2007 18:02:13 -0000	1.126
+++ perl.spec	18 Aug 2007 05:33:14 -0000
@@ -20,11 +20,15 @@
 
 Name:           perl
 Version:        %{perl_version}
-Release:        23%{?dist}
+Release:        21%{?dist}
 Epoch:          %{perl_epoch}
 Summary:        The Perl programming language
 Group:          Development/Languages
-License:        Artistic or GPL
+# Modules Tie::File and Getopt::Long are licenced under "GPLv2+ or Artistic,"
+# we have to reflect that in the sub-package containing them.
+# FIXME: Digest::MD5 has a must-advertise-RSA license with an exception,
+# the tag does not reflect that (yet).
+License:        (GPL+ or Artistic) and (GPLv2+ or Artistic)
 Url:            http://www.perl.org/
 Source0:        http://www.cpan.org/authors/id/N/NW/NWCLARK/%{name}-%{perl_version}.tar.bz2
 Source11:       filter-depends.sh
@@ -205,7 +209,7 @@
 
 %package libs
 Summary:        The libraries for the perl runtime
-Group:          Development/Languages
+License:        (GPL+ or Artistic)
 Requires:       perl = %{perl_epoch}:%{perl_version}-%{release}
 
 %description libs
@@ -214,7 +218,7 @@
 
 %package devel
 Summary:        Header files for use in perl development
-Group:          Development/Languages
+License:        (GPL+ or Artistic)
 Requires:       perl = %{perl_epoch}:%{perl_version}-%{release}
 
 %description devel
@@ -224,7 +228,7 @@
 
 %package suidperl
 Summary:        Suidperl, for use with setuid perl scripts
-Group:          Development/Languages
+License:        (GPL+ or Artistic)
 Requires:       perl = %{perl_epoch}:%{perl_version}-%{release}
 
 %description suidperl
@@ -233,7 +237,7 @@
 
 %package CPAN
 Summary:        Query, download and build perl modules from CPAN sites
-Group:          Development/Languages
+License:        (GPL+ or Artistic)
 Epoch:          0
 Version:        1.76_02
 Requires:       perl = %{perl_epoch}:%{perl_version}-%{release}
@@ -244,7 +248,7 @@
 
 %package ExtUtils-Embed
 Summary:        Utilities for embedding Perl in C/C++ applications
-Group:          Development/Languages
+License:        (GPL+ or Artistic)
 Epoch:          0
 Version:        1.26
 Requires:       perl-devel
@@ -255,7 +259,7 @@
 
 %package ExtUtils-MakeMaker
 Summary:        Create a module Makefile
-Group:          Development/Languages
+License:        (GPL+ or Artistic)
 Epoch:          0
 Version:        6.30
 Requires:       perl-devel
@@ -267,7 +271,7 @@
 
 %package Test-Harness
 Summary:        Run Perl standard test scripts with statistics
-Group:          Development/Languages
+License:        (GPL+ or Artistic)
 Epoch:          0
 Version:        2.56
 Requires:       perl-devel
@@ -278,7 +282,7 @@
 
 %package Test-Simple
 Summary:        Basic utilities for writing tests
-Group:          Development/Languages
+License:        (GPL+ or Artistic)
 Epoch:          0
 Version:        0.62
 Requires:       perl-devel
@@ -289,7 +293,8 @@
 
 %package core
 Summary:        Base perl metapackage
-Group:          Development/Languages
+# FIXME: Hey, I'm not sure this rpm contains any copyrightable material!
+License:        (GPL+ or Artistic)
 Epoch:          0
 Version:        %{perl_version}
 Requires:       perl = %{perl_epoch}:%{perl_version}-%{release}
@@ -748,6 +753,10 @@
 # Nothing. Nada. Zilch. Zarro. Uh uh. Nope. Sorry.
 
 %changelog
+* Sat Aug 18 2007 Stepan Kasal <skasal at redhat.com> - 4:5.8.8-24
+- Fix the License: tags.
+- Remove the redundant Group: tags from sub-packages.
+
 * Fri Aug 17 2007 Stepan Kasal <skasal at redhat.com> - 4:5.8.8-23
 - perl rpm requires the corresponding version of perl-libs rpm
 - Resolves: rhbz#240540