bkirkbri/im4clj — comment on commit 5c57891a7e4508c6b41523af730561b22d9ea967
File:  line 0
By neatonk:

I like the use of `stringify` and protocols, and I appreciate the reduction it total lines of code. However, I wonder what is gained by combining `im4clj.run` into `im4clj.im4java`?

My goal in maintaining a strict string-only interface to `im4java.run`, was to allow allow the use of other commandline runners in the future... `clojure.java.shell` or`pallet.shell`. Both of these libs accept a sequence of strings arguments followed by zero or more keyword options. The code in `im4clj.im4java` attempts to wrap a similar interface around im4java. This interface could be extended to accept keyword options as well, as a way of configuring the ImageCommand being created. If your additions were moved to `im4clj.run` i would be agree with them 100%.