Skip to content

depends: fix for llvm-ranlib (etc): 'No such file or directory' macOS 15.0 #30994

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.

Sign up for GitHub

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

Jump to bottom
Closed
itornaza wants to merge 1 commit into from
Closed

depends: fix for llvm-ranlib (etc): 'No such file or directory' macOS 15.0 #30994

itornaza wants to merge 1 commit into from

Conversation

@itornaza
Copy link
Contributor

@itornaza itornaza commented Sep 28, 2024 •
edited
Loading

This is an attempt to fix the issue #30978 .

To briefly summarize, @Sjors found that while compiling dependencies on macOS 15.0 the following "No such file or directory" are generated while these tools are installed.

$ cd depends
$ make
/bin/sh: command -v llvm-ranlib: No such file or directory
/bin/sh: command -v llvm-strip: No such file or directory
/bin/sh: command -v llvm-nm: No such file or directory
/bin/sh: command -v llvm-objdump: No such file or directory
/bin/sh: command -v dsymutil: No such file or directory

The proposed fix conditionally defines the necessary variables for these tools in the ./depends/Makefile when building on macOS. The rationale is described in more detail in #30978 (comment) as the Solution number 3.

Update 1:

It turns out that Xcode's make is not setting the flags that Gnu does as @hebasto described in #30978 (comment). So, to avoid any architectural changes in the depends directory that may lead to more confusion, the current patch manually sets the required flag in order for the above errors to be resolved.

cc @hebasto

@DrahtBot
Copy link
Contributor

DrahtBot commented Sep 28, 2024 •
edited
Loading

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/30994.

Reviews

See the guideline for information on the review process.
A summary of reviews will appear here.

Conflicts

No conflicts as of last run.

@DrahtBot DrahtBot mentioned this pull request Sep 28, 2024
Merged
@fanquake
Copy link
Member

fanquake commented Oct 2, 2024

Not sure about this issue, or making this change. Looks like it needs more investigation. See my comment here: #30978 (comment).

@itornaza itornaza marked this pull request as draft October 4, 2024 16:30
@itornaza
Copy link
Contributor Author

itornaza commented Oct 4, 2024

Not sure about this issue, or making this change. Looks like it needs more investigation. See my comment here: #30978 (comment).

Yes, switched to draft until we have a better understanding of the issue.

@hebasto
Copy link
Member

hebasto commented Jan 10, 2025

Not sure about this issue, or making this change. Looks like it needs more investigation. See my comment here: #30978 (comment).

Yes, switched to draft until we have a better understanding of the issue.

See #30978 (comment).

@itornaza itornaza force-pushed the ion-issue-30978 branch 2 times, most recently from 32aba7e to ba06659 Compare February 25, 2025 11:24
@itornaza
Copy link
Contributor Author

itornaza commented Feb 25, 2025

Rebased and updated the description.

The errors are now suppressed by setting a flag for make in the depends/hosts/darwin.mk that make implementation of Xcode does not, see #30978 (comment) and #30978 (comment).

@itornaza itornaza marked this pull request as ready for review February 25, 2025 12:50
@DrahtBot
Copy link
Contributor

DrahtBot commented Oct 3, 2025

There hasn't been much activity lately. What is the status here?

Finding reviewers may take time. However, if the patch is no longer relevant, please close this pull request. If the author lost interest or time to work on this, please close it and mark it 'Up for grabs' with the label, so that it can be picked up in the future.

@maflcko
Copy link
Member

maflcko commented Oct 3, 2025

Is this still needed after #30978 (comment) ?

@itornaza itornaza closed this Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Build system

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@itornaza @DrahtBot @fanquake @hebasto @maflcko