| 1 Jun 2024 |
kloenk | No worries. I waited 4 years in nixcpp :p | 22:06:15 |
| Charles joined the room. | 22:31:41 |
ma27 | Question: is there any way in a DerivationGoal to find out whether you're in a daemon or in a nix-build as a root? I'm a little out of ideas since the store is always a LocalStore here and I didn't manage to write dynamic_cast<SimpleLogger> since that's not implemented in any of the headers.
Context: a while ago I submitted https://gerrit.lix.systems/c/lix/+/913 to get SSH errors in the stderr of a nix-build and I think I found a nicer solution than this (using logger->log basically in DerivationGoal::tryBuildHook).
THe idea is to still use writeStderr (to have errors actually appear in nix-daemon stderr) and to use logger to make it appear in the client's stderr. Then we can probably get rid of the log-fd=4 thingy, at least for ssh-ng (for legacy ssh tests are breaking badly and I'm not entirely sure why yet).
Any ideas? Or is my idea even reasonable? | 22:46:03 |
ma27 | oh I should probably explain why I want to know: if I do sudo nix-build, I don't go via the daemon and logger->log & writeStderr would write into the "same" stderr, i.e. SSH messages would always appear twice. | 22:47:05 |
⚠️ eldritch horrors operating in this area ⚠️ | it'd probably be better to always go through nix-daemon if one exists and treat nix-build without faemin as disaster recovery that needn't be super prettt | 22:52:07 |
| artemist joined the room. | 22:55:20 |
ma27 | yeah I only realized that because lazy me was doing sudo ./result/bin/nix to avoid changing my system's daemon %)
but, do you want me to submit a change going into that direction?
and regarding the change I'm currently working on, your suggestion is basically to just accept duplicated messages when "bypassing" the daemon on a remote build? | 22:57:51 |
ma27 | or did I misunderstand that? :) | 22:57:56 |
⚠️ eldritch horrors operating in this area ⚠️ | no you got it right | 22:58:23 |
artemist | Is there a plan to setup gerrit for lix-installer? I'm working on fixing the tests so I can do more development | 22:58:36 |
ma27 | ok, thanks for your input!
will try to submit a a CL for that soonish.
but I think I'll go to sleep first :) | 22:59:35 |
Qyriad | In reply to @pennae:matrix.eno.space it'd probably be better to always go through nix-daemon if one exists and treat nix-build without faemin as disaster recovery that needn't be super prettt yes, agreed. root defaulting to the local store instead of the daemon is wild behavior and it's better to just fix that instead imo | 23:34:00 |
| 2 Jun 2024 |
| Federico Damián Schonborn changed their profile picture. | 00:25:40 |
| Federico Damián Schonborn changed their profile picture. | 00:28:10 |
jade_ | In reply to @ma27:nicht-so.sexy
yeah I only realized that because lazy me was doing sudo ./result/bin/nix to avoid changing my system's daemon %)
but, do you want me to submit a change going into that direction?
and regarding the change I'm currently working on, your suggestion is basically to just accept duplicated messages when "bypassing" the daemon on a remote build? https://git.lix.systems/lix-project/lix/issues/18 | 00:34:33 |
jade_ | In reply to @artemist:mildlyfunctional.gay Is there a plan to setup gerrit for lix-installer? I'm working on fixing the tests so I can do more development we can set it up, that wouldn't be hard at all. we just don't have CI for it at the moment. | 00:35:19 |
jade_ | if you'd like to have gerrit for it, we can just gerrit it | 00:35:34 |
artemist | I have some patches to fix the tests | 00:35:37 |
jade_ | im just going to gerrit it for you, and disable prs for the repo and change our hacky thing to link to that one on gerrit also
if anyone has objections to this i can just revert | 00:40:27 |
@irenes:matrix.org | sounds good to me | 00:49:46 |
artemist | I'll push my commits to gerrit once the repo is synced | 00:51:49 |
artemist | (I don't have create privileges so I can't push main to it | 00:58:01 |
artemist | * (I don't have create privileges so I can't push main to it) | 00:58:03 |
jade_ | i am fighting it right now | 00:58:14 |
jade_ | i also don't have such permissions | 00:58:18 |
jade_ | give me a little bit | 00:58:21 |
artemist | good luck | 01:02:54 |
artemist | https://gerrit.lix.systems/c/lix-installer/+/1375 (and that chain) is about the minimum of what I needed to get tests to build | 01:33:13 |
jade_ | (i did knock over the gerrit like a cat knocking over a vase) | 01:35:13 |
jade_ | (updating stuff and configs) | 01:35:20 |