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

[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

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

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]

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]

Superbe, ça a fonctionné et j'ai mon diff !
Merci Martin

PS : vive la doc fragmentée !

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

[calimo]

Pour la postérité : bug 484659

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.]

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]

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...

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]

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...

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.