diff options
| author | Jacob Keller <jacob.e.keller@intel.com> | 2014-09-08 16:55:08 -0700 |
|---|---|---|
| committer | Jacob Keller <jacob.e.keller@intel.com> | 2014-09-08 17:06:59 -0700 |
| commit | 1f7a45b8a148056606cadc7281c4926bb80a6a4c (patch) | |
| tree | 6ca2868d1e3fdefc17ca2b385cc4aec2b14cff67 | |
| parent | a57817f4dc7a6eb9cca21b4197a0df332c7d78a0 (diff) | |
| download | aiaiai-1f7a45b8a148056606cadc7281c4926bb80a6a4c.tar.gz | |
aiaiai: test all patches when testing bisection
Instead of only testing until a patch series fails, we should test every
patch. This provides more information, and does not seriously harm the
output. To take advantage of this, we also have to modify our check for
"base commit failed but it is fixed in the first patch". We simply say
that the base commit failed and is fixed as long as the series has no
failures.
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
| -rwxr-xr-x | helpers/aiaiai-test-bisectability | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/helpers/aiaiai-test-bisectability b/helpers/aiaiai-test-bisectability index 66d5cc5..b5e0f43 100755 --- a/helpers/aiaiai-test-bisectability +++ b/helpers/aiaiai-test-bisectability @@ -251,6 +251,7 @@ if build_failed "$base_stderr_log"; then fi n=0 +series_failed_bisect= while [ "$n" -lt "$patch_cnt" ]; do formail +$n -1 -s < "$mbox" > "$tmp_mbox" compile_patch "$tmp_mbox" "$stdout_log" "$stderr_log" @@ -264,10 +265,10 @@ while [ "$n" -lt "$patch_cnt" ]; do printf "\n\n" fi build_failure "$defconfig_orig" "HEAD" "patch #$n"< "$stderr_log" - exit 0 + series_failed_bisect="yes" fi done -if [ -n "$base_commit_failed" ]; then - printf "%s\n" "Base commit build failed, but the first patch fixed the build failure." +if [ -n "$base_commit_failed" ] && [ -z "$series_failed_bisect" ]; then + printf "%s\n" "Base commit build failed, but the series fixed the build failure." fi |