Re: Composer et SPIP sont dans un bateau

Bonjour

> Le 08/04/2019 à 20:06, [email protected] a écrit :
> > On peut faire des PR depuis git.spip.net mais avec les indications
> > communiquées il m'est impossible de comprendre ce qui ne fonctionne
> > pas et/ou pourquoi et donc donner une aide quelconque.
>
> Voici plus en détail dans les différents cas de figure.
>
> >> A propos du repo git :
> >>
> >> J'ai voulu faire un PR sur le repo GIT du core
> >> mais bien que pratiquant git à ma mesure depuis quelques années,
> >> je n'avais encore jamais fait ça, et je n'y suis pas parvenu.
>
> Localement j'ai fait
> git clone https://git.spip.net/SPIP/spip.git
> Là j'ai appliqué ma modif.
> Je n'ai pas su comment pull request ensuite.
> Quelle serait la commande ?

Tu ne peux pas tu n'as pas les droits. Le principe d'une PR c'est de
proposer une branche à appliquer sur une autre (sur le même dépot ou
non).
SPIP/spip.git n'est pas disponible à l'écriture. Donc il n'est pas
possible de créer une branche, ce qui signifie qu'il n'est pas
possible de faire une PR.

Seules les personnes ayant des droits d'écriture sur ce dépôt
pourraient faire un PR. Mais en mon sens cela serait une très mauvaise
pratique.

> >> Pour info, j'ai aussi essayé de le faire par l'UI du gitea,
> >> et je n'y suis pas parvenu (mais j'ai pas insisté).
> >> C'est possible ?
> J'avais créé un clone sur git.spip il y a 2 mois
> https://git.spip.net/JLuc/spip
> Je ne parviens pas à le mettre à jour via l'UI fournie.
> Ya moyen ?

Mettre à jour le dépot pas à ma conaissance. Faire une PR oui

Une fois sur ton fork/bifurcation.
Tu navigues dans la branche et le fichier qui t'intéresse. Par exemple
la branche master et le fichier spip.php

Tu verras un crayon sur le coté du fichier :
https://framapic.org/U6bzHLHsUGB6/XqL1p68dTJ9i.png

Si tu cliques dessus tu peux éditer en ligne le fichier et créer une
nouvelle branche (cf plus haut un PR c'est une branche)
https://framapic.org/AzzIteo0QmSY/l3HvD9MEC4kk.png

Ensuite tu vas sur "Demandes d'ajout", nouvelle demande
https://framapic.org/xREEahib9VCV/7BXz9kNuC63q.png

Et enfin tu précise sur quelle branche tu veux voir appliquer ton
correctif (autrement dit la PR)
https://framapic.org/XghNg3jxgYLS/GgA2xUQuw7Vy.png

Ce qui donnera
https://git.spip.net/SPIP/spip/pulls/8

Mais c'est pour moi une mauvaise pratique de vouloir à tout pris
passer par l'interface graphique. Cela n'a de vrai sens que si on
constate une modification restreinte et évidente.
Si on doit modifier plusieurs fichiers ou raisonner en plusieurs
commit il es préférable de passer par un vrai clone local de son fork.
Lorsqu'on pousse ensuite sa branche sur le dépot, gitea fournit en
réponse un lien pour créer la PR au besoin. (cf mail concernant
comment faire un PR)

> >> Par contre, j'ai réussi à le faire par l'UI de github.
> En effet le bouton qui permet les merge est "inversible" :
> on peut merger du clone vers l'origine (pour faire une PR)

cf dessus.

> mais aussi de l'origine vers le clone (pour synchroniser = rattraper les commits en retard)

Tiens ça c'est une nouveauté jusqu'à peu github ne proposait pas la
mise jour graphique de l'upstream.
Pas possible encore dans gitea à ma connaissance.

En espérant avoir été clair.

Km

PS il serait bien de centraliser tout ça car, il me semble que c'est
le 3eme ou 4eme fil de discussion ouvert sur la même question.