Unverified Commit c174d176 authored by Ashley Ruglys's avatar Ashley Ruglys
Browse files

opensubdiv: add metal support 



Apply suggestions from code review

Co-authored-by: default avatarPol Dellaiera <pol.dellaiera@protonmail.com>

Apply suggestions: split darwin frameworks into individual lines, use cmakeBool for metal support
parent 750c0ff4

pkgs/development/libraries/opensubdiv/default.nix

+12 −4
Original line number Diff line number Diff line
@@ -31,9 +31,17 @@ stdenv.mkDerivation rec { 
      glew xorg.libX11 xorg.libXrandr xorg.libXxf86vm xorg.libXcursor

      xorg.libXinerama xorg.libXi

    ]

    ++ lib.optional (openclSupport && !stdenv.isDarwin) ocl-icd

    ++ lib.optionals stdenv.isDarwin (with darwin.apple_sdk.frameworks; [OpenCL Cocoa CoreVideo IOKit AppKit AGL ])

    ++ lib.optional cudaSupport [

    ++ lib.optionals (openclSupport && !stdenv.isDarwin) [ ocl-icd ]

    ++ lib.optionals stdenv.isDarwin (with darwin.apple_sdk.frameworks; [

      OpenCL

      Cocoa

      CoreVideo

      IOKit

      AppKit

      AGL

      MetalKit

    ])

    ++ lib.optionals cudaSupport [

      cudaPackages.cuda_cudart

    ];
@@ -50,7 +58,7 @@ stdenv.mkDerivation rec { 
    [ "-DNO_TUTORIALS=1"

      "-DNO_REGRESSION=1"

      "-DNO_EXAMPLES=1"

      "-DNO_METAL=1" # don’t have metal in apple sdk

      (lib.cmakeBool "NO_METAL" (!stdenv.isDarwin))

      (lib.cmakeBool "NO_OPENCL" (!openclSupport))

      (lib.cmakeBool "NO_CUDA" (!cudaSupport))

    ] ++ lib.optionals (!stdenv.isDarwin) [