New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Haiku: Update arcade support #13755
Haiku: Update arcade support #13755
Conversation
10106c9
to
af775fd
Compare
7e4ec38
to
7b784d7
Compare
0b315b6
to
1d0176a
Compare
62a9525
to
d752f5e
Compare
d752f5e
to
b0a6ea7
Compare
b0a6ea7
to
480f2d9
Compare
480f2d9
to
1d6ce99
Compare
1d6ce99
to
4df669b
Compare
- Fix Haiku toolchain detection code. - Register Haiku to XUnit test platforms.
4df669b
to
6d1d15d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry this fell off my radar. LGTM!
cc @ViktorHofer
@@ -207,6 +207,7 @@ elseif(ILLUMOS) | |||
set(CMAKE_CXX_STANDARD_LIBRARIES "${CMAKE_CXX_STANDARD_LIBRARIES} -lssp") | |||
elseif(HAIKU) | |||
set(CMAKE_SYSROOT "${CROSS_ROOTFS}") | |||
set(CMAKE_PROGRAM_PATH "${CMAKE_PROGRAM_PATH};${CROSS_ROOTFS}/cross-tools-x86_64/bin") | |||
|
|||
set(TOOLSET_PREFIX ${TOOLCHAIN}-) | |||
function(locate_toolchain_exec exec var) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could hoist this function out of this if-else block and reuse it for both Haiku and illumos. (perhaps as a separate PR)
To double check: