[Nix-dev] npm prepublish hook

[Shea Levy]

Heh, OK. I will make an issue to track/fix this.

On 12/10/2013 09:34 AM, Mathijs Kwik wrote:
> I'm not gonna push broken stuff :)
>
> On Tue, Dec 10, 2013 at 3:23 PM, Shea Levy <shea at shealevy.com> wrote:
>> On 12/10/2013 09:16 AM, Mathijs Kwik wrote:
>>> On Tue, Dec 10, 2013 at 3:01 PM, Shea Levy <shea at shealevy.com> wrote:
>>>> maintaining a manual list, as unfortunate as that sounds. Hopefully this
>>>> will be fixed soon in most packages.
>>>>
>>>> For packages that are installed from a tarball (which is most), issues
>>>> with
>>>> read-only paths shouldn't come up. If there's a common set of utilities
>>>> always expected in $PATH we may as well resign ourselves to adding those
>>>> to
>>>> buildInputs unconditionally.
>>> The 2 cases I found are:
>>> bytebuffer-2.1.0:
>>>     bug in the package itself, wants to run hardcoded path
>>> ./node_modules/testjs/bin/testjs
>>>     but of course this path is not available in the (extracted) tarball.
>>> protobufjs:
>>>     runs tests like "test1 > tests/test1.out", which will fail because
>>> tests is not writable, so the issue _does_ come up.
>> Hmm, that first one could be fixed by running the npm install in $out since
>> we've already set up the deps there. Not sure about the second one, I can't
>> seem to find it in nixpkgs?