Podify the social contract about contributed modules

perlvar $1 clarification

$1 is currently documented as being set by the "last pattern match".
But it is left alone by unsuccessful pattern match attempts (continuing
to hold a value from an earlier successful match).

Saying "last successful pattern match" clarifies this; it's also the
phrase used to document $&.

Second, the entry for $1 in perlvar doesn't actually contain the text
"$1" anywhere.  As such, doing man perlvar then using /\$1 to search for
it in less doesn't locate it (though does match other places in that
file where $1 happens to be used).

Change S_ckwarn_common() to looping over the packed warning value.

The core never uses WARN3() or WARN4(), and rarely uses WARN2(), so the
previous code, effectively an unwrapped loop, wasn't a speed up. Functionally
equivalent smaller code fits better into CPU caches.

Migrate common code in Perl_ckwarn() and Perl_ckwarn_d() to S_ckwarn_common()

In Perl_ckwarn() and Perl_ckwarn_d() disentangle the complex conditionals.

It's much easier to see what is going on, if we use multiple return statements.

Replace Perl_deprecate() with a macro that calls Perl_ck_warner()

Perl_deprecate was not part of the public API, and did not have a deprecate()
shortcut macro defined without -DPERL_CORE. Neither codesearch.google.com nor
CPAN::Unpack show any users outside the core.

Move the code related to deprecate("comma-less variable list") into a function.

This makes all calls to deprecate() string constants.

Remove category 'syntax' from 5 warnings that should just be in 'deprecated'.

None were documented as also being in 'syntax'. Effectively, this completes the
reorganisation of commits 12bcd1a617c74d6e and 9014280dc8264580. See
http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2009-10/msg00601.html and
http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2002-03/msg00850.html

Use deprecate() in place of deprecate_old(), and remove S_deprecate_old().

ckWARNreg() and ckWARNregdep() now use string concatenation, like their brethren

They had been concatenating "%s" REPORT_LOCATION, as they weren't passing in a
format string, which wasn't consistent with the 2-5 argument versions. None of
the strings passed in have % characters in them, so this is safe (and any static
analyser will be able to see this).

Replace "%s"/SvPVX(sv_name) with "%"SVf/SVfARG(sv_name), which is UTF-8 safe.

In regcomp.c, where possible replace vWARN* with ckWARN*, using Perl_ck_warner.

This folds many pairs of ckWARN*() && Perl_warner() calls into singles call to
Perl_ck_warner(). vWARN(), vWARNdep() and vWARN2() are no longer used, so are
removed.

overload no longer implicitly unsets fallback on repeated 'use overload' lines - Fix for RT#68916

Subject: overload::import resets the setting of 'fallback'
Date: Tue, 04 Aug 2009 17:54:34 -0700
From: "Geoffrey T. Dairiki" <dairiki at dairiki.org>

Remove Test-Simple's 00compile.t from core

It required a hand-edited version of Test-Simple's MANIFEST due to the
exclusion of a .pm file from core, but we shouldn't have edited versions
of CPAN files in core. The presence of the MANIFEST file also caused
warnings from the Makefile.PL due to other files being excluded from core,
so rather than further editing of the MANIFEST it is simplest to just
remove it and the test that requires it.

Add Perl_ck_warner_d(), which combines Perl_ckwarn_d() and Perl_warner().

Replace ckWARN_d{,2,3,4}() && Perl_warner() with it, which trades reduced code
size for 1 more function call if warnings are not enabled.

Add Perl_ck_warner(), which combines Perl_ckwarn() and Perl_warner().

Replace ckWARN{,2,3,4}() && Perl_warner() with it, which trades reduced code
size (about 0.2%), for 1 more function call if warnings are not enabled.
However, if we're now in the L1 or L2 cache when we weren't previously, that's
still going to be a speed win.

Normalise 3 fatal pack/unpack error messages to "panic: %s"

In uni_to_bytes(), fix ckWARN() anomaly.

Re-order a conditional with ckWARN() last, as it is a function call.

version's t/survey_locales is not in core so mark it as EXCLUDED

deprecate_old() is not public, and only used within toke.c, so can be static.

In Perl_ckwarn(), simplify the check of isLEXWARN_on/isLEXWARN_off.

Don't install filter-util.pl

This file was getting incorrectly installed with perl since commit
2adbc9b6919cad1240a834325b355e9b2d5efd67, that moved it from t/ to ext/.

Implement TODO support locally in t/comp/opsubs.t

Removes the FAILED for test 22 on Win32.

Fix failed() calls in t/comp

Try for smarter (more portable) skip criteria in Encode's piconv.t.

We already differ from upstream, which skips when PERL_CORE is set.
Since we apparently think we can run it in core now, it seems like
we ought to sort out the proper skip criteria before sending it
upstream.

No longer need to skip t/comp/opsubs.t #22 on VMS.

