py25-ode, py26-ode: #error You must #define dSINGLE or dDOUBLE

[dershow]

I tried to install py26-ode. It first installs ode, which seems to build fine. But then it gives an error when it tries to build py26-ode. Here is the error:

sudo port install py26-ode
--->  Computing dependencies for py26-ode
--->  Fetching archive for py26-ode
--->  Attempting to fetch py26-ode-1.2.0_0.darwin_10.x86_64.tbz2 from http://packages.macports.org/py26-ode
--->  Fetching distfiles for py26-ode
--->  Attempting to fetch PyODE-1.2.0.tar.bz2 from http://voxel.dl.sourceforge.net/pyode
--->  Verifying checksum(s) for py26-ode
--->  Extracting py26-ode
--->  Applying patches to py26-ode
--->  Configuring py26-ode
--->  Building py26-ode
Error: org.macports.build for port py26-ode returned: command execution failed
Please see the log file for port py26-ode for details:
    /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_release_ports_python_py26-ode/py26-ode/main.log
To report a bug, follow the instructions in the guide:
    http://guide.macports.org/#project.tickets
Error: Processing of port py26-ode failed

I will attach the log file as well.

[dershow]

py26-ode build log file

[ryandesign (Ryan Carsten Schmidt)]

The first error in the log seems to be:

/opt/local/include/ode/common.h:111:2: error: #error You must #define dSINGLE or dDOUBLE

So py25-ode and py26-ode need to define that, and it needs to be defined to match what the ode port is doing. Since the ode port uses "--enable-double-precision", ports using ode need to define dDOUBLE.

[dershow]

Can this be done from the Port file? I see that the port is pretty simple, in that it just uses python setup. How can that variable be set so that it is passed into python, so that setup then compiles with the correct settings?

[dershow]

I got it to work, but making a change to the patch file. I don't have permission to directly make the change, but the attached file works for me, when I use it instead of the one supplied by macports. All it does is set the dDOUBLE, as you suggested. Can you apply this as an update to the port?

Thanks,

[dershow]

replacement patch file

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #35055.

[ryandesign (Ryan Carsten Schmidt)]

r94960 (maintainer timeout)