Re: [Nix-dev] Leiningen package

2012-01-19 Thread Yury G. Kudryashov
Phil Hagelberg wrote: A few comments. 1. I'd prefer more '-v' in builder.sh. E.g., mkdir -pv, cp -v etc. Good idea. I've attached an updated patch for this. # patchPhase seems to assume you have a tarball, not a single file patchPhase assume you have a '-p1' patch. Makes sense.

Re: [Nix-dev] Leiningen package

2012-01-19 Thread Phil Hagelberg
Yury G. Kudryashov urkud.ur...@gmail.com writes: Pushed. Sorry, forgotten that I'm using git-svn, not git-push, so didn't added patch by Phil Hagelberg to commit-message. No worries. Do you want to recieve e-mails from our continuous build system? That would be great; thank you! -Phil

Re: [Nix-dev] Leiningen package

2012-01-18 Thread Marc Weber
Excerpts from Phil Hagelberg's message of Wed Jan 18 02:24:11 +0100 2012: Makes sense. Should I try to adapt it to patchPhase, or does it not matter since it's such a simple invocation? Either add leading a/ b/ or use patchPhase=patch ${your-patch}; I'd not spend too much time on a patch

[Nix-dev] Leiningen package

2012-01-17 Thread Phil Hagelberg
Hello! I'm the author of Leiningen, the primary build tool used by Clojure developers. I've packaged it up for nix in the attached patch. Would this be acceptable to add to nixpkgs? thanks, Phil From b9284771628c4beeefce8c28c826c98b58063ff0 Mon Sep 17 00:00:00 2001 From: Phil Hagelberg

Re: [Nix-dev] Leiningen package

2012-01-17 Thread Yury G. Kudryashov
Phil Hagelberg wrote: Hello! I'm the author of Leiningen, the primary build tool used by Clojure developers. I've packaged it up for nix in the attached patch. Would this be acceptable to add to nixpkgs? A few comments. 1. I'd prefer more '-v' in builder.sh. E.g., mkdir -pv, cp -v etc.

Re: [Nix-dev] Leiningen package

2012-01-17 Thread Phil Hagelberg
A few comments. 1. I'd prefer more '-v' in builder.sh. E.g., mkdir -pv, cp -v etc. Good idea. I've attached an updated patch for this. # patchPhase seems to assume you have a tarball, not a single file patchPhase assume you have a '-p1' patch. Makes sense. Should I try to adapt it to