casadi build from source (from git repo) failing with SWIG 4.2.0

[traversaro]

casadi build from source (from git repo) failing with SWIG 4.2.0

[jaeandersson]

Building CasADi with mainline SWIG is currently not supported. Afaik, all binary builds are made with the matlab-customdoc branch of jaeandersson/swig. It would be great to be compliant with latest SWIG (and to align the MATLAB/SWIG interfaces), but until now there has been no projects justifying work in this direction.

[jaeandersson]

By the way, I guess this was totally by chance, but until SWIG 4.1.0 everything worked fine.

If I remember right, you get quite a few compiler warnings with mainline SWIG. But yes, the work to get it compliant is probably minimal.

[wsfulton]

@traversaro, the change to make should be fairly clear if you look at the related issues you posted as well as the changes within the SWIG code base in f89dd59d4b82ece899087682fdb86e94d2611513. If you can point me to the casadi code base where SWIG_Python_str_AsChar is, I can probably suggest a fix.

[traversaro]

Building CasADi with mainline SWIG is currently not supported. Afaik, all binary builds are made with the matlab-customdoc branch of jaeandersson/swig. It would be great to be compliant with latest SWIG (and to align the MATLAB/SWIG interfaces), but until now there has been no projects justifying work in this direction.

No problem, I totally understand that, thanks for the reply! Thanks to the use of SWIG_IMPORT the workaround is simply to use a release tarball and avoid to use SWIG at all. I mainly opened the issue to track what I found in case someone else experienced the same problem. By the way, I guess this was totally by chance, but until SWIG 4.1.0 everything worked fine.

[jaeandersson]

@traversaro, the change to make should be fairly clear if you look at the related issues you posted as well as the changes within the SWIG code base in f89dd59d4b82ece899087682fdb86e94d2611513. If you can point me to the casadi code base where SWIG_Python_str_AsChar is, I can probably suggest a fix.

@wsfulton All instances should be in this file: https://github.com/casadi/casadi/blob/main/swig/casadi.i

[wsfulton]

I've raised #3520 which should fix it. I'm not sure how to tweak your build system to set it to use swig-4.2.0 though.

[jgillis]

Should be fixed now with that pull request merged