Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

poppler: 21.05.0 -> 21.06.01 #127832

Merged
merged 3 commits into from
Jun 23, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 34 additions & 10 deletions pkgs/development/libraries/poppler/0.61.nix
Original file line number Diff line number Diff line change
@@ -1,5 +1,20 @@
{ stdenv, lib, fetchurl, cmake, ninja, pkg-config, libiconv, libintl
, zlib, curl, cairo, freetype, fontconfig, lcms, libjpeg, openjpeg, fetchpatch
{ stdenv
, lib
, fetchurl
, fetchpatch
, cairo
, cmake
, curl
, fontconfig
, freetype
, lcms
, libiconv
, libintl
, libjpeg
, ninja
, openjpeg
, pkg-config
, zlib
, withData ? true, poppler_data
, qt5Support ? false, qtbase ? null
, introspectionSupport ? false, gobject-introspection ? null
Expand All @@ -12,25 +27,38 @@ let
mkFlag = optset: flag: "-DENABLE_${flag}=${if optset then "on" else "off"}";
in
stdenv.mkDerivation rec {
name = "poppler-${suffix}-${version}";
pname = "poppler-${suffix}";
inherit version;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not just move version to here?


src = fetchurl {
url = "${meta.homepage}/poppler-${version}.tar.xz";
url = "https://poppler.freedesktop.org/poppler-${version}.tar.xz";
sha256 = "1afdrxxkaivvviazxkg5blsf2x24sjkfj92ib0d3q5pm8dihjrhj";
};

outputs = [ "out" "dev" ];

patches = [
# Fix internal crash: a negative number that should not be
(fetchpatch {
name = "CVE-2018-13988";
url = "https://cgit.freedesktop.org/poppler/poppler/patch/?id=004e3c10df0abda214f0c293f9e269fdd979c5ee";
sha256 = "1l8713s57xc6g81bldw934rsfm140fqc7ggd50ha5mxdl1b3app2";
})
# Fix internal crash: a negative number that should not be (not the above!)
./0.61-CVE-2019-9959.patch
];

buildInputs = [ libiconv libintl ] ++ lib.optional withData poppler_data;
nativeBuildInputs = [
cmake
ninja
pkg-config
];

buildInputs = [
libiconv
libintl
]
++ lib.optional withData poppler_data;
Comment on lines +60 to +61
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
]
++ lib.optional withData poppler_data;
] ++ lib.optional withData poppler_data;


# TODO: reduce propagation to necessary libs
propagatedBuildInputs = with lib;
Expand All @@ -39,8 +67,6 @@ stdenv.mkDerivation rec {
++ optional qt5Support qtbase
++ optional introspectionSupport gobject-introspection;

nativeBuildInputs = [ cmake ninja pkg-config ];

# Not sure when and how to pass it. It seems an upstream bug anyway.
CXXFLAGS = lib.optionalString stdenv.cc.isClang "-std=c++11";

Expand All @@ -58,12 +84,10 @@ stdenv.mkDerivation rec {
meta = with lib; {
homepage = "https://poppler.freedesktop.org/";
description = "A PDF rendering library";

longDescription = ''
Poppler is a PDF rendering library based on the xpdf-3.0 code base.
'';

license = licenses.gpl2;
license = licenses.gpl2Plus;
platforms = platforms.all;
maintainers = with maintainers; [ ttuegel ];
};
Expand Down
45 changes: 23 additions & 22 deletions pkgs/development/libraries/poppler/default.nix
Original file line number Diff line number Diff line change
@@ -1,44 +1,46 @@
{ stdenv
, lib
{ lib
, stdenv
, fetchurl
, fetchpatch
, cairo
, cmake
, ninja
, pkg-config
, libiconv
, libintl
, zlib
, pcre
, boost
, cups-filters
, curl
, cairo
, freetype
, fontconfig
, freetype
, inkscape
, lcms
, libiconv
, libintl
, libjpeg
, ninja
, openjpeg
, pkg-config
, scribusUnstable
, texlive
, zlib
, withData ? true, poppler_data
, qt5Support ? false, qtbase ? null
, introspectionSupport ? false, gobject-introspection ? null
, utils ? false, nss ? null
Comment on lines 25 to 27
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
, qt5Support ? false, qtbase ? null
, introspectionSupport ? false, gobject-introspection ? null
, utils ? false, nss ? null
, qt5Support ? false, qtbase
, introspectionSupport ? false, gobject-introspection
, utils ? false, nss

, minimal ? false
, suffix ? "glib"
, inkscape
, cups-filters
, texlive
, scribusUnstable
}:

let
mkFlag = optset: flag: "-DENABLE_${flag}=${if optset then "on" else "off"}";
in
stdenv.mkDerivation rec {
name = "poppler-${suffix}-${version}";
version = "21.05.0"; # beware: updates often break cups-filters build, check texlive and scribusUnstable too!
pname = "poppler-${suffix}";
version = "21.06.1"; # beware: updates often break cups-filters build, check texlive and scribusUnstable too!

outputs = [ "out" "dev" ];

src = fetchurl {
url = "${meta.homepage}/poppler-${version}.tar.xz";
sha256 = "sha256-2v1Te2gPrRIVvED8U9HzjoRJ18GFvGDVqJ4dJskNvYw=";
url = "https://poppler.freedesktop.org/poppler-${version}.tar.xz";
sha256 = "sha256-hrCeWgLeQAgaORbvhxHFEo6vSx/FnV+H0Oxm8E9ZXbQ=";
};

nativeBuildInputs = [
Expand All @@ -48,6 +50,8 @@ stdenv.mkDerivation rec {
];

buildInputs = [
boost
pcre
libiconv
libintl
] ++ lib.optional withData [
Expand Down Expand Up @@ -98,13 +102,10 @@ stdenv.mkDerivation rec {
meta = with lib; {
homepage = "https://poppler.freedesktop.org/";
description = "A PDF rendering library";

longDescription = ''
Poppler is a PDF rendering library based on the xpdf-3.0 code
base. In addition it provides a number of tools that can be
installed separately.
Poppler is a PDF rendering library based on the xpdf-3.0 code base. In
addition it provides a number of tools that can be installed separately.
'';

license = licenses.gpl2Plus;
platforms = platforms.all;
maintainers = with maintainers; [ ttuegel ] ++ teams.freedesktop.members;
Expand Down
5 changes: 4 additions & 1 deletion pkgs/top-level/all-packages.nix
Original file line number Diff line number Diff line change
Expand Up @@ -17697,7 +17697,10 @@ in
suffix = "min";
};

poppler_utils = poppler.override { suffix = "utils"; utils = true; };
poppler_utils = poppler.override {
suffix = "utils";
utils = true;
};

popt = callPackage ../development/libraries/popt { };

Expand Down