Modifier la formulation de discourse-setup

Salut
J’ai fait pas mal de nouvelles installations récemment.
Ce que j’ai remarqué, c’est que la formulation de la première réponse lors du processus de vérification si Docker est installé.

« Échec de la recherche de Docker… etc.
L’utilisation du mot « échec » comme première réponse dans la plupart des installations n’est pas un bon retour d’information pour quiconque n’est pas familier avec la ligne de commande.
Vous copiez et collez

sudo -s
git clone https://github.com/discourse/discourse_docker.git /var/discourse
cd /var/discourse

et cela entraîne un échec à moins que Docker ne soit déjà installé, ce qui est assez improbable dans la plupart des cas.

La formulation peut-elle être changée pour quelque chose de moins négatif ?

  • « Docker introuvable…

  • « [COMME PRÉVU]… Docker n’est pas installé…

  • « La prochaine étape consiste à installer Docker car il n’est pas actuellement installé…

  • « Cela a fonctionné ! - La prochaine étape consiste à installer Docker car il n’est pas actuellement installé…

@pfaffman Je sais que vous faites beaucoup d’installations, des suggestions ?

3 « J'aime »

Je me réponds… une réflexion après coup.

Est-ce juste une courtoisie de demander la permission d’installer Docker ?
Cet arrêt dans le processus d’installation est-il vraiment nécessaire ?

Ne pourrait-on pas simplement indiquer que Docker sera installé s’il ne l’est pas déjà ?

2 « J'aime »

Notez que cette formulation ne figure pas dans les instructions, elle provient du script discourse-setup.

2 « J'aime »

Oui,
Je parle de la réponse de la ligne de commande - echo

Le mot Failed peut-il être supprimé et remplacé par une autre formulation ?
Ou mieux encore, l’installation peut-elle simplement continuer sans s’arrêter pour une réponse ici ?

Si vous effectuez une installation, le dernier mot que vous voulez voir est Failed
Surtout après avoir copié et collé (correctement) les instructions d’installation.

3 « J'aime »

C’est un bon point. Il y a aussi un tas de commandes grep qui crachent des choses confuses et qui ressemblent à des erreurs. J’ai pensé à nettoyer cela auparavant. Cela prendra probablement une heure ou deux et je suis quelque peu réticent à le faire de mon propre temps car la plupart des gens sont capables de supporter ces messages effrayants.

La prochaine fois que je ferai des changements, cependant (peut-être le mois prochain pour faciliter la désactivation des nouvelles choses d’ember-cli), je verrai comment rendre ce langage moins effrayant.

4 « J'aime »

Rome ne s’est pas construite en un jour :slight_smile:

Donc le nettoyage peut prendre un peu de temps :+1:

3 « J'aime »

Salut Jay,
Oui, j’ai assisté à quelques installations et la panique diminue avec l’expérience.

Le langage utilisé reflète le résultat négatif de l’instruction if à la ligne 131.
En termes de codage, c’est correct, mais en réalité, il est attendu de ne pas trouver Docker installé car la plupart des gens viennent de faire une installation propre.

J’ai toujours pensé que ce code que j’ai collé dans la ligne de commande installerait Docker sans avoir besoin de dialogue ?

sudo -s
git clone https://github.com/discourse/discourse_docker.git /var/discourse
cd /var/discourse
2 « J'aime »

Apparemment, j’écrivais pour des programmeurs et non pour des gens quand j’ai écrit ça !

4 « J'aime »

Ce qui est présenté ici peut être facilement évité en remplaçant les messages par des chaînes de caractères linguistiques. Deux façons possibles d’y parvenir :

  • implémenter des appels API ( lire ) et utiliser Weblate,
  • stocker toutes les chaînes de caractères linguistiques localement ( au sein de l’installation de Discourse ) et remplacer tous les messages par des espaces réservés.

Les deux méthodes ci-dessus nécessitent une modification importante du cœur de Discourse.

1 « J'aime »

Il m’est venu à l’esprit que plutôt que de corriger toutes les choses que je pense être problématiques, le changement que vous demandez ne représente que quelques lignes, j’ai donc soumis ceci :

3 « J'aime »

Pour être honnête, la nouvelle formulation me donne l’impression que Docker est là, mais qu’il n’est juste pas dans le PATH.

3 « J'aime »

Oh, zut. Oui.

Les mots sont si difficiles.

Alors peut-être

Échec de la recherche de docker ou docker.io dans votre PATH.

ce qui, je pensais, aurait du sens car

Ni docker ni docker.io ne se trouvent dans votre PATH.

Devrait être

Docker introuvable.

Et les gens qui voient ça et

Appuyez sur Entrée pour installer Docker depuis https://get.docker.com/ ou Ctrl+C pour quitter

sauront s’ils doivent appuyer sur Entrée ou non.

5 « J'aime »

Salut Jay, merci d’avoir pris le temps de faire cela, je sais que tu es occupé avec ton propre site web qui est d’ailleurs très beau. Meilleurs vœux.

Docker introuvable.

Parfait.

1 « J'aime »

Je pense que mentionner PATH dans le message est un indicateur important que docker/docker.io peut ou non être installé, c’est-à-dire que le problème peut être que PATH n’inclut pas l’emplacement où il a été installé.

Si le but est simplement d’éviter d’utiliser un mot fortement négatif, je pense que remplacer « Échec de » par « Impossible de » ferait l’affaire.

Impossible de trouver docker ou docker.io dans votre PATH.

Retour au négatif…
Je n’ai pas besoin de le savoir… Je veux juste installer Docker quand je colle…

Pourquoi arrêter une installation pour signaler que ce que vous installez n’a pas été installé précédemment ?
Continuez l’installation de Docker.

Si Docker est trouvé dans le PATH - tant mieux - Installez Discourse

Supprimez complètement les deux lignes ?

Jay, Simon

Il est probablement préférable de ne pas perdre de temps là-dessus.
Plusieurs opinions sur la « copie » sont difficiles à résoudre sans contrarier quelqu’un.

Quand je ferai ma prochaine installation propre… je pourrai sourire à ce sujet et aller dans mon « endroit de bien-être » et comme on me dit que Docker n’a pas été trouvé sur mon installation fraîche et que j’appuie sur « Y » pour installer Docker, je pourrai le faire sans jurer entre les dents « Je pensais que c’était ce que je faisais – Dépêche-toi ».

1 « J'aime »

Je pense que c’est parce que vous n’installez pas docker. Ce que vous faites au moment de ce message est d’exécuter la configuration de Discourse, qui repose sur la disponibilité de docker. Cela vous aide à l’installer s’il ne le trouve pas, mais vous le demande car il ne connaît rien à votre système.

Peut-être qu’il ne le trouve pas parce qu’il n’est pas installé, auquel cas l’installation est évidemment la bonne réponse. Peut-être qu’il ne le trouve pas parce que vous avez délibérément installé docker ailleurs pour une raison quelconque, qui ne se trouve pas dans le PATH par défaut du système, auquel cas l’installation est la mauvaise réponse. (Une réponse correcte serait d’interrompre la configuration et de la relancer avec PATH=/où/est/docker:$PATH ./discourse-setup)

Est-il possible que vous interprétiez le nom du dépôt, discourse_docker, comme étant docker ? Il ne l’est pas, c’est la couche externe de Discourse, les composants qui doivent interagir avec docker comme le script de configuration, le script de lancement, etc.

1 « J'aime »