Modify

Opened 11 years ago

Closed 10 years ago

#36 closed enhancement (fixed)

support for torque 4.2

Reported by: stijn.deweirdt@… Owned by: bas
Priority: major Milestone: Torque 4.X support
Component: pbs Version: 4.3.5
Keywords: Cc:

Description

although pbs_python code compiles, one cannot load the _pbs.so

>>> import pbs
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib64/python2.6/site-packages/pbs/pbs.py", line 25, in <module>
    _pbs = swig_import_helper()
  File "/usr/lib64/python2.6/site-packages/pbs/pbs.py", line 21, in swig_import_helper
    _mod = imp.load_module('_pbs', fp, pathname, description)
ImportError: /usr/lib64/python2.6/site-packages/pbs/_pbs.so: undefined symbol: log_record

a lots has changed codewise in torque 4.2, i assume a new swig interface file has to be generated.

Attachments (2)

pbs_wrap_4.2.c (271.3 KB) - added by stijn.deweirdt@… 11 years ago.
pbs_wrap for 4.2
setup.py.in (2.3 KB) - added by stijn.deweirdt@… 11 years ago.
support for 4.2 in setup.py

Download all attachments as: .zip

Change History (13)

comment:1 Changed 11 years ago by bas

  • Status changed from new to assigned
  • Type changed from defect to enhancement

That is correct. In our environment we only use 2.4.X and 2.5.X. On the list i read a lot problems with the 4.X versions. you can check ticket #34. He left out some functions and it worked for Glenn.

Changed 11 years ago by stijn.deweirdt@…

pbs_wrap for 4.2

Changed 11 years ago by stijn.deweirdt@…

support for 4.2 in setup.py

comment:2 Changed 11 years ago by anonymous

the above attachements adds support for 4.2.

pbs_wrap_4.2 is based on the wrap for 2.4 setup.py.in now uses Looseversion to compare versions; it also assumes that you have torque include files installed somewhere (eg /usr/include/torque).

warning: current torque releases do not package log.h; get it from somewhere and add it to the inlcude/torque dir. i opened https://github.com/adaptivecomputing/torque/issues/166 to have this fixed in later versions.

comment:3 Changed 10 years ago by anonymous

these 3 steps worked for me, but I had to set os.environCC?=os.environCXX? instead of hardcoding 'g++' in setup.py.in

comment:4 Changed 10 years ago by bas

Thanks for the info. I have to make some time to port it. Actually these file are generated by swig.

comment:5 Changed 10 years ago by cjfields@…

I can confirm that this patch works as well (using it for our local Galaxy installation).

comment:6 Changed 10 years ago by keberflores@…

I tried these patches, but I'm getting another error message after that:

>>> import pbs
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/pymodules/python2.7/pbs/pbs.py", line 25, in <module>
    _pbs = swig_import_helper()
  File "/usr/lib/pymodules/python2.7/pbs/pbs.py", line 21, in swig_import_helper
    _mod = imp.load_module('_pbs', fp, pathname, description)
ImportError: /usr/lib/pymodules/python2.7/pbs/_pbs.so: undefined symbol: _Z9log_eventiiPKcPc

Over Ubuntu 12.04, torque 4.2.16.

What I've done:

  • Download current stable pbs-python.tar.gz from Installation Wiki (4.3.5)
  • Download files pbs_wrap_4.2.c and setup.py.in and placed in src/ and ./ respectively
  • dpkg-buildpackage -d
  • dpkg --install pbs-python-4.3.5-1_amd64.deb (generated in the previous step)

Same results when using make install.

Any hints? Thanks.

comment:7 Changed 10 years ago by bas

As today i have just installed 2.4-dev on a test node and succeeded in porting pbs_python to this torque release. I will release an updated pbs_python version shortly.

Version 0, edited 10 years ago by bas (next)

comment:8 Changed 10 years ago by bas

  • Milestone set to Torque 4.X support

comment:9 Changed 10 years ago by bas

In 300:

C directory is used for Torque version 2.X.
rewrite setup.py inspired by stijn de weirdt (University Gent), see #36

comment:10 Changed 10 years ago by bas

support is added in the trunk version that will shortly be released as version 4.4.0

comment:11 Changed 10 years ago by bas

  • Resolution set to fixed
  • Status changed from assigned to closed

Version 4.4.0 is released

Add Comment

Modify Ticket

Change Properties
Action
as closed The owner will remain bas.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.