From: | Qi Huang <huangqiyx(at)outlook(dot)com> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Git diff patch in context diff format |
Date: | 2012-08-09 02:44:47 |
Message-ID: | BAY002-W2037E664CC4D2024C29B736B0CC0@phx.gbl |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> Date: Wed, 8 Aug 2012 15:05:06 -0400
> From: andrew(at)dunslane(dot)net
> To: bruce(at)momjian(dot)us
> CC: huangqiyx(at)outlook(dot)com; pgsql-hackers(at)postgresql(dot)org
> Subject: Re: [HACKERS] Git diff patch in context diff format
>
>
> On 08/08/2012 01:29 PM, Bruce Momjian wrote:
> > On Thu, Aug 2, 2012 at 05:03:04PM +0800, Qi Huang wrote:
> >> Hi, hackers
> >> I was exporting my project to a patch file. As the patch review requires,
> >> the patch needs to be in context diff format (http://wiki.postgresql.org/wiki/
> >> Reviewing_a_Patch). But the git diff exports in a format similar to unified
> >> format. What is everyone doing with patching now? Is there any standard way?
> > Have you read our wiki about git and diffs?
> >
> > http://wiki.postgresql.org/wiki/Working_with_Git#Context_diffs_with_Git
>
>
>
> I must confess that, like Robert, I just use:
>
> git diff | filterdiff --format=context
>
> I tried the git config stuff mentioned on the wiki, and it bit me a few
> times, I forget exactly how, and this just works without hassle.
>
> cheers
>
> andrew
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
I'm sorry, but actually this issue has already been resolved. I don't why the system sent another mail... Thanks for your response.
Best RegardsHuang Qi VictorComputer Science of National University of Singapore
From | Date | Subject | |
---|---|---|---|
Next Message | Jesper Krogh | 2012-08-09 06:59:52 | Re: Inserting heap tuples in bulk in COPY |
Previous Message | Simon Riggs | 2012-08-08 23:08:17 | Re: Prevent restored WAL files from being archived again Re: Unnecessary WAL archiving after failover |