Ceci dit, la première chose à faire est d'en parler avec les développeurs d'Audacity. Tu (pas cosmocat, mais Xavier, l'auteur de la dépêche) sembles te baser entièrement sur une supposition pour ne même pas proposer un patch et t'imaginer qu'il sera rejeté de toutes façons. Alors oui il est tout à fait possible que des choses soient à revoir (ça arrive à tout le monde, dans tous les projets). Peut-être même qu'ils refuseront de but en blanc l'idée même du patch, qui sait! (je connais pas l'équipe d'Audacity ni s'ils sont arrangeants, ou braqués par exemple)
Mais tout ça t'en sauras rien si tu demandes pas. Va y humblement et diplomatiquement avec ton patch, et sois prêt à recevoir les critiques (s'il y a lieu, ça se trouve, ils le trouveront même bien tel quel, qui sait!) et améliorer le patch en conséquence. Et cela ira probablement.
Ceci dit, petit avis perso: ajouter des millisecondes ne me semble pas contredire le nom de la fonction "secondes" à ce point selon moi. Les millisecondes sont une variante (ou une subdvision) de l'unité seconde. Je ne vois pas de contradiction.
Mais peut-être que les dévs d'Audacity verront les choses autrement, j'en sais rien. Et dans ce cas là, ajouter une option peut en effet être une autre possibilité (ou encore changer le nom de la fonction). Et franchement je vois déjà une dizaine d'option pour l'affichage du temps, je doute que les dévs upstream aient un gros problème si tu souhaites en ajouter une avec une bonne raison (et tu en as une). Mais cela, je te conseille de voir cela directement avec les dévs du projet. Pas la peine de perdre ton temps pour du code qui sera pas pris.
Perso ma manière de travailler est la suivante quand je veux une fonctionnalité: je fais la base dans mon coin, car ça me sert de "preuve de concept", donc ça illustre bien mieux ce que je veux que du texte dans une requête de fonctionnalité, et surtout les dévs du projet me tiennent au sérieux dès le début car ils ont un patch à se mettre sous la dent et donc ils répondent beaucoup plus vite. Ça c'est ce que t'as fait et c'est très bien.
La seconde étape est de proposer ton patch de base une fois qu'il est un peu propre. Et à partir de là, la discussion avec upstream peut commencer et c'est le moment où tu fignoles, et où tu auras peut-être des concessions à faire en effet, ou bien du code à ajouter ou changer pour aller dans le sens du projet tout en intégrant ta nouvelle fonctionnalité intégralement.
Et quand c'est fait, le projet upstream est content et intègre ton patch. En général les projets sont contents d'avoir de nouvelles contributions. Ils espèrent même que les contributeurs restent et proposent plus de patchs à l'avenir. Faut pas croire mais la plupart des projets (même certains qui ont l'air gros, car connus à une certaine échelle) ont très peu de contributeurs. Je pense donc que tu as toutes tes chances.
Reste pas là sur linuxfr à piétiner et à te demander si oui ou non ils intègreront. Va direct sur leur logiciel de suivi, et donne ton patch. S'ils répondent pas vite, n'hésite pas à lancer un petit message sur la mailing list par exemple (ou autre de leurs canaux habituels de discussion, par exemple s'ils ont IRC), poliment et gentillement (genre "hey I submitted a patch a few days ago, would someone be kind enough to review it? I'm happy to change things if there are issues", un lien et quelques petits smileys bien placés). Et tout ira bien.
Film d'animation libre en CC by-sa/Art Libre, fait avec GIMP et autre logiciels libres: ZeMarmot [ http://film.zemarmot.net ]
[^] # Re: Contribution
Posté par Jehan (site web personnel, Mastodon) . En réponse à la dépêche Modeste contribution à Audacity sur l'affichage des temps. Évalué à 10.
Ceci dit, la première chose à faire est d'en parler avec les développeurs d'Audacity. Tu (pas cosmocat, mais Xavier, l'auteur de la dépêche) sembles te baser entièrement sur une supposition pour ne même pas proposer un patch et t'imaginer qu'il sera rejeté de toutes façons. Alors oui il est tout à fait possible que des choses soient à revoir (ça arrive à tout le monde, dans tous les projets). Peut-être même qu'ils refuseront de but en blanc l'idée même du patch, qui sait! (je connais pas l'équipe d'Audacity ni s'ils sont arrangeants, ou braqués par exemple)
Mais tout ça t'en sauras rien si tu demandes pas. Va y humblement et diplomatiquement avec ton patch, et sois prêt à recevoir les critiques (s'il y a lieu, ça se trouve, ils le trouveront même bien tel quel, qui sait!) et améliorer le patch en conséquence. Et cela ira probablement.
Ceci dit, petit avis perso: ajouter des millisecondes ne me semble pas contredire le nom de la fonction "secondes" à ce point selon moi. Les millisecondes sont une variante (ou une subdvision) de l'unité seconde. Je ne vois pas de contradiction.
Mais peut-être que les dévs d'Audacity verront les choses autrement, j'en sais rien. Et dans ce cas là, ajouter une option peut en effet être une autre possibilité (ou encore changer le nom de la fonction). Et franchement je vois déjà une dizaine d'option pour l'affichage du temps, je doute que les dévs upstream aient un gros problème si tu souhaites en ajouter une avec une bonne raison (et tu en as une). Mais cela, je te conseille de voir cela directement avec les dévs du projet. Pas la peine de perdre ton temps pour du code qui sera pas pris.
Perso ma manière de travailler est la suivante quand je veux une fonctionnalité: je fais la base dans mon coin, car ça me sert de "preuve de concept", donc ça illustre bien mieux ce que je veux que du texte dans une requête de fonctionnalité, et surtout les dévs du projet me tiennent au sérieux dès le début car ils ont un patch à se mettre sous la dent et donc ils répondent beaucoup plus vite. Ça c'est ce que t'as fait et c'est très bien.
La seconde étape est de proposer ton patch de base une fois qu'il est un peu propre. Et à partir de là, la discussion avec upstream peut commencer et c'est le moment où tu fignoles, et où tu auras peut-être des concessions à faire en effet, ou bien du code à ajouter ou changer pour aller dans le sens du projet tout en intégrant ta nouvelle fonctionnalité intégralement.
Et quand c'est fait, le projet upstream est content et intègre ton patch. En général les projets sont contents d'avoir de nouvelles contributions. Ils espèrent même que les contributeurs restent et proposent plus de patchs à l'avenir. Faut pas croire mais la plupart des projets (même certains qui ont l'air gros, car connus à une certaine échelle) ont très peu de contributeurs. Je pense donc que tu as toutes tes chances.
Reste pas là sur linuxfr à piétiner et à te demander si oui ou non ils intègreront. Va direct sur leur logiciel de suivi, et donne ton patch. S'ils répondent pas vite, n'hésite pas à lancer un petit message sur la mailing list par exemple (ou autre de leurs canaux habituels de discussion, par exemple s'ils ont IRC), poliment et gentillement (genre "hey I submitted a patch a few days ago, would someone be kind enough to review it? I'm happy to change things if there are issues", un lien et quelques petits smileys bien placés). Et tout ira bien.
Film d'animation libre en CC by-sa/Art Libre, fait avec GIMP et autre logiciels libres: ZeMarmot [ http://film.zemarmot.net ]