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

Add support for precompiled libraries #253

Conversation

pablogs9
Copy link
Contributor

@pablogs9 pablogs9 commented Oct 21, 2020

This PR adds support for precompiled libraries in OpenCR Arduino platform.

Purpose:
At micro-ROS we are developing a port of our libraries to your platform that can be integrated into the Arduino IDE. Due to the complexity of the ROS 2 build system, we are providing it as pre-compiled library (check here for more details).

Currently, if you try to build an OpenCR project with a pre-compiled library you obtain:

The platform does not support 'compiler.libraries.ldflags' for pre-compiled libraries.

So in order to provide support, we are adding an empty compiler.libraries.ldflags variable (that will be filled automatically by the Arduino IDE when it found precompiled libraries) to your platform.txt.

Related: https://github.com/micro-ROS/micro_ros_arduino

@ROBOTIS-Will ROBOTIS-Will changed the base branch from master to develop October 21, 2020 23:58
@ROBOTIS-Will
Copy link
Contributor

Hi @pablogs9 Thank you for your contribution. I'll merge this into our develop branch first then merge into the master.

@ROBOTIS-Will ROBOTIS-Will merged commit 9f2f564 into ROBOTIS-GIT:develop Oct 22, 2020
@pablogs9 pablogs9 deleted the feature/add_support_precompiled_libraries branch October 22, 2020 05:11
@pablogs9
Copy link
Contributor Author

Hi @ROBOTIS-Will, thanks. Do you know when are you going to release a new tagged version approximately?

@ROBOTIS-Will
Copy link
Contributor

@pablogs9
I'm also preparing for the TurtleBot3 firmware updates for ROS1 Noetic at the same time and will be merged into the master by tomorrow at the latest.
Thank you.

@pablogs9
Copy link
Contributor Author

Awesome, thanks for the quick response!

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