public inbox for [email protected]  
help / color / mirror / Atom feed
From: Michael Paquier <[email protected]>
To: Japin Li <[email protected]>
Cc: Pg Hackers <[email protected]>
Subject: Re: test_escape: invalid option -- 'c'
Date: Tue, 18 Feb 2025 12:46:50 +0900
Message-ID: <[email protected]> (raw)
In-Reply-To: <ME0P300MB04451FB20CE0346A59C25CADB6FA2@ME0P300MB0445.AUSP300.PROD.OUTLOOK.COM>
References: <ME0P300MB04451FB20CE0346A59C25CADB6FA2@ME0P300MB0445.AUSP300.PROD.OUTLOOK.COM>

On Tue, Feb 18, 2025 at 11:12:13AM +0800, Japin Li wrote:
> When I test test_escape, I find the usage of test_escape declares it
> support -c option, however, when I use it, it complains it is an
> invalid option.
> 
> Here is a patch to fix it.

You are right, obviously.  The only reason why this is not showing up
is that the TAP test uses the long option --conninfo.  Speaking of
which, this had better use fat commas to combine the option/value
pairs in these commands.
--
Michael


Attachments:

  [application/pgp-signature] signature.asc (833B, ../[email protected]/2-signature.asc)
  download

view thread (2+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: test_escape: invalid option -- 'c'
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox