actually that was needed, but bug remains.

seems like the environment::set for TMP is not carrying through to the
child process's environment somehow?

diff --git a/nucleus/applications/bundler/bundle_creator.cpp b/nucleus/applications/bundler/bundle_creator.cpp
index 52affc8f6bfe87109278e3f3b12cdfc35d9768d4..c537aa4f7c92ec710a530316ffe85966c134e330 100644 (file)
--- a/nucleus/applications/bundler/bundle_creator.cpp
+++ b/nucleus/applications/bundler/bundle_creator.cpp
@@ -465,12 +465,12 @@ int bundle_creator::read_manifest()
       BASE_LOG(astring("will set ") + _manifest_list[i]._payload + " = "
           + _manifest_list[i]._parms);
       astring new_value = parser_bits::substitute_env_vars(_manifest_list[i]._parms);
-///why oh why are we allowing them to overwrite our actual environment just because they're bundling something?      environment::set(_manifest_list[i]._payload, new_value);
+      environment::set(_manifest_list[i]._payload, new_value);
           
-#ifdef DEBUG_BUNDLER
+//#ifdef DEBUG_BUNDLER
       BASE_LOG(astring("** variable ") + _manifest_list[i]._payload + " should have value=" + new_value);
       BASE_LOG(astring("** variable ") + _manifest_list[i]._payload + " now does have value=" + environment::get(_manifest_list[i]._payload));
-#endif
+//#endif
 
       continue;
     } else if (ini.get(section_name, "assert_defined", value)) {