Re: Git diff patch in context diff format

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

In response to

Browse pgsql-hackers by date

  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