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 objcopy and nm on ARM platforms #29

Merged
merged 6 commits into from
Oct 29, 2024
Merged

Fix objcopy and nm on ARM platforms #29

merged 6 commits into from
Oct 29, 2024

Conversation

gmartin82
Copy link
Collaborator

@gmartin82 gmartin82 commented Oct 22, 2024

Build script now executes the objcopy and nm commands via cmake to ensure the correct version is used for the architecture.

Partly addresses #24.

@gmartin82 gmartin82 changed the title Fix objcopy on ARM platforms Fix objcopy and nm on ARM platforms Oct 22, 2024
@gmartin82 gmartin82 assigned JEnoch and unassigned JEnoch Oct 23, 2024
@gmartin82 gmartin82 requested a review from JEnoch October 23, 2024 13:31
@gmartin82 gmartin82 added the bug Something isn't working label Oct 23, 2024
@gmartin82
Copy link
Collaborator Author

Changes tested as part of this draft PR on the DDS plugin: eclipse-zenoh/zenoh-plugin-dds#376

Also tested with the zenoh-all-in-one repository.

@gmartin82 gmartin82 marked this pull request as ready for review October 23, 2024 13:31
@gabrik gabrik merged commit cd79443 into master Oct 29, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants