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

Enable tls and improve config. #3

Merged
merged 1 commit into from
May 2, 2016

Conversation

jcvernaleo
Copy link
Member

Config changed to match dcrctl since that is closer to what the miner
does. This removed some totally unneeded options and changed some
others.

gominer now uses its own directory for config rather than reusing the
dcrd one.

Add back btc/dcr copyright in config.go (since the whole file came
from there).

Add sample config file.

Config changed to match dcrctl since that is closer to what the miner
does.  This removed some totally unneeded options and changed some
others.

gominer now uses its own directory for config rather than reusing the
dcrd one.

Add back btc/dcr copyright in config.go (since the whole file came
from there).

Add sample config file.
@jolan
Copy link

jolan commented May 2, 2016

tACK.

When we eventually support mining pools we'll have to improve the url/protocol handling so mining urls work, i.e. stratum+tcp://dcr.coinmine.pl:2222.

Totally fine with how it is now though.

@jcvernaleo
Copy link
Member Author

Yep, I agree that we will need to change when we support pools (which is what I'm looking at now) but I wanted to get it in the same place as the rest of dcr* first.

@jcvernaleo jcvernaleo merged commit 39550c6 into decred:master May 2, 2016
@jcvernaleo jcvernaleo deleted the jcv_connect branch May 2, 2016 18:08
@jcvernaleo jcvernaleo added this to the v0.2.0 milestone Jul 27, 2016
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.

2 participants