fix: allow envars to be overriden in active environments

env.go

@@ -748,6 +748,7 @@ func (e *Env) Exec(l *ui.UI, pkg *manifest.Package, binary string, args []string
 		return errors.WithStack(err)
 	}
 	ops := e.allEnvarOpsForPackages(runtimeDeps, pkg, installed...)
+	ops = append(ops, e.systemEnvOverrideOps(ops)...)
 	packageHermitBin, err := e.getPackageRuntimeEnvops(pkg)
 	if err != nil {
 		return errors.WithStack(err)
@@ -777,6 +778,25 @@ func (e *Env) Exec(l *ui.UI, pkg *manifest.Package, binary string, args []string
 	return errors.Errorf("%s: could not find binary %q", pkg, binary)
 }
 
+// systemEnvOverrideOps returns environment variables overrode in an already activated system environment
+func (e *Env) systemEnvOverrideOps(ops envars.Ops) envars.Ops {
+	if activeEnv, ok := os.LookupEnv("HERMIT_ENV"); !ok || activeEnv != e.envDir {
+		return envars.Ops{}
+	}
+
+	system := envars.Parse(os.Environ())
+	changed := system.Apply(e.Root(), ops).Changed(false)
+
+	var overrides envars.Ops
+	for envar, v_new := range changed {
+		if v_system, ok := system[envar]; ok && v_new != v_system {
+			overrides = append(overrides, &envars.Force{Name: envar, Value: v_system})
+		}
+	}
+
+	return overrides
+}
+
 func (e *Env) getPackageRuntimeEnvops(pkg *manifest.Package) (envars.Op, error) {
 	// If the package contains a Hermit env, add that to the PATH for runtime dependencies
 	pkgEnv, err := OpenEnv(pkg.Root, e.state, e.packageSource, nil, e.httpClient, e.scriptSums)

integration/integration_test.go

@@ -265,6 +265,16 @@ func TestIntegration(t *testing.T) {
 			assert test "$(testbin1.sh)" = "FOO=runtimefoo"
 			assert test "$(testbin2.sh)" = "BAR=hermitbar"
 			`},
+		{name: "SystemEnvOverridesAlreadyAcitvatedHermitEnv",
+			preparations: prep{fixture("testenv4"), activate(".")},
+			script: `
+			hermit install testbin1
+			hermit install testbin2
+			export FOO=systemfoo
+			assert test "$(testbin1.sh)" = "FOO=systemfoo"
+			export BAR=systembar
+			assert test "$(testbin2.sh)" = "BAR=systembar"
+			`},
 	}
 
 	checkForShells(t)

it/full/spec/it_spec.sh

@@ -318,21 +318,21 @@ Describe "Hermit"
 
   Describe "Runtime dependencies"
     . bin/activate-hermit
-    It "Does not install runtime-dependencies to the environment"
+    It "do not install to the environment"
       When call hermit install testbin1
       The status should be success
       The stderr should be blank
       The file ./bin/testbin1 should be exist
       The file ./bin/testbin2 should not be exist
     End
 
-    It "allows installing packages with binaries conflicting with runtime dependencies"
+    It "allow installing packages with conflicting binaries"
       When call hermit install faketestbin2
       The status should be success
       The stderr should be blank
     End
 
-    It "Calls the runtime dependency correctly"
+    It "calls the runtime dependency correctly"
       When call ./bin/testbin1
       The status should be success
       The stdout should equal "Hello from testbin2"
@@ -344,7 +344,7 @@ Describe "Hermit"
     ln -s . ./symlinked
     cd ./symlinked
     . bin/activate-hermit
-    It "Allows calling binaries in the environment"
+    It "allows calling binaries in the environment"
       When call ./bin/testbin1
       The status should be success
       The stdout should not be blank