Les derniers commits sur Bonux sont complètement foireux.
Je n’arrive pas à faire marcher mon site en local.
Il est urgent de remettre la dernière version qui fonctionne à savoir 3.5.4 et de debug les ajouts si ceux-ci sont pertinents.
Je répète aussi que les tests doivent être faits avant les commits et avec toutes les notices.
Je suis revenu en 3.5.4 pour restaurer le fonctionnement du site.
Eric Lupinacci a écrit le 21/06/2020 à 16:17 :
Les derniers commits sur Bonux sont complètement foireux.
Je n’arrive pas à faire marcher mon site en local.Il est urgent de remettre la dernière version qui fonctionne à savoir 3.5.4 et de debug les ajouts si ceux-ci sont pertinents.
Je répète aussi que les tests doivent être faits avant les commits et avec toutes les notices.
Je suis revenu en 3.5.4 pour restaurer le fonctionnement du site.
Chez moi ça marche™
Et j'ai étudié le diff sans les espaces : la seule change qui change, c'est une ligne de commentaires (l'url dans le commentaire).
Pour le reste, c'est identique (puisque JLuc a rajouté puis enlevé quelque chose, pour revenir à l'état initial).
On a fait un pas de géant ! Merci.
Je viens de réessayer :
Warning: Cannot modify header information - headers already sent by (output started at /Users/eric/Sites/ZONEGIT/gitea/spip-contrib-extensions/spip-bonux/spip_bonux_options.php:1) in /Users/eric/Sites/SPIP/ecrire/inc/actions.php on line 141
En plus l’indentation a changé et d’ailleurs elle est foireuse sous phpstorm.
Donc tant qu’à faire faut revenir vraiment à la version précédente du fichier svp.
Le 21/06/2020 à 21:21, Eric Lupinacci a écrit :> Je viens de réessayer :> Warning: Cannot modify header information - headers already sent by (output started at > /Users/eric/Sites/ZONEGIT/gitea/spip-contrib-extensions/spip-bonux/spip_bonux_options.php:1) in > /Users/eric/Sites/SPIP/ecrire/inc/actions.php on line /141/> En plus l’indentation a changé et d’ailleurs elle est foireuse sous phpstorm.> Donc tant qu’à faire faut revenir vraiment à la version précédente du fichier svp.
OK
J'ai pas eu de réponse pour réverter
alors je vais commiter une nouvelle couche
JLuc
je sais pas quel était la question exacte pour reverter, mais sans doute que git revert <hash du commit> répond au besoin
Sérieusement ?
alors que tu vois qu’il y a des espaces en plus partout, donc AVANT le <?php et qu’on sait tout que c’est la plaie et une horreur parce que ça envoie un espace sur la sortie standard ?
Je suis pas sur que faire le malin soit le plus constructif...
Hello,
J’ai donc envoyé un revert complet des 4 commits de JLuc que tout soit bien clair, et recommit le seul changement de cette salve, qui était une URL dans le PHPDoc.
JLuc, ça serait quand même malin de tester un peu avant d’envoyer ce genre de gros diff, et si tu n’est pas certain ET que tu as besoin de commit pour déployer et tester, le mieux c’est de faire une branche : comme ça en ligne tu checkout sur cette branche, tu teste, tu debug, et quand ça marche tu peux faire une Pull Request, ou cherry-picker pour envoyer sur le master, ou demander à quelqu’un de le faire.
Et donc, instant documentation :
Comme indiqué par Maieul, pour générer un commit qui revert un autre commit il faut faire
git revert xxxxxx
(avec le numéro du commit à revert )
Pour voir le diff entre 2 commits c'est
git diff abcdef..123456
(la référence du premier commit, deux petits points et la référence du second commit).
Et si on ajoute l’option -w ça ignore les changements qui ne sont que des espaces.
Dans notre cas ça donnait :
$ git diff -w e350597..1900248
diff --git a/spip_bonux_options.php b/spip_bonux_options.php
index 5d83500..b2c6b6f 100644
--- a/spip_bonux_options.php
+++ b/spip_bonux_options.php
@@ -202,7 +202,7 @@ if (!function_exists('text_truncate')) {
* @param array $options An array of html attributes and options.
* @return string Trimmed string.
* @access public
-* @link http://book.cakephp.org/view/1469/Text#truncate-1625
+ * @link https://api.cakephp.org/4.0/class-Cake.Utility.Text.html#truncate
*/
function text_truncate($text, $length = 100, $options = array()) {
$default = array(
J’ai donc envoyé un revert complet des 4 commits de JLuc que tout soit bien clair, et recommit le seul changement de cette salve, qui était une URL dans le PHPDoc.
Merci Cerdic !
JLuc, ça serait quand même malin de tester un peu avant d’envoyer ce genre de gros diff, et si tu n’est pas certain ET que tu as besoin de commit pour déployer et tester, le mieux c’est de faire une branche : comme ça en ligne tu checkout sur cette branche, tu teste, tu debug, et quand ça marche tu peux faire une Pull Request, ou cherry-picker pour envoyer sur le master, ou demander à quelqu’un de le faire.
Oui je suis en environnement de dev temporairement bancal
et visiblement je ferais mieux de m'abstenir tant que c'est pas upgradé consolidé.
Pour info du coup j'avais commit via l'UI gitea
et c'est donc probablement ça qui a ajouté les espaces,
alors pourtant que ça ne s'était pas produit toutes les fois précédentes où j'ai commit comme ça
(alors je pige pas le pourquoi).
Et donc, instant documentation :
Comme indiqué par Maieul,
Non ya pas eu indiqué à ma connaissance.
C'est une commandes très basique finalement pour revert.
Encore faut il savoir que c'est aussi simple car c'est souvent compliqué de corriger un truc déjà pushé.
re-merci
JL
Cerdic a écrit le 22/06/2020 à 09:30 :
Sérieusement ?
Oui.
alors que tu vois qu’il y a des espaces en plus partout, donc AVANT le <?php et qu’on sait tout que c’est la plaie et une horreur parce que ça envoie un espace sur la sortie standard ?
J'ai vérifié : je n'ai pas les espaces au début du fichier.
Je suis pas sur que faire le malin soit le plus constructif...
Je ne cherchais pas à faire le malin.
Par contre, il y a peut-être une différence (qui va disparaître au 1er juillet) : j'ai mis à jour par SVN.
C'est d'autant plus surprenant que https://zone.spip.net/trac/spip-zone/changeset/125253/spip-zone/ a bien les espaces en début de <?php
Donc, je ne sais pas pourquoi ça a marché chez moi.
Désolé pour le bruit.
Je viens d'ajouter ces 2 commandes à la doc: https://contrib.spip.net/Equivalences-des-commandes-SVN-GIT#Voir-les-differences-diff
hier soir à 23h10 
mais bien après tes premières questions j'en convient
Ah en effet. Merci aussi alors.
JL