On Sun, Oct 2, 2011 at 4:08 PM, Dimitri Fontaine <dimi...@2ndquadrant.fr> wrote: > Ok I needed `git apply' to apply the patches, now that I used that I can > confirm that the 3 patches apply, compile, pass tests, and that I could > play with them a little. I think I'm going to mark that ready for > commiter. I don't have enough time for a more deep review but at the > same time patch reading and testing both passed :) > > You might need to post a version that patch will be happy with, though, > using e.g. > > git diff |filterdiff --format=context > /tmp/foo.patch > > Then diffstat reports: > 35 files changed, 932 insertions(+), 1913 deletions(-), 345 modifications(!)
I think that new versions of patch can handle unified diffs without a problem, but older versions choke on them. My Mac has 2.5.8 and handles unidiffs no problem. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers