On Fri, 2016-06-17 at 15:37 +0300, Dmitry V. Levin wrote:
On Fri, Jun 10, 2016 at 02:06:43PM +0200, Mark Wielaard wrote:
> +The ideal patch contains a ChangeLog entry and a test case for the
> +bug fixed or feature added. The testsuite (make check) is zero FAIL.
The last sentence looks confusing, could you reword it, please?
I suppose the idea is that testsuite is expected to have zero
failing tests.
Yes. I meant that people that do add new tests should make sure they
PASS (or are skipped). How about replacing that sentence with the
following paragraph explaining the intent:
+The testsuite (make check) is expected to have zero failing tests.
+Do not knowingly add tests that FAIL. If there are architectures or
+configurations where a tests is not supported make sure they are
+skipped instead of failing. Adding "exit 77" in the test shell wrapper
+indicates that a test was SKIPPED.
Thanks,
Mark