Re: [glean-dev] --overwrite option

[Allen A. <ak...@po...>]

Good point.  I hadn't considered the debugging process when I originally
chose that behavior.

The reason simple overwrite wasn't allowed was that subsequent runs
might overwrite different files in the results directory, leaving it in
an inconsistent state.  

Johan and I talked about allowing overwrite, and we had decided that the
right behavior was to do just as you did -- delete the entire existing
results directory tree, then create the new one as usual.  But we never
got around to implementing it.

So, bottom line is that I think this is the right approach.  Using
system() is a little sleazy, but not enough to worry about.  There
probably ought to be an assertion in the Windows version of the code to
complain if the overwrite option is used there, since it isn't
implemented.

Allen