]> git.dujemihanovic.xyz Git - u-boot.git/commit
buildman: Be more selective about which directories to remove
authorSimon Glass <sjg@chromium.org>
Wed, 18 Mar 2020 15:42:45 +0000 (09:42 -0600)
committerTom Rini <trini@konsulko.com>
Sat, 11 Apr 2020 01:21:06 +0000 (21:21 -0400)
commit925f6adfa53c2a80a1d7ce731e628cded13f5363
tree936dab0a376597af9a3a815ccf0c345184928be6
parent7beb43c9807159463ad6dd2a29517d4cee1e7478
buildman: Be more selective about which directories to remove

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
tools/buildman/builder.py
tools/buildman/test.py