From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Magnus Hagander <magnus(at)hagander(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: git diff script is not portable |
Date: | 2011-03-10 23:13:17 |
Message-ID: | 201103102313.p2ANDHb20199@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Bruce Momjian wrote:
> Magnus Hagander wrote:
> > On Wed, Nov 17, 2010 at 19:54, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> > > Looking here:
> > >
> > > ? ? ? ?http://wiki.postgresql.org/wiki/Working_with_Git
> > >
> > > the script we recommend for doing context diffs for git,
> > > git-external-diff, is not portable:
> > >
> > > ? ? ? ?http://anarazel.de/pg/git-external-diff
> > >
> > > It uses diff -L, which is not supported by FreeBSD, and I imagine many
> > > other operating systems.
> > >
> > > If we want people to use this to produce context diffs, we should
> > > provide a portable script. ?I can modify it to be portable, but it is
> > > currently hosted on some other site. ?How should I handle this? ? ?How
> > > do I contact the author, or perhaps I should create a new on on
> > > ftp.postgresql.org and link to that.
> >
> > Do it the git way - fork it and put it on your github page.
> >
> > Or do it the pg way - fork it and put it in src/tools.
>
> src/tools is a very good idea. Objections?
Done, and linked to from our git wiki page:
http://wiki.postgresql.org/wiki/Working_with_Git#Context_diffs_with_Git
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2011-03-10 23:15:01 | Re: Indent authentication overloading |
Previous Message | Tom Lane | 2011-03-10 23:12:00 | Can't specify default collation? |