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

Side by Side Diff: native_client_sdk/src/tools/common.mk

Issue 22766003: [NaCl SDK] Fix PNACL for debugging. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 4 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # 5 #
6 # GNU Make based build file. For details on GNU Make see: 6 # GNU Make based build file. For details on GNU Make see:
7 # http://www.gnu.org/software/make/manual/make.html 7 # http://www.gnu.org/software/make/manual/make.html
8 # 8 #
9 9
10 # 10 #
(...skipping 459 matching lines...) Expand 10 before | Expand all | Expand 10 after
470 run: check_for_chrome all $(PAGE) 470 run: check_for_chrome all $(PAGE)
471 $(RUN_PY) -C $(CURDIR) -P $(PAGE_TC_CONFIG) \ 471 $(RUN_PY) -C $(CURDIR) -P $(PAGE_TC_CONFIG) \
472 $(addprefix -E ,$(CHROME_ENV)) -- $(CHROME_PATH) $(CHROME_ARGS) \ 472 $(addprefix -E ,$(CHROME_ENV)) -- $(CHROME_PATH) $(CHROME_ARGS) \
473 --register-pepper-plugins="$(PPAPI_DEBUG),$(PPAPI_RELEASE)" 473 --register-pepper-plugins="$(PPAPI_DEBUG),$(PPAPI_RELEASE)"
474 474
475 .PHONY: run_package 475 .PHONY: run_package
476 run_package: check_for_chrome all 476 run_package: check_for_chrome all
477 $(CHROME_PATH) --load-and-launch-app=$(CURDIR) $(CHROME_ARGS) 477 $(CHROME_PATH) --load-and-launch-app=$(CURDIR) $(CHROME_ARGS)
478 478
479 479
480 GDB_ARGS += -D $(TC_PATH)/$(OSNAME)_x86_$(TOOLCHAIN)/bin/$(SYSARCH)-nacl-gdb 480 # name gdb for everyone ?!?!?
Sam Clegg 2013/08/10 00:54:01 /name/same/. Perhaps a better comment? Perhaps
noelallen1 2013/08/10 19:08:49 Adding a bug to fix this. BUG=271183
481 GDB_ARGS += -D $(TC_PATH)/$(OSNAME)_x86_newlib/bin/$(SYSARCH)-nacl-gdb
481 GDB_ARGS += -D $(abspath $(OUTDIR))/$(TARGET)_$(SYSARCH).nexe 482 GDB_ARGS += -D $(abspath $(OUTDIR))/$(TARGET)_$(SYSARCH).nexe
482 483
483 .PHONY: debug 484 .PHONY: debug
484 debug: check_for_chrome all $(PAGE) 485 debug: check_for_chrome all $(PAGE)
485 $(RUN_PY) $(GDB_ARGS) \ 486 $(RUN_PY) $(GDB_ARGS) \
486 -C $(CURDIR) -P $(PAGE_TC_CONFIG) \ 487 -C $(CURDIR) -P $(PAGE_TC_CONFIG) \
487 $(addprefix -E ,$(CHROME_ENV)) -- $(CHROME_PATH) $(CHROME_ARGS) \ 488 $(addprefix -E ,$(CHROME_ENV)) -- $(CHROME_PATH) $(CHROME_ARGS) \
488 --enable-nacl-debug \ 489 --enable-nacl-debug \
489 --register-pepper-plugins="$(PPAPI_DEBUG),$(PPAPI_RELEASE)" 490 --register-pepper-plugins="$(PPAPI_DEBUG),$(PPAPI_RELEASE)"
490 endif 491 endif
491 492
492 493
493 # uppercase aliases (for backward compatibility) 494 # uppercase aliases (for backward compatibility)
494 .PHONY: CHECK_FOR_CHROME DEBUG LAUNCH RUN 495 .PHONY: CHECK_FOR_CHROME DEBUG LAUNCH RUN
495 CHECK_FOR_CHROME: check_for_chrome 496 CHECK_FOR_CHROME: check_for_chrome
496 DEBUG: debug 497 DEBUG: debug
497 LAUNCH: run 498 LAUNCH: run
498 RUN: run 499 RUN: run
499 500
500 endif # TOOLCHAIN is valid... 501 endif # TOOLCHAIN is valid...
501 502
502 endif # TOOLCHAIN=all 503 endif # TOOLCHAIN=all
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698