Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Clarify API for dir.[ch] and unpack-trees.[ch] -- mark relevant fields as internal #1149

Closed
wants to merge 13 commits into from
Closed

Clarify API for dir.[ch] and unpack-trees.[ch] -- mark relevant fields as internal #1149

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
7bbc457
t2021: fix platform-specific leftover cruft
newren Feb 26, 2023
8ffdb6c
unpack-trees: heed requests to overwrite ignored files
newren Feb 25, 2023
879a93a
dir: separate public from internal portion of dir_struct
newren Oct 6, 2021
4ce9fae
dir: add a usage note to exclude_per_dir
newren Jan 17, 2022
1234440
dir: mark output only fields of dir_struct as such
newren Sep 2, 2022
4e86e39
unpack-trees: clean up some flow control
newren Feb 13, 2022
cd3d489
sparse-checkout: avoid using internal API of unpack-trees
newren Feb 13, 2022
09140cb
sparse-checkout: avoid using internal API of unpack-trees, take 2
newren Feb 23, 2023
27f2d47
unpack_trees: start splitting internal fields from public API
newren Jan 17, 2022
4236c0d
unpack-trees: mark fields only used internally as internal
newren Jan 17, 2022
76f4a54
unpack-trees: rewrap a few overlong lines from previous patch
newren Jan 17, 2022
ee36935
unpack-trees: special case read-tree debugging as internal usage
newren Jan 17, 2022
6575c00
unpack-trees: add usage notices around df_conflict_entry
newren Jan 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 4 additions & 1 deletion t/t2021-checkout-overwrite.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,13 @@ test_expect_success 'checkout commit with dir must not remove untracked a/b' '

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Git mailing list, Derrick Stolee wrote (reply to this):

On 2/27/2023 10:28 AM, Elijah Newren via GitGitGadget wrote:
> From: Elijah Newren <newren@gmail.com>

>  test_expect_success SYMLINKS 'the symlink remained' '
>  
> -	test_when_finished "rm a/b" &&
>  	test -h a/b
>  '
>  
> +test_expect_success 'cleanup after previous symlink tests' '
> +	rm a/b
> +'

I was confused why this worked without "rm -f a/b" and it seems
the path exists in all cases, it's just a symlink on the filesystem
in the case of the SYMLINKS prerequisite.

Thanks,
-Stolee

test_expect_success SYMLINKS 'the symlink remained' '

test_when_finished "rm a/b" &&
test -h a/b
'

test_expect_success 'cleanup after previous symlink tests' '
rm a/b
'

test_expect_success SYMLINKS 'checkout -f must not follow symlinks when removing entries' '
git checkout -f start &&
mkdir dir &&
Expand Down