Skip to content

Comments

krb5: add a fix for broken configure test#58191

Merged
claui merged 1 commit intoHomebrew:masterfrom
mistydemeo:fix_krb5
Jul 19, 2020
Merged

krb5: add a fix for broken configure test#58191
claui merged 1 commit intoHomebrew:masterfrom
mistydemeo:fix_krb5

Conversation

@mistydemeo
Copy link
Contributor

@mistydemeo mistydemeo commented Jul 18, 2020

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

This fixes krb5 on macOS 11, both ARM and x86_64. It turns out that clang's new pickiness about calling functions without their header definitions broke this test. The test is actually testing destructor attributes, and for convenience is using the C unlink function. That is defined in unistd.h, but the configure test doesn't #include this header.

I'll be submitting an upstream fix as well, but this inreplace fix gets us going.

@mistydemeo mistydemeo added the 11 Big Sur is specifically affected label Jul 18, 2020
@claui
Copy link
Contributor

claui commented Jul 19, 2020

The CI failures only affect stellar-core, and are timeouts due to a long-running self test.

Both krb5 and stellar-core are security-sensitive formulae, and certainly deserve more attention than most other formulae in that respect. However, the fact that the 10.14 CI run passed is strong evidence that the timeout is not related to krb5. I think we can safely merge this.

@claui claui merged commit 694e769 into Homebrew:master Jul 19, 2020
@claui
Copy link
Contributor

claui commented Jul 19, 2020

Thanks @mistydemeo! 🍻

@mistydemeo mistydemeo deleted the fix_krb5 branch July 19, 2020 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

11 Big Sur is specifically affected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants