Done at r1100795

http://ant-contrib.sourceforge.net/tasks/tasks/index.html

Jacques

From: "Jacques Le Roux" <j...@les7arts.com>
OK thanks, makes totaly sense to me

Jacques

From: "René Scheibe" <rene.sche...@tngtech.com>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

No, I haven't changed it. I just proposed to use ant-contrib instead of
implementing "if" in OFBiz itself.

Regards,
René Scheibe

On 05/08/2011 10:56 AM, Jacques Le Roux wrote:
I'd like to include ant-contrib in OFBiz and clean macros.xml, have you
something to contribute René?

Thanks

Jacques

From: "Jacques Le Roux" <jacques.le.r...@les7arts.com>
I have not looked into details, this has been committed by Adam
http://markmail.org/message/wnioayvhpqoo6p6u
Could you please enlighten us Adam?

Thanks

Jacques

René Scheibe wrote:
Some time ago the Ant "if" task implementation was replaced by another
one using a Javascript version.

I was wondering why this is named If-ant.py and not If-ant.js?

The scriptdef of this "if" task in macros.xml could also be cleaned.

<scriptdef name="if" language="javascript"
classpathref="rhino.class.path" loaderref="rhino.class.path.loader"
src="./If-ant.py">
 <element name="condition"
classname="org.apache.tools.ant.taskdefs.condition.And"/>
 <element name="commands"
classname="org.apache.tools.ant.taskdefs.Sequential"/>
 <element name="else"
classname="org.apache.tools.ant.taskdefs.Sequential"/>
</scriptdef>

The referenced loaderref "rhino.class.path.loader" is defined nowhere.
The referenced "rhino.class.path" references a Rhino jar file
("${ofbiz.home.dir}/extension/rhino/lib/js-1.7.R1.jar") which is not
existing. There is no "extension" folder at all in trunk.
If the classpathref and loaderref is removed, the "if" task is still
working as they are optional. So it looks like a Javascript engine is
included somewhere else.
If also have found framework/base/ant-scripts/if-script.groovy and it
looks like this is not used either.

I would anyway suggest to use ant-contrib which also includes an "if"
task.
See: http://ant-contrib.sourceforge.net/tasks/tasks/if.html

Regards
René Scheibe






- -- René Scheibe * rene.sche...@tngtech.com * +49-176-62192936
TNG Technology Consulting GmbH, Betastr. 13a, 85774 Unterföhring
Geschäftsführer: Henrik Klagges, Gerhard Müller, Christoph Stock
Sitz: Unterföhring * Amtsgericht München * HRB 135082
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk3Gj1oACgkQUXs9EHvIuCpuEwCeM8z/uU/uxhaAO14vFhDPxy8u
NmAAnjHH1wzr4Em5FXPPj92iLzkJXqu2
=A/b1
-----END PGP SIGNATURE-----




Reply via email to