Skip to content
This repository has been archived by the owner on Apr 24, 2022. It is now read-only.

Update README #106

Merged
merged 2 commits into from
Jul 3, 2017
Merged

Update README #106

merged 2 commits into from
Jul 3, 2017

Conversation

chfast
Copy link
Contributor

@chfast chfast commented Jul 3, 2017

No description provided.

@chfast chfast force-pushed the readme branch 2 times, most recently from e0240c7 to 0f8ba52 Compare July 3, 2017 09:42
README.md Outdated

```sh
mkdir build; cd build
cmake ..
Copy link

Choose a reason for hiding this comment

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

Is there a difference between .. and ../, also it might be good to point out this is the step where you pass the -DETHASHCUDA=ON option

README.md Outdated
mkdir build; cd build
cmake ..
cmake --build .
```
Copy link

Choose a reason for hiding this comment

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

This is linux specific but maybe add a make install at the end so the binary gets copied to /usr/local/bin

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Does it work?

Copy link

Choose a reason for hiding this comment

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

yes, it's how I built and installed from master on Ubuntu 17

@chfast
Copy link
Contributor Author

chfast commented Jul 3, 2017

I extended the "building from source" section. How does it look now?

@CamonZ
Copy link

CamonZ commented Jul 3, 2017

LGTM 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants