diff options
| author | André Draszik <andre.draszik@jci.com> | 2018-08-13 11:09:34 +0100 |
|---|---|---|
| committer | Richard Leitner <richard.leitner@skidata.com> | 2018-08-13 15:01:28 +0200 |
| commit | 3737e0d040dd89737ef4d3a7c45f71c4ff9ad2a8 (patch) | |
| tree | f9c4dfe02eb1acaf7b441d017c63e471ef106a55 | |
| parent | 1471f1789e002b0381d0df9dec09581c2a9866b1 (diff) | |
| download | meta-java-3737e0d040dd89737ef4d3a7c45f71c4ff9ad2a8.tar.gz | |
Revert "openjdk-8: fix build for gcc8.x"
This reverts commit 52fb41cec7d5125bb11c718705158696ffef11f8.
The change being reverted has two problems:
- it still doesn't produce working binaries
- compilation on pre-gcc v7 compilers fails (which is
relevant for compiling openjdk-8-native, as that
uses the build machine's gcc, not yocto's gcc):
| At global scope:
| cc1plus: error: unrecognized command line option ‘-Wno-stringop-overflow’ [-Werror]
| cc1plus: all warnings being treated as errors
We now use a different approach to address the issues
than that patch, and it is thusly not needed anymore.
We fully support gcc < 7 again.
Signed-off-by: André Draszik <andre.draszik@jci.com>
Signed-off-by: Richard Leitner <richard.leitner@skidata.com>
| -rw-r--r-- | recipes-core/openjdk/openjdk-8-release-16xbyy.inc | 1 | ||||
| -rw-r--r-- | recipes-core/openjdk/patches-openjdk-8/hotspot_fix_gcc8x_build.patch | 21 |
2 files changed, 0 insertions, 22 deletions
diff --git a/recipes-core/openjdk/openjdk-8-release-16xbyy.inc b/recipes-core/openjdk/openjdk-8-release-16xbyy.inc index bd440c6..8b30c37 100644 --- a/recipes-core/openjdk/openjdk-8-release-16xbyy.inc +++ b/recipes-core/openjdk/openjdk-8-release-16xbyy.inc | |||
| @@ -16,7 +16,6 @@ PATCHES_URI = "\ | |||
| 16 | file://0008-jdk-use-correct-include-for-signal.patch \ | 16 | file://0008-jdk-use-correct-include-for-signal.patch \ |
| 17 | file://0009-jdk-disable-backtrace-musl-build-fix.patch \ | 17 | file://0009-jdk-disable-backtrace-musl-build-fix.patch \ |
| 18 | file://0010-build-fix-build-on-as-needed-toolchains-generic.patch \ | 18 | file://0010-build-fix-build-on-as-needed-toolchains-generic.patch \ |
| 19 | file://hotspot_fix_gcc8x_build.patch \ | ||
| 20 | file://0011-hotspot-backport-patch-to-fix-misuses-of-strncpy-str.patch \ | 19 | file://0011-hotspot-backport-patch-to-fix-misuses-of-strncpy-str.patch \ |
| 21 | " | 20 | " |
| 22 | # some patches extracted from http://cr.openjdk.java.net/~rkennke/shark-build-hotspot/webrev.01/hotspot.patch | 21 | # some patches extracted from http://cr.openjdk.java.net/~rkennke/shark-build-hotspot/webrev.01/hotspot.patch |
diff --git a/recipes-core/openjdk/patches-openjdk-8/hotspot_fix_gcc8x_build.patch b/recipes-core/openjdk/patches-openjdk-8/hotspot_fix_gcc8x_build.patch deleted file mode 100644 index 1a406f2..0000000 --- a/recipes-core/openjdk/patches-openjdk-8/hotspot_fix_gcc8x_build.patch +++ /dev/null | |||
| @@ -1,21 +0,0 @@ | |||
| 1 | From: Richard Leitner <richard.leitner@skidata.com> | ||
| 2 | Date: Thu, 14 Jun 2018 23:42:11 +0200 | ||
| 3 | Subject: [PATCH] openjdk-8: hotspot: fix gcc8 compilation | ||
| 4 | |||
| 5 | As oe-core/yocto-project master migrated to gcc8.x we need to support this. | ||
| 6 | Therefore disable following gcc warnings/errors which caused the build to fail: | ||
| 7 | + stringop-overflow | ||
| 8 | + return-type | ||
| 9 | --- | ||
| 10 | |||
| 11 | --- a/hotspot/make/linux/makefiles/gcc.make | ||
| 12 | +++ b/hotspot/make/linux/makefiles/gcc.make | ||
| 13 | @@ -197,7 +197,7 @@ | ||
| 14 | endif | ||
| 15 | |||
| 16 | # Compiler warnings are treated as errors | ||
| 17 | -WARNINGS_ARE_ERRORS = -Werror | ||
| 18 | +WARNINGS_ARE_ERRORS = -Werror -Wno-stringop-overflow -Wno-error=return-type | ||
| 19 | |||
| 20 | ifeq ($(USE_CLANG), true) | ||
| 21 | # However we need to clean the code up before we can unrestrictedly enable this option with Clang | ||
