Pour faire fonctionner Imagick sous Windows, il faut ajouter realpath #5209

Closed
RealET wants to merge 4 commits from <deleted>:4.1 into 4.1
RealET commented 4 weeks ago

Le cas est sans doute anecdotique, mais j'en ai eu besoin pour tester Imagick en local

Le cas est sans doute anecdotique, mais j'en ai eu besoin pour tester Imagick en local
RealET added 1 commit 4 weeks ago
Owner

Je n’ai aucun moyen de vérifier cela.

Mais effectivement d’autres personnes l’indiquent, mais c’est il y a 10 ans sur PHP 5.4. Ça n’a pas changé depuis ?

Est-ce qu’on est sûr que ça n’impacte pas les autres OS ? Tu avais regardé Real3t aussi ?
D’autres avis dessus ?

Et par ailleurs il faudrait proposer en master, pas 4.1 la PR :)

Je n’ai aucun moyen de vérifier cela. Mais effectivement d’autres personnes l’indiquent, mais c’est il y a 10 ans sur PHP 5.4. Ça n’a pas changé depuis ? - https://www.php.net/manual/en/imagick.construct.php#108764 - https://stackoverflow.com/questions/9954764/imagick-php-5-4-extension-does-not-work-with-relative-paths-windows Est-ce qu’on est sûr que ça n’impacte pas les autres OS ? Tu avais regardé Real3t aussi ? D’autres avis dessus ? Et par ailleurs il faudrait proposer en master, pas 4.1 la PR :)
Poster

Ben j'ai testé sous Windows avec PHP 8.1.

Je n'ai pas de SPIP 4.1 installé sous Linux :(

Je suis curieux : comment je teste en 4.1 et pourrais faire une PR en master ? (à part aller directement utiliser l'éditeur de code source de Gitea) ?

Ben j'ai testé sous Windows avec PHP 8.1. Je n'ai pas de SPIP 4.1 installé sous Linux :( Je suis curieux : comment je teste en 4.1 et pourrais faire une PR en master ? (à part aller directement utiliser l'éditeur de code source de Gitea) ?
RealET added 1 commit 3 weeks ago
Poster

Par ailleurs, on a déjà un exemple dans SPIP qui fonctionne:
https://git.spip.net/spip/filtres_images/src/branch/master/filtres/images_typo.php#L524

Et d'ailleurs, il restait ça à faire : spip/filtres_images#4700

Par ailleurs, on a déjà un exemple dans SPIP qui fonctionne: https://git.spip.net/spip/filtres_images/src/branch/master/filtres/images_typo.php#L524 Et d'ailleurs, il restait ça à faire : https://git.spip.net/spip/filtres_images/pulls/4700
Owner

J’ai testé que ça cassait fonctionnait encore sans souci apparents.
Donc hop, c’est fusionné.

J’ai testé que ça cassait fonctionnait encore sans souci apparents. Donc hop, c’est fusionné.
marcimat closed this pull request 3 weeks ago
Owner

J’ai testé trop rapidement. J’ai bien des soucis

Ça va pas sur writeImage

On cherche à écrire une vignette, donc par défaut elle n’existe pas, donc realpath($vignette) retourne false.

Tu avais vraiment pu faire fonctionner ça ???

J’ai testé trop rapidement. J’ai bien des soucis Ça va pas sur `writeImage` On cherche à écrire une vignette, donc par défaut elle n’existe pas, donc `realpath($vignette)` retourne false. Tu avais vraiment pu faire fonctionner ça ???
Owner

Du coup j’ai enlevé les realpath sur les writeImage.
Je ne sais pas ce que ça donne sous Windows par contre.

Du coup j’ai enlevé les realpath sur les writeImage. Je ne sais pas ce que ça donne sous Windows par contre.
Poster

Du coup j’ai enlevé les realpath sur les writeImage.
Je ne sais pas ce que ça donne sous Windows par contre.

Je vais tester, je te dis.

> Du coup j’ai enlevé les realpath sur les writeImage. > Je ne sais pas ce que ça donne sous Windows par contre. Je vais tester, je te dis.
Owner

N’oublie pas pour tester

  • d’afficher les erreurs php
  • de vider local/vignettes ou d’utiliser var_mode=images
N’oublie pas pour tester - d’afficher les erreurs php - de vider local/vignettes ou d’utiliser var_mode=images
Poster

Bon, j'ai mieux testé et voici ma proposition :

ad944cd642

Bon, j'ai mieux testé et voici ma proposition : https://git.spip.net/spip/spip/commit/ad944cd642a92a0bc976f4770032af4ec75383b3
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Loading…
There is no content yet.