Add alive? function to expose java.lang.Process::isAlive

[grzm]

I saw a couple of options for detecting features in process.cljc and chose the more explicit one.

I'm not sure how to test pre-JDK8 behavior when .isAlive is not available. Does babushka/process explicitly support pre-JDK 8? If so, any recommendations on how to test?

[borkdude]

Thanks! Clojure itself even doesn’t support Java 1.7 anymore so we don’t either. You can assume alive to be there

On Fri, 7 Oct 2022 at 08:29, Michael Glaesemann ***@***.***> wrote: I saw a couple of options for detecting features in process.cljc and chose the more explicit one. I'm not sure how to test pre-JDK8 behavior when .isAlive is not available. Does babushka/process explicitly support pre-JDK 8? If so, any recommendations on how to test? ------------------------------ You can view, comment on, or merge this pull request online at: #92 Commit Summary - 069fcf2 <069fcf2> Add alive? function to expose java.lang.Process::isAlive File Changes (2 files <https://github.com/babashka/process/pull/92/files>) - *M* src/babashka/process.cljc <https://github.com/babashka/process/pull/92/files#diff-c49a0dfdcfc60e10e3ba98144d615104abf6e6adaaa5355ef8e629a3d2de88c8> (14) - *M* test/babashka/process_test.cljc <https://github.com/babashka/process/pull/92/files#diff-3beb7fcb4af94de65dc3b82e9af2852ecb8d5c6a89125cc8a902072d7c0acebc> (19) Patch Links: - https://github.com/babashka/process/pull/92.patch - https://github.com/babashka/process/pull/92.diff — Reply to this email directly, view it on GitHub <#92>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACFSBSV2R5DJ7WNFY644H3WB67L7ANCNFSM6AAAAAAQ7JDCNI> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

-- https://www.michielborkent.nl https://www.eetvoorjeleven.nu

[grzm]

Thanks!