[riot-notifications] [RIOT-OS/RIOT] dist/tools/build_system_sanity_check: BUG fix errors being ignored (#11695)

Gaƫtan Harter notifications at github.com
Fri Jun 14 11:04:29 CEST 2019


### Contribution description

On error messages the exit code was still 0 due to the typo.
Having error messages again properly return with an exit code of 1.


### Testing procedure

With the test commit the static tests will fail

### Issues/PRs references

Bug introduced by https://github.com/RIOT-OS/RIOT/pull/11672 which led to having this test commit merged https://github.com/RIOT-OS/RIOT/pull/11694/files

You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/11695

-- Commit Summary --

  * dist/tools/build_system_sanity_check: BUG fix errors being ignored
  * TEST COMMIT Revert "Revert "REMOVE ME TEST COMMIT""

-- File Changes --

    M dist/tools/buildsystem_sanity_check/check.sh (6)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/11695.patch
https://github.com/RIOT-OS/RIOT/pull/11695.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11695
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190614/a63a9883/attachment.html>


More information about the notifications mailing list