Skip to content

Changes to allow the app to run with openJDK 7 on OS X. #1787

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

Closed
wants to merge 2 commits into from
Closed

Changes to allow the app to run with openJDK 7 on OS X. #1787

wants to merge 2 commits into from

Conversation

haavar
Copy link

@haavar haavar commented Jan 4, 2014

These changes will allow the code to run on openJDK 7 under OS X. This is a preprerequisite for building the IDE without java 6 dependency.

#1665

Haavar Valeur added 2 commits January 3, 2014 17:55
…oolkit() after a call to FileManager.findFolder() will cause Toolkit.getDefaultToolkit() to hang.
@haavar haavar closed this Jan 4, 2014
@cmaglie
Copy link
Member

cmaglie commented Jan 9, 2014

I see that you closed this pull request, but should I merge the change in a8db52d anyway?
It's sill a prerequisite to support JDK 7 on OSX?

@haavar
Copy link
Author

haavar commented Jan 9, 2014

Yes, I had some issues with the serial library. You can't run the master branch on Java 7, because of the serial lib. However you can run the 1.5.x branch, because the lib has changed.

I've made changes to use java 7 on the 1.5.x branch, but file association is not working without a change in the oracle appbundler lib. I've sent oracle a patch, and I'm waiting to see what's going to happen with that. Other than that, it's working great so far.

Sent from my iPhone

On Jan 9, 2014, at 5:30, Cristian Maglie [email protected] wrote:

I see that you closed this pull request, but should I merge the change in a8db52d anyway?
It's sill a prerequisite to support JDK 7 on OSX?


Reply to this email directly or view it on GitHub.

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.

2 participants