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

draft: docs: note about pcntl PHP extension #182

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pierreboissinot
Copy link
Member

TLDR; j'ai ajouté une note indiquant que l'extension est activée par défaut dans les packages Debian.

Dans le cadre de la configuration d'une nouvelle machine suivant les prérequis 2024.07, je me suis demandé pour quelle raison figurait l'extension PHP pcntl.

Cette extension n'est pas installable mais est à activer lors du build de php.

Debian package php avec pcntl activé. Cf. code source
Cette PR devrait résoudre cette issue #125

Cette extension est a priori utile pour symfony/messenger, je me suis demandé pourquoi cela ne figurait pas dans les dépendances symfony/messenger => parce que l'extension n'est pas disponible sur Windows et donc ce cas est géré. Voir détails.

@thislg étant donné cette issue, dois-je update que la dernière version dans ce cas ?

@lucasmirloup je n'ai pas pu tester en local que je n'ai rien cassé sur le site car j'ai une erreur SSL
Screenshot from 2024-09-06 10-35-39
Saurais-tu m'aider stp ?

@pierreboissinot
Copy link
Member Author

@thislg @lucasmirloup

Je vous propose d'ajouter aussi ces notes (c'est un extrait de playbook ne faites pas attention au format). Qu'en dites-vous ? Je fais ça dans une autre PR ou ici ?

#- 'php{{ php_version }}-ctype' # enabled by default https://www.php.net/book.ctype
# - 'php{{ php_version }}-iconv' # enabled by default https://www.php.net/manual/en/iconv.installation.php
#- 'php{{ php_version }}-pcre' # enabled by default https://www.php.net/book.pcre
#- 'php{{ php_version }}-session' # enabled by default https://www.php.net/book.session
#- 'php{{ php_version }}-simplexml' # enabled by default https://www.php.net/book.simplexml
#- 'php{{ php_version }}-tokenizer' # enabled by default https://www.php.net/book.tokenizer

@thislg
Copy link
Member

thislg commented Sep 6, 2024

Ça date mais je pense que #125 a été créée parce qu'on a eu besoin de demander explicitement l'activation de l'extension en CLI pour Messenger (sur un projet non hébergé sur Debian j'imagine).

Pour ce qui est de l'ajout du commentaire "activé par défaut sur Debian", je ne suis pas convaincu. On voudrait que l'hébergeur s'assure que tout soit installé / activé, Debian ou non. Et si Debian retire une extension comme ça peut arriver sur des nouvelles versions de php, le commentaire ne sera plus valide et induira en erreur.

@lucasmirloup
Copy link
Member

Je n'ai pas encore eu le temps de tout lire, mais juste pour qu'on ait une trace :
Le problème que j'ai rencontré à plusieurs reprises sur des projets est que l'hébergeur a bien activé l'extension, mais a désactivé toutes les fonctions PHP avec l'option PHP disable-functions, probablement pour des raisons de sécurité pré-paramétrées dans leurs outils d'hébergement.

@lucasmirloup
Copy link
Member

#125

@pierreboissinot
Copy link
Member Author

Pour ce qui est de l'ajout du commentaire "activé par défaut sur Debian", je ne suis pas convaincu. On voudrait que l'hébergeur s'assure que tout soit installé / activé, Debian ou non. Et si Debian retire une extension comme ça peut arriver sur des nouvelles versions de php, le commentaire ne sera plus valide et induira en erreur.

@thislg tu as raison, je l'ai mis car ça me sert de ne pas oublier et me reposer la question plus tard quand j'écris un playbook qui configure php.

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

Successfully merging this pull request may close these issues.

3 participants