Meson build updates
Tristan Partin <tristan@neon.tech>
Attachments
- v1-0001-Remove-triple-quoted-strings.patch (text/x-patch) patch v1-0001
- v1-0002-Use-consistent-casing-in-Meson-option-description.patch (text/x-patch) patch v1-0002
- v1-0003-Use-consistent-Meson-option-description-formats.patch (text/x-patch) patch v1-0003
- v1-0004-Attach-colon-to-keyword-argument.patch (text/x-patch) patch v1-0004
- v1-0005-Use-the-not_found_dep-constant.patch (text/x-patch) patch v1-0005
- v1-0006-Remove-old-comment.patch (text/x-patch) patch v1-0006
- v1-0007-Tie-adding-C-support-to-the-llvm-Meson-option.patch (text/x-patch) patch v1-0007
- v1-0008-Mention-the-correct-way-to-disable-readline-suppo.patch (text/x-patch) patch v1-0008
- v1-0009-Remove-return-code-check.patch (text/x-patch) patch v1-0009
- v1-0010-Fix-some-grammar-usage-in-Meson-comments.patch (text/x-patch) patch v1-0010
- v1-0011-Pass-feature-option-through-to-required-kwarg.patch (text/x-patch) patch v1-0011
- v1-0012-Make-finding-pkg-config-python3-more-robust.patch (text/x-patch) patch v1-0012
- v1-0013-Make-some-Meson-style-more-consistent-with-surrou.patch (text/x-patch) patch v1-0013
- v1-0014-Reduce-branching-on-Meson-version.patch (text/x-patch) patch v1-0014
- v1-0015-Use-a-better-error-message-in-an-impossible-case.patch (text/x-patch) patch v1-0015
- v1-0016-Add-Meson-overrides.patch (text/x-patch) patch v1-0016
- v1-0017-Remove-Meson-program-options-for-specifying-paths.patch (text/x-patch) patch v1-0017
Hi, I took some time to look at the Meson build for Postgres. I contribute some of the time to Meson, itself. Within this patchset you will find pretty small changes. Most of the commits are attempting to create more consistency with the surrounding code. I think there is more that can be done to improve the build a bit like including subprojects for optional dependencies like lz4, zstd, etc. While I was reading through the code, I also noticed a few XXX/FIXMEs. I don't mind taking a look at those in the future, but I think I need more context for almost all of them since I am brand new to Postgres development. Since I also have _some_ sway in the Meson community, I can help get more attention to Meson issues that benefit Postgres. I currently note 3 Meson issues that are commented in the code for instance. If anyone ever has any problems or questions about Meson or specifically the Meson build of Postgres, I am more than happy to receive an email to see if I can help out. Highlighting the biggest changes in this patchset: - Add Meson overrides - Remove Meson program options for specifying paths Everything but the last patch could most likely be backported to previous maintained releases including Meson, though the patchset is initially targeting the master branch. -- Tristan Partin Neon (https://neon.tech)
Commits
-
meson: Make detection of python more robust
- c79ca0485a62 16.1 landed
- 41da94fd5c74 17.0 landed
-
meson: Pass more feature option through to required kwargs
- 5a7280d9880f 16.0 landed
- 47caa710cebe 17.0 landed
-
meson: Tie adding C++ support to the llvm Meson option
- 7fc064d9b70f 16.0 landed
- 7a7f13afee92 17.0 landed
-
meson: Fix intl misspelling
- b750e74e6ec3 16.0 landed
-
meson: Use a better error message in an impossible case
- 99e57663518b 16.0 landed
-
meson: Make some Meson style more consistent with surrounding code
- af492eb6d6c9 16.0 landed
-
meson: Fix some grammar usage in Meson comments
- 74668cbf0fcc 16.0 landed
-
meson: Mention the correct way to disable readline support
- f5159c734fe4 16.0 landed
-
meson: Remove old comment
- 45c88eecb4a6 16.0 landed
-
meson: Use the not_found_dep constant
- 38977dd2bdac 16.0 landed
-
meson: Attach colon to keyword argument
- c55d31e59c89 16.0 landed
-
meson: Use consistent Meson option description formats
- b2ec7f543f9e 16.0 landed
-
meson: Use consistent casing in Meson option descriptions
- 529449c379b9 16.0 landed
-
meson: Remove triple-quoted strings
- 559bdd58eb71 16.0 landed