James C. McPherson wrote:

I think it's reasonable to remove the

<delete dir="${maven.home}"/>

from the extract-assembly target. Thoughts?

No, this task should stay in place as it ensures the installation directory actually contains the just built distribution and not some remains from a previous build. So I'm for "Not a bug" regarding MNG-4663 in its current form.

What could however be changed/discussed is the semantics of the maven.home directory being specified to the script. While one is supposed to enter something like
  /opt/local/apache-maven-<version>
the script effectively ignores the last directory part but uses the root directory from the distro. I think using a file mapper for the <unzip> task one could chop off the archive's root directory. This would allow users to specify
  /opt/local/apache-maven/2.2.1
or whathever they like as home directory.

Thoughts?


Benjamin

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to