Re: echo task documentation error
Dominique Devienne wrote:
> empty. If foo is empty by mistake, how will one troubleshoot this?
and not a blank line. Undefined properties are echoed as literal
text, not skipped.
I'm well aware of that ;-) That's why I wrote "empty", not "not
defined". As in empty string vs null string.
The proper way to output a line separator is by using
${line.separator}. That this doesn't work in Maven is a
bug for Maven, not a "feature" of Ant.
AFAIK, *any* task output is prefixed by the task's name. A line
separator is output in my book ;-) I fail to see why it would be good
to make an exception for somehow.
Go ahead and fix the "doc" bug if you want. Strictly speaking this is
in keeping with BC; but this is yet another one of these little quirks
Ant is full of. --DD
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
I stand corrected, thanks.
I checked in the manual change.
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
Re: echo task documentation error
--- Dominique Devienne <[EMAIL PROTECTED]> wrote: [SNIP] > AFAIK, *any* task output is prefixed by the task's > name. A line > separator is output in my book ;-) I fail to see why > it would be good > to make an exception for somehow. FWIW, I agree. -Matt > > Go ahead and fix the "doc" bug if you want. Strictly > speaking this is > in keeping with BC; but this is yet another one of > these little quirks > Ant is full of. --DD > > - > To unsubscribe, e-mail: > [EMAIL PROTECTED] > For additional commands, e-mail: > [EMAIL PROTECTED] > > Low, Low, Low Rates! Check out Yahoo! Messenger's cheap PC-to-Phone call rates (http://voice.yahoo.com) - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: echo task documentation error
> empty. If foo is empty by mistake, how will one troubleshoot this?
and not a blank line. Undefined properties are echoed as literal text, not
skipped.
I'm well aware of that ;-) That's why I wrote "empty", not "not
defined". As in empty string vs null string.
The proper way to output a line separator is by using
${line.separator}. That this doesn't work in Maven is a
bug for Maven, not a "feature" of Ant.
AFAIK, *any* task output is prefixed by the task's name. A line
separator is output in my book ;-) I fail to see why it would be good
to make an exception for somehow.
Go ahead and fix the "doc" bug if you want. Strictly speaking this is
in keeping with BC; but this is yet another one of these little quirks
Ant is full of. --DD
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
Re: echo task documentation error
Dominique Devienne wrote:
unless someone comes up with a good reason
why this is a bug rather than a feature.
It's a bug if it doesn't prefix that line with [echo] in non-emacs mode.
is not much different from ${foo} with foo being
empty. If foo is empty by mistake, how will one troubleshoot this?
Sure, not a very good example, but still. Any task output should be
prefixed with the task name in non-emacs mode. --DD
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
This is the sort of input I wanted before I went off blindly changing
documentation to match current behavior.
But your example isn't quite correct either.
${foo} (with foo not defined)
will produce
[echo] ${foo}
and not a blank line. Undefined properties are echoed as literal text, not
skipped.
So, although the omission of the task tag may be unintended, I don't see it as
quite as bad a thing as you do. It is handy to be able to emit a totally blank
line. Although, to be perfectly consistent, we should probably define something
like or some such to get that behavior rather than this
accidental thing.
My feeling for now is to simply update the doc to say (on echo.html, for the
"required" column of the message attribute):
"No. Text may also be included in a character section within this element. If
neither is included a blank line will be emitted in the output."
leaving the question of the missing task label to be disposed of separately.
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
Re: echo task documentation error
unless someone comes up with a good reason
why this is a bug rather than a feature.
It's a bug if it doesn't prefix that line with [echo] in non-emacs mode.
is not much different from ${foo} with foo being
empty. If foo is empty by mistake, how will one troubleshoot this?
Sure, not a very good example, but still. Any task output should be
prefixed with the task name in non-emacs mode. --DD
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
Re: echo task documentation error
Antoine Levy-Lambert wrote: Hi Steve, I think you are a committer too. Feel free to edit the documentation. Regards, Antoine Steve Cohen wrote: A very minor point: The ant manual states that the echo task's message attribute is required "unless data is included in a character section within this element." This appears to be false. I have by accident discovered that has the effect of inserting a totally blank line into the output. It doesn't even show the normal [echo] prefix. I don't know if this is intentional, but it is actually quite useful. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] You're right. I have been quite inactive, but I can certainly do this. That is, unless someone comes up with a good reason why this is a bug rather than a feature. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: echo task documentation error
This appears to be false. I have by accident discovered that
has the effect of inserting a totally blank line into the output. It doesn't
Damn! That is exactly what I need! the maven ant-run plugin seems to
ignore the ${line.separator} property in an echo
Thanks for discovering this and +1 on keeping it!
Kev
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
Re: echo task documentation error
Hi Steve, I think you are a committer too. Feel free to edit the documentation. Regards, Antoine Steve Cohen wrote: > A very minor point: > > The ant manual states that the echo task's message attribute is > required "unless data is included in a character section within this > element." > > This appears to be false. I have by accident discovered that > > > > has the effect of inserting a totally blank line into the output. It > doesn't even show the normal [echo] prefix. I don't know if this is > intentional, but it is actually quite useful. > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
