28 Jul 2025 |
| Quentin joined the room. | 14:08:55 |
| Guilherme Lima joined the room. | 20:30:15 |
31 Jul 2025 |
| @unwary:matrix.org joined the room. | 00:24:26 |
| @unwary:matrix.org left the room. | 00:25:00 |
1 Aug 2025 |
| Cathal joined the room. | 02:39:58 |
3 Aug 2025 |
aleksana 🏳️⚧️ (force me to bed after 18:00 UTC) | https://github.com/NixOS/nixpkgs/pull/430575/files#diff-1ea2c8559b007cf7ff7861872d6572a11bdb8f6a5f41754cd8e424f7dac5bf5aR59 | 12:20:49 |
aleksana 🏳️⚧️ (force me to bed after 18:00 UTC) | aarch64-linux gcc doesn't support c++20? | 12:21:04 |
emily | nope, that's nonsense | 12:25:32 |
emily | but the comment was already there | 12:25:46 |
emily | it is just being moved around with formatting churn | 12:26:04 |
emily | in this case the formatting churn goes around conventions, meta is generally not alphabetically-ordered | 12:26:14 |
emily | and changelog before description and homepage is just weird | 12:26:37 |
emily | nor do build inputs go after postPatch (there is no strict convention for all of this, but if you are going to do this kind of churn, explicitly making things comply less with the guidelines in https://github.com/jtojnar/nixpkgs-hammering/blob/b0817c8269798a830bc42f58a01c7543e7c5268a/explanations/attribute-ordering.md is hard to justify) | 12:27:32 |
emily | so "bencode: sort" should be rejected, but checking and fixing the vintage broken comment is not a blocker | 12:28:00 |
emily | (I can comment as much if you want, but I've found that contributor a little strange to interact with so if you are already reviewing it probably makes sense for you to handle it) | 12:29:11 |
4 Aug 2025 |
Randy Eckenrode | In reply to @emilazy:matrix.org nor do build inputs go after postPatch (there is no strict convention for all of this, but if you are going to do this kind of churn, explicitly making things comply less with the guidelines in https://github.com/jtojnar/nixpkgs-hammering/blob/b0817c8269798a830bc42f58a01c7543e7c5268a/explanations/attribute-ordering.md is hard to justify) Is this documented anywhere as the convention to use? I’ve been grouping by phases (so build phase stuff together, etc) …. | 23:24:34 |
emily | I don't think there's any real consensus, but I think the nixpkgs-hammering ordering is something close to what's most common (i.e. probably many packages violate it in some way or another but if you were going to write down the most consistent set of rules you could based on patterns of relative ordering it would be close to that) | 23:25:41 |
emily | I think some things are pretty strong consensus (passthru and meta last, pname , version , src , and usually outputs and patches before other stuff) | 23:26:31 |
emily | and everything in between is a little fuzzy | 23:26:35 |
5 Aug 2025 |
| Axel Karjalainen joined the room. | 17:39:37 |
Axel Karjalainen | Was there any reason for libcCrossChooser to use targetPackages ? See also my comment here: https://github.com/NixOS/nixpkgs/pull/409851#discussion_r2254887700
Is this the right Matrix room for this?
| 17:51:17 |
vcunat | #cross-compiling:nixos.org also exists, but I don't know what's the best place. | 17:54:58 |
6 Aug 2025 |
| vidariondr set a profile picture. | 18:31:02 |
8 Aug 2025 |
| problems changed their profile picture. | 01:54:56 |
| 0x4A6F joined the room. | 06:32:02 |
| André Lima joined the room. | 18:41:24 |
12 Aug 2025 |
connor (he/him) (UTC-7) | If you were trying to add a stdenv for a new compiler which doesn’t support the same arguments as GCC and Clang, is there an easy way to filter out unsupported arguments?
As an example, suppose -frandom-seed wasn’t supported (which is used by https://github.com/NixOS/nixpkgs/blob/0300b04b2e48edec1f9934cda0fb6d4a2a2c1fbd/pkgs/stdenv/generic/default.nix#L83) — what can be done? | 20:51:30 |
emily | I think we just conditionalize those as appropriate, but for something out-of-tree or where it would be invasive, you could drop them in a wrapper I suppose | 20:52:27 |
emily | (of course ideally it would support the flags) | 20:52:36 |
connor (he/him) (UTC-7) | Any references for prior art for how to do that, or recommendations for where such a wrapper would go? | 20:54:34 |