Perldelta for exec PROGRAM LIST with empty LIST

[perl #131730] Fix exec PROGRAM LIST with empty LIST on VMS

Only when being Unixish, because I have no idea if
setup_argstr()/vms_do_exec() handle it.

[perl #131730] Fix exec PROGRAM LIST with empty LIST

This should call execvp() with an empty argv array (containing only the
terminating NULL pointer), but was instead just returning false (and not
setting $!).

Executing a program with an empty argv array is valid ISO C: it merely
requires argc to be nonnegative and argv[argc] to be NULL.  POSIX states
that that argv[0] "should point to a filename string that is associated
with the process being started", but "should" only applies to
applications claiming strict POSIX conformance.  Perl does not, and
certainly should not impose it on perl programs.

This also requires handling the case where both PROGRAM and LIST are
empty, to avoid calling execvp(NULL, …), which _is_ invalid.  I've made
it return ENOENT, which it the error POSIX specifies for execvp("", …).

Upgrade Test-Simple from version 1.302101 to 1.302103

the unopened test should use a file handle distinct from "OTHER"

used a name and added a comment so this isn't broken again

[perl #127063] suppress warning on assigning 0 to $[

PERL5LIB documentation: '.' is no longer in @INC by default

customise Pod::Perldoc to fix output misbehaviour

Pod::Perldoc has, since version 3.20, exhibited various kinds of
misbehaviour relating to a bad default choice of formatter.  Output has
sometimes appeared mangled due to the newly-default formatter emitting
unportable escape sequences, and sometimes there has been a more
severe output failure due to perldoc making unportable changes to pager
configuration in an attempt to make the escape sequences work.  This is
discussed in [perl #131762].  In the upstream instance of the module
there have been tweaks to the unportable behaviour, but not an actual fix.

In order to make the core distro ship a reliably-working version of
perldoc, this patch customises Pod::Perldoc to implement the obvious
fix for the portability problems.  The fixed version defaults to the
ToText formatter, which produces properly plain text that will go through
any pager.  It never attempts any change to pager configuration.

[perl #124349] deprecation warning on hostname() with arguments

Use preprocessor check for some DEBUG_X_TEST

Most of the DEBUG_?_TEST calls are already protected
by one '#idef DEBUGGING' check, but noticed a few of them
which were not protected in sv.c and toke.c

We should avoid these extra 'if' statements if perl
is not compiled with debug option: -DDEBUGGING.

(perl #131982) update the set/getpriority documentation too

(perl #131982) provide constants for get/setpriority in POSIX.pm

<sys/time.h> isn't necessarily needed, but apparently increases
portability, since the rusage structure defined in <sys/resource.h>
has struct timeval members.

Document that POSIX::abs, ::alarm, ::atan2, ::chdir, ::chmod need explicit argument.

Reformat code in one foreach block for readability.  Make podcheck happy.

For: RT #132145

pp.c: delete dead cpp-conditional declaration

This was added in commit dfe9444ca7881e716e9e8feaf20b55da491363ca (February
1998, for Perl 5.004_60) by Andy Dougherty, and its comment says that, even
then, he thought it was unneeded. But the perl5 repo has ever defined the
NEED_GETPID_PROTO cpp symbol that guards this declaration, so this ability
has clearly never been used.

Remove out-of-date mention of USA president

Avoid having to continue to update this every time the president changes
by replacing Barack Obama with Ada Lovelace.

RT#131000: splice doesn't honour read-only flag

The push and unshift builtins were correctly throwing a "Modification of a
read-only value attempted" exception when modifying a read-only array, but
splice was silently modifying the array. This commit adds tests that all
three builtins throw such an exception.

One discrepancy between the three remains: push has long silently accepted
a push of no elements onto an array, whereas unshift throws an exception in
that situation. This seems to have been originally a coincidence. The
pp_unshift implementation first makes space for the elements it unshifts
(which croaks for a read-only array), then copies the new values into the
space thus created. The pp_push implementation, on the other hand, calls
av_push() individually on each element; that implicitly croaks, but only one
there's at least one element being pushed.

The pp_push implementation has subsequently been changed: read-only checking
is now done first, but that was done to fix a memory leak. (If the av_push()
itself failed, then the new SV that had been allocated for pushing onto the
array would get leaked.) That leak fix specifically grandfathered in the
acceptance of empty-push-to-readonly-array, to avoid changing behaviour.

I'm not fond of the inconsistency betwen push on the one hand and unshift &
splice on the other, but I'm disinclined to make empty-push-to-readonly
suddenly start throwing an exception after all these years, and it seems
best not to extend that exemption-from-exception to the other builtins.

Use library functions instead of manual TAP in t/io/tell.t.

Committer: minor corrections to descriptions.

Dmitry Ulanov is now a Perl author.

For: RT #132276

hints/aix.sh: remove trailing spaces

miniperl also needs -Ilib for perl.exp on AIX etc

Document that -I. is now needed with -MTestInit

g++ is like gcc on AIX too

note that we cannot use g*, as AIX/vac ships gxlc as xlc with gcc-like
option handling

delete unused file from SDBM_File

get rid of "implicit fallthrough" warnings in SDBM_File

SDBM_File: tabs -> spaces

Delete dead mktemp() call

Commit 9c12f1e5a87cce227357eea4b0780c0323f952f0 provided support for Cray
XT4 Catamount/Qk. It added some code with a call to (insecure) mktemp(),
guarded with "#ifdef HAS_MKTEMP". But nothing has ever attempted to define
that preprocessor symbol (including the hints file that was added in that
commit), so this is clearly unused.

fix comment on GCC_DIAG_IGNORE/GCC_DIAG_RESTORE

actually fix "implicit fallthrough" warnings in gcc 7

remove the GCC_DIAG_RESTORE pragma as well

remove GCC_DIAG_IGNORE from sbox32_hash.h

fixups for portability

a) some platforms want #define to start at col 0
b) some compilers want to know whether fallthrough is expected or not.

Delete unused files from ext/SDBM_File

skip bench.t if valgrind is installed but broken

use standard STMT_START/STMT_END macros in STATIC_ASSERT_STMT

avoid gcc 7 warning about * in boolean context

get rid of "implicit fallthrough" warnings in ext/re/re_exec.c

get rid of "implicit fallthrough" warnings with gcc 7

Fix typo in comment about Configure modification

Clarify policy on updates to Configure

Further metaconfig updates to Porting/pumpkin.pod

Add Porting/mksample

This script was previously in the metaconfig repository, and also dealt
with updating Porting/Glossary, but it wasn't being regularly used.
It does not use any data from that repository, so is better placed
here.

There are some outstanding ambiguities to resolve in when this script
should be run in the documentation, since the generated files
Porting/config.sh and Porting/config_H are also updated by other
means.

Actually test for memrchr in VMS

Update Config-Perl-V to CPAN version 0.29

  [DELTA]

0.29  - 12 Oct 2017, H.Merijn Brand
    * It's 2017
    * Add test for 5.26.1-RC1
    * Fix signature with argument
    * Ignore ccache and compiler path in signature

Finish adding memrchr Configure probe

Commit 1e436e33 accidentally added the probe to Configure, this finishes
the job by regenerating Glossary, config_h.SH and friends.

Strip ccache from $Config{cc}

When building with -Dcc="ccache cc", $Config{cc} would also end up with
this setting. If that perl is then distributed, no XS will be able to
build if the target machine has no ccache installed/available

With this change, the whole build of perl itself will use ccache, but
$Config{cc} will have it stripped, so all modules (lib/cpan/ext) will
not use ccache. A correct installation of ccache however does not need
$CC to include the ccache prefix, so it is essentially a setup problem
on the building host.

fix comment for upstream meta

Update metaconfig related information

Upgrade Test-Simple from version 1.302100 to 1.302101

hopefully better macros for building on x86

perl no longer builds on MacOS Classic

Upgrade Test-Simple from vesion 1.302098 to 1.302100

Upgrade Encode from version 2.92 to 2.93

Eliminate recently introduced compilation warnings.

Warnings like the following started to appear within last 24 hours while
running 'make' on Linux/gcc and FreeBSD-10.3/clang.

op.c: In function ‘Perl_newATTRSUB_x’:
op.c:8593:28: warning: suggest parentheses around ‘&&’ within ‘||’
[-Wparentheses]
        :   IN_PERL_RUNTIME && PL_curstash != CopSTASH(PL_curcop)

Increase B::Xref::VERSION to 1.07

Fix B::Xref to handle sub refs

This only applies to non-threaded builds.  Threaded builds were
already fine.

B: Really remove unused var

6a4fc5265ba102 removed the useless assignment to it, but not the
var itself.

[Merge] [perl #129916] Allow sub-in-stash outside of main

The sub-in-stash optimization introduced in 2eaf799e only applied to
subs in the main stash, not in other stashes, due to a problem with
the logic in newATTRSUB.

This branch includes various commits to fix the issue and other prob-
lems that the fix uncovered.

Make pp_multideref handle local $::{subref}

Based on a patch by Nicholas R.

[perl #129916] Allow sub-in-stash outside of main

The sub-in-stash optimization introduced in 2eaf799e only applied to
subs in the main stash, not in other stashes, due to a problem with
the logic in newATTRSUB.

This comment:

   Also, we may be called from load_module at run time, so
   PL_curstash (which sets CvSTASH) may not point to the stash the
   sub is stored in.

explains why we need the PL_curstash != CopSTASH(PL_curcop) check.
(Perl_load_module will fail without it.) But that logic does not work
properly at compile time (when PL_curcop == &PL_compiling).

The value of CopSTASH(&PL_compiling) is never actually used.  It is
always set to the main stash.  So if we check that PL_curstash !=
CopSTASH(PL_curcop) and forego the optimization in that case, we will
never optimize subs outside of the main stash.

What we really need is to check IN_PERL_RUNTIME && PL_curstash !=
opSTASH(PL_curcop).  I.e., forego the optimization at run time if the
stashes differ.  That is what this commit implements.

One observable side effect of this change is that deleting a stash
element no longer anonymizes the CV if the CV had no GV that it was
depending on to provide its name.  Since the main thing in such situa-
tions is that we do not get a crash, I think this change (arguably an
improvement) is acceptable.)

-----------

A bit of explanation of various other changes:

gv.c:require_tie_mod needed a bit of help, since it could not handle
sub refs in stashes.

To keep localisation of stash elements working the same way,
local($Stash::{foo}) now upgrades a coderef to a full GV before the
localisation.  (Changes in two pp*.c files and in scope.c:save_gp.)

t/op/stash.t contains a test that makes sure that perl does not crash
when a GV with a CV pointing to it gets deleted.  This commit tweaks
the test so that it continues to test that.  (There has to be a GV for
the test to test what it is meant to test.)

Similarly with t/uni/caller.t and t/uni/stash.t.

op.c:rv2cv_op_cv with the _MAYBE_NAME_GV flag was returning the cal-
ling GV in those cases where a GV-less sub is called via a GV.  E.g.,
*main = \&Foo::foo; main().  This meant that errors like ‘Not enough
arguments’ were giving the wrong sub name.

newATTRSUB was not calling mro_method_changed_in when storing a
sub as an RV.

gv_init needs to arrange for the new GV to have the file and line num-
ber corresponding to the sub in it.  These are taken from CvSTART,
which may be off by a few lines, but is the closest we have to the
place the sub was declared.

Increase $B::Deparse::VERSION to 1.43

Deparse: Better constant-dumping heuristics

Constants created via sub foo () { 1 } are stored in the stash as
simple scalar references, under the CV-in-stash optimisation.  That
optimisation currently only applies to the main package, but will
shortly be extended to other packages.  This means B::Deparse’s
heuristics for dumping the constants needs to be improved, to avoid
dumping B::Deparse’s own constants for every program.

The heuristic I am using (since CvFILE is not present on a scalar ref)
is to record whether other subroutines in the same package as the con-
stant are being dumped by virtue of having CvFILE pointing to a file
that is being dumped.  This assumption is that constants and subroutines
in the same package are likely to be in the same file.

Provisional version bump for NEXT.pm

A patch ha- been submitted upstream already, so hopefully this version
number will be short-lived.

Increase B::Concise::VERSION to 1.002

Make B::Concise handle subrefs in stashes

The concise_stashref sub, for dumping all subroutines in a package,
would assign the value of a stash element to *s, and then use *s
to access the code ref in it.  If you do *s = *foo and then later
*s = \&bar, then you have assigned \&bar to *foo{CODE}, and even
a localisation of *s beforehand will not help.  That is exactly
what B::Concise was doing when dumping a package with some subref
elements.

[rt.cpan.org #123002] Fix NEXT.pm to work with GLOB stubs

I need this in order to fix perl bug #129916.

Add isGV_or_RVCV macro

This will be useful for a few code paths that need to treat a sub
ref in a stash the same way as a GV.

Let Deparse.t be run from the top-level

It used to work before 851f7bb3.  It is helpful when debugging tests
to be able to run ‘./perl -Ilib lib/B/Deparse.t’ without chdir-
ring around.

Unbreak gv_fetchmeth_sv

Commit v5.21.6-383-gc290e18 stopped gv_fetchmeth_sv from working cor-
rectly when fetching a constant with a utf8 name, because it no longer
passed the utf8 flag to the underlying functions.

That utf8 flag gets passed to gv_init when upgrading a glob proxy
(such as a constant) into a real glob.

Increase $B::VERSION to 1.70

B.pm: Remove unused var

This variable stopped being used in perl-5.005_02-1108-g8bac7e0 but
continued to exist until now.

Don’t assign PL_curstash twice in init_main_stash

This commit:

commit 8990e3071044a96302560bbdb5706f3e74cf1bef
Author: Larry Wall <larry@netlabs.com>
Date:   Fri Mar 18 00:00:00 1994 +0000

    perl 5.0 alpha 6

added ‘curstash = defstash’ to perl.c:init_main_stash, which already
had such an assignment a few lines above.  So it is redundant, and
always has been.

Set PL_curstname in pp_ctl.c:doeval

Otherwise we get the wrong name in sub error and warning messages:

$ ./miniperl -we 'package bar { sub bar { eval q"sub foo ([)" } } bar::bar'
Missing ']' in prototype for main::foo : [ at (eval 1) line 1.

(PL_curstname is probably used for other things too.  I didn’t check.)

I can arbitrarily set the package name in the warning to what-
ever I want:

$ ./miniperl -we 'package bar { sub bar { eval q"sub foo ([)" } } package fwipm; BEGIN { bar::bar }'
Missing ']' in prototype for fwipm::foo : [ at (eval 1) line 1.

Move PERL_INTERNAL_RAND_SEED set-up after init_stacks().

This code may be calling things that put SVs on the stack, so
we should have a stack before doing so.  That risk might be
theoretical on most platforms, but on VMS, the getenv
implementation mortalizes an SV when doing lookups.  This meant
that on a DEBUGGING build with PERL_DESTRUCT_LEVEL set, any code
or no code at all would warn like so:

  $ perl -e ";"
  Attempt to free temp prematurely: SV 0x845718 during global destruction.
  Scalars leaked: 1

Getting the stack initialized first fixes that.

Update Test-Simple to CPAN version 1.302098

  [DELTA]

1.302098  2017-10-03 06:13:49-07:00 America/Los_Angeles

    - Add docs for test2_stdout and test2_stderr
    - Fix 5.6 support

Update Digest-SHA to CPAN version 5.98

  [DELTA]

5.98  Wed Oct  4 00:40:02 MST 2017
  - removed "portable" mode from shasum and addfile
    -- rarely used, mostly in outdated systems
    -- potentially confusing features (e.g. \r\r\n -> \n)
    -- Universal Newlines mode (-U) a much cleaner approach
      -- mimics Universal Newlines in Python
  - shasum now uses Digest::SHA explicitly
    -- no longer loads Digest::SHA::PurePerl as an option
      -- hence no need for -R switch
    -- Digest::SHA::PurePerl has its own shasum: shasumpp

perldelta for 0cbfaef69bb7

pp_require: return earlier when module is already loaded

Note that vutil.c is customized in blead

This was done by 7394beb140 but didn't make it into the last version
upgrade (0.9918).

Fix mapping of Filter-Util-Call/filter-util.pl

and note that t/z_manifest.t is excluded.

Note Devel-PPPort/parts/embed.fnc is customized in blead

Change BadMETA.yml EOLs to match CPAN distro's version

Note perlfaq/lib/perlfaq[58].pod are customized in blead

Note pod/perlpodstyle.pod is customized in blead

Note Net-Ping/t/510_ping_udp.t is also customized in blead

Regen customized.dat

Pod-Simple-3.35 is KHW, not MARCGREEN

sync-with-cpan: curl support

Porting/sync-with-cpan: 5.12 compatibility

So I can type simply ‘Porting/sync-with-cpan Test::Simple’ and run
it with the system perl.

Upgrade Test-Simple from version 1.302096 to 1.302097

Upgrade Test-Simple from version 1.302073 to 1.302096

(includes regen/lib_cleanup.pl)

fix some thinkos in the new hash code

pointed out by Tux on irc. I dont think this will solve
all his problems, but it is a first step....

Add tests for 'p' and 'x' commands without subsequent whitespace.

Tests pass on perl-5.16.3 but should fail (until source code is corrected) on
subsequent versions.

For: RT #120174

Debugger cmds not requiring spaces

Make debugger commands like these work again, not requiring a space
between a single-letter command and a following argument which starts with
punctuation:

  p$^V
  x@ARGV
  x\@ARGV
  x\%INC

Regressions were in d478d7a0 and 8f144dfc.

Remove outdated references to 'swap'.

Per: RT # 131532

Revert "Revert "Fallbacks for Perl_fp_class_denorm().""

This reverts commit 2749d0395cc1e21d69af38cde41490699fc6e3ab.

The original commit:

    commit e77299d3416e7e737523afdc0642734205e46d59
    Author: Jarkko Hietaniemi <jhi@iki.fi>
    Date:   Mon May 29 09:28:30 2017 +0300

        Fallbacks for Perl_fp_class_denorm().

        These may be needed if the compiler doesn't expose the C99 math
        without some special switches.

This provides a fix for CentOS 5.

Document that POSIX::localtime needs explicit argument.

For: RT #132145

Update File-Fetch to CPAN version 0.54

  [DELTA]

0.54        Sun Sep 24 19:46:04 2017

* Switch test URL from www.cpan.org to httpbin.org

More than twist in my sobriety

Tick off 5.24.3 and 5.26.1

Add perldeltas for 5.24.3 and 5.26.1

Import Module::CoreList data for 5.24.3 and 5.26.1