On Wed, Mar 30, 2022 at 8:30 AM Thomas Munro <thomas.mu...@gmail.com> wrote: > On Wed, Mar 30, 2022 at 3:08 AM Robert Haas <robertmh...@gmail.com> wrote: > > Here's a new version, hopefully rectifying that deficiency. I also add > > a second patch here, documenting basebackup_to_shell.required_role, > > because Joe Conway observed elsewhere that I forgot to do that. > > Here are your patches again, plus that kludge to make the CI run your > TAP test on Windows.
It failed: https://cirrus-ci.com/task/5567070686412800 https://api.cirrus-ci.com/v1/artifact/task/5567070686412800/log/contrib/basebackup_to_shell/tmp_check/log/001_basic_primary.log https://api.cirrus-ci.com/v1/artifact/task/5567070686412800/log/contrib/basebackup_to_shell/tmp_check/log/regress_log_001_basic