rotdrop / nextcloud-app-files-archive

Archive inspection and extraction as Nextcloud app.
Other
9 stars 4 forks source link

Your Composer dependencies require a PHP version ">= 8.0.2". #19

Closed user023456789 closed 1 year ago

user023456789 commented 1 year ago

Hello, Not really an issue but much more a prerequisite verification issue, it seems that some dependencies could not be resolved. On Debian stable this composer version is not available for the moment. By installing the files_archive package the whole Nextcloud instance is broken until 'php occ app:remove files_archive' execution. Regards

rotdrop commented 1 year ago

Actually, I am not sure whether the beast still works with php 7.

Also, as Nextcloud comes with pre-installed PHP packages in its 3rdparty/ directory there is always the danger that things break.

One way out would be have different releases for the different Nextcloud versions which address mismatching PHP dependencies in a per-nextcloud-version manner.

Can you please report about the Nextcloud and PHP versions you are actually using?

Thanks for reporting!

Claus

rotdrop commented 1 year ago

BTW, there is also this PPA

https://launchpad.net/~ondrej/+archive/ubuntu/php

from which you can install PHP versions which are less outdated than the ones installed by Debian (the PPA also supports Debian stable and oldstable).

adegans commented 1 year ago

This hit me too, when I was still on PHP7.4... Add better dependency checks please.

rotdrop commented 1 year ago

This hit me too, when I was still on PHP7.4... Add better dependency checks please.

Thanks for reporting problems: but: this is not helpful "better dependency checks". What do you mean this phrase?

adegans commented 1 year ago

When I install the app it says nowhere that it needs PHP 8.0.2. Instead it just activated and broke my Nextcloud because I was still on php7.4. That shouldn’t be possible, or at least it should be a warning in the description of the app.

rotdrop commented 1 year ago

Actually, problem here is that info.xml claims to support 7.4. I have changed this now 8.0 which should render the app as not installable (at least not through the GUI) on PHP 7.4 system.

The composer file is not so much the problem here but rather the code which presumably uses features that are not present PHP 7.4. As least I assume that this is the case. More could only be concluded by a look at your Nextcloud or web server logs.

I am closing this bug now as resolved per 3b370931aedc5a2484a8528a6bfba4f6b5e7fb79