Commit 4d2ee242 authored by scottmg's avatar scottmg Committed by Commit bot

gn: Add missing NDEBUG define in Release

This is required at least on Windows as it's used by system headers to
control things like assert, etc.

R=brettw@chromium.org
BUG=335824,354261

Review URL: https://codereview.chromium.org/475153002

Cr-Commit-Position: refs/heads/master@{#293788}
parent 9f7acae4
...@@ -210,6 +210,9 @@ config("debug") { ...@@ -210,6 +210,9 @@ config("debug") {
} }
config("release") { config("release") {
defines = [
"NDEBUG",
]
} }
# Default libraries ------------------------------------------------------------ # Default libraries ------------------------------------------------------------
......
...@@ -44,6 +44,11 @@ static_library("libphonenumber_without_metadata") { ...@@ -44,6 +44,11 @@ static_library("libphonenumber_without_metadata") {
"src/phonenumbers/utf/unilib.cc", "src/phonenumbers/utf/unilib.cc",
] ]
# TODO(scottmg): Temporary until https://codereview.appspot.com/134650043/
# lands and rolls. http://crbug.com/335824
configs -= [ "//build/config/compiler:chromium_code" ]
configs += [ "//build/config/compiler:no_chromium_code" ]
direct_dependent_configs = [ ":libphonenumber_config" ] direct_dependent_configs = [ ":libphonenumber_config" ]
deps = [ deps = [
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment