Re: Variable substitution in target descriptions

Hi Roger

sorry for the delay

On 2022/07/20 16:43:16 Roger Whitcomb wrote:
> Would it be as simple as calling PropertyHelper.replaceProperties(=E2=80=A6=
) at line 1143 in Main with the =E2=80=9CtargetDescription=E2=80=9D string?

or a few lines later when you can be sure targetDescription is not null.

Yes, that would be one option, the other place could be Target's geDescriptio=
n method. If you change it in Main it will only be fixed for the Ant CLI. If =
you change it in Target it would also start replacing properties for other ty=
pes of integrations like IDEs.

To be honest I'm a bit torn here as users of Target.getDescription may not ex=
pect property replacement to happen and thus a change there could be viewed a=
s a fix as well as breaking current behavior. In a way this is the same backw=
ards compatibility discussion that has been part of Ant's history for more th=
an twenty years now.

Stefan