-Dmad minitest failure bisect

Tony Cook wrote:
>Smokes with -Dmad have been failing during make minitest since the
>middle of last month.

Mostly fixed by the attached patch.  The fault is a logic error on my
part, probably from the early phase of developing the lexer API patch,
when I didn't properly understand the various buffer pointer variables.

In my tests with -Dmad, I'm still getting a test failure ("panic: input
overflow") from t/op/incfilter.t.  The underlying problem is the filter
layer mishandling things when a filter function gives it a multiline
string, so it generates an invalid SV state (strlen(SvPVX(PL_linestr))
> SvCUR(PL_linestr)).  This faulty state also occurs without -Dmad,
and so doesn't appear to be Mad-related, it just doesn't in practice
cause the test panic without -Dmad.  I'm investigating this bug now.

-zefram

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

perlbug reports "Message sent" even when it hasn't

perlbug doesn't check the return status of Mail::Send and so reports
"Message sent" even if it hasn't:

perl-5.11.2 breaks NYTProf savesrc option (Lexer API suspected)

Tim Bunce wrote:
>The primary issue is the off-by-one error in the array indexing.

There's a bit more to it than that.  The indexing was off-by-one for
*some* places that process a new line, but correct for others, so the
saved source as a whole was mangled rather than simply offset.  Also,
there were some redundant calls to update_debugger_info(), so some lines
got saved twice, in some cases off-by-one for one saving and not for
the other.  The saved source is, therefore, hopelessly broken in 5.11.2.

Attached patch fixes the source saving.  Includes a new test, which works
through all reachable places that source lines get saved.  This should
close RT #70804.

-zefram

Jerry's last patch to add a TODO test didn't update the test count.

Thanks to Nicholas++ for pointing this out

Moved the original test file from the previous patch into t/op/lex.t,

Aded details on the bug it fixes to the tests.

