!lymvtcwDJ7ZA9Npq:lix.systems

Lix Development

422 Members
(Technical) development of Lix, the package manager, a Nix implementation. Please be mindful of ongoing technical conversations in this channel.142 Servers

Load older messages


SenderMessageTime
1 Aug 2025
@charles:computer.surgeryCharlesi could try to consolidate the conversation into an issue if that would be helpful, assuming the involved parties (raito and emily) are okay with that16:55:01
@charles:computer.surgeryCharlesprobably just copy+pasting the relevant/substantive bits directly from matrix lol16:55:22
@xokdvium:matrix.orgSergei Zimmerman (xokdvium)FWIW cppnix codebase has been reformatted.16:56:02
@emilazy:matrix.orgemilyif you'd like to see such an issue I'll see if I have energy to put my thoughts in more organized form in a little while, if you'd like to save yourself the trouble16:57:14
@emilazy:matrix.orgemily(I just think it is only worth having an issue if there is a possibility that the information it contains could result in a changed decision)16:57:31
@charles:computer.surgeryCharlesi mean it seems like the chance is nonzero at least16:58:39
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)
In reply to @charles:computer.surgery
i could try to consolidate the conversation into an issue if that would be helpful, assuming the involved parties (raito and emily) are okay with that
yep, would be good
16:59:33
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)
In reply to @emilazy:matrix.org
(I just think it is only worth having an issue if there is a possibility that the information it contains could result in a changed decision)
I won't lie, I think the bar is very high for such a decision to happen
16:59:55
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)So I don't want to be deceptive and snipe you17:00:00
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)I'm open to approach it with an open mind and look at it for what it is17:00:08
@emilazy:matrix.orgemily was that assessment made with or without knowledge of the existence of git mergetool? 😅 17:00:42
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)It was done with the knowlege of git mergetool17:00:52
@emilazy:matrix.orgemilyI am really confused as to the conflict issues that are being envisioned then. but ok. I will leave it be17:12:20
@emilazy:matrix.orgemilyI put up https://git.lix.systems/lix-project/lix/issues/94817:51:03
@emilazy:matrix.orgemilyto have the discussion recorded17:51:09
@emilazy:matrix.orgemilyhttps://gerrit.lix.systems/c/lix/+/3858 could probably use an urgent +2 btw17:51:14
@qyriad:katesiria.orgQyriad done 17:53:04
@emilazy:matrix.orgemilyty17:53:19
@emilazy:matrix.orgemilysorry for the regression17:53:22
@qyriad:katesiria.orgQyriadwe approved it, so we bear that responsibility too17:53:40
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)we should rerun CI manually on d5cfc6f19c and f077a6f36e18:09:44
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)taking care of it18:10:14
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)
In reply to @emilazy:matrix.org
I am really confused as to the conflict issues that are being envisioned then. but ok. I will leave it be
I double checked again and one reason that I see is that clang-format output is bad in certain (common in the Lix codebase) corner cases
18:35:29
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)And this is reason enough to proceed blindly with a full reformat18:35:36
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)* And this is reason enough to not proceed blindly with a full reformat to me18:35:41
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)If being broken on multiple lines with their literals cut in half, call being laid out on multiple lines breaking the logical manual grouping18:36:08
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)We already have some nolint and I remember that we already have troubles with incremental reformatting from time to time but manageable on the scale of it18:37:00
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)All the nested ternaries lose a lot of their meaning18:37:13
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)Also, reformatting is not even nolint-preserving18:38:14
@raitobezarius:matrix.orgraitobezarius (DECT: 7248)So this will cause a lot of manual churn18:38:17

Show newer messages


Back to Room ListRoom Version: 10