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

rmw_cyclonedds_cpp/CMakeLists.txt: add -latomic for RISC-V #332

Merged
merged 4 commits into from Aug 30, 2021

Conversation

guillaume-pais-siemens
Copy link
Contributor

Co-authored-by: Ivan Santiago Paunovic ivanpauno@ekumenlabs.com

See Pull Request: add -latomic in CMakelist.txt #327

Co-authored-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
@guillaume-pais-siemens guillaume-pais-siemens changed the title Update rmw_cyclonedds_cpp/CMakeLists.txt Update rmw_cyclonedds_cpp/CMakeLists.txt: add -latomic for RISC-V Aug 26, 2021
@guillaume-pais-siemens guillaume-pais-siemens changed the title Update rmw_cyclonedds_cpp/CMakeLists.txt: add -latomic for RISC-V rmw_cyclonedds_cpp/CMakeLists.txt: add -latomic for RISC-V Aug 26, 2021
@ivanpauno
Copy link
Member

ci.ros2.org is currently down, I will run CI when it gets back

@ivanpauno
Copy link
Member

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno
Copy link
Member

My bad, I ran CI using the wrong branch.
Again:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno
Copy link
Member

third time's the charm:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno
Copy link
Member

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno
Copy link
Member

Finally ...

It seems that macOS has issues when -latomic is passed and windows shows a warning as well.
@guillaume-pais-siemens could you add some platform dependent logic? Thanks!

@ivanpauno
Copy link
Member

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

Copy link
Collaborator

@eboasson eboasson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM too!

@ivanpauno ivanpauno merged commit 91646ce into ros2:master Aug 30, 2021
@ivanpauno
Copy link
Member

Thanks @guillaume-pais-siemens !!

@ivanpauno
Copy link
Member

@Mergifyio backport foxy galactic

@mergify
Copy link

mergify bot commented Aug 30, 2021

Command backport foxy galactic: success

Backports have been created

ivanpauno pushed a commit that referenced this pull request Aug 30, 2021
(cherry picked from commit 91646ce)

Co-authored-by: guillaume-pais-siemens <79512275+guillaume-pais-siemens@users.noreply.github.com>
ivanpauno pushed a commit that referenced this pull request Aug 30, 2021
(cherry picked from commit 91646ce)

Co-authored-by: guillaume-pais-siemens <79512275+guillaume-pais-siemens@users.noreply.github.com>
clalancette pushed a commit to eboasson/rmw_cyclonedds that referenced this pull request May 18, 2022
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.

None yet

3 participants