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.

Restore MAD handling of package statements

Since the version seems unused by MAD in "use", do the same
for "package"...

Let tell() report warnings on unopened filehandles

(fixes a bug introduced by previous patch)

Make tell() fail properly if called without an argument and when no previous file was read

This makes tell() return -1, and sets errno to EBADF, thus
restoring the 5.8.x behaviour

Properly return a syntax error instead of segfaulting if each/keys/values is used without an argument

Updated Object-Accessor to CPAN version 0.36

  Changes for 0.36    Thu Oct  8 09:42:36 BST  2009
  =================================================
  *   Apply patch to fix ABSTRACT in core, RT #49563
      from Jerry Hedden.

Move script.t from t/comp to t/run, as it's a test for invoking perl.

CPAN::YACSmoke has been deprecated and minismokebox is a much more robust CPAN testing framework. Updated the links accordingly.

Add Porting/release_schedule.pod to MANIFEST.

Fix Pod errors spotted by podcheck.t

Makefile and makefile only exist on case-sensitive file systems, so allow them.

Remove BEGIN{}, use '..', part deux

Add tests for syntax errors in "package Name VERSION"

Expand on cookbook todo

Add perltodo: write an XS cookbook

Porting/checkAUTHORS.pl now runs clean

Further clean up checkAUTHORS.pl output (fixing encoded @ signs)

Add 'package NAME VERSION' syntax

This patch adds support for setting the $VERSION of a namespace
when the namespace is declared with 'package'.  It eliminates the
need for 'our $VERSION = ...' and similar constructs.  E.g.

  package Foo::Bar 1.23;
  # $Foo::Bar::VERSION == 1.23

There are several advantages to this:

  * VERSION is parsed in *exactly* the same way as 'use NAME VERSION'

  * $VERSION is set at compile time

  * Eliminates '$VERSION = ...' and 'eval $VERSION' clutter

  * As it requires VERSION to be a numeric literal or v-string
    literal, it can be statically parsed by toolchain modules
    without 'eval' the way MM->parse_version does for '$VERSION = ...'

  * Alpha versions with underscores do not need to be quoted; static
    parsing will preserve the underscore, but during compilation, Perl
    will remove underscores as it does for all numeric literals

During development of this, there was discussion on #corehackers and
elsewhere that this should also allow other metadata to be set such as
"status" (stable/alpha) or "author/authority".  On reflection, those
metadata are not very well defined yet and likely should never be
encoded into Perl core parsing so they can be freely changed in the
future.  (They could perhaps be achieved via a comment on the same line
as 'package NAME VERSION'.)

Version numbers, however, already have a very specific definition and
use defined in the core through 'use NAME VERSION'.  This patch merely
provides appropriate symmetry for setting $VERSION with the exact same
parsing and semantics as 'use'.

It does not break old code with only 'package NAME', but code that
uses 'package NAME VERSION' will need to be restricted to perl 5.11.X.
This is analogous to the change to open() from two-args to three-args.
Users requiring the latest Perl will benefit, and perhaps N years from
now it will become standard practice when Perl 5.12 is targeted the
way that 5.6 is today.

The patch does not prevent 'package NAME VERSION' from being used
multiple times for the same package with different version numbers, but
nothing prevents $VERSION from being modified arbitrarily at runtime,
either, so I see no urgen reason to add limitations or warnings so
long as Perl uses a global $VERSION variable for package version
numbers.

I am posting this patch to the p5p list for discussion and review.  If
there seems to be general assent (or lack of dissent), I will go ahead
and commit the patch to blead.

sort @files once, outside the loop.

Standardise t/lib/*.t on ./test.pl, and remove BEGIN boilerplate.

Remove commented out reference to Test::More

Standarise t/porting/*.t on ./test.pl, and remove BEGIN boilerplate.

Replace a hardcoded temporary file name with test.pl's tempfile().

Standardise t/uni/*.t on ./test.pl, and remove PERL_CORE boilerplate.

TestInit sets up the directory for us.

Also remove unncessary C<use Encode>s.