2013-12-19 08:08:10 +08:00
|
|
|
#!/bin/sh
|
|
|
|
|
|
|
|
test_description='diff order'
|
|
|
|
|
|
|
|
. ./test-lib.sh
|
|
|
|
|
|
|
|
create_files () {
|
|
|
|
echo "$1" >a.h &&
|
|
|
|
echo "$1" >b.c &&
|
|
|
|
echo "$1" >c/Makefile &&
|
|
|
|
echo "$1" >d.txt &&
|
|
|
|
git add a.h b.c c/Makefile d.txt &&
|
|
|
|
git commit -m"$1"
|
|
|
|
}
|
|
|
|
|
|
|
|
test_expect_success 'setup' '
|
|
|
|
mkdir c &&
|
|
|
|
create_files 1 &&
|
|
|
|
create_files 2 &&
|
|
|
|
|
|
|
|
cat >order_file_1 <<-\EOF &&
|
|
|
|
*Makefile
|
|
|
|
*.txt
|
|
|
|
*.h
|
|
|
|
EOF
|
|
|
|
|
|
|
|
cat >order_file_2 <<-\EOF &&
|
|
|
|
*Makefile
|
|
|
|
*.h
|
|
|
|
*.c
|
|
|
|
EOF
|
|
|
|
|
|
|
|
cat >expect_none <<-\EOF &&
|
|
|
|
a.h
|
|
|
|
b.c
|
|
|
|
c/Makefile
|
|
|
|
d.txt
|
|
|
|
EOF
|
|
|
|
|
|
|
|
cat >expect_1 <<-\EOF &&
|
|
|
|
c/Makefile
|
|
|
|
d.txt
|
|
|
|
a.h
|
|
|
|
b.c
|
|
|
|
EOF
|
|
|
|
|
|
|
|
cat >expect_2 <<-\EOF
|
|
|
|
c/Makefile
|
|
|
|
a.h
|
|
|
|
b.c
|
|
|
|
d.txt
|
|
|
|
EOF
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success "no order (=tree object order)" '
|
|
|
|
git diff --name-only HEAD^..HEAD >actual &&
|
|
|
|
test_cmp expect_none actual
|
|
|
|
'
|
|
|
|
|
diff: let "git diff -O" read orderfile from any file and fail properly
The -O flag really shouldn't silently fail to do anything when given
a path that it can't read from.
However, it should be able to read from un-mmappable files, such as:
* pipes/fifos
* /dev/null: It's a character device (at least on Linux)
* ANY empty file:
Quoting Linux mmap(2), "SUSv3 specifies that mmap() should fail if
length is 0. However, in kernels before 2.6.12, mmap() succeeded in
this case: no mapping was created and the call returned addr. Since
kernel 2.6.12, mmap() fails with the error EINVAL for this case."
We especially want "-O/dev/null" to work, since we will be documenting
it as the way to cancel "diff.orderfile" when we add that.
(Note: "-O/dev/null" did have the right effect, since the existing error
handling essentially worked out to "silently ignore the orderfile". But
this was probably more coincidence than anything else.)
So, lets toss all of that logic to get the file mmapped and just use
strbuf_read_file() instead, which gives us decent error handling
practically for free.
Signed-off-by: Samuel Bronson <naesten@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2013-12-19 08:08:11 +08:00
|
|
|
test_expect_success 'missing orderfile' '
|
|
|
|
rm -f bogus_file &&
|
|
|
|
test_must_fail git diff -Obogus_file --name-only HEAD^..HEAD
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success POSIXPERM,SANITY 'unreadable orderfile' '
|
|
|
|
>unreadable_file &&
|
|
|
|
chmod -r unreadable_file &&
|
|
|
|
test_must_fail git diff -Ounreadable_file --name-only HEAD^..HEAD
|
|
|
|
'
|
|
|
|
|
2016-01-21 19:48:44 +08:00
|
|
|
test_expect_success "orderfile using option from subdir with --output" '
|
|
|
|
mkdir subdir &&
|
|
|
|
git -C subdir diff -O../order_file_1 --output ../actual --name-only HEAD^..HEAD &&
|
|
|
|
test_cmp expect_1 actual
|
|
|
|
'
|
|
|
|
|
2013-12-19 08:08:10 +08:00
|
|
|
for i in 1 2
|
|
|
|
do
|
|
|
|
test_expect_success "orderfile using option ($i)" '
|
|
|
|
git diff -Oorder_file_$i --name-only HEAD^..HEAD >actual &&
|
|
|
|
test_cmp expect_$i actual
|
|
|
|
'
|
diff: let "git diff -O" read orderfile from any file and fail properly
The -O flag really shouldn't silently fail to do anything when given
a path that it can't read from.
However, it should be able to read from un-mmappable files, such as:
* pipes/fifos
* /dev/null: It's a character device (at least on Linux)
* ANY empty file:
Quoting Linux mmap(2), "SUSv3 specifies that mmap() should fail if
length is 0. However, in kernels before 2.6.12, mmap() succeeded in
this case: no mapping was created and the call returned addr. Since
kernel 2.6.12, mmap() fails with the error EINVAL for this case."
We especially want "-O/dev/null" to work, since we will be documenting
it as the way to cancel "diff.orderfile" when we add that.
(Note: "-O/dev/null" did have the right effect, since the existing error
handling essentially worked out to "silently ignore the orderfile". But
this was probably more coincidence than anything else.)
So, lets toss all of that logic to get the file mmapped and just use
strbuf_read_file() instead, which gives us decent error handling
practically for free.
Signed-off-by: Samuel Bronson <naesten@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2013-12-19 08:08:11 +08:00
|
|
|
|
|
|
|
test_expect_success PIPE "orderfile is fifo ($i)" '
|
|
|
|
rm -f order_fifo &&
|
|
|
|
mkfifo order_fifo &&
|
|
|
|
{
|
|
|
|
cat order_file_$i >order_fifo &
|
|
|
|
} &&
|
|
|
|
git diff -O order_fifo --name-only HEAD^..HEAD >actual &&
|
|
|
|
wait &&
|
|
|
|
test_cmp expect_$i actual
|
|
|
|
'
|
2013-12-19 08:08:12 +08:00
|
|
|
|
|
|
|
test_expect_success "orderfile using config ($i)" '
|
|
|
|
git -c diff.orderfile=order_file_$i diff --name-only HEAD^..HEAD >actual &&
|
|
|
|
test_cmp expect_$i actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success "cancelling configured orderfile ($i)" '
|
|
|
|
git -c diff.orderfile=order_file_$i diff -O/dev/null --name-only HEAD^..HEAD >actual &&
|
|
|
|
test_cmp expect_none actual
|
|
|
|
'
|
2013-12-19 08:08:10 +08:00
|
|
|
done
|
|
|
|
|
2014-01-21 00:20:39 +08:00
|
|
|
test_expect_success 'setup for testing combine-diff order' '
|
|
|
|
git checkout -b tmp HEAD~ &&
|
|
|
|
create_files 3 &&
|
|
|
|
git checkout master &&
|
|
|
|
git merge --no-commit -s ours tmp &&
|
|
|
|
create_files 5
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success "combine-diff: no order (=tree object order)" '
|
|
|
|
git diff --name-only HEAD HEAD^ HEAD^2 >actual &&
|
|
|
|
test_cmp expect_none actual
|
|
|
|
'
|
|
|
|
|
|
|
|
for i in 1 2
|
|
|
|
do
|
|
|
|
test_expect_success "combine-diff: orderfile using option ($i)" '
|
|
|
|
git diff -Oorder_file_$i --name-only HEAD HEAD^ HEAD^2 >actual &&
|
|
|
|
test_cmp expect_$i actual
|
|
|
|
'
|
|
|
|
done
|
|
|
|
|
2013-12-19 08:08:10 +08:00
|
|
|
test_done
|