Faire l'ajout de la config mini dans le paquet.xml d'ecrire #4350

Closed
opened 3 years ago by Franck · 13 comments
Franck commented 3 years ago

Hello :)
Depuis spip 3.2, il est possible de faire l'ajout de necessite (et des "utilises" ?) concernant des lib, donc, je propose que nous profitions de cette fonction pour faire l'ajout dans le paquet.xml https://core.spip.net/projects/spip/repository/entry/spip/ecrire/paquet.xml
L'info est présente à deux endroits https://www.spip.net/fr_article4351.html et en moins complet https://core.spip.net/projects/spip/repository/entry/spip/INSTALL.txt
Je ne suis pas certain que nous puissions mettre toute la config mini (apache) mais déjà, cela réduirait le risque que des gens n'aient pas config mini (même s'il faut reconnaitre qu'il doivent êtes peu nombreux...)

Hello :) Depuis spip 3.2, il est possible de faire l'ajout de necessite (et des "utilises" ?) concernant des lib, donc, je propose que nous profitions de cette fonction pour faire l'ajout dans le paquet.xml https://core.spip.net/projects/spip/repository/entry/spip/ecrire/paquet.xml L'info est présente à deux endroits https://www.spip.net/fr_article4351.html et en moins complet https://core.spip.net/projects/spip/repository/entry/spip/INSTALL.txt Je ne suis pas certain que nous puissions mettre toute la config mini (apache) mais déjà, cela réduirait le risque que des gens n'aient pas config mini (même s'il faut reconnaitre qu'il doivent êtes peu nombreux...)
b_b commented 3 years ago
Owner

Oui ça ne mange pas de pain, mais rien d'urgent car tout ça pourra être pris en charge par composer dans le futur.

Oui ça ne mange pas de pain, mais rien d'urgent car tout ça pourra être pris en charge par composer dans le futur.
b_b commented 3 years ago
Owner

Pour revenir sur la proposition d'augmenter à PHP 5.6 mini je pense qu'on peut le faire sans problème. Pour info, on nécessitait PHP 5.4 mini pour raison de compat avec debian 7, cf r23619 => "On nécessite PHP 5.4 minimum. Note: Debian 7 Wheezy est supportée avec PHP 5.4 jusque fin mai 2018…"

À ce jour, la version mini de PHP sous debian est la 5.6 proposée par debian 8 qui est supportée jusqu'au 30 juin 2020, ref https://wiki.debian.org/fr/LTS & https://packages.debian.org/fr/jessie/php5
Statut changé à En cours

Pour revenir sur la proposition d'augmenter à PHP 5.6 mini je pense qu'on peut le faire sans problème. Pour info, on nécessitait PHP 5.4 mini pour raison de compat avec debian 7, cf r23619 => "On nécessite PHP 5.4 minimum. Note: Debian 7 Wheezy est supportée avec PHP 5.4 jusque fin mai 2018…" À ce jour, la version mini de PHP sous debian est la 5.6 proposée par debian 8 qui est supportée jusqu'au 30 juin 2020, ref https://wiki.debian.org/fr/LTS & https://packages.debian.org/fr/jessie/php5 **Statut changé à En cours**
Owner

Allez on y va pour 5.6 mini sur la 3.3

Allez on y va pour 5.6 mini sur la 3.3
b_b commented 3 years ago
Owner

Champomy pour tout le monde :p

Champomy pour tout le monde :p
Owner

Évidemment +1 pour moi :)

Évidemment +1 pour moi :)
b_b commented 3 years ago
Owner

Je viens d'envoyer la modification pour PHP 5.6 mini, fais péter le patch pour le paquet.xml de SPIP `Francky !

Je viens d'envoyer la modification pour PHP 5.6 mini, fais péter le patch pour le paquet.xml de SPIP `Francky !
Poster

Bon, je vais faire plus simple, car git m'agace...
En lisant les docs:
https://www.spip.net/fr_article4351.html
https://core.spip.net/projects/spip/repository/entry/spip/INSTALL.txt

