Commit 0c0d8006 by Frank Henigman Committed by Jamie Madill

Revert "Gyp build support for Chrome OS."

http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_rel_ng/builds/189745 ../../third_party/angle/src/libANGLE/renderer/gl/glx/DisplayGLX.cpp:87:27: error: assigning to 'Display *' (aka '_XDisplay *') from incompatible type 'EGLNativeDisplayType' (aka 'long') mXDisplay = display->getNativeDisplayId(); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../../third_party/angle/src/libANGLE/renderer/gl/glx/DisplayGLX.cpp:93:19: error: comparison between pointer and integer ('Display *' (aka '_XDisplay *') and 'EGLNativeDisplayType' (aka 'long')) if (mXDisplay == EGL_DEFAULT_DISPLAY) ~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~ TBR=kbr@chromium.org TBR=jmadill@chromium.org BUG=angleproject:1297 This reverts commit b5aa26bd. Change-Id: Icf3bd84df4493e66e78f8351b1a3879893ae9d0e Reviewed-on: https://chromium-review.googlesource.com/336849Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org>
parent 3477f3a6
...@@ -15,14 +15,11 @@ ...@@ -15,14 +15,11 @@
'angle_build_winphone%': '0', 'angle_build_winphone%': '0',
'conditions': 'conditions':
[ [
['OS=="linux" and use_x11==1', ['OS=="linux" and use_x11==1 and chromeos==0', {
{
'angle_use_glx%': 1, 'angle_use_glx%': 1,
'angle_use_drm%': 0,
}, },
{ {
'angle_use_glx%': 0, 'angle_use_glx%': 0,
'angle_use_drm%': 1,
}], }],
], ],
...@@ -41,10 +38,6 @@ ...@@ -41,10 +38,6 @@
], ],
'conditions': 'conditions':
[ [
['chromeos==1',
{
'defines': [ 'USE_OZONE' ],
}],
['component=="shared_library"', ['component=="shared_library"',
{ {
'defines': [ 'COMPONENT_BUILD' ], 'defines': [ 'COMPONENT_BUILD' ],
......
...@@ -5,22 +5,24 @@ ...@@ -5,22 +5,24 @@
{ {
'variables': 'variables':
{ {
# Chrome OS chroot builds need a special pkg-config, # Assume for the time being that we're never compiling
# so make it possible to change. # standalone ANGLE on Chrome OS.
'pkg-config%': 'pkg-config', 'chromeos': 0,
# Chromium puts the pkg-config command in a variable because
# calling pkg-config directly doesn't work in a sysroot image.
# See Chromium bug 569947 for more information.
# For standalone builds we don't have this problem so we use
# the regular command.
'pkg-config': 'pkg-config',
# Use a nested variable trick to get use_x11 evaluated more # Use a nested variable trick to get use_x11 evaluated more
# eagerly than other conditional variables. # eagerly than other conditional variables.
'variables': 'variables':
{ {
'variables':
{
'chromeos%': 0,
},
'conditions': 'conditions':
[ [
['OS=="linux" and chromeos==0', ['OS=="linux"',
{ {
'use_x11': 1, 'use_x11': 1,
}, },
...@@ -28,13 +30,9 @@ ...@@ -28,13 +30,9 @@
'use_x11': 0, 'use_x11': 0,
}], }],
], ],
# Copy conditionally-set variables out one scope.
'chromeos%': '<(chromeos)',
}, },
# Copy conditionally-set variables out one scope. # Copy conditionally-set variables out one scope.
'use_x11%': '<(use_x11)', 'use_x11%': '<(use_x11)',
'chromeos%': '<(chromeos)',
}, },
} }
...@@ -30,7 +30,7 @@ ...@@ -30,7 +30,7 @@
'angle_enable_d3d11%': 1, 'angle_enable_d3d11%': 1,
'angle_enable_hlsl%': 1, 'angle_enable_hlsl%': 1,
}], }],
['OS=="linux"', ['OS=="linux" and use_x11==1 and chromeos==0',
{ {
'angle_enable_gl%': 1, 'angle_enable_gl%': 1,
}], }],
......
...@@ -1013,9 +1013,6 @@ ...@@ -1013,9 +1013,6 @@
[ [
'<(deqp_path)/framework/platform/x11', '<(deqp_path)/framework/platform/x11',
], ],
}],
['OS=="linux"',
{
'deqp_defines': 'deqp_defines':
[ [
# Ask the system headers to expose all the regular function otherwise # Ask the system headers to expose all the regular function otherwise
...@@ -1376,7 +1373,7 @@ ...@@ -1376,7 +1373,7 @@
{ {
'sources': [ '<@(deqp_libtester_sources_win)', ], 'sources': [ '<@(deqp_libtester_sources_win)', ],
}], }],
['OS=="linux" or OS=="mac"', ['(OS=="linux" and use_x11==1) or OS=="mac"',
{ {
'sources': [ '<@(deqp_libtester_sources_unix)', ], 'sources': [ '<@(deqp_libtester_sources_unix)', ],
}], }],
......
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