Re: [RFC/PATCHv2] t5525: test the tagopt variable and that it can be overridden
- From
Daniel Johnson <computerdruid@gmail.com>
- Date
- Aug 14, 2010, 21:32 UTC
- Message-ID
- <201008141732.16214.ComputerDruid@gmail.com>
- In-Reply-To
- <AANLkTikP5xsvfBt7Rq9CoNnF=7SE4-V6LBML=tDMtT_Q@mail.gmail.com>
On Friday 13 August 2010 17:39:04 Ævar Arnfjörð Bjarmason wrote:
> On Fri, Aug 13, 2010 at 21:27, Daniel Johnson <computerdruid@gmail.com>
wrote:
Show 16 quoted lines
> > --- > > > >>> @@ -0,0 +1,44 @@ > >>> + > >>> +#!/bin/sh > >> > >> Is that an empty line before the test begins? The shebang should be on > >> the first line. > > > > Embarrassing. That's what I get for using yank/put and not paying closer > > attention. The rest is fixed too. > > Nice, I haven't actually *run it* but it looks good, so provided that > it passes tests when our beloved maintainer applies it: > > Reviewed-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Well, now that we have a fix and a test to help guard against future breakages, can someone tell me the next step as to getting them included?
Thanks, -Dan