Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy ibmjvmti.h into $JAVA_HOME/include directory #93

Merged
merged 1 commit into from
Jun 12, 2018

Conversation

babsingh
Copy link
Member

@babsingh babsingh commented Jun 8, 2018

Closes: eclipse-openj9/openj9#1252

Signed-off-by: Babneet Singh [email protected]

@babsingh
Copy link
Member Author

@Mesbah-Alam This change should fix eclipse-openj9/openj9#1252. I had provided you JDK8 with the above fix earlier. Please post in this pull request if JDK8 with the above fix resolves the issue in eclipse-openj9/openj9#1252.

@pshipton
Copy link
Member

jenkins test sanity

@babsingh
Copy link
Member Author

The following tests in Sanity - JDK8 - win_x86-64_cmprssptrs are failing:

	cmdLineTester_jvmtitests_extended_3
	cmdLineTester_jvmtitests_extended_7
	cmdLineTester_jvmtitests_extended_8
	cmdLineTester_jvmtitests_extended_9
	cmdLineTester_jvmtitests_extended_11
	decompileAtMethodResolve_0

Reason of failure - NPT ERROR: Cannot open library

Solution - Create a link to libnpt.so in $JAVA_HOME/jre/lib from /usr/local/lib
ln -s <JAVA_HOME>/jre/lib/amd64/libnpt.so /usr/local/lib/libnpt.so

@pshipton
Copy link
Member

pshipton commented Jun 12, 2018

Babneet, please mention your solution for the NPT ERROR problem in eclipse-openj9/openj9#2129

@pshipton pshipton changed the title WIP: Copy ibmjvmti.h into $JAVA_HOME/include directory Copy ibmjvmti.h into $JAVA_HOME/include directory Jun 12, 2018
@pshipton
Copy link
Member

I'm going to go ahead and merge this. Even if it doesn't fix the original problem the file needs to be copied.

@pshipton pshipton merged commit d587cde into ibmruntimes:openj9 Jun 12, 2018
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