add interpolation test for [perl #70091]

(Updated by Jesse Vincent to put the test in comp rather than a new
toplevel test directory)

Also skip spaces after variable if we are within lexical brackets. Fixes #70091: Segmentation fault in hash lookup in regex substitution

Add assertions that pp_padav and pp_padhv push scalars of the correct type.

further windows makefiles fixes

Message-ID: <4B0D4F71.3010606@khwilliamson.com>

wrap uniprops.t; makefile changes for mktables

Message-ID: <4B0C4744.7080401@khwilliamson.com>

Add TODO test for perl #70748

Bump version and regenerate warnings.pm

Add tests.

Add code and starting perldoc for warnings::fatal_enabled.

This is an analog for warnings::enabled, except it tests whether the
given category has been set fatal using "use warnings FATAL => foo".
This is mostly for symmetry.

Assumes that the fatal bit for a category will have an offset one higher
than the regular bit for the category, because otherwise much rewriting
of __chk would be required.

Add missing library to VC++ section of Win32 dmake makefile

Avoid circularity in $(LINK_FLAGS) definition in Win32 dmake makefile

cleanup get_arena param-names, mark as May-Change

new param-names reflect actual usage.
Mark as may-change so we can add a reqid field later.

Slight rewording of last patch

add mention of PERL_MEM_LOG in perlrun.pod

Specify in Makefile what generates pod/perluniprops.pod

Fix Win32 makefiles as per f86864acbf97469fd9e5d5233d51ff743f4d8d6e

Remove generated file pod/perluniprops.pod from the source distribution

mktables not run unless needed

record version of Intel C during configure

Currently building with icc records an empty ccversion, the attached
change fixes that.

Tony

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

* Remove some end-of-line whitespace from perlebcdic

* Convert some pod files to UTF-8 (like all of the other pod files)

* Reword the sentence about checking unlink failures

Fix code-before-declaration error on Win32 with recent VC++ compilers

(We only #define SET_INVALID_PARAMETER_HANDLER for VC2005 and later, so
users of earlier compilers will not have seen this error)

Upgrade to threads 1.75

Don't suggest Mail::Send isn't availble if we're ignoring it

Move change c35076938c7236fb into embed.fnc, from the generated file proto.h

Added a perltodo about enabling/disabling individual warnings

Fix mktables calls in Win32 makefiles

Subject: Re: Smoke [5.11.2] v5.11.2-42-g02b203e FAIL(M) MSWin32 WinXP/.Net SP3 (x86/2 cpu)
Date: Mon, 23 Nov 2009 11:02:17 +0000
Message-ID: <20091123110217.GU2582@plum.flirble.org>

Make unicode semantics the default

disable Module::Build bundle_inc.t

More info about camel in perlrepository.pod

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

fix logic for a workaround in POSIX.xs

There is the assignment "RETVAL = 0" to silence the compiler in case
it incorrectly reports return of an undefined value.
This assignment should be present whenever *any* of the symbols is
not defined.  But the code was written so that the assignment was
compiled only when *none* of the symbols was defined.

Use hex instead of octal for larger code points

Add also mktables.lst to gitignore, since it's generated

Add new pod file to various lists and makefiles

Simplify title of perluniprops

The man standard requires the title to be on one line.

Add deprecation of goto to inner scopes to perldelta

Add Karl's text describing his Unicode property changes to perldelta

Merge branch 'mktables' into blead

Remove mktables.lst, since it's generated

Add new generated file lib/unicore/Heavy.pl to gitignore

Also remove from there files that are no longer generated by mktables.

Fix plan syntax in TAP output

mktables revamp

Fix unindented code in perlfunc after 5ed4f2ecec5b66feed399b9a24471aeac4242cdb

Resort MANIFEST - Latest Module::Build MANIFEST update broke sorting.

Add new warning introduced by last patch in perldiag.

deprecate "goto" to jump into a construct

Force OP_REQUIRE to scalar context at the end of ck_require and don't let it become void context. Fixes problem with require not always being in scalar context.

Add a TODO test for "require" always being in scalar context.

Make -UDEBUGGING the default on VMS for 5.12.0.

Like it has been everywhere else for ages and ages.  Also make
command-line selection of -UDEBUGGING and -DDEBUGGING work in
configure.com; before the only way to turn it off was by saying
no in answer to the interactive question.

Perl_pad_sv and Perl_hv_assert don't exist without -DDEBUGGING.

So let's not make them part of the API when they don't exist.

Declaration nit in vms.c's Perl_my_fwrite.

Don't use register storage class for something we later take the
address of.

Update generated Module::Build::ConfigData

Avoid logical name conflicts in File::Path::_rmtree on VMS.

already submitted upstream at:

http://rt.cpan.org/Public/Bug/Display.html?id=51588

Always add a manifest resource to perl.exe to specify the <trustInfo>
settings for Windows Vista and later.  Without this setting Windows
will treat perl.exe as a legacy application and apply various
heuristics like redirecting access to protected file system areas
(like the "Program Files" folder) to the users "VirtualStore"
instead of generating a proper "permission denied" error.

For VC8 and VC9 this manifest setting is automatically generated by
the compiler/linker (together with the binding information for their
respective runtime libraries); for all other compilers we need to
embed the manifest resource explicitly in the external resource file.

This change also requests the Microsoft Common-Controls version 6.0
(themed controls introduced in Windows XP) via the dependency list
in the assembly manifest.  For VC8 and VC9 this is specified using the
/manifestdependency linker commandline option instead.

quick-fix broken Module::Build::Platforms::VMS

Update count

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

Fix bug #68260

File::Find was not resolving paths of the form "/..////../" correctly.
Fixed by adding a quantifier to the substitution parameter in
contract_name().

Parse embed.fnc first, then cross-check consistency between it and =for apidoc

Synchronise "may change"/"experimental" flags between embed.fnc and =for apidoc

Merge near-duplicate code for 'api' and 'guts'.

Merge %apidocs and %gutsdocs into $docs{api} and $docs{guts}.

Clarify docs for implicit "next" on all "when" blocks

Missing static storage class for some new static functions

Fix up the NAME in perl5112delta.pod

(It needs to be this way to make search.cpan.org generate the right links)
This step is already described in release_managers_guide.pod.

Always bless filehandles into IO::Handle:: - don't try for FileHandle:: first.

Given that FileHandle "is now a front-end to the IO::* classes" it seems
wasted effort to indirect through it.

Updated Module::CoreList Changes etc for a CPAN release

create perl5113delta

Fix perldelta_template extension

Tiny English fix

* Note that unlink sets $! on failure.

The docs to unlink didn't explicitly note that it set $! on
failure, unlike the docs on some other system calls do.

While I was in there, I cleansed the entry a little and
added an example of unlinking files one-by-one to find
the ones that fail.

Modern Perl fix: let's call a glob a glob() and not a <*>.

This problem was noted on the Perl Beginner's list:

http://www.nntp.perl.org/group/perl.beginners/2009/11/msg110062.html

* Fixing inconsistent use of tabs in perlfunc. Now it's spaces spaces spaces.

* FAQ sync for Nov blead release

This comes from 028b6d17a07335707c2b234cb69ac4051ed48435 in
git@github.com:briandfoy/perlfaq.git

* Fixed sort example using =(\d+)

The example wanted to sort a list like qw(=1 =2 =a =3 =d). One
example tried to be clever with array indices and precomputed
an array in @nums. However, it forgot to leave holes for the
elements where it could not extract a run of digits. Once the
indices were misaligned, the sort didn't give the right answer.

I know you can read the patch, but since I fixed whitespace too,
a simple diff gives you a lot of output. The old example had:

    for (@old) {
        push @nums, /=(\d+)/;
        push @caps, uc($_);
    }

The new one keeps the indices aligned by using undef when the
match failed:

    for (@old) {
        push @nums, ( /=(\d+)/ ? $1 : undef );
        push @caps, uc($_);
    }

This issue was reported on Stackoverflow:

http://stackoverflow.com/questions/1754441

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

Module::CoreList updated for the real 5.11.2, including Module::Build changes

Convert the shell pipeline for the list of committers to a Perl 1 liner.

To me it feels better to promote our own dogfood.

The lexer API is very exciting, but only mention it once

Run Porting/podtidy

Add details of module updates

Delete a few more sections

Delete sections, add a bunch more

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

Sync AUTHORS and Porting/checkAUTHORS.pl with Changelogs

Mention how to generate stats for the Acknowledgements

Fix POD: C<...->...> => C<< ...-> ... >>

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

Updated Module::Build to 0.35_09

0.35_09 - Thu Nov 19 01:30:42 EST 2009

 Bug fixes:

 - The DB package should not be included in 'provides' in META files
   [David Golden]

 - Fixed t/xs.t build failures in bleadperl for noexec temp directories
   [Nicholas Clark]

 - Adjusted order of @INC in resume() (fixes par.t, ppm.t, xs.t fails):
   @INC = @new_additions_to_inc, @saved_additions_to_inc, @default_inc
   [David Golden]

 - Skip bundle_inc.t tests if bundled Module::Build for test can't be
   tweaked (Works around test crashes on Win2) [David Golden]

 Other:

 - 'C_support' is no longer an optional feature.  Modern ExtUtils::CBuilder
   and ExtUtils::ParseXS added to the 'requires' list.  This ensures that
   upgrading Module::Build will upgrade this critical modules.
   ExtUtils::CBuilder no longer requires a compiler, so it is "safe" to
   require.

Fill in some details about the release

Tweak tense, remove double spaces

Bump version of Module::CoreList to 2.23

Abolish xio_lines from struct PVIO - store IoLINES() in the IVX slot.

PVIOs don't use the SvIVX slot.

lexer API fixes

The attached patch contains these fixes for the lexer API work:

* fix MinGW-revealed problem in BOM logic (replacing Jan's patch)
* fix warnings from t/op/incfilter.t
* probably fix g++ failure due to goto bypassing initialisation
* perl5112delta update

-zefram

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

minitest fixes

Attached patch fixes two small bugs that interfere with minitest.

-zefram

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