cypht-org / cypht

Cypht: Lightweight Open Source webmail aggregator [PHP, JS]
http://cypht.org
GNU Lesser General Public License v2.1
949 stars 147 forks source link

composer create-project installing older version #940

Closed ConnectGrid closed 2 months ago

ConnectGrid commented 3 months ago

🐛 Bugreport

Running composer require jason-munro/cypht installs the latest version (1.4.1 at time of this writing). However, calling composer create-project jason-munro/cypht is installing v1.4.0, which has outdated dependencies that don't support PHP 8.3.x.

I'm not familiar with enough the create-project command to know how to fix this.

Version & Environment

Rev: []

OS: []

Steps to reproduce

  1. Run composer create-project jason-munro/cypht

On PHP 8.3, dependency validation fails.

marclaporte commented 3 months ago

@christer77 has volunteered on this one.

Shadow243 commented 2 months ago

@ConnectGrid It could be due to caching.

Did you try this:composer clear-cache and then composer create-project jason-munro/cypht ??

marclaporte commented 2 months ago

On PHP 8.3, dependency validation fails.

What is the exact error message?

Cypht 1.4.x supports minimum PHP 5.6 Cypht 2.0.x (soon to be released) supports minimum PHP 7.4

It could be a pain to support PHP 5.6 to PHP 8.4 so we may want to put a max PHP version for Cypht 1.4.x

Can you try Cypht master? We did some work to support PHP 7.4 to PHP 8.4

christer77 commented 2 months ago

ERROR MESSAGE : The error is not related to cypht but to the htmlpurifier package which was fixed to "^4.13" while this version does not support php 8.3.x, so since 2022-09-16 to fix some bugs including php 8.3.x support htmlpurifier-releases

christer77 commented 2 months ago

ERROR MESSAGE : cypht-jason-munro-error

marclaporte commented 2 months ago

ok, so the next version of HTML Purifier will support PHP 8.4 https://github.com/ezyang/htmlpurifier/pull/396

christer77 commented 2 months ago

Exactly, and this since version 4.17, this means that I am returning the PR because this of which is 4.16 is limited to 8.3

marclaporte commented 2 months ago

@christer77 You need to generate a composer.lock file as well.

christer77 commented 2 months ago

It's noted @marclaporte

marclaporte commented 2 months ago

@christer77 please see comments for https://github.com/cypht-org/cypht/pull/943

christer77 commented 2 months ago

Consideration of comments [#943 ] in PR