| 8 May 2023 |
 hldmna | La suggestion était d'ajouter un 'nouveau' paquet, genre netdata_1_37 | 18:37:03 |
 raitobezarius | Non? | 18:37:05 |
| raitobezarius | Non ça c'est pas bien | 18:37:13 |
| raitobezarius | Il faut prendre les PRs qui ont fixé la security vuln | 18:37:21 |
| raitobezarius | Isoler les fix | 18:37:24 |
| raitobezarius | Les appliquer sur la version stable | 18:37:29 |
| raitobezarius | C'est ça qui est la proposition acceptée dans cette issue | 18:37:39 |
| raitobezarius | Y a un warning mais ça on s'en fout on le patch aussi | 18:38:00 |
| raitobezarius | Pour appliquer un patch dans nixpkgs hldmna: | 18:39:14 |
| raitobezarius | Tu peux utiliser `patches` qui prend une liste de chemins | 18:39:25 |
| raitobezarius | (grep nixpkgs pour des exemples c'est assez documenté) | 18:39:41 |
| 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 |
