adoptium / adoptium/TKG

SPEC for hotspot IMPL JDK setting by AUTO_DETECT as without _cmprssptrs

Open
#47 3 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
22
Forks
114
Avg merge
4d 2h
Merged PRs (30d)
8

Description

Running with github action or running in developer's local environment without export SPEC EnvDetector will generate autoGenEnv.mk to setting DETECTED_SPEC. And DETECTED_SPEC will be set as SPEC as nodefine SPEC.
https://github.com/AdoptOpenJDK/TKG/blob/master/envSettings.mk#L25

For non-openj9 JDK, DETECTED_SPEC will be set as without _cmprssptrs.
https://github.com/AdoptOpenJDK/TKG/blob/master/src/org/openj9/envInfo/JavaInfo.java#L75

This is inconsistent with jenkins story, which set SPEC per platform. All non large heap 64 platforms is setting as _cmprssptrs.
https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/buildenv/jenkins/openjdk_tests

Also this inconsistence is hidden by the check https://github.com/AdoptOpenJDK/TKG/blob/master/envSettings.mk#L33

Contributor guide

Open the contributing guide

Research direction

Compare the SPEC handling in envSettings.mk at lines 25 and 33 with detection in src/org/openj9/envInfo/JavaInfo.java at line 75 and the platform settings in buildenv/jenkins/openjdk_tests. Trace AUTO_DETECT for non-OpenJ9 JDKs and verify that generated DETECTED_SPEC matches the Jenkins behavior, including the _cmprssptrs distinction.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, java
Domain
build-system, ci-cd
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.