On Mon, Jun 06, 2011 at 12:11:43PM -0400, Jeff King wrote: > I'll try the nicer fix Sverre mentioned and post a real patch. I wanted > to add a test for multiple refs to the test suite, but I got bogged down > by other breakages. Some of which I think are the fault of testgit, but > one of which I think may be a problem in the transport-helper code. > > I gave up in frustration after many hours of dealing with it on Friday, > but I'll give it another go today. :) OK, I finally hit a point where I think all of my patches are doing sane things. There are a few cleanups at the beginning, and then 4/8 demonstrates five different bugs. All but the last one are fixed by the remaining patches (see the comments in 4/8 for why the fifth bug is tricky). [1/8]: transport-helper: fix minor leak in push_refs_with_export [2/8]: git-remote-testgit: exit gracefully after push [3/8]: t5800: factor out some ref tests [4/8]: t5800: document some non-functional parts of remote helpers [5/8]: teach remote-testgit to import non-HEAD refs [6/8]: teach remote-testgit to import multiple refs [7/8]: transport-helper: don't feed bogus refs to export push [8/8]: git_remote_helpers: push all refs during a non-local export -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