On Fri, Jun 27, 2008 at 04:37:33PM -0700, Junio C Hamano wrote: > > With this patch, the output takes into account the status of > > updating the local ref: > > > > error: there are still refs under 'refs/remotes/origin/foo' > > From $url_of_repo > > ! [new branch] foo -> origin/foo (unable to update local ref) > > Makes sense --- thanks. This is something we can have automated tests, > isn't it? We don't currently have any tests for either the fetch output or the push output. Note that we aren't changing the output _status_. Fetch always knew that this condition was a failure, and exited appropriately. So it really would just be testing the expected human-readable output in these situations, something I thought we usually didn't include in the tests. But if you think it is worth doing, I can whip up a few tests. -Peff -- 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