pgsql: Remove distprep

Поиск
Список
Период
Сортировка
От Peter Eisentraut
Тема pgsql: Remove distprep
Дата
Msg-id E1r019a-004Bqr-Bf@gemulon.postgresql.org
обсуждение исходный текст
Список pgsql-committers
Remove distprep

A PostgreSQL release tarball contains a number of prebuilt files, in
particular files produced by bison, flex, perl, and well as html and
man documentation.  We have done this consistent with established
practice at the time to not require these tools for building from a
tarball.  Some of these tools were hard to get, or get the right
version of, from time to time, and shipping the prebuilt output was a
convenience to users.

Now this has at least two problems:

One, we have to make the build system(s) work in two modes: Building
from a git checkout and building from a tarball.  This is pretty
complicated, but it works so far for autoconf/make.  It does not
currently work for meson; you can currently only build with meson from
a git checkout.  Making meson builds work from a tarball seems very
difficult or impossible.  One particular problem is that since meson
requires a separate build directory, we cannot make the build update
files like gram.h in the source tree.  So if you were to build from a
tarball and update gram.y, you will have a gram.h in the source tree
and one in the build tree, but the way things work is that the
compiler will always use the one in the source tree.  So you cannot,
for example, make any gram.y changes when building from a tarball.
This seems impossible to fix in a non-horrible way.

Second, there is increased interest nowadays in precisely tracking the
origin of software.  We can reasonably track contributions into the
git tree, and users can reasonably track the path from a tarball to
packages and downloads and installs.  But what happens between the git
tree and the tarball is obscure and in some cases non-reproducible.

The solution for both of these issues is to get rid of the step that
adds prebuilt files to the tarball.  The tarball now only contains
what is in the git tree (*).  Getting the additional build
dependencies is no longer a problem nowadays, and the complications to
keep these dual build modes working are significant.  And of course we
want to get the meson build system working universally.

This commit removes the make distprep target altogether.  The make
dist target continues to do its job, it just doesn't call distprep
anymore.

(*) - The tarball also contains the INSTALL file that is built at make
dist time, but not by distprep.  This is unchanged for now.

The make maintainer-clean target, whose job it is to remove the
prebuilt files in addition to what make distclean does, is now just an
alias to make distprep.  (In practice, it is probably obsolete given
that git clean is available.)

The following programs are now hard build requirements in configure
(they were already required by meson.build):

- bison
- flex
- perl

Reviewed-by: Michael Paquier <michael@paquier.xyz>
Reviewed-by: Andres Freund <andres@anarazel.de>
Discussion: https://www.postgresql.org/message-id/flat/e07408d9-e5f2-d9fd-5672-f53354e9305e@eisentraut.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/721856ff24b3722ce8e894e5a32c9c063cd48455

