Re: [PATCH 2/3] stripspace: respect repository config
- From
Jeff King <peff@peff.net>
- Date
- Nov 23, 2016, 00:12 UTC
- Message-ID
- <20161123001259.y7wjo75bhdgxmeme@sigill.intra.peff.net>
- In-Reply-To
- <xmqq7f7vp4ck.fsf@gitster.mtv.corp.google.com>
On Tue, Nov 22, 2016 at 01:55:07PM -0800, Junio C Hamano wrote:
Show 7 quoted lines
> > And this test makes sense. Even without "sub", it would show the > > regression, but it's a good idea to test the sub-directory case to cover > > the path-munging. > > Yup. Obviously during my initial attempt I was scratching my head > wondering where these two files went--they were later found inside > t/ directory which was really bad ;-)
Heh. Yeah, it's nice when the failure mode doesn't escape the trash directory, but it's probably not worth worrying about too much.
Show 7 quoted lines
> > In the "archive --remote" test I added, we may want to do the same to > > show that "--output" points at the correct path. > > Perhaps something like this. By going down one level, we make sure > that it is not sufficient to accidentally read from .git/config to > find out what 'foo' is, and also ../b5-nick.tar that is relative to > the prefix (aka 'a/') ends up at the top-level.
Yeah, your modification looks good.
-Peff