Skip to content

Avoid GREATER_EQUAL in CMake files - #15113

Merged
keithc-ca merged 1 commit into
eclipse-openj9:masterfrom
babsingh:ojdk_mhs_jdk8_v1
May 25, 2022
Merged

Avoid GREATER_EQUAL in CMake files#15113
keithc-ca merged 1 commit into
eclipse-openj9:masterfrom
babsingh:ojdk_mhs_jdk8_v1

Conversation

@babsingh

@babsingh babsingh commented May 25, 2022

Copy link
Copy Markdown
Contributor

GREATER_EQUAL is not available in the minimum CMake version that OpenJ9
supports.

So, Java version >= 11 is rewritten as an "else()".

Signed-off-by: Babneet Singh sbabneet@ca.ibm.com

@babsingh
babsingh requested a review from keithc-ca May 25, 2022 15:31
Comment thread runtime/jcl/exports.cmake Outdated
@keithc-ca keithc-ca self-assigned this May 25, 2022
GREATER_EQUAL is not available in the minimum CMake version that OpenJ9
supports.

So, Java version >= 11 is rewritten as an "else()".

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
@babsingh
babsingh force-pushed the ojdk_mhs_jdk8_v1 branch from 492b1df to c1572f3 Compare May 25, 2022 15:56
@keithc-ca

Copy link
Copy Markdown
Contributor

jenkins compile x86-64_windows_ojdk292 jdknext

@keithc-ca

Copy link
Copy Markdown
Contributor

jenkins compile x86-64_windows_ojdk292 jdk8,jdk11

@keithc-ca

Copy link
Copy Markdown
Contributor

jenkins compile x86-64_windows_ojdk292 jdk17

@keithc-ca

Copy link
Copy Markdown
Contributor

jenkins compile win jdk17

@keithc-ca

Copy link
Copy Markdown
Contributor

@keithc-ca

Copy link
Copy Markdown
Contributor

An example failure without this can be found in https://openj9-jenkins.osuosl.org/job/Build_JDK11_x86-64_windows_ojdk292_Personal/3:

[2022-05-25T19:19:56.741Z] CMake Error at runtime/jcl/exports.cmake:640 (if):
[2022-05-25T19:19:56.741Z]   if given arguments:
[2022-05-25T19:19:56.741Z] 
[2022-05-25T19:19:56.741Z]     "JAVA_SPEC_VERSION" "GREATER_EQUAL" "11"
[2022-05-25T19:19:56.741Z] 
[2022-05-25T19:19:56.741Z]   Unknown arguments specified
[2022-05-25T19:19:56.741Z] Call Stack (most recent call first):
[2022-05-25T19:19:56.741Z]   runtime/jcl/CMakeLists.txt:226 (include)
[2022-05-25T19:19:56.741Z] 
[2022-05-25T19:19:56.741Z] 
[2022-05-25T19:19:56.741Z] -- Configuring incomplete, errors occurred!

@keithc-ca
keithc-ca merged commit e929d6b into eclipse-openj9:master May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants