Skip to content

Improve NocgoOpenSSL.md#2198

Merged
qmuntal merged 7 commits intomicrosoft/mainfrom
dev/qmuntal/clarnoocgo
Mar 20, 2026
Merged

Improve NocgoOpenSSL.md#2198
qmuntal merged 7 commits intomicrosoft/mainfrom
dev/qmuntal/clarnoocgo

Conversation

@qmuntal
Copy link
Member

@qmuntal qmuntal commented Mar 19, 2026

Clarify how the nocgo mode is selected and add a runtime dependencies section.

@qmuntal qmuntal requested a review from a team as a code owner March 19, 2026 15:03
Copilot AI review requested due to automatic review settings March 19, 2026 15:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Clarifies selection behavior for the nocgo OpenSSL backend and documents runtime library dependencies when using the cgo-less implementation.

Changes:

  • Rewrites the backend-selection section into explicit decision bullets (CGO_ENABLED=1/0/default behavior).
  • Adds a “Runtime dependencies” section describing required glibc libs and notes about glibc 2.34+.

You can also share your feedback on Copilot code review. Take the survey.

qmuntal and others added 2 commits March 19, 2026 16:14
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
qmuntal and others added 3 commits March 20, 2026 09:10
Co-authored-by: Davis Goodin <dagood@users.noreply.github.com>
Co-authored-by: Davis Goodin <dagood@users.noreply.github.com>
@qmuntal qmuntal enabled auto-merge March 20, 2026 08:43
@dagood
Copy link
Member

dagood commented Mar 20, 2026

/azp run microsoft-go

(IIRC this will get it to use a newer merge commit to skip #2200...)

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@dagood
Copy link
Member

dagood commented Mar 20, 2026

(IIRC this will get it to use a newer merge commit to skip #2200...)

Didn't work, but some more of the macOS failures passed. Maybe it's over? Hitting retry again.

Closing and re-opening would probably have worked, but that would redo the GitHub Actions workflows too I believe.

@qmuntal qmuntal merged commit 3931bcf into microsoft/main Mar 20, 2026
42 checks passed
@qmuntal qmuntal deleted the dev/qmuntal/clarnoocgo branch March 20, 2026 23:59
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