Skip to content

Commit e45e229

Browse files
committed
fix putenv()
1 parent 8ae8e3f commit e45e229

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

os/os.module.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,19 +117,19 @@ func getpid(self py.Object, args py.Tuple) (py.Object, error) {
117117
// putenv sets the value of an environment variable named by the key.
118118
func putenv(self py.Object, args py.Tuple) (py.Object, error) {
119119
if len(args) != 2 {
120-
//... return error ...
120+
return nil, py.ExceptionNewf(py.TypeError, "missing required arguments: 'key:str' and 'value:str'")
121121
}
122122
k, ok := args[0].(py.String)
123123
if !ok {
124-
//... return error ...
124+
return nil, py.ExceptionNewf(py.TypeError, "str expected (pos 1), not "+args[0].Type().Name)
125125
}
126126
v, ok := args[1].(py.String)
127127
if !ok {
128-
//... return error ...
128+
return nil, py.ExceptionNewf(py.TypeError, "str expected (pos 2), not "+args[1].Type().Name)
129129
}
130130
err := os.Setenv(string(k), string(v))
131131
if err != nil {
132-
return ...
132+
return nil, py.ExceptionNewf(py.OSError, "Unable to set enviroment variable")
133133
}
134134
return py.None, nil
135135
}

0 commit comments

Comments
 (0)