-
Notifications
You must be signed in to change notification settings - Fork 79
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
Migrate to the modern linker input API. #1437
Conversation
54215bb
to
318c530
Compare
LGTM, but assigning @aherrmann for an additional review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Should this instance also be changed or is there a reason to keep using libraries_to_link
in that case?
Did you try enabling --incompatible_require_linker_input_cc_api
in .bazelrc
? Would be good to check against regressions on this on CI if possible.
ada01e2
to
9a33859
Compare
Thank you for the review.
Yes, indeed! This makes me realize I accidentally pushed an old version of my branch that didn't fix that case through a git snafu. Should be okay now.
Yes, that's how I was testing locally. I committed this to the branch now, too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
@Mergifyio rebase |
Command
|
9a33859
to
ddb7a93
Compare
See bazelbuild/bazel#10860.