And it's a good thing since the skip infrastructure was pulled
out from under us with 6b077bebea000af2a5477d50d7604bab33ee75c2.

The Win32 TODO probably still doesn't work and will need to be
implemented locally without the require of test.pl.

porting/manifest.t now tests that all files listed in MANIFEST are present.

Default to running tests in cpan/ with relative paths for perl and @INC

Explicitly turn paths absolute for the 33 extensions in cpan/ that fail tests
with relative paths.

Run tests in ext/ and dist/ with relative paths for perl and @INC.

Also, as only tests in cpan/ are using %no_abs and %temp_no_core, only consult
these look-up hashes for tests in cpan/

Swap the chdir and the call to perl_lib(), to work with relative paths in @INC.

MakeMaker::Test::Utils::perl_lib now copes with relative paths for core testing.

In the core, @INC already contains the moral equivalent of blib/lib. However,
it's a relative path (by default), so make it absolute. It's easier to KISS if
this is done *before* any change of directory, so document this, and change the
non-core case to add the absolute path of 'blib/lib' to @INC, rather than the
absolute path of '../blib/lib'.

Remove core @INC setting and chdir boilerplate from Thread::Semaphore's tests.

Remove core @INC setting and chdir boilerplate from Thread::Queue's tests.

Remove core @INC setting and chdir boilerplate from SelfLoader's tests.

Remove core @INC setting boilerplate from lib's test.

Correct the location of the DISTRIBUTION tarball for Parse::CPAN::Meta

Possibly we shouldn't store this, if it can be determined unambiguously from the
PAUSE index files.

Exclude CGI/t/fast.t from the core, as it relies on FCGI, which is not in core.

Whilst the test is designed to skip if FCGI is not present, that logic is now
broken, and even if it were fixed, the as-implemented skip count is now wrong.
We're never going to run this test, as core tests run with just the core's
libraries in @INC, so I see no reason to ship it.

Correct the name of subtest.t, added in 2c4d5b9bac4cc261

There isn't actually an "all files in MANIFEST are present" test in t/porting.
Right now we rely on ./Configure on *nix doing it, which doesn't work if you
aren't on *nix, or are, but don't re-run Configure. Maybe we should add one.

Correct the listed names of the two files uudecoded in commit 7b742a041dc3ba8c

It removed the '.uu' from the end of the name, but didn't change the preceding
'_' to '.'. They'd been changed from '.' to avoid having multiple '.'s in the
name, as some VMS filesystems don't like multiple '.'s. It's not easy being
green.

Darwin's groups can have spaces in them

Fix Test-Simple's new 00compile.t test in core

It was failing even with its MANIFEST file now in place because that
MANIFEST listed an IO::Scalar file that is EXCLUDED from core.

Add Test-Simple's MANIFEST file

It is needed by the new 00compile.t test script. We don't normally include
CPAN distributions' MANIFEST files in core, but there is an existing
precedent for this in ExtUtils-MakeMaker.

Remove test boilerpates from ExtUtils-CBuilder tests

I was obviously wrong to restore in 1051afdbc0a9e60b8f65ed22fefafd8ecf771171
what Nicholas had removed in 4d984b75f1859fc27f556ee42f31a04df42a2004 because
the tests don't work in core with the boilerplates in place.
Don't undo that which has been done by Nicholas!

Correct MANIFEST, broken by Test-Simple upgrade

Upgrade to Test-Simple-0.94

Upgrade to Storable-2.21

Retained local changes to *.t (mostly test boilerplates) since blead
is canonical for this distribution

Upgrade to Pod-Simple-3.08

Sync Pod-Simple test scripts with CPAN versions

Retained local changes in:
corpus.t, reinit.t, search20.t, search22.t, search50.t
because they look more significant

Upgrade to Parse-CPAN-Meta-1.40

Upgrade to ExtUtils-Manifest-1.57

Upgrade to ExtUtils-CBuilder-0.260301

Sync ExtUtils-CBuilder test scripts with CPAN versions

This restores the test boilerplates that were removed when ExtUtils-CBuilder
was moved from lib/ to ext/ (4d984b75f1859fc27f556ee42f31a04df42a2004)

Upgrade to CGI.pm-3.48

Replace UU encoded files in CGI with their binary originals

There is no longer any need to avoid having binary files in the perl
distribution, and these files are not UU encoded in CGI on CPAN.

Move CGI's Changes file to its proper place

Applied a patch from David Fifield to fix an error message in perlipc.pod

Subject: Comment doesn't match code in perlipc.pod.
Date: Thu, 30 Jul 2009 07:31:48 -0600
To: perlbug@perl.org
From: David Fifield <david@bamsoftware.com>

Added a TODO test for RT#69332 - PerlIO returns undef instead of 0 on EOL

