On Wed, Mar 30, 2022 at 12:30 PM Andres Freund <and...@anarazel.de> wrote:
> # Reconfigure to restrict access and require a detail.
> $shell_command =
>         $PostgreSQL::Test::Utils::windows_os
>         ? qq{$gzip --fast > "$escaped_backup_path\\\\%d.%f.gz"}
>     : qq{$gzip --fast > "$escaped_backup_path/%d.%f.gz"};
>
> I don't think the branch is needed anymore, forward slashes should work for
> output redirection.

We have similar things in src/test/perl/PostgreSQL/Test/Cluster.pm. Do
you think those can also be removed? I'm not sure it's the place of
this patch to introduce a mix of styles.

-- 
Robert Haas
EDB: http://www.enterprisedb.com


Reply via email to