From b8d4b62158a6eaa14d70015777306a84c8cfb6d3 Mon Sep 17 00:00:00 2001 From: Ben Fry Date: Sun, 4 May 2014 17:03:41 -0400 Subject: [PATCH] fix for #2349, use embedded Java with Linux --- app/src/processing/mode/java/JavaBuild.java | 4 ++++ todo.txt | 11 ++++++----- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/app/src/processing/mode/java/JavaBuild.java b/app/src/processing/mode/java/JavaBuild.java index 3e1b68a5c..5bb3bcf4f 100644 --- a/app/src/processing/mode/java/JavaBuild.java +++ b/app/src/processing/mode/java/JavaBuild.java @@ -1559,6 +1559,10 @@ public class JavaBuild { pw.print("APPDIR=$(dirname \"$0\")\n"); // more posix compliant // another fix for bug #234, LD_LIBRARY_PATH ignored on some platforms //ps.print("LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$APPDIR\n"); + if (embedJava) { + // https://github.com/processing/processing/issues/2349 + pw.print("$APPDIR/java/bin/"); + } pw.print("java " + Preferences.get("run.options") + " -Djava.library.path=\"$APPDIR:$APPDIR/lib\"" + " -cp \"" + exportClassPath + "\"" + diff --git a/todo.txt b/todo.txt index 64e79a823..a25038197 100644 --- a/todo.txt +++ b/todo.txt @@ -11,6 +11,9 @@ X sketchbook location is set to an actual sketch (huh?) X exported apps on Windows 64 not working? X https://github.com/processing/processing/issues/2468 X just needed to add the local path for Java +X when exporting with local Java embedded, use that version +X https://github.com/processing/processing/issues/2349 +X (we can do this now since we're actually doing the embedding) high @@ -40,10 +43,11 @@ _ maybe OS X Java can't look in subfolders? (just auto-adds things) medium _ import static causes exception (with fix) _ https://github.com/processing/processing/issues/8 -_ https://github.com/processing/processing/pull/2273 +o https://github.com/processing/processing/pull/2273 +X can't use this patch, too confusing _ re/move things from Google Code downloads _ https://code.google.com/p/support/wiki/DownloadsFAQ -_ actual help with cleaning out the repo +_ clean out the repo _ https://github.com/processing/processing/issues/1898 _ requires re-forking, so still a ton of work _ remove non-web stuff from web @@ -53,9 +57,6 @@ _ add font fixes to the rest of the API _ https://github.com/processing/processing/commit/eaff673d173b2d27f276cf5c59e3abf6c0fab86b _ g2.setRenderingHint(RenderingHints.KEY_FRACTIONALMETRICS, _ RenderingHints.VALUE_FRACTIONALMETRICS_ON); -_ when exporting with local Java embedded, use that version -_ https://github.com/processing/processing/issues/2349 -_ (we can do this now since we're actually doing the embedding) _ get stack trace issues with NPE _ https://github.com/processing/processing/pull/2359 _ dataPath() not working when app is not run from app dir