[PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-28 Thread Carl Worth
On Fri, 28 Jan 2011 08:45:35 +0100, Thomas Schwinge wrote: > Matches my thoughts :-) -- and as it occurs to me right now, doing it in > one here document should be possible like this, if additional_headers is > changed to have the newline *at the beginning* of the string: ... > Date: ${templa

[PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-28 Thread Carl Worth
On Thu, 27 Jan 2011 02:17:21 -0700, Thomas Schwinge wrote: > +# Note that in the way we're setting it above and using it below, > +# `additional_headers' will also serve as the header / body separator > +# (empty line in between). Thanks, Thomas. I've merged this now. I'd even pref

[PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-28 Thread Thomas Schwinge
Hallo! On Fri, 28 Jan 2011 15:36:25 +1000, Carl Worth wrote: > On Thu, 27 Jan 2011 02:17:21 -0700, Thomas Schwinge > wrote: > > +# Note that in the way we're setting it above and using it below, > > +# `additional_headers' will also serve as the header / body separator > > +# (empty

Re: [PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-28 Thread Carl Worth
On Fri, 28 Jan 2011 08:45:35 +0100, Thomas Schwinge wrote: > Matches my thoughts :-) -- and as it occurs to me right now, doing it in > one here document should be possible like this, if additional_headers is > changed to have the newline *at the beginning* of the string: ... > Date: ${templa

Re: [PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-27 Thread Thomas Schwinge
Hallo! On Fri, 28 Jan 2011 15:36:25 +1000, Carl Worth wrote: > On Thu, 27 Jan 2011 02:17:21 -0700, Thomas Schwinge > wrote: > > +# Note that in the way we're setting it above and using it below, > > +# `additional_headers' will also serve as the header / body separator > > +# (empty

Re: [PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-27 Thread Carl Worth
On Thu, 27 Jan 2011 02:17:21 -0700, Thomas Schwinge wrote: > +# Note that in the way we're setting it above and using it below, > +# `additional_headers' will also serve as the header / body separator > +# (empty line in between). Thanks, Thomas. I've merged this now. I'd even pref

[PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-27 Thread Michal Sojka
On Thu, 27 Jan 2011, Thomas Schwinge wrote: > Signed-off-by: Thomas Schwinge > --- > test/test-lib.sh |5 - > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/test/test-lib.sh b/test/test-lib.sh > index d179426..f536172 100755 > --- a/test/test-lib.sh > +++ b/test/test-l

Re: [PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-27 Thread Michal Sojka
On Thu, 27 Jan 2011, Thomas Schwinge wrote: > Signed-off-by: Thomas Schwinge > --- > test/test-lib.sh |5 - > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/test/test-lib.sh b/test/test-lib.sh > index d179426..f536172 100755 > --- a/test/test-lib.sh > +++ b/test/test-l

[PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-27 Thread Thomas Schwinge
Signed-off-by: Thomas Schwinge --- test/test-lib.sh |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/test/test-lib.sh b/test/test-lib.sh index d179426..f536172 100755 --- a/test/test-lib.sh +++ b/test/test-lib.sh @@ -351,8 +351,11 @@ ${additional_headers}" ${additiona

[PATCH] Clarify usage of `additional_headers' in test/test-lib.sh:generate_message.

2011-01-27 Thread Thomas Schwinge
Signed-off-by: Thomas Schwinge --- test/test-lib.sh |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/test/test-lib.sh b/test/test-lib.sh index d179426..f536172 100755 --- a/test/test-lib.sh +++ b/test/test-lib.sh @@ -351,8 +351,11 @@ ${additional_headers}" ${additiona