[Nix-dev] npm prepublish hook

[Mathijs Kwik]

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?