Avoid unnecessary "touch meson.build" in vpath builds
authorAndres Freund
Thu, 25 Apr 2024 14:51:33 +0000 (07:51 -0700)
committerAndres Freund
Thu, 25 Apr 2024 14:51:33 +0000 (07:51 -0700)
In e6927270cd1 I added a 'touch meson.build' to configure.ac, to ensure
conflicts between in-tree configure based builds and meson builds are
automatically detected. Unfortunately I omitted spaces around the condition
restricting this to in-tree builds, leading to touch meson.build to also be
executed in vpath builds. While the only consequence of this buglet is an
unnecessary empty file in build directories, it seems worth backpatching.

Reported-by: Christoph Berg
Discussion: https://postgr.es/m/20240417230002[email protected]
Backpatch: 16-, where the meson based build was added

configure
configure.ac

index 375a8b7dbc45189cdab3dee13a467d691b736a4a..0d29e35c20f3fc2c47f7a5527eaae29ef16c6627 100755 (executable)
--- a/configure
+++ b/configure
@@ -20827,6 +20827,6 @@ fi
 
 # Ensure that any meson build directories would reconfigure and see that
 # there's a conflicting in-tree build and can error out.
-if test "$vpath_build"="no"; then
+if test "$vpath_build" = "no"; then
   touch meson.build
 fi
index aed940410c3fc32ba3b603da94d18850c6074fdf..ee4e3e2b664ee084883d6e36cde4744581fdda65 100644 (file)
@@ -2500,6 +2500,6 @@ AC_OUTPUT
 
 # Ensure that any meson build directories would reconfigure and see that
 # there's a conflicting in-tree build and can error out.
-if test "$vpath_build"="no"; then
+if test "$vpath_build" = "no"; then
   touch meson.build
 fi