Fork me on GitHub
#tools-deps
<
2021-03-22
>
imre13:03:55

Should gitlibs be nuked when upgrading from clojure tools 1.10.2.796 to 1.10.3.814?

borkdude13:03:25

It's always safe to nuke those, so if in doubt, go ahead I would say

imre14:03:22

I know it's safe 🙂 just wanted to know if it is necessary or recommended. I'd prefer to skip doing it on all the build agents in case it isn't needed

seancorfield15:03:22

I think if you went through the incremental prerelease builds you could get into a bad state but 814 fixed that. Or at least fixed part of that. I would upgrade to 814 and see if a build fails. It's pretty obvious if the gitlibs cache is broken.

3
borkdude17:03:39

I'm trying to get deps.clj up to date with the latest and greatest tools.deps / clojure bash. I tried coming up with a test that would have failed under the old clj and works with the newest clj with regards to the corfield comma / whitespace problem. So I tried this:

clj -Srepro -Sdeps '{:aliases {:space {:jvm-opts ["-Dfoo=\"foo bar\""] }}}' -M:space -e "(prn (System/getProperty \"foo\"))"
The string prints as "\"foo bar\"". 1. Should it? 2. Is this a representative test?

borkdude18:03:10

You can also do, deps.edn:

{:aliases {:space {:jvm-opts ["-Dfoo=foo bar"] }}}
Not sure if that's how it should be done. Just looking for a good test case.

borkdude18:03:12

Both seemed to fail with 1.10.2.781

borkdude19:03:08

I think I caught up with 814 now. Thanks! duckie

grzm21:03:28

Good afternoon, all! What’s the syntax for specifying multiple Java properties at the command line using -J For example, I want to specify a custom truststore using -.ssl.trustStore=/path/to/truststore and -.ssl.trustStorePassword=changeme. Space-separated like this?

clojure -J'-Djavax.net.ssl.trustStore=/path/to/truststore -Djavax.net.ssl.trustStorePassword=changeme' -M http-test.clj

thheller21:03:59

@grzm a -J for each I believe, -J-Dfoo -J-Dbar

thanks3 3