From: Tom Lane <tgl@sss.pgh.pa.us>
To: Fabien COELHO <coelho@cri.ensmp.fr>
cc: Robert Haas <robertmhaas@gmail.com>,
        Pavel Stehule <pavel.stehule@gmail.com>,
        Corey Huinker <corey.huinker@gmail.com>,
        Daniel Verite <daniel@manitou-mail.org>,
        PostgreSQL Developers <pgsql-hackers@postgresql.org>
Subject: Re: Variable substitution in psql backtick expansion
In-reply-to: <alpine.DEB.2.20.1708261740590.17521@lancre>
References: <bc15034d-b421-42ac-89b1-c3485fa1728b@manitou-mail.org>
 <alpine.DEB.2.20.1704020917220.4632@lancre>
 <CAFj8pRCVFp+68Tt29sZfn9tE28c2D7u_VzTcXrqjyCVBzAzRsA@mail.gmail.com>
 <alpine.DEB.2.20.1704021303300.4632@lancre>
 <CAFj8pRACH1M=egydH4dAAoo9rRVTkuYCaoyZQK2Yf4BTQuRM5w@mail.gmail.com>
 <CADkLM=dzsbgJdP7KAJnbihxU1XFnWZgit9ye5cWnvkjZXyPbGw@mail.gmail.com>
 <alpine.DEB.2.20.1704021805550.4632@lancre>
 <CADkLM=c+hm2rc0tkKgC-ZgrLttHT2KkfppE+BC-=i-xj+7V-TQ@mail.gmail.com>
 <3926.1491159242@sss.pgh.pa.us> <alpine.DEB.2.20.1704022151080.9265@lancre>
 <CAFj8pRDyaC+FaSxd92CLcuV+UFGhwY7NVVBhqFvxPkhvKNhyww@mail.gmail.com>
 <alpine.DEB.2.20.1705211737030.6053@lancre>
 <CA+TgmobT3rD6jaZMCs=PWH_khMu6Bb4TwX+r4tbTNA41hx32=w@mail.gmail.com>
 <13303.1503698976@sss.pgh.pa.us>
 <CA+TgmobXCcEM7uB8_g+tKfB-1gvEyp+Qmmre9bcXU9mgpmW38Q@mail.gmail.com>
 <14596.1503701034@sss.pgh.pa.us> <alpine.DEB.2.20.1708260747110.3627@lancre>
 <20786.1503760136@sss.pgh.pa.us> <alpine.DEB.2.20.1708261740590.17521@lancre>
Comments: In-reply-to Fabien COELHO <coelho@cri.ensmp.fr>
	message dated "Sat, 26 Aug 2017 18:40:05 +0200"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <24849.1503766425.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Sat, 26 Aug 2017 12:53:45 -0400
Message-ID: <24850.1503766425@sss.pgh.pa.us>
Precedence: bulk
Sender: pgsql-hackers-owner@postgresql.org

Fabien COELHO <coelho@cri.ensmp.fr> writes:
> So basically the only thing needed from Robert & you seems to change=20
> "11.0" to "11devel", which is fine with me.
> The attached v5 does that.

I think you are taking unreasonable shortcuts here:

+	SetVariable(pset.vars, "SERVER_VERSION_NAME", PQparameterStatus(pset.db, =
"server_version"));

The existing code in connection_warnings() does this:

            const char *server_version;

            /* Try to get full text form, might include "devel" etc */
            server_version =3D PQparameterStatus(pset.db, "server_version");
            /* Otherwise fall back on pset.sversion */
            if (!server_version)
            {
                formatPGVersionNumber(pset.sversion, true,
                                      sverbuf, sizeof(sverbuf));
                server_version =3D sverbuf;
            }

and I think you should duplicate that logic verbatim.  Now admittedly,
server_version has been available for a long time, so that this might
never matter in practice.  But we shouldn't be doing this one way
in one place and differently somewhere else.

			regards, tom lane


--=20
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers