[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#39078: Path.pm change -> deltree.pl -> t/uninstall-fail failure
From: |
Jim Meyering |
Subject: |
bug#39078: Path.pm change -> deltree.pl -> t/uninstall-fail failure |
Date: |
Thu, 16 Jan 2020 20:05:02 -0800 |
On Sun, Jan 12, 2020 at 6:41 PM Karl Berry <address@hidden> wrote:
>
> Regarding the new failure of Path.pm:rmtree to do removals, here is the
> change I had in mind to use chmod and rm instead. The previously-"ERROR"
> tests (uninstall-fail and instspc) work for me with this change.
>
> Before I bother spending time on all the related overhead changes, any
> objections, improvements, comments on this approach?
>
> The rest of the patch is unrelated to the functional change: checking
> for any of the elements existing after the purported removal. In my
> testing I found it useful for this to be reported, and the abort to
> occur, right away.
>
> Then, the recursive ls report on failure might be overkill, but I found
> it useful to understand the failing situation, so here it is for
> consideration. (I guess it would be better to prepend the prefix to
> every line of the ls output, which would be easy enough.) --thanks, karl.
Hi Karl,
I like your patch. With our without prefixes.
Thanks for working on this!