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

Vous vous penchez sur le développement d'applications ou d'extensions pour Mozilla ? C'est alors vers ce forum que se dirigeront naturellement vos requêtes...

Modérateur : chBok

Répondre
calimo
Animal mythique
Messages : 14118
Enregistré le : 26 déc. 2003, 11:51

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

Message par calimo » 22 mars 2009, 11:50

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
Modifié en dernier par calimo le 22 mars 2009, 17:24, modifié 1 fois.

martin
Varan
Messages : 1074
Enregistré le : 21 janv. 2004, 16:23

Message par martin » 22 mars 2009, 14:19

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.

calimo
Animal mythique
Messages : 14118
Enregistré le : 26 déc. 2003, 11:51

Message par calimo » 22 mars 2009, 16:50

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
Modifié en dernier par calimo le 22 mars 2009, 17:25, modifié 1 fois.

calimo
Animal mythique
Messages : 14118
Enregistré le : 26 déc. 2003, 11:51

Message par calimo » 22 mars 2009, 17:24

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

arno.
Varan
Messages : 1347
Enregistré le : 19 août 2004, 12:26

Message par arno. » 23 mars 2009, 00:27

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)

calimo
Animal mythique
Messages : 14118
Enregistré le : 26 déc. 2003, 11:51

Message par calimo » 23 mars 2009, 09:03

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

martin
Varan
Messages : 1074
Enregistré le : 21 janv. 2004, 16:23

Message par martin » 23 mars 2009, 09:27

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.

Répondre

Qui est en ligne

Utilisateurs parcourant ce forum : Aucun utilisateur enregistré et 1 invité