chore: move xbox toolchain to sentry-xbox repository#1329
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
supervacuus
left a comment
There was a problem hiding this comment.
Just blocking this so it doesn't get accidentally merged.
| elseif(CMAKE_SYSTEM_NAME STREQUAL "Prospero") | ||
| set(PROSPERO TRUE) | ||
| elseif("${CMAKE_GENERATOR_PLATFORM}" MATCHES "Gaming\\.Xbox\\.(Scarlett|XboxOne)\\.x64") | ||
| set(XBOX TRUE) |
There was a problem hiding this comment.
Bug: Xbox Build Failures Due to Incorrect Variable Placement
Xbox support is broken. The new XBOX variable detection logic is incorrectly placed within an elseif chain that checks CMAKE_SYSTEM_NAME. Since Xbox builds set CMAKE_SYSTEM_NAME to "WINDOWS", the CMAKE_GENERATOR_PLATFORM condition for Xbox is never reached, and XBOX is never set. This, coupled with the removal of Xbox toolchain files that generated gdk_build.props (a file referenced by later CMake logic when XBOX is true), leads to build failures for Xbox targets.
There was a problem hiding this comment.
that's not true, there's no CMAKE_SYSTEM_NAME == windows condition in this if-elseif chain
Having the xbox support spread out between multiple repositories (sentry-native and sentry-xbox) induces a maintenance burden. Also, it's a bit of a pain when including toolchains in the downstream sentry-xbox due to having first fetch this repo before being able to use them in cmake.
This PR removes the toolchains from this repo. Additionally, I've removed changelog items from the unreleased versionnot relevant here anymore.