[oe] [meta-java][PATCH 4/4] openjdk-8: add recipes for openjdk-8 and openjre-8

Jens Rehsack rehsack at gmail.com
Thu Oct 29 11:24:55 UTC 2015


> Am 29.10.2015 um 11:46 schrieb Otavio Salvador <otavio.salvador at ossystems.com.br>:
> 
> On Thu, Oct 29, 2015 at 7:22 AM, Jens Rehsack <rehsack at gmail.com> wrote:
>>> Am 29.10.2015 um 08:41 schrieb Richard Leitner <richard.leitner at skidata.com>:
>>> To cite Otavio:
>>> We could rename the patches directory for openjdk-7 and avoid the
>>> version number on it. This would make easier for upgrades and to see
>>> the diff between the patches.
>> 
>> So git diff -M doesn't work for you? I don't understand the reasoning.
> 
> It does but reusing the directory easy the upgrade and as the
> openjdk-8 is in maintenance it is expected that it does not change
> much in the patches.

Partial. As I told you, I updated OpenJDK from 8u40 to 8u72 when I first
encountered massive crashes on the target device with zeroshark until
llvm guys told me, that legacy JIT for ARM was utterly broken and they
force people to MCJIT.

Having both directories during the upgrade helped me massively to
avoid breaking patches by fixing against new upstream adoptions.

Surely, this could be handled developer-side by keeping an foo.old
directory - but it smells the same smell ;)

I don't fight against renaming the directory, I just argue, both
way have their own kind of smell.

>>>> create mode 100644 recipes-core/openjdk/openjdk-8-common.inc
>>>> create mode 100644 recipes-core/openjdk/openjdk-8-cross.inc
>>>> create mode 100644 recipes-core/openjdk/openjdk-8-native.inc
>>>> create mode 100644 recipes-core/openjdk/openjdk-8-native_72b00.bb
>>>> create mode 100644 recipes-core/openjdk/openjdk-8-release-72b00.inc
>>>> create mode 100644 recipes-core/openjdk/openjdk-8_72b00.bb
>>>> create mode 100644 recipes-core/openjdk/openjre-8_72b00.bb
>>> 
>>> Furthermore I had problems building OpenJDK-7 with this patch applied.
>>> But I will verify that again and come back to you!
>> 
>> Yes, please :)
>> 
>> I'm sorry that I cannot verify on own, because OpenJDK-7 still requires X11
>> and I have no X11 available in my distro and not enough resources to create
>> a playground for such things :(
>> 
>> But with sane errors I'm happy to dig and fix.
> 
> Once we merge this, I would like to try to simplify the recipes a
> little. Maybe avoiding the .inc files as much as possible as it makes
> harder to follow the code and in fact we just support (and this should
> not change) a single version for each openjdk major version.

I tried to reuse as much as possible from existing *.inc files, but

OpenJDK-7 is derived from icedtea-2.1 - which has a complete different
approach to build. OpenJDK-8 is plain, pure OpenJDK with it's own set
of pitfalls, smells and edges. Unfortunately figuring that all out costs
me days to adopt code from common.inc in a way both can use until I
finally gave up because of to many weird assumptions in openjdk-common.inc :/

Cheers
-- 
Jens Rehsack - rehsack at gmail.com




More information about the Openembedded-devel mailing list