-
Notifications
You must be signed in to change notification settings - Fork 140
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 makefile to use / fall back on built-in "ar" #2716
Comments
To be clear, did this work with 0.25.0? |
@ParadoxV5 do you think you could take a look at this? |
Running |
Also @forthrin that value is coming from |
⬆. The new release switched from
|
I'm on macOS 14.
|
Prism 0.19 is a default gem for Ruby 3.3, though |
This Help us out @forthrin (and thank you for your patience):
@kddnewton any other thoughts?
So we support ARM? 👌 |
I'm still getting this error after trying the workaround. |
@nickmccurdy This is an issue indeed (a recurring one, sounds like), but I don’t want to just push some workaround (like what the title says). I want to cure it at its root. |
@ParadoxV5: Happy to help
Never compiled Ruby, at least not manually or intentionally. It's a cask.
Don't have Xcode, just CLT. Might have experimentally run
Like which? (Generally haven't had problems with Ruby or gems) |
Google says this’ll brew a formula, not a cask?
That could be something. https://www.unix.com/man-page/OSX/1/xcode-select (from 2013?) implies that this Xcode util manages building tools including but not limited to
Anything that says “Building native extensions. This could take a while” when Does [Edit] |
@kddnewton |
Yes, formula. Ruby is not compiled on the local machine. All brews on the local machine are freshened daily by
|
So if I understand correctly, the issue is that the Ruby is compiled on a different machine and contains config settings for the machine that compiled it, which may or may not function correctly on the actual machine installing prism. In this case, I think it makes sense to check if the AR exists, and if it doesn't then fall back to |
Step by step terminal commands for testing this? Normally just running |
Ensure current Ruby is >= 3.0.0 with
|
Sure the above commands include the PR? Lamentably:
|
Oh sorry, I should have added, please run |
Ditto. cmd = ([ENV['AR']] + `which -a ar`.lines.map(&:chomp)).compact.find { |e| File.exist?(e) } |
Ahh maybe the issue is not in |
Alright, can you pull the latest version of that branch and try again with |
Houston, we have lift-off! |
Fixed on |
What date will this be available in the central Gem repository? (It's breaking |
I'll make a new release today, I'll update the ticket when I do. |
Released with v0.27.0 |
$ gem install prism
Successfully installed prism-0.27.0
Great detective work, you guys. |
No thank you, @kddnewton did all the thing. Though I’m still puzzled on how your Also, the PR logs told that you needed the |
@forthrin I have been trying to understand this issue in more details (here).
? I wonder if maybe an incorrect |
$ xcode-select --print-path
/Library/Developer/CommandLineTools
$ ls $(xcode-select --print-path)
Library SDKs usr
$ ls $(xcode-select --print-path)/bin
ls: /Library/Developer/CommandLineTools/bin: No such file or directory
$ ls $(ruby -e 'puts RbConfig::CONFIG["CC"]')
ls: clang: No such file or directory
$ ls $(ruby -e 'puts RbConfig::CONFIG["AR"]')
ls: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ar: No such file or directory
FYI: $ ls -l /Library/Developer/CommandLineTools/usr/bin
/Library/Developer/CommandLineTools/usr/bin/ar
/Library/Developer/CommandLineTools/usr/bin/clang
# snip |
Thank you, that confirms that This seems very much a case of Homebrew/homebrew-portable-ruby#187 or very similar. |
I used GitHub Actions (with
|
I filed Homebrew/homebrew-core#170978 |
@eregon: Kudos for digging deeper and locating the core issue! |
The text was updated successfully, but these errors were encountered: