| 8 May 2023 |
 raitobezarius | Tu prends la PR de GitHub et tu rajoutes .patch à la fin | 18:39:50 |
| raitobezarius | Ça te donne la version patch prête à l'emploi de la PR | 18:39:59 |
| raitobezarius | Tu fais fetchpatch dessus | 18:40:06 |
| raitobezarius | Là déjà c'est pas mal, faut juste fixer le warning | 18:40:18 |
 hldmna | effectivement, j'ai lu un peu trop vite les commentaires
je vais voir pour faire une PR avec les patch | 18:42:20 |
| hldmna | https://github.com/NixOS/nixpkgs/pull/230758 good ? | 19:25:54 |
 Julien | Tu peux utiliser fetchpatch au lieu de commiter les patchs | 19:26:44 |
| Julien | par exemple tu as cette PR : https://github.com/netdata/netdata/pull/14063, tu as le patch ici https://github.com/netdata/netdata/pull/14063.patch | 19:27:51 |
| Julien | https://nixos.org/manual/nixpkgs/stable/#fetchpatch | 19:28:31 |
 symphorien | c'est mieux de prendre l'url du commit https://github.com/netdata/netdata/commit/811028aea2f146cc0ac2bc403f7d692add400d63.patch que la PR au cas où qqn force push la PR (bon celle là est fermée donc normalement c'est bon mais voilà) | 19:29:04 |
 immae | Tu es sûr que ça marche ? Il me semblait avoir remarqué que les commits se font immédiatement garbage collect en cas de force push sur github (d’ailleurs leur lien vers les anciens commits dans le fil d’activité d’une MR est cassé pour cette raison) | 19:31:21 |
| Julien | Après ya pas de hash pour fetchpatch ? | 19:32:36 |
| hldmna | Julien merci, je connaissais pas. Je pense que je vais laisser comme ça pour l'instant. j'utiliserai fetchpatch si on me le demande dans la PR | 19:33:26 |
| symphorien | In reply to @immae:immae.eu
Tu es sûr que ça marche ? Il me semblait avoir remarqué que les commits se font immédiatement garbage collect en cas de force push sur github (d’ailleurs leur lien vers les anciens commits dans le fil d’activité d’une MR est cassé pour cette raison) le commit est dans master donc c'est bon il se fera pas gc | 19:33:25 |
| immae | ah oui ok | 19:33:50 |
| Julien | In reply to @hldmna:beeper.com
Julien merci, je connaissais pas. Je pense que je vais laisser comme ça pour l'instant. j'utiliserai fetchpatch si on me le demande dans la PR Je pense que on va te le demander. C'est utile aussi pour savoir d'où tu sors ton patch | 19:34:00 |
| hldmna | J'ai mis le lien des prs en commentaires dans les patches ^^ au cas où | 19:34:49 |
 Minijackson | si jamais la PR change, le build va casser tout de même de manière non évidente, puisque le build sera probablement passé une fois sur cache.nixos.org | 19:36:00 |
| raitobezarius | on t'a demandé fetchpatch :P | 19:38:18 |
| Julien | You got it :P https://github.com/NixOS/nixpkgs/pull/230758#discussion_r1187819603 | 19:38:25 |
| Julien | Et en plus je me suis retenu d'aller review la PR pour le faire :P | 19:38:50 |
| raitobezarius | déso j'ai envoyé un screen de la conv à winter, ça la fera marrer je pense | 19:40:05 |
| Julien | Elle parle fr ? | 19:40:26 |
| raitobezarius | non | 19:42:16 |
| Minijackson | ça bully les first-time contributors, c'est pas gentil... | 19:42:21 |
| raitobezarius | pardon, hldmna si c'est mal pris, c'était pas du tout le but | 19:42:45 |
| Julien | G mis une petite fusée sur la PR pour saluer la contribution | 19:43:08 |
| raitobezarius | (mais comme il m'a expliqué que la situation de netdata était critique au dernier meetup j'ai été un peu joueur) | 19:43:12 |
| hldmna | In reply to @raitobezarius:matrix.org
pardon, hldmna si c'est mal pris, c'était pas du tout le but pas de soucis, je le prends pas mal
j'apprends :p | 19:44:43 |
| hldmna | pour le sha256, j'arrive pas à trouvé mieux que de mettre un truc bidon et de remplacer avec ce que donne l'execution. Il y a vraiment que ça ? | 19:52:41 |
