Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(135)

Unified Diff: native_client_sdk/src/tools/common.mk

Issue 272243006: [NaCl SDK] Various fixes so "make debug" works on Windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge master Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | native_client_sdk/src/tools/run.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: native_client_sdk/src/tools/common.mk
diff --git a/native_client_sdk/src/tools/common.mk b/native_client_sdk/src/tools/common.mk
index e09f3e518bb74133413d512d7044b16ac87b9ea1..15cf783d2a3f5d8d85a1816794017046058e33d0 100644
--- a/native_client_sdk/src/tools/common.mk
+++ b/native_client_sdk/src/tools/common.mk
@@ -477,7 +477,14 @@ CHROME_PATH ?= $(shell $(GETOS) --chrome 2> $(DEV_NULL))
NULL :=
SPACE := $(NULL) # one space after NULL is required
-CHROME_PATH_ESCAPE := $(subst $(SPACE),\ ,$(CHROME_PATH))
+ifneq ($(OSNAME),win)
+ CHROME_PATH_ESCAPE := $(subst $(SPACE),\ ,$(CHROME_PATH))
+ SANDBOX_ARGS :=
+else
+ CHROME_PATH_ESCAPE := $(CHROME_PATH)
+ SANDBOX_ARGS := --no-sandbox
+endif
+
GDB_PATH := $(shell $(NACL_CONFIG) -t $(TOOLCHAIN) --tool=gdb)
.PHONY: check_for_chrome
@@ -495,7 +502,7 @@ PAGE_TC_CONFIG ?= "$(PAGE)?tc=$(TOOLCHAIN)&config=$(CONFIG)"
run: check_for_chrome all $(PAGE)
$(RUN_PY) -C $(CURDIR) -P $(PAGE_TC_CONFIG) \
$(addprefix -E ,$(CHROME_ENV)) -- $(CHROME_PATH_ESCAPE) \
- $(CHROME_ARGS) --no-sandbox \
+ $(CHROME_ARGS) \
--register-pepper-plugins="$(PPAPI_DEBUG),$(PPAPI_RELEASE)"
.PHONY: run_package
@@ -515,7 +522,7 @@ debug: check_for_chrome all $(PAGE)
$(RUN_PY) $(GDB_ARGS) \
-C $(CURDIR) -P $(PAGE_TC_CONFIG) \
$(addprefix -E ,$(CHROME_ENV)) -- $(CHROME_PATH_ESCAPE) \
- $(CHROME_ARGS) --enable-nacl-debug \
+ $(CHROME_ARGS) $(SANDBOX_ARGS) --enable-nacl-debug \
--register-pepper-plugins="$(PPAPI_DEBUG),$(PPAPI_RELEASE)"
.PHONY: serve
« no previous file with comments | « no previous file | native_client_sdk/src/tools/run.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698