summaryrefslogtreecommitdiff
path: root/pkg/env/env.go
diff options
context:
space:
mode:
authorMatthew Heon <matthew.heon@pm.me>2020-07-31 13:52:14 -0400
committerMatthew Heon <matthew.heon@pm.me>2020-08-18 15:17:46 -0400
commita7e864e6e7de894d4edde4fff00e53dc6a0b5074 (patch)
tree9a6381d9ce5ce0045ffa07dcfea2d10c725ab50d /pkg/env/env.go
parentff1f81b14a2944a1651e1b404681972ff3483d36 (diff)
downloadpodman-a7e864e6e7de894d4edde4fff00e53dc6a0b5074.tar.gz
podman-a7e864e6e7de894d4edde4fff00e53dc6a0b5074.tar.bz2
podman-a7e864e6e7de894d4edde4fff00e53dc6a0b5074.zip
Ensure DefaultEnvVariables is used in Specgen
When we rewrote Podman's pkg/spec, one of the things that was lost was our use of a set of default environment variables, that ensure all containers have at least $PATH and $TERM set. While we're in the process of re-adding it, change it from a variable to a function, so we can ensure the Join function does not overwrite it and corrupt the defaults. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
Diffstat (limited to 'pkg/env/env.go')
-rw-r--r--pkg/env/env.go14
1 files changed, 8 insertions, 6 deletions
diff --git a/pkg/env/env.go b/pkg/env/env.go
index a16007a50..0d55e5560 100644
--- a/pkg/env/env.go
+++ b/pkg/env/env.go
@@ -12,14 +12,16 @@ import (
"github.com/pkg/errors"
)
-// DefaultEnvVariables sets $PATH and $TERM.
-var DefaultEnvVariables = map[string]string{
- "PATH": "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
- "TERM": "xterm",
-}
-
const whiteSpaces = " \t"
+// DefaultEnvVariables returns a default environment, with $PATH and $TERM set.
+func DefaultEnvVariables() map[string]string {
+ return map[string]string{
+ "PATH": "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
+ "TERM": "xterm",
+ }
+}
+
// Slice transforms the specified map of environment variables into a
// slice. If a value is non-empty, the key and value are joined with '='.
func Slice(m map[string]string) []string {