Skip to content

Commit

Permalink
Fix test script issues
Browse files Browse the repository at this point in the history
Ignore a couple of harmless warnings from CXXTEST harness.

Yet another workaround for #91:
ensure the build-tree libopenasip.so is first in LD_LIBRARY_PATH when
running unit tests through compiletest.sh.
  • Loading branch information
pjaaskel committed Nov 15, 2022
1 parent 512b83a commit 29a4b38
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions openasip/test/Makefile_test.defs
Original file line number Diff line number Diff line change
Expand Up @@ -116,6 +116,7 @@ all: ${GCOV_TEST} ${INITIALIZATION} ${TESTS} ${GCOV_TEST2} ${FINALIZATION}

%-runner.o: %-runner.cpp
${CXX} ${TCE_INCLUDES} ${CPPFLAGS} ${CXXFLAGS} \
-Wno-sign-compare -Wno-ignored-qualifiers -Wno-catch-value \
${EDITLINE_INCLUDES} ${BOOST_INCLUDES} -I ${CXXDIR} -I ${CXXDIR}/.. \
${EXTRA_COMPILER_FLAGS} \
-g -c $< -o $@
Expand Down
4 changes: 4 additions & 0 deletions openasip/tools/scripts/compiletest.sh
Original file line number Diff line number Diff line change
Expand Up @@ -322,6 +322,10 @@ MAKE_OPT="-k"
export TCE_DEVEL_MODE=1
export PATH="$PATH:/bin:/usr/local/bin:/usr/bin"

# Yet another workaround for https://github.com/cpc/openasip/issues/91
# Ensure we are testing the build tree openasip.so.
export LD_LIBRARY_PATH=$PWD/src/.libs:$LD_LIBRARY_PATH

USER=`whoami`

if [ "$USER" == "root" ]; then
Expand Down

0 comments on commit 29a4b38

Please sign in to comment.