From 6609d09e91898223c5a60984603adf2d3351aade Mon Sep 17 00:00:00 2001 From: Kirill Zaitsev Date: Fri, 12 Dec 2014 12:01:12 +0300 Subject: [PATCH] Move env variables outsude docker build #PR2 --- shared/build/build.go | 20 +++++++++----------- shared/build/build_test.go | 5 +++++ 2 files changed, 14 insertions(+), 11 deletions(-) diff --git a/shared/build/build.go b/shared/build/build.go index 1af2f01bd..03aa42a4d 100644 --- a/shared/build/build.go +++ b/shared/build/build.go @@ -445,25 +445,16 @@ func (b *Builder) writeDockerfile(dir string) error { // is the "ubuntu" user, since all build images // inherit from the ubuntu cloud ISO dockerfile.WriteUser("ubuntu") - dockerfile.WriteEnv("HOME", "/home/ubuntu") - dockerfile.WriteEnv("LANG", "en_US.UTF-8") - dockerfile.WriteEnv("LANGUAGE", "en_US:en") dockerfile.WriteEnv("LOGNAME", "ubuntu") - dockerfile.WriteEnv("TERM", "xterm") - dockerfile.WriteEnv("SHELL", "/bin/bash") + dockerfile.WriteEnv("HOME", "/home/ubuntu") dockerfile.WriteRun("sudo chown -R ubuntu:ubuntu /var/cache/drone") dockerfile.WriteRun("sudo chown -R ubuntu:ubuntu /usr/local/bin/drone") default: // all other images are assumed to use // the root user. dockerfile.WriteUser("root") - dockerfile.WriteEnv("HOME", "/root") - dockerfile.WriteEnv("LANG", "en_US.UTF-8") - dockerfile.WriteEnv("LANGUAGE", "en_US:en") dockerfile.WriteEnv("LOGNAME", "root") - dockerfile.WriteEnv("TERM", "xterm") - dockerfile.WriteEnv("SHELL", "/bin/bash") - dockerfile.WriteEnv("GOPATH", "/var/cache/drone") + dockerfile.WriteEnv("HOME", "/root") } dockerfile.WriteAdd("proxy.sh", "/etc/drone.d/") @@ -479,6 +470,13 @@ func (b *Builder) writeDockerfile(dir string) error { func (b *Builder) writeBuildScript(dir string) error { f := buildfile.New() + // add environment variables for user env + f.WriteEnv("LANG", "en_US.UTF-8") + f.WriteEnv("LANGUAGE", "en_US:en") + f.WriteEnv("TERM", "xterm") + f.WriteEnv("GOPATH", "/var/cache/drone") + f.WriteEnv("SHELL", "/bin/bash") + // add environment variables about the build f.WriteEnv("CI", "true") f.WriteEnv("DRONE", "true") diff --git a/shared/build/build_test.go b/shared/build/build_test.go index 25eee4a15..e1eaea07e 100644 --- a/shared/build/build_test.go +++ b/shared/build/build_test.go @@ -537,6 +537,11 @@ func TestWriteBuildScript(t *testing.T) { } f := buildfile.New() + f.WriteEnv("LANG", "en_US.UTF-8") + f.WriteEnv("LANGUAGE", "en_US:en") + f.WriteEnv("TERM", "xterm") + f.WriteEnv("GOPATH", "/var/cache/drone") + f.WriteEnv("SHELL", "/bin/bash") f.WriteEnv("CI", "true") f.WriteEnv("DRONE", "true") f.WriteEnv("DRONE_REMOTE", "git://github.com/drone/drone.git")