Skip to content

Commit f6c6988

Browse files
committed
micro-ROS Kilted patch
Signed-off-by: Antón Casas <[email protected]>
1 parent 0f06574 commit f6c6988

19 files changed

+503
-34
lines changed

.github/workflows/fork_checker.yml

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
name: micro-ROS fork Update Checker
2+
on:
3+
workflow_dispatch:
4+
inputs:
5+
name:
6+
description: "Manual trigger"
7+
schedule:
8+
- cron: '0 4 * * *'
9+
10+
jobs:
11+
micro_ros_fork_update_check:
12+
runs-on: ubuntu-latest
13+
container: ubuntu:24.04
14+
strategy:
15+
fail-fast: false
16+
matrix:
17+
branches: [humble, jazzy, kilted, master]
18+
steps:
19+
- name: Check
20+
id: check
21+
shell: bash
22+
run: |
23+
apt update; apt install -y git
24+
REPO=$(echo ${{ github.repository }} | awk '{split($0,a,"/"); print a[2]}')
25+
git clone -b ${{ matrix.branches }} https://github.com/micro-ros/$REPO
26+
cd $REPO
27+
git remote add ros2 https://github.com/ros2/$REPO
28+
git fetch ros2
29+
git fetch origin
30+
echo "::set-output name=merge_required::true"
31+
CMP=$(git rev-list --left-right --count ros2/${{ matrix.branches }}...origin/${{ matrix.branches }} | awk '{print $1}')
32+
if [ $CMP = "0" ]; then echo "::set-output name=merge_required::false"; fi
33+
34+
- name: Alert
35+
if: ${{ steps.check.outputs.merge_required == 'true' }}
36+
run: exit 1

.vscode/settings.json

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
{
2+
"files.associations": {
3+
"*.uml": "plantuml",
4+
"*.in": "c"
5+
}
6+
}

CMakeLists.txt

Lines changed: 16 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
1-
cmake_minimum_required(VERSION 3.20)
1+
cmake_minimum_required(VERSION 3.12)
22

33
project(rcutils)
44

5+
option(RCUTILS_NO_THREAD_SUPPORT "Disable thread support." OFF)
6+
option(RCUTILS_NO_FILESYSTEM "Disable filesystem usage." OFF)
7+
option(RCUTILS_AVOID_DYNAMIC_ALLOCATION "Disable dynamic allocations." OFF)
8+
option(RCUTILS_NO_64_ATOMIC "Enable alternative support for 64 bits atomic operations in platforms with no native support." OFF)
9+
option(RCUTILS_MICROROS "Flag for building micro-ROS." ON)
10+
511
# Default to C11
612
if(NOT CMAKE_C_STANDARD)
713
set(CMAKE_C_STANDARD 11)
@@ -17,21 +23,6 @@ find_package(ament_cmake REQUIRED)
1723
find_package(ament_cmake_python REQUIRED)
1824
find_package(ament_cmake_ros_core REQUIRED)
1925

20-
# By default, without the settings below, find_package(Python3) will attempt
21-
# to find the newest python version it can, and additionally will find the
22-
# most specific version. For instance, on a system that has
23-
# /usr/bin/python3.10, /usr/bin/python3.11, and /usr/bin/python3, it will find
24-
# /usr/bin/python3.11, even if /usr/bin/python3 points to /usr/bin/python3.10.
25-
# The behavior we want is to prefer the "system" installed version unless the
26-
# user specifically tells us othewise through the Python3_EXECUTABLE hint.
27-
# Setting CMP0094 to NEW means that the search will stop after the first
28-
# python version is found. Setting Python3_FIND_UNVERSIONED_NAMES means that
29-
# the search will prefer /usr/bin/python3 over /usr/bin/python3.11. And that
30-
# latter functionality is only available in CMake 3.20 or later, so we need
31-
# at least that version.
32-
cmake_policy(SET CMP0094 NEW)
33-
set(Python3_FIND_UNVERSIONED_NAMES FIRST)
34-
3526
find_package(Python3 REQUIRED COMPONENTS Interpreter)
3627

3728
ament_python_install_package(${PROJECT_NAME})
@@ -46,7 +37,7 @@ if(UNIX AND NOT APPLE)
4637
endif()
4738
endif()
4839

49-
if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
40+
if(NOT RCUTILS_MICROROS AND (CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang"))
5041
# enables building a static library but later link it into a dynamic library
5142
add_compile_options(-fPIC)
5243
endif()
@@ -93,6 +84,7 @@ set(rcutils_sources
9384
src/time.c
9485
${time_impl_c}
9586
src/uint8_array.c
87+
$<$<BOOL:${RCUTILS_NO_64_ATOMIC}>:src/atomic_64bits.c>
9688
)
9789
set_source_files_properties(
9890
${rcutils_sources}
@@ -147,8 +139,14 @@ target_compile_definitions(${PROJECT_NAME} PRIVATE "RCUTILS_BUILDING_DLL")
147139
if(BUILD_TESTING AND NOT RCUTILS_DISABLE_FAULT_INJECTION)
148140
target_compile_definitions(${PROJECT_NAME} PUBLIC RCUTILS_ENABLE_FAULT_INJECTION)
149141
endif()
142+
configure_file(
143+
"${PROJECT_SOURCE_DIR}/include/rcutils/configuration_flags.h.in"
144+
"${PROJECT_BINARY_DIR}/include/rcutils/configuration_flags.h"
145+
)
150146

151-
target_link_libraries(${PROJECT_NAME} ${CMAKE_DL_LIBS})
147+
if(NOT RCUTILS_NO_FILESYSTEM)
148+
target_link_libraries(${PROJECT_NAME} ${CMAKE_DL_LIBS})
149+
endif()
152150

153151
# Needed if pthread is used for thread local storage.
154152
if(IOS AND IOS_SDK_VERSION LESS 10.0)

include/rcutils/allocator.h

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,21 @@ RCUTILS_WARN_UNUSED
8585
rcutils_allocator_t
8686
rcutils_get_zero_initialized_allocator(void);
8787

88+
/// Set rcutils default allocators.
89+
/**
90+
* <hr>
91+
* Attribute | Adherence
92+
* ------------------ | -------------
93+
* Allocates Memory | No
94+
* Thread-Safe | Yes
95+
* Uses Atomics | No
96+
* Lock-Free | Yes
97+
*/
98+
RCUTILS_PUBLIC
99+
RCUTILS_WARN_UNUSED
100+
bool
101+
rcutils_set_default_allocator(rcutils_allocator_t * allocator);
102+
88103
/// Return a properly initialized rcutils_allocator_t with default values.
89104
/**
90105
* This defaults to:
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
2+
#ifndef RCUTILS__CONFIGURATION_FLAGS_H_
3+
#define RCUTILS__CONFIGURATION_FLAGS_H_
4+
5+
#ifdef __cplusplus
6+
extern "C"
7+
{
8+
#endif
9+
10+
#cmakedefine RCUTILS_NO_FILESYSTEM
11+
#cmakedefine RCUTILS_AVOID_DYNAMIC_ALLOCATION
12+
#cmakedefine RCUTILS_NO_THREAD_SUPPORT
13+
#cmakedefine RCUTILS_MICROROS
14+
15+
#ifdef __cplusplus
16+
}
17+
#endif
18+
19+
#endif // RCUTILS__CONFIGURATION_FLAGS_H_

include/rcutils/error_handling.h

Lines changed: 28 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,19 +39,21 @@ extern "C"
3939
#include "rcutils/testing/fault_injection.h"
4040
#include "rcutils/types/rcutils_ret.h"
4141
#include "rcutils/visibility_control.h"
42+
#include "rcutils/configuration_flags.h"
4243

43-
#ifdef __STDC_LIB_EXT1__
44+
#if defined(__STDC_LIB_EXT1__) && !defined(RCUTILS_NO_FILESYSTEM)
4445
/// Write the given msg out to stderr, limiting the buffer size in the `fwrite`.
4546
/**
4647
* This ensures that there is an upper bound to a buffer overrun if `msg` is
4748
* non-null terminated.
4849
*/
4950
#define RCUTILS_SAFE_FWRITE_TO_STDERR(msg) \
5051
do {fwrite(msg, sizeof(char), strnlen_s(msg, 4096), stderr);} while (0)
51-
#else
52-
/// Write the given msg out to stderr.
52+
#elif !defined(RCUTILS_NO_FILESYSTEM)
5353
#define RCUTILS_SAFE_FWRITE_TO_STDERR(msg) \
5454
do {fwrite(msg, sizeof(char), strlen(msg), stderr);} while (0)
55+
#else
56+
#define RCUTILS_SAFE_FWRITE_TO_STDERR(msg)
5557
#endif
5658

5759
/// Set the error message to stderr using a format string and format arguments.
@@ -63,6 +65,8 @@ extern "C"
6365
* \param[in] format_string The string to be used as the format of the error message.
6466
* \param[in] ... Arguments for the format string.
6567
*/
68+
69+
#if !defined(RCUTILS_AVOID_DYNAMIC_ALLOCATION)
6670
#define RCUTILS_SAFE_FWRITE_TO_STDERR_WITH_FORMAT_STRING(format_string, ...) \
6771
do { \
6872
char output_msg[RCUTILS_ERROR_MESSAGE_MAX_LENGTH]; \
@@ -73,7 +77,11 @@ extern "C"
7377
RCUTILS_SAFE_FWRITE_TO_STDERR(output_msg); \
7478
} \
7579
} while (0)
80+
#else
81+
#define RCUTILS_SAFE_FWRITE_TO_STDERR_WITH_FORMAT_STRING(format_string, ...)
82+
#endif
7683

84+
#if !defined(RCUTILS_AVOID_DYNAMIC_ALLOCATION)
7785
/// The maximum length a formatted number is allowed to have.
7886
#define RCUTILS_ERROR_STATE_LINE_NUMBER_STR_MAX_LENGTH 20 // "18446744073709551615"
7987

@@ -100,6 +108,13 @@ extern "C"
100108
RCUTILS_ERROR_STATE_LINE_NUMBER_STR_MAX_LENGTH - \
101109
RCUTILS_ERROR_FORMATTING_CHARACTERS - \
102110
1)
111+
#else
112+
#define RCUTILS_ERROR_STATE_LINE_NUMBER_STR_MAX_LENGTH 1
113+
#define RCUTILS_ERROR_FORMATTING_CHARACTERS 1
114+
#define RCUTILS_ERROR_MESSAGE_MAX_LENGTH 1
115+
#define RCUTILS_ERROR_STATE_MESSAGE_MAX_LENGTH 1
116+
#define RCUTILS_ERROR_STATE_FILE_MAX_LENGTH 1
117+
#endif // RCUTILS_AVOID_DYNAMIC_ALLOCATION
103118

104119
/// Struct wrapping a fixed-size c string used for returning the formatted error string.
105120
typedef struct rcutils_error_string_s
@@ -121,7 +136,7 @@ typedef struct rcutils_error_state_s
121136
} rcutils_error_state_t;
122137

