Unverified Commit d807892d authored by Fernando Rodrigues's avatar Fernando Rodrigues Committed by Morgan Jones
Browse files

check-meta: wrap maintainers attribute to include team members 



This is a work-around that resolves the issues that spawned from #400458 by adding the team members from teams declared in `meta.teams` to `meta.maintainers`, effectively restoring the original behaviour of the `maintainers` attribute, and thus allowing downstreams to keep consuming `meta.maintainers` without any extra changes to support `meta.teams`.

Follow-up to #394797.

Signed-off-by: default avatarFernando Rodrigues <alpha@sigmasquadron.net>
parent 246654b4

pkgs/stdenv/generic/check-meta.nix

+5 −0
Original line number Diff line number Diff line
@@ -594,6 +594,11 @@ let 
    // optionalAttrs (pos != null) {

      position = pos.file + ":" + toString pos.line;

    }

    // {

      maintainers =

        attrs.meta.maintainers or [ ]

        ++ concatMap (team: team.members or [ ]) attrs.meta.teams or [ ];

    }

    // {

      # Expose the result of the checks for everyone to see.

      unfree = hasUnfreeLicense attrs;