OMPython setup.py assumes omniidl is installed

[sigurd.jakobsen@…]

Hi,
I recently installed openmodelica from build.openmodelica.org_apt_dists_xenial_nightly_binary-amd64_Packages .
However, I ran into some trouble installing OMPython. The problem was that when using pip install setup.py fails at line 34 because omniidl is not installed. The error message is quite cryptic:

Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-zmpu5w-build/setup.py", line 38, in <module>
        generateIDL()
      File "/tmp/pip-zmpu5w-build/setup.py", line 34, in generateIDL
        if 0<>call(["omniidl","-bpython","-Wbglobal=_OMCIDL","-Wbpackage=OMPythonIDL",idl]):
      File "/usr/lib/python2.7/subprocess.py", line 523, in call
        return Popen(*popenargs, **kwargs).wait()
      File "/usr/lib/python2.7/subprocess.py", line 711, in __init__
        errread, errwrite)
      File "/usr/lib/python2.7/subprocess.py", line 1343, in _execute_child
        raise child_exception
    OSError: [Errno 2] No such file or directory

I was therefore thinking that it would be better if omniidl was listed as a dependency or that the setup script printed an error telling the user to install omniidl.

Best regards
Sigurd

[sjoelund.se]

omniidl is not a pip package (it is part of omniorb, which the user needs to install in order to use the Python omniorb package which is also not listed in pip). Making the error-message better could be done though. Feel free to create a pull request: ​https://github.com/OpenModelica/OMPython

[dietmarw]

I'd say this is fixed since the dependencies are listed in the README now.