Attention is currently required from: lynxis lazus.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/libosmocore/+/41362?usp=email )
Change subject: tests/testsuite.at: fix invalid if-endif syntax ......................................................................
Patch Set 1:
(1 comment)
File tests/testsuite.at:
https://gerrit.osmocom.org/c/libosmocore/+/41362/comment/b00c708a_7b0cd54e?u... : PS1, Line 86: AT_SKIP_IF([test ! -e $abs_top_builddir/tests/msgfile/msgfile_test])
Checking the file might be problematic, because old artifact might be still present. […]
Well, we already do this kind of checking in several projects. Checking variables is more complicated because they do not magically materialize here. You would need to define `tests/atlocal.in` like libosmocore.git does for the `sim_test`.
It's always a good practice to do `make distclean` before building the project with different configure flags, so I don't see a big problem here...