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

audit complains about linking to system frameworks that don't exist #9015

Closed
fxcoudert opened this issue Oct 30, 2020 · 2 comments · Fixed by #9031
Closed

audit complains about linking to system frameworks that don't exist #9015

fxcoudert opened this issue Oct 30, 2020 · 2 comments · Fixed by #9031
Labels
outdated PR was locked due to age

Comments

@fxcoudert
Copy link
Member

Testing of the salt update PR gives a very weird audit error:

==> brew audit salt --online --git --skip-style
==> FAILED
Error: 1 problem in 1 formula detected
Missing libraries:
  unexpected (/System/Library/Frameworks/AuthenticationServices.framework/Versions/A/AuthenticationServices)
  unexpected (/System/Library/Frameworks/OSLog.framework/Versions/A/OSLog)
  unexpected (/System/Library/Frameworks/PushKit.framework/Versions/A/PushKit)
  unexpected (/System/Library/Frameworks/Speech.framework/Versions/A/Speech)
  unexpected (/System/Library/Frameworks/SystemExtensions.framework/Versions/A/SystemExtensions)
Unexpected missing library linkage detected
salt:
  * salt has broken dynamic library links:

I suspect something is wrong with the audit, because it's perfectly legitimate for a formula to link against system frameworks. If there really is something wrong here, the error message is not clear enough that I understand what the problem is.

@MikeMcQuaid
Copy link
Member

I suspect something is wrong with the audit, because it's perfectly legitimate for a formula to link against system frameworks.

Are you sure those system frameworks exist?

@MikeMcQuaid MikeMcQuaid changed the title audit complains about linking to system frameworks audit complains about linking to system frameworks that don't exist Nov 2, 2020
@MikeMcQuaid
Copy link
Member

Looking at the code: they do not.

@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Dec 3, 2020
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Dec 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants