Skip to content
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

Fix Zephyr includes #20

Open
wants to merge 7 commits into
base: humble
Choose a base branch
from
Open

Fix Zephyr includes #20

wants to merge 7 commits into from

Conversation

pablogs9
Copy link
Member

@pablogs9 pablogs9 commented Aug 22, 2022

pablogs9 and others added 7 commits May 30, 2022 12:34
* micro-ROS changes over dashing

Feature/add security directory (#1)

* Added security directory

* Updated security directory

Feature/avoid filesystem and allocation (#2)

* Included RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION

* Added no filesystem options

* Default allocators write access

* Avoid dynamic allocation and no filesytem on error handling

* Typo

* New flags for filesystem and avoid dynamic

* Error handling template

* New allocator approach

Add test_security_directory test from rcl. (#3)

Merge pull request #4 from micro-ROS/feature/zephyr_fixes

Feature/zephyr fixes

CMake refactor (#5)

Update approach (#6)

* Update approach

* Remove target_compile_definitions and refactor flags install

* Added RCUTILS_NO_FILESYSTEM on new functions

* Added RCUTILS_NO_FILESYSTEM on new functions

Co-authored-by: Pablo Garrido <[email protected]>

Updates 17092020


Fix atomics 64bits (#9)


* micro-ROS changes over dashing

Feature/add security directory (#1)

* Added security directory

* Updated security directory

Feature/avoid filesystem and allocation (#2)

* Included RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION

* Added no filesystem options

* Default allocators write access

* Avoid dynamic allocation and no filesytem on error handling

* Typo

* New flags for filesystem and avoid dynamic

* Error handling template

* New allocator approach

Add test_security_directory test from rcl. (#3)

Merge pull request #4 from micro-ROS/feature/zephyr_fixes

Feature/zephyr fixes

CMake refactor (#5)


Update approach (#6)

* Update approach

* Remove target_compile_definitions and refactor flags install

* Added RCUTILS_NO_FILESYSTEM on new functions

* Added RCUTILS_NO_FILESYSTEM on new functions

Co-authored-by: Pablo Garrido <[email protected]>

* Initial changes

* Add hashing and lock pool

* Updates

Co-authored-by: Jose Antonio Moral <[email protected]>
Fix atomics 64bits (#9)



Updates 09102020

* Release micro-ROS Foxy (#8)

Update


Cleaning


Update


Update filesystem


Updates


Adjust logger level
* Avoid not used warnings

* Update
This reverts commit befc608.

Reduce error handling static size (#14) (#15)

Signed-off-by: Pablo Garrido <[email protected]>
(cherry picked from commit 1176652)

Co-authored-by: Pablo Garrido <[email protected]>
Revert "Revert "Install headers to include\${PROJECT_NAME} (ros2#351)""

This reverts commit 4546892.
(cherry picked from commit 85efa4a)

Co-authored-by: Pablo Garrido <[email protected]>
Signed-off-by: Pablo Garrido <[email protected]>
own implementation allowing to fix error
regarding CLOCK_MONOTONIC definition.

Signed-off-by: Felipe <[email protected]>
Signed-off-by: Pablo Garrido <[email protected]>
@pablogs9
Copy link
Member Author

@mergify backport master foxy galactic

@mergify
Copy link

mergify bot commented Aug 22, 2022

backport master foxy galactic

🟠 Waiting for conditions to match

  • merged [📌 backport requirement]

@pablogs9
Copy link
Member Author

Do not merge until decisions on Zephyr version support are taken

@rosterloh
Copy link

@pablogs9 you'll need to pass KERNEL_VERSION_MINOR in too as this change came is for 3.1.0 not 3.0.0 see

@pablogs9
Copy link
Member Author

@pablogs9 you'll need to pass KERNEL_VERSION_MINOR in too as this change came is for 3.1.0 not 3.0.0 see

Ok thanks, we will modify this

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.

4 participants