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

Openblas failed to pour the bottle on macOS #15990

Closed
4 tasks done
alazarolop opened this issue Sep 11, 2023 · 7 comments · Fixed by #15992
Closed
4 tasks done

Openblas failed to pour the bottle on macOS #15990

alazarolop opened this issue Sep 11, 2023 · 7 comments · Fixed by #15992
Assignees
Labels
bug Reproducible Homebrew/brew bug outdated PR was locked due to age

Comments

@alazarolop
Copy link

alazarolop commented Sep 11, 2023

brew gist-logs <formula> link OR brew config AND brew doctor output

Your system is ready to brew.

Verification

  • My "brew doctor output" says Your system is ready to brew. and am still able to reproduce my issue.
  • I ran brew update and am still able to reproduce my issue.
  • I have resolved all warnings from brew doctor and that did not fix my problem.
  • I searched for recent similar issues at https://github.com/Homebrew/homebrew-core/issues?q=is%3Aissue and found no duplicates.

What were you trying to do (and why)?

I have already installed the formula openblas and I wanted to upgrade it. I have seen a similar issue in another formula #15805

What happened (include all command output)?

==> Upgrading openblas
  0.3.23 -> 0.3.24

Error: Parameter 'old': Expected type Pathname, got type String with value "@@HOMEBREW_PREFIX@@/opt/gc.../gcc/x86_64-apple-darwin22/13"
Caller: /usr/local/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:41
Definition: /usr/local/Homebrew/Library/Homebrew/os/mac/keg.rb:40

What did you expect to happen?

Pouring of the bottle properly.

Step-by-step reproduction instructions (by running brew commands)

brew upgrade openblas
@alazarolop alazarolop added the bug Reproducible Homebrew/brew bug label Sep 11, 2023
@Bo98 Bo98 transferred this issue from Homebrew/homebrew-core Sep 11, 2023
@Bo98
Copy link
Member

Bo98 commented Sep 11, 2023

Looks like change_rpath has the wrong type signature (old and new are always String and never Pathname and ruby-macho expects Strings)

@alazarolop
Copy link
Author

Thank you for the quick reply! Is this something I should fix in my installation or it depends on the formula?

@MikeMcQuaid
Copy link
Member

@alazarolop Can I see your brew config output? Working on a fix, thanks.

@alazarolop
Copy link
Author

Thank you a lot @MikeMcQuaid .

@MikeMcQuaid
Copy link
Member

You're welcome @alazarolop!

@scpeters
Copy link
Member

Looks like change_rpath has the wrong type signature (old and new are always String and never Pathname and ruby-macho expects Strings)

apologies, I added that in #15980 but didn't know what I was doing. It was needed for intermediate form of the pull request, but I think the type declarations aren't needed now. Should I remove the other type signatures that I added to os/mac/keg?

@MikeMcQuaid
Copy link
Member

@scpeters No worries, it happens! I think fine to leave other signatures in unless you know them to be wrong but, lesson learned for me I guess on being wary of these sorts of changes that aren't covered by test.

@github-actions github-actions bot added the outdated PR was locked due to age label Oct 13, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Reproducible Homebrew/brew bug outdated PR was locked due to age
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants