Skip to content

Switch back to JRE 8u77 on ARM#4454

Merged
benfry merged 1 commit into
processing:masterfrom
gohai:arm-fix-jre-regression
May 9, 2016
Merged

Switch back to JRE 8u77 on ARM#4454
benfry merged 1 commit into
processing:masterfrom
gohai:arm-fix-jre-regression

Conversation

@gohai

@gohai gohai commented May 9, 2016

Copy link
Copy Markdown
Contributor

JRE 8u91 or higher cause performance problems with the GL Video library - specifically: SimpleCapture example doesn't show a texture from USB-attached webcam on Raspberry Pi 2, and TwoVideos example doesn't play two videos smoothly.

This was tested with Processing compiled on the Pi (JDK 8u65b17), and processing-glvideo 2d6de36a33fde3573b335e9a95cc3e380466f328.

@benfry Spent all morning tracking down this regression ... would you mind if I re-spun the ARM tar-ball for 3.1 with this change? (Holding off with the Raspbian image for now)

JRE 8u91 or higher cause performance problems with the GL Video library - specifically: SimpleCapture example doesn't show a texture from USB-attached webcam on Raspberry Pi 2, and TwoVideos example doesn't play two videos smoothly.

This was tested with Processing compiled on the Pi (JDK 8u65b17), and processing-glvideo 2d6de36a33fde3573b335e9a95cc3e380466f328.
@gohai gohai force-pushed the arm-fix-jre-regression branch from 346a935 to 3c2b96f Compare May 9, 2016 13:02
@gohai

gohai commented May 9, 2016

Copy link
Copy Markdown
Contributor Author

@benfry I went ahead and updated the tar ball, after realizing that my previous re-spin had the wrong build number (250). So the processing tar ball for ARM is now essentially: https://github.com/gohai/processing/commits/arm-work

@benfry

benfry commented May 9, 2016

Copy link
Copy Markdown
Contributor

Doesn't this have to go after these two lines?

<property name="jdk.update" value="92" />
<property name="jdk.build" value="14" />

@gohai

gohai commented May 9, 2016

Copy link
Copy Markdown
Contributor Author

Thought properties in ant can only be set exactly once - hence setting it conditionally for the special case ahead of the default. (Most importantly: seems to work :) )

@benfry benfry merged commit 7fbd4a1 into processing:master May 9, 2016
@benfry

benfry commented May 9, 2016

Copy link
Copy Markdown
Contributor

Happy to trust you on it. ;-)

@gohai gohai deleted the arm-fix-jre-regression branch May 9, 2016 15:07
@gohai

gohai commented May 9, 2016

Copy link
Copy Markdown
Contributor Author

@benfry Is there any place to report/triage issues with JDK releases? A phone number where Larry Ellison picks up, perhaps?

@benfry

benfry commented May 9, 2016

Copy link
Copy Markdown
Contributor

Nah, Larry's on a boat somewhere so he doesn't have to pick up the phone.

But the openjdk mailing lists work for discussion, and then there's a bug reporting system (JIRA-based?) that handles the official things.

@gohai

gohai commented May 10, 2016

Copy link
Copy Markdown
Contributor Author

@benfry I filled JI-9037261 & will write to openjdk on the weekend!

@github-actions

Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants