-
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
Fix UpdateManager on Linux #13120
Fix UpdateManager on Linux #13120
Conversation
private record ReleaseInfo(Version LatestClientVersion, List<string> AssetDownloadLinks) | ||
{ | ||
public string InstallerDownloadUrl { get; set; } = ""; | ||
public string InstallerFileName { get; set; } = ""; |
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.
I didn't want to deal with Nullable
@@ -248,22 +246,22 @@ private async Task DownloadAndValidateWasabiSignatureAsync(string sha256SumsFile | |||
} | |||
} | |||
|
|||
private (string url, string fileName) GetAssetToDownload(List<string> assetDownloadURLs) | |||
private (string url, string fileName) GetAssetToDownload(List<string> assetDownloadLinks) |
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.
assetDownloadURLs
gives a Warning
@@ -28,7 +28,7 @@ public UpdateManager(TimeSpan period, string dataDir, bool downloadNewVersion, I | |||
GithubHttpClient = githubHttpClient; | |||
WasabiClient = sharedWasabiClient; | |||
// The feature is disabled on linux at the moment because we install Wasabi Wallet as a Debian package. | |||
DownloadNewVersion = downloadNewVersion && !RuntimeInformation.IsOSPlatform(OSPlatform.Linux); | |||
DownloadNewVersion = downloadNewVersion && (RuntimeInformation.IsOSPlatform(OSPlatform.OSX) || RuntimeInformation.IsOSPlatform(OSPlatform.Windows)); |
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.
Here I changed slightly because it better fits with the rest of the logic in GetAssetToDownload
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 on deb12, fixes the problem
(both with auto download enabled and disabled)
Fixes #13118