All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] More maintainers
Date: Thu, 3 Sep 2020 18:24:09 +0200	[thread overview]
Message-ID: <20200903182409.6b337059@windsurf.home> (raw)
In-Reply-To: <CAJX3zAuYgzv6_d2Fb8wUrHwPnyg0FEmNSfkiJkQ=zyqEqXKcrA@mail.gmail.com>

Hello Avraham,

On Thu, 3 Sep 2020 18:44:27 +0300
Avraham Shukron <avraham.shukron@gmail.com> wrote:

> For what it's worth I think that the mail-based contribution process is
> part of the problem.
> With a decent Git server like GitHub/GitLab patches could be reviewed more
> easily.
> A CI pipeline could run tests that will give immediate feedback for any
> pull request.
> More importantly, it'll dramatically reduce the barrier for new and young
> contributors.

This has been discussed multiple times in the past in Buildroot, and in
other open-source projects. There is even as we speak some pretty
intense debate in the Linux kernel community about this.

As we've seen from the discussion here, the Buildroot issue is not a
lack of contribution, but a lack of review from trusted and experienced
reviewers, and a lack of maintainers time. So while I'm all for new
contributors and contributions, I don't think reducing the barrier is
really key here. Also, I've always been skeptical about this statement
that using Github reduces the barrier to entry. When you contribute to
a project, is really sending a patch over e-mail the difficult part
compared to understanding the code base, debugging the issue you've
found or implementing the feature you wanted ? Really installing "git
send-email" is a no-brainer straightforward process that is
ridiculously easy even compared writing any single change in the code
base with a decent commit message. Aren't we using this "reducing the
barrier" argument a bit inappropriately here ?

I believe I can say that all four Buildroot maintainers have a very
strong preference for and a very optimized workflow to work with e-mail
based patch submission and review.

Somewhat related, recently a patch series I submitted last year to
OpenWrt (which wasn't merged) got picked up by someone else, and
re-submitted with new updates and fixes. Due to being the original
author, I was in copy of all the Github discussion that took place. And
I found it absolutely impossible and awful to follow the different
revisions of the patch series, to which version of the patch series the
comments were being made, etc.

Perhaps for some people the Github pull request workflow makes sense,
but I believe it's important to recognize and realize that there are
also people for which this workflow doesn't make sense.

> Buildroot is all about using simple and familiar tools like make and
> Kconfig, and I personally think that this principle should also be applied
> to the contribution process and right now buildroot is one of the last
> active open source projects using the mailing list approach.

This is a very biased statement: there are still plenty of open-source
projects that use e-mail based contribution workflow. I don't think we
can call Linux or U-Boot "inactive" projects. Can we? :-)

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2020-09-03 16:24 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-27 17:34 [Buildroot] More maintainers Adam Duskett
2020-08-27 19:35 ` Angelo Compagnucci
2020-08-27 20:39 ` Thomas Petazzoni
2020-08-29  9:50   ` Yann E. MORIN
2020-09-02 17:53     ` Adam Duskett
2020-09-02 20:08       ` Arnout Vandecappelle
2020-09-02 20:11         ` Adam Duskett
2020-09-02 21:51           ` Christian Stewart
2020-09-03 15:44             ` Avraham Shukron
2020-09-03 16:24               ` Thomas Petazzoni [this message]
2020-09-03 16:44                 ` Angelo Compagnucci
2020-09-03 17:29                   ` Adam Duskett
2020-09-04  8:31                     ` Nicolas Cavallari
2020-09-04 17:39                   ` Peter Korsgaard
2020-09-04 18:12                     ` Michael Nazzareno Trimarchi
2020-09-04 19:05                       ` Peter Korsgaard
2020-09-04 19:36                     ` Angelo Compagnucci
2020-09-04 21:10                       ` Peter Korsgaard
2020-09-05  4:52                         ` Christian Stewart
2020-09-05  6:44                           ` Peter Korsgaard
2020-09-05 16:20                             ` Adam Duskett
2020-09-05 17:25                               ` Peter Korsgaard
2020-09-05 19:16                               ` Thomas Petazzoni
2020-09-05 20:10                                 ` Angelo Compagnucci
2020-09-05 20:38                                   ` Thomas Petazzoni
2020-09-05 20:55                                     ` Angelo Compagnucci
2020-09-05 21:04                                       ` Thomas Petazzoni
2020-09-05 21:34                                         ` Angelo Compagnucci
2020-09-05 22:25                                           ` Adam Duskett
2020-09-06  7:43                                             ` Peter Korsgaard
2020-09-06  7:58                                               ` Yegor Yefremov
2020-09-07 15:02                                                 ` Heiko Thiery
2020-09-07 15:20                                                   ` Thomas Petazzoni
2020-09-07 15:43                                                     ` Heiko Thiery
2020-09-10 22:30                                                     ` Christian Stewart
2020-09-05 20:25                               ` Yann E. MORIN
2020-09-05 20:54                                 ` Yann E. MORIN
2020-09-05 18:59                             ` Alexander Dahl
2020-09-05 19:17                               ` Peter Korsgaard
2020-09-05 20:06                                 ` Christian Stewart
2020-09-04 17:30                 ` James Hilliard
2020-09-03 16:28               ` Angelo Compagnucci
2020-09-04 18:52                 ` Peter Korsgaard
2020-09-03 18:39               ` Christian Stewart
2020-09-07 15:57               ` Michael Nosthoff
2020-09-07 18:35                 ` Alexander Dahl
2020-09-09  6:14                   ` Peter Korsgaard
2020-09-05 13:06 ` [Buildroot] More maintainers: a live discussion ? Thomas Petazzoni
2020-09-07  8:02   ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200903182409.6b337059@windsurf.home \
    --to=thomas.petazzoni@bootlin.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.