From 7343a3b642cbdbac47770fe24d65aa9f9d23e6fa Mon Sep 17 00:00:00 2001 From: Adam Joseph Date: Sun, 6 Nov 2022 23:08:33 -0800 Subject: [PATCH] boehmgc: switch from versionAtLeast to == for powerpc workaround --- pkgs/development/libraries/boehm-gc/default.nix | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/pkgs/development/libraries/boehm-gc/default.nix b/pkgs/development/libraries/boehm-gc/default.nix index 0e7a936b4c2a..34e26f547093 100644 --- a/pkgs/development/libraries/boehm-gc/default.nix +++ b/pkgs/development/libraries/boehm-gc/default.nix @@ -30,10 +30,14 @@ stdenv.mkDerivation (finalAttrs: { ++ lib.optional enableMmap "--enable-mmap" ++ lib.optional enableLargeConfig "--enable-large-config"; - # this stanza can be dropped when a release fixes this issue: - # https://github.com/ivmai/bdwgc/issues/376 + # This stanza can be dropped when a release fixes this issue: + # https://github.com/ivmai/bdwgc/issues/376 + # The version is checked with == instead of versionAtLeast so we + # don't forget to disable the fix (and if the next release does + # not fix the problem the test failure will be a reminder to + # extend the set of versions requiring the workaround). makeFlags = if (stdenv.hostPlatform.isPower64 && - lib.versionAtLeast finalAttrs.version "8.2.2") + finalAttrs.version == "8.2.2") then [ # do not use /proc primitives to track dirty bits; see: # https://github.com/ivmai/bdwgc/issues/479#issuecomment-1279687537