-
-
Notifications
You must be signed in to change notification settings - Fork 21.3k
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
Make sysctl calls on FreeBSD #90295
Make sysctl calls on FreeBSD #90295
Conversation
You didn't update your branch, it is still about 18 months out of date, please update your branch with |
There are still style issues that would be fixed by clang-format (or checking surrounding code and using the same style). And the commits are still not linked to your GH account. |
@akien-mga hi, when I look at the commit history it says I authored and committed the changes, so I'm not sure how its not linked to my account, or what I would need to change. |
See: Why are my commits linked to the wrong user? for more info. |
Seems there was a capital letter in my github email that makes it different to the commit info. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't test, but assuming it works, the changes seem fine.
Could you squash the commits? See PR workflow for instructions.
While doing so, you would also get a change to update your Git email to make it match your GitHub account's (with git commit --amend --reset-author
after changing your local or global Git email).
You added unexpected changes, you probably rebased on your old local You can go back with
And for good measure, you could also do:
to stop going back to 18 months old code :) |
6839dc3
to
e0b76d7
Compare
I think it looks right this time |
Yes, looks great! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Could you squash the commits?
The OS module get_unique_id and get_processor_name rely on linux files which don't exist on a standard FreeBSD install, make sysctl calls to get the required data.
@akien-mga should be good to go now. Note that I also added the |
Thanks! And congrats for your first merged Godot contribution 🎉 |
Cherry-picked for 4.2.2. |
The OS module get_unique_id and get_processor_name rely on linux files which don't exist on a standard FreeBSD install, make sysctl calls to get the required data.
I expect OpenBSD and NetBSD would require similar changes.