Maybe this should be done with a additional attribute like : - ignorePrefix - disablePrefix
something like <import file="..." disablePrefix="true"/> WDYT? 2008/11/12 Jean-Louis BOUDART <[EMAIL PROTECTED]> > 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] >> >> >
