Skip to content
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

Fix fail message construction in cc_shared_library #14697

Closed
wants to merge 1 commit into from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Feb 3, 2022

Previously, if there were many unaccounted for libraries, this would simply fail with:

Error in join: 'string' is not iterable

Previously, if there were many unaccounted for libraries, this would simply fail with:
```
Error in join: 'string' is not iterable
```
@fmeum
Copy link
Collaborator Author

fmeum commented Feb 3, 2022

@oquenchil This is a one-line Starlark fix.

@gregestren gregestren added the team-Rules-CPP Issues for C++ rules label Feb 3, 2022
Copy link
Contributor

@oquenchil oquenchil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bazel-io bazel-io closed this in 536f8d9 Apr 7, 2022
@ankit-agarwal-ai
Copy link
Contributor

Can we cherrypick this onto 5.1.1. It kind of blocks a core migration workflow (i.e analyzing unaccounted for deps)

ankit-agarwal-ai pushed a commit to AppliedIntuition/bazel that referenced this pull request May 12, 2022
Previously, if there were many unaccounted for libraries, this would simply fail with:
```
Error in join: 'string' is not iterable
```

Closes bazelbuild#14697.

PiperOrigin-RevId: 440090957
@fmeum
Copy link
Collaborator Author

fmeum commented May 12, 2022

@bazel-io flag

@fmeum fmeum deleted the patch-2 branch May 12, 2022 05:50
@bazel-io bazel-io added the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label May 12, 2022
@ckolli5
Copy link

ckolli5 commented Jun 17, 2022

@bazel-io fork 5.3.0

@bazel-io bazel-io removed the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Jun 17, 2022
ckolli5 added a commit that referenced this pull request Jun 21, 2022
Previously, if there were many unaccounted for libraries, this would simply fail with:
```
Error in join: 'string' is not iterable
```

Closes #14697.

PiperOrigin-RevId: 440090957

Co-authored-by: Fabian Meumertzheim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-CPP Issues for C++ rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants