Fix ambiguous segfault for test requiring MORPHEUS_ROOT #514

cwharris · 2022-12-02T18:26:40Z

Fixes #401

cwharris · 2022-12-03T06:44:00Z

rerun tests

…ssue_401

morpheus/_lib/tests/CMakeLists.txt

morpheus/_lib/tests/test_morpheus.hpp

mdemoret-nv

Can merge once CI is passing

cwharris · 2022-12-08T07:02:27Z

cwharris added 2 commits December 2, 2022 12:09

add MORPHEUS_ROOT as ENV to container build

27618c1

throw exception in tests if MORPHEUS_ROOT not set

83e2004

cwharris requested a review from a team as a code owner December 2, 2022 18:26

cwharris added bug Something isn't working non-breaking Non-breaking change labels Dec 2, 2022

cwharris and others added 7 commits December 5, 2022 13:31

fix styles

c7c5bd2

Merge branch 'branch-23.01' of github.com:nv-morpheus/Morpheus into i…

e302622

…ssue_401

move get_morpheus_root to test_morpheus.hpp

a3543c3

move get_morpheus_root to its own file.

e4eaa14

fix styles

7261384

Merge branch 'branch-23.01' of github.com:nv-morpheus/Morpheus into i…

5c66df9

…ssue_401

Merge branch 'branch-23.01' into issue_401

04e2c83

mdemoret-nv requested changes Dec 7, 2022

View reviewed changes

morpheus/_lib/tests/CMakeLists.txt Outdated Show resolved Hide resolved

morpheus/_lib/tests/test_morpheus.hpp Show resolved Hide resolved

cwharris added 2 commits December 7, 2022 12:57

address review feedback

d4d8f8d

Merge branch 'issue_401' of github.com:cwharris/Morpheus into issue_401

5aab9c7

cwharris requested a review from mdemoret-nv December 7, 2022 18:58

mdemoret-nv approved these changes Dec 7, 2022

View reviewed changes

ghost merged commit e681951 into nv-morpheus:branch-23.01 Dec 8, 2022

This pull request was closed.

Provide feedback