20 Mar 2024 |
hexa | no π | 01:28:25 |
adamcstephens | only idea i have is to pull the inner condition out to a variable, but it's just shifting the code around | 01:29:05 |
hexa | yeah, don't bother | 01:29:32 |
hexa | lib.optional should not exis | 01:31:27 |
hexa | * lib.optional should not exist | 01:31:29 |
adamcstephens | because it's not clearly a list? | 01:32:27 |
hexa | I remember that I confused optionals and optional very often early on | 01:33:44 |
hexa | but now I think just optionals should be sufficient for everything | 01:33:58 |
adamcstephens | ok, works for me :) | 01:34:22 |
hexa | making the argument a proper list makes extending it much simpler | 01:34:56 |
adamcstephens | yep | 01:35:03 |
hexa | also the reason I'm a friend of always expanding lists cough | 01:35:09 |
hexa | but that ship has sailed | 01:35:12 |
adamcstephens | what do you mean always expanding lists? | 01:35:37 |
hexa | to one line per item | 01:35:47 |
adamcstephens | ahh | 01:36:05 |
adamcstephens | we get that, after we get two items | 01:36:31 |
adamcstephens | i feel like the ovs line should split anyway given its lenght | 01:41:21 |
adamcstephens | 101 characters | 01:41:47 |
hexa | yeah, but that it flips around between one and two item π€· | 01:41:57 |
adamcstephens | yeah extra diff churn | 01:42:12 |
hexa | I know the discussion is probably not that simple | 01:42:20 |
adamcstephens | probably not | 02:14:44 |
aanderse | anyone ever run a container in github actions? no matter what command i run via `lxc exec machine -- X` i get command not found
and i can't even `lxc console` into the machine... always says the password is wrong even though i built and loaded custom lxc image
again... only happening when run from github actions | 20:54:34 |
aanderse | and for reference an ubuntu container runs fine π | 20:55:11 |
adamcstephens | What nixos image are you using? | 21:11:02 |
adamcstephens | You could see if itβs a path issue by using /run/current-system/sw/bin/β¦. | 21:12:39 |
aanderse | yeah that was my first thought
not PATH related
thanks though
just thought I'd mention here in case anyone has done this, save me some time banging my head π | 21:18:54 |
21 Mar 2024 |
adamcstephens | In reply to @aanderse:nixos.dev yeah that was my first thought
not PATH related
thanks though
just thought I'd mention here in case anyone has done this, save me some time banging my head π I donβt use GHA much and definitely not with lxd/incus.Β I do know the incus repo does though. | 11:06:34 |
| NixOS Moderation Botchanged room power levels. | 18:02:49 |