| 8 May 2023 |
 raitobezarius | et met moi en reviewer | 17:22:38 |
| raitobezarius | ou ping moi | 17:22:43 |
 hldmna | Ok | 17:23:01 |
| hldmna | J'aurai dû regarder avant.., il y a déjà eu une PR pour ça mais elle a été rejeté à cause de breaking change. https://github.com/NixOS/nixpkgs/pull/207270 | 18:35:13 |
| raitobezarius | Winter a été un peu brutal mais oui il faut isoler les patchs | 18:35:38 |
| hldmna | * J'aurai dû regarder avant.., il y a déjà eu une PR pour ça mais elle a été rejeté à cause de breaking change (entre autre). https://github.com/NixOS/nixpkgs/pull/207270 | 18:35:43 |
| raitobezarius | In reply to @raitobezarius:matrix.org
si tu peux chopper les patchs ^ c'est le sens de ce propos | 18:35:52 |
| raitobezarius | Trouve les commits, prend les patchs et applique par dessus et si tu ouvres une telle PR, c'est tout bon ;) | 18:36:22 |
| hldmna | Yep, mais pour ça aussi les gens étaient contre dans la PR | 18:36:25 |
| 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 |
