Subject: Re: [bug#27852] [PATCH] gnu: node: Update to 8.2.1.

From: [email-masked] (Ludovic Courtès)
Date: Sat, 02 Sep 2017 00:20:11 +0200

Jelle, it looks like this fell through the cracks a warm summer
inevitably creates, no?  :-)

Ludo’.

[email-masked] (Ludovic Courtès) skribis:

> Jelle Licht <[email-masked]> skribis:
>
>> From f88297638a82858dff61176f646cdc4ddca748cf Mon Sep 17 00:00:00 2001
>> From: Jelle Licht <[email-masked]>
>> Date: Thu, 27 Jul 2017 17:11:08 +0200
>> Subject: [PATCH] gnu: node: Update to 8.2.1.
>>
>> * gnu/packages/node.scm (node): Update to 8.2.1.
>> (node)[arguments]: Removed broken linter test.
>
> LGTM!
>
>> -             ;; Having the build fail because of linter errors is insane!
>> +             ;; Linting is no longer supported in the release tarball
>>               (substitute* '("Makefile")
>> -               (("	\\$\\(MAKE\\) jslint") "")
>> -               (("	\\$\\(MAKE\\) cpplint\n") ""))
>> +               (("	\\$\\(MAKE\\) lint") ""))
>
> Maybe add a link to <https://github.com/nodejs/node/issues/14513> for
> future reference?
>
> Thanks,
> Ludo’.