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

chore: drop support for Windows 7 / 8 / 8.1 #36427

Merged
merged 11 commits into from
Dec 1, 2022

Conversation

miniak
Copy link
Contributor

@miniak miniak commented Nov 22, 2022

Description of Change

https://support.google.com/chrome/thread/185534985/sunsetting-support-for-windows-7-8-8-1-in-early-2023?hl=en
https://www.electronjs.org/blog/windows-7-to-8-1-deprecation-notice

Closes #35597

Checklist

Release Notes

Notes: Windows 7, Windows 8, and Windows 8.1 are not supported anymore as Chromium 110 dropped support.

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Nov 22, 2022
@miniak miniak changed the title chore: statically link Windows APIs available on Windows 10+ chore: drop support for Windows 7 & 8 Nov 22, 2022
@miniak miniak force-pushed the miniak/drop-win-7-8-support branch 2 times, most recently from c0043a7 to 67736f8 Compare November 22, 2022 20:44
@miniak miniak requested a review from a team as a code owner November 22, 2022 20:44
@miniak miniak force-pushed the miniak/drop-win-7-8-support branch 3 times, most recently from b1eb6d0 to 5fa490e Compare November 22, 2022 20:52
Copy link
Member

@nornagon nornagon left a comment

Choose a reason for hiding this comment

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

Needs an update to breaking-changes.md and a blog post, I think!

@miniak
Copy link
Contributor Author

miniak commented Nov 22, 2022

Needs an update to breaking-changes.md and a blog post, I think!

@nornagon I will do that. How would you formulate it?

docs/breaking-changes.md Outdated Show resolved Hide resolved
Copy link
Member

@VerteDinde VerteDinde left a comment

Choose a reason for hiding this comment

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

Left one nit comment, but otherwise looks good!

docs/breaking-changes.md Outdated Show resolved Hide resolved
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
@miniak miniak requested a review from nornagon November 28, 2022 17:32
Copy link
Member

@nornagon nornagon left a comment

Choose a reason for hiding this comment

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

Let's also plan to write a blog post about this. cc @erickzhao and @electron/wg-ecosystem more generally

README.md Outdated Show resolved Hide resolved
Co-authored-by: Jeremy Rose <jeremya@chromium.org>
README.md Outdated Show resolved Hide resolved
Copy link
Member

@nornagon nornagon left a comment

Choose a reason for hiding this comment

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

let's update this with a link to our blog post once we have it. electron/website#319

@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Nov 29, 2022
@miniak miniak changed the title chore: drop support for Windows 7 & 8 chore: drop support for Windows 7 / 8 / 8.1 Nov 30, 2022
@miniak
Copy link
Contributor Author

miniak commented Nov 30, 2022

@nornagon the blog post is up https://www.electronjs.org/blog/windows-7-to-8-1-deprecation-notice. Should I replace the Chromium support links with this one? Or keep both?

README.md Outdated Show resolved Hide resolved
docs/breaking-changes.md Outdated Show resolved Hide resolved
miniak and others added 2 commits November 30, 2022 22:26
Co-authored-by: Jeremy Rose <jeremya@chromium.org>
@VerteDinde VerteDinde added target/23-x-y PR should also be added to the "23-x-y" branch. and removed no-backport labels Dec 1, 2022
@VerteDinde VerteDinde merged commit eb29148 into main Dec 1, 2022
@VerteDinde VerteDinde deleted the miniak/drop-win-7-8-support branch December 1, 2022 01:13
@release-clerk
Copy link

release-clerk bot commented Dec 1, 2022

Release Notes Persisted

Windows 7, Windows 8, and Windows 8.1 are not supported anymore as Chromium 110 dropped support.

@trop
Copy link
Contributor

trop bot commented Dec 1, 2022

I have automatically backported this PR to "23-x-y", please check out #36513

@trop trop bot added in-flight/23-x-y merged/23-x-y PR was merged to the "23-x-y" branch. and removed target/23-x-y PR should also be added to the "23-x-y" branch. in-flight/23-x-y labels Dec 1, 2022
@GitMensch
Copy link

What does this mean for Windows Server 2012 R2 ?

@pushkin-
Copy link

pushkin- commented Dec 15, 2022

@nornagon @miniak Can you confirm my assumption that Electron 23+ still run on Windows Server 2012 R2 after the loss of support, but it just won't get patches and might run into issues down the line.

@CommonLoon102
Copy link

@nornagon @miniak Can you confirm my assumption that Electron 23+ still run on Windows Server 2012 R2 after the loss of support, but it just won't get patches and might run into issues down the line.

Windows Server was never mentioned in the README.md as a supported OS.

@miniak
Copy link
Contributor Author

miniak commented Jan 23, 2023

Windows Server 2012 R2 is actually the same as Windows 8.1 and I can confirm that Electron 23 no longer works there most likely due to changes in Chromium.

@miniak
Copy link
Contributor Author

miniak commented Jan 23, 2023

v23.0.0-alpha.1 (Chromium 110.0.5415.0) still runs on Windows 8.1
v23.0.0-alpha.2 (Chromium 110.0.5451.0) crashes on Windows 8.1

@miniak
Copy link
Contributor Author

miniak commented Jan 23, 2023

There is an issue tracking this #36909, let's continue the discussion there

@miniak miniak mentioned this pull request Jan 24, 2023
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
* chore: drop support for Windows 7 & 8

* chore: remove disable-redraw-lock.patch

* chore: update patches

* Update docs/breaking-changes.md

Co-authored-by: Erick Zhao <erick@hotmail.ca>

* Update docs/breaking-changes.md

Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>

* fix breaking-changes.md

* chore: note last supported version

Co-authored-by: Jeremy Rose <jeremya@chromium.org>

* chore: add link to deprecation policy

* Update docs/breaking-changes.md

Co-authored-by: Jeremy Rose <jeremya@chromium.org>

* update README.md

Co-authored-by: Milan Burda <miburda@microsoft.com>
Co-authored-by: PatchUp <73610968+patchup[bot]@users.noreply.github.com>
Co-authored-by: Erick Zhao <erick@hotmail.ca>
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
Co-authored-by: Jeremy Rose <jeremya@chromium.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-review/requested 🗳 merged/23-x-y PR was merged to the "23-x-y" branch. semver/major incompatible API changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Review and clarify the Windows version support
8 participants