Skip to content
Snippets Groups Projects

Debug build

Merged Daniel Hornung requested to merge f-debug-build into f-release-0.1

Summary

Debug building should "just work" now.

Focus

Correct options changed?

Test Environment

make conan-debug

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

  • All automated tests pass
  • Reference related Issues caosdb/customers/lfpb/management#433
  • 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
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Daniel Hornung
  • Daniel Hornung
  • Daniel Hornung marked the checklist item Annotations in code (Gitlab comments) as completed

    marked the checklist item Annotations in code (Gitlab comments) as completed

  • Daniel Hornung marked the checklist item All automated tests pass as completed

    marked the checklist item All automated tests pass as completed

  • Daniel Hornung requested review from @timm

    requested review from @timm

  • Daniel Hornung added 14 commits

    added 14 commits

    • 54e91c79 - WIP: move constructor and assignment operator for Entity
    • 9de7cf1d - EHN: copy assignment for entity
    • 52e89cd1 - Merge branch 'f-mixed-write-transactions' into f-move-entity
    • 00e64d74 - Merge branch 'f-mixed-write-transactions' into f-move-entity
    • d535de96 - FIX entity move constructor
    • facb19fd - Merge branch 'f-mixed-write-transactions' into f-move-entity
    • b7cefe64 - Merge branch 'dev' into f-move-entity
    • 5ccac6d9 - WIP: move entity
    • 69c2497d - Merge branch 'dev' into f-move-entity
    • d6f573ff - more tests for moving entities
    • 6583f355 - Merge branch 'f-move-entity' of gitlab.indiscale.com:caosdb/src/caosdb-cpplib into f-move-entity
    • afcf530c - WIP: fix logging segfaults
    • 5ad56ce9 - Merge remote-tracking branch 'origin/f-logging-segfaults' into f-debug-and-segfault
    • d3fde05e - ENH: Utility function to reset the grpc arena.

    Compare with previous version

  • Daniel Hornung marked this merge request as draft from 54e91c79

    marked this merge request as draft from 54e91c79

  • Daniel Hornung marked this merge request as ready

    marked this merge request as ready

  • Daniel Hornung added 1 commit

    added 1 commit

    Compare with previous version

  • Daniel Hornung assigned to @henrik and @timm and unassigned @daniel

    assigned to @henrik and @timm and unassigned @daniel

  • Daniel Hornung assigned to @daniel and unassigned @henrik

    assigned to @daniel and unassigned @henrik

  • Daniel Hornung added 1 commit

    added 1 commit

    • 5e0b3974 - WIP: Debug builds with dependencies.

    Compare with previous version

  • Daniel Hornung marked this merge request as draft from 5e0b3974

    marked this merge request as draft from 5e0b3974

  • Timm Fitschen added 4 commits

    added 4 commits

    • f85d05ed - 1 commit from branch dev
    • e5f2855a - Merge branch 'dev' into f-move-entity
    • c88d8980 - Merge branch 'f-move-entity' into f-logging-segfaults
    • 11042207 - Merge branch 'f-logging-segfaults' into f-debug-build

    Compare with previous version

  • Timm Fitschen changed target branch from dev to f-logging-segfaults

    changed target branch from dev to f-logging-segfaults

  • Timm Fitschen resolved all threads

    resolved all threads

  • Timm Fitschen
  • Daniel Hornung added 1 commit

    added 1 commit

    • 657d62bf - FIX: Linking with debugging should work now.

    Compare with previous version

  • Daniel Hornung added 1 commit

    added 1 commit

    • a103192c - ENH: Make code coverage optional and depend on available libraries.

    Compare with previous version

  • Daniel Hornung added 1 commit

    added 1 commit

    • 80590802 - FIX: Debug building works now without clang-tidy installed.

    Compare with previous version

  • Daniel Hornung resolved all threads

    resolved all threads

  • Daniel Hornung added 1 commit

    added 1 commit

    • 4458d8bc - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Daniel Hornung added 1 commit

    added 1 commit

    • bb4f27fe - DOC: Extended previous commit.

    Compare with previous version

  • Daniel Hornung added 1 commit

    added 1 commit

    Compare with previous version

  • Daniel Hornung
  • Daniel Hornung
  • Timm Fitschen added 7 commits

    added 7 commits

    Compare with previous version

  • Timm Fitschen added 13 commits

    added 13 commits

    Compare with previous version

  • Timm Fitschen added 3 commits

    added 3 commits

    Compare with previous version

  • Timm Fitschen added 1 commit

    added 1 commit

    Compare with previous version

  • Timm Fitschen added 1 commit

    added 1 commit

    Compare with previous version

  • Timm Fitschen added 1 commit

    added 1 commit

    • 5bc7d1aa - DEPS: update boost dependency

    Compare with previous version

  • Timm Fitschen added 1 commit

    added 1 commit

    Compare with previous version

  • Timm Fitschen added 1 commit

    added 1 commit

    Compare with previous version

  • Timm Fitschen added 12 commits

    added 12 commits

    Compare with previous version

  • Timm Fitschen added 19 commits

    added 19 commits

    • 22abf083 - BUG: Add enum name to cases.
    • e35253f6 - MAINT: Rename DELETE for windows build
    • 3b739b68 - Replace boost::filesystem with std::filesystem
    • 111c8ebd - Fix load_string_file
    • aacdcc54 - Cherry-pick changes from f-debug-build
    • 940ffaab - Add option for static linkage
    • 2e6e09f6 - Rename TransactionType cases to ..._ONLY
    • eee42d19 - Fix CMakeLists.txt (BUILD_SHARED_LIBS option)
    • 5538d36e - Set CMAKE_MSVC_RUNTIME_LIBRARY for WIN32
    • 9b37d1ca - Fix includes
    • e22afbc1 - Define __attribute__ for non-gnuc compilers
    • 11fc7fba - Update CMakeLists for WIN32
    • dba650f6 - WIP mixed shared/static setup
    • 7c6601d8 - fix test clients
    • cc650271 - Merge branch 'dev' into f-fix-windows-build
    • 7cdbe084 - REL: prepare release
    • 226cb1e2 - Merge branch 'f-release-0.1' into f-debug-build
    • de5b6541 - STY: formatting
    • d3f36157 - WIP: fix CMakeLists and includes

    Compare with previous version

  • Timm Fitschen added 1 commit

    added 1 commit

    Compare with previous version

  • Timm Fitschen added 1 commit

    added 1 commit

    Compare with previous version

  • Timm Fitschen changed target branch from f-logging-segfaults to f-release-0.1

    changed target branch from f-logging-segfaults to f-release-0.1

  • Timm Fitschen added 3 commits

    added 3 commits

    • 5a46497f - DEPS: update compatible server version
    • 081e1552 - Remove unused code
    • 3d181764 - Merge branch 'dev' into f-debug-build

    Compare with previous version

  • Timm Fitschen added 1 commit

    added 1 commit

    Compare with previous version

  • Timm Fitschen resolved all threads

    resolved all threads

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading