Merge: fix nitcc assert failure and nitcc continuous integration
authorJean Privat <jean@pryen.org>
Wed, 14 Aug 2019 15:22:21 +0000 (11:22 -0400)
committerJean Privat <jean@pryen.org>
Wed, 14 Aug 2019 15:22:21 +0000 (11:22 -0400)
commit72681ae381a89d97b15c1cb7e2a32f411c7d525f
treef49b5444f1c3bc2ea051e9fb0f9966dda2eb0a9c
parent17ea05c8a942c42ed83c3206c5226e22c26eee8e
parentb032dd421ed716bcee4cfae3ee26de2af166557b
Merge: fix nitcc assert failure and nitcc continuous integration

Since #2573, some nitcc tests failed on a self-check (because the trim function did not correctly remove tags on trimmed nodes).

The bug was benign and easily fixable, but the subversion of the expectation was that the script kind of forgot about correctly reporting the error (an error message was printed but a success error code was returned). So the whole CI job was a success as failed tests where unreported.

Reported-by: Florian Deljarry <deljarry.florian@gmail.com>

Pull-Request: #2772