| 15 Nov 2024 |
John Ericson | meson one is passing now too, nicely | 16:24:46 |
John Ericson | With these changes, we can do one mega dev shell with nix perl bindings, nix-eval-jobs, and hydra, and change everything at once if we want :D | 16:25:39 |
Mic92 | John Ericson: you want one mega repo? | 17:15:41 |
John Ericson | Mic92: nope! | 17:15:50 |
John Ericson | just I unpack a bunch of repos locally | 17:15:58 |
John Ericson | and then make separate PRs | 17:16:07 |
das_j | In reply to @Ericson2314:matrix.org das_j: Mic92 should we decide for nix-eval-jobs and Hydra? I repeat my Github point which is that there is currently nobody willing and/or capable of maintaining that in-tree for hydra | 17:48:36 |
das_j | So it can only go up from here | 17:48:42 |
John Ericson | @janne.hess:helsinki-systems.de: maybe I saw those words twice but didn't understand them | 19:42:15 |
Rick (Mindavi) | Thanks for fixing up CA derivations while you were at it ππΌ, guess it was a useful test to add πΆβπ«οΈ | 20:02:38 |
John Ericson | Mindavi: oh you can be sure I won't cause a CA regression haha | 20:03:56 |
John Ericson | In reply to @janne.hess:helsinki-systems.de So it can only go up from here are you saying it is worth merging? because no one wants to maintain hydra-eval-jobs? | 20:04:55 |
Rick (Mindavi) | I would think it's a good thing to use nix-eval-jobs in hydra instead of trying to maintain in the hydra repo | 20:07:26 |
John Ericson | Mindavi: I think we all agree, I just don't wanna merge my own PR | 20:08:58 |
das_j | In reply to @Ericson2314:matrix.org @janne.hess:helsinki-systems.de: maybe I saw those words twice but didn't understand them I'm sorry, not sure what was wrong with my grammar. The hydra component lacks maintainership which makes a switch to a maintained component a good idea. Is this grammar more understandable? | 21:39:30 |
John Ericson | das_j: gotcha, so you are saying this is definitely a net improvement, right? | 21:40:06 |
John Ericson | if you like it, feel free to hit the merge button | 21:40:42 |
John Ericson | I can inform the rest of the SC but I would not block a decision on this | 21:40:58 |
das_j | In reply to @Ericson2314:matrix.org das_j: gotcha, so you are saying this is definitely a net improvement, right? Pretty much, apart from the fact that it's nix-community. Which is not something I have a personal issue with but I can imagine other people do | 21:41:31 |
John Ericson | I say better to ask for forgiveness than permission :) | 21:41:54 |
das_j | In reply to @Ericson2314:matrix.org if you like it, feel free to hit the merge button I will take a look this weekend I hope. If I don't, feel free to ping and annoy me again because I likely forgot ;) I'll do my best reviewing the change | 21:42:10 |
John Ericson | OK thanks! | 21:42:19 |
John Ericson | all the code is due to the lix people, who are already using this | 21:42:29 |
das_j | In reply to @Ericson2314:matrix.org OK thanks! No, thank you for upstreaming all that work | 21:42:41 |
John Ericson | I just changed nix-eval-jobs (already merged) and stuff in the flake | 21:42:43 |
John Ericson | so feel free to give it a quick and dirty review | 21:42:56 |
John Ericson | since the code is already in production....somewhere | 21:43:02 |
John Ericson | frankly, I don't mind if you don't look at the code at all | 21:43:48 |
John Ericson | I just want to not merge it myself since you brought up the nix-comunity issue, want to make sure no one reads that as me merging over your objection | 21:44:14 |
das_j | In reply to @Ericson2314:matrix.org frankly, I don't mind if you don't look at the code at all I'd feel bad if I didn't tbh. That's something I would expect someone in my role to do | 21:44:23 |