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

DO NOT MERGE - Mac smart mining #1936

Closed

Conversation

Projects
None yet
4 participants
@revler1082
Copy link

commented Mar 28, 2017

I'm just abusing the CI to see if this will compile since I don't have a mach, don't hate me =(

Dion Ahmetaj Dion Ahmetaj
@revler1082

This comment has been minimized.

Copy link
Author

commented Mar 28, 2017

whoo, something compiled, if someone wants to build the branch on their mac and let me know if it actually runs that'd be awesome.

@ghost

This comment has been minimized.

Copy link

commented Mar 28, 2017

I can give it a shot later today

@ghost

This comment has been minimized.

@revler1082

This comment has been minimized.

Copy link
Author

commented Mar 29, 2017

@ghost

This comment has been minimized.

Copy link

commented Mar 29, 2017

The fpaste shows me trying it with the last parameter set both to true and false. See line 11 and 20.

@fluffypony

This comment has been minimized.

Copy link
Collaborator

commented Apr 10, 2017

@revler1082 is this still WIP and must stay open?

@revler1082

This comment has been minimized.

Copy link
Author

commented Apr 10, 2017

@fluffypony

This comment has been minimized.

Copy link
Collaborator

commented Apr 24, 2017

@revler1082 just giving this a little bump - no rush if you haven't gotten to it :)

@Jaqueeee Jaqueeee referenced this pull request May 2, 2017

Merged

Enable smart mining #703

@vtnerd

This comment has been minimized.

Copy link
Contributor

commented May 28, 2017

I've been ignoring this for no good reason - @revler1082 do you need a tester? I have a Mac Mini here that I could test against and debug on.

@revler1082

This comment has been minimized.

Copy link
Author

commented May 28, 2017

@fluffypony

This comment has been minimized.

Copy link
Collaborator

commented May 30, 2017

Ok waiting on feedback from @vtnerd before merging

@NanoAkron

This comment has been minimized.

Copy link
Contributor

commented Jun 30, 2017

@revler1082 @fluffypony @vtnerd Should this be closed in favour of #2055?

@revler1082

This comment has been minimized.

Copy link
Author

commented Jun 30, 2017

@fluffypony fluffypony closed this Jul 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.