Package: perl-base
Version: 5.10.1-11
Severity: serious
Tags: patch, pending

Debian Perl Policy, section 2.3 (Module Path), states that site
directories of binary compatible earlier Perl versions should be on @INC:

site (old)

    site directories (as above) for modules installed with previously
    released perl packages for which the current package is binary
    compatible are included if present.

In the case of the current version, 5.10.1, we should append
/usr/local/share/perl/5.10.0 and /usr/local/lib/perl/5.10.0 to @INC if
they exist.

The code that's supposed to do this is the #ifdef PERL_INC_VERSION_LIST
block in

 
http://patch-tracker.debian.org/patch/series/view/perl/5.10.1-11/debian/mod_paths.diff

but unfortunately it doesn't currently work as intended, apparently
because the 5.10 series added an additional Configure variable,
d_inc_version_list, that we don't currently override from config.over,
so PERL_INC_VERSION_LIST never gets defined.

This would apply to the lenny version too, but there are no earlier
binary compatible versions to include there as lenny shipped with 5.10.0.

Patch attached. I'll look at including a couple of other fixes in 5.10.1-12
so it may take a few days until the upload.
-- 
Niko Tyni   nt...@debian.org
>From 34e9b0d3bbaf426500788a93aa6fbc2835a83f99 Mon Sep 17 00:00:00 2001
From: Niko Tyni <nt...@debian.org>
Date: Mon, 22 Mar 2010 23:47:08 +0200
Subject: [PATCH] Properly include the 5.10.0 site directories on @INC as per Perl policy.

---
 debian/changelog   |    1 +
 debian/config.over |    3 +++
 2 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 61b6f34..7b9bc55 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -3,6 +3,7 @@ perl (5.10.1-12) UNRELEASED; urgency=low
   * Fix the location of an Archive::Tar test file.
   * Update perl-base conflict versions on libscalar-list-utils-perl
     and libxsloader-perl.
+  * Properly include the 5.10.0 site directories on @INC as per Perl policy.
 
  -- Niko Tyni <nt...@debian.org>  Tue, 16 Mar 2010 00:57:27 +0200
 
diff --git a/debian/config.over b/debian/config.over
index 91abbdf..c8cccc4 100644
--- a/debian/config.over
+++ b/debian/config.over
@@ -48,6 +48,9 @@ do
     inc_version_list="$ver${inc_version_list:+ }$inc_version_list"
     inc_version_list_init="\"$ver\",$inc_version_list_init"
 done <debian/released-versions
+if [ -n "$inc_version_list" ]; then
+    d_inc_version_list=define
+fi
 
 # set generic email addresses, host/domain names
 cf_by='Debian Project'
-- 
1.7.0.2

Reply via email to