clean up debugger environment before testing with it

The perl debugger uses various environment variables to configure
itself, or pick which debugger to use.  Clean up those environment
variables before activating the debugger so that it behaves predictably.

Fixes RT#127516

Adjust gitignore - we do track inc/ as of 58cd9d45

Version 0.27

Work around the upcoming -Dfortify_inc :/

Switch to EU::HC (with an inc::latest-like approach)

Implement semi-elaborate Makefile.PL boilerplate for EU::HC use. While at
this point EU::HC seems rather stable, it still hasn't settled enough for
complete comfort.

This allow loading of "more-recent-than INC", in order to decrease amount of
needed release churn to work around newly-found problems.

Fix silly inconsistency in docs from 253c58bc

Remove incorrect SKIP for 5.8.1

The reason this test failed is srictly the construct of the test itself, and
not part of namespace::clean's functionality. reqiure() in list-assign context
combined with an @INC coderef hook would result in visible stack corruption
as per http://www.nntp.perl.org/group/perl.perl5.porters/2003/10/msg83192.html

Allow packages to be cleaned multiple times

Originally the functionality was implmented via Filter::Util::Call, but when
it was removed in [1] the "one per scope" logic remained. Given the reliance
on B::Hooks::EndOfScope none of this is longer required.

[1] https://metacpan.org/diff/file?target=PHAYLON%2Fnamespace-clean-0.07%2Flib%2Fnamespace%2Fclean.pm&source=PHAYLON%2Fnamespace-clean-0.06%2Flib%2Fnamespace%2Fclean.pm

Switch to GH url in meta for metacpan's sake

See how it behaves when url/web mismatch during a trial, will fix properly
if a priblem

Restrict debugger workaround to when DB::sub will be used

The issues with the debugger only arise when DB::sub will be called.
Since the workaround requires XS modules, avoid it unless actually
needed.  This avoids XS prereqs when using modules like Devel::Confess
that use debugger features but don't require the workaround.

Ditch done_testing - less Test::More upgrade-churn

Make sure t/10-pure-perl.t runs are identical regardless of invocation

Previously `prove -lr ./t` would trip things up

Get tests running under -T (cosmetics only)

