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

zstd 1.5.0 #77434

Closed
wants to merge 2 commits into from
Closed

zstd 1.5.0 #77434

wants to merge 2 commits into from

Conversation

miccal
Copy link
Member

@miccal miccal commented May 17, 2021

Created with brew bump-formula-pr.

@BrewTestBot BrewTestBot added the bump-formula-pr PR was created using `brew bump-formula-pr` label May 17, 2021
@carlocab
Copy link
Member

carlocab commented May 17, 2021

The clazy linkage failure is

==> brew linkage --test clazy
==> FAILED
Broken dependencies:
  /opt/homebrew/opt/llvm@11/lib/libc++.1.dylib (llvm@11)

Won't be fixed by a revision bump. I suggest you drop it from this PR and we can fix it in a separate PR.

Edit: That was a lie.

@BrewTestBot
Copy link
Member

:shipit: @carlocab has triggered a merge.

@carlocab
Copy link
Member

Hmm, no, revision bump didn't help. From the newly built bottle:

❯ brew linkage clazy
System libraries:
  /usr/lib/libSystem.B.dylib
  /usr/lib/libncurses.5.4.dylib
  /usr/lib/libxml2.2.dylib
  /usr/lib/libz.1.dylib
Variable-referenced libraries:
  @rpath/ClazyPlugin.dylib
Broken dependencies:
  /usr/local/opt/llvm@11/lib/libc++.1.dylib (llvm@11)
Dependencies with no linkage:
  qt

Not sure why that's passing in CI.

@miccal miccal deleted the bump-zstd-1.5.0 branch May 18, 2021 01:16
@github-actions github-actions bot added the outdated PR was locked due to age label Jun 18, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants