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

add initial support for OpenBSD #53633

Merged
merged 9 commits into from
Mar 29, 2024

Conversation

semarie
Copy link
Contributor

@semarie semarie commented Mar 7, 2024

These commits add initial support of OpenBSD in julia.

It isn't strictly enough to make julia runable on OpenBSD (see #53632), but it covers the larger part.

semarie added 6 commits March 7, 2024 07:55
- libsuitesparse: use $(MAKE) instead of hardcoding "make" (it is "gmake" on BSD)
- patchelf: pass MAKE variable
- libssh2: pass -fPIC if need (permit libgit2 to link on OpenBSD)
- libgit2: search iconv.h in right directory
- pcre2: avoid RWX memory by disabling jit on OpenBSD
- libssh2
- libsuitesparse
- mbedtls
@semarie semarie mentioned this pull request Mar 7, 2024
10 tasks
src/gc-stacks.c Show resolved Hide resolved
src/sys.c Show resolved Hide resolved
Copy link
Member

@Keno Keno left a comment

Choose a reason for hiding this comment

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

Seems like a reasonable start overall

base/binaryplatforms.jl Outdated Show resolved Hide resolved
@semarie
Copy link
Contributor Author

semarie commented Mar 25, 2024

there is something I should do to make progress here ?

@oscardssmith oscardssmith added the building Build system, or building Julia or its dependencies label Mar 25, 2024
@oscardssmith
Copy link
Member

No. I think this looks ready to merge.

@oscardssmith oscardssmith added the merge me PR is reviewed. Merge when all tests are passing label Mar 25, 2024
Copy link
Member

@fingolfin fingolfin left a comment

Choose a reason for hiding this comment

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

Thank you for working on this!

@StefanKarpinski
Copy link
Member

Do we have any prospect for CI for OpenBSD?

@semarie
Copy link
Contributor Author

semarie commented Mar 25, 2024

@IanButterworth
Copy link
Member

Do you want any of the commit messages to be retained in the squash merge description here? Github's default squash merge message is just the original PR post, so if you do perhaps elaborate there?

@ararslan
Copy link
Member

ararslan commented Mar 29, 2024

Do we have any prospect for CI for OpenBSD?

Buildkite provides official OpenBSD binaries for the agent with every release. Like Windows, our FreeBSD CI is set up to run under KVM on Linux, so we could theoretically make an analogous setup for OpenBSD.

@semarie
Copy link
Contributor Author

semarie commented Mar 29, 2024

@IanButterworth I am fine with default squash merge. I did several differents commits to help reviewing (and not mix too many things)

@IanButterworth IanButterworth merged commit e26d140 into JuliaLang:master Mar 29, 2024
8 checks passed
@semarie semarie deleted the openbsd-initial branch March 29, 2024 16:52
@IanButterworth IanButterworth removed the merge me PR is reviewed. Merge when all tests are passing label Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants