It looks like we need to reorder two of the patches. This patch needs to be moved to later in the series and happen after the delete_ref conversion : refs.c: make delete_ref use a transaction refs.c: add an err argument to delete_ref_loose I will respin a v19 with these patches reordered. Thanks, ronine sahlberg On Wed, Jun 18, 2014 at 1:47 PM, Michael Haggerty <mhagger@xxxxxxxxxxxx> wrote: > On 06/17/2014 05:53 PM, Ronnie Sahlberg wrote: >> Add an err argument to delete_loose_ref so that we can pass a descriptive >> error string back to the caller. Pass the err argument from transaction >> commit to this function so that transaction users will have a nice error >> string if the transaction failed due to delete_loose_ref. >> >> Add a new function unlink_or_err that we can call from delete_ref_loose. This >> function is similar to unlink_or_warn except that we can pass it an err >> argument. If err is non-NULL the function will populate err instead of >> printing a warning(). >> >> Simplify warn_if_unremovable. >> [...] > > I'm getting test failures starting with this commit: > >> Test Summary Report >> ------------------- >> t5514-fetch-multiple.sh (Wstat: 256 Tests: 11 Failed: 3) >> Failed tests: 6, 8-9 >> Non-zero exit status: 1 >> t6050-replace.sh (Wstat: 256 Tests: 28 Failed: 1) >> Failed test: 15 >> Non-zero exit status: 1 >> t1400-update-ref.sh (Wstat: 256 Tests: 133 Failed: 4) >> Failed tests: 86-87, 130-131 >> Non-zero exit status: 1 >> t5540-http-push-webdav.sh (Wstat: 256 Tests: 19 Failed: 2) >> Failed tests: 8-9 >> Non-zero exit status: 1 >> t5505-remote.sh (Wstat: 256 Tests: 76 Failed: 5) >> Failed tests: 11, 45-48 >> Non-zero exit status: 1 >> t9903-bash-prompt.sh (Wstat: 256 Tests: 51 Failed: 1) >> Failed test: 19 >> Non-zero exit status: 1 >> t9300-fast-import.sh (Wstat: 256 Tests: 170 Failed: 1) >> Failed test: 71 >> Non-zero exit status: 1 >> t6030-bisect-porcelain.sh (Wstat: 256 Tests: 55 Failed: 47) >> Failed tests: 2-5, 7-11, 13-14, 16-30, 32-34, 36-37, 39-44 >> 46-55 >> Non-zero exit status: 1 >> t7512-status-help.sh (Wstat: 256 Tests: 35 Failed: 1) >> Failed test: 27 >> Non-zero exit status: 1 >> t5516-fetch-push.sh (Wstat: 256 Tests: 80 Failed: 3) >> Failed tests: 47-49 >> Non-zero exit status: 1 > > Let me know if you need more information. > > Michael > > -- > Michael Haggerty > mhagger@xxxxxxxxxxxx > http://softwareswirl.blogspot.com/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html