Skip to content

Conversation

@BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

@github-actions github-actions bot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Jun 13, 2025
@alebcay
Copy link
Member

alebcay commented Jun 13, 2025

Looks like a template arg list is required now after the template keyword, probably due to llvm/llvm-project#80801. Probably need some changes similar to llvm/llvm-project#94204 in the area where the error occurred.

@alebcay
Copy link
Member

alebcay commented Jun 14, 2025

The error occurs in the vendored copy of iguana in include/ylt/standalone/iguana - this copy is built against, but not installed into the final prefix.

The error is already fixed in the most recent release of iguana via qicosmos/iguana@ef81508. The newest release of iguana also seems to ship a copy of vendored ylt (in the ylt subdirectory).

It's unclear to me how best to resolve this issue. Which project depends on which? Both seem to be vendoring a copy of the other.

@github-actions
Copy link
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. To keep this pull request open, add a help wanted or in progress label.

@github-actions github-actions bot added the stale No recent activity label Jun 16, 2025
@chenrui333 chenrui333 added the build failure CI fails while building the software label Jun 16, 2025
@github-actions github-actions bot removed stale No recent activity labels Jun 17, 2025
@chenrui333 chenrui333 added the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label Jun 18, 2025
@github-actions
Copy link
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. To keep this pull request open, add a help wanted or in progress label.

yalantinglibs: add clang build patch

Signed-off-by: Rui Chen <[email protected]>
@chenrui333 chenrui333 force-pushed the bump-yalantinglibs-1.2.0 branch from 2aeb50b to e0e216a Compare June 21, 2025 12:47
@github-actions github-actions bot removed the stale No recent activity label Jun 21, 2025
@chenrui333
Copy link
Member

The error occurs in the vendored copy of iguana in include/ylt/standalone/iguana - this copy is built against, but not installed into the final prefix.

The error is already fixed in the most recent release of iguana via qicosmos/iguana@ef81508. The newest release of iguana also seems to ship a copy of vendored ylt (in the ylt subdirectory).

It's unclear to me how best to resolve this issue. Which project depends on which? Both seem to be vendoring a copy of the other.

just dropped a note about the patch in the upstream pr, alibaba/yalantinglibs#947

@chenrui333 chenrui333 added ready to merge PR can be merged once CI is green and removed build failure CI fails while building the software labels Jun 21, 2025
@github-actions
Copy link
Contributor

🤖 An automated task has requested bottles to be published to this PR.

Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch.

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Jun 21, 2025
@BrewTestBot BrewTestBot enabled auto-merge June 21, 2025 14:52
@BrewTestBot BrewTestBot added this pull request to the merge queue Jun 21, 2025
Merged via the queue into master with commit 5822b1a Jun 21, 2025
17 checks passed
@BrewTestBot BrewTestBot deleted the bump-yalantinglibs-1.2.0 branch June 21, 2025 15:00
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. CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. ready to merge PR can be merged once CI is green

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants