Max Rydahl Andersen
Max Rydahl Andersen
> Edit: Personally I'd just look to all the places where we call the resolver and see if we can't put the property replacement there. that made me go look...
note: just realizing prperty replacement as is implemented now also support env. for environment variable lookup so definitly dont want it available *everywhere*
well heck - we already support property replacement in all Tags (incl. files) I just realize. look for "replaceProperties" we avoid/reduce the impact on //FILES by files not allowing absolute...
so yeah "//FILES myfile=../../../.bash_profile" works ...but so does Path.of("../../../.bash_profile") in a main method of any java code ...so yeah; not any worse or better risk than what running anything from...
@quintesse assuming tests passes I think this is good to go.
@quintesse yes, it wont help for remote usage - but this fix won't change that. its already the case.
idea: make "!" a shorthand to trigger this behavior: //DEPS log4j! jfx!
@quintesse wdyt ? feels like should be fairly trivial ?
Btw. This is super annoying when using jbang in jupyter as it doesn't get the error stream so it only gets what's in the JSON
I think we want to record a "errors" list in the output. having to look both in output and parse stderr as well as .json is going to be too...