On Sun, Jul 02, 2017 at 05:12:15PM +0100, Edd Barrett wrote:
> 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
We used to have a port of leiningen, but I removed it more than 3 years ago:
------------8<------------
Revision 1.6, Sat Nov 9 10:40:20 2013 UTC (3 years, 7 months ago) by jasper
Branch: MAIN
CVS Tags: HEAD
Changes since 1.5: +1 -1 lines
FILE REMOVED
remove leiningen; the upstream script is not really intended to get packaged
anyway.
------------8<------------
Might be worth considering before proceeding to re-add the files if the
situation has improved since 2013.
--
jasper
