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

Fix issues with -qvisibility=hidden on AIX #99

Merged
merged 1 commit into from
Jan 23, 2019
Merged

Fix issues with -qvisibility=hidden on AIX #99

merged 1 commit into from
Jan 23, 2019

Conversation

groeges
Copy link
Member

@groeges groeges commented Jan 23, 2019

This is a backport of a fix from jdk12 fix to jdk11 extensions
http://hg.openjdk.java.net/jdk/jdk/rev/fc54d27e58d8

Also backout additional change
#15

Reported in issue:
https://github.ibm.com/runtimes/openjdk-contribution/issues/743

Signed-off-by: Steven Groeger [email protected]

This is a backport of a fix from jdk12 fix to jdk11 extensions
http://hg.openjdk.java.net/jdk/jdk/rev/fc54d27e58d8

Also backout additional change
#15

Signed-off-by: Steven Groeger <[email protected]>
@groeges
Copy link
Member Author

groeges commented Jan 23, 2019

Copyright checker will fail as we are reverting some files to the base files from OpenJDK.

@groeges
Copy link
Member Author

groeges commented Jan 23, 2019

jenkins compile aix

@groeges
Copy link
Member Author

groeges commented Jan 23, 2019

jenkins compile xlinux

@groeges groeges changed the title Fix issues with -qvisibility=hidden on AIX WIP: Fix issues with -qvisibility=hidden on AIX Jan 23, 2019
@andrew-m-leonard andrew-m-leonard merged commit bd54ad4 into ibmruntimes:openj9 Jan 23, 2019
@andrew-m-leonard andrew-m-leonard changed the title WIP: Fix issues with -qvisibility=hidden on AIX Fix issues with -qvisibility=hidden on AIX Jan 23, 2019
jerrylui803 pushed a commit to jerrylui803/openj9-openjdk-jdk11 that referenced this pull request Jan 25, 2019
We do not need to comment-out Xlint JDK code
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