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

gnutls-guile 3.6.14 (new formula) #59481

Closed
wants to merge 1 commit into from

Conversation

aconchillo
Copy link
Contributor

  • 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 formula builds gnutls library again in order to build the bindings, however it only installs the bindings. By only installing the bindings we have the following issue, however the compiled library is actually not needed since the gnutls one will be used. I'm not sure how to do this well without creating this new formula, unless we enable Guile in the gnutls formula, but I feel it's better to have a gnutls-guile one.

❯ brew audit --strict gnutls-guile
Broken dependencies:
  /usr/local/Cellar/gnutls-guile/3.6.14/lib/libgnutls.30.dylib (gnutls-guile)
No broken library linkage detected
gnutls-guile:
  * Formula gnutls-guile contains deprecated SPDX licenses: ["LGPL-2.1"].
  * gnutls-guile has broken dynamic library links:

Error: 2 problems in 1 formula detected

@BrewTestBot BrewTestBot added deprecated license Formula uses a deprecated SPDX license which should be updated new formula PR adds a new formula to Homebrew/homebrew-core labels Aug 11, 2020
@aconchillo
Copy link
Contributor Author

The other issue about doing it this way is that it would probably be necessary to update gnutls-guile when updating gnutls (e.g. when updating to a newer version).

@aconchillo aconchillo mentioned this pull request Aug 12, 2020
5 tasks
@aconchillo
Copy link
Contributor Author

Closing in favor of #59490 since there's no way to fix the broken dynamic library link issue.

@aconchillo aconchillo closed this Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deprecated license Formula uses a deprecated SPDX license which should be updated new formula PR adds a new formula to Homebrew/homebrew-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants