Merge branch 'maint-1.6.0' into maint-1.6.1
* maint-1.6.0: bisect: fix another instance of eval'ed string bisect: fix quoting TRIED revs when "bad" commit is also "skip"ped Support "\" in non-wildcard exclusion entries Conflicts: git-bisect.sh
This commit is contained in:
commit
bf0fe35c93
2
dir.c
2
dir.c
@ -134,7 +134,7 @@ int match_pathspec(const char **pathspec, const char *name, int namelen, int pre
|
|||||||
|
|
||||||
static int no_wildcard(const char *string)
|
static int no_wildcard(const char *string)
|
||||||
{
|
{
|
||||||
return string[strcspn(string, "*?[{")] == '\0';
|
return string[strcspn(string, "*?[{\\")] == '\0';
|
||||||
}
|
}
|
||||||
|
|
||||||
void add_exclude(const char *string, const char *base,
|
void add_exclude(const char *string, const char *base,
|
||||||
|
@ -284,62 +284,74 @@ filter_skipped() {
|
|||||||
_skip="$2"
|
_skip="$2"
|
||||||
|
|
||||||
if [ -z "$_skip" ]; then
|
if [ -z "$_skip" ]; then
|
||||||
eval "$_eval"
|
eval "$_eval" | {
|
||||||
|
while read line
|
||||||
|
do
|
||||||
|
echo "$line &&"
|
||||||
|
done
|
||||||
|
echo ':'
|
||||||
|
}
|
||||||
return
|
return
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# Let's parse the output of:
|
# Let's parse the output of:
|
||||||
# "git rev-list --bisect-vars --bisect-all ..."
|
# "git rev-list --bisect-vars --bisect-all ..."
|
||||||
eval "$_eval" | while read hash line
|
eval "$_eval" | {
|
||||||
do
|
VARS= FOUND= TRIED=
|
||||||
case "$VARS,$FOUND,$TRIED,$hash" in
|
while read hash line
|
||||||
# We display some vars.
|
do
|
||||||
1,*,*,*) echo "$hash $line" ;;
|
case "$VARS,$FOUND,$TRIED,$hash" in
|
||||||
|
1,*,*,*)
|
||||||
# Split line.
|
# "bisect_foo=bar" read from rev-list output.
|
||||||
,*,*,---*) ;;
|
echo "$hash &&"
|
||||||
|
;;
|
||||||
# We had nothing to search.
|
,*,*,---*)
|
||||||
|
# Separator
|
||||||
|
;;
|
||||||
,,,bisect_rev*)
|
,,,bisect_rev*)
|
||||||
echo "bisect_rev="
|
# We had nothing to search.
|
||||||
|
echo "bisect_rev= &&"
|
||||||
VARS=1
|
VARS=1
|
||||||
;;
|
;;
|
||||||
|
|
||||||
# We did not find a good bisect rev.
|
|
||||||
# This should happen only if the "bad"
|
|
||||||
# commit is also a "skip" commit.
|
|
||||||
,,*,bisect_rev*)
|
,,*,bisect_rev*)
|
||||||
echo "bisect_rev=$TRIED"
|
# We did not find a good bisect rev.
|
||||||
|
# This should happen only if the "bad"
|
||||||
|
# commit is also a "skip" commit.
|
||||||
|
echo "bisect_rev='$TRIED' &&"
|
||||||
VARS=1
|
VARS=1
|
||||||
;;
|
;;
|
||||||
|
|
||||||
# We are searching.
|
|
||||||
,,*,*)
|
,,*,*)
|
||||||
|
# We are searching.
|
||||||
TRIED="${TRIED:+$TRIED|}$hash"
|
TRIED="${TRIED:+$TRIED|}$hash"
|
||||||
case "$_skip" in
|
case "$_skip" in
|
||||||
*$hash*) ;;
|
*$hash*) ;;
|
||||||
*)
|
*)
|
||||||
echo "bisect_rev=$hash"
|
echo "bisect_rev=$hash &&"
|
||||||
echo "bisect_tried=\"$TRIED\""
|
echo "bisect_tried='$TRIED' &&"
|
||||||
FOUND=1
|
FOUND=1
|
||||||
;;
|
;;
|
||||||
esac
|
esac
|
||||||
;;
|
;;
|
||||||
|
,1,*,bisect_rev*)
|
||||||
# We have already found a rev to be tested.
|
# We have already found a rev to be tested.
|
||||||
,1,*,bisect_rev*) VARS=1 ;;
|
VARS=1
|
||||||
,1,*,*) ;;
|
;;
|
||||||
|
,1,*,*)
|
||||||
# ???
|
;;
|
||||||
*) die "filter_skipped error " \
|
*)
|
||||||
"VARS: '$VARS' " \
|
# Unexpected input
|
||||||
"FOUND: '$FOUND' " \
|
echo "die 'filter_skipped error'"
|
||||||
"TRIED: '$TRIED' " \
|
die "filter_skipped error " \
|
||||||
"hash: '$hash' " \
|
"VARS: '$VARS' " \
|
||||||
"line: '$line'"
|
"FOUND: '$FOUND' " \
|
||||||
;;
|
"TRIED: '$TRIED' " \
|
||||||
esac
|
"hash: '$hash' " \
|
||||||
done
|
"line: '$line'"
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
done
|
||||||
|
echo ':'
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
exit_if_skipped_commits () {
|
exit_if_skipped_commits () {
|
||||||
|
@ -19,6 +19,9 @@ do
|
|||||||
>$dir/a.$i
|
>$dir/a.$i
|
||||||
done
|
done
|
||||||
done
|
done
|
||||||
|
>"#ignore1"
|
||||||
|
>"#ignore2"
|
||||||
|
>"#hidden"
|
||||||
|
|
||||||
cat >expect <<EOF
|
cat >expect <<EOF
|
||||||
a.2
|
a.2
|
||||||
@ -42,6 +45,9 @@ three/a.8
|
|||||||
EOF
|
EOF
|
||||||
|
|
||||||
echo '.gitignore
|
echo '.gitignore
|
||||||
|
\#ignore1
|
||||||
|
\#ignore2*
|
||||||
|
\#hid*n
|
||||||
output
|
output
|
||||||
expect
|
expect
|
||||||
.gitignore
|
.gitignore
|
||||||
@ -79,9 +85,10 @@ test_expect_success \
|
|||||||
>output &&
|
>output &&
|
||||||
test_cmp expect output'
|
test_cmp expect output'
|
||||||
|
|
||||||
cat > excludes-file << EOF
|
cat > excludes-file <<\EOF
|
||||||
*.[1-8]
|
*.[1-8]
|
||||||
e*
|
e*
|
||||||
|
\#*
|
||||||
EOF
|
EOF
|
||||||
|
|
||||||
git config core.excludesFile excludes-file
|
git config core.excludesFile excludes-file
|
||||||
|
@ -224,6 +224,31 @@ test_expect_success 'bisect skip: cannot tell between 2 commits' '
|
|||||||
fi
|
fi
|
||||||
'
|
'
|
||||||
|
|
||||||
|
# $HASH1 is good, $HASH4 is both skipped and bad, we skip $HASH3
|
||||||
|
# and $HASH2 is good,
|
||||||
|
# so we should not be able to tell the first bad commit
|
||||||
|
# among $HASH3 and $HASH4
|
||||||
|
test_expect_success 'bisect skip: with commit both bad and skipped' '
|
||||||
|
git bisect start &&
|
||||||
|
git bisect skip &&
|
||||||
|
git bisect bad &&
|
||||||
|
git bisect good $HASH1 &&
|
||||||
|
git bisect skip &&
|
||||||
|
if git bisect good > my_bisect_log.txt
|
||||||
|
then
|
||||||
|
echo Oops, should have failed.
|
||||||
|
false
|
||||||
|
else
|
||||||
|
test $? -eq 2 &&
|
||||||
|
grep "first bad commit could be any of" my_bisect_log.txt &&
|
||||||
|
! grep $HASH1 my_bisect_log.txt &&
|
||||||
|
! grep $HASH2 my_bisect_log.txt &&
|
||||||
|
grep $HASH3 my_bisect_log.txt &&
|
||||||
|
grep $HASH4 my_bisect_log.txt &&
|
||||||
|
git bisect reset
|
||||||
|
fi
|
||||||
|
'
|
||||||
|
|
||||||
# We want to automatically find the commit that
|
# We want to automatically find the commit that
|
||||||
# introduced "Another" into hello.
|
# introduced "Another" into hello.
|
||||||
test_expect_success \
|
test_expect_success \
|
||||||
|
Loading…
Reference in New Issue
Block a user