Skip to content

Conversation

@botantony
Copy link
Member

@botantony botantony commented Nov 6, 2025

Formulae with problems (tested in #252206):

        Illegal instruction (core dumped)
      /var/tmp/dotnet-20251102-464859-hcev1i/dotnet-9.0.109/src/runtime/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Crossgen2.sfxproj(53,5): error MSB3073: The command "/var/tmp/dotnet-20251102-464859-hcev1i/dotnet-9.0.109/src/runtime/artifacts/bin/crossgen2_publish/x64/Release/linux-x64/publish/crossgen2 /var/tmp/dotnet-20251102-464859-hcev1i/dotnet-9.0.109/src/runtime/artifacts/bin/coreclr/linux.x64.Release/IL/System.Private.CoreLib.dll --out /var/tmp/dotnet-20251102-464859-hcev1i/dotnet-9.0.109/src/runtime/artifacts/obj/Microsoft.NETCore.App.Crossgen2/Release/net9.0/ubuntu.22.04-x64/S.P.C.tmp" exited with code 132.

In progress:
TODO PRs:
  • statesmith (depends on dotnet)
Merged PRs:

@botantony botantony added in progress Stale bot should stay away CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. no long build conflict Do not allow merging other pull requests when files conflict with this one labels Nov 6, 2025
@github-actions github-actions bot added automerge-skip `brew pr-automerge` will skip this pull request new formula PR adds a new formula to Homebrew/homebrew-core alias labels Nov 6, 2025
@github-actions

This comment was marked as resolved.

@github-actions github-actions bot added CI-linux-self-hosted-deps Test dependents on Linux self-hosted runner long dependent tests Set a long timeout for dependent testing CI-skip-recursive-dependents Pass --skip-recursive-dependents to brew test-bot. long build Set a long timeout for formula testing labels Nov 6, 2025
@daeho-ro daeho-ro mentioned this pull request Nov 7, 2025
postgis: revision bump (icu4c@78 78.1)
@chenrui333
Copy link
Member

looks like more PRs got blocked by this PR, especially the nodejs ones.

@botantony
Copy link
Member Author

Hope I could finish it soon but unfortunately I do not see any progress. If nothing promising comes out today, I'm closing this PR until someone else bothers to fix x64 linux build for dotnet

@daeho-ro
Copy link
Member

Can we update icu4c@77 syntax only and then keep qt@5 and dotnet bottles? @Homebrew/core

@carlocab
Copy link
Member

If nothing promising comes out today, I'm closing this PR until someone else bothers to fix x64 linux build for dotnet

That's not a good idea. See https://docs.brew.sh/Homebrew-homebrew-core-Maintainer-Guide#summary.

Before making use of a staging branch, there is one important disadvantage to consider: once you have merged bottle updates to the staging branch, it is very difficult to take them back. This typically involves deleting uploaded bottles, which will occasionally require an owner of the Homebrew GitHub organisation to delete uploaded bottles one at a time.

You will probably need to merge this without dotnet x64 Linux bottles.

@carlocab
Copy link
Member

Can we update icu4c@77 syntax only and then keep qt@5 and dotnet bottles? @Homebrew/core

The dependents will also need to be on icu4c@77. Could cause problems with versioned dependency conflicts. Assuming that isn't an issue, it might work. But I don't think you can get around the versioned dependency conflict issue.

@cho-m
Copy link
Member

cho-m commented Nov 12, 2025

First thing I would consider is checking if the formulae with version conflicts are functionally impacted by the conflict. If low risk, then would just use versioned_dependencies_conflicts_allowlist.json to get PR over the line.

ICU uses versioned symbols and the risk would be even lower if the ICU usage is in shelled off commands rather than loaded together at once.

@cho-m
Copy link
Member

cho-m commented Nov 12, 2025

To give an example, undercutf1 has a mixed ICU usage due to its ffmpeg dependency which indirectly needs ICU.

The way it uses FFmpeg is via https://github.com/rosenbjerg/FFMpegCore which is a wrapper around the executable not the library

This means the ICU loaded by .NET shouldn't interact with the one loaded by FFmpeg.

@botantony
Copy link
Member Author

cdxgen uses dotnet runtime for dosai plugin so it should not create conflicts with node

@botantony botantony marked this pull request as ready for review November 13, 2025 00:40
@github-actions

This comment was marked as resolved.

@cho-m cho-m removed the automerge-skip `brew pr-automerge` will skip this pull request label Nov 13, 2025
@github-actions

This comment was marked as resolved.

@botantony botantony added this pull request to the merge queue Nov 13, 2025
Merged via the queue into main with commit 6b1ab8c Nov 13, 2025
30 of 32 checks passed
@botantony botantony deleted the icu4c@78-staging branch November 13, 2025 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alias CI-linux-self-hosted-deps Test dependents on Linux self-hosted runner CI-skip-recursive-dependents Pass --skip-recursive-dependents to brew test-bot. CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. in progress Stale bot should stay away long build Set a long timeout for formula testing long dependent tests Set a long timeout for dependent testing new formula PR adds a new formula to Homebrew/homebrew-core no long build conflict Do not allow merging other pull requests when files conflict with this one

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants