[icedtea-web] RFC: allow icedtea-web to run with just a jre

[Jiri Vanek]

On 08/22/2011 10:20 PM, Deepak Bhole wrote:
> * Dr Andrew John Hughes<ahughes at redhat.com>  [2011-08-22 15:54]:
>> On 07:28 Fri 19 Aug     , Jiri Vanek wrote:
>>> Hi!!
>>>
>>
>> Please don't top-post.
>>
>>> Just to keep in loop - This is related to https://bugzilla.redhat.com/show_bug.cgi?id=731358 and https://bugzilla.redhat.com/show_bug.cgi?id=731345
>>>
>>> The patch itself looks ok. The only point I can see is that now there are three java paths set by configure -
>>> --with-jdk-home can specify SYSTEM_JDK_DIR
>>> --with-java can specify JAVA
>>> and now --with-jre is added.
>>>
>>> Is there really any reason why to keep all thee variables?
>>>
>>
>> Yes, a jdk directory is required for building and it still should be possible to allow an alternative
>> binary to be set using --with-java.  All three have sensible defaults, so in most cases, none will have
>
> Out of curiosity, why would one want to use a different binary via
> --with-java than the one in jdk-home? Just trying to understand what the
> use case there is..

I'm more wondering why to use different java then the one in jre home...

But as Omair mentioned - gcj is root of all evil :) and unknown parameters can cause harm.

As dr Andrew told - all variables have sensible defaults. But I still  think that there should be only two of them jdk-home and jre-home. (with coresponding  java comand and javac command)

I'm not sure whether  using "java stub" isn't to much complex for this...

Regards
J.

>
> Cheers,
> Deepak
>
>> to actually be specified and, when it is required, --with-jdk-home will work fine in most cases.
>>
>> The jre directory was being hardcoded before so allowing it to be set by --with-jre-home is sensible IMHO.
>> --
>> Andrew :)
>>
>> Free Java Software Engineer
>> Red Hat, Inc. (http://www.redhat.com)
>>
>> Support Free Java!
>> Contribute to GNU Classpath and IcedTea
>> http://www.gnu.org/software/classpath
>> http://icedtea.classpath.org
>> PGP Key: F5862A37 (https://keys.indymedia.org/)
>> Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37