| 8 May 2023 |
 raitobezarius | écris [Backport] devant ton titre de PR | 17:22:28 |
| raitobezarius | met un tag security | 17:22:35 |
| 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 |
