This is the mail archive of the mailing list for the elfutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] CONTRIBUTING: Extend patch, committer and maintainer guidelines.

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.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]