Skip to content

Initial FreeBSD support #35

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

Closed
wants to merge 1 commit into from
Closed

Initial FreeBSD support #35

wants to merge 1 commit into from

Conversation

vkrasnov
Copy link
Contributor

Implement initial FreeBSD support.

Fixes #29

@vkrasnov
Copy link
Contributor Author

I deployed a FreeBSD instance on azure with pipelines workers, but Docker is not supported on FreeBSD the same way as on Linux, therefore I need to make the tests work with jails instead.

Copy link
Contributor

@ashleygwilliams ashleygwilliams left a comment

Choose a reason for hiding this comment

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

a few nits/questions

@@ -8,16 +8,16 @@ pub mod drop_privileges;
mod integration_tests;
pub mod peer;

#[cfg(any(target_os = "macos", target_os = "ios"))]
#[cfg(any(target_os = "macos", target_os = "freebsd"))]
Copy link
Contributor

Choose a reason for hiding this comment

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

was that target_os = "ios" not load bearing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, I'm afraid ios is never gonna support utun :(

#[cfg(any(target_os = "macos", target_os = "ios"))]
#[path = "tun_darwin.rs"]
#[cfg(any(target_os = "macos", target_os = "freebsd"))]
#[path = "tun_bsd.rs"]
Copy link
Contributor

Choose a reason for hiding this comment

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

we may want this file name to indicate that it's both darwin and bsd

Copy link
Contributor Author

Choose a reason for hiding this comment

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

darwin is considered bsd

@decke decke mentioned this pull request Mar 28, 2019
@valpackett
Copy link

Hi, any updates on this? Looks like there's merge conflicts now..

@vkrasnov
Copy link
Contributor Author

Hi, any updates on this? Looks like there's merge conflicts now..

I plan to have official FreeBSD support by the end of this year

kevincox pushed a commit to Sovereign-Engineering/boringtun that referenced this pull request Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Freebsd
4 participants