-
Notifications
You must be signed in to change notification settings - Fork 58
Added JNI Wrappers for Sanitizer #1359
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
Conversation
lalitb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ideal scenario in future would be to have all the submodule code (including JNI wrappers) in the private repo. Not an issue for this PR as it just follows the existing pattern.
|
@frank-msodsp I've pulled the code that include this PR and it fails to build on my local windows 11 machine (had to disable the Sanitizer includes to build), can you verify that there is no issue? These are the errors |
i just ran build-Win32Debug.bat with the lastest and didnt have issues. What are running? |
|
I'm running the android build(gradlew maesdk:build), this failure is new, as I've built it few weeks ago without any issues, had to disable Sanitizer feature in to build it locally. |
|
i'll try that can you share the steps to setup this up? Its also interesting that the ci build for android was successful. |
Added JNI Wrappers for sanitizer