Unverified Commit eddb12ac authored by Yueh-Shun Li's avatar Yueh-Shun Li Committed by Philip Taron
Browse files

mkDerivation: make allowedReferences/allowedRequisites nullable when __structuredAttrs is true

parent 4d3c76c4

pkgs/stdenv/generic/make-derivation.nix

+7 −2
Original line number Diff line number Diff line
@@ -15,6 +15,7 @@ let 
    elemAt

    extendDerivation

    filter

    filterAttrs

    getDev

    head

    imap1
@@ -709,8 +710,12 @@ let 
              ];

          }

          // (

            let

              attrsOutputChecks = makeOutputChecks attrs;

              attrsOutputChecksFiltered = filterAttrs (_: v: v != null) attrsOutputChecks;

            in

            if !__structuredAttrs then

              makeOutputChecks attrs

              attrsOutputChecks

            else

              {

                outputChecks = builtins.listToAttrs (
@@ -719,7 +724,7 @@ let 
                    value =

                      let

                        raw = zipAttrsWith (_: builtins.concatLists) [

                          (makeOutputChecks attrs)

                          attrsOutputChecksFiltered

                          (makeOutputChecks attrs.outputChecks.${name} or { })

                        ];

                      in

pkgs/test/overriding.nix

+10 −0
Original line number Diff line number Diff line
@@ -68,6 +68,16 @@ let 
          }).pname;

        expected = "hello-no-final-attrs-overridden";

      };

      structuredAttrs-allowedRequisites-nullability = {

        expr =

          lib.hasPrefix builtins.storeDir

            (pkgs.stdenv.mkDerivation {

              __structuredAttrs = true;

              inherit (pkgs.hello) pname version src;

              allowedRequisites = null;

            }).drvPath;

        expected = true;

      };

    };



  test-extendMkDerivation =