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

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