On Wed, Aug 26, 2015 at 9:18 PM, Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> wrote: > Junio C Hamano <gitster@xxxxxxxxx> writes: > >> Junio C Hamano <gitster@xxxxxxxxx> writes: >> >>> Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> writes: >>> >>>>> For the current code %(if:empty)%(align)%(end)%(then)Empty%(else)Not Empty%(end) >>>>> would print non-empty, I guess the documentation holds in that case. >>>>> Not sure if we require it to print non-empty. >>>> >>>> You don't want the %(if) condition to depend on whether >>>> --shell/--python/... is used. Since %(if:empty)%(align)%(end)%(then) >>>> holds when you don't use --shell, you also want it to hold when you >>>> quote. IOW, you should check for emptyness before (or actually without) >>>> doing the quoting. I guess this is what you're doing, and if so, I think >>>> it's "The Right Thing". >>> >>> I agree that %(align)%(end) should expand to empty and %(if:empty)...%(then) >>> should look at that empty string without quoting. So >>> >>> %(if:empty)%(align)%(end)%(then)Empty%(else)Not Empty%(end) >>> >>> should give "Empty"; otherwise the code is buggy, I think. >> >> (I shouldn't be typing while eating...) >> >> It should give "Empty", but the --shell/--python/... may make the >> whole "Empty", as the result of %(if:...)...%(end), be quoted. So >> you may see "'Empty'" in the output. > > Agreed (with both points). > Thanks, will work on this. -- Regards, Karthik Nayak -- 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