Don't define FRONTEND for initdb
authorAndres Freund <andres@anarazel.de>
Tue, 23 Aug 2022 03:39:30 +0000 (20:39 -0700)
committerAndres Freund <andres@anarazel.de>
Tue, 23 Aug 2022 03:39:30 +0000 (20:39 -0700)
No headers requiring FRONTED to be defined are included as of af1a949109d.

Since this is the last user of (contrib|frontend)_defines in Mkvcbuild.pm,
remove them.

Discussion: https://postgr.es/m/20220820194550.725755r6fj2ro3rx@awork3.anarazel.de

src/bin/initdb/Makefile
src/tools/msvc/Mkvcbuild.pm

index b0dd13dfbdfa7b74fcfef7b0215108f2971bf7ea..6737938c3f878577e851236f2c54b5d36c7e61cf 100644 (file)
@@ -16,7 +16,7 @@ subdir = src/bin/initdb
 top_builddir = ../../..
 include $(top_builddir)/src/Makefile.global
 
-override CPPFLAGS := -DFRONTEND -I$(libpq_srcdir) -I$(top_srcdir)/src/timezone $(CPPFLAGS)
+override CPPFLAGS := -I$(libpq_srcdir) -I$(top_srcdir)/src/timezone $(CPPFLAGS)
 
 # Note: it's important that we link to encnames.o from libpgcommon, not
 # from libpq, else we have risks of version skew if we run with a libpq
index ee963d85f30066cc1c11bc670dce88b12e95ed10..156428d908e8f5f2d9805c486622008ea4b051b2 100644 (file)
@@ -35,7 +35,6 @@ my $libpq;
 my @unlink_on_exit;
 
 # Set of variables for modules in contrib/ and src/test/modules/
-my $contrib_defines        = {};
 my @contrib_uselibpq       = ();
 my @contrib_uselibpgport   = ();
 my @contrib_uselibpgcommon = ();
@@ -53,7 +52,6 @@ my @contrib_excludes       = (
    'unsafe_tests');
 
 # Set of variables for frontend modules
-my $frontend_defines = { 'initdb' => 'FRONTEND' };
 my @frontend_uselibpq =
   ('pg_amcheck', 'pg_ctl', 'pg_upgrade', 'pgbench', 'psql', 'initdb');
 my @frontend_uselibpgport = (
@@ -1126,10 +1124,10 @@ sub AdjustContribProj
 {
    my $proj = shift;
    AdjustModule(
-       $proj,                    $contrib_defines,
-       \@contrib_uselibpq,       \@contrib_uselibpgport,
-       \@contrib_uselibpgcommon, $contrib_extralibs,
-       $contrib_extrasource,     $contrib_extraincludes);
+       $proj,                  \@contrib_uselibpq,
+       \@contrib_uselibpgport, \@contrib_uselibpgcommon,
+       $contrib_extralibs,     $contrib_extrasource,
+       $contrib_extraincludes);
    return;
 }
 
@@ -1137,17 +1135,16 @@ sub AdjustFrontendProj
 {
    my $proj = shift;
    AdjustModule(
-       $proj,                     $frontend_defines,
-       \@frontend_uselibpq,       \@frontend_uselibpgport,
-       \@frontend_uselibpgcommon, $frontend_extralibs,
-       $frontend_extrasource,     $frontend_extraincludes);
+       $proj,                   \@frontend_uselibpq,
+       \@frontend_uselibpgport, \@frontend_uselibpgcommon,
+       $frontend_extralibs,     $frontend_extrasource,
+       $frontend_extraincludes);
    return;
 }
 
 sub AdjustModule
 {
    my $proj                  = shift;
-   my $module_defines        = shift;
    my $module_uselibpq       = shift;
    my $module_uselibpgport   = shift;
    my $module_uselibpgcommon = shift;
@@ -1156,13 +1153,6 @@ sub AdjustModule
    my $module_extraincludes  = shift;
    my $n                     = $proj->{name};
 
-   if ($module_defines->{$n})
-   {
-       foreach my $d ($module_defines->{$n})
-       {
-           $proj->AddDefine($d);
-       }
-   }
    if (grep { /^$n$/ } @{$module_uselibpq})
    {
        $proj->AddIncludeDir('src\interfaces\libpq');