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 Ubuntu build tutorial for rippled v1.1.0 [DO NOT MERGE YET] #439

Closed
wants to merge 2 commits into from

Conversation

mDuo13
Copy link
Collaborator

@mDuo13 mDuo13 commented Aug 1, 2018

  • Needs a version-setting commit from rippled 1.1.0.

The changes from rippled 1.0.0/1.0.1 are as follows:

  • Requires Boost 1.67.0 (or higher, presumably, though that's the newest Boost as of this writing).
  • CMake instructions have changed slightly. The detailed Linux build instructions from that PR say to use -DCMAKE_BUILD_TYPE=Debug but per @mellery451 that's actually chosen by default anyway, so these instructions omit that flag and include a note about specifying the Release type for production servers.

Resolves #429.

@mDuo13 mDuo13 requested review from mellery451, jbheron and gbarr01 and removed request for jbheron August 1, 2018 21:40
Copy link
Contributor

@gbarr01 gbarr01 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@mellery451 mellery451 left a comment

Choose a reason for hiding this comment

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

👍

- consolidated initial install dependencies
- BOOST_ROOT dir will be incorrectly set if your username is not ubuntu
- (Revoved by @mDuo13: automatically setting number of processors to use to compile Boost libraries to half of what users would have)
- git log to now show v1.0.1
- numbering changes
@mDuo13
Copy link
Collaborator Author

mDuo13 commented Aug 21, 2018

Included in #448

@mDuo13 mDuo13 closed this Aug 21, 2018
@mDuo13 mDuo13 deleted the doc1650 branch June 14, 2019 20:44
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.

4 participants