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

Fix double jsonification and setpool controls #6

Merged
merged 3 commits into from
Mar 15, 2018
Merged

Fix double jsonification and setpool controls #6

merged 3 commits into from
Mar 15, 2018

Conversation

kkkrackpot
Copy link
Contributor

Hopefully partially fixes the ccminer issue mentioned here #5

@kkkrackpot kkkrackpot changed the title Fix double jsonification, to keep ccminer happy. Fix double jsonification and setpool controls Mar 9, 2018
@p4u
Copy link
Owner

p4u commented Mar 14, 2018

I'm not maintaing this code and I've not the infrastructure for testing it with real miners and real pools.

So I'd merge your pool request, just need a confirmation that you tested it deep enough.

@kkkrackpot
Copy link
Contributor Author

@p4u I tested it for several evenings doing pool-hopping :)
At least for me it works as I want it to work, including reconnections to the same pool with different user/pass credentials. ccminer's errors're also gone.
However, I cant guarantee that it will work for everyone the way they want.
It's a pity that you lost interest to the tool, it's quite a useful thing...

@p4u p4u merged commit 13af244 into p4u:master Mar 15, 2018
@p4u
Copy link
Owner

p4u commented Mar 15, 2018

It's a pity that you lost interest to the tool, it's quite a useful thing...

Lack of time and resources...

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