| 14 Dec 2022 |
 Winter (she/her) | In reply to @philipp:xndr.de
I think the default of services.matrix-appservice-discord.settings is really unfortunate: The database.filename is set to the assumed default value but that makes using database connections a pain because those aren't used if the former is set. cc Notkea | 20:00:45 |
 @pacien:pacien.net | In reply to @philipp:xndr.de
I think the default of services.matrix-appservice-discord.settings is really unfortunate: The database.filename is set to the assumed default value but that makes using database connections a pain because those aren't used if the former is set. what would you suggest instead? | 20:04:21 |
 philipp | I think not setting it would be fine. Wanted to test that though and this was half a note to myself. | 20:06:23 |
| @pacien:pacien.net | or perhaps setting the default differently (avoiding recursiveUpdate in the module).
Could set database = mkDefault { filename = "..."; } so that setting database:connString would also remove filename.
But I'm not sure whether mkDefault would actually allow doing that kind of thing. | 20:20:34 |
| 19 Dec 2022 |
 Sumner Evans | Is there a way to override matrix-react-sdk given the way that we currently build Element? | 16:54:29 |
 Dandellion | no, because the unwrapped was put inside the let block | 16:55:43 |
| Dandellion | https://github.com/NixOS/nixpkgs/pull/200336 | 16:56:31 |
| Dandellion | and we are as per usual terrible at merging things in this team >.< | 16:57:49 |
| Sumner Evans | I saw that PR, and I just didn't feel like I understood what it was doing well enough to approve it. | 16:58:42 |
| Dandellion | The idea is to let you do
pkgs.element-web.override {element-web-unwrapped = element-web-unwrapped.overrideAttrs ... }
| 17:04:51 |
| Dandellion | * The idea is to let you do
pkgs.element-web.override {element-web-unwrapped = pkgs.element-web-unwrapped.overrideAttrs ... }
| 17:05:05 |
| Dandellion | but since ele-unwrapped is its own drv, not have to build it just because you're pkgs.element-web.override {conf = ...} | 17:06:02 |
| Dandellion | * but since ele-unwrapped is its own drv, not have to build the whole element-web just because you're pkgs.element-web.override {conf = ...} | 17:06:20 |
| Dandellion | * but since ele-unwrapped is its own drv, not have to build the whole element-web just because you're pkgs.element-web.override {conf = ...}ing | 17:06:22 |
| Sumner Evans | it was still not entirely clear to me how I'd override element-web itself to have a different version of matrix-react-sdk... | 17:07:22 |
| Sumner Evans | I really hate the architecture of the Element (the matrix-react-sdk is really the element-react-sdk) | 17:07:48 |
| Dandellion | Nothing's changed on that front since you made it build from source | 17:08:29 |
| Dandellion | just which drv to override | 17:08:43 |
| Dandellion | (with that PR at least) | 17:08:57 |
| Sumner Evans | I thought it got reverted. | 17:08:58 |
