* Re: [PATCH] CONTRIBUTING: Extend patch, committer and maintainer guidelines.
@ 2016-06-17 13:16 Mark Wielaard
0 siblings, 0 replies; 5+ messages in thread
From: Mark Wielaard @ 2016-06-17 13:16 UTC (permalink / raw)
To: elfutils-devel
[-- Attachment #1: Type: text/plain, Size: 945 bytes --]
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] CONTRIBUTING: Extend patch, committer and maintainer guidelines.
@ 2016-06-21 19:48 Mark Wielaard
0 siblings, 0 replies; 5+ messages in thread
From: Mark Wielaard @ 2016-06-21 19:48 UTC (permalink / raw)
To: elfutils-devel
[-- Attachment #1: Type: text/plain, Size: 784 bytes --]
On Fri, Jun 17, 2016 at 02:02:33PM +0200, Mark Wielaard wrote:
> On Fri, 2016-06-10 at 14:06 +0200, Mark Wielaard wrote:
> > +As a maintainer/committer you should still post patches as described
> > +above. And ideally they are reviewed and approved as above. If no
> > +other committer has reviewed or objected to your patch for a week
> > +you may use your own judgement whether you ping your patch or push
> > +it after "self-review". If you do, you should post a message to the
> > +mailinglist that the patch has been pushed.
>
> Since this is a patch describing how we work together I wanted to give
> people one more chance to comment on it. If nobody objects I'll push it
> on Monday.
Pushed now with the wording change discussed with Dmitry.
Cheers,
Mark
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] CONTRIBUTING: Extend patch, committer and maintainer guidelines.
@ 2016-06-17 12:37 Dmitry V. Levin
0 siblings, 0 replies; 5+ messages in thread
From: Dmitry V. Levin @ 2016-06-17 12:37 UTC (permalink / raw)
To: elfutils-devel
[-- Attachment #1: Type: text/plain, Size: 368 bytes --]
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.
--
ldv
[-- Attachment #2: attachment.sig --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] CONTRIBUTING: Extend patch, committer and maintainer guidelines.
@ 2016-06-17 12:02 Mark Wielaard
0 siblings, 0 replies; 5+ messages in thread
From: Mark Wielaard @ 2016-06-17 12:02 UTC (permalink / raw)
To: elfutils-devel
[-- Attachment #1: Type: text/plain, Size: 637 bytes --]
On Fri, 2016-06-10 at 14:06 +0200, Mark Wielaard wrote:
> +As a maintainer/committer you should still post patches as described
> +above. And ideally they are reviewed and approved as above. If no
> +other committer has reviewed or objected to your patch for a week
> +you may use your own judgement whether you ping your patch or push
> +it after "self-review". If you do, you should post a message to the
> +mailinglist that the patch has been pushed.
Since this is a patch describing how we work together I wanted to give
people one more chance to comment on it. If nobody objects I'll push it
on Monday.
Cheers,
Mark
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] CONTRIBUTING: Extend patch, committer and maintainer guidelines.
@ 2016-06-10 12:06 Mark Wielaard
0 siblings, 0 replies; 5+ messages in thread
From: Mark Wielaard @ 2016-06-10 12:06 UTC (permalink / raw)
To: elfutils-devel
[-- Attachment #1: Type: text/plain, Size: 3261 bytes --]
Signed-off-by: Mark Wielaard <mjw@redhat.com>
---
CONTRIBUTING | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
ChangeLog | 4 ++++
2 files changed, 52 insertions(+)
diff --git a/CONTRIBUTING b/CONTRIBUTING
index 3f5e5aa..418b9c7 100644
--- a/CONTRIBUTING
+++ b/CONTRIBUTING
@@ -46,3 +46,51 @@ using your real name (sorry, no pseudonyms or anonymous contributions.)
git commit --signoff will add such a Signed-off-by line at the end of
the commit log message for you.
+
+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.
+Do not commit patches if you know they might break a testcase.
+
+We do allow binaries in the testsuite for tests that only need to
+read ELF or DWARF data and if generating the data in the testcase
+itself is difficult or would be architecture specific.
+The binaries should be bzip2 compressed. Add a note in the test
+wrapper run-<testcase>.sh script how to regenerate the binary.
+
+After sending your patch to the mailinglist one of the committers
+to the project will review it, give feedback, and if perfect they
+will commit it for you.
+
+The current maintainers/committers can be found at:
+https://admin.fedoraproject.org/accounts/group/members/gitelfutils/*
+
+You can become a maintainer/committer yourself after you have provided
+at least a handful of accepted patches and agree to the guidelines in
+this document for creating, reviewing, accepting and committing patches.
+
+To become a committer you need an FAS account at:
+https://admin.fedoraproject.org/accounts/
+Upload a SSH public key and have an existing maintainer sponsor you
+for the Elf Utils Group (gitelfutils).
+
+committers can push patches through:
+ssh://<fasname>@git.fedorahosted.org/git/elfutils.git
+
+As a maintainer/committer you should still post patches as described
+above. And ideally they are reviewed and approved as above. If no
+other committer has reviewed or objected to your patch for a week
+you may use your own judgement whether you ping your patch or push
+it after "self-review". If you do, you should post a message to the
+mailinglist that the patch has been pushed.
+
+committers may also create git branches starting with <fasname>/...
+patches on these branches are works in progress, so might not be perfect
+yet, but should follow the above guidelines as much as possible and should
+be aimed at integration into master. For merging a branch into master
+the same process as above should be followed by posting the patches
+to the list first.
+
+committers/maintainers who repeatedly ignore the above guidelines,
+are hostile or offensive towards other committers or contributors,
+and don't correct their behavior after being asked by other committers
+will be removed as maintainer/committer.
diff --git a/ChangeLog b/ChangeLog
index 1ec202a..1a7d9cc 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2016-06-10 Mark Wielaard <mjw@redhat.com>
+
+ * CONTRIBUTING: Extend patch, committer and maintainer guidelines.
+
2016-05-02 Filipe Brandenburger <filbranden@google.com>
* configure.ac (argp check): Pass pass &argv.
--
2.5.5
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-06-21 19:48 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-17 13:16 [PATCH] CONTRIBUTING: Extend patch, committer and maintainer guidelines Mark Wielaard
-- strict thread matches above, loose matches on Subject: below --
2016-06-21 19:48 Mark Wielaard
2016-06-17 12:37 Dmitry V. Levin
2016-06-17 12:02 Mark Wielaard
2016-06-10 12:06 Mark Wielaard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).