Skip to content

feat: added cmake for Ubuntu resolute#64

Closed
Kooolkimooov wants to merge 4 commits into
mc-rtc:mainfrom
Kooolkimooov:main
Closed

feat: added cmake for Ubuntu resolute#64
Kooolkimooov wants to merge 4 commits into
mc-rtc:mainfrom
Kooolkimooov:main

Conversation

@arntanguy arntanguy self-assigned this May 26, 2026

@arntanguy arntanguy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll have a look during the week.

In the meanwhile, I'm curious considering you went ahead and fixed a cython issue with Eigen3ToPython, I suppose you got everything to build with cython bindings?

If that's the case, didn't you also run into issues with:

  • having to create a virtualenv for python packages
  • use of setuptools/distutils
  • other issues?

If I recall correctly, this was a sticking point when we tried for ubuntu noble (then we went on rewriting python bindings with nanobind and never revisited the issue).

Comment thread cmake/linux/resolute.cmake Outdated
@Kooolkimooov

Copy link
Copy Markdown
Author

My goal was for the default superbuild to build correctly so I kept python bindings.

However you are right that I should have encountered some issues with the new python behaviour; I think my pyenv setup may have been detected and pre-commit was already installed (reason why I missed the change in the bootstrap script). I'll investigate while getting my environment as close to default Resolute as possible.

@arntanguy

Copy link
Copy Markdown
Contributor

I got it to build locally in an ubuntu 24.04 devcontainer with your cython patches: #56
Now fixing CI issues there, then I'll integrate your changes from this PR, thanks for your proactivity on this!

arntanguy added a commit to arntanguy/mc-rtc-superbuild that referenced this pull request Jun 1, 2026
Based on mc-rtc#64

Direct build on github actions in build.yml workflow is not enabled as
the runner is not yet available...
arntanguy added a commit to arntanguy/mc-rtc-superbuild that referenced this pull request Jun 1, 2026
Based on mc-rtc#64

Direct build on github actions in build.yml workflow is not enabled as
the runner is not yet available...
@arntanguy

Copy link
Copy Markdown
Contributor

Superseeded by #56 where CI and devcontainer generation is running for jammy, noble and resolute.
Many thanks for the patches, and the extra motivation to see this through!

@arntanguy arntanguy closed this Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants