Re: lang/sun-jdk7 and lang/oracle-jdk8 not using built-in

["J. Lewis Muir" <jlmuir%imca-cat.org@localhost>]

On 2/15/16 11:37 PM, J. Lewis Muir wrote:
> On 2/15/16 2:16 AM, Jonathan Perkin wrote:
>> In that case we should add some logic to handle this correctly.  If
>> on Darwin we could use /usr/libexec/java_home to set PKG_JVM and the
>> resulting variables correctly.
>>
>> Do you want to have a go at adding this?
> 
> Sure, attached.
> 
> The patch is against pkgsrc-2015Q4 and makes the builtin.mk of
> lang/sun-jre7 and lang/oracle-jre8 verify that "/Library/Internet
> Plug-Ins/JavaAppletPlugin.plugin/Contents/Home" contains the expected
> JRE version before using it.
> 
> Since I didn't "use /usr/libexec/java_home to set PKG_JVM and the
> resulting variables correctly" as you suggested, I suspect my patch is
> not at all what you had in mind.  I'm sorry, I didn't understand what
> you were suggesting.
> 
> Regards,
> 
> Lewis

Any thoughts on this patch?

Lewis