lkml.org 
[lkml]   [2012]   [Dec]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v7 00/27] x86, boot, 64bit: Add support for loading ramdisk and bzImage above 4G
Hi Yinghai,

On Tue, Dec 18, 2012 at 03:08:16PM -0800, Yinghai Lu wrote:
> On Tue, Dec 18, 2012 at 2:43 PM, Borislav Petkov <bp@alien8.de> wrote:
> > you don't care about properly written commit messages, as long as they work
>
> It seems that you are quite upset somehow.

sorry if I've made the wrong impression, but I'm not upset at all - I
was simply asking.

> Writing changelog that everyone could understand seems not be possible
> for me. Even I tried harder and splt patches to small pieces.
>
> This patchset is still on going stage, and still need to check with
> kgdb guys. I sent v7 out for more broader review, as some or most guys
> like to check that in plain-text email instead of git branch.
>
> So don't worry, commit log problem will be addressed.

I understand all that but I shouldn't be telling you how the whole
process works: you write a patch addressing one logical thing at a time,
write a commit message explaining the reason for the patch and then send
it out to everyone for review.

During the review, you read review emails and if you agree with the
suggested changes, you incorporate them in your patchset. This is the
continuous improvement we do until the patchset is mergeable.

Now you say, writing understandable changelogs are not possible for you
- well, I had one changelog written for you and asked you whether this
is what you mean. You didn't say anything. If it were, you could've
added it to the commit message and thus hopefully make the patch more
understandable. And I'm not saying I can write better, but I'm trying
and sometimes it seems to work.

All I'm saying is, if you have trouble with writing commit messages, we
can write them for you and you can tell us whether they make sense or
not. And use those pieces which make sense. And learn along the way.
I mean, we're all learning here; hell, I'm learning a lot from your
patchset too.

This is why I tried to review it, it teaches me stuff.

And the main thing is, if you don't write proper commit messages and do
not explain your code, this thing grows into an unmaintainable mess and
we don't do that crap in Linux. We want this code to be understandable
to more people not to less.

This is all I'm saying, please write more understandable commit messages
and if you can't, then ask us - we're more than glad to help.

Thanks.

--
Regards/Gruss,
Boris.

Sent from a fat crate under my desk. Formatting is fine.
--


\
 
 \ /
  Last update: 2012-12-19 00:41    [W:0.076 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site