!oeFELgatfbdiAEkJAY:nixos.org

Nix Zürich

116 Members
https://zurich.nix.ug/ - Next ZHF event 23-24 November 2024 Report notes: https://pad.lassul.us/dAqTN5maSqCRcvHuSun-Xg#28 Servers

Load older messages


SenderMessageTime
25 Nov 2024
@fricklerhandwerk:matrix.orgfricklerhandwerk *

It was a awesome seeing all of you (again), thanks everyone for coming! You made this the greatest Zürich ZHF hackathon ever.

I hope each of you could derive some pleasure from the event and take something away — I sure did. Would love to keep up that spirit in May!

07:18:54
@fricklerhandwerk:matrix.orgfricklerhandwerk Thanks a lot ners and john-rodewald for making all of this possible, and thanks to das-g for your invaluable support with the location. 07:20:04
@nebucatnetzer13:matrix.orgnebucatnetzer13 I absolutely agree, it is getting better every time.
Loved the group dinner 💪.
08:21:23
@lilalambda:matrix.orgCorsin Pfister This was my first time at ZHF, and I had a great time! Cool group of people! I really enjoyed seeing with how much fun and enthusiasm this was organized and put together. I'd like to thank ners , john-rodewald , das-g and everyone else who helped organizing this event! The group dinner was also a highlight for me 🙂 08:27:49
@fricklerhandwerk:matrix.orgfricklerhandwerk changed the room topic to "https://zurich.nix.ug/ - Next ZHF event 23-24 November 2024 Report notes: https://pad.lassul.us/dAqTN5maSqCRcvHuSun-Xg#" from "https://zurich.nix.ug/ - Next ZHF event 23-24 November 2024".09:18:46
@fricklerhandwerk:matrix.orgfricklerhandwerkHey all, I'd like to whip up a report to let others know what we've been up to. If you like to share you worked on, please do so here until this week Friday 2024-11-29: https://pad.lassul.us/dAqTN5maSqCRcvHuSun-Xg# One or two sentences is enough. Links and pictures would be great!09:22:08
@fricklerhandwerk:matrix.orgfricklerhandwerk * Hey all, I'd like to whip up a report to let others know what we've been up to. If you like to share what you worked on, please do so here until this week Friday 2024-11-29: https://pad.lassul.us/dAqTN5maSqCRcvHuSun-Xg# One or two sentences is enough. Links and pictures would be great!09:22:18
@ners:nixos.devners@room ping to help spread the word09:53:35
@ners:nixos.devners I‘ll send out a request for feedback later today or tomorrow. 09:57:38
@matthewcroughan:defenestrate.itmatthewcroughanIMG_20241125_171930.jpg
Download IMG_20241125_171930.jpg
17:20:47
@matthewcroughan:defenestrate.itmatthewcroughan @ners:nixos.dev !!! 17:20:51
26 Nov 2024
@koen:procolix.comKoen de Jonge (SynQ)

When I upgrade my system from nixos-24.05 to nixos-24.11 I get this:

building Nix...
building the system configuration...
error:
       … while calling the 'head' builtin
         at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/attrsets.nix:1:35741:
       … while evaluating the attribute 'value'
         at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:1:33591:
       … while evaluating the option `system.build.toplevel':

       … while evaluating definitions from `/nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/system/activation/top-level.nix':

       (stack trace truncated; use '--show-trace' to show the full, detailed trace)

       error:
       Failed assertions:
       - The option definition `sound' in `/etc/nixos/configuration.nix' no longer has any effect; please remove it.
       The option was heavily overloaded and can be removed from most configurations.

Is there any way we can make this a bit more 'user friendly'?

11:35:15
@koen:procolix.comKoen de Jonge (SynQ) *

When I upgrade my system from nixos-24.05 to nixos-24.11 I get this:

building Nix...
building the system configuration...
error:
       … while calling the 'head' builtin
         at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/attrsets.nix:1:35741:
       … while evaluating the attribute 'value'
         at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:1:33591:
       … while evaluating the option `system.build.toplevel':

       … while evaluating definitions from `/nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/system/activation/top-level.nix':

       (stack trace truncated; use '--show-trace' to show the full, detailed trace)

       error:
       Failed assertions:
       - The option definition `sound' in `/etc/nixos/configuration.nix' no longer has any effect; please remove it.
       The option was heavily overloaded and can be removed from most configurations.

