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 <[email protected]> > > [1] 409fc029c40 tools: buildman: Don't use the working dir as build dir
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature
