matthiasblaesing commented on PR #7248:
URL: https://github.com/apache/netbeans/pull/7248#issuecomment-2050286100

   > what does "the GraalVM.js tests work only when invoked manually" mean ? 
`ant test` from the commandline ?
   
   Sorry, bad wording. "the GraalVM.js tests work only when invoked manually" 
might be better phrased "the GraalVM.js tests work only when invoked 
individually". I mean, that when all tests of the GraalVM.js module are run I 
see 3 failing and 2 successful tests. The 3 failing tests work, when run 
isolated.
   
   I think it is ok, to support GaalJS 17 only on an untested basis as the 
platform itself is unstable (GraalVM as a whole) or a niche (GraalVM 17). I 
also think it is fair, that people running such an experimental platform must 
be prepared to take the pain (in the case of GraalVM 17, install JS).
   
   For a future perspective, this might become a problem:
   
   
https://github.com/apache/netbeans/blob/14fb065d23dca2fae74483b38bfe733504317c16/ide/libs.graalsdk/src/org/netbeans/libs/graalsdk/impl/Installer.java#L27-L41
   
   But I have some hope, that in the future Truffle/GraalVM will not stay with 
native libraries per platform or have a sane strategy to work then without 
failing hard. Seeing a Java runtime loading a native library by default and 
calling that the future of the JVM can't be the future.
   
   I reached the end of what I can do. I would only squash this.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to