add CopyBoth ExecStatus

[dgvncsz0f]

the reason I'm adding this is to handle responses from
START_REPLICATION command. I believe this is the only case
CopyBoth is ever needed.

Let me know if this is ok. i will gladly change anything you find
necessary. thanks!

[lpsmith]

Hmm, my main concern here is that this would then require a recent version of libpq to compile, because otherwise PGRES_COPY_BOTH won't be defined in the libpq headers, resulting in a compile error.

One possibility might be to hard-code the constant into postgresql-libpq, and/or use some CPP to work around the issue.

[dgvncsz0f]

fair enough! I'll work on a patch and will let you know!

[dgvncsz0f]

@lpsmith

i was reading the documentation to check when COPY_BOTH first show up. at least on the documentation, it is 9.1.x. this seems to happen before than PGRES_SINGLE_TUPLE, which the lib already depends on. the later only shows up in 9.2.x.

https://www.postgresql.org/docs/9.1/static/libpq-exec.html
https://www.postgresql.org/docs/9.2/static/libpq-exec.html

my guess is that it is safe to include it, as the minimum version is already set to 9.2.x. what you think?

[lpsmith]

Good call, yeah I thought this was added subsequent to libpq-9.2. So this looks good to me. Thanks!

[lpsmith]

Uploaded to hackage as postgresql-libpq-0.9.3.0. Take care!