Skip to content

Fix invalid linker name in argument '-fuse-ld=ld64.lld' on macOS#516

Closed
fmeum wants to merge 2 commits intobazelbuild:mainfrom
fmeum:patch-2
Closed

Fix invalid linker name in argument '-fuse-ld=ld64.lld' on macOS#516
fmeum wants to merge 2 commits intobazelbuild:mainfrom
fmeum:patch-2

Conversation

@fmeum
Copy link
Copy Markdown
Collaborator

@fmeum fmeum commented Nov 4, 2025

This is caused by this extra warning being printed by the clang invocation used to detect the linker path:

ld64.lld: warning: directory not found for option -L/usr/local/lib

Fix this by making the match for the path more specific.

This is caused by this extra warning being printed by the clang invocation used to detect the linker path:

```
ld64.lld: warning: directory not found for option -L/usr/local/lib
```

Fix this by making the match for the path more specific.
@fmeum fmeum marked this pull request as ready for review November 4, 2025 11:02
@fmeum
Copy link
Copy Markdown
Collaborator Author

fmeum commented Nov 4, 2025

FYI @keith @brentleyjones

comius
comius previously approved these changes Nov 4, 2025
pzembrod
pzembrod previously approved these changes Nov 4, 2025
Comment thread cc/private/toolchain/unix_cc_configure.bzl
@fmeum fmeum dismissed stale reviews from pzembrod and comius via 8e75534 November 4, 2025 11:15
@fmeum fmeum requested a review from pzembrod November 4, 2025 11:16
@fmeum
Copy link
Copy Markdown
Collaborator Author

fmeum commented Nov 17, 2025

@pzembrod Friendly ping

@fmeum fmeum deleted the patch-2 branch December 1, 2025 07:41
@fmeum
Copy link
Copy Markdown
Collaborator Author

fmeum commented Dec 1, 2025

@pzembrod It would be really helpful to have this fix in a release, could you cut one?

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.

3 participants