Make sure $] checks run correctly (another thing my smokers didn't catch)

Fix thinko from 18377f5b (RT#107813)

Version 0.26

Explicitly document the late runtime binding of `sort SUBNAME ...`

Rewrap Changes to be more compatible with metacpan

This commit should have never existed - the way metacpan tries to parse the
changelogs is broken in the first place, on a very deep conceptual level.
Nevertheless, compromising for the sake of readability.

Switch around the preference of S::U / S::N

This makes the code more complex, but will save on loading the rather heavy
B set of modules in case Sub::Util is in fact available

Streamline meta etc

No idea why $DebuggerFixup was originally introduced - fold it back

Also clarify Father C's original comment

Make sure to exercise P::S::PP under t/10-pure-perl.t

Reign in XS portions needed for certain perls under debugging, several things:

- Do not rely on Sub::Identify at all (it is too finicky) - either use
  Sub::Util::subname if already loaded, or just use B (RT#96945)

- Allow either Sub::Name or Sub::Util to be used for namings

- Only add Sub::Name to the install list if neither Sub::* is available

- Clarify constant names, move them to _Util, make sure we use them everywhere

- Make sure that t/07-debugger.t runs on all perls, including the ones not
  needing the XS dance

Add an once-and-for-all workaround for Package::Stash::XS + =< 5.8.6

The simple workaround in ac5c00e9 got broken by P::S::XS 0.36. Since at this
point this is more or less a pattern, add a ridiculous-ish workaround forcing
Package::Stash::PP-only on perls under 5.8.7.

While not ideal, the benefits of ::XS are virtually invisible in this use case
and moreover there is work underway to fix ::XS once and for all. Thus the
commit is considered a viable temporary measure to stave off further errors.

Eject pod to the end of the file

The current file structure is very chaotic, electing to move POD out out of the
way before doing any further changes

`git blame -M` is smart enough to preserve authorship etc in such cases

Version 0.25

correct EU::CBuilder check

Reorganize PP test to fail comprehensibly on failed prereqs

Prompted by http://www.cpantesters.org/cpan/report/33f54a42-3f59-11e2-b03f-503131b64f85

Version 0.24

B::H::EOS is now PP-capable on its own

Fix tests on space-containing paths RT#77528

Properly skip debugger test when optional deps not available

Version 0.23

Bump dep to B::Hooks::EndOfScope 0.10

Fix failures on 5.13.6, RT#74683, Zefram++

Version 0.22

Whoops, more hidden packages

Simplify even more - we don't care about context anymore

Even more simplifications of the PP callback

Another devrel to see how happy smokers are

Run the PP tests with or without Devek::Hide

The real fix for RT#73402 (abaondon tie %^H entirely), FC++

Disable the debugger fixup entirely on perl 5.15.5+

Do not rerun PP tests twice if the envvar is set

Trial release to smoketest the hinthash handling changes

Fix careless handling of the hinthash in PP mode (RT#73402)

Add an envvar to make it easier to smoke the PP codepath
Still use Devel::Hide for the actual testing, as to not mask breakage
of the B::H::EOS detector

Install the debugger fixup deps iff can_cc (RT#72368)

P::S::XS buggery strikes again

Fix all debugger issues from perl 5.8.1 on FC++ (RT#69862)

ignore mymetas

Add dropped NAME pod section (RT#70259)

Version 0.21

Build correct meta, not including B::H::EOS

Strictures and changelogging

Try harder to detect tie(%^H) clashes

Clarification as per doy's request

Fix borked git url

Trial release preparing for pure-perl-capable 0.21

De-dzilify. This may look odd and borderline revolting - read on

Currently available dzil plugins do not offer the dynconfig flexibility
that this module requires. Instead of taking a 2 day course in dzil
internals, I opted for using a tool I know to accomplish the same task.

I have absolutely no problem with someone reverting this commit and re-
doing things in dzil, as long as the resulting Makefile.PL is functionally
equivalent. Some points:

* Dzil's autorequires can't possibly understand what an optional dep is
  This is an easy fix - just specify all deps manually

* There is no plugin for "require list depending on can_cc". Package::Stash
  does something to the effect, but with some sort of hack I am not entirely
  capable of comprehending

* There is no plugin to "require list depending on smoking env", nor there
  is an easy mechanism (that I see) of combining it with the above

Pure-perlize

Spellfixoring RT#54388

This used to fail with old V::M, add it to make sure it behaves under PP

Moar stresstesting

Only ask for debugger bits when debugger detected

Mini $RemoveSubs refactoring: less loops, less concat

Small refactoring in $RemoveSubs in the cleanup of the GLOB entry:
- grep for undefined symbols just once
- less string concatenation

Version 0.20

Bump Package::Stash dependency to 0.22

This pulls in a bugfix in Package::Stash::XS 0.19.

Version 0.19

Remove back-compat for old Package::Stash versions

0.18 is stable and doesn't have the deprecation warnings for the old API
methods. That makes it safe for us to depend on it because the upgrade won't
break existing code using it, such as Moose.

work with new package::stash api

don't rely on package::stash's remove_package_symbol implementation

Version 0.18

vivify the storage var manually

Version 0.17

Allow the debugger test to be skipped early

Loading the debugger often goes wrong, so it's important we skip the
test before it gets loaded.

Version 0.16

Stop declaring unused variables

Credit doy

Convert to Dist::Zilla

changelog and version bump

s/Stash::Manip/Package::Stash/g

adjust deps and add changelog

start converting to use Stash::Manip

Version 0.14.

Turn the very faily debugger test into an author test.

Remove auto_install for mbm as using git is 'too much yak hair'.

Version 0.13.

Skip failing debugger tests on 5.8.8 and older.

Version 0.12.

Remove auto-generated README.

Stop relying on stash entries always being upgraded into real GVs.

work around $DB::sub

The variable is a string which is dereferenced as a sub that no longer
exists.

When we delete the glob where the sub was defined, we therefore alias it
to "namespace::clean::deleted::$fq" so that the debugger can still
dereference a symbolic ref and get back a working subroutine

Fix restoring of non-code symbols when cleaning.

Curtesy of Ben Morrows.

Add testcase for clobbering globs when trying to restore them.

Courtesy of Ben Morrows.

Version 0.11.

Add reference to RT ticket the recent changes fix.

updated README

removal subroutine named clean_subroutines, not remove_subroutines

implemented -cleanee option plus an external callable remove_subroutines class method to provide functionaltiy as per RT#41850