unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH]: Add Ant.
Date: Fri, 6 Feb 2015 16:51:45 +0100	[thread overview]
Message-ID: <idjsiej6nse.fsf@bimsb-sys02.mdc-berlin.net> (raw)
In-Reply-To: <87r3u44jwg.fsf@gnu.org>

>> * 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?

According to the docs[1],

    "ANT_HOME is used by the launcher script for finding the
     libraries. JAVA_HOME is used by the launcher for finding the
     JDK/JRE to use."

So, it would indeed make sense to modify the "launcher script"
(whichever this is, probably "ant") to set ANT_HOME before continuing.
I'm not a Java person, though, so I don't know if this is considered
bad.

JAVA_HOME, however, probably should not be set.  After all, Ant works
with different JDK/JRE versions, not only IcedTea 6.

I even wonder if we should make icedtea6 a build-time input only to
compile the libraries, so that one would not need to have icedtea6
installed at all.  Or should there be multiple variants of Java packages
akin to what we do with Python modules?  I must admit that I find this
rather confusing.  How closely do we have to tie Java applications /
libraries to a particular version of the JDK?  Are there any insights
you could share about how it's done in Nix?

~~ Ricardo

[1]: http://ant.apache.org/manual/index.html

  reply	other threads:[~2015-02-06 15:53 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-30 16:04 [PATCH]: Add Ant Ricardo Wurmus
2015-01-30 16:21 ` Ricardo Wurmus
2015-02-05 12:34   ` Ludovic Courtès
2015-02-06 15:51     ` Ricardo Wurmus [this message]
2015-02-07 22:55       ` Ludovic Courtès
2015-02-09 14:51         ` Ricardo Wurmus
2015-02-15 15:40           ` Andreas Enge
2015-02-15 16:13             ` Ricardo Wurmus
2015-02-15 16:21               ` Andreas Enge
2015-02-16 16:38               ` Ricardo Wurmus
2015-02-16 17:08                 ` Andreas Enge
2015-02-05 12:32 ` Ludovic Courtès

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=idjsiej6nse.fsf@bimsb-sys02.mdc-berlin.net \
    --to=ricardo.wurmus@mdc-berlin.de \
    --cc=guix-devel@gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).