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

Update to JNA 4.2.0 #4443

Merged
merged 1 commit into from May 8, 2016

Conversation

Projects
None yet
2 participants
@gohai
Contributor

gohai commented May 7, 2016

Initial motivation for this was to support ARM platforms on libraries that don't come with their own copy of JNA, but depend on the one that comes with Processing. (The Kinect library did that, but I wouldn't be surprised if there were more.)

JNA 4.2.0 and above should work with ARM - this is also what this PR updates to. (The Video library uses the same version since a couple of month with no complaints.)

jna-platform.jar is required for the Win32-specific functionality that the app makes use of. Note: The name of the subfolder containing jnidispatch.dll for 64-bit Windows changed - fixed this in build.xml.

This successfully starts up on Windows 10.

Update to JNA 4.2.0
The name of the subfolder containing jnidispatch.dll for 64-bit Windows changed as well.

@gohai gohai referenced this pull request May 7, 2016

Closed

Update JNA to 4.2.1 #4134

@benfry benfry merged commit 347945e into processing:master May 8, 2016

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry May 8, 2016

Member

I'd rather that the libraries include the version of JNA that they need, otherwise they'll just run into compatibility problems down the road. But I've opted to merge this so that we're using a fresher JNA.

Member

benfry commented May 8, 2016

I'd rather that the libraries include the version of JNA that they need, otherwise they'll just run into compatibility problems down the road. But I've opted to merge this so that we're using a fresher JNA.

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry May 8, 2016

Member

Are you sure this works on Windows? When double-clicking processing.exe I now get errors about native support not working properly.

Member

benfry commented May 8, 2016

Are you sure this works on Windows? When double-clicking processing.exe I now get errors about native support not working properly.

@gohai

This comment has been minimized.

Show comment
Hide comment
@gohai

gohai May 8, 2016

Contributor

@benfry I only tried ant run on my jna-update branch w/ Windows 10. Unfortunately I don't have my Windows desktop at hand to try more... perhaps, revert if release is imminent? (Sry sry)

Contributor

gohai commented May 8, 2016

@benfry I only tried ant run on my jna-update branch w/ Windows 10. Unfortunately I don't have my Windows desktop at hand to try more... perhaps, revert if release is imminent? (Sry sry)

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry May 8, 2016

Member

Hm, scratch that... processing.exe is working, but it seems that the new JNA just exacerbates #3543, so we no longer get the useful error message workaround. I'll keep fiddling.

Member

benfry commented May 8, 2016

Hm, scratch that... processing.exe is working, but it seems that the new JNA just exacerbates #3543, so we no longer get the useful error message workaround. I'll keep fiddling.

@gohai gohai deleted the gohai:jna-update branch May 8, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment