From: Jeff King <peff@peff.net>
Date: Thu, 29 Jan 2009 08:34:37 GMT
Subject: Re: [PATCH] symbolic ref: refuse non-ref targets in HEAD
Message-ID: <20090129083437.GA6297@coredump.intra.peff.net>
In-Reply-To: <20090129080145.GA777@coredump.intra.peff.net>

On Thu, Jan 29, 2009 at 03:01:45AM -0500, Jeff King wrote:

> > In addition, perhaps it may make sense to use test_create_repo to go one
> > level deeper before starting to play around, so that trash directory's
> > repository will prevent you from going any further up.
> 
> That sort of helps, but only by luck. Each test kills off one layer of
> repo. So the first one kills the test_create_repo, and the second one
> kills the trash directory. Adding another test would kill off the main
> repo. :) So you have to do something per-test. I'll do that in the
> re-roll.

It occurs to me you perhaps did mean to do something per-test, like:

  test_create_repo foo && (cd foo && do_the_actual_test)

That would work, too.

-Peff