Ooops i forgot "ignoring" prefix should ignore also prefix separator
2008/11/12 Jean-Louis BOUDART <[EMAIL PROTECTED]> > 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] >>> >>> >> >
