GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Update to JNA 4.2.0 by gohai · Pull Request #34 · processing/processing-video

Conversation

@gohai

@gohai gohai commented Sep 27, 2015

Copy link
Copy Markdown
Contributor

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@gohai
Update to JNA 4.2.0
d819d72
My fix from a656ad2 is now part upstreamed and part of a stable JNA release, so update to this one.
@gohai

gohai commented Sep 27, 2015

Copy link
Copy Markdown
Contributor Author

@codeanticode Hope this makes sense to you!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@benfry

benfry commented Sep 27, 2015

Copy link
Copy Markdown
Contributor

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@gohai

gohai commented Sep 27, 2015

Copy link
Copy Markdown
Contributor Author

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..

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@neilcsmith-net

neilcsmith-net commented Oct 2, 2015

Copy link
Copy Markdown

@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?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@codeanticode

codeanticode commented Oct 13, 2015

Copy link
Copy Markdown
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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

codeanticode added a commit that referenced this pull request Oct 13, 2015
@codeanticode
Merge pull request #34 from processing/jna-update
93d8a11
Update to JNA 4.2.0
@codeanticode
codeanticode merged commit 93d8a11 into master Oct 13, 2015
@gohai

gohai commented Oct 13, 2015

Copy link
Copy Markdown
Contributor Author

@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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@codeanticode

codeanticode commented Oct 13, 2015

Copy link
Copy Markdown
Contributor

Ok, here: https://github.com/processing/processing-video/releases/tag/latest

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@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

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@gohai @benfry @neilcsmith-net @codeanticode
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.