[perl #69686] Fix DragonFly thread lib selection

  Hey,

  attached patch fixes the thread library detection in DragonFly.  All
  versions of DragonFly can be used threaded with -pthread.  libc_r must
  not be used directly.

  Please let me know if you have any questions.

  Thanks,
     simon

Make _quote_args() return a result, rather than passing it a scalar reference.

This was the only use of scalar references in test.pl, and the only use of \
to create a reference to a scalar. We are now more robust in the face of parser
or runtime bugs.

Avoid using ++, op= and anon hash constructors in the testing code.

{} could be misparsed, ++ has a lot of internal implementation "magic" that we
don't need, but don't want to trip us up if it isn't working, and op= isn't
necessary when we already rely on the more general $a = $b op $c working.

Refactoring tests to use test.pl is a TODO.

Add a test for the bootstrap rules for tests in t/

Don't use require in comp/fold.t, as require isn't tested yet.

Emit TAP directly.

Give names to all tests in t/comp/fold.t

Move the require './test.pl' to the end of t/comp/hints.t

Ideally tests in t/comp wouldn't use require, as require isn't tested yet, but
this test really needs runperl(), and really wants to live in t/comp/hints.t,
so place it at the end, so that any catestrophic failure only fails the last
test. We don't use any other functionality of t/test.pl

This test uses hard-coded test numbers, but I'm not convinced that it would be
correct to re-write it to use an automatically incrementing counter, as that
wouldn't fail in an obvious fashion if some compile-time blocks ran out of
order. What we have *will* fail in an informative fashion if compile time
blocks do not run correctly.

Don't use require in comp/multiline.t, as require isn't tested yet.

Emit TAP directly.

Don't use require in comp/opsubs.t, as require isn't tested yet.

Emit TAP directly.

Don't use require in comp/our.t, as require isn't tested yet.

Emit TAP directly.

Don't use require in comp/parser.t, as require isn't tested yet.

Emit TAP directly.

Don't use require in comp/retainedlines.t, as require isn't tested yet.

Emit TAP directly.

Avoid relying on prototypes working for tests to pass. They aren't tested yet.

Don't use require in comp/uproto.t, as require isn't tested yet.

Emit TAP directly.

In opt(), use is(..., undef) rather than ok(!defined ...)

Move the test for require 5.11.0 not loading strictures to require.t from use.t

Don't use require in comp/utf.t, as require isn't tested yet.

So emit TAP directly, rather than utilising test.pl. Like test.pl, avoid using
++, as it has complexity, and that complexity isn't tested yet.

Record that Ask and Steve Hay have volunteered to be releng victims

Add missing IO-Compress test file

The original IO-Compress patch:

http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2009-03/msg00293.html

omitted the file (but strangely included the entry "t/cz-03zlib-v1.t*" [sic]
in the MANIFEST file that it mistakenly included).

Also note in Maintainers.pl that IO-Compress is now in cpan/ not dist/,
and mark UPSTREAM as 'cpan'.

Use require.t's bytes_to_utf() in place of PerlIO layers in utf.t

This avoids it needing to load Config, and means it can by run with miniperl,
and PerlIO-disabled perls.

Print the encoding name as part of the test, rather than on a separate line.

Refactor bytes_to_utf16() into a more generic routine that also handles UTF-8

Remove the direct code for testing UTF-8, calling bytes_to_utf() from a loop for
all 3 tested encodings.

Replace longhand invocations of test() with 3 nested loops.

Test utf8 with BOMs, like we already test utf16be and utf16le.

Move tests for use for "new style version numbers" to use.t from require.t

Slightly clean up the release manager guide by specifying an "End here for SNAPSHOT" per DAPM

Document that blead is released by a member of the sucker-of-the-month club.

Update the release manager guide to clean up the tagging instructions

dapm++ pointed out that I'd duplicated the "create a tag" instructions
while doing 5.11.0.  Because of dmq's new tag-based releng
infrastructure, the tag step needed to move earlier.

MakeMaker testing in core runs from one level deeper than before.

So when we set up a dummy root for the tests on VMS, we need to
make it two levels above current rather than one in order to
prevent a lot of test failures that say "PERL_CORE is set but
I can't find your PERL_SRC!"

Move &do_require to the top of require.t, so that @a is the first lexical.

The relevant line, with comment, is:

    my @a; # magic guard for scope violations (must be first lexical in file)

Move the test for RT #49472 to op/attrs.t from comp/require.t

Disable strictures while setting $VERSION in a "package" statement

Merge branch 'feature/package-name-version' into blead

Documentation typo

Simplify code that sets $VERSION on the "package" line

Avoid using bytes in require.t, as pack "C0U" does what we need.

Avoid using the strict pragma in retainedlines.t - use may not work yet.

Avoid using the strict pragma in colon.t - use may not work yet.

Avoid using pragmata in fold.t - use may not work yet.

Avoid using pragmata in opsubs.t - use may not work yet.

Avoid using the warnings pragma in proto.t - use may not work yet.