Typo in commit b641685 breaks Solaris-x64 with Sun cc

perlre: Don't treat /aa as a separate modifier

VMS-related known problems in 5.14.0.

Skup sigdispatch tests on VMS as they hang the VMS smokers.

On Wed, Apr 20, 2011 at 7:22 PM, Craig A. Berry <craigberry@mac.com>
wrote:
> In article <E1QCB3D-0002kP-5p@camel.ams6.corp.booking.com>,
>  davem@iabyn.com ("Dave Mitchell") wrote:
>
>> In perl.git, the branch blead has been updated
>>
>> <http://perl5.git.perl.org/perl.git/commitdiff/011c381477c2b48fc4fbb6c52c59dbd
>> 6a21bc7d6?hp=53777b0ce48433ad582498a56c60698a8fad29f6>
>>
>> - Log
>> -----------------------------------------------------------------
>> commit 011c381477c2b48fc4fbb6c52c59dbd6a21bc7d6
>> Author: David Mitchell <davem@iabyn.com>
>> Date:   Tue Apr 19 14:17:12 2011 +0100
>>
>>     dispatch signals when leaving an eval
>
> The good news is that this also fixes RT #76384.  The bad news is that
> the new tests hang the test suite on VMS and the watchdog is unable to
> break out of it (possibly only with threads -- still need to check
> that).

On Win32, skip the tests added in 011c3814, as alarm can't interrupt select.

perldelta: Fix broken links

Apparently the writers of perldiag realized that "%s%s" is the same
as %s for the purposes of diagnostics, so there is no anchor for the
removed link.

Revert "Missing bug number in d12b49d"

This reverts commit 53777b0ce48433ad582498a56c60698a8fad29f6.

constant.pm 1.21 has already been released on CPAN. Reverting this for
now just keeps things simple.

Authors sorting from tchrist

perlop: /o update

perllocale: Mention /l

Also the possibility of an undefined initial locale

perldelta: Fix remaining confusing double double quotes

C<foo"bar"> will translate into "foo"bar"" on some devices, which is
confusing.  Change the remaining ones to C<foo'bar'>

perldelta encoding error spotted by tchrist

Bump patchlevel to RC1

Update to Known Problems, removing several previously-known problems
I've verified as fixed.

Reflowing text

Ilmari pointed out that I incorrectly marked up some perl function()
statements as C<function()>

Update the documentation for rand() to note that it's not
cryptographically secure due to concerns that end-users are unaware of
this and use it in situations where security depends on the strength of
the randomness generated.

I'd have been happier getting this patch in earlier in the cycle.
We'd hoped to replace the RNG, but that didn't happen in time, so this
doc update is the "better, still not good" fallback.

Update Module::CoreList for 5.14.0

Add 5.14.0RC1 and 5.14.0 (hopeful) date to perlhist

Started to flesh out the AUTHORS section of perldelta

dispatch signals when leaving an eval

Currently PERL_ASYNC_CHECK is only called during scope exit in pp_leavetry
and pp_levaeeval. This means that if the signal handler calls die, the
eval won't catch it.

This broke Sys::AlarmCall's test suite, which was doing the equivalent of

    $SIG{ALRM} = sub { die };
    eval {
alarm(1);
select(undef, undef, undef, 10);
    }
    # expect the die to get caught and $@ set here.

Because the select was the last statement in the block, PERL_ASYNC_CHECK
wasn't called next until the leave_scope at the end of leavetry.
See RT #88774.

The simple fix is to add a PERL_ASYNC_CHECK at the top of
leavetry and leaveeval.

Missing bug number in d12b49d

Correct typos in 66b4c19

porting/cmp_version.t: skip threads 1.83

The threads version for blead was bumped for the 5.13.11 release,
but not all version numbers were bumped, so the '1.83' in 5.14.0
will be marginally different (1 line of text) than in the 5.13.11 release.

fix blurb at top of t/porting/cmp_version.t

t/porting/cmp_version.t: add version skip facility

Allow it to skip failing for particular versions of particular
modules

Removed a note about Test::Harness that was obsoleted by the release of
Test::Harness 3.23

I think I've now got all the funny quote business taken care of.
I've pod-, spell-, and dupword-checked this.

Update perldelta for push/keys changes

• Mark it as experimental
• Remove paragraphs that no longer apply

