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

Dingpf/upgrade to protobuf #35

Closed
wants to merge 4 commits into from
Closed

Conversation

dingp
Copy link
Contributor

@dingp dingp commented Aug 21, 2023

Adding absl as a dependency, also link protobuf and absl::status in the main library.

There was an inexplicit dependency on absl::status used by ers_schema_test, which is not explicitly specified and caused errors like:

 error adding symbols: DSO missing from command line 

The symbol was released to absl::lts_20230125::Status::UnrefNonInlined(unsigned long)

@dingp
Copy link
Contributor Author

dingp commented Aug 21, 2023

This is to be merged when we switch to the new external software stack (ext-1.1 for SL7, and ext-2.0 for AL9).

In the new external software stack, we have protobuf 3.22.2 and gRPC 1.56.2.

@dingp dingp closed this Aug 21, 2023
@dingp
Copy link
Contributor Author

dingp commented Aug 21, 2023

This is replaced by #36 (changed branch name so that it's better suited to be used in other repos).

@dingp dingp deleted the dingpf/upgrade_to_protobuf branch August 21, 2023 21:27
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.

1 participant