Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag

From	Javier Martinez Canillas <javier@dowhile0.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag
Date	2015-10-28 16:00 +0100
Message-ID	<qoAmL-3WD-67@gated-at.bofh.it> (permalink)
References	(6 earlier) <qoxRT-2uq-11@gated-at.bofh.it> <qoyEj-329-65@gated-at.bofh.it> <qoz7l-3cS-49@gated-at.bofh.it> <qoA3o-3Pi-29@gated-at.bofh.it> <qoA3o-3Pi-27@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hello Lee,

[removed since we have been discussing the same back and forth]

On Wed, Oct 28, 2015 at 3:38 PM, Lee Jones <lee.jones@linaro.org> wrote:
>>
>> No need for that, I already explained my point of view several times
>> and you just think I'm wrong because I don't agree with you. So let's
>> just agree on disagree ;-)
>
> I do think you're wrong.  And you think I am wrong.  My reasoning is
> based on logic, common sense and words, and yours is based on, well, I
> really don't know. ;)
>

My reasoning is based on two things basically:

1) Simplicity: it is easier to think in terms of developers (that post
patches) and maintainers (that reviews / acks patches) instead of
developers (that post patches), reviewers (that review patches) and
maintainers (that collect patches and push them upstream).

Individual developers posting patches should not even care what is the
flow of a patch into mainline IMHO, just who should be in copy (which
should be both the "own" of the file whatever is called and the
subsystem maintainer).

2) Consistency: I think that the developer / maintainer split is what
more people are familiar with than the developer / reviewer /
maintainer split.

I don't really have a strong opinion and I'm completely OK if all the
people that don't keep a tree are renamed to Reviewer, Author,
Whatever instead of Maintainer. The only thing that I ask is if this
could be made consistently across all subsystems and have an agreement
in consensus so developers knows what the semantics are regardless of
what subsystems they need to contribute.

> So yes, we'll have to agree to disagree or neither of us will get any
> real work done today.
>

Agreed, I spent most of my morning on this thread ;)

Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-27 16:50 +0100
  Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Sebastian Reichel <sre@kernel.org> - 2015-10-27 18:30 +0100
    Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-27 19:20 +0100
      Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Joe Perches <joe@perches.com> - 2015-10-27 19:50 +0100
        Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-28 02:50 +0100
          Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 09:40 +0100
            Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Javier Martinez Canillas <javier@dowhile0.org> - 2015-10-28 10:30 +0100
              Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Javier Martinez Canillas <javier@dowhile0.org> - 2015-10-28 10:30 +0100
              Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-28 10:40 +0100
              Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 11:30 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Javier Martinez Canillas <javier@dowhile0.org> - 2015-10-28 12:00 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Joe Perches <joe@perches.com> - 2015-10-28 12:10 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Javier Martinez Canillas <javier@dowhile0.org> - 2015-10-28 12:30 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 12:40 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 13:20 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Joe Perches <joe@perches.com> - 2015-10-28 13:30 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 13:30 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Joe Perches <joe@perches.com> - 2015-10-28 13:50 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Javier Martinez Canillas <javier@dowhile0.org> - 2015-10-28 14:10 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 14:40 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 15:40 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Javier Martinez Canillas <javier@dowhile0.org> - 2015-10-28 16:00 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-29 01:00 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Javier Martinez Canillas <javier@dowhile0.org> - 2015-10-29 01:20 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-30 18:00 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-30 18:00 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Javier Martinez Canillas <javier@dowhile0.org> - 2015-10-28 15:40 +0100
            Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-28 10:30 +0100
              Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Uwe Kleine-König   <u.kleine-koenig@pengutronix.de> - 2015-10-28 10:40 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 11:00 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-28 14:20 +0100
                [PATCH] get_maintainer: Add subsystem to reviewer output Joe Perches <joe@perches.com> - 2015-10-28 17:50 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Lee Jones <lee.jones@linaro.org> - 2015-10-28 18:10 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Joe Perches <joe@perches.com> - 2015-10-28 18:10 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Lee Jones <lee.jones@linaro.org> - 2015-10-28 18:30 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Joe Perches <joe@perches.com> - 2015-10-28 18:40 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Lee Jones <lee.jones@linaro.org> - 2015-10-28 18:50 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Joe Perches <joe@perches.com> - 2015-10-28 19:00 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Lee Jones <lee.jones@linaro.org> - 2015-10-29 10:30 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Joe Perches <joe@perches.com> - 2015-10-29 15:20 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Lee Jones <lee.jones@linaro.org> - 2015-10-29 17:20 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Joe Perches <joe@perches.com> - 2015-10-28 18:20 +0100
                Re: [PATCH] get_maintainer: Add subsystem to reviewer output Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-29 00:50 +0100
              Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 11:20 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-28 14:30 +0100
                Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 14:50 +0100
            Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-10-28 17:30 +0100
              Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones <lee.jones@linaro.org> - 2015-10-28 17:40 +0100
  Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Chanwoo Choi <cw00.choi@samsung.com> - 2015-10-28 10:00 +0100

csiph-web