123138
// make sure our math is right...
124-
#if __STDC_VERSION__ >= 201112L
139+
#if __STDC_VERSION__ >= 201112L && !defined(RCUTILS_AVOID_DYNAMIC_ALLOCATION)
125140
static_assert(
126141
sizeof(rcutils_error_string_t) == (
127142
RCUTILS_ERROR_STATE_MESSAGE_MAX_LENGTH +
@@ -233,8 +248,12 @@ rcutils_set_error_state(const char * error_string, const char * file, size_t lin
233248
*
234249
* \param[in] msg The error message to be set.
235250
*/
251+
#ifdef RCUTILS_AVOID_DYNAMIC_ALLOCATION
252+
#define RCUTILS_SET_ERROR_MSG(msg)
253+
#else
236254
#define RCUTILS_SET_ERROR_MSG(msg) \
237255
do {rcutils_set_error_state(msg, __FILE__, __LINE__);} while (0)
256+
#endif // RCUTILS_AVOID_DYNAMIC_ALLOCATION
238257

239258
/// Set the error message using a format string and format arguments.
240259
/**
@@ -245,6 +264,9 @@ rcutils_set_error_state(const char * error_string, const char * file, size_t lin
245264
* \param[in] format_string The string to be used as the format of the error message.
246265
* \param[in] ... Arguments for the format string.
247266
*/
267+
#ifdef RCUTILS_AVOID_DYNAMIC_ALLOCATION
268+
#define RCUTILS_SET_ERROR_MSG_WITH_FORMAT_STRING(format_string, ...)
269+
#else
248270
#define RCUTILS_SET_ERROR_MSG_WITH_FORMAT_STRING(format_string, ...) \
249271
do { \
250272
char output_msg[RCUTILS_ERROR_MESSAGE_MAX_LENGTH]; \
@@ -255,6 +277,8 @@ rcutils_set_error_state(const char * error_string, const char * file, size_t lin
255277
RCUTILS_SET_ERROR_MSG(output_msg); \
256278
} \
257279
} while (0)
280+
#endif // RCUTILS_AVOID_DYNAMIC_ALLOCATION
281+
258282

259283
/// Indicate that the function intends to set an error message and return an error value.
260284
/**

include/rcutils/logging.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ extern "C"
4040
* \def RCUTILS_DEFAULT_LOGGER_DEFAULT_LEVEL
4141
* \brief The default severity level of the default logger.
4242
*/
43-
#define RCUTILS_DEFAULT_LOGGER_DEFAULT_LEVEL RCUTILS_LOG_SEVERITY_INFO
43+
#define RCUTILS_DEFAULT_LOGGER_DEFAULT_LEVEL RCUTILS_LOG_SEVERITY_UNSET
4444

4545
/// The flag if the logging system has been initialized.
4646
RCUTILS_PUBLIC

include/rcutils/macros.h

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@ extern "C"
2222
{
2323
#endif
2424

25+
#include "rcutils/configuration_flags.h"
26+
2527
#ifndef _MSC_VER
2628
/// A macro to make the compiler warn when the return value of a function is not used.
2729
#define RCUTILS_WARN_UNUSED __attribute__((warn_unused_result))
@@ -32,7 +34,9 @@ extern "C"
3234

3335
/// \cond Doxygen_Suppress
3436
// This block either sets RCUTILS_THREAD_LOCAL or RCUTILS_THREAD_LOCAL_PTHREAD.
35-
#if defined _WIN32 || defined __CYGWIN__
37+
#if defined(RCUTILS_NO_THREAD_SUPPORT)
38+
#define RCUTILS_THREAD_LOCAL
39+
#elif defined _WIN32 || defined __CYGWIN__
3640
// Windows or Cygwin
3741
#define RCUTILS_THREAD_LOCAL __declspec(thread)
3842
#elif defined __APPLE__
Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
// Copyright 2018 Open Source Robotics Foundation, Inc.
2+
//
3+
// Licensed under the Apache License, Version 2.0 (the "License");
4+
// you may not use this file except in compliance with the License.
5+
// You may obtain a copy of the License at
6+
//
7+
// http://www.apache.org/licenses/LICENSE-2.0
8+
//
9+
// Unless required by applicable law or agreed to in writing, software
10+
// distributed under the License is distributed on an "AS IS" BASIS,
11+
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12+
// See the License for the specific language governing permissions and
13+
// limitations under the License.
14+
15+
#ifndef RCUTILS__SECURITY_DIRECTORY_H_
16+
#define RCUTILS__SECURITY_DIRECTORY_H_
17+
18+
#ifdef __cplusplus
19+
extern "C"
20+
{
21+
#endif
22+
23+
#include "rcutils/allocator.h"
24+
#include "rcutils/visibility_control.h"
25+
26+
#ifndef ROS_SECURITY_NODE_DIRECTORY_VAR_NAME
27+
#define ROS_SECURITY_NODE_DIRECTORY_VAR_NAME "ROS_SECURITY_NODE_DIRECTORY"
28+
#endif
29+
30+
#ifndef ROS_SECURITY_ROOT_DIRECTORY_VAR_NAME
31+
#define ROS_SECURITY_ROOT_DIRECTORY_VAR_NAME "ROS_SECURITY_ROOT_DIRECTORY"
32+
#endif
33+
34+
#ifndef ROS_SECURITY_LOOKUP_TYPE_VAR_NAME
35+
#define ROS_SECURITY_LOOKUP_TYPE_VAR_NAME "ROS_SECURITY_LOOKUP_TYPE"
36+
#endif
37+
38+
/// Return the secure root directory associated with a node given its validated name and namespace.
39+
/**
40+
* E.g. for a node named "c" in namespace "/a/b", the secure root path will be
41+
* "a/b/c", where the delimiter "/" is native for target file system (e.g. "\\" for _WIN32).
42+
* If no exact match is found for the node name, a best match would be used instead
43+
* (by performing longest-prefix matching).
44+
*
45+
* However, this expansion can be overridden by setting the secure node directory environment
46+
* variable, allowing users to explicitly specify the exact secure root directory to be utilized.
47+
* Such an override is useful for where the FQN of a node is non-deterministic before runtime,
48+
* or when testing and using additional tools that may not otherwise be easily provisioned.
49+
*
50+
* \param[in] node_name validated node name (a single token)
51+
* \param[in] node_namespace validated, absolute namespace (starting with "/")
52+
* \param[in] allocator the allocator to use for allocation
53+
* \returns machine specific (absolute) node secure root path or NULL on failure
54+
* returned pointer must be deallocated by the caller of this function
55+
*/
56+
RCUTILS_PUBLIC
57+
char * rcutils_get_secure_root(
58+
const char * node_name,
59+
const char * node_namespace,
60+
const rcutils_allocator_t * allocator
61+
);
62+
63+
#ifdef __cplusplus
64+
}
65+
#endif
66+
67+
#endif // RCUTILS__SECURITY_DIRECTORY_H_

include/rcutils/testing/fault_injection.h

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,8 @@ RCUTILS_WARN_UNUSED
8383
int_least64_t
8484
_rcutils_fault_injection_maybe_fail(void);
8585

86+
#ifdef RCUTILS_ENABLE_FAULT_INJECTION
87+
8688
/**
8789
* \def RCUTILS_FAULT_INJECTION_MAYBE_RETURN_ERROR
8890
* \brief This macro checks and decrements a static global variable atomic counter and returns
@@ -199,6 +201,17 @@ _rcutils_fault_injection_maybe_fail(void);
199201
rcutils_fault_injection_set_count(no_fault_injection_count); \
200202
} while (0)
201203

204+
#else
205+
206+
// Mocks for micro-ROS when fault injection not enabled
207+
208+
#define RCUTILS_FAULT_INJECTION_MAYBE_RETURN_ERROR(return_value_on_error)
209+
#define RCUTILS_FAULT_INJECTION_MAYBE_FAIL(failure_code)
210+
#define RCUTILS_FAULT_INJECTION_TEST(code)
211+
#define RCUTILS_NO_FAULT_INJECTION(code)
212+
213+
#endif
214+
202215
#ifdef __cplusplus
203216
}
204217
#endif

0 commit comments

Comments
 (0)