On Wed, Mar 18, 2020 at 09:42:45AM -0600, Simon Glass wrote:
At present buildman removes any directory it doesn't intend to write output into. This is overly expansive since if the output directory happens to be somewhere with existing files, they may be removed. Using an existing directory for buildman is not a good practice, but since the result might be catastrophic, it is best to guard against it.
A previous commit[1] fixed this by refusing to write to a subdirectory of the current directory, assumed to have U-Boot source code. But we can do better by only removing directories that look like the ones buildman creates.
Update the code to do this and add a test.
Signed-off-by: Simon Glass sjg@chromium.org
[1] 409fc029c40 tools: buildman: Don't use the working dir as build dir
Applied to u-boot/master, thanks!