Skip to content

Commit

Permalink
Fix chromium builds with is_cobalt = false (#4298)
Browse files Browse the repository at this point in the history
b/374988048

Set is_cobalt = false for chromium only(non cobalt builds)
  • Loading branch information
niranjanyardi authored Oct 23, 2024
1 parent 8333551 commit a828c6e
Show file tree
Hide file tree
Showing 11 changed files with 8 additions and 98 deletions.
1 change: 1 addition & 0 deletions build/config/clang/clang.gni
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
# found in the LICENSE file.

import("//build/toolchain/toolchain.gni")
import("//cobalt/build/configs/cobalt.gni")

default_clang_base_path = "//third_party/llvm-build/Release+Asserts"

Expand Down
1 change: 1 addition & 0 deletions cobalt/build/configs/android-arm/args.gn
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
target_os = "android"
target_cpu = "arm"
is_cobalt = false
1 change: 1 addition & 0 deletions cobalt/build/configs/android-arm64/args.gn
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
target_os = "android"
target_cpu = "arm64"
is_cobalt = false
2 changes: 0 additions & 2 deletions cobalt/build/configs/android-cobalt-arm/args.gn
Original file line number Diff line number Diff line change
@@ -1,4 +1,2 @@
target_os = "android"
target_cpu = "arm"

is_cobalt = true
2 changes: 0 additions & 2 deletions cobalt/build/configs/android-cobalt-arm64/args.gn
Original file line number Diff line number Diff line change
@@ -1,4 +1,2 @@
target_os = "android"
target_cpu = "arm64"

is_cobalt = true
2 changes: 0 additions & 2 deletions cobalt/build/configs/android-cobalt-x86/args.gn
Original file line number Diff line number Diff line change
@@ -1,4 +1,2 @@
target_os = "android"
target_cpu = "x86"

is_cobalt = true
1 change: 1 addition & 0 deletions cobalt/build/configs/android-x86/args.gn
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
target_os = "android"
target_cpu = "x86"
is_cobalt = false
4 changes: 3 additions & 1 deletion cobalt/build/configs/cobalt.gni
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,6 @@
# Get the path to the starboard implementation and include its GN
# configuration.

is_cobalt = true
declare_args() {
is_cobalt = true
}
2 changes: 0 additions & 2 deletions cobalt/build/configs/linux-cobalt-x64x11/args.gn
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
target_os = "linux"
target_cpu = "x64"

is_cobalt = true

enable_nacl = false
use_dawn = false
skia_use_dawn = false
Expand Down
89 changes: 0 additions & 89 deletions cobalt/build/configs/linux-x64/args.gn

This file was deleted.

1 change: 1 addition & 0 deletions cobalt/build/configs/linux-x64x11/args.gn
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
target_os = "linux"
target_cpu = "x64"
target_platform = "linux-x64x11"
is_cobalt = false

# Chromium flag overrides needed for building libstarboard

Expand Down

0 comments on commit a828c6e

Please sign in to comment.