From c856d7ff128b4d6ed87bae0a5b10692763a0e63b Mon Sep 17 00:00:00 2001 From: Alyssa Ross Date: Thu, 30 Jan 2020 19:37:29 +0000 Subject: [PATCH] dino: remove unnecessary private transitive deps As explained by Orivej (reformatted from Markdown): > These (except libsignal-protocol-c) are private dependencies of the > actual dependencies that should neither be propagated nor added > manually to dino. For example, libpsl and brotli come from > libsoup-2.4.pc: > > Requires: glib-2.0 >= 2.38, gobject-2.0 >= 2.38, gio-2.0 >= 2.38 > Requires.private: libxml-2.0, sqlite3, libpsl >= 0.20, libbrotlidec, zlib > > (To be precise, glib uses utillinuxMinimal rather than utillinux.) > > The warnings we see, such as Package 'mount', required by 'gio-2.0', > not found, come from CMake running both pkg-config --libs and > pkg-config --static --libs to populate both _LIBRARIES and > _STATIC_LIBRARIES[1], but dino has no use for the latter. > Currently these warnings can not be disabled: > https://gitlab.kitware.com/cmake/cmake/issues/18158 > > (They could be prevented by pruning Requires.private from shared-only > libraries akin to , > although it can not be detected if a library is shared-only from the > .pc file alone, and this is just a warning.) > > [1]: docs: https://cmake.org/cmake/help/v3.16/module/FindPkgConfig.html --- .../networking/instant-messengers/dino/default.nix | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/pkgs/applications/networking/instant-messengers/dino/default.nix b/pkgs/applications/networking/instant-messengers/dino/default.nix index 8928814e967..9cf0032cf49 100644 --- a/pkgs/applications/networking/instant-messengers/dino/default.nix +++ b/pkgs/applications/networking/instant-messengers/dino/default.nix @@ -2,7 +2,7 @@ , vala, cmake, ninja, wrapGAppsHook, pkgconfig, gettext , gobject-introspection, gnome3, glib, gdk-pixbuf, gtk3, glib-networking , xorg, libXdmcp, libxkbcommon -, libnotify, libsoup, libgee, utillinux, libselinux, libsepol, libpsl, brotli +, libnotify, libsoup, libgee , librsvg, libsignal-protocol-c , libgcrypt , epoxy @@ -52,18 +52,12 @@ stdenv.mkDerivation rec { pcre xorg.libxcb xorg.libpthreadstubs - xorg.libXtst libXdmcp libxkbcommon epoxy at-spi2-core dbus icu - utillinux - libselinux - libsepol - libpsl - brotli libsignal-protocol-c librsvg ];