atomisthq / jibbit

Dockerless Clojure Image builds using deps.edn
Eclipse Public License 1.0
111 stars 9 forks source link

main-opts and jvm-opts are being extracted from basis incorrectly #19

Closed puredanger closed 1 year ago

puredanger commented 1 year ago

In doing some work on the basis recently, I discovered that due to some sloppy coding on my part, the basis contained more data than it should have in the :resolve-args and :classpath-args keys.

In particular these code locations are pulling either :jvm-opts or :main-opts from the :classpath-args map. Neither of these are classpath args and were never expected to be data that should be there:

So, you should not be doing that. However, I do think it is reasonable to want those merged jvm-opts and main-opts and the basis now contains an official place to get that info in a new basis key :argmap.

To fix, update to at least tools.build 0.9.4, and replace :classpath-args with :argmap. The :classpath-args are deprecated and will be removed in the near future.

slimslenderslacks commented 1 year ago

@puredanger Thanks for this. I'll switch this over to use the :argmap and update to 0.9.4.