From: Todd Lyons <tlyons@exim.org>
Date: Tue, 22 Oct 2013 02:50:23 +0000 (-0700)
Subject: Extract build features from Makefile
X-Git-Url: https://git.exim.org/buildfarm-client.git/commitdiff_plain/fa6f7a1774685967756ac8c98d668193e355c97d

Extract build features from Makefile
---

diff --git a/build-farm.conf.template b/build-farm.conf.template
index 0f44dfc..7924b86 100644
--- a/build-farm.conf.template
+++ b/build-farm.conf.template
@@ -174,7 +174,7 @@ my $branch;
     # locales to test
     locales => [qw( C )],
 
-    # Unused
+    # Placemarker for auto-generated build features
     config_opts =>[
         qw()
     ],
diff --git a/run_build.pl b/run_build.pl
index 2943475..18f4cbf 100755
--- a/run_build.pl
+++ b/run_build.pl
@@ -194,7 +194,6 @@ if (ref($force_every) eq 'HASH')
     $force_every = $force_every->{$branch} || $force_every->{default};
 }
 
-my $config_opts = $EximBuild::conf{config_opts};
 my $scm = new EximBuild::SCM \%EximBuild::conf;
 
 my $buildport;
@@ -1061,19 +1060,6 @@ sub configure
     return unless step_wanted('configure');
     print time_str(),"creating configuration ...\n" if $verbose;
 
-    my @quoted_opts;
-    foreach my $c_opt (@$config_opts)
-    {
-        if ($c_opt =~ /['"]/)
-        {
-            push(@quoted_opts,$c_opt);
-        }
-        else
-        {
-            push(@quoted_opts,"'$c_opt'");
-        }
-    }
-
     my $env = $EximBuild::conf{makefile_set};
     my $add = $EximBuild::conf{makefile_add};
     my $features = $EximBuild::conf{makefile_regex};
@@ -1146,6 +1132,13 @@ sub configure
                 echo "Contents of Local/Makefile:"
                 egrep '^[^#]' $local_conf `;
         push @confout, @tmp;
+        # Build the config_opts array to send to the server
+        my @config_opts = grep s/(?:LOOKUP_|EXPERIMENTAL_|USE_)(\S+)=.*/$1/,
+                          @tmp;
+        push @config_opts, grep s/^(?:EXIM_)(PERL|PYTHON)=.*/$1/,
+                           @tmp;
+        $EximBuild::conf{config_opts} = @config_opts;
+        
         # Does not matter what the Exim version is, as long as it is valid.
         my $exim_ver = $EximBuild::conf{exim_test_version} || '4.82';
         `cd $exim