[icedtea-web] RFC: Fix for PR779: IcedTea-Web plug-in hardcodes jpi-version to 1.6.0

[Matthias Klose]

On 08/31/2011 01:48 AM, Dr Andrew John Hughes wrote:
> On 13:36 Tue 30 Aug     , Deepak Bhole wrote:
>> Hi,
>>
>> Attached patch fixes PR779:
>> http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=779
>>
>> It makes the jpi-version dynamic rather than static.
>>
>> Okay for HEAD and 1.1?
>>
>> Cheers,
>> Deepak
>
> No, this patch is incorrect.  There is no guarantee that the javac
> used to build will be of the same JDK version used at runtime,
> or even that it will be installed (your previous JRE patches allow
> IcedTea-Web to work without a JDK remember?)

No. There is nothing wrong with this approach. As long as you have the same 
version in SYSTEM_JDK_DIR and SYSTEM_JDK_DIR/jre, everthing is fine. Anything 
else is a broken configuration/installation anyway.