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

has no member named ‘always_print_primitive_fields’ #566

Closed
drbeco opened this issue Mar 24, 2024 · 7 comments
Closed

has no member named ‘always_print_primitive_fields’ #566

drbeco opened this issue Mar 24, 2024 · 7 comments

Comments

@drbeco
Copy link
Contributor

drbeco commented Mar 24, 2024

New protobuf3-26.0 dependency for Bear causes the error:

[ 35%] Building CXX object intercept/CMakeFiles/events_db_a.dir/source/collect/db/EventsDatabaseWriter.cc.o
/tmp/SBo/Bear-3.1.3/source/intercept/source/collect/db/EventsDatabaseWriter.cc: In function ‘google::protobuf::util::JsonPrintOptions {anonymous}::create_print_options()’:
/tmp/SBo/Bear-3.1.3/source/intercept/source/collect/db/EventsDatabaseWriter.cc:39:23: error: ‘using JsonPrintOptions = struct google::protobuf::json::PrintOptions’ {aka ‘struct google::protobuf::json::PrintOptions’} has no member named ‘always_print_primitive_fields’
   39 |         print_options.always_print_primitive_fields = true;
      |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
make[5]: *** [intercept/CMakeFiles/events_db_a.dir/build.make:90: intercept/CMakeFiles/events_db_a.dir/source/collect/db/EventsDatabaseWriter.cc.o] Error 1
make[4]: *** [CMakeFiles/Makefile2:388: intercept/CMakeFiles/events_db_a.dir/all] Error 2
make[3]: *** [Makefile:136: all] Error 2
make[2]: *** [CMakeFiles/BearSource.dir/build.make:87: subprojects/Stamp/BearSource/BearSource-build] Error 2
make[1]: *** [CMakeFiles/Makefile2:174: CMakeFiles/BearSource.dir/all] Error 2
make: *** [Makefile:156: all] Error 2

Using Slackware 15.0 as OS.

@drbeco
Copy link
Contributor Author

drbeco commented Mar 24, 2024

I'll create a pull request with an update on the offending source code

@rizsotto
Copy link
Owner

Thanks @drbeco for the report and for the PR!

rizsotto added a commit that referenced this issue Mar 26, 2024
@rizsotto
Copy link
Owner

reverting this change, since it breaks all CI pipelines.

need to pick up the dependency version and guard with preprocessor macros to assign the field conditionally.

@rizsotto rizsotto reopened this Mar 26, 2024
@drbeco
Copy link
Contributor Author

drbeco commented Mar 29, 2024

The CI pipelines error messages don't say much to help the case

@drbeco
Copy link
Contributor Author

drbeco commented Mar 29, 2024

This new PR is running the CI pipelines. Lets hope and see.

@drbeco
Copy link
Contributor Author

drbeco commented Mar 29, 2024

Failed (?) Confirm?
Now it is me who is running... running out of ideas.

@kovmir
Copy link

kovmir commented Apr 11, 2024

Broken on Slackware -current too.

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

No branches or pull requests

3 participants