| 31 Jan 2024 |
|  Federico Damián Schonborn changed their profile picture. | 06:22:02 |
| 9 Feb 2024 |
|  Sofie (AWAY) joined the room. | 07:28:31 |
| 20 Feb 2024 |
| Sofie (AWAY) changed their display name from Sofi to Sofie. | 07:39:13 |
| Sofie (AWAY) changed their profile picture. | 14:39:04 |
| Sofie (AWAY) changed their profile picture. | 14:41:24 |
| Sofie (AWAY) changed their profile picture. | 14:42:41 |
| Sofie (AWAY) changed their profile picture. | 14:44:00 |
| 27 Feb 2024 |
 K900 | Reviving this room to say that if no one objects, I want to rip out create_machine(dict) from the test driver in the next few days | 19:44:36 |
| K900 | PR is here: https://github.com/NixOS/nixpkgs/pull/291544 | 19:44:39 |
 Alyssa Ross | Oh I use that out of tree | 19:46:03 |
| Alyssa Ross | ah but it's literally just removing the need to wrap the start command in a dict? | 19:46:26 |
| Alyssa Ross | sgtm, but would we want a deprecation period with a warning? I don't mind myself, but maybe I'm not the only one using it out of tree. | 19:47:12 |
| K900 | As long as you're just passing in a start command, it's fine | 19:48:01 |
| K900 | But stuff like {"hda": ..., "cdrom": ...} will not work | 19:48:26 |
| Alyssa Ross | Oh I didn't even remember you could do that. | 19:48:43 |
| K900 | I don't know about the deprecation period, because the old way of generating startcommands was deprecated, with a warning, for YEARS | 19:48:52 |
| K900 | * I don't know about the deprecation period, because the old way of generating startcommands was deprecated, with a warning, for YEARS now | 19:48:53 |
| K900 | And just removing the outer dict layer is a trivial change that will be caught by mypy | 19:49:15 |
| Alyssa Ross | Do you mean the deprecation that I undid a year ago, or a different one? https://github.com/nixos/nixpkgs/commit/845576aac4c067b816886d5ef55aaec82ea37c17 | 19:50:59 |
| K900 | https://github.com/NixOS/nixpkgs/blob/b5c956262d950cd2accccbe6a1e7fe768d5efaad/nixos/lib/test-driver/test_driver/machine.py#L382-L386 | 19:54:04 |
| K900 | This one | 19:54:05 |
| K900 | Which is still there and 3 years old | 19:54:21 |
| Alyssa Ross | oh, so is it already possible to get equivalent functionality without passing a dict with startCommand? | 19:55:47 |
| K900 | Not quite | 19:57:15 |
| K900 | It's stupid | 19:57:19 |
| K900 | It takes a dict | 19:57:21 |
| K900 | Always | 19:57:23 |
| K900 | But if the dict has startCommand, it'll use that | 19:57:33 |
| K900 | And if it has anything else, it'll use THAT THING | 19:57:41 |
| Alyssa Ross | Then I'm not sure this deprecation warning counts? If there was no possible way to get that functionality without using startCommand… | 19:58:18 |
