Commit eed29ebb by Corentin Wallez

Simplify the assignment of use_x11.

Hopefully fixes the flaky compile errors in ChromeOS BUG=angleproject:892 BUG=503173 Change-Id: Ifd79f1e382c12be3aecd9efb4860287df7d43060 Reviewed-on: https://chromium-review.googlesource.com/281180Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org> Tested-by: 's avatarCorentin Wallez <cwallez@chromium.org>
parent 3018c206
...@@ -36,17 +36,23 @@ ...@@ -36,17 +36,23 @@
'-Wno-format-nonliteral', '-Wno-format-nonliteral',
], ],
'chromeos%': 0, # Nested dictionnary hack to allow setting a default value for
'conditions': # a variable and test it in the dict (merged in angle.gyp).
[ 'variables':
['OS=="linux" and chromeos==0', {
{ 'conditions':
'use_x11%': 1, [
}, ['OS=="linux"',
{ {
'use_x11%': 0, 'use_x11': 1,
}], },
], {
'use_x11': 0,
}],
],
},
# Copy conditionally-set variables out one scope.
'use_x11%': '<(use_x11)',
}, },
'target_defaults': 'target_defaults':
{ {
......
...@@ -17,25 +17,6 @@ ...@@ -17,25 +17,6 @@
'angle_enable_d3d11%': 0, 'angle_enable_d3d11%': 0,
'angle_enable_gl%': 0, 'angle_enable_gl%': 0,
'angle_enable_hlsl%': 0, 'angle_enable_hlsl%': 0,
# This nested dictionary hack comes from Chromium's build/common.gypi
# and is needed so we can both set a default value for this variable and
# test it in the same file.
'variables':
{
'chromeos%': 0,
'conditions':
[
['OS=="linux" and chromeos==0',
{
'use_x11%': 1,
},
{
'use_x11%': 0,
}],
],
},
# Copy conditionally-set variables out one scope.
'use_x11%': '<(use_x11)',
'conditions': 'conditions':
[ [
['OS=="win"', ['OS=="win"',
......
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