Skip to content

Commit 5f80016

Browse files
committed
log subprocesses on FINE
1 parent cc98f60 commit 5f80016

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,12 +66,14 @@
6666
import java.util.HashMap;
6767
import java.util.HashSet;
6868
import java.util.List;
69+
import java.util.Locale;
6970
import java.util.Map;
7071
import java.util.Map.Entry;
7172
import java.util.Random;
7273
import java.util.Set;
7374
import java.util.concurrent.TimeUnit;
7475

76+
import com.oracle.graal.python.PythonLanguage;
7577
import com.oracle.graal.python.builtins.Builtin;
7678
import com.oracle.graal.python.builtins.CoreFunctions;
7779
import com.oracle.graal.python.builtins.PythonBuiltinClassType;
@@ -135,7 +137,6 @@
135137
import com.oracle.truffle.api.profiles.ValueProfile;
136138
import com.sun.security.auth.UnixNumericGroupPrincipal;
137139
import com.sun.security.auth.UnixNumericUserPrincipal;
138-
import java.util.Locale;
139140

140141
@CoreFunctions(defineModule = "posix")
141142
public class PosixModuleBuiltins extends PythonBuiltins {
@@ -1340,6 +1341,7 @@ int system(String cmd) {
13401341
if (!context.isExecutableAccessAllowed()) {
13411342
return -1;
13421343
}
1344+
PythonLanguage.getLogger().fine(() -> "os.system: " + cmd);
13431345
String[] command = new String[]{shell[0], shell[1], cmd};
13441346
Env env = context.getEnv();
13451347
try {

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,7 @@
5151
import java.util.List;
5252
import java.util.Map;
5353

54-
import org.graalvm.nativeimage.ImageInfo;
55-
54+
import com.oracle.graal.python.PythonLanguage;
5655
import com.oracle.graal.python.builtins.Builtin;
5756
import com.oracle.graal.python.builtins.CoreFunctions;
5857
import com.oracle.graal.python.builtins.PythonBuiltinClassType;
@@ -77,6 +76,8 @@
7776
import com.oracle.truffle.api.dsl.NodeFactory;
7877
import com.oracle.truffle.api.dsl.Specialization;
7978

79+
import org.graalvm.nativeimage.ImageInfo;
80+
8081
@CoreFunctions(defineModule = "_posixsubprocess")
8182
public class PosixSubprocessModuleBuiltins extends PythonBuiltins {
8283
@Override
@@ -129,6 +130,7 @@ synchronized int forkExec(PList args, @SuppressWarnings("unused") PList execList
129130
}
130131
}
131132

133+
PythonLanguage.getLogger().fine(() -> "_posixsubprocess.fork_exec: " + String.join(" ", argStrings));
132134
ProcessBuilder pb = new ProcessBuilder(argStrings);
133135
if (p2cread != -1 && p2cwrite != -1) {
134136
pb.redirectInput(Redirect.PIPE);

0 commit comments

Comments
 (0)