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

Feature/upgrade to net8 #5111

Merged
merged 416 commits into from Apr 21, 2024
Merged

Feature/upgrade to net8 #5111

merged 416 commits into from Apr 21, 2024

Conversation

kenkendk
Copy link
Member

@kenkendk kenkendk commented Mar 3, 2024

Builds on #5107 and upgrades all projects to .Net8.
The code builds, but Cocoa support has been removed.
The code is not tested.

tsuckow and others added 30 commits April 1, 2021 20:31
* Also don't show a terminal window for the trayicon
Moved things around to make cleaner build files.
Implementing everything with `netstandard2.1` and then thin wrapper executables using .net5
…tstandard2.0` to avoid excessive warnings on build.

Still needs to include he native files.
Previously, we would have inconsistent paths to the executables (net5
vs. net5.0).  For example,

Executables/net5/Duplicati.CommandLine.BackendTester/bin/Debug/net5.0
Executables/net5/Duplicati.CommandLine/bin/Debug/net5

With these changes, the path to the Duplicati.CommandLine executables
becomes

Executables/net5/Duplicati.CommandLine/bin/Debug/net5.0
Use consistent target framework for .NET 5 entry executables
This fixes some additional issues in the experimental/net5-split branch:

* Replace call to Thread.Abort.
* Fix retrieval of embedded resource.
* Replace DNS client (DnsLite doesn't work on Mac or Linux with .NET 5).
* Avoid opening terminal when running TrayIcon project on Windows.
This fixes #4527.
The use of a backslash as a path separator only works for Windows.

This fixes #4528.
Remove unused installer things.
Added version numbers to MSI build.
@kenkendk kenkendk marked this pull request as ready for review April 21, 2024 07:34
@kenkendk kenkendk merged commit a31a288 into master Apr 21, 2024
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants