summaryrefslogtreecommitdiff
path: root/object-store.h
diff options
context:
space:
mode:
authorLibravatar Matheus Tavares <matheus.bernardino@usp.br>2021-02-22 22:10:35 -0300
committerLibravatar Junio C Hamano <gitster@pobox.com>2021-02-24 12:14:51 -0800
commit9ebd7fe15869910c81c42290bd80296664597dec (patch)
tree468e6cdeefb78a29eff87e4e7d649cb188a5f286 /object-store.h
parentadd: mark --chmod error string for translation (diff)
downloadtgif-9ebd7fe15869910c81c42290bd80296664597dec.tar.xz
add: propagate --chmod errors to exit status
If `add` encounters an error while applying the --chmod changes, it prints a message to stderr, but exits with a success code. This might have been an oversight, as the command does exit with a non-zero code in other situations where it cannot (or refuses to) update all of the requested paths (e.g. when some of the given paths are ignored). So make the exit behavior more consistent by also propagating --chmod errors to the exit status. Note: the test "all statuses changed in folder if . is given" uses paths added by previous test cases, some of which might be symbolic links. Because `git add --chmod` will now fail with such paths, this test would depend on whether all the previous tests were executed, or only some of them. Avoid that by running the test on a fresh repo with only regular files. Signed-off-by: Matheus Tavares <matheus.bernardino@usp.br> Reviewed-by: Taylor Blau <me@ttaylorr.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'object-store.h')
0 files changed, 0 insertions, 0 deletions