Je pense qu'il faudrait que nous mettions dans:
https://core.spip.net/projects/spip/repository/entry/spip/ecrire/paquet.xml

Par contre pour les autres je ne sais pas (php-xml, ...) Ce sont des "utilise" mais en faite, je sais pas comment faire :(

En plus, concernant par exemple "Freetype" je pense que sa place est plutôt dans https://zone.spip.org/trac/spip-zone/browser/spip-zone/core/plugins/filtres_images/paquet.xml

J'ajouterais bien dans dump https://zone.spip.org/trac/spip-zone/browser/spip-zone/core/plugins/dump/paquet.xml
Je ne suis même plus sûr que gd2 soit obligatoire en lisant: https://core.spip.net/issues/3171

PS, b_b tu as oublier de mettre 5.6 ici:
https://core.spip.net/projects/spip/repository/entry/spip/INSTALL.txt#L6

Franck

Bon, je vais faire plus simple, car git m'agace... En lisant les docs: https://www.spip.net/fr_article4351.html https://core.spip.net/projects/spip/repository/entry/spip/INSTALL.txt Je pense qu'il faudrait que nous mettions dans: https://core.spip.net/projects/spip/repository/entry/spip/ecrire/paquet.xml <necessite nom="php" compatibilite="[5.6.0;7.4.*]" /> <necessite nom="php:mysqli" compatibilite="[2.0.0;]" /> <necessite nom="php:gd2" compatibilite="[2.0.0;]" /> Par contre pour les autres je ne sais pas (php-xml, ...) Ce sont des "utilise" mais en faite, je sais pas comment faire :( En plus, concernant par exemple "Freetype" je pense que sa place est plutôt dans https://zone.spip.org/trac/spip-zone/browser/spip-zone/_core_/plugins/filtres_images/paquet.xml J'ajouterais bien <necessite nom="php:pdo_sqlite" compatibilite="[;]" /> dans dump https://zone.spip.org/trac/spip-zone/browser/spip-zone/_core_/plugins/dump/paquet.xml Je ne suis même plus sûr que gd2 soit obligatoire en lisant: https://core.spip.net/issues/3171 PS, b_b tu as oublier de mettre 5.6 ici: https://core.spip.net/projects/spip/repository/entry/spip/INSTALL.txt#L6 Franck

Appliqué par commit r24453.
Statut changé à Fermé

Appliqué par commit r24453. **Statut changé à Fermé**
b_b commented 3 years ago
Owner

Intégré donc, à noter que php:gd2 n'existe pas, c'est php:gd, et que php:mysqli n'existe pas du tout, sinon j'ai aussi modifié le fichier INSTALL.txt.

Intégré donc, à noter que php:gd2 n'existe pas, c'est php:gd, et que php:mysqli n'existe pas du tout, sinon j'ai aussi modifié le fichier INSTALL.txt.
b_b commented 3 years ago
Owner

Bon, il semblerait que ça génère des effets de bord lors de l'installation cf #4409

En attendant que le core prenne ces nécessites correctement en charge, je propose d'annuler les modifications en question.
Statut changé à En cours

Bon, il semblerait que ça génère des effets de bord lors de l'installation cf #4409 En attendant que le core prenne ces nécessites correctement en charge, je propose d'annuler les modifications en question. **Statut changé à En cours**
b_b commented 3 years ago
Owner

Et voilà, revert done.
Statut changé à Fermé

Et voilà, revert done. **Statut changé à Fermé**
b_b commented 3 years ago
Owner

Et un revert de plus avec r24465 car le paquet.xml du core semble tellement dérogatoire que l'ajout d'un necessite sur une version de PHP pétait le menu dans le privé...

Et un revert de plus avec r24465 car le paquet.xml du core semble tellement dérogatoire que l'ajout d'un necessite sur une version de PHP pétait le menu dans le privé...
b_b commented 3 years ago
Owner

wontfix donc

wontfix donc
Sign in to join this conversation.
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.