Don't issue env warning if env is empty

keep_environment needs to be mentioned in the runtime config.
Setting add_environment isn't enough to suppress the warning.

(cherry picked from commit 8e58ed807c77febfde61d3cf47928302f93cc99c)

diff --git a/doc/doc-docbook/spec.xfpt b/doc/doc-docbook/spec.xfpt
index f6c33fe01ad86f498ff473fdc2dca05a04059750..dd68ed56041a08390ee9125a8e7b0b11ef7ab59f 100644 (file)
--- a/doc/doc-docbook/spec.xfpt
+++ b/doc/doc-docbook/spec.xfpt
@@ -14007,9 +14007,10 @@ FOO and having FOO_HOME in your &%keep_environment%& option may have
 unexpected results. You may work around this using a regular expression
 that does not match the macro name: ^[F]OO_HOME$.
 
 unexpected results. You may work around this using a regular expression
 that does not match the macro name: ^[F]OO_HOME$.
 

-Current versions of Exim issue a warning during startupif you do not mention
-&%keep_environment%& or &%add_environment%& in your runtime configuration
-file.
+Current versions of Exim issue a warning during startup if you do not mention
+&%keep_environment%& in your runtime configuration file and if there is
+anything in your environment. Future versions may not issue that warning
+anymore.

 .wen
 
 
 .wen
 
 

diff --git a/src/src/readconf.c b/src/src/readconf.c
index 83d198d33ea6354190e32129a566ae6d1810b878..569c96c8ab775e62450b34eb4dfe76157cbe8d5c 100644 (file)
--- a/src/src/readconf.c
+++ b/src/src/readconf.c
@@ -3344,10 +3344,10 @@ if (openssl_options != NULL)
   }
 #endif
 
   }
 #endif
 

-if ((!add_environment || *add_environment == '\0') && !keep_environment)
+if (!keep_environment && environ && *environ)

   log_write(0, LOG_MAIN,
   log_write(0, LOG_MAIN,

-      "WARNING: purging the environment.\n"
-      " Suggested action: use keep_environment and add_environment.\n");
+      "Warning: purging the environment.\n"
+      " Suggested action: use keep_environment.");

 }
 
 
 }
 
 

diff --git a/test/confs/0615 b/test/confs/0615
index bbfc4978aa750917cb6c3caa19c7988a787a0081..86ecee108b9eefca52a9a66d6bb93b7bf423dc15 100644 (file)
--- a/test/confs/0615
+++ b/test/confs/0615
@@ -1,6 +1,7 @@
 # exim test configuration 0615
 exim_path = EXIM_PATH
 tls_advertise_hosts =
 # exim test configuration 0615
 exim_path = EXIM_PATH
 tls_advertise_hosts =

+spool_directory = DIR/spool

 perl_startup = $| = 1; \
        print "Environment visible in Perl:\n"; \
        print map { "$_=$ENV{$_}\n" } sort keys %ENV;
 perl_startup = $| = 1; \
        print "Environment visible in Perl:\n"; \
        print map { "$_=$ENV{$_}\n" } sort keys %ENV;

diff --git a/test/stderr/0615 b/test/stderr/0615
index 85ee6f7600b6763d04607c3bb06f94b63a9b3ccb..d8d0f8547dfd7533118ad13424ea4765a2585b5f 100644 (file)
--- a/test/stderr/0615
+++ b/test/stderr/0615
@@ -1,15 +1,11 @@
 LOG: MAIN
 LOG: MAIN

-  WARNING: purging the environment.
- Suggested action: use keep_environment and add_environment.
-
+  Warning: purging the environment.
+ Suggested action: use keep_environment.

 LOG: MAIN
 LOG: MAIN

-  WARNING: purging the environment.
- Suggested action: use keep_environment and add_environment.
-
+  Warning: purging the environment.
+ Suggested action: use keep_environment.

 LOG: MAIN
 LOG: MAIN

-  WARNING: purging the environment.
- Suggested action: use keep_environment and add_environment.
-
-1999-03-02 09:44:33 WARNING: purging the environment.
- Suggested action: use keep_environment and add_environment.
-
+  Warning: purging the environment.
+ Suggested action: use keep_environment.
+1999-03-02 09:44:33 Warning: purging the environment.
+ Suggested action: use keep_environment.