diff options
| author | Brian Norris <briannorris@chromium.org> | 2022-09-02 13:40:17 -0700 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-09-07 16:19:18 +0000 |
| commit | f6b4581c17e4ce2132cb04d4180c326d628b7426 (patch) | |
| tree | 078ef6fc232bda7105ff962cbf24bde10688a398 | |
| parent | 27ff437ee579e03bc0a584a0be1e11b2629bbffb (diff) | |
| download | chrome-ec-f6b4581c17e4ce2132cb04d4180c326d628b7426.tar.gz | |
ec: Fix DEBUG "variable" in usb_updater Makefile (retry)
DEBUG is not the way to refer to a variable in a Makefile expression;
$(DEBUG) is.
Make extra/rma_reset/Makefile consistent with this too.
Reland after gsctool fixed some build errors:
https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3876011
BUG=b:245028043
TEST=build (chromeos-cr50-dev)
Signed-off-by: Brian Norris <briannorris@chromium.org>
Change-Id: I6c9ac95522c987e520123b10f7c722a21bb5ecb8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3877390
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
| -rw-r--r-- | extra/rma_reset/Makefile | 2 | ||||
| -rw-r--r-- | extra/usb_updater/Makefile | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/extra/rma_reset/Makefile b/extra/rma_reset/Makefile index 4a640c5b4c..4e401b8d5d 100644 --- a/extra/rma_reset/Makefile +++ b/extra/rma_reset/Makefile @@ -19,7 +19,7 @@ CFLAGS := -std=gnu99 \ -Wredundant-decls \ -Wmissing-declarations -ifeq ($(DEBUG),1) +ifneq ($(DEBUG),) CFLAGS += -g -O0 else CFLAGS += -O3 diff --git a/extra/usb_updater/Makefile b/extra/usb_updater/Makefile index 04adfff4dd..ac21ba8e93 100644 --- a/extra/usb_updater/Makefile +++ b/extra/usb_updater/Makefile @@ -19,10 +19,10 @@ CFLAGS := -std=gnu99 \ -Wredundant-decls \ -Wmissing-declarations -ifeq (DEBUG,) -CFLAGS += -O3 -else +ifneq ($(DEBUG),) CFLAGS += -O0 +else +CFLAGS += -O3 endif # |