Re: [HACKERS] How to submit a patch. - Mailing list pgsql-hackers

From Marc G. Fournier
Subject Re: [HACKERS] How to submit a patch.
Date
Msg-id 9f8979f689453c9f01d6419b7469eb3d
Whole thread Raw
In response to [HACKERS] How to submit a patch.  (David Friend <dfriend@atlsci.atlsci.com>)
List pgsql-hackers
On Mon, 2 Jun 1997, David Friend wrote:

> On Sun, 1 Jun 1997, Sean Lyndersay wrote:
>
> > When running regression tests on my system (DEC Alpha running OSF1 v4.0
> > Release 564), several tests fail. Some key ones include the int2 and int4,
> > but these are caused by changes in the error messages for certain tests.
>
> This can't be helped.  The error messages differ on different machines.
> The tests that "fail" on your machine pass on mine and many others.  I
> would suggest you don't submit patches to change these.
>
> > I changed the expected files, but came up against another problem: simply
> > put, what is the preferred method for submitting a patch?
>
> Here is how I do it.  (I got this from Thomas Lockhart.):
>
>   1) Create directory /usr/src/pgsql/patches, e.g.
>        mkdir /usr/src/pgsql/patches
>
>   2) Make a backup of the files you will change, e.g.
>        cd /usr/src/pgsql/src/man
>        cp psql.1 psql.1.orig
>
>   3) Edit the files, e.g.
>        vi psql.1
>
>   4) Make the patch by using diff from the patch directory, e.g.
>        cd /usr/src/pgsql/patches
>        diff -c ../src/man/psql.1.orig ../src/man/psql.1 > psql.1.patch

    So *that* is why I don't like your patches :)  That ../ screws me
up every time...never think to add the -p1 to patch when applying :)  At
least its not like some ppl that send me a patch to <insert obscure .c file
here> and expect me to know *where* in the source tree it is :(

    Why not just do:

    cd /usr/src/pgsql
    diff -c src/man/psql.1.orig src/man/psql.1 > /tmp/psql.1.patch


Marc G. Fournier                                 scrappy@hub.org
Systems Administrator @ hub.org              scrappy@freebsd.org

------------------------------

pgsql-hackers by date:

Previous
From: proff@suburbia.net
Date:
Subject: Re: [HACKERS] Isn't Purify great? :)
Next
From: David Friend
Date:
Subject: Re: [HACKERS] Someone might already have done working versions