[PATCH RESEND] keyword: use wvfs.rmtree to remove kwdemo directory

Christian Ebert blacktrash at gmx.net
Tue May 5 13:23:14 CDT 2015


* Augie Fackler on Tuesday, May 05, 2015 at 10:11:59 -0400
> On Sun, May 03, 2015 at 02:24:59AM +0100, Christian Ebert wrote:
>> * Matt Mackall on Saturday, May 02, 2015 at 16:48:31 -0500
>>> That's quite a scary API, now that you've drawn it to my attention.
>>> Makes it a little too easy to accidentally delete repositories. We
>>> probably shouldn't have a default arg there.. and we might even consider
>>> not accepting empty strings.
>> 
>> I can pass the temp dir as argument explicitly. That works as
>> well in this case. You want me to resend again like that?
> 
> Yes please

It's already in, after I had sent a V2:
http://selenic.com/hg/rev/4ecbd88cde9a

thx

-- 
  Was heißt hier Dogma, ich bin Underdogma!
[ What the hell do you mean dogma, I am underdogma. ]
free movies       --->>> http://www.blacktrash.org/underdogma
http://itunes.apple.com/podcast/underdogma-movies/id363423596


More information about the Mercurial-devel mailing list