Re: [PATCH 10/13] transport-helper: implement marks location as capability

Heya,

On Sun, Aug 29, 2010 at 20:31, Jonathan Nieder [off-list ref] wrote:

This seems fragile to me: shouldn't the remote helper check somehow
that the line it read was actually a gitdir line?

You're probably right, the simplest way would be to check if repo.gitdir is set.

No other complaint on my part.  Requiring a flush seems entirely
appropriate to me, and if someone comes up with something nicer than
the "capabilities" sequence for requesting information, it would not
be the end of the world to have two ways to discover the .git dir.

Agreed.

-- 
Cheers,

Sverre Rabbelier