conda-forge / idyntree-feedstock

A conda-smithy repository for idyntree.
BSD 3-Clause "New" or "Revised" License
3 stars 7 forks source link

ENH: update package version to 13.1.0 #111

Closed conda-forge-admin closed 3 weeks ago

conda-forge-admin commented 3 weeks ago

Hi! This is the friendly automated conda-forge-webservice.

I've started a version update as instructed in #110.

I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!

Fixes #110

conda-forge-admin commented 3 weeks ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

traversaro commented 3 weeks ago

Oh cool, compilation error that was not there in local CI:

2024-10-23T08:28:32.1623907Z $SRC_DIR/build/bindings/python/iDynTreePYTHON_wrap.cxx: In function 'PyObject* _wrap_MatrixDynSize_toNumPy(PyObject*, PyObject*)':
2024-10-23T08:28:32.1624664Z $SRC_DIR/build/bindings/python/iDynTreePYTHON_wrap.cxx:27620:41: error: too few arguments to function 'PyObject* SWIG_Python_AppendOutput(PyObject*, PyObject*, int)'
2024-10-23T08:28:32.1625180Z 27620 |     resultobj = SWIG_Python_AppendOutput(resultobj,obj);
2024-10-23T08:28:32.1625581Z       |                 ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~

perhaps a SWIG 4.3.0 problem?

github-actions[bot] commented 3 weeks ago

Hi! This is the friendly conda-forge automerge bot!

Commits were made to this PR after the automerge label was added. For security reasons, I have disabled automerge by removing the automerge label. Please add the automerge label again (or ask a maintainer to do so) if you'd like to enable automerge again!

traversaro commented 3 weeks ago

Pinning to SWIG 4.2.1 fix the problem, we need to replicate this in idyntree's CI and open an issue in swig.