Opened 8 years ago

Closed 8 years ago

#4029 closed enhancement (fixed)

OMPython setup.py assumes omniidl is installed

Reported by: sigurd.jakobsen@… Owned by: Alachew Mengist
Priority: low Milestone: 1.12.0
Component: OMPython Version: v1.11.0
Keywords: Cc:

Description

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

Change History (3)

comment:1 by Martin Sjölund, 8 years ago

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

comment:2 by Dietmar Winkler, 8 years ago

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

comment:3 by Adeel Asghar, 8 years ago

Milestone: Future1.12.0
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.