compiler-research / cppyy-backend

1 stars 6 forks source link

Return the error set. #86

Closed vgvassilev closed 1 year ago

vgvassilev commented 1 year ago

This fixes a subtle error when libcppyy-backend has all of its dependencies loaded and it exits successfully at the first time.

In that case python thinks that the cdll object is the error set and tries to iterate it causing a error:

Traceback (most recent call last):
  File "/opt/conda/envs/.venv/lib/python3.10/site-packages/cppyy/__init__.py", line 80, in <module>
    from ._cpython_cppyy import *
  File "/opt/conda/envs/.venv/lib/python3.10/site-packages/cppyy/_cpython_cppyy.py", line 21, in <module>
    c = loader.load_cpp_backend()
  File "/home/jovyan/cppyy-backend/python/cppyy_backend/loader.py", line 99, in load_cpp_backend
    c, err2 = _load_helper(name)
  File "/opt/conda/envs/.venv/lib/python3.10/ctypes/__init__.py", line 392, in __getitem__
    func = self._FuncPtr((name_or_ordinal, self))
TypeError: function name must be string, bytes object or integer
vgvassilev commented 1 year ago

The build fails for other reasons. Let's merge this PR.