lordgamez commented on code in PR #1781:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1781#discussion_r1591365928
##########
extensions/python/PythonDependencyInstaller.cpp:
##########
@@ -136,6 +136,7 @@ void PythonDependencyInstaller::evalScript(std::string_view
script) {
}
OwnedDict bindings = OwnedDict::create();
+ bindings.put("__builtins__", OwnedObject(PyImport_ImportModule("builtins")));
Review Comment:
Unfortunately I cannot precisely explain what was the issue here, but it
seems that the python builtin variables and functions need to be added in the
python bindings provided for the PyEval_EvalCode to evaluate the `sys.path`
modification script to be executed. It seems that in some platform+python
version combinations it is not needed as the python builtins are available by
default, but other times it needs to be specified explicitly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
