Ooops my bad, i've merged with my previous patch :$
This is now fixed in:
http://easyant.abrm.info/trac/browser/trunk/src/main/patches/ant-713373-easyant-patch.diff

The patch contains only things related to phases concept.

In addition to this, i think that we should add a way to disable prefixing,
i have a use case in easyant where i don't want to use prefix.
In the patch i removed this control:
  if (configuredValue != null && configuredValue.length() == 0) {
                configuredValue = null;
  }

This allow me to have a dirty fix to import a build script without prefix :)
by setting as attribute to an empty string (<import file="..." as=""/>).




2008/11/12 Stefan Bodewig <[EMAIL PROTECTED]>

> On 2008-11-12, Jean-Louis BOUDART <[EMAIL PROTECTED]> wrote:
>
> > EasyAnt switched to ant-713373
>
> > Every things seems ok, except prefix separator should be applied on
> targets
> > (ie. in targetHelper) too.
>
> >
> http://easyant.abrm.info/trac/browser/trunk/src/main/patches/ant-713373-easyant-patch.diff
>
> Both places where your patch replaces "." with prefixSeparator (line
> 861 and line 888 of the original) are correct in svn, you must be
> using some inconsistent state since your patch won't apply.
>
> I'm a bit puzzled since I never committed a version that had
> prefixSeparator only in one place but not the other two.
>
> Stefan
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>

Reply via email to