On Sun, Jan 12, 2020 at 6:41 PM Karl Berry <k...@freefriends.org> 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!