Øystein Walle <oys...@gmail.com> writes: > Junio C Hamano <gitster <at> pobox.com> writes: > >> >> ... >> >> is easier to read and maintain if written like so (with using HT >> properly---our MUAs may damage it and turn the indentation into >> spaces): >> >> ... && >> sed -e "s/ Z$/ /" >>expect <<-\EOF && >> Fixes: Z >> Acked-by= Z >> Reviewed-by: Z >> Signed-off-by: Z >> EOF >> ... >> > > How about: > > printf '%s: \n' Fixes Acked-by Reviewed-by Signed-off-by >> expect
Not really. > This solution scores high marks in both readability and maintainability > in my mind. I actually considered that approach while I was writing the message you responded to, but discarded it because it forces us to commit to the view that we forsee no need to test an output that does not end with a trailing whitespace. And I do not think that is a limitation we want to place on this test. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
