!VRULIdgoKmKPzJZzjj:nixos.org

Nix Package Manager development

864 Members
For people hacking on Nix: https://github.com/NixOS/nix Nix maintainers can be reached here.184 Servers

Load older messages


SenderMessageTime
6 Apr 2025
@Las:matrix.orgLasHonestly it’s not clear to me why the output path is in the derivation at all11:18:56
@Las:matrix.orgLasDoing content addressed drvs only btw11:19:11
@flokli:matrix.orgflokliYou shouldn't be manually adding your own derivations, especially if you're not also calculating Output paths (correctly)11:24:22
@flokli:matrix.orgflokliThe only "official" way to do is by writing nix code, and having the evaluator emit it. For everything else, you're off the well-tested path.11:25:10
@flokli:matrix.orgflokliIt's unfortunate derivations are added as aterm into the store in first place. IMHO they should only be valid for the lifetime of the evaluation itself, and not be put in the store 11:26:05
@sinan:sinanmohd.comsinan changed their profile picture.11:32:59
@emilazy:matrix.orgemilykind of at odds with the dynamic-derivations model, right?11:51:49
@emilazy:matrix.orgemily(not saying that I necessarily disagree, just that it seems like there's a commitment to some serialized derivation form at this point)11:52:21
@p14:matrix.orgp14
In reply to @roberthensing:matrix.org
Maybe nix.overrideAttrs could throw a useful error referring to a monolithic nixVersions.nix_mono package to ease the transition
Just hit this and would have appreciated such an error.
11:54:32
@p14:matrix.orgp14Actually struggling a bit to untangle what is going on and why I can’t apply a patch. Does nix mono exist yet?11:55:32
@Las:matrix.orgLas
In reply to @flokli:matrix.org
You shouldn't be manually adding your own derivations, especially if you're not also calculating Output paths (correctly)
I’m doing dyn drvs
12:39:18
@Las:matrix.orgLasI don’t want to depend on the nix evaluator in my drv12:39:27
@emilazy:matrix.orgemily (are you sure your problem is not just missing ^*?) 12:51:09
@Las:matrix.orgLas
In reply to @emilazy:matrix.org
(are you sure your problem is not just missing ^*?)
What
13:25:30
@Las:matrix.orgLasI never considered that13:25:41
@Las:matrix.orgLasWhy would you need that13:25:46
@emilazy:matrix.orgemily nix build /nix/store/foo.drv builds that drv, …^out builds that output 13:25:52
@emilazy:matrix.orgemilythere was a warning for it for many releases but it got removed13:25:58
@emilazy:matrix.orgemily the idea is that nix build X always produces X 13:26:03
@emilazy:matrix.orgemily so nix build /nix/build/non-drv-store-path will try to substitute that path, e.g. 13:26:16
@Las:matrix.orgLasMakes sense13:37:12
@Las:matrix.orgLas I think it’s a misnomer 13:37:44
@Las:matrix.orgLasShould be nix realize13:37:49
@vcunat:matrix.orgvcunatFYI, some (cross-)builds got broken now on Hydra, e.g. https://hydra.nixos.org/build/294450042/nixlog/1/tail14:59:18
@vcunat:matrix.orgvcunat I'm a bit surprised myself that -Wundef triggers on preprocessor this way. 15:00:08
@vcunat:matrix.orgvcunat I suppose you'd simply do #ifdef __APPLE__ and #elif defined(__APPLE__). Most likely easier than fighting what compiler considers a warning. 15:03:47
@vcunat:matrix.orgvcunat * I suppose you'd simply do #ifdef __APPLE__ and #elif defined(__APPLE__). Most likely it's easier than fighting what compiler considers a warning. 15:04:00
7 Apr 2025
@ss:someonex.netSomeoneSerge (back on matrix)
In reply to @flokli:matrix.org
It's unfortunate derivations are added as aterm into the store in first place. IMHO they should only be valid for the lifetime of the evaluation itself, and not be put in the store

RE: validity

Resolved drvs are ok though, aside from aterm? In the sense their contents are inherently consistent with path. Assuming nix doesn't change.

06:37:08
@joerg:thalheim.ioMic92https://github.com/NixOS/nixpkgs/pull/396773 nixVersions.latest: 2.26 -> 2.2811:30:47
@joerg:thalheim.ioMic92 https://github.com/NixOS/nixpkgs/pull/396750 John Ericson nix-eval-jobs: 2.26.0 -> 2.28.0 12:14:46

Show newer messages


Back to Room ListRoom Version: 6