build: fix RELEASE check (URGENT)

[rvagg]

Ref: #1405

please review @jbergstroem || @bnoordhuis asap or I'm just going to merge this and release a 1.7.1, 1.7.0 is a write-off because this is broken unfortunately.

[rvagg]

for reference, the error on a release build is:

#NODE_VERSION_IS_RELEASE is set to (hell sed -ne 's/.
Did you remember to update src/node_version.h?

I was weary about this change and even went to the effort of testing the suggested sed change on a few platforms, pity I missed the syntax error in the process.

[rvagg]

meh, merging without review

[rvagg]

landed @ fa91b18

[jbergstroem]

For what its worth: post-merge LGTM. Sorry about the slip-up.

[rvagg]

still no good, on OSX:

Makefile:200: *** unterminated call to function `shell': missing `)'.  Stop.

I don't even ...

Makefiles :shakesfist: and cross-platform sed :shakesfist:

[jbergstroem]

I can't reproduce. Tried on gnu make 3.81:

# cat foo 
all:
    RELEASE=$(shell sed -ne 's/#define NODE_VERSION_IS_RELEASE \([01]\)/\1/p' src/node_version.h)

# make -f foo
RELEASE=0

[rvagg]

force-pushed another fix in this from

RELEASE=$(shell sed -ne 's/#define NODE_VERSION_IS_RELEASE \([01]\)/\1/p' src/node_version.h)

to

RELEASE=$(shell sed -ne 's/\#define NODE_VERSION_IS_RELEASE \([01]\)/\1/p' src/node_version.h)

it's now aee86a2

thanks @jbergstroem for spotting that as the problem