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

Update brewsci-r.rb #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Gasman2014
Copy link

Unable to install brewsci-mumps due to error in associated package, brewsci-r.

I think this pr should fix it but I seem to be going round in circles. I have installed other formulae from this repo (eg Tetgen) but Brew insists on going back to tap the original brewsci/num repo and then failing on brewsci-r - which is a package I don't even want to install from here (I have it already).

$brew tap
gasman2014/num
homebrew/cask
homebrew/core

$brew install gasman2014/num/brewsci-mumps
==> Installing brewsci-mumps from gasman2014/num
==> Tapping brewsci/num                                          <------------- WHY?????
Cloning into '/usr/local/Homebrew/Library/Taps/brewsci/homebrew-num'...
remote: Enumerating objects: 69, done.
remote: Counting objects: 100% (69/69), done.
remote: Compressing objects: 100% (49/49), done.
remote: Total 470 (delta 43), reused 34 (delta 20), pack-reused 401
Receiving objects: 100% (470/470), 98.92 KiB | 1.87 MiB/s, done.
Resolving deltas: 100% (215/215), done.
Error: Invalid formula: /usr/local/Homebrew/Library/Taps/brewsci/homebrew-num/Formula/brewsci-r.rb
brewsci-r: Calling depends_on :java is disabled! Use "depends_on "openjdk@11", "depends_on "openjdk@8" or "depends_on "openjdk" instead.
Please report this issue to the brewsci/num tap (not Homebrew/brew or Homebrew/core), or even better, submit a PR to fix it:
  /usr/local/Homebrew/Library/Taps/brewsci/homebrew-num/Formula/brewsci-r.rb:25

@dpo
Copy link
Contributor

dpo commented Jan 17, 2021

What's gasman2014/num? Is it a fork of this repo? If so, I think brew insists on tapping this repo because formulae tell it to, e.g., https://github.com/brewsci/homebrew-num/blob/master/Formula/brewsci-mumps.rb#L24.

One build error is

/usr/local/Homebrew/Library/Taps/brewsci/homebrew-num/Formula/brewsci-r.rb:25:3: C: [Correctable] dependency "openjdk@8" (line 25) should be put before dependency "xz" (line 24)

The other seems to be occurring during the R configure step:

checking whether bzip2 support suffices... configure: error: bzip2 library and headers are required

I can't look into this right now but I'll be happy to review a PR.

You may be able to fix things temporarily in your local tap by changing occurrences of brewsci/num in the formulae to gasman2014/num.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants