Skip to content

Commit

Permalink
ls-files: fix broken --no-empty-directory
Browse files Browse the repository at this point in the history
Commit ce8e880 converted ls-files to use parseopt; the
--no-empty-directory option was converted as an
OPT_BIT for "empty-directory" to set the
DIR_HIDE_EMPTY_DIRECTORY flag. However, that makes it do the
opposite of what it should: --empty-directory would hide,
but --no-empty-directory would turn off hiding.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Jeff King authored and Junio C Hamano committed Mar 8, 2009
1 parent 8a3b25d commit 2fb6d6d
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 3 deletions.
4 changes: 2 additions & 2 deletions builtin-ls-files.c
Original file line number Diff line number Diff line change
Expand Up @@ -454,8 +454,8 @@ int cmd_ls_files(int argc, const char **argv, const char *prefix)
OPT_BIT(0, "directory", &dir.flags,
"show 'other' directories' name only",
DIR_SHOW_OTHER_DIRECTORIES),
OPT_BIT(0, "empty-directory", &dir.flags,
"list empty directories",
OPT_BIT(0, "no-empty-directory", &dir.flags,
"don't show empty directories",
DIR_HIDE_EMPTY_DIRECTORIES),
OPT_BOOLEAN('u', "unmerged", &show_unmerged,
"show unmerged files in the output"),
Expand Down
14 changes: 13 additions & 1 deletion t/t3000-ls-files-others.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,13 @@ filesystem.
path2/file2 - a file in a directory
path3-junk - a file to confuse things
path3/file3 - a file in a directory
path4 - an empty directory
'
. ./test-lib.sh

date >path0
ln -s xyzzy path1
mkdir path2 path3
mkdir path2 path3 path4
date >path2/file2
date >path2-junk
date >path3/file3
Expand All @@ -28,13 +29,16 @@ git update-index --add path3-junk path3/file3
cat >expected1 <<EOF
expected1
expected2
expected3
output
path0
path1
path2-junk
path2/file2
EOF
sed -e 's|path2/file2|path2/|' <expected1 >expected2
cat <expected2 >expected3
echo path4/ >>expected2

test_expect_success \
'git ls-files --others to show output.' \
Expand All @@ -53,4 +57,12 @@ test_expect_success \
'git ls-files --others --directory should not get confused.' \
'test_cmp expected2 output'

test_expect_success \
'git ls-files --others --directory --no-empty-directory to show output.' \
'git ls-files --others --directory --no-empty-directory >output'

test_expect_success \
'--no-empty-directory hides empty directory' \
'test_cmp expected3 output'

test_done

0 comments on commit 2fb6d6d

Please sign in to comment.