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

Change Releases warning Message to "will require >PHP8.0 or <PHP8.2 " #12357

Open
2 tasks done
nivadis opened this issue Jan 16, 2023 · 18 comments
Open
2 tasks done

Change Releases warning Message to "will require >PHP8.0 or <PHP8.2 " #12357

nivadis opened this issue Jan 16, 2023 · 18 comments
Assignees

Comments

@nivadis
Copy link

nivadis commented Jan 16, 2023

Debug mode

Describe the bug

please change the messeage of the releases to >PHP8.0 or <PHP8.2 and not PHP8.0 or greater

the cli with php8.2 gave the hint to avoid php8.2 ;-)

thanks

Reproduction steps

...

Expected behavior

PHP8.0 or <PHP8.2 and not PHP8.0 or greater

Screenshots

No response

Snipe-IT Version

v6.0.14 build 9161 (g799c9c910)

Operating System

debian

Web Server

apache

PHP Version

8.2.X

Operating System

No response

Browser

No response

Version

No response

Device

No response

Operating System

No response

Browser

No response

Version

No response

Error messages

No response

Additional context

No response

@welcome
Copy link

welcome bot commented Jan 16, 2023

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

@nivadis nivadis changed the title Change Releases warning Message to "will require PHP8.0 or <PHP8.2 " Change Releases warning Message to "will require >PHP8.0 or <PHP8.2 " Jan 16, 2023
@snipe
Copy link
Owner

snipe commented Jan 17, 2023

What message are you getting with PHP8.2? It should work with 8.2 (should being the operative word here.)

@uberbrady
Copy link
Collaborator

composer install does not work on 8.2 right now, I can confirm that on my local. Let me see if I can twiddle some stuff.

@p-rintz
Copy link

p-rintz commented Feb 9, 2023

Without running "composer update", the following will error out:

Installing dependencies from lock file
Verifying lock file contents can be installed on current platform.
Your lock file does not contain a compatible set of packages. Please run composer update.

  Problem 1
    - nette/schema is locked to version v1.2.2 and an update of this package was not requested.
    - nette/schema v1.2.2 requires php >=7.1 <8.2 -> your php version (8.2.2) does not satisfy that requirement.
  Problem 2
    - nette/utils is locked to version v3.2.7 and an update of this package was not requested.
    - nette/utils v3.2.7 requires php >=7.2 <8.2 -> your php version (8.2.2) does not satisfy that requirement.
  Problem 3
    - phpspec/prophecy is locked to version v1.15.0 and an update of this package was not requested.
    - phpspec/prophecy v1.15.0 requires php ^7.2 || ~8.0, <8.2 -> your php version (8.2.2) does not satisfy that requirement.
  Problem 4
    - nette/schema v1.2.2 requires php >=7.1 <8.2 -> your php version (8.2.2) does not satisfy that requirement.
    - league/config v1.1.1 requires nette/schema ^1.2 -> satisfiable by nette/schema[v1.2.2].
    - league/config is locked to version v1.1.1 and an update of this package was not requested.

@KorvinSzanto
Copy link
Contributor

@snipe is the fix at this point to run composer update with php8.2 or to downgrade to PHP8.1?

@carbon-evolution
Copy link

carbon-evolution commented Aug 2, 2023

Is the issue resolved?
Getting same error as of @p-rintz.

PHP 8.2.8 (cli) (built: Jul 8 2023 07:10:21) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.2.8, Copyright (c) Zend Technologies
with Zend OPcache v8.2.8, Copyright (c), by Zend Technologies

Problem 1
- Root composer.json requires php >=7.4.3 <8.2 but your php version (8.2.8) does not satisfy that requirement.
Problem 2
- nette/schema is locked to version v1.2.2 and an update of this package was not requested.
- nette/schema v1.2.2 requires php >=7.1 <8.2 -> your php version (8.2.8) does not satisfy that requirement.
Problem 3
- nette/utils is locked to version v3.2.7 and an update of this package was not requested.
- nette/utils v3.2.7 requires php >=7.2 <8.2 -> your php version (8.2.8) does not satisfy that requirement.
Problem 4
- phpspec/prophecy is locked to version v1.15.0 and an update of this package was not requested.
- phpspec/prophecy v1.15.0 requires php ^7.2 || ~8.0, <8.2 -> your php version (8.2.8) does not satisfy that requirement.
Problem 5
- nette/schema v1.2.2 requires php >=7.1 <8.2 -> your php version (8.2.8) does not satisfy that requirement.
- league/config v1.1.1 requires nette/schema ^1.2 -> satisfiable by nette/schema[v1.2.2].
- league/config is locked to version v1.1.1 and an update of this package was not requested.

@uberbrady
Copy link
Collaborator

Your best bet is to downgrade to php8.1

@KiralyCraft
Copy link

I'm running PHP 8.2.10 on Arch Linux. I've modified composer.json to edit the restrictions for < 8.2.11 instead of < 8.2, ran the composer command with "update" instead of "install ....", and then installed as indicated (this may be redundant though).

So far it works wonders, I haven't found any issues with it.

@CharmingRhino
Copy link

I'm running PHP 8.2.10 on Arch Linux. I've modified composer.json to edit the restrictions for < 8.2.11 instead of < 8.2, ran the composer command with "update" instead of "install ....", and then installed as indicated (this may be redundant though).

So far it works wonders, I haven't found any issues with it.

Which lines in the composer.json file did you have to change?

@KiralyCraft
Copy link

Which lines in the composer.json file did you have to change?

This one

@nemchik
Copy link

nemchik commented Jan 4, 2024

Just chiming in; It was pointed out above that this line

"php": ">=7.4.3 <8.2",
prevents using php 8.2 (and now also php 8.3) but actually still allows using php 7.4 (EOL) and php 8.0 (EOL).

Per https://www.php.net/supported-versions.php as of writing, php 8.1 (the only version of php that is not EOL that works to install snipe-it) is no longer in active support. It is still receiving security updates and is not yet EOL, but it will be as of 25 Nov 2024. The two versions of php currently in active support are php 8.2 and php 8.3.

The composer.json file should be updated to reflect the supported versions of php if php 8.2 is meant to be included. Whether EOL versions of php are supported is up to the project to decide, but if the only versions of php that a project supports are outside of php's active support window it paints a bit of a negative picture.

@snipe an update to the composer.json would be greatly appreciated if the supported php version range has changed.

As a side note, I am also involved in maintaining https://github.com/linuxserver/docker-snipe-it and have been unable to make a few changes due to being locked in with the older (now EOL) php 8.1.

GitHub
Alpine/Nginx container for the Asset Management software Snipe-IT - GitHub - linuxserver/docker-snipe-it: Alpine/Nginx container for the Asset Management software Snipe-IT

@snipe
Copy link
Owner

snipe commented Jan 4, 2024

@nemchik Supported versions have not been changed. Snipe-IT v7 will phase out 7.4, but we're stuck with supporting it until that comes out.

@vithusel
Copy link

@snipe

Im currently using php 8.2.10 and have to perform the upgrades manually which isn't an issue in the short term.

Will Snipe-IT v7 introduce support to be able to upgrade with this PHP version? I'm questioning if life would be made easier downgrading PHP or if it makes more sense to just wait.

The most recent version of upgrade.php introduces a a requirements JSON file which only supports 8.2.0 any newer minor versions aren't supported.

@snipe
Copy link
Owner

snipe commented Jan 28, 2024

@vithusel Yeah, that new file is to make upgrading easier moving forward. Since we don’t know what version someone is upgrading from/to, this at least gives us a way to stop people from blowing up their installations if they missed the PHP requirements (both max and min) in the documentation. We’re still refining that file though, since it’s not actually used yet.

8.2 isn’t supported in the Snipe-IT v6 line (due to dependencies, not because we don’t want to), but v7 will be.

@vithusel
Copy link

@vithusel Yeah, that new file is to make upgrading easier moving forward. Since we don’t know what version someone is upgrading from/to, this at least gives us a way to stop people from blowing up their installations if they missed the PHP requirements (both max and min) in the documentation. We’re still refining that file though, since it’s not actually used yet.

8.2 isn’t supported in the Snipe-IT v6 line (due to dependencies, not because we don’t want to), but v7 will be.

Thank you for the confirmation. 8.2.10 seems to be playing well for me so perhaps I'm narrowly avoiding a particular feature that's broken with it. I'll freeze my servers for now and will wait till v7 to upgrade

@pvsaintpe
Copy link

image

@pvsaintpe
Copy link

good night, can you do something about this?
Laravel 11 works with php8.2/8.3 why can't I install dependencies for php 8.2/8.3 and Laravel 11?

@snipe
Copy link
Owner

snipe commented Apr 29, 2024

@pvsaintpe - per the requirements documentation, we do not yet support PHP 8.2. We will in Snipe-IT v7 which should be out soon.

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

No branches or pull requests