Tweak environment block resolve strategy to be more ergonomic

Widen · Jul 31, 2019 · 4c957bf · 4c957bf
1 parent 18bec66
commit 4c957bf
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/buildSrc/src/main/groovy/com/widen/plugins/buildkite/ConfigurableEnvironment.groovy b/buildSrc/src/main/groovy/com/widen/plugins/buildkite/ConfigurableEnvironment.groovy
@@ -22,9 +22,11 @@ trait ConfigurableEnvironment {
      * Set environment variables using a configure block.
      */
     void environment(@DelegatesTo(Map) Closure closure) {
-        [:].with {
-            with(closure)
-            environment(it)
-        }
+        def map = [:]
+        closure = (Closure) closure.clone()
+        closure.delegate = map
+        closure.resolveStrategy = Closure.OWNER_FIRST
+        closure()
+        environment(map)
     }
 }