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

formula_cellar_checks: check_non_libraries only for new formulae #2997

Merged
merged 1 commit into from
Aug 3, 2017

Conversation

ilovezfs
Copy link
Contributor

@ilovezfs ilovezfs commented Aug 3, 2017

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew tests with your changes locally?

This check is overly opinionated and leads to audit failures no one cares to fix, so let's only run it for new formulae. For example, tcl-tk has failed this audit for years:

$ brew audit tcl-tk
tcl-tk:
  * Non-libraries were installed to "/usr/local/opt/tcl-tk/lib"
    Installing non-libraries to "lib" is discouraged.
    The offending files are:
      /usr/local/opt/tcl-tk/lib/Tk.icns
      /usr/local/opt/tcl-tk/lib/Tk.tiff
Error: 1 problem in 1 formula

@MikeMcQuaid
Copy link
Member

Could we make this a --new-formula check?

@MikeMcQuaid
Copy link
Member

(NOOOOO is a reasonable response 😆)

This check is overly opinionated and leads to audit failures no one
cares to fix, so let's only run it for new formulae.
@ilovezfs ilovezfs changed the title formula_cellar_checks: remove check_non_libraries formula_cellar_checks: check_non_libraries only for new formulae Aug 3, 2017
@ilovezfs
Copy link
Contributor Author

ilovezfs commented Aug 3, 2017

@MikeMcQuaid yes, it's always good to have extra reasons to reject new formulae.

@ilovezfs ilovezfs merged commit 1cae6dd into Homebrew:master Aug 3, 2017
@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants