!VRULIdgoKmKPzJZzjj:nixos.org

Nix Hackers

909 Members
For people hacking on the Nix package manager itself190 Servers

Load older messages


SenderMessageTime
17 Mar 2025
@jade_:matrix.orgjade_this is the wrong channel for this. please check out a nixpkgs dev channel or so instead05:30:45
@egrieco:matrix.orgegrieco joined the room.06:49:11
@piegames:flausch.socialpiegames joined the room.08:07:02
@Ericson2314:matrix.orgJohn Ericson Robert Hensing (roberth): https://github.com/NixOS/nixpkgs/pull/390479 can we merge this manual backport of your PR to 24.11? 15:42:50
@roberthensing:matrix.orgRobert Hensing (roberth)I would think so, yeah15:43:34
@Ericson2314:matrix.orgJohn EricsonOK thanks15:45:12
@emilazy:matrix.orgemily pkgsStatic.nixVersions.nix_2_26 is still broken 15:57:48
@emilazy:matrix.orgemily(at eval time)15:58:09
@Ericson2314:matrix.orgJohn Ericson emily: wait is that new in this PR? 16:15:18
@emilazy:matrix.orgemilyno, I reported it here before16:16:34
@emilazy:matrix.orgemilythe complicated splicing stuff that I don't understand why it's there is broken16:16:47
@Ericson2314:matrix.orgJohn Ericson Eelco: So the reason it is not a CanonPath is the symlink target and "location the source tree is for symlink resolution purposes" is an OS path 16:17:02
@Ericson2314:matrix.orgJohn Ericsoneven more confusing, in the case of remote file systems it might be an OS path of another operating system16:17:33
@Ericson2314:matrix.orgJohn EricsonI am wary of canonicalizing because we don't want a situation where the symlink points outside the source tree, but then another symlink points back inside, to allow the symlink to be resolved16:18:33
@Ericson2314:matrix.orgJohn Ericsonrather as soon as any intermediate encountered during symlink resolution leaves the source tree, it's game over16:19:30
@Ericson2314:matrix.orgJohn Ericson Eelco: Also happy with https://github.com/NixOS/nix/pull/12658 now? 17:18:53
@emilazy:matrix.orgemily (I was just mentioning it wrt bumping nixVersions.latest) 17:36:36
@emilazy:matrix.orgemily(and that more fixes will need backporting later)17:36:45
@aidetechbot:matrix.orgaidetechbotoh woops sorry22:57:37
18 Mar 2025
@sinan:sinanmohd.comsinan changed their profile picture.06:43:31
@trofi:matrix.orgtrofiAny chance to merge https://github.com/NixOS/nix/pull/12105? it's 3 months old.07:27:11
@roberthensing:matrix.orgRobert Hensing (roberth)It seems that an assertion is failing; see comment12:40:39
@thorhop:norge.chatBigBabyThor joined the room.14:48:57
@schromp:echsen.clubSchromp joined the room.15:56:47
@Ericson2314:matrix.orgJohn Ericson Las: I feel like we should merge my commits prior to my last one splitting the files, and then do your commits 18:17:50
@Ericson2314:matrix.orgJohn Ericson Robert Hensing (roberth): have any opinions on that? 18:18:01
@Las:matrix.orgLas
In reply to @Ericson2314:matrix.org
Las: I feel like we should merge my commits prior to my last one splitting the files, and then do your commits
From your PR or the restricted store one or?
18:19:00
@Ericson2314:matrix.orgJohn Ericson Las: from the big one (though yes that includes restricted store) 18:19:17
@Las:matrix.orgLasMy PR has the benefit of being simple, but it’s also simpler to rebase possibly18:19:39
@Las:matrix.orgLasMaybe smaller PR then with your changes? Which would include what!18:19:57

Show newer messages


Back to Room ListRoom Version: 6