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 cross-compilation by forwarding cmake_toolchain_file #216

Closed
wants to merge 1 commit into from

Conversation

dgoel
Copy link

@dgoel dgoel commented May 13, 2018

Without this change, compiler detection fails for eprosima libraries
since the toolchain file, which may contain the sysroot path necessary
for compilation to work properly, is not forwarded to cmake.

Without this change, compiler detection fails for eprosima libraries
since the toolchain file, which may contain the sysroot path necessary
for compilation to work properly, is not forwarded to cmake.
@dgoel
Copy link
Author

dgoel commented May 17, 2018

See this comment for more details about the build failure that this PR is trying to address:

ros2-for-arm/ros2#4 (comment)

@richiware
Copy link
Member

Sorry for the delay. We had drastic changes on the file you are modifying in an internal branch. Now we are not using Add_ExternalProjectAdd. Instead of it we are using add_subdirectory and those kind of configuration parameters now are not needed to be passed. These changes are already on master branch. I hope it works for you.

@dgoel
Copy link
Author

dgoel commented Jun 18, 2018

Thanks. I'll close the PR then.

@dgoel dgoel closed this Jun 18, 2018
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