Avoid deduplicating linkopts of cc_shared_library dependencies.

[katzdm]

Linker inputs for dependencies of cc_shared_library are currently passing their linkopts as a depset, which leads to deduplication of the flags thereby passed. Consequently, a sequence of well-formed flags like -framework Security -framework IOKit gets collapsed to -framework Security IOKit; the linker then interprets IOKit as an object file that it's unable to locate, which yields a malformed link line.

Passing linkopts directly rather than wrapping it in depset appears to resolve the issue. I've added a test that replicates the original issue; since -framework isn't meaningful on all platforms, I've only enabled it for MacOS.

Note that the tests in this repository utilize the rules_cc repository rather than the built-in rules; therefore, I don't think the new test can be expected to pass until the change has been propagated there, and the controlling MODULE.bazel.lock entry has been updated. I've tested it locally by cating a local_path_override directive to the MODULE.bazel file that gets synthesized for these tests, in order to point the tests towards a local fork of rules_cc with this same change applied; under these conditions, the test passes.

Closes #27735.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[fmeum]

@pzembrod

[fmeum]

@bazel-io fork 9.0.0

[bharadwaj08-one]

@katzdm Could you please take a look at the failing checks?

[katzdm]

@bharadwaj08-one Please see the explanation in the PR description:

Note that the tests in this repository utilize the rules_cc repository rather than the built-in rules; therefore, I don't think the new test can be expected to pass until the change has been propagated there, and the controlling MODULE.bazel.lock entry has been updated. I've tested it locally by cating a local_path_override directive to the MODULE.bazel file that gets synthesized for these tests, in order to point the tests towards a local fork of rules_cc with this same change applied; under these conditions, the test passes.