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

Allow the patcher project to open and build on Visual Studio 2022 #281

Merged
merged 11 commits into from
Nov 12, 2023
Merged

Conversation

prolium
Copy link
Contributor

@prolium prolium commented May 21, 2023

.NET Framework 4.0 is deprecated in vs2022 and the targeting packs are no longer available to download from Microsoft.
Targeting v4.6 allows it to be opened and built, also v4.6 was an in-place update for v4.0-v4.5.2: Microsoft .NET Framework 4.6 (Web Installer)

@LennardF1989
Copy link
Member

.NET 4 technically allows compatibility with Windows XP. The next best choice would be 4.5, since that ships with Windows 8 by default. Only starting Windows 10 is .NET 4.6 installed by default.

Not saying this is wrong, but something to consider.

@prolium
Copy link
Contributor Author

prolium commented May 21, 2023

Yeah I guess, maybe 4.5.2 would be a sweet spot between old/new ?
For reference, these are the available targeting packs by default in vs2022 (installer v17.5.4)
image

@AnthonyFuller
Copy link
Contributor

AnthonyFuller commented May 21, 2023

.NET 4 technically allows compatibility with Windows XP.

The game can't even run on XP (and the server itself can't run on XP or Win7), so this is definitely not a concern if it ever even was one.

@RDIL
Copy link
Member

RDIL commented May 21, 2023

Dropping XP support isn't a concern at all. My only concern would be if the relevant runtimes are shipped by default on Win10 and Win11. If not, and this is another download, then I'm -1 on this.

Signed-off-by: Reece Dunham <me@rdil.rocks>
@RDIL RDIL self-requested a review as a code owner May 21, 2023 18:15
Signed-off-by: Reece Dunham <me@rdil.rocks>
@RDIL
Copy link
Member

RDIL commented May 21, 2023

Looks like the CI image doesn't ship with the TP by default either, I don't have a ton of time to dive into the image and see it's contents though so I'm not sure how to proceed.

@RDIL
Copy link
Member

RDIL commented May 21, 2023

Thank you for the PR btw! This is something I've been meaning to do for a few months now 😄

@RDIL RDIL self-assigned this May 21, 2023
@RDIL RDIL added backlog This needs to be reviewed. major This will need to be in a semver-major release. labels May 21, 2023
@prolium
Copy link
Contributor Author

prolium commented May 21, 2023

There's a page from Microsoft detailing which version is preinstalled with the OS: https://learn.microsoft.com/en-us/dotnet/framework/get-started/system-requirements

v4.6 is shipped with Win10 (version 1507)
It's ok if you want to discard this, totally understandable especially for backward compatibility and having less deps.

@TheBoxyBear
Copy link

Where can I acquire the targeting pack? It also works in 4.6 for me but I would have to downgrade before making a PR until this is merged.

@RDIL
Copy link
Member

RDIL commented Oct 27, 2023

@TheBoxyBear the 4.0 targeting pack is shipped with the VS2019 installer - https://learn.microsoft.com/en-us/visualstudio/releases/2019/redistribution#vs2019-download

RDIL added 8 commits October 27, 2023 17:30
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
@RDIL RDIL added this to the v7 milestone Nov 12, 2023
@grappigegovert
Copy link
Member

I trust that the CI works as intended; looks fine otherwise 👍

Copy link
Contributor

@AnthonyFuller AnthonyFuller left a comment

Choose a reason for hiding this comment

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

I concur with @grappigegovert.

@RDIL
Copy link
Member

RDIL commented Nov 12, 2023

Awesome, thank you @prolium!

@RDIL RDIL merged commit 063f6f1 into thepeacockproject:v7 Nov 12, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog This needs to be reviewed. major This will need to be in a semver-major release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants