Skip to content

F grpc f acm

Timm Fitschen requested to merge f-grpc-f-acm into dev

Summary

Two things happen here. First, the Boost log headers are being removed from our own headers. This is in itself not an enhancement, but it will help in the future making boost a pure build dependency of our library. Currently, boost is a "peer dependency" and all code which uses our headers also have to include the boost headers.

Second, there was a bug in the server (fixed in caosdb-server!53 (merged)) and during fixing it, I noticed that there is also a bug in the cpplib which is only one missing line. As I was at it, I also moved the file_descriptor struct into it's own source.

Focus

  • Look how all occurrences of "boost.log" disappear from the header files and move to the source files instead.
  • The missing line is annotated explicitly.

Test Environment

Manual testing is not necessary. See corresponding caosdb-cppinttest!19 (merged)

Check List for the Author

  • All automated tests pass
  • Reference related Issues
  • Up-to-date CHANGELOG.md
  • Annotations in code (Gitlab comments)
    • Intent of new code
    • Problems with old code
    • Why this implementation?

Check List for the Reviewer

  • I understand the intent of this MR
  • All automated tests pass
  • Up-to-date CHANGELOG.md
  • The test environment setup works and the intended behavior is reproducible in the test environment
  • In-code documentation and comments are up-to-date.
  • Check: Are there specifications? Are they satisfied?

For further good practices have a look at our review guidelines.

Edited by Timm Fitschen

Merge request reports

Loading