-
Notifications
You must be signed in to change notification settings - Fork 492
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
Upgrade to Tor 0.4.8.5 #11501
Upgrade to Tor 0.4.8.5 #11501
Conversation
f1a4930
to
ccb3cc2
Compare
b5966cf
to
99555e2
Compare
How to test this? Run Wasabi and see if it works? |
Yes. Ideally, try to coinjoin. |
@@ -7,7 +7,7 @@ | |||
`git update-index --chmod=+x bitcoind` | |||
`git update-index --chmod=+x .\lin64\Tor\tor` | |||
`git update-index --chmod=+x .\win64\Tor\tor.exe` | |||
`git update-index --chmod=+x .\osx64\Tor\tor.real` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this change about? Changed filename because of Tor Browser release vs Tor release?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no tor.real
file. It's called tor
in the latest release. Presumably they unified it.
EDIT: Today it just works well. One time issue was probably on my side. |
What is your problem exactly? Because I tested this PR and had no issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK on MacOS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested coinjoining, LGTM.
@kristapsk Could you test as well please? Then we would have all OSs covered. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't mind my previous comment. Today it just works perfectly well.
I will quickly test this on Linux also.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tack 99555e2 deb 11
Wow, GJ. |
* Upgrade to the latest Tor * macOS seems to distribute `tor` binary instead of `tor.real` binary * Fix hashes * Make sure that `tor` file is treated as binary file (cherry picked from commit e7a3d66)
This reverts commit d19e877.
Changelog: https://gitlab.torproject.org/tpo/core/tor/-/blob/tor-0.4.8.5/ReleaseNotes:
Related to #10833
This PR upgrades Tor version from
0.4.7.13
to0.4.8.5
to test the new conflux feature and if it has any impact on round disruptions.Changelog
Changelog mentions:
Resources
Older upgrades:
0.4.7.8
#8613