Skip to content

Updated gfortran.rb#37

Open
toyoxx wants to merge 1 commit into
ros:masterfrom
toyoxx:patch-1
Open

Updated gfortran.rb#37
toyoxx wants to merge 1 commit into
ros:masterfrom
toyoxx:patch-1

Conversation

@toyoxx

@toyoxx toyoxx commented Feb 7, 2019

Copy link
Copy Markdown

Substituted MacOS.prefer_64_bit? for Hardware::CPU.is_64_bit? to allow new Homebrew installations tap on ros/deps

Substituted MacOS.prefer_64_bit? for Hardware::CPU.is_64_bit? to allow new Homebrew installations tap on ros/deps
@Coffee-Boyy

Coffee-Boyy commented Mar 4, 2019

Copy link
Copy Markdown

To use this tap until the PR gets pulled in, do:
brew tap toyoxx/homebrew-deps

@mattkanwisher

Copy link
Copy Markdown

Any reason why its taking a year to merge this in?

@HeroCC

HeroCC commented Jan 10, 2022

Copy link
Copy Markdown

+1 -- as the MacOS homebrew install docs are updated for melodic, it would be great to have this merged!

EDIT I ran into this error, not sure if this is an upstream issue or something related to this patch, would appreciate your feedback @toyoxx

brew tap toyoxx/homebrew-deps                                                                                                                                                                                                                      1
==> Tapping toyoxx/deps
Cloning into '/usr/local/Homebrew/Library/Taps/toyoxx/homebrew-deps'...
remote: Enumerating objects: 108, done.
remote: Total 108 (delta 0), reused 0 (delta 0), pack-reused 108
Receiving objects: 100% (108/108), 28.61 KiB | 1010.00 KiB/s, done.
Resolving deltas: 100% (49/49), done.
Error: Invalid formula: /usr/local/Homebrew/Library/Taps/toyoxx/homebrew-deps/gfortran.rb
gfortran: private method `include?' called for #<BuildOptions:0x00007f87828a7580>
Error: Cannot tap toyoxx/deps: invalid syntax in tap!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants