17 Feb 2023 |
| kevincox changed their profile picture. | 20:31:56 |
| kevincox changed their profile picture. | 20:33:07 |
18 Feb 2023 |
| Quinten joined the room. | 05:02:59 |
21 Feb 2023 |
| Tobias Stenzel changed their display name from Tobias Stenzel (escaP) to Tobias Stenzel (extern). | 17:22:39 |
| Tobias Stenzel changed their display name from Tobias Stenzel (extern) to Tobias Stenzel. | 17:22:49 |
26 Feb 2023 |
dotlambda | https://github.com/NixOS/nixpkgs/pull/211304 | 16:40:53 |
6 Mar 2023 |
ma27 | I applied some hacks against element-* to make sure we get pings when somebody touches the package: https://github.com/NixOS/nixpkgs/pull/219821
kinda hacky, but I think it's still pretty useful :) | 11:28:49 |
@piegames:matrix.org | OfBorg is notoriously broken, so why not register as Codeowners insteead? | 11:56:20 |
ma27 | for codeowners everyone in the maintainer team must have write access, also I don't like having to maintain individual paths at another place in the codebase. | 12:06:25 |
ma27 | see https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners | 12:06:28 |
15 Mar 2023 |
hexa | the matrix-synapse dependency management is bit over all over the place regarding optional dependencies | 09:39:48 |
hexa | some get dumped into to propagatedBuildInputs, as if they were required | 09:40:20 |
hexa | some become optional using enableSomething flags | 09:40:32 |
hexa | some get entirely ignored | 09:40:36 |
hexa | also not a fan of the undiscoverable enableFlag design | 09:46:29 |
hexa | also toggling these flags forces a rebuild of the package, when providing a proper PYTHONPATH would be sufficient | 10:02:25 |
hexa | ma27: config inspection to derive required extras? yay/nay? | 11:36:43 |
ma27 | so your suggestion would be to handle optional dependencies (such as systemd/redis and probably more, e.g. prometheus_client which is IIRC only needed when actually exposing metrics) the same way as we do it with plugins (i.e. adding them to PYTHONPATH in the service)? Havent' thought too much about it, but after a first glance I'd say it sounds pretty reasonable, hence Yay from me %) | 11:57:36 |
hexa | yep 🙂 | 12:00:36 |
hexa | make the default closure small | 12:00:44 |
hexa | make sure extras and plugins don't cause a rebuild, so everyone can rely on the cached package | 12:01:01 |
hexa | Redacted or Malformed Event | 12:01:54 |
hexa | Redacted or Malformed Event | 12:02:03 |
hexa | prometheus_client is actually not an optional dependency | 12:06:47 |
hexa | the idea is to make it more feasible to expose these extras listed here https://github.com/matrix-org/synapse/blob/develop/pyproject.toml#L249-L273 | 12:07:17 |
hexa | which are all the packages marked as optional in the dependencies section | 12:07:27 |
hexa | but grouping them like this makes their use much clearer | 12:07:37 |
hexa | and allows us to select them intelligently | 12:07:44 |
ma27 | In reply to @hexa:lossy.network prometheus_client is actually not an optional dependency heh, that was a wild guess and served just as an example :) | 12:10:14 |
hexa | yeah, I feel the same about prometheus, why isn't it optional? | 12:10:34 |