builtin modules have no __file__ attribute?

[minesworld]

Python code like

import os
os.__file__

fails as there is no __file__ set.

Don't know why - in pythonnet the attribute is created on the builtin modules...

[minesworld]

"Interesting" - the main difference between CSnakes CAPI and pythonnet's PythonEngine is how the PYTHONPATH is set.

After Py_InitializeEx pytonnet changes sys.path on the module. the builtin modules have __file__ set.

CSnake does a Py_SetPath before Py_Initalize . the builtin do not have __file__ set.

By changing runtime.cs to

private void InitializeEmbeddedPython()
{
    lock (initLock)
    {
        /*
        if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
            Py_SetPath_UCS2_UTF16(PythonPath);
        else
            Py_SetPath_UCS4_UTF32(PythonPath);
        Debug.WriteLine($"Initializing Python on thread {GetNativeThreadId()}");
        */
        Py_Initialize();

  }
}

and adding the virtuel environment path to the sys module like:

  using (CSnakes.Runtime.Python.GIL.Acquire())
  {
      var pyoPtrSysName = CAPI.AsPyUnicodeObject("sys");
      var pyoPtrSysModule = CAPI.PyImport_Import(pyoPtrSysName);
      var pyoPtrPathAttr = CAPI.AsPyUnicodeObject("path");
      var pyoPtrPathList = CAPI.PyObject_GetAttr(pyoPtrSysModule, pyoPtrPathAttr);
      var pyoPtrPathValue = CAPI.AsPyUnicodeObject(venvSitePackagesPath);
      CAPI.PyList_Append(pyoPtrPathList, pyoPtrPathValue);

     // doesn't care about referencing ...
  }

  // maybe a Runtime.Exec($"import sys\nsys.path.append(r\"{venvSitePackagesPath}\")") would be simpler solution...

CSnakes works correctly. Maybe its a bug in the nuget python package (3.12.7) ...

PS: please think about making the whole CAPI stuff public. My branch of CSnakes does so and without it would be a hard task writing and testing such changes... . Maybe there is some C# magic which enables to use it as public and those who are more concerned just have APIs above and such can not do "unsafe" stuff...

[minesworld]

The code in the pull request append the missing paths from PytonPath to sys.path . This way the CPython's default remain untouched and in order. Maybe the DLL implementors did the right thing...