README.md is confusing

[fniephaus]

The README.md is confusing because when using GraalVM 1.0.0 RC1, you run into problems executing mx --dynamicimports /compiler build, as recommended in "With Graal". I guess what is meant is "with the Graal compiler", but the wording is confusing here. What that README is probably talking about is "using LabsJDK + Graal compiler". It would be nice to know how to run from source with the default GraalVM and to distinguish that in the README from running with the Graal compiler also built from source.

[wirthi]

Thanks for your comment, Fabio. We will clarify in the README.md that it is indeed meant when you use a Graal compiler that you built from source yourself.

Regarding using Graal.js with a GraalVM: I am not sure what you mean with that? The GraalVMs binaries we provide are shipped with Graal.js already. If you build a GraalVM yourself, the documentation for that can be found in GraalVM, not in Graal.js.

[fniephaus]

@wirthi thanks for the swift response and for revising the README.md.

I actually meant building Graal.js with GraalVM 1.0.0 RC1. I've just checked out the repository again and tried to use GraalVM to mx --dynamicimports /compiler build Graal.js from scratch. AFAIK, the GraalVM can't compile its own compiler, but LabsJDK can.

[wirthi]

I have updated the README a bit. For the compilation problem there is another issue open.

[fniephaus]

Thanks! Although, I think it would also be useful to mention what vm should be used for which step. mx --dynamicimports /compiler build, for example, requires LabsJDK and won't work on a GraalVM (which is not obvious for beginners).