From 297b9ae147b590f03597e00d8bb4eee8ea7c347d Mon Sep 17 00:00:00 2001 From: Karl Skomski Date: Mon, 7 Sep 2015 23:57:18 +0200 Subject: [PATCH] build: fix v8_enable_handle_zapping override It was previously ignored by features.gypi and therefore enabled by default for release builds. See https://code.google.com/p/chromium/issues/detail?id=318206 PR-URL: https://github.com/nodejs/node/pull/2731 Reviewed-By: Fedor Indutny Reviewed-By: Ben Noordhuis --- common.gypi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/common.gypi b/common.gypi index 7819816bbbd..c57cc5b7b3a 100644 --- a/common.gypi +++ b/common.gypi @@ -56,7 +56,7 @@ 'configurations': { 'Debug': { 'variables': { - 'v8_enable_handle_zapping%': 1, + 'v8_enable_handle_zapping': 1, }, 'defines': [ 'DEBUG', '_DEBUG' ], 'cflags': [ '-g', '-O0' ], @@ -83,7 +83,7 @@ }, 'Release': { 'variables': { - 'v8_enable_handle_zapping%': 0, + 'v8_enable_handle_zapping': 0, }, 'cflags': [ '-O3', '-ffunction-sections', '-fdata-sections' ], 'conditions': [