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

Feature request: node_uname_info on *BSD #1222

Closed
sachaz opened this issue Jan 4, 2019 · 9 comments · Fixed by #1433
Closed

Feature request: node_uname_info on *BSD #1222

sachaz opened this issue Jan 4, 2019 · 9 comments · Fixed by #1433

Comments

@sachaz
Copy link

sachaz commented Jan 4, 2019

Related to #1221 the node_uname_info is not enabled on *BSD

@SuperQ
Copy link
Member

SuperQ commented Jan 4, 2019

We should be able to do this on *BSD with the uname(2) syscall.

@sachaz
Copy link
Author

sachaz commented Jan 5, 2019

is https://golang.org/pkg/runtime/#GOOS is not more reliable ?

@matthiasr
Copy link
Contributor

matthiasr commented Jan 7, 2019 via email

@sachaz
Copy link
Author

sachaz commented Jan 17, 2019

thanks for considering this request, uname is usefull on all platforms !

@sachaz
Copy link
Author

sachaz commented Feb 13, 2019

up !

@discordianfish
Copy link
Member

@sachaz This is accepted, now we need somebody to add support for that. PRs are welcome :)

@philipgough
Copy link
Contributor

This seems to have been added for FreeBSD already. Is it reasonable to add this for Darwin? If yes I can take that.

@SuperQ
Copy link
Member

SuperQ commented Jul 18, 2019

@philipgough Go for it, PR would be welcome.

@philipgough
Copy link
Contributor

philipgough commented Jul 18, 2019

@SuperQ great -will take a stab at it. What needs to be done to close this issue if FreeBSD is covered? I guess adding and testing on both openbsd and netbsd

Might as well do both.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants