Horton

Thread

Commits

  1. doc: Mention environment variable ZSTD in the TAP tests for MSVC

  2. Add support for building with ZSTD.

  1. pgsql: Add support for building with ZSTD.

    Robert Haas <rhaas@postgresql.org> — 2022-02-18T18:53:10Z

    Add support for building with ZSTD.

This commit doesn't actually add anything that uses ZSTD; that will be
done separately. It just puts the basic infrastructure into place.

Jeevan Ladhe, Robert Haas, and Michael Paquier. Reviewed by Justin
Pryzby and Andres Freund.

Discussion: http://postgr.es/m/CA+TgmoatQKGd+8SjcV+bzvw4XaoEwminHjU83yG12+NXtQzTTQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/6c417bbcc8ff98875234ca269979fc7defde58e5

Modified Files
--------------
configure                         | 265 ++++++++++++++++++++++++++++++++++++++
configure.ac                      |  33 +++++
doc/src/sgml/install-windows.sgml |   9 ++
doc/src/sgml/installation.sgml    |   9 ++
src/Makefile.global.in            |   1 +
src/include/pg_config.h.in        |   6 +
src/tools/msvc/Solution.pm        |  13 ++
src/tools/msvc/config_default.pl  |   1 +
src/tools/msvc/vcregress.pl       |   1 +
9 files changed, 338 insertions(+)

  2. Re: pgsql: Add support for building with ZSTD.

    Michael Paquier <michael@paquier.xyz> — 2022-02-19T06:18:57Z

    Hi,

On Fri, Feb 18, 2022 at 06:53:10PM +0000, Robert Haas wrote:
> Add support for building with ZSTD.
> 
> This commit doesn't actually add anything that uses ZSTD; that will be
> done separately. It just puts the basic infrastructure into place.
> 
> Jeevan Ladhe, Robert Haas, and Michael Paquier. Reviewed by Justin
> Pryzby and Andres Freund.

I completely forgot that the section of the docs dedicated to the TAP
tests with MSVC also needs a refresh to mention the new variable ZSTD,
as of the attached.  Do you mind if I apply that?

Thanks,
--
Michael

  3. Re: pgsql: Add support for building with ZSTD.

    Robert Haas <robertmhaas@gmail.com> — 2022-02-20T14:45:05Z

    On Sat, Feb 19, 2022 at 1:19 AM Michael Paquier <michael@paquier.xyz> wrote:
> On Fri, Feb 18, 2022 at 06:53:10PM +0000, Robert Haas wrote:
> > Add support for building with ZSTD.
> >
> > This commit doesn't actually add anything that uses ZSTD; that will be
> > done separately. It just puts the basic infrastructure into place.
> >
> > Jeevan Ladhe, Robert Haas, and Michael Paquier. Reviewed by Justin
> > Pryzby and Andres Freund.
>
> I completely forgot that the section of the docs dedicated to the TAP
> tests with MSVC also needs a refresh to mention the new variable ZSTD,
> as of the attached.  Do you mind if I apply that?

No issues at all with you adjusting this, but I think that sentence
reads a little awkwardly. Perhaps instead of "The default is
<literal>zstd</literal>, that would be the command found in
<varname>PATH</varname>." you could write something like "The default
is <literal>zstd</literal>, which will search for a command by that
name in the configured <varname>PATH</varname>." Or maybe something
else is better, not sure exactly, your version just seems a little odd
to me.

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

  4. Re: pgsql: Add support for building with ZSTD.

    Michael Paquier <michael@paquier.xyz> — 2022-02-21T01:02:58Z

    On Sun, Feb 20, 2022 at 09:45:05AM -0500, Robert Haas wrote:
> No issues at all with you adjusting this, but I think that sentence
> reads a little awkwardly.

Thanks.

> Perhaps instead of "The default is
> <literal>zstd</literal>, that would be the command found in
> <varname>PATH</varname>." you could write something like "The default
> is <literal>zstd</literal>, which will search for a command by that
> name in the configured <varname>PATH</varname>." Or maybe something
> else is better, not sure exactly, your version just seems a little odd
> to me.

Okay, done then.  We've been using the same wording for all the other
variables, and what you are suggesting here sounds much better to me,
so I have adjusted all the descriptions this way, and added the ZSTD
part.
--
Michael