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 CMAKE configuration for C standard version #340

Merged
merged 3 commits into from
Feb 16, 2023
Merged

Conversation

Acuadros95
Copy link
Contributor

Needed for ESP-IDF v5 support: micro-ROS/micro_ros_espidf_component#176

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@Acuadros95 Acuadros95 merged commit 8dfd8dc into develop Feb 16, 2023
@Acuadros95 Acuadros95 deleted the feature/c_version branch February 16, 2023 08:53
Acuadros95 added a commit that referenced this pull request Mar 29, 2023
* Add CMAKE configuration for C standard version (#340)

* Add C standard configuration

Signed-off-by: acuadros95 <[email protected]>

* Update C define

Signed-off-by: acuadros95 <[email protected]>

* Remove CMAKE conditional

Signed-off-by: acuadros95 <[email protected]>

---------

Signed-off-by: acuadros95 <[email protected]>

* Fix build for macOS (#346)

* Add Twitter and Readthedocs shields (#307) (#308)

* Add Twitter and Readthedocs shields

Signed-off-by: Pablo Garrido <[email protected]>

* Update

Signed-off-by: Pablo Garrido <[email protected]>

* Fix

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

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

* Fix build for macOS

- Add check for CMAKE_SYSTEM_NAME "Darwin".

Signed-off-by: Rhys Mainwaring <[email protected]>

---------

Signed-off-by: Rhys Mainwaring <[email protected]>
Co-authored-by: Pablo Garrido <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Fix newline-eof compiler warning (#347)

Add EOF newline to session.c.

Signed-off-by: Rhys Mainwaring <[email protected]>

* Fix doxygen warning (#352)

* Remove outdated navindex tag

Signed-off-by: acuadros95 <[email protected]>

* Revert unwanted removal

Signed-off-by: acuadros95 <[email protected]>

---------

Signed-off-by: acuadros95 <[email protected]>

* Add missing doxygen parameter (#353)

Signed-off-by: acuadros95 <[email protected]>

* Fix domain id argument in create_participant API (#348)

---------

Signed-off-by: acuadros95 <[email protected]>
Signed-off-by: Rhys Mainwaring <[email protected]>
Co-authored-by: Rhys Mainwaring <[email protected]>
Co-authored-by: Pablo Garrido <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
pablogs9 added a commit that referenced this pull request Mar 29, 2023
* Add CMAKE configuration for C standard version (#340)

* Add C standard configuration

Signed-off-by: acuadros95 <[email protected]>

* Update C define

Signed-off-by: acuadros95 <[email protected]>

* Remove CMAKE conditional

Signed-off-by: acuadros95 <[email protected]>

---------

Signed-off-by: acuadros95 <[email protected]>

* Fix build for macOS (#346)

* Add Twitter and Readthedocs shields (#307) (#308)

* Add Twitter and Readthedocs shields

Signed-off-by: Pablo Garrido <[email protected]>

* Update

Signed-off-by: Pablo Garrido <[email protected]>

* Fix

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

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

* Fix build for macOS

- Add check for CMAKE_SYSTEM_NAME "Darwin".

Signed-off-by: Rhys Mainwaring <[email protected]>

---------

Signed-off-by: Rhys Mainwaring <[email protected]>
Co-authored-by: Pablo Garrido <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Fix newline-eof compiler warning (#347)

Add EOF newline to session.c.

Signed-off-by: Rhys Mainwaring <[email protected]>

* Fix doxygen warning (#352)

* Remove outdated navindex tag

Signed-off-by: acuadros95 <[email protected]>

* Revert unwanted removal

Signed-off-by: acuadros95 <[email protected]>

---------

Signed-off-by: acuadros95 <[email protected]>

* Add missing doxygen parameter (#353)

Signed-off-by: acuadros95 <[email protected]>

* Fix domain id argument in create_participant API (#348)

---------

Signed-off-by: acuadros95 <[email protected]>
Signed-off-by: Rhys Mainwaring <[email protected]>
Co-authored-by: Rhys Mainwaring <[email protected]>
Co-authored-by: Pablo Garrido <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
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.

3 participants