summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Jackson <ijackson@chiark.greenend.org.uk>2018-10-13 13:31:52 +0100
committerIan Jackson <ijackson@chiark.greenend.org.uk>2018-10-13 16:46:47 +0100
commit1d5fcc08324200339c57ab8c6cac1c8ad9e9943a (patch)
tree048d929656bbe06e5515154d7e68550c4828887d
parentd48ae0d567fff38618c1fffa5c4265cbfbda8160 (diff)
dgit: Demand that all clean modes are handled by clean_tree_check
We've dealt with all the cases now, so we can add the default. Also, in the test suite. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
-rwxr-xr-xdgit2
-rw-r--r--tests/lib-build-modes1
2 files changed, 2 insertions, 1 deletions
diff --git a/dgit b/dgit
index f5092f5..e104838 100755
--- a/dgit
+++ b/dgit
@@ -6243,6 +6243,8 @@ sub clean_tree_check () {
# deleted. Since we're not, and not using the working tree
# anyway, we can just ignore them - nothing will use them.
} elsif ($cleanmode eq 'none') {
+ } else {
+ confess "$cleanmode ?";
}
}
diff --git a/tests/lib-build-modes b/tests/lib-build-modes
index 78ae236..bc8b2aa 100644
--- a/tests/lib-build-modes
+++ b/tests/lib-build-modes
@@ -161,7 +161,6 @@ bm-compute-expected () {
none|Cnone) ;;
Cdpkg-source*) ;; # handled below
Cgit|Cgit-ff) ;;
- C*) echo "TODO bm eff_cleanmode=$eff_cleanmode" ;;
*) fail "t-compute-expected-run $cleanmode ??" ;;
esac