RandomX branch - cmake required - documentation update #145

Closed
opened 2 years ago by jahway603 · 6 comments
Collaborator

TO-DO - add cmake as pre-req in INSTALL.md as required for RandomX branch

apt install cmake

  • Tested and noted on both Debian 11 and Ubuntu 20.04 LTS
TO-DO - add cmake as pre-req in INSTALL.md as required for RandomX branch `apt install cmake` * Tested and noted on both Debian 11 and Ubuntu 20.04 LTS
jahway603 self-assigned this 2 years ago
Poster
Collaborator
- [x] Add cmake to INSTALL.md - [x] Add cmake to https://git.hush.is/onryo/hush-smart-chain-how-to
Owner

@jahway603 I prefer to not add build dependencies, but RandomX requires cmake. In the future, we could change it such that if cmake is not installed, Hush can still be compiled but HSCs using RandomX will not be available. Not sure if it's worth pursuing that option.

@jahway603 I prefer to not add build dependencies, but RandomX requires cmake. In the future, we could change it such that if cmake is not installed, Hush can still be compiled but HSCs using RandomX will not be available. Not sure if it's worth pursuing that option.
Poster
Collaborator

@duke cmake isn't a large install and I was able to compile the RandomX branch with the package manager version of cmake (cmake version 3.16.3 on Ubuntu 20.04 and cmake version 3.18.4 on Debian 11)

@duke cmake isn't a large install and I was able to compile the RandomX branch with the package manager version of cmake (cmake version 3.16.3 on Ubuntu 20.04 and cmake version 3.18.4 on Debian 11)
Owner

Issue to check for cmake in build.sh is here:

#155

Issue to check for cmake in build.sh is here: https://git.hush.is/hush/hush3/issues/155
Poster
Collaborator

Added cmake to hush3 INSTALL.md as part of PR #158

Added cmake to hush3 INSTALL.md as part of PR https://git.hush.is/hush/hush3/pulls/158
Owner

ok, I think we can close this

ok, I think we can close this
duke closed this issue 2 years ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.