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 support for OpenBSD #747

Merged
merged 1 commit into from
Oct 4, 2023
Merged

Conversation

dl6tom
Copy link

@dl6tom dl6tom commented Oct 1, 2023

since OpenBSD has no API to get the path that was used to create the current process, the get_exe_dir() implementation makes use of argv[0] and the PATH environment variable.

@dl6tom dl6tom force-pushed the master branch 2 times, most recently from 815c9d8 to a8707b9 Compare October 1, 2023 19:29
@carrotIndustries
Copy link
Member

Is there any CI service that supports OpenBSD so we can make sure that this code at least compiles?

@dl6tom
Copy link
Author

dl6tom commented Oct 1, 2023

i have it running here, so can assure you it build and runs. it seems sourcehut supports OpenBSD in their CI (https://man.sr.ht/builds.sr.ht/compatibility.md). i can take a look what it takes to set this up.

since OpenBSD has no API to get the path that was used to create the
current process, the get_exe_dir() implementation makes use of argv[0]
and the PATH environment variable.
@dl6tom
Copy link
Author

dl6tom commented Oct 2, 2023

sourcehut has no free tier, so i setup a jenkins vm (is a bit underpowered though). it gets as far as it can w/o this commit: https://jenkins.0x90.space/job/horizon-eda/7/console
still have to figure out how this would integrate with github.

@carrotIndustries
Copy link
Member

I could set up a webhook for that purpose.

@dl6tom
Copy link
Author

dl6tom commented Oct 2, 2023

i think that would be https://jenkins.0x90.space/github-webhook/ then. not sure how to report the build status back to github.

@carrotIndustries carrotIndustries merged commit fff7134 into horizon-eda:master Oct 4, 2023
20 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants