Fix missing __file__ attribute on builtin modules by applying PythonPath as appending to sys.path #308
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This time it should work...
__file__
and__name__
attribute onos
module (first commit, soTestHasAttrFile
should fail without the patch)The patch:
Py_SetPath
usage and DLL definitionsAppendMissingPathsToSysPath()
toCPythonAPI
, called within GIL before initializing the typesAppendMissingPathsToSysPath
does as it says: it checks if a path is already insys.path
, if not it will be appendedPythonPath
is splitted "as is", its up to the setter ofCPythonAPI.PythonPath
to provide absolute pathsAppendMissingPathsToSysPath
is written using "raw"nint
C-API calls. Added the missing DLL calls as xxxRaw() .