Minor updates to the git tutorial and our commit message policies.

[perl #71676] GDBM_File support on Win32

Date: Thu, 07 Jan 2010 00:36:25 -0800
Message-ID: <rt-3.6.HEAD-1505-1262853384-1207.71676-15-0@perl.org>

Tweaked as suggested in:

From: "H.Merijn Brand" <h.m.brand@xs4all.nl>
Date: Fri, 8 Jan 2010 13:35:17 +0100
Message-ID: <20100108133517.1f2916d3@pc09.procura.nl>

Perl blead on AIX - fail: t/porting/test_bootstrap

Hello,

Nicholas Clark wrote:
> On Mon, Jan 18, 2010 at 11:33:07AM +0100, Rainer Tammer wrote:
>
>> Hello,
>> the 32 bit blead build looks quite good on AIX:
>>
>> A threaded, shared 32 bit build only produced one error:
>>
> Thanks for continuing to look out for AIX.
>
>
No problem ...
>> Is this a know error ??
>>
> Yes, but I believe that I committed a change to fix it about 30 minutes ago.
>
>
OK,
I try this fix.

I have also a fix for the GDBM problem.
Now GDBM is only disabled if the "problem" headers are installed,
e.g. if the gdbm rpm < 1.8.3-5 is installed.

I have attached the patch for this.

> Nicholas Clark
>
>
>
Bye
  Rainer

From 4755db7f86c38d3c2b69c668c5747368e98c3eae Mon Sep 17 00:00:00 2001
From: Rainer Tammer <tammer@tammer.net>
Date: Mon, 18 Jan 2010 11:40:27 +0100
Subject: [PATCH] Enabld GDBM on AIX if possible

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

Move packagev.t from t/comp to t/op, as it fouls the test bootstrapping policy.

The package VERSION syntax isn't strictly an op, but it needs to use several
features that aren't yet tested at the time that tests in t/comp are run.

Revert "[perl #62646] Maximum string length with substr"

This reverts commit b6d1426f94a845fb8fece8b6ad0b7d9f35f2d62e.

Conflicts:

pp.c

update AUTHORS for new commits, spelling

Add t/comp/packagev.t to MANIFEST

Upgrade to CPAN.pm 1.94_54

Clean up mktables intro comments; remove trailing white space

Correct VMS-specific handling of $! in Perl_magic_get.

0097b436152452e403cc71b4f1a1cfd30ec0ba1a had introduced a test failure
on VMS because it only set the POK flag in the non-VMS code path,
which was an easy mistake to make because accumulated patches had
over time made the #ifdef jungle way more tangled than it needed to
be.  There is really only one line that needs to be VMS-specific.

A fig leaf for calling sv_pos_u2b with IV* where it expects I32*.

Following v5.11.3-103-gb6d1426, any compiler paying attention
whines about the pointer mismatch (which on VMS breaks the
build).

Zefram reports a further patch in progress:
  Message-ID: <20100116022631.GA10264@fysh.org>

but this gets the warning out of the way so other work can proceed.

Parse 'use NAME VERSION' with C locale

wrap a link in L<> to avoid bad autolinking

[perl #72098] File::Copy stripping 06000 perms on cp for root

The problem is with the use of cp when perms & 06000.

There is logic that checks to see if the target file is owned by the
user ($>) running the copy and/or if $> is a member of the group that
owns the target file. If this is not the case, then the 06000 bits are
masked out before the chmod is called after the copy.

This is mostly good logic except when root is executing this, in which
case root should get to do whatever it wants to do.

Looking closer at the code to test for group membership, I think it can
be more easily and more cheaply be written using $). I've added this
change to the patch.

This will also fix the problem where someone has a group membership
based on /etc/passwd and is not mentioned in /etc/group.

* Tom's latest perlfunc cleanups, to ensure we're on the same track

* Backported Tom's perlfaq cleanup to the perlfaq repo, and now they are back here :)

Fix SV leaks in Perl_vnumify() and Perl_vnormal() for "Invalid version object"s

Add warning "Cannot find encoding"

Bump version of PerlIO::encoding

The warnings emitted by PerlIO::encoding should be silenceable.

Make them mandatory warnings.

Avoid a double SV leak in an error state branch of pp_require.

Avoid using a sprintf format for a constant value (0).

[perl #62646] Maximum string length with substr

(This is only a partial fix, since it doesn't handle lvalue substr)

Document last change in perldelta for 5.11.4.

This will have to be merged with other Unicode property changes
in the final 5.12 perldelta.

PATCH: [perl #71726] \p{xdigit} should match full-width forms

The Unicode standard suggests that xdigit match not only the ASCII hex
digits, but also the full width forms starting at U+FF10.

This patch just changes the source from the Unicode ASCII hex digit to
the Unicode normal hex digit.

This adds a constructor option to not output the comments containing the range counts; and uses that option on Decomposition.pl, used by normalize.pm.

Qualify pointer arguments of prescan_version in embed.fnc

Bump version to 2.21 and add changelog

Fixed the closure argument passing bug perl#72068

Fixed tests.

Added tests for perl#72068

Back out the {ENTER,LEAVE}_with_name("sub") part of d343c3ef45381352 for now.

It's conflicting with an established pattern in XS code, working around the
inability of the XS SCOPE: keyword to actually provide anything useful.
The minor amount of extra debugging it gives here is not worth the external
trouble it causes.

Revisit this once we can provide a meaningful option to disable the
ENTER/LEAVE around XSUBs.

CYG23-544-stat

Stable cygwin patch for root filetests (gid 0 root <= gid 544 Administrators).

On cygwin check for the Administrators group (544) which has root
rights regarding -r filetests.

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

Omnibus strict and lax version parsing

Authors: John Peacock, David Golden and Zefram

The goal of this mega-patch is to enforce strict rules for version
numbers provided to 'package NAME VERSION' while formalizing the prior,
lax rules used for version object creation.  Parsing for use() is
unchanged.

version.pm adds two globals, $STRICT and $LAX, containing regular
expressions that define the rules.  There are two additional functions
-- version::is_strict and version::is_lax -- that test an argument
against these rules.

However, parsing of strings that might contain version numbers is done
in core via the Perl_scan_version function, which may be called during
compilation or may be called later when version objects are created by
Perl_new_version or Perl_upg_version.

A new helper function, Perl_prescan_version, has been added to validate
a string under either strict or lax rules.  This is used in toke.c for
'package NAME VERSION' in strict mode and by Perl_scan_version in lax
mode.  It matches the behavior of the verison.pm regular expressions,
but does not use them directly.

A new test file, comp/packagev.t, validates strict and lax behaviors of
'package NAME VERSION' and 'version->new(VERSION)' respectively and
verifies their behavior against the $STRICT and $LAX regular
expressions, as well.  Validating these two implementation should help
ensure they each work as intended.

Other files and tests have been modified as necessary to support these
changes.

There is remaining work to be done in a few areas:

* documenting all changes in behavior and new functions

* determining proper treatment of "," as decimal separators in
  various locales

* updating diagnostics for new error messages

* porting changes back to the version.pm distribution on CPAN,
  including pure-Perl versions

Changed Copyright year as suggested by karl williamson in nntp.perl.org/group/perl.perl5.porters/155493

* Em dash cleanup in pod/

I looked at all the instances of spaces around -- and in most cases
converted the sentences to use more appropriate punctuation. In
general, the -- in the perl docs seem to be there only to make
really complicated and really long sentences.

I didn't look at the closed em-dashes. They probably have the same
sentence-complexity problem.

I left some open em-dashes in place. Those are the ones used in
lists.

* FAQ sync for Perl 5.12 release candidate

This is commit cfc60b072536b6232cec8a3a3551c8bdf3c3eefd from the
perlfaq repository at git@github.com:briandfoy/perlfaq.git

Tom Christiansen's perlfunc cleansing, part 2

From "PATCH: perlfunc cleanup, part 2" (28574.1263366404@chthon)

Quite a lot here, I know.

I probably spent the most time on pack(), as you'll see, but there are
plenty of rephrasings of awkward wordings plus the occasional orthographic
fix scattered throughout.  I've added a few more examples here and there,
and certain simple things like perl vs Perl have been set to canonical form.

I've eyeballed the diffs for my own typos, and I've looked at the whole
thing after running it both through nroff (not so pretty, but better) and
also through troff (*much* better).

I didn't try to make sure all code inserts were indented the same number of
spaces, not because it doesn't need being done, but because I didn't want
to blow this patch up that much more than it already is.  You can see the
ragged left in the code inserts if you troff->ps->pdf view it.  Oh, well.

Hope this helps,

--tom

Update File-Fetch to CPAN version 0.24

  Changes for 0.24        Wed Jan  6 23:32:19 2010
  =================================================
  * Applied a patch from brian d foy RT #53427
    that makes new() respect sub-classes.

test stash_name in less.pm

catch the one $class instance missed in less.pm

Document UNIVERSAL->import deprecation in perldelta

Mention name of C constant in feature.pm for greppability

The new lvalue warning should not be turned on by default

A check for the warning category was missing from commit
885ef6f56b61fd750ef3b1fa614d11480baac635. Also, document
the warning category in perldiag.

Document new lvalue warning in perldelta

The "illegalproto" warning category is new in 5.11.4, not 5.11.3

Nits in perldelta template

Add perl5114delta.pod

With $PERL_UNICODE we still need to find utf8.pm

fix for [perl #65582] anon globs segfaulting

The following code has had differing behaviours:

    my $io_ref = *STDOUT{IO};
    my $glob = *$io_ref;

          defined($glob)    "$glob"
          --------------    -------
5.8.8       false           "" with uninit warning
5.10.0      true            (coredump)
this commit true            ""

$glob is essentially an anonymous typeglob (no NAME, EGV or GvSTASH).
It shouldn't register as undefined since it's clearly a valid GV with a
valid IO slot; Stringifying to "" seems to be the right thing, and not
warning seems right too, since its not undef.

fix for [perl #61976] Errno ($!) not evaluated to a error message string
(5.10.0 in taint mode)

Change 27176 / 2a509ed3c095f7d712013e653f68821f6bb2d6db fixed a taint
bug, which as a side effect, meant that $! used within a tainted expression
failed to have a string value.

This quick fix just makes sure the POK flag is set (prior to it being
shifted back to pPOK).

Add note to .pod for compound form perl extensions

Bump less' version

allow indirection between less and its hints stash name

Remove [[:posix:]] references because changes not going into 5.12

fix for [perl #41138] $_ leaks under threads

It's possible for an interpreter to get cloned with an SV copied that ends
up only linked from @_. For example, local $x causes a link to the
original $x SV to be added to the save stack, but when cloning a thread
the save stack isn't copied. If the old $x was also in someone's @_,
then it gets copied, but because @_'s elements  aren't normally reference
counted, old $x ends up with a refcount of zero, and you get a "leaked"
warning when the thread exits.

The workaround is to reify any reify-able AVs in the cloned interpreter
during cloning.

Also fixes [perl #70602], [perl #70974]

PATCH: perlfunc cleanup, part 1

So I started tidying up the C<delete> and C<each> documentation,
but ended up diddling the file pretty much all over.  There are
some typo corrections, but most of this is just smoothing out
infelicitous wording.  I did add one brief example to C<glob>.

I've made no warnings about delete and slices with negative
indices; I'm hoping that may be fixed, as David has suggested
be done.  It's not a political edit, just a copy-edit.

This is obviously only the first portion of the file, but the
tweaks were getting extensive enough I thought I'd better send
in a checkpoint.  I'll finish it up over the next several days.

[perl #71788] Skip $) test when NGROUPS_MAX is too small or when on darwin

Remove an extraneous space

Re: [perl #71852] Warning "/* within comment" in Win32 config.h-related files

(Applied patch and regenerated canned config files)

Message-ID: <4B44FB2B.8080509@volny.cz>

Retain builtin attributes from pre-declaration. Fixes [perl #68758].

Ignore a lvalue attribute after the subroutine has been defined, and warn about it. Fixes part of [perl #68758].

[perl #71924] shmget/semget may return zero as valid id

Documentation in perlipc thus fixed by using // instead of ||

Completely avoid autovivification of CORE::GLOBAL::caller

(by using symbolic references as suggested by Vincent)

Typo fix : caller:: isn't caller

(Noticed by Aaron Crane)

Add new prototype warning test file to MANIFEST

Perldelta presentation nits

Move prototype parsing related warnings from the 'syntax' top level warnings category to a new 'illegalproto' subcategory.

Two warnings can be emitted when parsing a prototype -

  Illegal character in prototype for %s : %s
  Prototype after '%c' for %s : %s

The first one is emitted when any invalid character is found, the latter
when further prototype-type stuff is found after a slurpy entry (i.e. valid
character but in such a place as to be a no-op, and therefore likely a bug).

These warnings are distinct from those emitted when a sub is overwritten by
one with a different prototype, and when calls are made to subroutines with
prototypes - those are in the pre-existing sub-category 'prototype'.

Since modules such as signatures.pm and Web::Simple only need to disable
the warnings during parsing, I chose to add a new category containing only
these. Moving these warnings into the 'prototype' sub-category would have
forced authors to disable more warnings than they intended, and the entire
raison d'etre of this patch is to allow the specific warnings involved to
be disabled.

In order to maintain compatibility with existing code, the new location
needed to be a sub-category of 'syntax' - this means that

  no warnings 'syntax';

will continue to work as expected - even in cases like Web::Simple where all
subcategories extant prior to this patch are re-enabled (this is another
reason why a move into the 'protoype' category would not achieve the desired
goal).

The category name 'illegalproto' was chosen because the most common warning
to encounter is the "Illegal character" one, and therefore 'illegalproto'
while minorly inaccurate by ignoring the (relatively recent and unknown)
second warning is an easy name to spot on an initial skim of perllexwarn
and will behave as expected by also disabling the case of an unusual prototype
that happens to look like a normal one.

This patch updates pod/perllexwarn.pod, perldiag.pod and perl5113delta.pod
to document the new category, toke.c and warnings.pl to create and implement
the new category, and a new test t/op/protowarn.t that verifies the new
behaviour in a number of cases. It also includes the files generated by
regen.pl that are found in the repo - notably warnings.h and lib/warnings.pm.

2.024

[perl #71948] Documentation error for (*MARK)

The NAME portion of (*MARK:NAME) is not optional.

Update skip count in perlio.t.

Follow-up to af9379e9ed4daaed65ba42baa492afc842917dd5.

PL_sv_serial only exists ifdef DEBUG_LEAKING_SCALARS

Fix for 4149198fba64273f3fea8fc073ccb5d080059f4a which broke
debugging build. Whoops!

Update Archive-Extract to cpan version 0.38

  Changes for 0.38    Wed Jan  6 23:48:52 2010
  ============================================
  * Apply a patch from Michael G Schwern RT #53246
    extract() is vulnerable to print globals.

fix for [perl #66108] Leaked scalars

@DB::args is a hack: it gets set with non-refcounted aliases of the
caller's @_ elements.

Once the sub that ran caller() has exited, @DB::args will contain garbage:
elements will be SVs that have been freed, re-assigned etc.

So as a minimum, when cloning an interpreter, skip cloning @DB::args.

reset sv_serial for each new interpreter

to a value of 0 + 1000000*interpreter_number
where interpreter_number is currently just a dirty hash of my_perl

in perl_clone() move common code outside ifdef/else blocks

e diff looks confusing, but basically it changes:

    #ifdef PERL_IMPLICIT_SYS
      ...
      COMMON_BLOCK
    #else
      ...
      COMMON_BLOCK
    #end

into
    #ifdef PERL_IMPLICIT_SYS
      ...
    #else
      ...
    #end
    COMMON_BLOCK

in perl_clone() shuffle some ifdefs about a bit

No change in logic.

unify #ifdef indent on both sides of PERL_IMPLICIT_SYS in perl_clone()

include sv_debug_serial field in debugging output

Avoid race conditions with files in /tmp, by explicitly checking dev & inode.

(Concerns raised by and the form of the solution suggested by Bram.)

Correct errors in the use of multiple targets, which could break parallel make.

Where a single script (or program) generates multiple files, the Makefile rule
was written with those multiple files as targets on the same line. This would
be correct if that syntax meant "these $n things are built by these commands".

However it doesn't - it means "to build any of these targets, run these
commands", and hence to run them once for each of the targets. This can result
in race conditions with parallel makes, with files being trampled on and
strange failure modes.

Hence the correct style is to give one of the targets on the rule, and rules
that make all the other targets that are generated depend on that target.

Unlink PerlIO's tempfiles for the case of no -T, but bogus $ENV{TMPDIR}

When -T is enabled, or when $ENV{TMPDIR} is bogus, perlio.c used a pathname
matching </tmp/PerlIO_??????>. However, it was only correctly unlinking the
file for the case of -T enabled.

Cleanup PerlIO temp files

[perl #71870] Use of uninitialized value in bitwise and B::Deparse

It's better to just silence it in Deparse rather than stopping
B::PMOP::reflags from returning undef because of a non-constant regexp.
That way, we keep the possibility to test for this situation, and it stays
in line with B::PMOP::pregcomp.

Also bump $B::Deparse::VERSION.

Fix typo in reference

Merge branch 'blead' of ssh://perl5.git.perl.org/gitroot/perl into blead

Document that interpolating a '(??{ code })' construct in a regular
expression also requires "use re 'eval'", just as '(?{ code })' does.

Fix a couple of typos and standardize on using 'regexp' instead of
using a 'regex'/'regexp' mash.

vaproto for the other OS's

Follow-up to d03b3b00ac22f32af87a752669a46d9d06ae1561

Increase default pipe buffer size on VMS to 8192 on 64-bit systems.

As Charles Lane's piping code reaches its 10-year anniversary, it's
time to bump up the default to something slightly more appropriate
for modern systems.  In addition to other benefits, this fixes test
failures when a test script outputs a large amount of text on one
line (e.g., t/re/reg_posixcc.t).

vaproto for VMS.

Follow-up to d03b3b00ac22f32af87a752669a46d9d06ae1561, which broke
the build.

[perl #71748] Bleadperl f0e67a1 breaks CPAN: Template::Plugin::YAML::Encode 0.02

Unsurprisingly, the nature of the bug is that I accidentally changed
the logic of one of the several types of space skipping.  Fix attached.

Avoid a possible race condition where a parallel make might
attempt to update git_version.h and lib/Config_git.pl in
concurrent processes.

Sort the initialisers to non_bincompat_options[], and fix a whitespace "error".

Add USE_ATTRIBUTES_FOR_PERLIO to the list of -V's compile-time options.

Tie::Hash::NamedCapture::* shouldn't abort if passed bad input [RT #71828]

Note which t/ subdirs are not suitable for test.pl use

FD 4 is not (yet) open at this stage.

Regenerated after backporting 88a6f4fc380d30c40
Please *do* remember to notify the metaconfig folk when directly patching Configure
Bring back Missing parts

PATCH: minor typo cleanup of pod/ directory

These are all in the pod/ directory, and only the first is a code fix.
There was also a single lingering ISO 8859-1 encoding that missed the
UTF-8 upconvert.  The rest are cleanups for typos, some of which seem
to have been around for a rather long time: spelling errors, incorrect
possessives, and extra, missing, or duplicated words.

If you actually read through, I bet you'll realize what sparked this. :)

--tom

Signed-off-by: Abigail <abigail@abigail.be>

perlguts: mutli-threaded typo

--
Reini Urban
http://phpwiki.org/           http://murbreak.at/

From 76ebe81f50d97e606c4243d5060d4749f7408deb Mon Sep 17 00:00:00 2001
From: Reini Urban <rurban@x-ray.at>
Date: Mon, 4 Jan 2010 18:09:25 +0000
Subject: [PATCH 1/2] perlguts: mutli-threaded typo

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

perlguts: B::Generate already works

--
Reini Urban
http://phpwiki.org/           http://murbreak.at/

From 520f6e768bdf71ad35740698df2d4c7f0bb7ae9e Mon Sep 17 00:00:00 2001
From: Reini Urban <rurban@x-ray.at>
Date: Mon, 4 Jan 2010 18:16:14 +0000
Subject: [PATCH 2/2] perlguts: B::Generate already works

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>