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

Release v1.0.13-rc1 #2733

Merged
merged 4 commits into from
Nov 15, 2017
Merged

Release v1.0.13-rc1 #2733

merged 4 commits into from
Nov 15, 2017

Conversation

str4d
Copy link
Contributor

@str4d str4d commented Nov 15, 2017

No description provided.

@str4d str4d added the C-release Category: This is an issue corresponding to the creation of a specific release. label Nov 15, 2017
Copy link
Contributor

@nathan-at-least nathan-at-least left a comment

Choose a reason for hiding this comment

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

utACK. Some follow up work requested, but no blockers for this PR.

@@ -8,7 +8,7 @@
// Deprecation policy:
// * Shut down 18 weeks' worth of blocks after the estimated release block height.
Copy link
Contributor

Choose a reason for hiding this comment

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

[DOES NOT BLOCK] Looks like we missed #2686 for this release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, I was going to do that in the actual 1.0.13 release PR, but I can trivially do it in this PR instead (just requires one more commit).

.PP
In order to ensure you are adequately protecting your privacy when using Zcash,
please see <https://z.cash/support/security/index.html>.
please see <https://z.cash/support/security/>.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this a manual change outside of make-release.py? I prefer to separate make-release.py changes from manual changes in the future, but for now this is acceptable. (If it's somehow the result of make-release.py that would also be notable and maybe a bug.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is part of the make-release.py change - it updates man pages, which pull from changes to the help text in init.cpp.

@@ -68,7 +68,7 @@ Timeout in seconds during HTTP requests, or 0 for no timeout. (default:
.SH COPYRIGHT

In order to ensure you are adequately protecting your privacy when using Zcash,
please see <https://z.cash/support/security/index.html>.
please see <https://z.cash/support/security/>.
Copy link
Contributor

Choose a reason for hiding this comment

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

See above.

@@ -86,7 +86,7 @@ Set register NAME to given JSON\-STRING
.SH COPYRIGHT

In order to ensure you are adequately protecting your privacy when using Zcash,
please see <https://z.cash/support/security/index.html>.
please see <https://z.cash/support/security/>.
Copy link
Contributor

Choose a reason for hiding this comment

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

See above.

@@ -462,7 +462,7 @@ console, 600 otherwise)
.SH COPYRIGHT

In order to ensure you are adequately protecting your privacy when using Zcash,
please see <https://z.cash/support/security/index.html>.
please see <https://z.cash/support/security/>.
Copy link
Contributor

Choose a reason for hiding this comment

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

See above (all changes to this URL).

Copy link
Contributor

@nathan-at-least nathan-at-least left a comment

Choose a reason for hiding this comment

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

utACK on the #2686 change.

// * A warning is shown during the 2 weeks' worth of blocks prior to shut down.
static const int APPROX_RELEASE_HEIGHT = 219700;
static const int WEEKS_UNTIL_DEPRECATION = 18;
static const int WEEKS_UNTIL_DEPRECATION = 16;
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@str4d
Copy link
Contributor Author

str4d commented Nov 15, 2017

@zkbot r+

@zkbot
Copy link
Contributor

zkbot commented Nov 15, 2017

📌 Commit 66a9fd8 has been approved by str4d

@zkbot
Copy link
Contributor

zkbot commented Nov 15, 2017

⌛ Testing commit 66a9fd8 with merge c1282be...

zkbot added a commit that referenced this pull request Nov 15, 2017
@zkbot
Copy link
Contributor

zkbot commented Nov 15, 2017

☀️ Test successful - pr-merge
Approved by: str4d
Pushing c1282be to master...

@zkbot zkbot merged commit 66a9fd8 into zcash:master Nov 15, 2017
@daira
Copy link
Contributor

daira commented Nov 15, 2017

Post-hoc ACK

@daira daira added this to the 1.0.13 milestone Nov 15, 2017
@str4d str4d deleted the release-v1.0.13-rc1 branch January 27, 2023 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-release Category: This is an issue corresponding to the creation of a specific release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants