Re: NEW: devel/leiningen

Hi Jon,

On Thu, Jun 29, 2017 at 10:38:32AM -0400, Jon Bernard wrote:
> Attached is a port of leiningen, for automating clojure projects.

Comments:

* There's a hard-coded path to bash in a patch. You will need to use a
variable (LOCALBASE) in place of /usr/local and then use
${SUBST_CMD} in a custom make target, maybe post-install.

* Did you try that script with /bin/sh? If it works that would be
preferable. Otherwise, bash needs to be a RUN_DEPEND.

* mandoc linter is unhappy with the man page:

$ mandoc -Tlint /usr/local/man/man1/lein.1
mandoc: /usr/local/man/man1/lein.1:2:17: WARNING: cannot parse date, using it verbatim: 2011 June 30

Consider pathcing?

* I was surprised to see that your port did not RUN_DEPEND on clojure.
I see that upon first `lein repl` the tool will download a clojure
using (the horror) maven (which must be bundled?). I *think*, but
other porters can correct me, we would prefer to use the in-tree
clojure instead of a moving target. Is this possible?

* Even if I let it do its thing, the repl doesn't seem to work:

---8<---
$ lein repl
Retrieving org/clojure/clojure/1.8.0/clojure-1.8.0.pom from central
Retrieving org/sonatype/oss/oss-parent/7/oss-parent-7.pom from central
Retrieving org/clojure/tools.nrepl/0.2.12/tools.nrepl-0.2.12.pom from central
Retrieving org/clojure/pom.contrib/0.1.2/pom.contrib-0.1.2.pom from central
Retrieving clojure-complete/clojure-complete/0.2.4/clojure-complete-0.2.4.pom from clojars
Retrieving org/clojure/tools.nrepl/0.2.12/tools.nrepl-0.2.12.jar from central
Retrieving org/clojure/clojure/1.8.0/clojure-1.8.0.jar from central
Retrieving clojure-complete/clojure-complete/0.2.4/clojure-complete-0.2.4.jar from clojars
REPL server launch timed out.
--->8---

I know nothing about Clojure or lein, so I'm going to CC our clojure
MAINTAINER, Jasper.

Cheers

--
Best Regards
Edd Barrett

http://www.theunixzoo.co.uk