move postInit for posix module into posix module

Author: timfel

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/Python3Core.java

@@ -429,10 +429,6 @@ public void postInitialize() {
 
             loadFile(__BUILTINS_PATCHES__, PythonCore.getCoreHomeOrFail());
 
-            PythonModule os = lookupBuiltinModule("posix");
-            Object environAttr = os.getAttribute("environ");
-            ((PDict) environAttr).setDictStorage(createEnvironDict().getDictStorage());
-
             initialized = true;
         }
     }
@@ -639,16 +635,6 @@ private void loadFile(String s, String prefix) {
         callTarget.call(PArguments.withGlobals(mod));
     }
 
-    @TruffleBoundary
-    private PDict createEnvironDict() {
-        Map<String, String> getenv = System.getenv();
-        PDict environ = factory.createDict();
-        for (Entry<String, String> entry : getenv.entrySet()) {
-            environ.setItem(factory.createBytes(entry.getKey().getBytes()), factory.createBytes(entry.getValue().getBytes()));
-        }
-        return environ;
-    }
-
     public PythonObjectFactory factory() {
         return factory;
     }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/PosixModuleBuiltins.java

@@ -66,6 +66,8 @@
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.List;
+import java.util.Map;
+import java.util.Map.Entry;
 import java.util.Random;
 import java.util.Set;
 import java.util.concurrent.TimeUnit;
@@ -89,10 +91,12 @@
 import com.oracle.graal.python.builtins.objects.common.SequenceStorageNodes;
 import com.oracle.graal.python.builtins.objects.common.SequenceStorageNodes.GetItemNode;
 import com.oracle.graal.python.builtins.objects.common.SequenceStorageNodes.ToByteArrayNode;
+import com.oracle.graal.python.builtins.objects.dict.PDict;
 import com.oracle.graal.python.builtins.objects.exception.OSErrorEnum;
 import com.oracle.graal.python.builtins.objects.floats.PFloat;
 import com.oracle.graal.python.builtins.objects.function.PKeyword;
 import com.oracle.graal.python.builtins.objects.ints.PInt;
+import com.oracle.graal.python.builtins.objects.module.PythonModule;
 import com.oracle.graal.python.builtins.objects.str.PString;
 import com.oracle.graal.python.builtins.objects.tuple.PTuple;
 import com.oracle.graal.python.builtins.objects.type.PythonClass;
@@ -244,6 +248,22 @@ public void initialize(PythonCore core) {
         builtinConstants.put("environ", core.factory().createDict());
     }
 
+    @Override
+    public void postInitialize(PythonCore core) {
+        super.postInitialize(core);
+
+        // fill the environ dictionary with the current environment
+        Map<String, String> getenv = System.getenv();
+        PDict environ = core.factory().createDict();
+        for (Entry<String, String> entry : getenv.entrySet()) {
+            environ.setItem(core.factory().createBytes(entry.getKey().getBytes()), core.factory().createBytes(entry.getValue().getBytes()));
+        }
+
+        PythonModule posix = core.lookupBuiltinModule("posix");
+        Object environAttr = posix.getAttribute("environ");
+        ((PDict) environAttr).setDictStorage(environ.getDictStorage());
+    }
+
     @Builtin(name = "getcwd", fixedNumOfPositionalArgs = 0)
     @GenerateNodeFactory
     public abstract static class CwdNode extends PythonBuiltinNode {