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

Update Minimum Windows Server OS to Server 2008 R2 #2903

Closed
TheCakeIsNaOH opened this issue Nov 12, 2022 · 8 comments · Fixed by #3143
Closed

Update Minimum Windows Server OS to Server 2008 R2 #2903

TheCakeIsNaOH opened this issue Nov 12, 2022 · 8 comments · Fixed by #3143

Comments

@TheCakeIsNaOH
Copy link
Member

Is Your Feature Request Related To A Problem? Please describe.

As per the supported operating systems for .Net 4.8, Windows 7 or newer and Windows Server 2008 R2 or newer are supported.
https://learn.microsoft.com/en-us/dotnet/framework/get-started/system-requirements

Currently, from everywhere I have looked, Windows 7 is listed the minimum support Windows Client OS for Chocolatey CLI so that is good to go.

However, the same does not seem to go for Window Server, as Server 2003 or 2008 seem to be the minimum version listed, so things will have to be updated at or ahead of the 2.0 release.

Describe The Solution. Why is it needed?

There are a number of locations to update in this repository:

<!-- Windows Vista / Windows Server 2008 -->
<supportedOS Id="{e2011457-1546-43c5-a5fe-008deee3d3f0}"/>

* Windows Server 2003+ / Windows 7+

case "6.0":
name = isServer ? "Windows Server 2008" : "Windows Vista";
break;
case "5.2":
name = isServer ? "Windows Server 2003" : "Windows XP";
break;
case "5.1":
name = "Windows XP";
break;
case "5.0":
name = "Windows 2000";
break;

I think I got them, but I easily could have missed somewhere.

Additional Context.

Additional issues should be opened for other locations that may have minimum supported OS information:

  • Installation page on chocolatey.org
  • Installation course on chocolatey.org
  • Docs repository
  • Internal locations?

Related Issues

@pauby
Copy link
Member

pauby commented Nov 25, 2022

For clarity, we support Windows Server 2008 R2 in Azure as that is what is currently supported by Microsoft, up to January 2024.

@pauby
Copy link
Member

pauby commented Mar 16, 2023

In the README, we should remove the supported operating systems and point to the docs instead. This gives us one place to update this as it moves forward.

@vexx32
Copy link
Member

vexx32 commented Mar 16, 2023

I agree that it's not ideal that we have this documented in multiple places...

However, if we're going to point folks to a doc page instead of embedding information in the package, we'll need to ensure that the doc page is very explicit about which OSes are supported by which versions of the package, so that folks aren't misled if they're installing a slightly older package and going off the documentation which perhaps might be more recently updated than the package itself.

For example, say in future (choco v3.x?) we dropped support for Windows Server 2008(R2) entirely and then updated our docs to say that. Folks installing v2.x who're looking at that doc might think that also applies to v2.x when that's not the case.

@pauby
Copy link
Member

pauby commented Mar 16, 2023

If we do that, then we are going to have the issue of what the package supports and what Chocolatey Software supports.

In the example:

For example, say in future (choco v3.x?) we dropped support for Windows Server 2008(R2) entirely and then updated our docs to say that. Folks installing v2.x who're looking at that doc might think that also applies to v2.x when that's not the case.

We wouldn't support 2.x of Chocolatey CLI for open-source users, even though the software may actually run. In the same way that Chocolatey CLI currently works on Windows 7 (for some users) we do not support it.

@vexx32
Copy link
Member

vexx32 commented Mar 16, 2023

That makes sense, I guess I misunderstood a bit what we were talking about here, sorry. 🙂

@TheCakeIsNaOH
Copy link
Member Author

TheCakeIsNaOH commented Mar 16, 2023

If we do that, then we are going to have the issue of what the package supports and what Chocolatey Software supports.

It may be worth having two pieces of information separating out what is "supported" vs what is likely "compatible". That way, people who want to go off and run on unsupported operating systems will have some idea of what is possible.

@pauby
Copy link
Member

pauby commented Mar 17, 2023

"compatible" says "supported" to me. If something is "compatible" then, it should work. When it doesn't people will understandably ask questions as to why. If we don't support it, what do we say?

I know this sounds like its terminology, but I feel we have to set expectations clearly. There may be a way to do that, providing both.

@vexx32 vexx32 self-assigned this Apr 28, 2023
vexx32 added a commit to vexx32/choco that referenced this issue Apr 28, 2023
Update docs and templates referring to OSes, and application manifest to
put 2008r2 as the minimum supported OS.
@gep13 gep13 linked a pull request May 2, 2023 that will close this issue
10 tasks
vexx32 added a commit to vexx32/choco that referenced this issue May 2, 2023
Update docs and templates referring to OSes, and application manifest to
put 2008r2 as the minimum supported OS.
vexx32 added a commit to vexx32/choco that referenced this issue May 2, 2023
Update docs and templates referring to OSes, and application manifest to
put 2008r2 as the minimum supported OS.
corbob pushed a commit to vexx32/choco that referenced this issue May 10, 2023
Update docs and templates referring to OSes, and application manifest to
put 2008r2 as the minimum supported OS.
corbob added a commit that referenced this issue May 10, 2023
@vexx32 vexx32 closed this as completed May 17, 2023
@choco-bot
Copy link

🎉 This issue has been resolved in version 2.0.0 🎉

The release is available on:

Your GitReleaseManager bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants