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 support for NetBSD #1364

Closed
wants to merge 3 commits into from
Closed

Conversation

khorben
Copy link

@khorben khorben commented Oct 14, 2022

These two commits will hopefully help bring support for NetBSD to the project. I have not been able to verify if these changes are correct yet ("go test" did not yield anything apparently useful). Please review before merging, and do so only if you feel like it is not expected to break anything new.

For these changes, I have simply copied the OpenBSD files as NetBSD, as the two systems are usually very close to one another. As such I think at least some things should work once the module is confirmed to build.

In the current state, this is just a copy of the OpenBSD port as NetBSD.
I have been unable to run tests or confirm anything to be working so
far.
@shirou
Copy link
Owner

shirou commented Oct 14, 2022

Thank you for your contribution. I have not checked all of the files, but if these netbsd files are just copied from openbsd, how about simply add netbsd to the build tag in openbsd files, like this?

//go:build openbsd netbsd

@khorben
Copy link
Author

khorben commented Oct 14, 2022

It's a good idea, it might be a bit misleading though.

Let me try to actually test and verify if the changes work over the week-end.

@khorben
Copy link
Author

khorben commented Oct 16, 2022

For the cpu part, some changes were required for the code to work on NetBSD.
I will continue with the other parts of the code ASAP.

darkgeek added a commit to darkgeek/gopsutil that referenced this pull request Aug 27, 2023
Currently the changes are all copied from shirou#1364
@Lomanic
Copy link
Collaborator

Lomanic commented May 10, 2024

Closing, superseded by #1530 which is partially based off this PR, thanks @khorben.

@Lomanic Lomanic closed this May 10, 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.

None yet

3 participants