-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
DirectArgumentsMarshalNullableTest failure for s390x #1103
Comments
Could you do a bisect to see when this problem was introduced? I build the library in a qemu-chroot but the speed is only enough for that. If you can run it on real hardware, this should speed up bisecting considerably. Thank you. |
Will do. This was the 1st time I’ve tried this level of JNA the earlier versions I worked on (e.g. 4.4) didn’t have that test.
If you need an account for doing s390x stuff I can provide one on a CentOS 7.6 clone. I also have a Jenkins slave available if you want to do CI (and use Jenkins of course).
Neale
|
Am Mittwoch, den 05.06.2019, 08:56 -0700 schrieb Neale Ferguson:
Will do. This was the 1st time I’ve tried this level of JNA the
earlier versions I worked on (e.g. 4.4) didn’t have that test.
If you need an account for doing s390x stuff I can provide one on a
CentOS 7.6 clone. I also have a Jenkins slave available if you want
to do CI (and use Jenkins of course).
If you want to tackle it yourself - great I'm willing to help. If you
prefer me to have a look, an account would be great. Thank you for the
offer!
Matthias
|
Send me your public key.
.
|
The fix for the issue was merged to master. The problem was not in the unittest, but a real problem in the direct dispatch code path. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I inserted some debug code into
DirectArgumentsMarshalNullableTest
and intonative/testlib.c
to see what was being processed:returnInt32Argument
Note: All other tests pass.
The text was updated successfully, but these errors were encountered: