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

Update cmake so that rippled can build as a submodule #3449

Closed
wants to merge 1 commit into from

Conversation

ximinez
Copy link
Collaborator

@ximinez ximinez commented Jun 11, 2020

This is a relatively trivial fix to the build system which allows rippled or parts of it to be built when included as a submodule to another project. There is also the option used in dbc8f14, but this seemed simpler.


This change is Reviewable

Copy link
Contributor

@cjcobb23 cjcobb23 left a comment

Choose a reason for hiding this comment

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

@ximinez Can you add some simple instructions regarding how to test that this works as expected?

@ximinez
Copy link
Collaborator Author

ximinez commented Jun 11, 2020

@cjcobb23 In short, if you can build rippled in all of the current configurations, then it works.

Copy link
Contributor

@cjcobb23 cjcobb23 left a comment

Choose a reason for hiding this comment

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

Cloned this and it built for me.

Copy link
Contributor

@miguelportilla miguelportilla left a comment

Choose a reason for hiding this comment

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

👍 Builds on all configurations.

@ximinez ximinez added the Passed Passed code review & PR owner thinks it's ready to merge. Perf sign-off may still be required. label Jun 16, 2020
@ximinez
Copy link
Collaborator Author

ximinez commented Jun 16, 2020

Squashed and set the "passed" flag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build System Passed Passed code review & PR owner thinks it's ready to merge. Perf sign-off may still be required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants