Re: [PATCHv11 02/11] receive-pack.c: die instead of error in case of possible future bug

2015-01-06 Thread Jonathan Nieder
Stefan Beller wrote: > +++ b/builtin/receive-pack.c > @@ -1061,9 +1061,8 @@ static void warn_if_skipped_connectivity_check(struct > command *commands, > } > } > if (!checked_connectivity) > - error("BUG: run 'git fsck' for safety.\n" > - "If

[PATCHv11 02/11] receive-pack.c: die instead of error in case of possible future bug

2015-01-06 Thread Stefan Beller
Discussion on the previous patch revealed we rather want to err on the safe side. To do so we need to stop receive-pack in case of the possible future bug when connectivity is not checked on a shallow push. Also while touching that code we considered that removing the reported refs may be harmful