add a setup check for 64 bit

[szaimen]

See nextcloud/documentation#9071

Signed-off-by: szaimen [email protected]

[szaimen]

/rebase

[szaimen]

@MichaIng this check will only be present on NC25 and above and is not going to be backported.

[MichaIng]

Makes sense. We should link the related docs section as well then. I'll add a list of known issues with 32-bit systems there.

But keep in mind that this likely leads to user reports like "Why suddenly my system is not supported anymore?"/"How can I mute the warning?" 😉.

[szaimen]

But keep in mind that this likely leads to user reports like "Why suddenly my system is not supported anymore?"/"How can I mute the warning?" 😉.

That is why we will not backport this change

[szaimen]

We should link the related docs section as well then.

good idea!

[MichaIng]

That is why we will not backport this change

But users will upgrade their Nextcloud instance and then see the warning, and at least I'd be very unhappy if the upgrade would be denied on my little RPi 2 home server 😉. But yeah, major upgrade means breaking changes/support changes are possible.

[szaimen]

Thats why we are not actively blocking the upgrade. Only showing this warning then...

[PVince81]

👍

[PVince81]

in theory the check could also be added in the updater in NC < 25 close to the update button so that people get a warning before clicking "update". of course only applies for web updater, not CLI phar update or other ways

[MichaIng]

in theory the check could also be added in the updater in NC < 25 close to the update button so that people get a warning before clicking "update".

That could be irritating since nothing changed about code-wise support but only a new warning is shown, which shouldn't be a reason to not update.

Or is there a plan to stop end user support when a 32-bit system is used, also not related to the known issues with it, exactly with NC25, while keeping it with NC24?

The requirements update in docs has been backported, which answers this already.

[szaimen]

in theory the check could also be added in the updater in NC < 25 close to the update button so that people get a warning before clicking "update". of course only applies for web updater, not CLI phar update or other ways

I would rather not do this. Having the setup check is enough, imo.

[szaimen]

I'll do the link to the docs in a follow-up

[szaimen]

CI failure unrelated