Package Details: java8-openjfx-doc 8.u202-10

Git Clone URL: https://aur.archlinux.org/java8-openjfx.git (read-only, click to copy)
Package Base: java8-openjfx
Description: Java OpenJFX 8 client application platform (open-source implementation of JavaFX)
Upstream URL: https://wiki.openjdk.java.net/display/OpenJFX/Main
Keywords: java8-openjfx openjfx
Licenses: GPL
Submitter: freswa
Maintainer: Rogach
Last Packager: Rogach
Votes: 12
Popularity: 0.171898
First Submitted: 2022-03-09 18:41 (UTC)
Last Updated: 2024-05-25 07:41 (UTC)

Latest Comments

« First ‹ Previous 1 .. 4 5 6 7 8 9 10 11 12 13 Next › Last »

Rogach commented on 2023-02-03 05:38 (UTC)

@Pillgar Patched the PKGBUILD, please test that it now builds on your machine.

Pillgar commented on 2023-02-03 02:01 (UTC)

I think you figured it out.: https://gist.github.com/Pillgar/90931a304e57d75e02c2a3e366e247cb

ilikenwf commented on 2023-02-02 17:00 (UTC)

@Rogach yes on the patch, maybe bad in practice although we should also consider that because of how we handle Java, most people probably don't run java 8 as the default most of the time...and archlinux-java sets all the env vars out of the box.

The PKGBUILD may need to hard code the paths to use a java8 or lower path, and put that in the makedepends...circular self dependence maybe but still...

Rogach commented on 2023-02-02 15:21 (UTC)

Since the build is dependent on specific strings in java -version output, I'm planning to restrict dependencies to jdk8-openjdk and jre8-openjdk (instead of java-runtime-openjdk and java-environment-openjdk). I doubt that someone is running this build with newer JDK version.

However if somebody needs to compile with some other JDK version, then please leave a message here (and include the output of java -version, so that I can patch build.gradle accordingly).

Rogach commented on 2023-02-02 15:18 (UTC)

Aha, so _JAVA_OPTIONS line is throwing it off. Here's the updated test script, please test it: https://gist.github.com/Rogach/6f9639e3025ba744fb77b895c841acf4

Pillgar commented on 2023-02-02 15:07 (UTC)

Done: https://gist.github.com/Pillgar/40441a8186d52e6853d4fa9c05fe76ed

Rogach commented on 2023-02-02 12:30 (UTC)

@ilikenwf Thanks for the suggestion, I've updated the PKGBUILD with the patch.

I think this would be a bad idea in general - previous openjfx classes will be present on the classpath when building the new ones. However this package is probably going to be stuck on 8u202 version for the forceeable future, so this should be fine.

Rogach commented on 2023-02-02 11:21 (UTC)

@Pillgar Here's the updated script, I added some debugging printfs to get closer to the issue: https://gist.github.com/Rogach/6f9639e3025ba744fb77b895c841acf4

Can you please run it and paste the output?

ilikenwf commented on 2023-02-02 10:07 (UTC) (edited on 2023-02-02 10:08 (UTC) by ilikenwf)

So I was getting the error about "/usr/lib/jvm/java-8-openjdk/jre/lib/ext/jfxrt.jar" needing to be removed. I think this is because it assumes we're building in place... it's using $JAVA_HOME for this, which we can't change for the build since it needs java to build java...

This patch fixes it for me.

--- build.gradle.orig   2023-02-02 04:04:58.378846527 -0600
+++ build.gradle    2023-02-02 04:05:19.844677551 -0600
@@ -1070,12 +1070,7 @@

 // Task to check whether jfxrt.jar is present in the JDK
 task checkJfxrtJar {
-    doLast {
-        def jfxrtFile = new File("$JDK_HOME/jre/lib/ext/jfxrt.jar");
-        if (jfxrtFile.exists()) {
-            fail("$jfxrtFile must be removed before building sdk")
-        }
-    }
+    // we don't build in place
 }

 task updateCacheIfNeeded() {

Pillgar commented on 2023-02-02 07:13 (UTC)

Here ya go: https://gist.github.com/Pillgar/ca3840525166f9c7c0de8bdf6641e9f8

« First ‹ Previous 1 .. 4 5 6 7 8 9 10 11 12 13 Next › Last »