git.postgresql.org Git - postgresql.git/commit

git projects / postgresql.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: b71dad2) | patch
Move logging.h and logging.c from src/fe_utils/ to src/common/.
2019年5月14日 18:19:49 +0000 (14:19 -0400)
2019年5月14日 18:20:10 +0000 (14:20 -0400)
commit fc9a62af3f87f4bec1e8c904ea99ae50f3c881ef
Move logging.h and logging.c from src/fe_utils/ to src/common/.

The original placement of this module in src/fe_utils/ is ill-considered,
because several src/common/ modules have dependencies on it, meaning that
libpgcommon and libpgfeutils now have mutual dependencies. That makes it
pointless to have distinct libraries at all. The intended design is that
libpgcommon is lower-level than libpgfeutils, so only dependencies from
the latter to the former are acceptable.

We already have the precedent that fe_memutils and a couple of other
modules in src/common/ are frontend-only, so it's not stretching anything
out of whack to treat logging.c as a frontend-only module in src/common/.
To the extent that such modules help provide a common frontend/backend
environment for the rest of common/ to use, it's a reasonable design.
(logging.c does not yet provide an ereport() emulation, but one can
dream.)

Hence, move these files over, and revert basically all of the build-system
changes made by commit cc8d41511. There are no places that need to grow
new dependencies on libpgcommon, further reinforcing the idea that this
is the right solution.

Discussion: https://postgr.es/m/a912ffff-f6e4-778a-c86a-cf5c47a12933@2ndquadrant.com
71 files changed:
src/bin/initdb/initdb.c diff | blob | blame | history
src/bin/pg_archivecleanup/Makefile diff | blob | blame | history
src/bin/pg_archivecleanup/pg_archivecleanup.c diff | blob | blame | history
src/bin/pg_basebackup/pg_basebackup.c diff | blob | blame | history
src/bin/pg_basebackup/pg_receivewal.c diff | blob | blame | history
src/bin/pg_basebackup/pg_recvlogical.c diff | blob | blame | history
src/bin/pg_basebackup/receivelog.c diff | blob | blame | history
src/bin/pg_basebackup/streamutil.c diff | blob | blame | history
src/bin/pg_checksums/Makefile diff | blob | blame | history
src/bin/pg_checksums/pg_checksums.c diff | blob | blame | history
src/bin/pg_controldata/Makefile diff | blob | blame | history
src/bin/pg_controldata/pg_controldata.c diff | blob | blame | history
src/bin/pg_ctl/Makefile diff | blob | blame | history
src/bin/pg_ctl/pg_ctl.c diff | blob | blame | history
src/bin/pg_dump/common.c diff | blob | blame | history
src/bin/pg_dump/pg_backup_archiver.c diff | blob | blame | history
src/bin/pg_dump/pg_backup_custom.c diff | blob | blame | history
src/bin/pg_dump/pg_backup_db.c diff | blob | blame | history
src/bin/pg_dump/pg_backup_utils.h diff | blob | blame | history
src/bin/pg_dump/pg_dump.c diff | blob | blame | history
src/bin/pg_dump/pg_dump_sort.c diff | blob | blame | history
src/bin/pg_dump/pg_dumpall.c diff | blob | blame | history
src/bin/pg_dump/pg_restore.c diff | blob | blame | history
src/bin/pg_resetwal/Makefile diff | blob | blame | history
src/bin/pg_resetwal/pg_resetwal.c diff | blob | blame | history
src/bin/pg_rewind/Makefile diff | blob | blame | history
src/bin/pg_rewind/datapagemap.c diff | blob | blame | history
src/bin/pg_rewind/pg_rewind.h diff | blob | blame | history
src/bin/pg_test_fsync/Makefile diff | blob | blame | history
src/bin/pg_test_fsync/pg_test_fsync.c diff | blob | blame | history
src/bin/pg_upgrade/pg_upgrade.c diff | blob | blame | history
src/bin/pg_waldump/Makefile diff | blob | blame | history
src/bin/pg_waldump/pg_waldump.c diff | blob | blame | history
src/bin/pgbench/pgbench.c diff | blob | blame | history
src/bin/psql/command.c diff | blob | blame | history
src/bin/psql/common.c diff | blob | blame | history
src/bin/psql/copy.c diff | blob | blame | history
src/bin/psql/crosstabview.c diff | blob | blame | history
src/bin/psql/describe.c diff | blob | blame | history
src/bin/psql/help.c diff | blob | blame | history
src/bin/psql/input.c diff | blob | blame | history
src/bin/psql/large_obj.c diff | blob | blame | history
src/bin/psql/mainloop.c diff | blob | blame | history
src/bin/psql/psqlscanslash.l diff | blob | blame | history
src/bin/psql/startup.c diff | blob | blame | history
src/bin/psql/variables.c diff | blob | blame | history
src/bin/scripts/clusterdb.c diff | blob | blame | history
src/bin/scripts/common.c diff | blob | blame | history
src/bin/scripts/createdb.c diff | blob | blame | history
src/bin/scripts/createuser.c diff | blob | blame | history
src/bin/scripts/dropdb.c diff | blob | blame | history
src/bin/scripts/dropuser.c diff | blob | blame | history
src/bin/scripts/pg_isready.c diff | blob | blame | history
src/bin/scripts/reindexdb.c diff | blob | blame | history
src/bin/scripts/vacuumdb.c diff | blob | blame | history
src/common/Makefile diff | blob | blame | history
src/common/controldata_utils.c diff | blob | blame | history
src/common/file_utils.c diff | blob | blame | history
src/common/logging.c [moved from src/fe_utils/logging.c with 92% similarity] diff | blob | blame | history
src/common/pgfnames.c diff | blob | blame | history
src/common/restricted_token.c diff | blob | blame | history
src/common/rmtree.c diff | blob | blame | history
src/fe_utils/Makefile diff | blob | blame | history
src/fe_utils/psqlscan.l diff | blob | blame | history
src/include/common/logging.h [moved from src/include/fe_utils/logging.h with 92% similarity] diff | blob | blame | history
src/interfaces/ecpg/test/Makefile diff | blob | blame | history
src/nls-global.mk diff | blob | blame | history
src/test/isolation/Makefile diff | blob | blame | history
src/test/regress/GNUmakefile diff | blob | blame | history
src/test/regress/pg_regress.c diff | blob | blame | history
src/tools/msvc/Mkvcbuild.pm diff | blob | blame | history
This is the main PostgreSQL git repository.
RSS Atom

AltStyle によって変換されたページ (->オリジナル) /