From 1048867fe06f9473a51a7d33e95aaf5b0fefc0cc Mon Sep 17 00:00:00 2001 From: Daniel Hill Date: Tue, 3 Oct 2023 20:28:40 +1300 Subject: [PATCH] bcachefs: fix version not showing correctly buildLinux doesn't provide argsOverride by default. Prior to 663caaa, a default kernel was replaced with a direct invocation to buildLinux, this broke the use of argsOverride, in 663caaa, the use of argsOverride was removed, and then later the invocation of buildLinux was replaced with vanilla linux kernel that required argsOverride which then rebroke things again. Recommend either: put argsOverride inside buildLinux, or replace it with something less confusing to maintainers. --- .../linux/kernel/linux-testing-bcachefs.nix | 21 ++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/pkgs/os-specific/linux/kernel/linux-testing-bcachefs.nix b/pkgs/os-specific/linux/kernel/linux-testing-bcachefs.nix index e0fb6c7c4e0c..9709e698bfd7 100644 --- a/pkgs/os-specific/linux/kernel/linux-testing-bcachefs.nix +++ b/pkgs/os-specific/linux/kernel/linux-testing-bcachefs.nix @@ -10,18 +10,25 @@ , argsOverride ? {} , ... } @ args: - +let localversion = "-bcachefs-unstable-${version.date}"; +in # NOTE: bcachefs-tools should be updated simultaneously to preserve compatibility (kernel.override ( args // { - version = "${kernel.version}-bcachefs-unstable-${version.date}"; - extraMeta = { - branch = "master"; - broken = stdenv.isAarch64; - maintainers = with lib.maintainers; [ davidak Madouura pedrohlc raitobezarius YellowOnion ]; - }; + argsOverride = { + version = "${kernel.version}${localversion}"; + modDirVersion = "${kernel.version}${localversion}"; + + extraMeta = { + homepage = "https://bcachefs.org/"; + branch = "master"; + maintainers = with lib.maintainers; [ davidak Madouura pedrohlc raitobezarius YellowOnion ]; + }; + } // argsOverride; structuredExtraConfig = with lib.kernel; { + # we need this for uname + LOCALVERSION = freeform localversion; BCACHEFS_FS = module; BCACHEFS_QUOTA = option yes; BCACHEFS_POSIX_ACL = option yes;