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

tsduck 3.40-4165 #208703

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

tsduck 3.40-4165 #208703

wants to merge 1 commit into from

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

@github-actions github-actions bot added java Java use is a significant feature of the PR or issue bump-formula-pr PR was created using `brew bump-formula-pr` labels Feb 23, 2025
@lelegard
Copy link
Contributor

lelegard commented Feb 23, 2025

Hi Homebrew maintainers,

My guess is that the macOS running the build has an old version of clang.

Starting with this new version, TSDuck requires a C++20 compliant compiler. Even though gcc and clang are supposed to support C++20 for some time, they had several bugs, refusing to compile valid programs. The error in this run is one of those, as I experimented in the past.

On macOS, we now require clang 16.0.0 as minimum version.

How would you express that requirement in a Homebrew formula?

In the CI workflow of the TSDuck project, we enforce clang 16 as a prerequisite. In practice, we test the version of clang and, if it is 15 or lower, we enforce an upgrade of the "Xcode command line tools", which brings clang 16 or higher.

See the latest (successful) build here: https://github.com/tsduck/tsduck/actions/runs/13477350475

EDIT: also started a discussion here since more people watch discussions than failing PR's.

@chenrui333 chenrui333 added the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label Feb 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. java Java use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants