Re: [PATCH 4/5] refs.c: update rename_ref to use a transaction

2014-07-25 Thread Jonathan Nieder
Ronnie Sahlberg wrote: > Change refs.c to use a single transaction to copy/rename both the refs and > its reflog. Yay! > Since we are no longer using rename() to move the reflog file > we no longer need to disallow rename_ref for refs with a symlink for its > reflog so we can remove

[PATCH 4/5] refs.c: update rename_ref to use a transaction

2014-07-25 Thread Ronnie Sahlberg
Change refs.c to use a single transaction to copy/rename both the refs and its reflog. Since we are no longer using rename() to move the reflog file we no longer need to disallow rename_ref for refs with a symlink for its reflog so we can remove that test from the testsuite. Change the function to