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

subversion: explicitly specify the prefix for swig #57657

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

lembacon
Copy link
Member

@lembacon lembacon commented Jul 8, 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>)?

subversion, by default, will try to find swig in /usr/local, however if Howebrew is not installed into the standard prefix, subversion will fail to locate swig.

A revision bump is not needed here.

@SMillerDev SMillerDev added the CI-requeued PR has been re-added to the queue label Jul 8, 2020
@SeekingMeaning SeekingMeaning added almost there PR is nearly ready to merge and removed CI-requeued PR has been re-added to the queue labels Jul 28, 2020
`subversion`, by default, will try to find `swig` in `/usr/local`,
however if Howebrew is not installed into the standard prefix,
`subversion` will fail to locate `swig`.

A revision bump is not needed here.
@jonchang
Copy link
Contributor

jonchang commented Aug 5, 2020

Rebased again for CI issues.

@jonchang jonchang merged commit 525f6d6 into Homebrew:master Aug 5, 2020
@lembacon lembacon deleted the subversion--swig-prefix branch August 7, 2020 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
almost there PR is nearly ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants