candid82 / joker

Small Clojure interpreter, linter and formatter.
https://joker-lang.org/
Eclipse Public License 1.0
1.63k stars 65 forks source link

*command-line-args* Erroneously Includes Preceding OS/Wrapper Arguments #85

Closed jcburley closed 6 years ago

jcburley commented 6 years ago

E.g.:

$ ./joker --repl -- a b c
Welcome to joker v0.9.6. Use EOF (Ctrl-D) or SIGINT (Ctrl-C) to exit.
user=> (println (str *command-line-args*))
["--repl" "--" "a" "b" "c"]
nil
user=> ^D
$

However, both Clojure and ClojureScript set this to just:

("a" "b" "c")

So, Joker should be setting it such that it includes neither --repl nor --, in the above example.

(That Joker sets it to a vector instead of a sequence is a separate issue.)

jcburley commented 6 years ago

https://github.com/jcburley/joker/commit/c9e4a7ab92fe05af6fa0066a455b07726a7a52c8 is queued up to fix this.