Skip to content

Initial support for NetBSD#1364

Closed
khorben wants to merge 3 commits intoshirou:masterfrom
khorben:khorben/netbsd
Closed

Initial support for NetBSD#1364
khorben wants to merge 3 commits intoshirou:masterfrom
khorben:khorben/netbsd

Conversation

@khorben
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

3 participants