is there currently a good way to do piping between processes in bb (since conch is gone)? if not, does the following look interesting? or perhaps there is a way to use it as an external lib?

@sogaiu clojure.java.shell/sh accepts an :in argument. conch was just a layer over processbuilder, which is in bb. I'm using this directly in e.g. deps.clj: https://github.com/borkdude/deps.clj/blob/master/src/borkdude/deps.clj#L16-L52

thanks for the pointer!

https://github.com/lambdaisland/regal looks cool, might be nice for babashka since is it also helps make regex be a bit more readable.. but it's also kind of redundant since core.re-matches and core.re-find are there

Maybe it works as an external lib:

$ export BABASHKA_CLASSPATH=$(clojure -Spath -Sdeps '{:deps {regal
                       {:git/url "" :sha "e179f20a2ec78d47c8c24257e644ac80a70e33cb"}}}')
 $ bb "(require '[lambdaisland.regal :as re]) (re/regex [:range \a \z])"

oh I see! it's because it uses reader conditionals and there is no :bb reader conditional

so it just returns nil for some functions

Did you consider to have a mode where babashka would use the :clj reader conditional tag? In many cases that will work I'm guessing. I think that's what I saw in one of the patches with :bb reader conditionals

$ export BABASHKA_CLASSPATH=$(clojure -Spath -Sdeps '{:deps {regal {:git/url "" :sha "42906e57bf45e9d6cb699700ea19e653757f535d"}}}')
$ bb "(require '[lambdaisland.regal :as re]) (re/regex [:range \a \z])"
#"[a-z]"

@jeroenvandijk I think that makes sense. Maybe if the :bb branch appears before the :clj branch, it will take that one, else it will take the :clj branch?

Yeah so the only case where it could go wrong is where the writer intentionally left :bb out (i.e. meaning no implementation. Right now this seems like a rare case. When it's intentionally left blank we could also demand :bb nil as that would make it explicitly and unambiguous. So for all the cases where :clj is compatible (and no :bb is needed) it would just work

yep

this would only take a minor change in bb, most notably here:

src/babashka/main.clj
330:                 :features #{:bb}

still forking libs might be a good idea because the original author is not always willing to maintain compatibility with bb maybe

Yeah that's actually a good practise in general, also for :clj

what is the good practice you're referring to, forking?

yeah, forking sorry. I don't always do it, but it's a pity when your favourite library has been removed from one day to the other and you didn't fork it

https://github.com/borkdude/babashka/issues/267

maybe making it explicit in the import instead? like

$ export BABASHKA_CLASSPATH=$(clojure -Spath -Sdeps '{:deps {regal {:git/url "" :sha "42906e57bf45e9d6cb699700ea19e653757f535d" :reader/tag :clj}}}')

that command just produces a classpath using the clojure tool, which just ignores extra stuff you put inside -Sdeps

ah dangit

but bb could be configurable like --features bb,clj or something, but that might work out very confusing if some libraries do expect a bb tag and others don't. I think it might be better to just have one kind of behavior for everything

yeah makes sense.

it would be a breaking change, but since the amount of bb-compatible libs in the wild might not be that large yet, it may be a good time to do it now

I mean the ones it has seems to cover the vast majority of the "bash script" domain

yeah, I mean these guys: https://github.com/borkdude/babashka/#external-resources

I think I may have found a bug, would you rather I make a small issue with example or just say it here? (it seems the alias for tools.cli is not working? (clojure.tools.cli/parse-opts *command-line-args* cli-options) works but (tools.cli/parse-opts *command-line-args* cli-options) doesn't (java.lang.ClassCastException: clojure.lang.PersistentList cannot be cast to clojure.lang.IFn)

lemme check

I see the bug:

(def aliases
  '{tools.cli 'clojure.tools.cli
    edn clojure.edn
    wait babashka.wait
    signal babashka.signal
    shell clojure.java.shell
    io 
    async clojure.core.async
    csv clojure.data.csv
    json cheshire.core})

there's a quote too many. PR welcome!

for now you can also do (require '[clojure.tools.cli :as foo])

I did that yeah

Fixed it

Yay! awesome!