Skip to content
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

Initial build configuration for Haiku #86303

Merged
merged 3 commits into from
May 17, 2023

Conversation

trungnt2910
Copy link
Contributor

@trungnt2910 trungnt2910 commented May 16, 2023

Add support for Haiku in build configuration files in eng/native/**

Part of #55803.

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label May 16, 2023
@ghost
Copy link

ghost commented May 16, 2023

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

Add support for Haiku build configuration files in eng/native/**

Part of #55803.

Author: trungnt2910
Assignees: -
Labels:

area-Infrastructure-libraries, community-contribution

Milestone: -

@ghost
Copy link

ghost commented May 16, 2023

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

Add support for Haiku in build configuration files in eng/native/**

Part of #55803.

Author: trungnt2910
Assignees: -
Labels:

area-Infrastructure, community-contribution

Milestone: -

eng/build.sh Outdated Show resolved Hide resolved
@@ -214,6 +214,12 @@ if(CLR_CMAKE_HOST_OS STREQUAL sunos)
endif(SUNOS_KERNEL_KIND STREQUAL illumos OR CMAKE_CROSSCOMPILING)
endif(CLR_CMAKE_HOST_OS STREQUAL sunos)

if(CLR_CMAKE_HOST_OS STREQUAL haiku)
set(CLR_CMAKE_HOST_UNIX 1)
set(CLR_CMAKE_HOST_UNIX_AMD64 1)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does Haiku support building on x64 host only?

Copy link
Contributor Author

@trungnt2910 trungnt2910 May 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the purpose of this PR and the ones by me in the near future, yes.

eng/native/tryrun.cmake Outdated Show resolved Hide resolved
eng/native/tryrun.cmake Outdated Show resolved Hide resolved
Initially recognize Haiku as a supported platform in
Directory.Build.props and all scripts in eng/**.
@trungnt2910 trungnt2910 force-pushed the dev/trungnt2910/haiku-eng branch 3 times, most recently from 8b2c462 to d5b5866 Compare May 17, 2023 01:00
trungnt2910 and others added 2 commits May 17, 2023 11:02
Drop some unused configure values from tryrun.cmake,
configure.cmake and config.h.in.
Simplified an error message for the `--os` option to avoid maintaining a duplicate list of OSes.

Co-authored-by: Adeel Mujahid <[email protected]>
Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jkotas jkotas merged commit 445dac9 into dotnet:main May 17, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Jun 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Infrastructure community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants