Re: [evolution-patches] Patch submission

[Jeffrey Stedfast <fejj novell com>]

What I meant was that a patch file should contain both the diff for the
source code changes and the changelog. Not that you should send the
changelog separately. 

On Sat, 2005-07-16 at 20:06 -0700, riyaz ahmed wrote:
> On 7/16/05, Jeffrey Stedfast <fejj novell com> wrote:
> > please include a changelog in each patch
> > 
> > On Sat, 2005-07-16 at 02:36 -0700, riyaz ahmed wrote:
> > >
> > >
> > > On 7/13/05, Jeffrey Stedfast <fejj novell com> wrote:
> > >         it would be nice if you made a single diff and added the
> > >         changelog
> > >         entries to the proper changelog file.
> > >
> > >         If you used emacs, it would auto-magically find the proper
> > >         ChangeLog
> > >         file for you and properly add the filename to the ChangeLog
> > >
> > >         eg, the ChangeLog entry should look more like this:
> > >
> > >         2005-07-13  H.A.Riyaz Ahmed  <sunez riyaz gmail com>
> > >
> > >                 Fixes bug #246244 and bug #246247
> > >
> > >                 * src/menubar.c (function_modified): changed the value
> > >         of the argument
> > >
> > >                 * src/propeties.c (function_modified): added the
> > >         condition statement
> > >
> > >                 * src/body.c (function_modified): changed the argument
> > >         as required
> > >
> > >                 * src/utils.c (function_modified): changed the display
> > >         as mentioned
> > >
> > >
> > >         (Note: I'm assuming the files are in gtkhtml/src/ but that
> > >         might not be
> > >         the case)
> > >
> > >         that said... you should also be a tad more descriptive in the
> > >         changelog.
> > >         For example, body.c's change says "changed the argument as
> > >         required".
> > >         That doesn't at all describe what you changed. Instead, a
> > >         better
> > >         description might be:
> > >
> > >         "changed the capitalisation of some words to conform to the
> > >         HIG"
> > >
> > >         your changes to properties.c does not follow the coding style.
> > >         Please
> > >         read the README.HACKING file and keep your coding style
> > >         consistant with
> > >         the surrounding code.
> > >
> > >         also... why are there multiple patches to the same files?
> > >         which diff is
> > >         the one we should be reviewing?
> > >
> > >
> > >         I suggest you make the following changes above (using `cvs
> > >         diff -up`
> > >         would be extremely useful as well) and then resend the single
> > >         resulting
> > >         patch.
> > >
> > >    Hi ....
> > > I have remodified the patch as you have mentioned
> > > ithink that this will be fine
> > >  Truly
> > >  H.A.Riyaz Ahmed
> > >
> > >
> > >
> > >
> > >         Jeff
> > >
> > >         On Wed, 2005-07-13 at 17:49 +0530, riyaz ahmed wrote:
> > >         > hi,
> > >         >
> > >         >    i have attached the patches for  the bugs 246244( three
> > >         files ) ,
> > >         > 246247( three files ) and a
> > >         > changelog file
> > >         > these fixes the label and HIG-layout suggestions ,
> > >         >
> > >         >
> > >         > Truly
> > >         > H.A.Riyaz Ahmed
> > >         > _______________________________________________
> > >         > evolution-patches mailing list
> > >         > evolution-patches lists ximian com
> > >         > http://lists.ximian.com/mailman/listinfo/evolution-patches
> > >         --
> > >         Jeffrey Stedfast
> > >         Evolution Hacker - Novell, Inc.
> > >         fejj ximian com  - www.novell.com
> > >
> > >
> > 
> Hi..
> I have attached the bugs seperately and the patches for the bug
> seperately as you have specified