Is there any way we can make this a bit more 'user friendly'?
Also, how should I submit a ticket for this and at what location?

11:35:51
@koen:procolix.comKoen de Jonge (SynQ)

It is triggered by this entry in configuration.nix which comes as a default (in a previous install method):

  # Enable sound.
  sound.enable = true;
11:36:42
@elikoga:matrix.orgelikoga

This is a combination of both nix errors being verbose in the wrong way

And nixos evaluation assertions triggering overly verbose errors.

If only the error message showed up with a reference to the definition, would that be a good behavior?

11:36:53
@koen:procolix.comKoen de Jonge (SynQ)I fear we will lose a few users of nixos over this.11:36:57
@koen:procolix.comKoen de Jonge (SynQ)I would prefer that NixOS would just ignore the statement and throw a warning that it is depreciated.11:37:34
@elikoga:matrix.orgelikoga

This is done/possible with some other settings

However not sound apparently

11:38:00
@micmine:matrix.orgemilyAre the pictures that were taken available somewhere?11:39:43
@infinisil:matrix.orginfinisil
In reply to @koen:procolix.com
I would prefer that NixOS would just ignore the statement and throw a warning that it is depreciated.
Here's the relevant code: https://github.com/NixOS/nixpkgs/blob/dafe47b3c8bb54c65e5a28bcad88894e4bb6a6e0/lib/modules.nix#L1104. This could relatively easily be changed to use config.warnings instead
11:41:31
@elikoga:matrix.orgelikoga Maybe there should be a mkDeprecatedOptionModule that warns for one release cycle? 11:42:15
@john-rodewald:nixos.devjohn-rodewald
In reply to @micmine:matrix.org
Are the pictures that were taken available somewhere?
We’re working on editing them. The group photo will be posted here later. :)
11:42:19
@elikoga:matrix.orgelikoga

But still, this is an example of breaking changes when upgrading from a half yearly release to the next

Outlined in the changelog and migration guide too: https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2411.section.md#sound-options-removal-sec-release-2411-migration-sound

11:45:08
@koen:fediversity.euKoen (SynQ) de Jonge
In reply to @elikoga:matrix.org

But still, this is an example of breaking changes when upgrading from a half yearly release to the next

Outlined in the changelog and migration guide too: https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2411.section.md#sound-options-removal-sec-release-2411-migration-sound

Do you mean that end users should not complain about that?
11:46:11
@elikoga:matrix.orgelikoga

I'm saying that in some cases, no better alternive exists. Breaking means breaking sometimes.

I know that if the enable option had an equivalent replacement "mkRenamedOptionModule" could've been used to warn about the rename.

In this case however, the external Api has changed and the courtesy of notifying an upgrading user has been fulfilled. Little can be done except delay the change associated with external api change.

11:48:05
@elikoga:matrix.orgelikoga Usually, to upgrade a deployment or project from one half-yearly release to the next, some work needs to be put in to adjust to breaking changes 11:49:42
@elikoga:matrix.orgelikoga

Also "kill [...] with fire" seems a bit heavy handed, it could certainly be spaced out on 2 release cycles with a warning in the first one

https://github.com/NixOS/nixpkgs/pull/326262

11:52:10
@infinisil:matrix.orginfinisilI think it's pretty standard to at least warn for a release before failing. Only in rare cases this isn't feasible, but here it is (by just ignoring the option being set). Honestly seems like a bit of an oversight, because it's rare for an option to be just removable like that.11:52:47
@koen:fediversity.euKoen (SynQ) de Jonge
In reply to @infinisil:matrix.org
I think it's pretty standard to at least warn for a release before failing. Only in rare cases this isn't feasible, but here it is (by just ignoring the option being set). Honestly seems like a bit of an oversight, because it's rare for an option to be just removable like that.
I like this answer 😍
11:53:38
@elikoga:matrix.orgelikogaI actually disagree on this one11:54:23

Show newer messages


Back to Room ListRoom Version: 10