[Buildroot] [PATCH 2/2] samba4: bump to version 4.2.0
thomas.petazzoni at free-electrons.com
Fri Mar 6 09:40:10 UTC 2015
Dear Gustavo Zacarias,
On Fri, 06 Mar 2015 06:34:59 -0300, Gustavo Zacarias wrote:
> > I'd say readability. Renaming patches is clearly separated from .mk
> > changes. In the case of your Samba 4.2 bump, the changes within the .mk
> > file are mixed between bump-related changes, and indentation-related
> > changes, and this is what bothers me.
> It doesn't break any bisectability so what's the problem, you don't like
> a patch that changes too many things at once?
Exactly. I'm surprised you even ask, separate logical changes is just
the 101 of open-source contribution in many projects. See
> Sorry to rehash an old thread but this kind of things don't motivate me
> at all.
I don't quite understand your feeling here. What I'm asking you to do
is something we ask to *all* contributors, including newcomers who have
never contributed a single patch to Buildroot. Why would we have more
relaxed/special rules for long-term contributors like you ?
If I was annoying you with something unusual, which I never bother
other people with, I would understand. But here I'm just asking a very
basic thing, which I also ask to every other contributor.
> One thing is when you say "typo" which well, yes, it's a typo and it's
> fine to correct it.
Sorry I did not understand this part.
Anyway, I'll take care of doing the split of the Samba 4.2 patch and
I'll apply. I would have expected a bit more help and understanding from
a long term contributor such as you.
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
More information about the buildroot