cpan/CPAN/lib/CPAN/FirstTime.pm | 4 2 + 2 - 0 !
1 file changed, 2 insertions(+), 2 deletions(-)

 provide a sensible installdirs default for modules installed from
 CPAN.

Some modules which are included in core set INSTALLDIRS => 'perl'
explicitly in Makefile.PL or Build.PL.  This makes sense for the normal @INC
ordering, but not ours.

cpan/DB_File/version.c | 2 2 + 0 - 0 !
1 file changed, 2 insertions(+)

 remove overly restrictive db_file version check.

Bug-Debian: http://bugs.debian.org/340047

Package dependencies ensure the correct library is linked at run-time.

pod/perl.pod | 12 10 + 2 - 0 !
1 file changed, 10 insertions(+), 2 deletions(-)

 replace generic man(1) instructions with debian-specific information.

Indicate that the user needs to install the perl-doc package.

cpan/Encode/bin/enc2xs | 8 4 + 4 - 0 !
t/porting/customized.t | 3 3 + 0 - 0 !
2 files changed, 7 insertions(+), 4 deletions(-)

 tweak enc2xs to follow symlinks and ignore missing @inc directories.

Bug-Debian: http://bugs.debian.org/290336

- ignore missing directories,
- follow symlinks (/usr/share/perl/5.8 -> 5.8.4).
- filter "." out when running "enc2xs -C", it's unnecessary and causes
  issues with follow => 1 (see #603686 and [rt.cpan.org #64585])

ext/Errno/Errno_pm.PL | 16 0 + 16 - 0 !
1 file changed, 16 deletions(-)

 remove errno version check due to upgrade problems with long-running
 processes.

Bug-Debian: http://bugs.debian.org/343351

Remove version check which can cause problems for long running
processes embedding perl when upgrading to a newer version,

lib/ExtUtils/Embed.pm | 3 3 + 0 - 0 !
1 file changed, 3 insertions(+)

 note that libperl-dev package is required for embedded linking

Bug-Debian: http://bugs.debian.org/186778

cpan/ExtUtils-Install/lib/ExtUtils/Install.pm | 18 9 + 9 - 0 !
cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm | 18 9 + 9 - 0 !
2 files changed, 18 insertions(+), 18 deletions(-)

 respect umask during installation

This is needed to satisfy Debian policy regarding group-writable
site directories.

cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm | 6 3 + 3 - 0 !
1 file changed, 3 insertions(+), 3 deletions(-)

 set umask approproately for site install directories

Policy requires group writable site directories

cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 eu:mm: set location of libperl.a under /usr/lib

The Debian packaging moves libperl.a a couple of levels up from the
CORE directory to match other static libraries.

cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm | 35 3 + 32 - 0 !
1 file changed, 3 insertions(+), 32 deletions(-)

 don't install .packlist or perllocal.pod for perl or vendor

Makefile.SH | 7 2 + 5 - 0 !
1 file changed, 2 insertions(+), 5 deletions(-)

 postpone ld_library_path evaluation to the binary targets.

Modify the setting of LD_LIBRARY_PATH to append pre-existing values at the
time the rule is evaluated rather than when the Makefile is created.

This is required when building packages with dpkg-buildpackage and fakeroot,
since fakeroot (which now sets LD_LIBRARY_PATH) is not used for the "build"
rule where the Makefile is created, but is for the clean/binary* targets.

cpan/ExtUtils-MakeMaker/bin/instmodsh | 4 3 + 1 - 0 !
1 file changed, 3 insertions(+), 1 deletion(-)

 debian policy doesn't install .packlist files for core or vendor.

cpan/ExtUtils-MakeMaker/lib/ExtUtils/Liblist/Kid.pm | 3 3 + 0 - 0 !
1 file changed, 3 insertions(+)

 remove standard libs from ld_run_path as per debian policy.

cpan/libnet/lib/Net/Config.pm | 7 3 + 4 - 0 !
1 file changed, 3 insertions(+), 4 deletions(-)

 set location of libnet.cfg to /etc/perl/net as /usr may not be
 writable.

perl.c | 58 58 + 0 - 0 !
1 file changed, 58 insertions(+)

 tweak @inc ordering for debian

Our order is:

    etc (for config files)
    site (5.8.1)
    vendor (all)
    core (5.8.1)
    site (version-indep)
    site (pre-5.8.1)

The rationale being that an admin (via site), or module packager
(vendor) can chose to shadow core modules when there is a newer
version than is included in core.

Configure | 5 2 + 3 - 0 !
1 file changed, 2 insertions(+), 3 deletions(-)

 prune the list of libraries wanted to what we actually need.

Bug-Debian: http://bugs.debian.org/128355

We want to keep the dependencies on perl-base as small as possible,
and some of the original list may be present on buildds (see Bug#128355).

cpan/libnet/lib/Net/SMTP.pm | 1 1 + 0 - 0 !
1 file changed, 1 insertion(+)

 document the net::smtp 'port' option

Bug-Debian: http://bugs.debian.org/100195
Bug: http://rt.cpan.org/Public/Bug/Display.html?id=36038

utils/perlivp.PL | 1 1 + 0 - 0 !
1 file changed, 1 insertion(+)

 make perlivp skip include directories in /usr/local

Bug-Debian: http://bugs.debian.org/510895

On Sat, Jan 10, 2009 at 12:37:18AM +1100, Brendan O'Dea wrote:
> On Wed, Jan 7, 2009 at 12:21 AM, Niko Tyni <ntyni@debian.org> wrote:

> > We could create the directories in a postinst script, but I'm not sure
> > I see the point. They will be created automatically when installing
> > CPAN modules.
>
> The directories are intentionally not created, as this way they are
> excluded from the search path at start-up, saving a bunch of wasted
> stats at use/require time in the common case that the user has not
> installed any local packages.  As Niko points out, they will be
> created as required.

Signed-off-by: Niko Tyni <ntyni@debian.org>

lib/deprecate.pm | 15 14 + 1 - 0 !
1 file changed, 14 insertions(+), 1 deletion(-)

 point users to debian packages of deprecated core modules

Bug-Debian: http://bugs.debian.org/747628

Several modules are being deprecated with perl 5.20.
To get a clean transition, perl/perl-modules is going to recommend the
separate Debian packages of these for one release cycle so that they will be
pulled in by default on upgrades. This is taking place for perl 5.20/jessie.

However, on systems configured to ignore recommendations the deprecation
warnings will still be useful, so modify them slightly to point to the
separate packages instead.

locale.c | 5 4 + 1 - 0 !
pod/perllocale.pod | 8 8 + 0 - 0 !
2 files changed, 12 insertions(+), 1 deletion(-)

 squelch locale warnings in debian package maintainer scripts

Bug-Debian: http://bugs.debian.org/508764

The system locales are rather frequently out of sync with the C library
during package upgrades, causing a huge amount of useless Perl locale
warnings. Squelch them when running package maintainer scripts, detected
by the DPKG_RUNNING_VERSION environment variable.

Any real locale problem will show up after the system upgrade too, and
the warning will be triggered normally again at that point.

t/test.pl | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 skip tests specific to the upstream git repository

patchlevel.h | 3 3 + 0 - 0 !
1 file changed, 3 insertions(+)

 list packaged patches in patchlevel.h

t/op/threads.t | 4 4 + 0 - 0 !
1 file changed, 4 insertions(+)

 skip a crashing test case in t/op/threads.t on gnu/kfreebsd

Bug: http://rt.perl.org/rt3/Ticket/Display.html?id=96272
Bug-Debian: http://bugs.debian.org/628493

The crash is not a regression in 5.14, it just gets triggered there by
a new unrelated test case.

Skip the test until the culprit is found.

cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker.pm | 4 4 + 0 - 0 !
1 file changed, 4 insertions(+)

 document that ccflags should include $config{ccflags}

Bug: https://rt.cpan.org/Public/Bug/Display.html?id=68613
Bug-Debian: http://bugs.debian.org/628522

Compiling XS extensions without $Config{ccflags} can break the
binary interface on some platforms.

cpan/CPAN/lib/CPAN/Distribution.pm | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 configure cpan::distribution with correct name of html2text

Bug-Debian: http://bugs.debian.org/640479

lib/perl5db.pl | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 invoke x-terminal-emulator rather than xterm in perl5db.pl

In Debian systems, xterm might not exist or might not be the preferred
terminal emulator. Use x-terminal-emulator instead

Bug-Debian: http://bugs.debian.org/668490

cpan/CPAN/lib/CPAN/FirstTime.pm | 31 27 + 4 - 0 !
1 file changed, 27 insertions(+), 4 deletions(-)

 fix cpan::firsttime defaults with nonexisting site dirs if a parent
 is writable

The site directories do not exist on a typical Debian system.  The build
systems will create them when necessary, so there's no need for a prompt
suggesting local::lib if the first existing parent directory is writable.

Also, writability of the core directories is not interesting as we
explicitly tell CPAN not to touch those with INSTALLDIRS=site.

Bug-Debian: http://bugs.debian.org/688842

cpan/Memoize/Memoize/Storable.pm | 2 1 + 1 - 0 !
cpan/Memoize/t/tie_storable.t | 24 20 + 4 - 0 !
2 files changed, 21 insertions(+), 5 deletions(-)

 memoize::storable: respect 'nstore' option not respected
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Memoize(3perl) says:

                tie my %cache => 'Memoize::Storable', $filename, 'nstore';
                memoize 'function', SCALAR_CACHE => [HASH => \%cache];

        Include the nstore option to have the "Storable" database
        written in network order.  (See Storable for more details
        about this.)

In fact the "nstore" option does no such thing.  Option parsing looks
like this:

        @options{@_} = ();

$self->{OPTIONS}{'nstore'} is accordingly set to undef.  Later
Memoize::Storable checks if the option is true, and since undef is
not true, the "else" branch is always taken.

        if ($self->{OPTIONS}{'nstore'}) {
                Storable::nstore($self->{H}, $self->{FILENAME});
        } else {
                Storable::store($self->{H}, $self->{FILENAME});
        }

Correcting the condition to (exists $self->{OPTIONS}{'nstore'}) fixes
it.

Noticed because git-svn, which uses the 'nstore' option for its
on-disk caches, was producing

        Byte order is not compatible at ../../lib/Storable.pm

regen/lib_cleanup.pl | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 skip a regeneration check in unrelated git repositories

If the test is run in a git repository without lib/.gitignore,
for instance because the repository only imported the Perl tarball,
the regeneration check is broken because lib/.gitignore is missing.

cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 pass ld settings through to subdirectories

[rt.cpan.org #28632] fixed most issues with passing LD down to
subdirectory Makefile.PL files. However, there are some distributions
(including Wx, Par::Packer, and Verilog-Perl) that explicitly set
LD. Those still can't be overridden from the command line. Adding LD to
the PASTHRU list fixes this.

Bug-Debian: https://bugs.debian.org/758471

cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Any.pm | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 make eu::makemaker honour mannext settings in generated manpage
 headers

This was inherited from early changes by Brendan O'Dea, previously

cpan/Devel-PPPort/PPPort_xs.PL | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 sort the list of xs code files when generating realppport.xs

all_files_in_dir() uses readdir() ordering to make the list of
input files. This can vary between build systems, breaking build
reproducibility.

Bug: https://github.com/mhx/Devel-PPPort/pull/28
Bug-Debian: https://bugs.debian.org/801523

cpan/Encode/Unicode/Unicode.pm | 4 3 + 1 - 0 !
1 file changed, 3 insertions(+), 1 deletion(-)

 document debian backport of encode::unicode fix

Bug-Debian: https://bugs.debian.org/798727

t/op/stat.t | 1 1 + 0 - 0 !
1 file changed, 1 insertion(+)

 work around debian bug#796798

kFreeBSD 10 (possibly only with softupdates enabled) may defer
calculating the mtime for more than 2 seconds.  Work around this
with a stat() call to calculate the mtime immediately.

(Modified to only stat() on kfreebsd by Niko Tyni)

Bug-Debian: https://bugs.debian.org/796798

cpan/autodie/lib/Fatal.pm | 7 6 + 1 - 0 !
cpan/autodie/t/no-all.t | 22 22 + 0 - 0 !
cpan/autodie/t/no-default.t | 23 23 + 0 - 0 !
3 files changed, 51 insertions(+), 1 deletion(-)

 fix a scoping issue with "no autodie" and the "system" sub

Don't queue nonexisting subs for reinstalling later when
exiting the 'no autodie' scope.

FIXME: if the original sub 'can be undef for "CORE::" subs', does this
break for those? Is that the case when $symbol =~ /::/, as guarded for
on L566?

Bug: https://github.com/pjf/autodie/issues/69
Bug-Debian: https://bugs.debian.org/798096

Configure | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 fix the configure escape with usecrosscompile but no targethost

Commit 97076f2 added a graceful exit to Configure when targethost is
not defined but usecrosscompile is. However, this is not reached because
there is a similar check a bit earlier that makes Configure bail out.

Make the earlier check warn instead of croaking.

The use case for this combination is supplying an external config.sh
suitable for the target platform, avoiding the need for configuration
probes.

Bug: https://rt.perl.org/Ticket/Display.html?id=127234

cpan/Memoize/Memoize.pm | 4 2 + 2 - 0 !
1 file changed, 2 insertions(+), 2 deletions(-)

 fix pod errors in memoize

Bug: https://rt.cpan.org/Public/Bug/Display.html?id=89441

cpan/Test-Simple/lib/ok.pm | 4 4 + 0 - 0 !
1 file changed, 4 insertions(+)

 added encoding for pod.

t/op/stat.t | 1 1 + 0 - 0 !
1 file changed, 1 insertion(+)

 fix t/op/stat.t failures on hurd

We already do this on GNU/kFreeBSD, and GNU/Hurd seems to need same
treatment.

Bug-Debian: https://bugs.debian.org/822735

cpan/libnet/lib/Net/NNTP.pm | 6 3 + 3 - 0 !
1 file changed, 3 insertions(+), 3 deletions(-)

 net::nntp: correct innd/nnrpd confusion in relation to reader option

In the 'Reader' paragraph of the 'Constructor' section, the text has
nnrpd and innd the wrong way round

Bug: https://github.com/steve-m-hay/perl-libnet/pull/25
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=51962

cpan/Math-Complex/lib/Math/Trig.pm | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 math::trig: clarify definition of great_circle_midpoint

Bug: https://rt.cpan.org/Ticket/Display.html?id=114104
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697567

cpan/Math-Complex/lib/Math/Trig.pm | 2 2 + 0 - 0 !
1 file changed, 2 insertions(+)

 math::trig: add missing see also

Bug: https://rt.cpan.org/Ticket/Display.html?id=114105
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697568

cpan/Math-Complex/lib/Math/Trig.pm | 5 5 + 0 - 0 !
1 file changed, 5 insertions(+)

 math::trig: document angle units

Bug: https://rt.cpan.org/Ticket/Display.html?id=114106
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=731505

cpan/CPAN/lib/CPAN.pm | 3 3 + 0 - 0 !
1 file changed, 3 insertions(+)

 cpan: add link to main cpan web site

Bug: https://github.com/andk/cpanpm/pull/96
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=367291

cpan/Time-Piece/Piece.pm | 4 2 + 2 - 0 !
1 file changed, 2 insertions(+), 2 deletions(-)

 time::piece: improve documentation for add_months and add_years

These methods don't act on the time of the object they are called from,
so the previous documentation was open to misinterpretation.

Bug: https://github.com/rjbs/Time-Piece/pull/22
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=817925

utils/perlbug.PL | 25 13 + 12 - 0 !
1 file changed, 13 insertions(+), 12 deletions(-)

 perlbug: refactor duplicated file reading code

_send_message_mailsend() needs to build the message itself rather than
calling build_complete_message() like the other backends, but they can
still share the file reading code, and so can the 'display report' part.

utils/perlbug.PL | 13 12 + 1 - 0 !
1 file changed, 12 insertions(+), 1 deletion(-)

 perlbug: wrap overly long lines

Mail transport agents limit the length of message lines at SMTP time.
One observed limit is 1000 characters per line. Mail user agents typically
work around these limits by MIME-encoding the message. Since perlbug
doesn't do that, it needs to limit the length of its lines manually to
make sure bug reports get delivered.

The longest lines in perlbug reports normally come from Config::myconfig
output, particularly 'config_args', which has been observed to exceed
1000 characters on some configurations, causing report rejection. While
less likely, the list of local patches is another potential source of
overly long lines.

Use Text::Wrap (if available) to wrap the body of the report at an
arbitrarily chosen and hopefully safe limit of 900 characters. No
indentation or continuation line markers are added, though it would
be easy to add those if desired. Attachments and mail headers are not
wrapped.

(Backported for Debian 5.24 by removing lib/perlbug.t changes)

ext/POSIX/t/sigaction.t | 4 3 + 1 - 0 !
1 file changed, 3 insertions(+), 1 deletion(-)

 ext/posix/t/sigaction.t: skip uid and pid tests on gnu/hurd

SA_SIGINFO does not include uid and pid on GNU/Hurd, and this is
unlikely to change in the near future, so skip these tests.

Thanks to Samuel Thibault for the report.

hints/gnu.sh | 33 29 + 4 - 0 !
1 file changed, 29 insertions(+), 4 deletions(-)

 modify hints for hurd per debian ticket 825020.

For:  RT #128279

Patch written by Samuel Thibault, forwarded by Dominic Hargreaves.

Bug: https://rt.perl.org/Ticket/Display.html?id=128279
Bug-Debian: https://bugs.debian.org/825020

dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS.pm | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 make the output of extutils::parsexs reproducible

perl.c | 5 5 + 0 - 0 !
pod/perlrun.pod | 5 5 + 0 - 0 !
2 files changed, 10 insertions(+)

 look for sitecustomize.pl in /etc/perl rather than sitelib on debian
 systems

Debian uses this sitecustomize.pl as a transition mechanism for removing
the current working directory (".") from @INC by default while leaving
the local administrator a temporary means of re-adding it back.

This mechanism is expected to be removed after one Debian stable release.

Makefile.SH | 2 1 + 1 - 0 !
Porting/pod_rules.pl | 2 1 + 1 - 0 !
TestInit.pm | 2 1 + 1 - 0 !
lib/strict.t | 2 1 + 1 - 0 !
lib/warnings.t | 2 1 + 1 - 0 !
makedef.pl | 2 1 + 1 - 0 !
regen.pl | 2 1 + 1 - 0 !
regen/ebcdic.pl | 3 3 + 0 - 0 !
regen/genpacksizetables.pl | 2 1 + 1 - 0 !
regen/mg_vtable.pl | 2 1 + 1 - 0 !
t/comp/line_debug.t | 2 2 + 0 - 0 !
t/lib/warnings/op | 1 1 + 0 - 0 !
t/op/goto.t | 2 1 + 1 - 0 !
t/porting/regen.t | 2 1 + 1 - 0 !
t/re/pat.t | 4 2 + 2 - 0 !
t/run/switches.t | 6 3 + 3 - 0 !
t/test.pl | 3 2 + 1 - 0 !
17 files changed, 24 insertions(+), 17 deletions(-)

 patch unit tests to explicitly insert "." into @inc when needed.

Backported to perl 5.22 by Dominic Hargreaves and forward
to 5.24 by Niko Tyni

Bug: https://rt.perl.org/Public/Bug/Display.html?id=127810

cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm | 4 2 + 2 - 0 !
cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker.pm | 5 5 + 0 - 0 !
2 files changed, 7 insertions(+), 2 deletions(-)

 add perl_use_unsafe_inc support to eu::mm for fortify_inc support.

This change allows the majority of Perl modules that cannot build/test/install
without . in INC to be able to do so, while maintaining a safer perl under
normal use.

Backported to perl 5.22 by Dominic Hargreaves and forward to 5.24
by Niko Tyni

Bug: https://rt.perl.org/Public/Bug/Display.html?id=127810

cpan/CPAN/scripts/cpan | 5 5 + 0 - 0 !
1 file changed, 5 insertions(+)

 set perl_use_unsafe_inc for cpan usage

This change allows the majority of Perl modules to build/test/install from the
cpan client without having to modify them.

Bug: https://rt.perl.org/Public/Bug/Display.html?id=127810

pod/perlvar.pod | 8 5 + 3 - 0 !
1 file changed, 5 insertions(+), 3 deletions(-)

 document in perlvar that we remove '.' from @inc by default

cpan/ExtUtils-MakeMaker/lib/ExtUtils/Command/MM.pm | 3 2 + 1 - 0 !
cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm | 2 1 + 1 - 0 !
2 files changed, 3 insertions(+), 2 deletions(-)

 make perllocal.pod files reproducible

Bug-Debian: https://bugs.debian.org/834190
Bug-Debian: https://bugs.debian.org/835815

t/run/runenv.t | 4 1 + 3 - 0 !
1 file changed, 1 insertion(+), 3 deletions(-)

 remove test for '.' in @inc as it might not be

The intention is that sitecustomize.pl can be changed to remove '.'
from @INC, and this might become the default in future, so ensure that
this doesn't cause a build failure of perl.

cpan/File-Path/t/Path.t | 3 2 + 1 - 0 !
1 file changed, 2 insertions(+), 1 deletion(-)

 file-path: fix test failure in hurd due to hard-coded enoent

cflags.SH | 6 6 + 0 - 0 !
1 file changed, 6 insertions(+)

 temporarily lower the optimization of op.c on hppa due to gcc-6
 problems

This fixes a crash in ext/XS-APItest/t/customop.t

It's currently unclear if the problem is with the code or the
gcc-6 optimizer.

Bug-Debian: https://bugs.debian.org/838613

cpan/Test-Simple/lib/Test/Builder.pm | 1 1 + 0 - 0 !
1 file changed, 1 insertion(+)

 silence a 'used only once' warning in test::builder

Symptom of the problem:

 % perl -w -e 'use Test::Requires::Git'
 # git version 2.9.3
 Name "Test::Requires::Git::TODO" used only once: possible typo at /usr/share/perl/5.24/Test/Builder.pm line 2249.