Re: [PATCH]: Add Ant.

[Ludovic Courtès]

Ricardo Wurmus <address@hidden> skribis:

> A couple more things:
>
> * the installation script copies MS DOS batch scripts to $out/bin, which
>   we could do without.

Indeed.  :-)  Would be best to get rid of them.

> * there are many launcher scripts for Ant: bash scripts, perl scripts,
>   Python scripts ... Does ant-minimal need to get perl and python as
>   inputs or can we do without these additional dependencies?

By launcher scripts, do you mean the ‘ant’ command?  I suppose it’s
enough to have one of them, and the Bash version is probably the one
with the smallest disk space footprint, no?

> * to use Ant without problems ANT_HOME and JAVA_HOME really should be
>   set appropriately (ANT_HOME to $out of the ant-minimal package,
>   JAVA_HOME to $out of icedtea6).  Should I suggest these two paths as
>   search paths by adding something like this to ant-minimal (for
>   ANT_HOME) and icedtea6 (for JAVA_HOME)?
>
>     (native-search-paths
>      (list (search-path-specification
>             (variable "ANT_HOME")
>             (files '(".")))))

Since Ant is generally used via the ‘ant’ command (AIUI), what about
wrapping that command so that ANT_HOME and JAVA_HOME are set
appropriately?

Thanks,
Ludo’.