Hi, On Thu, Jan 7, 2010 at 5:41 AM, Tay Ray Chuan <rctay89@xxxxxxxxx> wrote: > Regarding this point, I now think that we should ignore the > helper-reported status only if that status is none, and continue > updating the ref status in the top-level transport if the helper did > push successfully/failed, even if we didn't tell it to push: > > @@ -429,7 +429,7 @@ static int push_refs(struct transport *transport, > > ref->status = status; > ref->remote_status = msg; > - if (ref->status == REF_STATUS_NONE) { > + if (ref->status == REF_STATUS_NONE && status == REF_STATUS_NONE) { > ref->status = status; > ref->remote_status = msg; > } else { sorry for this broken hunk. I'll be re-sending shortly to make things clearer. -- Cheers, Ray Chuan -- 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