public inbox for [email protected]  

help / color / mirror / Atom feed

From: Michael Meskes <[email protected]>
To: Tom Lane <[email protected]>
To: Ashutosh Sharma <[email protected]>
Cc: [email protected]
Subject: Re: ecpg: issue related to preprocessor directives
Date: Mon, 03 Aug 2020 13:16:48 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <CAE9k0P=dQk9X0cU2tN49S7a9tv733-e1pVdpB1P-pWJ5PdTktg@mail.gmail.com>
	<[email protected]>
	<[email protected]>

> Here's a proposed patch, which also clarifies the documentation,
> which seemed a bit confused/misleading to me.

Thanks Tom.

> As stated, I'm not sure it's wise to back-patch this aggressively
> ... but maybe it'd be okay to squeeze it into v13?

Fully agreed, back-patching sounds like a pretty risky approach. I
guess almost everyone who ran into this, fixed their code to work
correctly by now. I'd also prefer squeezing it into 13, let's not keep
this around for longer.

Michael
-- 
Michael Meskes
Michael at Fam-Meskes dot De
Michael at Meskes dot (De|Com|Net|Org)
Meskes at (Debian|Postgresql) dot Org

view thread (7+ messages)  latest in thread

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], [email protected]
  Subject: Re: ecpg: issue related to preprocessor directives
  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