JoliNotif icon indicating copy to clipboard operation
JoliNotif copied to clipboard

RFC on removing support for Windows < 10

Open pyrech opened this issue 2 months ago • 4 comments

Windows 7 is EOL since 01/2020 and Windows 8.1 since 01/2023.

So I propose to remove support for Windows 8 and lower.

Since #127, we now have binary-less driver for Windows and WSL. So we can already remove the wsl-notify-send driver (wsl only) and its 3.36 MB binary that does not bring anything compared to #127.

But, if we remove support for Windows < 10, it will also allow us to:

  • remove the Notifu driver (windows 7 only) and its 200kb binary
  • remove the SnoreToast driver (windows>= 8) and its 1.97 MB binary

Then Windows support (either true Windows or WSL) would rely on a unique powershell-based driver, not requiring to package binaries inside JoliNotif, making it sooooo much lighter, and same for Castor.

Any opinions on this?

pyrech avatar Oct 11 '25 09:10 pyrech

So I propose to remove support for Windows 8 and lower.

yes 👍🏼

But, if we remove support for Windows < 10, it will also allow us to:

I don't know enough windows to answer. But it looks like it's still used a lot :/ https://gs.statcounter.com/os-version-market-share/windows/desktop/worldwide/

Anyway, I'm not against removing support for windows 10. Old version of this lib are still available :)

lyrixx avatar Oct 16 '25 09:10 lyrixx

Oh, < 10, and not <= 10.

Let's drop all deprecated things !

lyrixx avatar Oct 16 '25 09:10 lyrixx

I think JoliNotif is widely used, outside Castor as well.

So aiming for maximum support is important IMO - there is no point always pushing the latest version on a library like this. But it's already not the case, for example it requires PHP 8.2 for some reasons?!

PHP 8.3 will de facto makes this not usable in Windows 7 according to https://github.com/php/php-src/issues/12762.

damienalexandre avatar Oct 16 '25 09:10 damienalexandre

there is no point always pushing the latest version on a library like this.

Size of the library matter!

For example it requires PHP 8.2 for some reasons?!

8.2 is not supported anymore https://www.php.net/supported-versions.php

lyrixx avatar Oct 16 '25 09:10 lyrixx