From: | Ryan Murphy <ryanfmurphy(at)gmail(dot)com> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Cc: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Subject: | Re: proposal: psql \setfileref |
Date: | 2016-09-26 12:57:42 |
Message-ID: | 20160926125742.1310.38255.pgcf@coridan.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Pavel,
I just tried to apply your patch psql-setfileref-initial.patch (using git apply) to the newest revision of postgres at the time (da6c4f6ca88) and it failed to patch startup.c. Thinking that the patch was for some previous revision, I then checked out d062245b5, which was from 2016-08-31, and tried applying the patch from there. I had the same problem:
$ git apply psql-setfileref-initial.patch
error: patch failed: src/bin/psql/startup.c:106
error: src/bin/psql/startup.c: patch does not apply
However, when I applied the changes to startup.c manually and removed them from the patch, the rest of the patch applied without a problem. I don't know if I may have done something wrong in trying to apply the patch, but you may want to double check if you need to regenerate your patch from the latest revision so it will apply smoothly for reviewers.
In the meantime, I haven't had a chance to try out the fileref feature yet but I'll give it a go when I get a chance!
Thanks!
Ryan
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2016-09-26 13:27:31 | Re: Allowing GIN array_ops to work on anyarray |
Previous Message | Tom Lane | 2016-09-26 12:53:51 | Re: pgsql: pg_ctl: Detect current standby state from pg_control |