Skip to content
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

Unable to start JavaFX UI #5795

Closed
thron7 opened this issue May 6, 2024 · 2 comments · Fixed by #5963
Closed

Unable to start JavaFX UI #5795

thron7 opened this issue May 6, 2024 · 2 comments · Fixed by #5963
Labels

Comments

@thron7
Copy link

thron7 commented May 6, 2024

Describe the bug

First off, thanks for Spoon!

I'm new to Spoon and tried to start the new JavaFX UI following the description in spoon-visualisation/README.md. After running mvn clean package and changing into the target folder, there is no visualisation*.jar (recursively), and I haven't figured out how to build it, so I couldn't run the documented java ... -jar visualisation-1.1.jar command.

ping @arnobl

Thanks, Thomas

Source code you are trying to analyze/transform

No response

Source code for your Spoon processing

No response

Actual output

No response

Expected output

No response

Spoon Version

11.0.1-SNAPSHOT (It's the Git "master" branch at commit 36610b433bf16)

JVM Version

17

What operating system are you using?

Linux

@thron7 thron7 added the bug label May 6, 2024
@I-Al-Istannen
Copy link
Collaborator

I-Al-Istannen commented Aug 30, 2024

The Javafx build is a bit fiddly, it would really benefit from somebody cleaning it up.

Currently you need to:

  1. Fix the version of the spoon-pom parent
  2. mvn clean package
  3. cd target/modules
  4. Run
mkdir output
unzip ecj-3.37.0.jar -d output
unzip org.eclipse.jdt.core-3.37.0.jar -d output # overwrite
rm ecj-3.37.0.jar
rm org.eclipse.jdt.core-3.37.0.jar
cd output
zip -r ../org.eclipse.jdt.core-3.37.0.jar *
cd ..
java --module-path . --add-modules=ALL-MODULE-PATH -jar visualisation-1.1.jar

This might not be entirely obvious to casual users...

@I-Al-Istannen
Copy link
Collaborator

This should be fixed with #5963.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants