Re: git diff script is not portable - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: git diff script is not portable
Date
Msg-id 201103102313.p2ANDHb20199@momjian.us
Whole thread Raw
In response to Re: git diff script is not portable  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
Bruce Momjian wrote:
> Magnus Hagander wrote:
> > On Wed, Nov 17, 2010 at 19:54, Bruce Momjian <bruce@momjian.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@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Can't specify default collation?
Next
From: Bruce Momjian
Date:
Subject: Re: Indent authentication overloading