Modified Files
--------------
GNUmakefile.in                                    |  6 +-
config/perl.m4                                    |  9 +--
config/programs.m4                                | 21 ++----
configure                                         | 62 +++--------------
contrib/cube/Makefile                             |  7 +-
contrib/fuzzystrmatch/Makefile                    |  9 +--
contrib/seg/Makefile                              |  7 +-
doc/Makefile                                      |  2 +-
doc/src/Makefile                                  |  2 +-
doc/src/sgml/Makefile                             | 16 ++---
doc/src/sgml/installation.sgml                    | 83 ++++++++++-------------
doc/src/sgml/sourcerepo.sgml                      | 10 ---
meson.build                                       |  2 +-
src/Makefile                                      |  5 +-
src/Makefile.global.in                            | 30 +++-----
src/backend/Makefile                              | 62 ++++-------------
src/backend/bootstrap/Makefile                    |  6 +-
src/backend/catalog/Makefile                      | 14 +---
src/backend/jit/llvm/Makefile                     |  2 +-
src/backend/nls.mk                                |  2 +-
src/backend/nodes/Makefile                        | 12 ++--
src/backend/parser/Makefile                       |  8 +--
src/backend/port/Makefile                         |  2 +-
src/backend/replication/Makefile                  | 10 ++-
src/backend/replication/libpqwalreceiver/Makefile |  2 +-
src/backend/replication/pgoutput/Makefile         |  2 +-
src/backend/snowball/Makefile                     |  4 --
src/backend/storage/lmgr/Makefile                 |  4 +-
src/backend/utils/Makefile                        | 28 +++-----
src/backend/utils/activity/Makefile               |  2 +-
src/backend/utils/adt/Makefile                    |  5 +-
src/backend/utils/mb/Makefile                     |  2 +-
src/backend/utils/mb/Unicode/Makefile             |  2 -
src/backend/utils/mb/conversion_procs/proc.mk     |  2 +-
src/backend/utils/misc/Makefile                   |  3 +-
src/bin/initdb/Makefile                           |  2 +-
src/bin/pg_amcheck/Makefile                       |  2 +-
src/bin/pg_archivecleanup/Makefile                |  2 +-
src/bin/pg_basebackup/Makefile                    |  2 +-
src/bin/pg_checksums/Makefile                     |  2 +-
src/bin/pg_config/Makefile                        |  2 +-
src/bin/pg_controldata/Makefile                   |  2 +-
src/bin/pg_ctl/Makefile                           |  2 +-
src/bin/pg_dump/Makefile                          |  2 +-
src/bin/pg_resetwal/Makefile                      |  2 +-
src/bin/pg_rewind/Makefile                        |  2 +-
src/bin/pg_test_fsync/Makefile                    |  2 +-
src/bin/pg_test_timing/Makefile                   |  2 +-
src/bin/pg_upgrade/Makefile                       |  2 +-
src/bin/pg_verifybackup/Makefile                  |  2 +-
src/bin/pg_waldump/Makefile                       |  2 +-
src/bin/pgbench/Makefile                          |  4 --
src/bin/psql/Makefile                             |  6 --
src/bin/scripts/Makefile                          |  2 +-
src/common/Makefile                               |  5 --
src/common/unicode/Makefile                       |  2 -
src/fe_utils/Makefile                             |  6 --
src/include/Makefile                              |  2 +-
src/interfaces/ecpg/Makefile                      |  2 +-
src/interfaces/ecpg/compatlib/Makefile            |  2 -
src/interfaces/ecpg/ecpglib/Makefile              |  2 -
src/interfaces/ecpg/include/Makefile              |  2 +-
src/interfaces/ecpg/pgtypeslib/Makefile           |  2 -
src/interfaces/ecpg/preproc/Makefile              |  6 --
src/interfaces/ecpg/test/Makefile                 |  4 +-
src/interfaces/libpq/Makefile                     |  3 -
src/interfaces/libpq/test/Makefile                |  2 +-
src/makefiles/pgxs.mk                             |  2 +-
src/nls-global.mk                                 |  2 +-
src/pl/plperl/GNUmakefile                         |  2 +-
src/pl/plpgsql/src/Makefile                       |  7 --
src/pl/plpython/Makefile                          |  6 +-
src/pl/tcl/Makefile                               |  5 --
src/port/Makefile                                 |  2 +-
src/test/authentication/Makefile                  |  2 +-
src/test/examples/Makefile                        |  2 +-
src/test/icu/Makefile                             |  2 +-
src/test/isolation/Makefile                       |  6 --
src/test/kerberos/Makefile                        |  2 +-
src/test/ldap/Makefile                            |  2 +-
src/test/locale/Makefile                          |  2 +-
src/test/recovery/Makefile                        |  2 +-
src/test/regress/GNUmakefile                      |  2 +-
src/test/ssl/Makefile                             |  2 +-
src/test/ssl/sslfiles.mk                          |  4 +-
src/test/subscription/Makefile                    |  2 +-
src/timezone/Makefile                             |  2 +-
src/tools/ifaddrs/Makefile                        |  2 +-
src/tools/pg_bsd_indent/Makefile                  |  2 +-
89 files changed, 175 insertions(+), 416 deletions(-)


В списке pgsql-committers по дате отправления:

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: Re: pgsql: Migrate logical slots to the new node during an upgrade.
Следующее
От: Tom Lane
Дата:
Сообщение: pgsql: Compute aggregate argument types correctly in transformAggregate