Chromium Code Reviews| Index: build/config/BUILDCONFIG.gn |
| diff --git a/build/config/BUILDCONFIG.gn b/build/config/BUILDCONFIG.gn |
| index c91db167b34dcc8adcd98dab2e09835599b89727..77f08ad9d06c419c75f308333761ec78316ecb25 100644 |
| --- a/build/config/BUILDCONFIG.gn |
| +++ b/build/config/BUILDCONFIG.gn |
| @@ -125,8 +125,14 @@ declare_args() { |
| # Debug build. |
| is_debug = true |
| + # Whether we're building components and interfaces for ChromeOS |
| + # This is for backwards compatability until the rename in complete in all |
| + # dependent files. See use_cros_ui in ui.gni. They should be the same. |
| + # TODO(petermayo): Remove this when it doesn't break known compiles. |
| + is_chromeos = current_os == "chromeos" |
| + |
| # Whether we're a traditional desktop unix. |
| - is_desktop_linux = current_os == "linux" && current_os != "chromeos" |
| + is_desktop_linux = current_os == "linux" && !is_chromeos |
| # Set to true when compiling with the Clang compiler. Typically this is used |
| # to configure warnings. |
| @@ -162,12 +168,16 @@ declare_args() { |
| # We set these various is_FOO booleans for convenience in writing OS-based |
| # conditions. |
| # |
| -# - is_android, is_chromeos, is_ios, and is_win should be obvious. |
| +# - is_android, is_ios, and is_win should be obvious. |
| # - is_mac is set only for desktop Mac. It is not set on iOS. |
| # - is_posix is true for mac and any Unix-like system (basically everything |
| # except Windows). |
| # - is_linux is true for desktop Linux and ChromeOS, but not Android (which is |
| # generally too different despite being based on the Linux kernel). |
| +# - is_chromeos_os is true if the results are expected to run on a ChromeOS |
| +# machine, using the filesystem and runtime libraries specifed by |
| +# a build of that OS, rather than local linux. E.g. presume an update |
| +# system, journalled files system, storage modes, no other apps, etc. |
| # |
| # Do not add more is_* variants here for random lesser-used Unix systems like |
| # aix or one of the BSDs. If you need to check these, just check the |
| @@ -176,7 +186,7 @@ declare_args() { |
| if (current_os == "win" || current_os == "winrt_81" || |
| current_os == "winrt_81_phone" || current_os == "winrt_10") { |
| is_android = false |
| - is_chromeos = false |
| + is_chromeos_os = false |
| is_ios = false |
| is_linux = false |
| is_mac = false |
| @@ -185,7 +195,7 @@ if (current_os == "win" || current_os == "winrt_81" || |
| is_win = true |
| } else if (current_os == "mac") { |
| is_android = false |
| - is_chromeos = false |
| + is_chromeos_os = false |
| is_ios = false |
| is_linux = false |
| is_mac = true |
| @@ -194,7 +204,7 @@ if (current_os == "win" || current_os == "winrt_81" || |
| is_win = false |
| } else if (current_os == "android") { |
| is_android = true |
| - is_chromeos = false |
| + is_chromeos_os = false |
| is_ios = false |
| is_linux = false |
| is_mac = false |
| @@ -203,7 +213,8 @@ if (current_os == "win" || current_os == "winrt_81" || |
| is_win = false |
| } else if (current_os == "chromeos") { |
| is_android = false |
| - is_chromeos = true |
| + assert(is_chromeos, "ChromiumOS requires is_chromeos.") |
| + is_chromeos_os = true |
| is_ios = false |
| is_linux = true |
| is_mac = false |
| @@ -215,7 +226,7 @@ if (current_os == "win" || current_os == "winrt_81" || |
| # It is not set by default or on the command line. We treat is as a |
| # Posix variant. |
| is_android = false |
| - is_chromeos = false |
| + is_chromeos_os = false |
| is_ios = false |
| is_linux = false |
| is_mac = false |
| @@ -224,7 +235,7 @@ if (current_os == "win" || current_os == "winrt_81" || |
| is_win = false |
| } else if (current_os == "ios") { |
| is_android = false |
| - is_chromeos = false |
| + is_chromeos_os = false |
| is_ios = true |
| is_linux = false |
| is_mac = false |
| @@ -233,7 +244,7 @@ if (current_os == "win" || current_os == "winrt_81" || |
| is_win = false |
| } else if (current_os == "linux") { |
| is_android = false |
| - is_chromeos = false |
| + is_chromeos_os = false |
| is_ios = false |
| is_linux = true |
| is_mac = false |
| @@ -328,6 +339,8 @@ if (!is_android) { |
| "*\bandroid/*", |
| ] |
| } |
| + |
| +#TODO(petermayo): Move this to ui.gni with the removal of is_chromeos |
|
Peter Mayo
2015/08/19 21:42:05
This affects a lot of places. North of 400 direct
|
| if (!is_chromeos) { |
| sources_assignment_filter += [ |
| "*_chromeos.h", |