Attention is currently required from: fixeria.

osmith has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/simtrace2/+/43530?usp=email )

Change subject: build: do distcheck for the host tools ......................................................................

Patch Set 1: Code-Review+2

(1 comment)

File contrib/jenkins.sh:

https://gerrit.osmocom.org/c/simtrace2/+/43530/comment/7783eead_1b5c6642?usp... : PS1, Line 90: $MAKE distcheck || cat-testlogs.sh I was about to write: the check wouldn't abort on error here unless you explicitly add the exit 1:

```suggestion

if ! $MAKE distcheck; then cat-testlogs.sh exit 1

fi```

... but cat-testlogs.sh already has "exit 1" so this is fine: https://gitea.osmocom.org/osmocom/osmo-ci/src/branch/master/scripts/cat-test...