Re: [HACKERS] Small patch for pg_basebackup argument parsing - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: [HACKERS] Small patch for pg_basebackup argument parsing
Date
Msg-id CAB7nPqRSNHVPYGbhwEAGj5R1xrfE=qY2MLY9gNFQwRMmz7Ob0w@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] Small patch for pg_basebackup argument parsing  (Ryan Murphy <ryanfmurphy@gmail.com>)
Responses Re: [HACKERS] Small patch for pg_basebackup argument parsing
List pgsql-hackers
On Wed, Jul 5, 2017 at 2:57 PM, Ryan Murphy <ryanfmurphy@gmail.com> wrote:
> I tried to apply your patch to test it (though reading Robert's last comment it seems we wish to have it adjusted
beforecommitting)... but in any case I was not able to apply your patch to the tip of the master branch (my git apply
failed). I'm setting this to Waiting On Author for a new patch, and I also agree with Robert that the test can be
simplerand can go in the other order.  If you don't have time to make another patch, let me know, I may be able to make
one.

git is unhappy even if forcibly applied with patch -p1. You should
check for whitespaces at the same time:
$ git diff --check
src/bin/pg_basebackup/pg_receivewal.c:483: indent with spaces.
+    char       *strtol_endptr = NULL
And there are more than this one.
--
Michael



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [HACKERS] Extra Vietnamese unaccent rules
Next
From: Michael Paquier
Date:
Subject: Re: [HACKERS] pg_stop_backup(wait_for_archive := true) on standby server