Failures in basic.mk recipes can be lost in pipes

[mykter]

basic.mk is my go-to starting point for Go projects, thanks!

I noticed that a CI job of mine was missing golint, but make all was succeeding. The golint | tee approach was hiding the error return code.

Adding .SHELLFLAGS = -o pipefail -c fixed it for me (i.e. the build now correctly fails if golint is missing).

I see basic.mk has moved on a bit from the version I'm working from, I don't know if the greps in the pipes might mean this approach isn't viable any more?

Just raising this issue in case it's helpful - this isn't an issue with the weather tool, but I couldn't see a 'home' repo for basic.mk. Please do close it if this is the wrong project or this isn't a problem in your use case!

[issue-label-bot]

Issue-Label Bot is automatically applying the label bug to this issue, with a confidence of 0.76. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.