From 9e9cc471b663ae318fbddd98232137640500c195 Mon Sep 17 00:00:00 2001 From: Benjamin Hipple Date: Sun, 22 Apr 2018 18:24:44 -0400 Subject: [PATCH] dejavu-fonts: fix missing Perl IOString dependency (#39341) Per the docs, IOString was recently added as a dependency: https://github.com/dejavu-fonts/dejavu-fonts/blob/master/BUILDING.md As-is, attempting to rebulid from source errors out with this missing perl module. --- pkgs/data/fonts/dejavu-fonts/default.nix | 4 ++-- pkgs/top-level/all-packages.nix | 4 +--- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/pkgs/data/fonts/dejavu-fonts/default.nix b/pkgs/data/fonts/dejavu-fonts/default.nix index 23158a38f2189a..251654e2d58dd5 100644 --- a/pkgs/data/fonts/dejavu-fonts/default.nix +++ b/pkgs/data/fonts/dejavu-fonts/default.nix @@ -1,4 +1,4 @@ -{fetchFromGitHub, stdenv, fontforge, perl, FontTTF}: +{ fetchFromGitHub, stdenv, fontforge, perl, perlPackages }: let version = "2.37"; @@ -25,7 +25,7 @@ let full-ttf = stdenv.mkDerivation { name = "dejavu-fonts-full-${version}"; - nativeBuildInputs = [fontforge perl FontTTF]; + nativeBuildInputs = [fontforge perl perlPackages.IOString perlPackages.FontTTF]; src = fetchFromGitHub { owner = "dejavu-fonts"; diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix index 5bd83347ab0929..d07ac59c80f12a 100644 --- a/pkgs/top-level/all-packages.nix +++ b/pkgs/top-level/all-packages.nix @@ -14241,9 +14241,7 @@ with pkgs; crimson = callPackage ../data/fonts/crimson {}; - dejavu_fonts = lowPrio (callPackage ../data/fonts/dejavu-fonts { - inherit (perlPackages) FontTTF; - }); + dejavu_fonts = lowPrio (callPackage ../data/fonts/dejavu-fonts {}); # solve collision for nix-env before https://github.com/NixOS/nix/pull/815 dejavu_fontsEnv = buildEnv {