Add example for embedding simple python scripts and debugging them using VSCode #13
Conversation
oracle-contributor-agreement
bot
added
the
OCA Verified
timfel
force-pushed
the
tim/graalpy-vscode
branch
from
b8d6ae6
to
e3c3cbc
Compare
timfel
force-pushed
the
tim/graalpy-vscode
branch
from
e3c3cbc
to
9b75d60
Compare
| ## 4.3 Calling the Python script from Java | ||
|
|
||
| `App.java` | ||
| ```java |
There was a problem hiding this comment.
If I want to follow just the guide without looking at the reference solution, then this snippet is not enough. In other guides like the Micronaut/Spring guide we included even the large html listing just so that the guide itself is self-contained. We should probably be consistent in this?
There was a problem hiding this comment.
Good point, I'll expand the readme
|
|
||
| ```shell | ||
| ./mvnw compile | ||
| ./mvnw javafx:run |
There was a problem hiding this comment.
When I followed the JavaFX quickstart from https://openjfx.io/openjfx-docs/#maven it did not generate the wrapper. Maybe this should be mvn compiler etc. or mention something like "if you don't use Maven wrapper, the commands would be ...".
There was a problem hiding this comment.
Ouch, the project generated by javafx-archetype-simple seem to be broken on many fronts...
- it sets
maven-compiler-pluginto version3.8.0, but that seems to be broken so one must manually update to at least3.8.1 - it generates
module-info.java, so javac does module stuff and since polyglot is a proper module, one has to addrequires org.graalvm.polyglot;to the generatedmodule-info.java. - after fixing all that it fails in runtime on some Java module permission issue
All in all, maybe it's easier to not even mention this Java FX quickstart guide..?
|
|
||
|  | ||
|  | ||
|  |
There was a problem hiding this comment.
I am getting this:
have you seen anything like that before?
There was a problem hiding this comment.
Argh. They haven't pushed the new version out, yet, and I still had my local changes that I pushed upstream. They updated the debugpy version in the extension to one with the fix (microsoft/vscode-python-debugger#505) but haven't pushed a new extension release
No description provided.