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

Propagate toolchain-file to external-project #374

Merged
merged 1 commit into from
Jan 29, 2019
Merged

Propagate toolchain-file to external-project #374

merged 1 commit into from
Jan 29, 2019

Conversation

lmayencourt
Copy link
Contributor

If defined, propagate the CMAKE_TOOLCHAIN_FILE argument to the cmake argument of freetype, zlib and ogre projects.

If defined, propagate the CMAKE_TOOLCHAIN_FILE argument to the cmake
argument of freetype, zlib and ogre projects.

Change-Id: Ibf2802b96c2238a06191e78a1b2a3128769a83af
Signed-off-by: Louis Mayencourt <louis.mayencourt@arm.com>
@tfoote tfoote added the in progress Actively being worked on (Kanban column) label Jan 28, 2019
@wjwwood wjwwood added in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Jan 28, 2019
@wjwwood
Copy link
Member

wjwwood commented Jan 28, 2019

Thanks for the pr, does this need to be done for the other vendor packages?

@lmayencourt
Copy link
Contributor Author

Some of them already have the same test: libyaml_vendor, poco_vendor, console_bridge_vendor and uncrustify_vendor.
I fell on that one because of a cross-compilation issue. I didn't check the other one yet...

@wjwwood
Copy link
Member

wjwwood commented Jan 28, 2019

Ok, maybe you can let me know when you've gotten father along and then I can review merge this pr? Or would you prefer to get this one merged and do follow prs? My preference is slightly for the single pr since each one takes some time.

@lmayencourt
Copy link
Contributor Author

I prefer single pr, as I don't think we will need to modify other vendor packages

@wjwwood
Copy link
Member

wjwwood commented Jan 28, 2019

Ok, here's CI then:

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

@wjwwood wjwwood merged commit a8eea33 into ros2:ros2 Jan 29, 2019
@wjwwood wjwwood removed the in review Waiting for review (Kanban column) label Jan 29, 2019
@lmayencourt lmayencourt deleted the topic/ros2_cc branch January 30, 2019 10:00
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