-
Notifications
You must be signed in to change notification settings - Fork 249
ci: add Ubuntu 26.04 to CI + minor flags #238
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 5 commits
23e7442
899adb6
416bc1b
cdd8e1d
3a62857
efaf833
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,9 +5,19 @@ index 20a6a05..5577e99 100644 | |
| @@ -3,7 +3,7 @@ macro(add_urdfdom_library) | ||
| set(multiValueArgs SOURCES LINK) | ||
| cmake_parse_arguments(add_urdfdom_library "${options}" "${oneValueArgs}" "${multiValueArgs}" ${ARGN}) | ||
|
|
||
| - add_library(${add_urdfdom_library_LIBNAME} SHARED | ||
| + add_library(${add_urdfdom_library_LIBNAME} STATIC | ||
|
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this was a "patch on the patch" from my side because I saw the BUILD_SHARED_LIBS_OFF would be overriden by the SHARED keyword. I am not sure whether there may be consequences I am not aware of @AndreasKuhner @francando
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Probably fine? Can be verified by checking the generated .so and check if some urdfdom shows up |
||
| ${add_urdfdom_library_SOURCES}) | ||
| target_include_directories(${add_urdfdom_library_LIBNAME} PUBLIC | ||
| "$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>" | ||
| diff --git a/urdf_parser/include/urdf_parser/urdf_parser.h b/urdf_parser/include/urdf_parser/urdf_parser.h | ||
| --- a/urdf_parser/include/urdf_parser/urdf_parser.h | ||
| +++ b/urdf_parser/include/urdf_parser/urdf_parser.h | ||
| @@ -49,5 +49,6 @@ | ||
| #include <urdf_world/types.h> | ||
|
|
||
| #include "exportdecl.h" | ||
| +#include <cstdint> | ||
|
|
||
| namespace tinyxml2{ | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -34,7 +34,7 @@ Key Features | |
|
|
||
| - **Low-level control**: Access precise motion control for research robots. | ||
| - **Real-time communication**: Interact with the robot in real-time. | ||
| - **Multi-platform support**: Ubuntu 20.04, 22.04, and 24.04 LTS | ||
| - **Multi-platform support**: Ubuntu 20.04, 22.04, 24.04, and 26.04 LTS | ||
|
|
||
|
|
||
| 1. System Requirements | ||
|
|
@@ -46,6 +46,7 @@ Before using **libfranka**, ensure your system meets the following requirements: | |
| - Ubuntu 20.04 LTS (Focal Fossa) | ||
| - Ubuntu 22.04 LTS (Jammy Jellyfish) | ||
| - Ubuntu 24.04 LTS (Noble Numbat) | ||
| - Ubuntu 26.04 LTS (Resolute Raccoon) | ||
| - `Linux with PREEMPT_RT patched kernel <https://frankarobotics.github.io/docs/doc/libfranka/docs/real_time_kernel.html>`_ recommended for real-time control | ||
|
|
||
| **Build Tools** (for building from source): | ||
|
|
@@ -59,7 +60,7 @@ Before using **libfranka**, ensure your system meets the following requirements: | |
|
|
||
| .. _installation-debian-package: | ||
|
|
||
| 2. Installation from Debian Package (Recommended) | ||
| 1. Installation from Debian Package (Recommended) | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should that be 2?
r-simonelli marked this conversation as resolved.
Outdated
|
||
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
|
|
||
| The easiest way to install **libfranka** is by using the pre-built Debian packages | ||
|
|
@@ -94,6 +95,12 @@ Supported Platforms | |
| - .. image:: https://img.shields.io/github/actions/workflow/status/frankarobotics/libfranka/libfranka-build.yml?label=Ubuntu+24.04&logo=ubuntu&logoColor=white | ||
| :target: https://github.com/frankarobotics/libfranka/actions/workflows/libfranka-build.yml | ||
| :alt: Ubuntu 24.04 build status | ||
| * - 26.04 LTS | ||
| - resolute | ||
| - amd64 | ||
| - .. image:: https://img.shields.io/github/actions/workflow/status/frankarobotics/libfranka/libfranka-build.yml?label=Ubuntu+26.04&logo=ubuntu&logoColor=white | ||
| :target: https://github.com/frankarobotics/libfranka/actions/workflows/libfranka-build.yml | ||
| :alt: Ubuntu 26.04 build status | ||
|
|
||
| Quick Install | ||
| ^^^^^^^^^^^^^ | ||
|
|
@@ -231,6 +238,9 @@ Method B: Using Docker Command Line | |
| # For Ubuntu 24.04 | ||
| docker build --build-arg UBUNTU_VERSION=24.04 -t libfranka-build:24.04 .ci/ | ||
|
|
||
| # For Ubuntu 26.04 | ||
| docker build --build-arg UBUNTU_VERSION=26.04 -t libfranka-build:26.04 .ci/ | ||
|
|
||
| 2. **Run the container and build** | ||
|
|
||
| .. code-block:: bash | ||
|
|
@@ -540,6 +550,10 @@ For more examples, see the `Usage Examples documentation <https://frankarobotics | |
| - .. image:: https://img.shields.io/github/actions/workflow/status/frankarobotics/libfranka/pylibfranka-wheels.yml?label=python3.12&logo=python&logoColor=white | ||
| :target: https://github.com/frankarobotics/libfranka/actions/workflows/pylibfranka-wheels.yml | ||
| :alt: Python 3.12 wheel build status | ||
| * - Ubuntu 26.04 | ||
| - .. image:: https://img.shields.io/github/actions/workflow/status/frankarobotics/libfranka/pylibfranka-wheels.yml?label=python3.14&logo=python&logoColor=white | ||
| :target: https://github.com/frankarobotics/libfranka/actions/workflows/pylibfranka-wheels.yml | ||
| :alt: Python 3.14 wheel build status | ||
|
|
||
| **Pylibfranka** provides Python bindings for libfranka, allowing robot control with Python. | ||
|
|
||
|
|
@@ -678,6 +692,8 @@ Platform Compatibility | |
| - Python 3.9, 3.10, 3.11, 3.12 | ||
| * - 24.04 (Noble) | ||
| - Python 3.9, 3.10, 3.11, 3.12 | ||
| * - 26.04 (Resolute Raccoon) | ||
| - Python 3.14 | ||
|
|
||
| **Note:** Ubuntu 20.04 users must use Python 3.9 due to glibc compatibility requirements. | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is llvm 22 working with our code base without throwing errors?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @AndreasKuhner, thanks for pointing it out. Are you able to visualize the workflows?
fork workflows
The workflows appear to run successfully. Is this enough as a guarantee? Or are you thinking of runtime problems?

The pywheels failure is due to token problems related to publishing:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E.g. a newer version for clang-tidy usually comes with additional checks etc. which might make the build broke. But I think the workflows should be verifying that... :)