Page 1 sur 1

[résolu] Comment générer un patch pour bugzilla

Publié : 22 mars 2009, 11:50
par calimo
Hello,

Suite à ce sujet j'aimerais faire un patch pour html.css. Comme c'est du CSS, je pense pouvoir me débrouiller au niveau du code :wink:

Mon seul problème, c'est que quand on regarde d'autres patches pour html.css (autre exemple) il y a des lignes au début du patch qui indiquent la version du html.css.

Donc j'ai installé mercurial, créé un fichier ~/.hgrc selon les instructions : https://developer.mozilla.org/en/Mercur ... figuration
Ensuite j'ai créé un répertoire "mozilla" sur mon bureau et fait

Code : Tout sélectionner

hg pull -u
Et j'ai reçu le message d'erreur suivant :

Code : Tout sélectionner

abort: There is no Mercurial repository here (.hg not found)!
Je pense qu'il s'agit du fichier définissant le serveur de Mozilla et tout ça, mais je n'ai pas réussi à trouver ce que je dois utiliser exactement... et il n'y a rien dans la doc Mercurial Basics à ce sujet :?

Est-ce que quelqu'un a déjà réussi à se dépatouiller avec ça, et si oui, comment ?

Merci :wink:

Message envoyé avec : Mozilla/5.0 (X11; U; Linux x86_64; fr; rv:1.9.0.7) Gecko/2009030423 Ubuntu/8.10 (intrepid) Firefox/3.0.7

Publié : 22 mars 2009, 14:19
par martin
Je n'ai pas encore essayer de récupérer les sources mercurial de mozilla, uniquement cvs, mais je pense qu'il faut que tu commences par importer l'arbre existant, voir cette page:
https://developer.mozilla.org/en/Mozill ... Mercurial)

clone:
hg clone http://hg.mozilla.org/releases/mozilla-1.9.1/ 191src

ou récupérer un bundle puis le mettre à jour.

Publié : 22 mars 2009, 16:50
par calimo
Superbe, ça a fonctionné et j'ai mon diff ! 8-)
Merci Martin :wink:

PS : vive la doc fragmentée ! :lol:

Message envoyé avec : Mozilla/5.0 (X11; U; Linux x86_64; fr; rv:1.9.0.7) Gecko/2009030423 Ubuntu/8.10 (intrepid) Firefox/3.0.7

Publié : 22 mars 2009, 17:24
par calimo
Pour la postérité : bug 484659 :wink:

Message envoyé avec : Mozilla/5.0 (X11; U; Linux x86_64; fr; rv:1.9.0.7) Gecko/2009030423 Ubuntu/8.10 (intrepid) Firefox/3.0.7

Publié : 23 mars 2009, 00:27
par arno.
Maintenant que tu as fait le plus facile, il te faut demander une "review", puis suivre la procédure pour que ton patch soit accepté.

Message envoyé avec : Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.0.7) Gecko/2009030810 Iceweasel/3.0.7 (Debian-3.0.7-1)

Publié : 23 mars 2009, 09:03
par calimo
Il ne faut pas que le bug soit confirmé avant de faire tout ça ?
Je pense que je peux raisonnablement sauter les étapes de vérification que le code compile. C'est du CSS.
Pour les tests, le problème c'est que c'est à faire sur des sites réels qui pourraient dépendre de la valeur pre pour s'afficher correctement... :oops:

Message envoyé avec : Mozilla/5.0 (X11; U; Linux x86_64; fr; rv:1.9.0.7) Gecko/2009030423 Ubuntu/8.10 (intrepid) Firefox/3.0.7

Publié : 23 mars 2009, 09:27
par martin
calimo a écrit : Pour les tests, le problème c'est que c'est à faire sur des sites réels qui pourraient dépendre de la valeur pre pour s'afficher correctement... :oops:
Mais rien ne t'empêche de créer néanmoins un fichier test simplifié, montrant le problème que tu souhaites mettre en évidence ;) .

En même temps, personnellement je ne suis pas sûr de vouloir la correction de ce bug, c'est le comportement habituel depuis longtemps, je préférerai une extension.