Skip to content

Conversation

@gohai
Copy link
Contributor

@gohai gohai commented Sep 27, 2015

My fix from a656ad2 is now part upstreamed and part of a stable JNA release, so update to this one.

My fix from a656ad2 is now part upstreamed and part of a stable JNA release, so update to this one.
@gohai
Copy link
Contributor Author

gohai commented Sep 27, 2015

@codeanticode Hope this makes sense to you!

@benfry
Copy link
Contributor

benfry commented Sep 27, 2015

Is this going to conflict with the JNA used by the PDE?

@gohai
Copy link
Contributor Author

gohai commented Sep 27, 2015

If it does, it would be an issue with processing-video since 038d80f. (Note: the Download Manager is still distributing an even oder version of processing-video currently, so it would only show up for people that get their library from GitHub.)

@benfry Is PDE's JNA used on all platforms? I'll definitely go test those..

@neilcsmith-net
Copy link

@gohai Have you tested this on Windows at all? Some issues have been seen with GStreamer-Java on Windows with later versions of JNA. Would love to know if we can ignore that issue now. If not, please report at https://github.com/gstreamer-java

@benfry one JNA version per process, so could be OK?

@codeanticode
Copy link
Contributor

I tested the video library using JNA 4.2.0 on Windows and Mac OS X, and didn't get any conflict with PDE's JNA.

codeanticode added a commit that referenced this pull request Oct 13, 2015
@codeanticode codeanticode merged commit 93d8a11 into master Oct 13, 2015
@gohai
Copy link
Contributor Author

gohai commented Oct 13, 2015

@neilcsmith-net I guess we'll try and revert if needed.

@codeanticode Thanks a lot! Would it be possible to distribute the new version through the Library Manager before the next version is released? Right now I am bundling the updated version with the tar-balls for ARM, as well as the Raspbian images, but it be nicer if folks could just download it as usual through the UI when they need it.

@codeanticode
Copy link
Contributor

@gohai gohai deleted the jna-update branch February 10, 2016 00:45
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.

5 participants