| 23 Jan 2024 |
@delroth:delroth.net | specifically: how can a ::Machine get built from a nix::Machine there | 17:44:53 |
John Ericson | delroth: The ::machine has no fields which have to be set construction time, it just needs the new super class to be construct=ed | 17:45:45 |
@delroth:delroth.net | sure, but there isn't a ::Machine constructor from nix::Machine here, as far as I can tell - is that implicitly defined? | 17:47:48 |
John Ericson | delroth: yes that is implicitly defined | 17:49:23 |
John Ericson | merely because ::Machine has no explicit constructor, and it has the public base class | 17:49:44 |
John Ericson | I would probably get rid of the inheritence eventually because inheritance is ugly :) | 17:50:11 |
John Ericson | but that would be more churn because field names, so I am saving that for after the Nix-side code is cleaned up and matching Hydra | 17:50:35 |
@delroth:delroth.net | In reply to @Ericson2314:matrix.org delroth: yes that is implicitly defined after some digging the answer I was looking for was https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/p0017r1.html from C++17 :p | 18:05:50 |
@delroth:delroth.net | TIL | 18:05:59 |
@delroth:delroth.net | I think this might be the first time I actually encounter this, somehow | 18:06:25 |
John Ericson | delroth: ah OK | 18:07:14 |
John Ericson | I never did C++ pre C++17 and Nix :) | 18:07:26 |
| 24 Jan 2024 |
| donotdividebyzero joined the room. | 11:04:27 |
| 25 Jan 2024 |
@delroth:delroth.net | https://github.com/NixOS/hydra/pull/1344 it finally annoyed me enough to spend 5min on it | 08:32:16 |
K900 | Oh no | 08:32:33 |
K900 | You could just | 08:32:38 |
K900 | Do that | 08:32:38 |
K900 | The entire time | 08:32:41 |
K900 | Not like you as in you personally | 08:32:52 |
K900 | But the ability was there | 08:32:56 |
K900 | I just assumed Google makes it forced | 08:33:09 |
@delroth:delroth.net | don't assume someone else's malice when you could instead assume it's our own incompetence | 08:33:36 |
@delroth:delroth.net | is that a saying? maybe it should be | 08:33:43 |
@delroth:delroth.net | John Ericson: https://hydra.nixos.org/eval/1803854 looks like the nix-2.19 update broke stuff | 08:47:09 |
@delroth:delroth.net | error: access to URI 'github:NixOS/nixpkgs/e9f06adb793d1cca5384907b3b8a4071d5d7cb19' is forbidden in restricted mode | 08:47:20 |
@delroth:delroth.net | only impacts projects using flakes so not imo critical, if you want to suggest a fix I can try to apply it later today | 09:08:37 |
John Ericson | delroth: changing a nix setting will fix this | 13:54:37 |
ma27 | In reply to @delroth:delroth.net error: access to URI 'github:NixOS/nixpkgs/e9f06adb793d1cca5384907b3b8a4071d5d7cb19' is forbidden in restricted mode allowed-uris = [ "github:" ] is what I did on my Hydra. | 14:06:17 |
John Ericson | https://github.com/NixOS/ngi0-infra/blob/40fdf9ed77ff9e0ec196ed9c751d6827ca196740/makemake/hydra.nix#L57 | 14:07:14 |
John Ericson | there is a few colon-ending ones in there | 14:07:25 |