Update perlfunc for [perl #80626]

Add a test for keys $tied

This was ‘inadvertently’ fixed with commit 7ac5715

Make push, etc., work on tied scalars

I broke this with commit d4fc441

update threads::shared version in perldelta.

Upgrade to threads 1.83

Upgrade to threads::shared 1.37

perllocale: Corrections

This pod misled some people, including this author, as to the initial
state of locales.  This fleshes out some details, and changes some
wording.

perlrecharclass: Nits

Perldelta patches from Ilmari and Abigail

Fix a few pod nits

Make keys $scalar an lvalue

This does a run-time check to see whether $scalar is a hash ref, and
dies if it is not.

This is to keep keys \@_ consistent with keys @_.

I cannot simply use OPf_MOD, since that indicates *potential* lvalue
context (including subroutine args).

So, instead, I take advantage of the fact that OPf_SPECIAL is always
set on the LHS of an assignment (usually to indicate that local()
should not erase the value).

perlop: Update for some 5.14 changes

perluniintro: Update for 5.14 changes

perlunicode.pod: Nits

perlre: Remove false statement about locales

VMS section clean-up for perldelta.

perldelta editing pass

Remove ambigous warning from perldiag

This no longer happens as of commit 7ac5715.

Make keys/value/each $scalar accept only unblessed refs

See ticket #80626.

collapse plibpth to one line and remove trailing /

The recent change to hints/linux.sh,
40f026236b9959b7ad3260fedc6c66cd30bb7abc
set the plibpth variable.
It was supposed to set all entries on a single line, but it didn't.
Do it now, and also remove trailing /'s.

(The collapsing is a more robust version of the previous commit,
since reverted, that davem wrote independently).

Revert "collapse plibpth to one line"

This reverts commit 55e4a474ad63535e486bd657f45b5339709cbcd3.

In improved version is coming next...

collapse plibpth to one line

The recent change to hints/linux.sh,
40f026236b9959b7ad3260fedc6c66cd30bb7abc
set the plibpth variable.
It was supposed to set all entries on a single line, but it didn't.
Do it now,

configpm: handle multi-line key='value\n...'

There is old code in configpm to handle mulit-line entries in config.sh
along the lines of

    plibpth='/lib/x86_64-redhat-linux/4.4.5/
    /lib/../lib64/
    /usr/lib/x86_64-redhat-linux/4.4.5/
    /usr/lib/../lib64/
    /lib/
    /usr/lib/'

which was broken, and produced
    Use of uninitialized value $1
warnings, and messed up the content of lib/Config_heavy.pl.

We probably normally don't have multi-line entries, which is why no-one
noticed it before, but 40f026236b9959b7ad3260fedc6c66cd30bb7abc
has started generating the entry above.

fix for pp.c under win32 etc

Commit d4fc4415aac96132fac5b1e43e73bcba33a41b79 added two definitions
of the DEREF_PLAIN_ARRAY array; the non-GCC one had syntax errors

Minor fixes to perldelta.pod

Reviewed section "Core Enhancements".

Signed-off-by: Ævar Arnfjörð Bjarmason <avar@cpan.org>

Consistent use of 'e.g.,' in perldelta.pod

There were some places, where 'e.g.' was used without a comma. I made it
consistent.

Signed-off-by: Ævar Arnfjörð Bjarmason <avar@cpan.org>

Consistent use of two spaces after full stop in perldelta.pod

There were more occurences of two spaces after full stops, so I made
those two that were only one.

Signed-off-by: Ævar Arnfjörð Bjarmason <avar@cpan.org>

perldelta: typos

pod/perldelta.pod: POSIX should be in all-caps (not "Posix")

pod/perldelta.pod: clarify "Use of qw(...) as parentheses" deprecation

Clarify the note about qw(...) as parentheses deprecation. Without
being clarified this could be misunderstood to deprecate most uses of
qw(...).

This came up on the Git mailing list[1] after I submitted a patch[2]
to fix this issue in Gitweb[3].

1. http://permalink.gmane.org/gmane.comp.version-control.git/167293
2. http://permalink.gmane.org/gmane.comp.version-control.git/167283
3. http://permalink.gmane.org/gmane.comp.version-control.git/167297

pod/perldelta.pod: link to pertinent feature.pm POD

Instead of just linking to L<feature> link to the "the
'unicode_strings' feature" section.

Make push/shift $scalar accept only unblessed aryrefs

See ticket #80626.

perldelta for d12b49d (Unicode constants)

check --whole-archive is supported (in cc.cbu) before using it

netbsd for pre 4.6 gcc requires --whole-archive to build shared
libraries, but this is rejected and not required in 4.6.0.

Revert "check --whole-archive is supported before using it"

This reverts commit 673d8593b7ef274dadbfff97fd641e3c563fc716.

I meant to apply the cc.cbu version of this patch.

Lots of perldelta editing. Still miles to go before we sleep

Make Unicode constants under use utf8 work again

Because sub lookup (and glob lookup in general) ignores the UTF8
flag, such subs are actually ‘correctly’ stored under the utf8-encoded
equivalent of the name, and not the name itself.

TODO some of the new sigdispatch tests on VMS.

Needs more signal foo than I can muster at the moment to figure out
what the intent is here and why it's not working.

As part of their switch to a multi-arch library layout,
Ubuntu 11.04 (and later, presumably) doesn't keep most libraries
(such as -lm) in /lib or /usr/lib.  So we have to ask gcc to tell us
where to look.  We don't want gcc's own libraries, however, so we
filter those out.

This could be conditional on Ubuntu, but other distributions have
announced their intent follow suit, and this scheme seems to work even
on rather old gcc's.  This unconditionally uses gcc because even if the
user is using another compiler, we still need to find the math library
and friends, and I don't know how other compilers will cope with that
situation.  Still, as an escape hatch, allow Configure command line
overrides to plibpth to bypass this check.

perlre: Nits in recently added text

Nits in perlunicode

Followup to 088225f/[perl #88132]: packages ending with :

Commit 088225f was not sufficient to fix the regression. It still
exists for packages whose names end with a single colon.

I discovered this when trying to determine why RDF::Trine was crashing
with 5.14-to-be.

In trying to write tests for it, I ended up triggering the same crash
that RDF::Trine is having, but in a different way.

In the end, it was easier to fix about three or four bugs (depending
on how you count them), rather than try to fix only the regression
that #88132 deals with (isa caches not updating when packages ending
with colons are aliased), as they are all intertwined.

The changes are as follows:

Concerning the if (!(flags & ~GV_NOADD_MASK)...) statement in
gv_stashpvn: Normally, gv_fetchpvn_flags (which it calls and whose
retval is assigned to tmpgv) returns NULL if it has not been told
to add anything and if the gv requested looks like a stash gv (ends
with ::). If the number of colons is odd (foo:::), that code path is
bypassed, so gv_stashpvn returns a GV without a hash. So gv_stashpvn
tries to used that NULL hash and crashes. It should instead return
NULL, to be consistent with the two-colon case.

Blindly assigning a name to a stash does not work if the stash has
multiple effective names. A call to mro_package_moved is required as
well. So what gv_stashpvn was doing was insufficient.

The parts of the mro code that check for globs or stash elems that
contain stashes by looking for :: at the end of the name now take into
account that the name might consist of a single : instead.

Give EU::CB 04-base.t an absolute path on VMS.

catfile() with an empty string as a first argument does *not*
produce an absolute path on VMS (and probably only does so by
accident elsewhere).  For purposes of the test, it seems that
any absolute path would do, so this could probably be done
portably, but on the eve of 5.14 let's make a separate code
path for VMS and leave everyone else as is for now.

IPC::Open3 needs porting to work on VMS.

So skip the test for now.  Something like the Win32 mechanism
spawn_with_handles() may be workable but needs investigation.

Don't copy all of %ENV in prereq.t.

It's sufficient for purposes of enabling prerequisite warnings to
set a localized $ENV{PERL_CORE} to undef rather than making a local
copy of the entire %ENV hash.  This makes the test pass on VMS,
where copying %ENV in toto is not supported.

This fixes a smoke failure, so borrow it from upstream, where it's
already released as part of MM 6.57_10.  6.58 will contain an
additional tweak that works with Perl 5.6, but no need to worry
about that here.

perlrebackslash: Slight edits

perlop: Slight edits

perlunicode: More 5.14 edits

perlunitut: clarification

perluniprops: Linked to wrong pod; slight edits

perlunicode: Edits for 5.14

In the release guide, mention the AdvanceCOMP can compress gzip files better.

It's an option release managers (on *nix) might want to be aware of.

stop waithires.t failing under high load

In threads::shared, the waithires.t test checks cond_timedwait() with
sub-second timeouts. If the newly-minted child doesn't manage to grab the
lock within 0.05s, the cond_timedwait() will timeout, and the child and
the test will hang, until eventually killed off by the watchdog.  This can
easily happen on a slow/loaded system.

We fix this by putting the cond_timedwait() in a retry loop, only giving
up after 10 seconds of repeated timeouts.

perllocale: Clean up recent additions

perldelta: 342c852 changed $re::VERSION

[perl #88486] IO::File does not always export SEEK*

Commit d963bf0 made perl set @IO::File::ISA automagically.

Commit 15e6cdd made filehandle methods load IO::File automagically.

Commit efc5c7c attempted to solve the problem that having IO::Handle
loaded causes IO::File to be bypassed, rendering 15e6cdd only
semi-effective (see [perl #87940]), by requiring IO::File inside
IO::Handle.

That commit ended up breaking several CPAN modules, because IO::File
adds @IO::Seekable::EXPORT to its own exports.

If IO::Seekable is loaded first (before IO::File), before setting up
its @EXPORT it loads IO::Handle, which loads IO::File, which tries to
load IO::Seekable, which is in %INC already, and which is hence wait-
ing for IO::File to load before it sets its own @EXPORT. So IO::File
sees @IO::Seekable::EXPORT empty.

Hence, every piece of code that tries to import SEEK_END from IO::File
will simply not get it if IO::Seekable is already loaded (explicitly
or, e.g., by File::Temp).

This commit hopefully fixes the breakage and the problem that efc5c7c
attempted to fix by loading IO::File only inside IO::Handle::new (the
only method that IO::File overrides).

perldelta entries for 088225f

[perl #88420] BOM support on Windows broken in 5.13.11

When Perl reads the script in text mode, then the tell() position
on the script handle may include stripped carriage return characters.
Therefore the file position after reading the first line of the
script may be one larger than the length of the input buffer.

[perl #88132] broken ISA lookup after aliasing packages ending with ::

gv_fetchpvn_flags did not always assign a name to a return HV ending
with ::. This would result in code in various places skipping certain
‘stashes’ (in quotes because nameless HVs are technically not stashes)
because they were nameless when they should not have been.

So sometimes ISA caches would end up being out of date, as in the test
cases posted with [perl #88132] (and incorporated into this patch).

This commit fixes that by changing the parsing of glob names.

Formerly, a :: was not considered a package separator if it came imme-
diately after a ::. So foo:::: would become foo::/:: (with the final
:: considered a regular stash entry, not a ‘stash’ stash entry) and
foo:::::: would become foo::/:::/:.

Now a :: is always a package separator. So *foo::::bar is accessible
via $foo::{"::"}{bar} and *$foo:::::: via $foo::{"::"}{"::"}.

This happens to fix [perl #88134] as well.

perldelta: Clarify entry

perlre.pod: Clarify

Several confusions have arisen about how things work, and this
addresses them.

perlunicode: Update for 5.14

perllocale: Update for 5.14

In testargs.t in Test::Harness, don't run a world-writable file.

The test writes a file, then changes the mode, then executes it. The file needs
to be +x to be executable (on many platforms). The file will need to be +w to
be deletable on some platforms. But setting the file world writable just before
running it feels like a bad idea, given that the file's name is as predictable
as process IDs, as there's a race condition to break into the account running
perl's tests.

handle freed backref array in global cleanup

[perl #88330]

If a thinggy is heavily leaked, so that it takes multiple passes through
Perl_sv_clean_all to get its refcount to zero, then if it has weak refs to
it, its backref array may get freed before it.  We already set the
refcount of the array to 2 to preserve it across one pass of
Perl_sv_clean_all, but I can't think of a way of protecting it more
generally (short of using a private array structure rather than an AV).

In the past, this caused a scary assertion failure.

Now instead, just skip if we're in global cleanup and the array is freed.
This isn't ideal, but its reasonably robust, as we don't reuse freed SVs
once in global cleanup (so the freed AV hangs around to be identified as
such).

check --whole-archive is supported before using it

netbsd for pre 4.6 gcc requires --whole-archive to build shared
libraries, but this is rejected and not required in 4.6.0.

Simply changing the option to -Wl,--whole-archive works on gcc 4.6.0
but fails for the system cc on NetBSD 5.1/x64.

perlrebackslash: Update for 5.14 changes

perlretut: Update for 5.14 /a, /u

perlrecharclass: Update for 5.14 changes

perlre.pod: Update for 5.14

reg_eval_scope.t: skip coring tests on all arches

Some TODO tests that were known to core dump are already skipped on
Windows on production releases. Extend this to all platforms,
to avoid scaring the punters with spurious 'Aborted' messages in the
test harness output.