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

Change regtest port to 18344. Closes #2269. #2270

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

bitcartel
Copy link
Contributor

No description provided.

@bitcartel bitcartel added the A-networking Area: Networking code label Apr 16, 2017
@bitcartel bitcartel added this to the 1.0.9 milestone Apr 16, 2017
@bitcartel bitcartel requested review from daira and str4d April 16, 2017 21:52
Copy link
Contributor

@daira daira left a comment

Choose a reason for hiding this comment

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

utACK. I confirmed that 18444 does not appear as a port number anywhere else in the source and docs. There should be no compatibility obstacles to changing this for regtest.

Copy link
Contributor

@str4d str4d left a comment

Choose a reason for hiding this comment

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

utACK

@arcalinea
Copy link
Contributor

arcalinea commented Apr 18, 2017

Tested, works fine.

ACK. @zkbot try

@zkbot
Copy link
Contributor

zkbot commented Apr 18, 2017

⌛ Trying commit 05844ba with merge 997ad12...

@daira
Copy link
Contributor

daira commented Apr 18, 2017

Hmm, why is this still pending after 11 hours? @ageis?

@zkbot
Copy link
Contributor

zkbot commented Apr 18, 2017

💥 Test timed out

@ageis
Copy link
Contributor

ageis commented Apr 18, 2017

@zkbot retry

@zkbot
Copy link
Contributor

zkbot commented Apr 18, 2017

⌛ Trying commit 05844ba with merge 7a36a60...

zkbot added a commit that referenced this pull request Apr 18, 2017
@zkbot
Copy link
Contributor

zkbot commented Apr 18, 2017

☀️ Test successful - zcash

@daira
Copy link
Contributor

daira commented Apr 19, 2017

@zkbot r+

@zkbot
Copy link
Contributor

zkbot commented Apr 19, 2017

📌 Commit 05844ba has been approved by daira

@zkbot
Copy link
Contributor

zkbot commented Apr 19, 2017

⌛ Testing commit 05844ba with merge 6142843...

@zkbot
Copy link
Contributor

zkbot commented Apr 19, 2017

💔 Test failed - zcash

@daira
Copy link
Contributor

daira commented Apr 19, 2017

Known-nondeterministic test failure unrelated to this PR. @zkbot retry

@zkbot
Copy link
Contributor

zkbot commented Apr 19, 2017

⌛ Testing commit 05844ba with merge 20d61ac...

zkbot added a commit that referenced this pull request Apr 19, 2017
@zkbot
Copy link
Contributor

zkbot commented Apr 19, 2017

☀️ Test successful - zcash

@zkbot zkbot merged commit 05844ba into zcash:master Apr 19, 2017
@arcalinea
Copy link
Contributor

arcalinea commented Apr 20, 2017

Are we merging this soon?

And oh ugh, I think I fixed the nondeterministic test failure in the latest commit to #2200, should merge that to prevent this from happening

@daira
Copy link
Contributor

daira commented Apr 20, 2017

@arcalinea yes, it had already merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-networking Area: Networking code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants