Re: [RFC 0/5] Date Mode: Add --time-zone; deprecate --date=local

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Michael Witten <mfwitten@xxxxxxxxx> writes:

>>> I took a cursory look, but I've spent so much time on this series already
>>> that I don't really care what it says...
>>
>> This is not a very good way to motivate somebody who is already tired at
>> the end of the day to reivew the RFC series, I would have to say.
>
> Then you can save us both time and just apply my series immediately :-P

That's not how things work around here.

The default is not to apply, unless the goal of the change is worthy and
the patch implements that change well.  It is submitter's job to convince
people that the change is worthy, justify that it is worth reviewers' time
to review the patch, and respond to questions and suggestions for
improvements.

By doing these responsibly, you win support for the particular patchset,
and win trust from others in you, which would affect further patches from
you.

It is not exactly a good way to win support to dump a patchset, to declare
that you are beyond the point of rethinking, to say a take-it-or-leave-it,
nor to defend your private style that goes against the project's coding
style (style is subjective and there is no right or wrong).

I know you know all of the above from your ":-P" (and your past patch
submissions), but I am writing these down so that people new to the list
do not get a wrong impression from this exchange.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]