diff options
| author | Alexander Tsoy <alexander@tsoy.me> | 2016-02-02 20:56:15 +0300 |
|---|---|---|
| committer | Heiko Schlittermann (HS12-RIPE) <hs@schlittermann.de> | 2016-02-05 11:17:52 +0100 |
| commit | 75286da3fea0f1640b8b4271ffd0098a6ac0a2f8 (patch) | |
| tree | 897eab3ee7266b554da4dfaffba474fe168ff076 /src | |
| parent | 06685b44bbff151655dd0e2fbaa8cbb8cd44c08a (diff) | |
Rename build-time option TMPDIR to EXIM_TMPDIR
Build-time option TMPDIR included in Makefile clashes with environment
variable of the same name. This breaks tools that make use of that
variable, such as distcc.
The following example demonstrates what's going on:
$ cat Makefile
TMPDIR="/tmp"
all:
env
$ export TMPDIR=test
$ make | grep ^TMPDIR
TMPDIR="/tmp"
distcc error (note the extra quotes):
ERROR: can't use TMPDIR ""/tmp"": No such file or directory
Diffstat (limited to 'src')
| -rw-r--r-- | src/src/EDITME | 2 | ||||
| -rw-r--r-- | src/src/config.h.defaults | 2 | ||||
| -rw-r--r-- | src/src/exim.c | 12 |
3 files changed, 8 insertions, 8 deletions
diff --git a/src/src/EDITME b/src/src/EDITME index 30a296e89..6afe0c74c 100644 --- a/src/src/EDITME +++ b/src/src/EDITME @@ -1123,7 +1123,7 @@ SYSTEM_ALIASES_FILE=/etc/aliases # it replaces the value with what is defined here. Commenting this setting # suppresses the check altogether. -TMPDIR="/tmp" +EXIM_TMPDIR="/tmp" #------------------------------------------------------------------------------ diff --git a/src/src/config.h.defaults b/src/src/config.h.defaults index 14de083fe..c1cf1a9b4 100644 --- a/src/src/config.h.defaults +++ b/src/src/config.h.defaults @@ -150,7 +150,7 @@ it's a default value. */ #define TCP_WRAPPERS_DAEMON_NAME "exim" #define TIMEZONE_DEFAULT -#define TMPDIR +#define EXIM_TMPDIR #define TRANSPORT_APPENDFILE #define TRANSPORT_AUTOREPLY diff --git a/src/src/exim.c b/src/src/exim.c index ebc71dd37..c134bf5e6 100644 --- a/src/src/exim.c +++ b/src/src/exim.c @@ -3899,20 +3899,20 @@ if (log_oneline) temporary files are created; Exim doesn't use these (apart from when delivering to MBX mailboxes), but called libraries such as DBM libraries may require them. If TMPDIR is found in the environment, reset it to the value defined in the -TMPDIR macro, if this macro is defined. */ +EXIM_TMPDIR macro, if this macro is defined. */ -#ifdef TMPDIR +#ifdef EXIM_TMPDIR { uschar **p; for (p = USS environ; *p != NULL; p++) { if (Ustrncmp(*p, "TMPDIR=", 7) == 0 && - Ustrcmp(*p+7, TMPDIR) != 0) + Ustrcmp(*p+7, EXIM_TMPDIR) != 0) { - uschar *newp = malloc(Ustrlen(TMPDIR) + 8); - sprintf(CS newp, "TMPDIR=%s", TMPDIR); + uschar *newp = malloc(Ustrlen(EXIM_TMPDIR) + 8); + sprintf(CS newp, "TMPDIR=%s", EXIM_TMPDIR); *p = newp; - DEBUG(D_any) debug_printf("reset TMPDIR=%s in environment\n", TMPDIR); + DEBUG(D_any) debug_printf("reset TMPDIR=%s in environment\n", EXIM_TMPDIR); } } } |