jline triggers a restricted method warning on modern JVMs
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 55/100
Línea de trabajo
Start with the :repl alias and the clojure-cli.repl.server entry point, then trace how the spawned client receives JVM options. Reproduce the warning on JDK 27 and verify that --enable-native-access=ALL-UNNAMED reaches the client and removes the JLine warning.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
[edited, now that I understand more of how the CLI REPL works and the handling of aliases in tools.deps/create-basis]
Running the CLI REPL on JDK 27 yields this warning:
WARNING: A restricted method in java.lang.System has been called
WARNING: java.lang.System::load has been called by org.jline.nativ.JLineNativeLoader in an unnamed module (file:/home/sean/.m2/repository/org/jline/jline/4.3.1/jline-4.3.1.jar)
WARNING: Use --enable-native-access=ALL-UNNAMED to avoid a warning for callers in this module
WARNING: Restricted methods will be blocked in a future release unless native access is enabled
The client needs to have this added to the alias (:attach):
:jvm-opts ["--enable-native-access=ALL-UNNAMED"]
If you have an :attach alias in your deps.edn file, per this project's README, you can add that line and it will solve it for clojure -M:attach (as expected).
Because of the way aliases defined in multiple sources (root, user, project, extra, via -Sdeps) are handled when a basis is created, the way that the CLI REPL server starts the client (for the :repl alias) will pick up the :jvm-opts from your deps.edn file!
That is because alias definitions are merged across sources, per this Ask about aliases and sources.
As long as that behavior does not change, adding the :jvm-opts shown above to the :attach alias in deps.edn is sufficient. If that behavior changes—or users create a different alias for starting the client (@practicalli-johnny this applies to you since you used a different name in your config!)—then the server code will need updating to add the :jvm-opts when spawning the client.
I think the safest approach is to update the server code to add that, since it works on JDK17+.
- Lenguaje dominante
- Clojure
- Estrellas
- 20
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de clojure/clojure-cli.repl
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
clojure/clojure-cli.repl#5 · 1 reacción ·
Todos los issues de clojure/clojure-cli.repl
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
.Needs Triage .Team/UXWest Priority:P2 Type:Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
metabase/metabase#83998 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Install script not workingAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Clojure 1.13 compatibilityAbiertobug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
clojure-emacs/cider-nrepl#1053 · 1 comentario ·
Los mantenedores suelen responder en 2 días