> -----Original Message----- > From: ASSI > Sent: Monday, May 10, 2021 1:06 AM > > Jason Pyeron writes: > > Cygport test is not returning a non zero exit status on failure! Is > > this expected? > > Yes, it is using 'make -k' under the hood. Again, if you need to do > something else than what the standard invocation provides, you would > generally write your own src_test() function.
$ cat -n /usr/share/cygport/lib/src_test.cygpart <snip/> 33 cygtest() { 34 if [ -e Makefile -o -e GNUmakefile -o -e makefile ] 35 then 36 if make -n check &> /dev/null 37 then 38 make -k check || true 39 elif make -n test &> /dev/null 40 then 41 make -k test || true 42 else 43 inform "No testsuite detected."; 44 fi 45 else 46 inform "No testsuite detected."; 47 fi 48 } <snip/> What is the historic rationale behind the "OR true" after the make check? It seems silly to have to redefine src_test() as { cd ${B} make check }, just to have a failure exit code if the test fail. It seems to have come in at https://github.com/cygwin/cygport/commit/ef93794f35de38b28f473b8f14c4a9e5a4a4a5a6 and never changed since. The commit does not indicate why the exit status is suppressed. -Jason