Skip to content

Conversation

oherrala
Copy link

  • 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?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

Node can already be build with Homebrew'd OpenSSL. With this change it's possible to build with Homebrew'd LibreSSL.

Building with OpenSSL and LibreSSL is disallowed.

Node can already be build with Homebrew'd OpenSSL. With this change
it's possible to build with Homebrew'd LibreSSL.

Building with OpenSSL and LibreSSL is disallowed.
@oherrala
Copy link
Author

Apparently I had mistake somewhere and Node doesn't build with LibreSSL. Closing PR for now.

@oherrala oherrala closed this Dec 19, 2016
@oherrala
Copy link
Author

There exists nodejs/node#9376.

Check back this PR when that is merged.

@MikeMcQuaid
Copy link
Member

@oherrala I'm afraid we won't include this option even after that as we've standardised on OpenSSL in Homebrew.

@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