search

jvoisin / fortify-headers

Standalone portable header-based implementation of FORTIFY_SOURCE=3
BSD Zero Clause License
19 stars 3 forks source link

The test runner is broken #42

Open q66 opened 11 months ago

q66 commented 11 months ago

I just noticed that the test runner does not actually fail on any failures.

jvoisin commented 10 months ago

Fixed in f3c0bfd

q66 commented 10 months ago

that's not a fix, it's the make command that should exit with 1 if anything fails

jvoisin commented 10 months ago

I don't think there is an easy elegant way to make the makefile exit with 1 should a test fail, while making it run all the tests.

q66 commented 10 months ago

sure there is, you're already running a shell snippet, just make it a little bigger and if any of the tests returns non-zero, set a variable and if the variable is set at the end, error

or make it a separate helper script and invoke that