Message-ID: <38332.166.70.238.45.1217860413.squirrel@webmail.wolfmountaingroup.com>
In-Reply-To: <4897070E.5050509@s5r6.in-berlin.de>
References: <17494.166.70.238.46.1217784156.squirrel@webmail.wolfmountaingroup.com>
    <43318.166.70.238.45.1217792210.squirrel@webmail.wolfmountaingroup.com>
       <1217808849.2328.43.camel@localhost.localdomain>
    <58898.166.70.238.45.1217816372.squirrel@webmail.wolfmountaingroup.com>
    <4897070E.5050509@s5r6.in-berlin.de>
Date: Mon, 4 Aug 2008 08:33:33 -0600 (MDT)
Subject: Re: [ANNOUNCE] Merkey's Kernel Debugger
From: jmerkey@wolfmountaingroup.com
To: "Stefan Richter" <stefanr@s5r6.in-berlin.de>
Cc: jmerkey@wolfmountaingroup.com, "Josh Boyer" <jwboyer@gmail.com>,
       linux-kernel@vger.kernel.org
User-Agent: SquirrelMail/1.4.6
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7BIT
Importance: Normal
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2646
Lines: 69


OK,  Sounds like I get a D- on patch format submission.   I will rework
the patches, switch back to GPL2 (since I guess GPL 3 is still not there
yet) and clean up this list of issues.   ULONG, etc. is Microsoft syntax
for cross platform compatibility.  Since this is a LINUX SPECIFIC PATCH,
I'll rip out and rework the Gates-isms in the code.

All that aside, the damn works so at least folks can start using it while
I perform code beautification.

Jeff


> jmerkey@wolfmountaingroup.com wrote:
>>> On Sun, 2008-08-03 at 13:36 -0600, jmerkey@wolfmountaingroup.com wrote:
>>>>
>>>> This patch is formally submitted for consideration for inclusion in
>>>> the
>>>> base linux kernel.
>>>>
>>>> ftp://ftp.wolfmountaingroup.org/pub/mdb/mdb-2.6.26-ia32-08-02-08.patch
>>>
>>> Formally submitted patches should be sent to the list inline.
>>> Reviewing
>>> something on an FTP server just becomes that much harder.
>>>
>>> josh
>>>
>>>
>>
>> Submitted as inline patches.
>
> Some non-technical comments to the patch series:
>   - Each patch posting in a patch series should have an own Subject and
>     changelog which specifically describes the included patch.
>   - The Developer's Certificate of Origin is written simply as a single
>     line:
>     Signed-off-by: Jeffrey Vernon Merkey <email@address>
>     This line needs to be included in the changelog of each patch, i.e.
>     precedes the diff.  (Tools which harvest patches from mboxes are
>     trained to pick the changelog up from before the diff.)
>   - The MUA rewrapped some lines.
>   - File name and date of last change are redundant information and are
>     better left out of the source files.
>   - Understandably for a port from other kernels, there are clashes with
>     Linux kernel's coding style like CamelCase names, comment style,
>     indentations.
>   - Why define LONGLONG, WORD, BYTE and so on?  They could be plain
>     unsigned char etc., or u8 etc. if you like it brief.
>   - Boolean values should be the standard true and false, not locally
>     defined TRUE and FALSE.
>   - Usually the #include's are not collected in an intermediary header
>     (as in patch 7/25) but put directly into the files which require
>     a particular #include.
>
> I haven't looked in detail at the patches; it's far out of my area of
> experience...
> --
> Stefan Richter
> -=====-==--- =--- --=--
> http://arcgraph.de/sr/
>


--
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/