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

Add support for OpenBSD #455

Merged
merged 1 commit into from
May 10, 2022
Merged

Add support for OpenBSD #455

merged 1 commit into from
May 10, 2022

Conversation

qbit
Copy link
Contributor

@qbit qbit commented Feb 11, 2022

This adds PhysicalMemoryBytes functions and tests for OpenBSD.

@CLAassistant
Copy link

CLAassistant commented Feb 11, 2022

CLA assistant check
All committers have signed the CLA.

iamemilio
iamemilio previously approved these changes Feb 15, 2022
Copy link
Contributor

@iamemilio iamemilio left a comment

Choose a reason for hiding this comment

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

This looks good to me, its nearly identical to what we support for freebsd, @RichVanderwal has verified that this works with OpenBSD in vagrant, go vet did not find any risky pointer conversions, and the tests look good. Thank you for the PR!

@RichVanderwal RichVanderwal changed the base branch from master to develop February 15, 2022 21:05
@RichVanderwal RichVanderwal dismissed iamemilio’s stale review February 15, 2022 21:05

The base branch was changed.

@qbit
Copy link
Contributor Author

qbit commented Mar 6, 2022

Do I need to do anything else for this?

@iamemilio iamemilio self-requested a review May 10, 2022 17:25
@iamemilio iamemilio removed the request for review from RichVanderwal May 10, 2022 17:27
@iamemilio iamemilio merged commit aee135d into newrelic:develop May 10, 2022
qbit added a commit to qbit/certificates that referenced this pull request Jul 28, 2022
OpenBSD support was added to the following deps:
 - github.com/go-piv/piv-go in go-piv/piv-go#101
 - github.com/newrelic/go-agent in newrelic/go-agent#455
 - github.com/miekg/pkcs11 in miekg/pkcs11#140

With these deps bumped, tests all pass on OpenBSD amd64.
qbit added a commit to qbit/certificates that referenced this pull request Aug 4, 2022
OpenBSD support was added to the following deps:
 - github.com/go-piv/piv-go in go-piv/piv-go#101
 - github.com/newrelic/go-agent in newrelic/go-agent#455
 - github.com/miekg/pkcs11 in miekg/pkcs11#140

With these deps bumped, tests all pass on OpenBSD amd64.
qbit added a commit to qbit/certificates that referenced this pull request Aug 4, 2022
OpenBSD support was added to the following deps:
 - github.com/go-piv/piv-go in go-piv/piv-go#101
 - github.com/newrelic/go-agent in newrelic/go-agent#455
 - github.com/miekg/pkcs11 in miekg/pkcs11#140

With these deps bumped, tests all pass on OpenBSD amd64.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants