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

Export omrthread_attr_set_detachstate #4658

Merged
merged 1 commit into from
Dec 9, 2019

Conversation

ashu-mehra
Copy link
Contributor

@ashu-mehra ashu-mehra commented Dec 9, 2019

For some reason omrthread_attr_set_detachstate was not exported,
while other similar functions are.
omrthread_attr_set_detachstate is now needed by OpenJ9 project as part
of this PR: eclipse-openj9/openj9#7940.

Signed-off-by: Ashutosh Mehra mehra.ashutosh@ibm.com

@rwy7
Copy link
Contributor

rwy7 commented Dec 9, 2019

@genie-omr build all

Have you tested this change against J9?

@youngar
Copy link
Contributor

youngar commented Dec 9, 2019

Can you update the copyright date in thread/thread_include.mk? This seems like a mistake not to have exported the function.

For some reason omrthread_attr_set_detachstate was not exported,
while other similar functions are.
omrthread_attr_set_detachstate is now needed by OpenJ9 project as part
of this PR: eclipse-openj9/openj9#7940.

Signed-off-by: Ashutosh Mehra <mehra.ashutosh@ibm.com>
@ashu-mehra
Copy link
Contributor Author

Have you tested this change against J9?

Yes, I tested these changes with OpenJ9 PR eclipse-openj9/openj9#7940 on Linux x86 and power.

Can you update the copyright date in thread/thread_include.mk

Done

@youngar
Copy link
Contributor

youngar commented Dec 9, 2019

It passed previous CI and OpenJ9 integration, so there is no need to launch another build for a copyright update. Thanks @ashu-mehra

@youngar youngar merged commit 730abd7 into eclipse-omr:master Dec 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants