tree 02d6f1c0d7ba01960b8de5dc3cca62d9c354060b
parent 01fade1fc521f30d8ee55b75c8f736b674c31fbc
author Nicolas Capens <capn@google.com> 1665148272 -0400
committer Nicolas Capens <nicolascapens@google.com> 1665151613 +0000

Fix enabling XCB and Wayland WSI builds on Linux

CMake variables need to be dereferenced using ${...} before passing them
as function arguments, so we can't just use LINUX as the default value
for the cache variables. However, we also can't use ${LINUX} because the
variable isn't defined on non-Linux platforms and the expression
evaluates to the empty string, meaning we're missing an argument for the
option_if_not_defined() function.

Instead this change simply does not define these cache variables if
we're not building for Linux, and sets the XCB and Wayland ones to
literal TRUE as default.

Bug: b/249015129
Change-Id: I2d171cf8dae025e75f4ce81f717f38051664c0f8
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/68749
Tested-by: Nicolas Capens <nicolascapens@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>