Thread: Git diff patch in context diff format
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?
Thanks.
Best Regards
Best Regards
Huang Qi Victor
Computer Science of National University of Singapore
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 -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + It's impossible for everything to be true. +
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
> Date: Wed, 8 Aug 2012 15:05:06 -0400
> From: andrew@dunslane.net
> To: bruce@momjian.us
> CC: huangqiyx@outlook.com; pgsql-hackers@postgresql.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@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
> From: andrew@dunslane.net
> To: bruce@momjian.us
> CC: huangqiyx@outlook.com; pgsql-hackers@postgresql.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@postgresql.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 Regards
Best Regards
Huang Qi Victor
Computer Science of National University of Singapore