Skip to content

dotnet: switch to openssl@3#134587

Closed
mchoo7 wants to merge 3 commits into
Homebrew:masterfrom
mchoo7:dotnet-openssl3
Closed

dotnet: switch to openssl@3#134587
mchoo7 wants to merge 3 commits into
Homebrew:masterfrom
mchoo7:dotnet-openssl3

Conversation

@mchoo7

@mchoo7 mchoo7 commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

See #134251.

@github-actions github-actions Bot added long build Set a long timeout for formula testing icu4c ICU use is a significant feature of the PR or issue labels Jun 22, 2023
@mchoo7

mchoo7 commented Jun 22, 2023

Copy link
Copy Markdown
Contributor Author

CI-long-timeout needed.

@chenrui333 chenrui333 added openssl-3-migration Related to switching to an OpenSSL 3 dependency staging-branch-pr This pull request targets a *-staging branch. labels Jun 22, 2023
@p-linnane p-linnane added CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. labels Jun 22, 2023
@github-actions github-actions Bot added autosquash Automatically squash pull request commits according to Homebrew style. and removed CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. labels Jun 23, 2023
@mchoo7

mchoo7 commented Jun 23, 2023

Copy link
Copy Markdown
Contributor Author

Related:
#95339
dotnet/source-build#2795

Maybe this PR solved this issue: dotnet/runtime#84591

Comment thread Formula/dotnet.rb Outdated
Comment on lines 5 to 8

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Source-build is only supported on 7.0.1xx.

This may change for .NET 8 but for .NET 7 it's documented here.

@Bo98 Bo98 Jun 23, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We could/should update this to v7.0.108 however, though I'm guessing there's already been attempts to do that.

@mchoo7 mchoo7 force-pushed the dotnet-openssl3 branch 2 times, most recently from bc833ad to 90f1f8a Compare June 23, 2023 23:41
@mchoo7 mchoo7 force-pushed the dotnet-openssl3 branch from 90f1f8a to 66cbb7d Compare June 24, 2023 04:32
@github-actions github-actions Bot added the automerge-skip `brew pr-automerge` will skip this pull request label Jun 26, 2023
@p-linnane p-linnane changed the base branch from openssl-migration-staging to master June 26, 2023 23:57
@p-linnane p-linnane removed the staging-branch-pr This pull request targets a *-staging branch. label Jun 26, 2023
@pthariensflame

Copy link
Copy Markdown
Contributor

This may need a bump to 7.0.109?

@mchoo7 mchoo7 force-pushed the dotnet-openssl3 branch from 154fe71 to 9ff5f21 Compare July 12, 2023 00:59
@github-actions github-actions Bot removed the automerge-skip `brew pr-automerge` will skip this pull request label Jul 12, 2023
@github-actions

github-actions Bot commented Aug 2, 2023

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions Bot added the stale No recent activity label Aug 2, 2023
@github-actions github-actions Bot closed this Aug 10, 2023
@mchoo7 mchoo7 deleted the dotnet-openssl3 branch December 25, 2023 01:37
@github-actions github-actions Bot added the outdated PR was locked due to age label Jan 25, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jan 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosquash Automatically squash pull request commits according to Homebrew style. CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. icu4c ICU use is a significant feature of the PR or issue long build Set a long timeout for formula testing openssl-3-migration Related to switching to an OpenSSL 3 dependency outdated PR was locked due to age stale No recent activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants