dbus: minor update, reorganize and polish the expression

pkgs/development/libraries/dbus/default.nix

@@ -3,39 +3,27 @@
 , libX11, libICE, libSM, useX11 ? true }:
 
 let
-  version = "1.6.4";
+  version = "1.6.8"; # 1.7.* isn't recommended, even for gnome 3.8
+  sha256 = "1b0vq5b81synr0hqsfyypyq5yw305q0fq1f9alzv3vmb73pp04zw";
+
+  inherit (stdenv) lib;
+
+  buildInputsX = lib.optionals useX11 [ libX11 libICE libSM ];
+
+  # also other parts than "libs" need this statically linked lib
+  makeInternalLib = "(cd dbus && make libdbus-internal.la)";
+
+
+  # A generic builder for individual parts (subdirs) of D-Bus
+  dbus_drv = name: subdirs: merge: stdenv.mkDerivation (lib.mergeAttrsByFuncDefaultsClean [{
+
+    name = "dbus-${name}-${version}";
 
     src = fetchurl {
       url = "http://dbus.freedesktop.org/releases/dbus/dbus-${version}.tar.gz";
-    sha256 = "1wacqyfkcpayg7f8rvx9awqg275n5pksxq5q7y21lxjx85x6pfjz";
+      inherit sha256;
     };
 
-  patches = [
-    ./ignore-missing-includedirs.patch ./implement-getgrouplist.patch
-    ./ucred-dirty-hack.patch ./no-create-dirs.patch
-  ];
-
-  # build only the specified subdirs
-  postPatch = subdirs : "sed '/SUBDIRS/s/=.*/=" + subdirs + "/' -i Makefile.am\n"
-    # use already packaged libdbus instead of trying to build it again
-    + stdenv.lib.optionalString (subdirs != "dbus") ''
-        for mfile in */Makefile.am; do
-          sed 's,\$(top_builddir)/dbus/\(libdbus-[0-9]\),${libs}/lib/\1,g' -i "$mfile"
-        done
-      '';
-
-  nativeBuildInputs = [ pkgconfig ];
-
-  buildInputs = [ autoconf automake libtool expat ]; # ToDo: optional selinux?
-
-  buildInputsWithX = buildInputs ++ stdenv.lib.optionals useX11 [ libX11 libICE libSM ];
-
-  preConfigure = ''
-    patchShebangs .
-    substituteInPlace tools/Makefile.am --replace 'install-localstatelibDATA:' 'disabled:'
-    autoreconf -fi
-  '';
-
     configureFlags = [
       "--localstatedir=/var"
       "--sysconfdir=/etc"
@@ -43,88 +31,74 @@ let
       "--with-systemdsystemunitdir=$(out)/lib/systemd"
     ];
 
-  # also other parts than "libs" need this statically linked lib
+    preConfigure = ''
-  postConfigure = "(cd dbus && make libdbus-internal.la)";
+      patchShebangs .
-
+      substituteInPlace tools/Makefile.am --replace 'install-localstatelibDATA:' 'disabled:'
-  doCheck = true;
+      autoreconf -fi
+    '';
 
     installFlags = "sysconfdir=$(out)/etc";
 
+    doCheck = true;
 
-  libs = stdenv.mkDerivation {
+    patches = [
-    name = "dbus-library-" + version;
+      ./ignore-missing-includedirs.patch ./implement-getgrouplist.patch
+      ./ucred-dirty-hack.patch ./no-create-dirs.patch
+    ];
 
-    inherit src nativeBuildInputs preConfigure configureFlags doCheck installFlags;
+    nativeBuildInputs = [ pkgconfig ];
+    propagatedBuildInputs = [ expat ];
+    buildInputs = [ autoconf automake libtool ]; # ToDo: optional selinux?
 
-    buildInputs = buildInputs ++ [ systemd.headers ];
+    # build only the specified subdirs
+    postPatch = "sed '/SUBDIRS/s/=.*/=" + subdirs + "/' -i Makefile.am\n"
+      # use already packaged libdbus instead of trying to build it again
+      + lib.optionalString (name != "libs") ''
+          for mfile in */Makefile.am; do
+            sed 's,\$(top_builddir)/dbus/\(libdbus-[0-9]\),${libs}/lib/\1,g' -i "$mfile"
+          done
+        '';
 
-    patches = patches ++ [ ./systemd.patch ]; # bypass systemd detection
+  } merge ]);
 
-    postPatch = postPatch "dbus";
+  libs = dbus_drv "libs" "dbus" {
+    buildInputs = [ systemd.headers ];
+    patches = [ ./systemd.patch ]; # bypass systemd detection
 
-    # Enable X11 autolaunch support in libdbus.  This doesn't actually
+    # Enable X11 autolaunch support in libdbus. This doesn't actually depend on X11
-    # depend on X11 (it just execs dbus-launch in dbus.tools),
+    # (it just execs dbus-launch in dbus.tools), contrary to what the configure script demands.
-    # contrary to what the configure script demands.
     NIX_CFLAGS_COMPILE = "-DDBUS_ENABLE_X11_AUTOLAUNCH=1";
   };
 
-  tools = stdenv.mkDerivation {
+in rec {
-    name = "dbus-tools-" + version;
 
-    inherit src patches nativeBuildInputs preConfigure doCheck installFlags;
+  # This package has been split because most applications only need dbus.lib
+  # which serves as an interface to a *system-wide* daemon,
+  # see e.g. http://en.wikipedia.org/wiki/D-Bus#Architecture .
+  # Also some circular dependencies get split by this (like with systemd).
 
-    configureFlags = configureFlags ++ [ "--with-dbus-daemondir=${daemon}/bin" ];
+  inherit libs;
-
-    buildInputs = buildInputsWithX ++ [ libs daemon systemd dbus_glib ];
-
-    NIX_CFLAGS_LINK = "-Wl,--as-needed -ldbus-1";
-
-    postPatch = postPatch "tools";
-  };
-
-  daemon = stdenv.mkDerivation {
-    name = "dbus-daemon-" + version;
-
-    inherit src patches nativeBuildInputs
-      preConfigure configureFlags postConfigure doCheck installFlags;
-
-    buildInputs = buildInputs ++ [ systemd ];
-
-    postPatch = postPatch "bus";
-  };
-
-  tests = stdenv.mkDerivation {
-    name = "dbus-tests-" + version;
-
-    inherit src patches nativeBuildInputs
-      preConfigure configureFlags postConfigure doCheck installFlags;
-
-    buildInputs = buildInputsWithX ++ [ systemd libs tools daemon dbus_glib python ];
-
-    postPatch = postPatch "test";
 
+  tools = dbus_drv "tools" "tools" {
+    configureFlags = [ "--with-dbus-daemondir=${daemon}/bin" ];
+    buildInputs = buildInputsX ++ [ libs daemon systemd dbus_glib ];
     NIX_CFLAGS_LINK = "-Wl,--as-needed -ldbus-1";
   };
 
-  docs = stdenv.mkDerivation {
+  daemon = dbus_drv "daemon" "bus" {
-    name = "dbus-docs-" + version;
+    preBuild = makeInternalLib;
+    buildInputs = [ systemd ];
+  };
 
-    inherit src patches nativeBuildInputs
+  # Some of the tests don't work yet; in fact, @vcunat tried several packages
-      preConfigure configureFlags doCheck installFlags;
+  # containing dbus testing, and all of them have some test failure.
-
+  tests = dbus_drv "tests" "test" {
-    buildInputs = buildInputs;
+    preBuild = makeInternalLib;
-
+    buildInputs = buildInputsX ++ [ systemd libs tools daemon dbus_glib python ];
-    postPatch = postPatch "doc";
+    NIX_CFLAGS_LINK = "-Wl,--as-needed -ldbus-1";
+  };
 
+  docs = dbus_drv "docs" "doc" {
     postInstall = ''rm -r "$out/lib"'';
   };
-
-in {
-  inherit libs daemon tools tests docs;
-
-  dbus_libs   = libs;
-  dbus_daemon = daemon;
-  dbus_tools  = tools;
-  dbus_tests  = tests;
-  dbus_docs   = docs;
 }

pkgs/lib/misc.nix

@@ -315,6 +315,8 @@ rec {
       )
     ];
   mergeAttrsByFuncDefaults = foldl mergeAttrByFunc { inherit mergeAttrBy; };
+  mergeAttrsByFuncDefaultsClean = list: removeAttrs (mergeAttrsByFuncDefaults list) ["mergeAttrBy"];
+
   # sane defaults (same name as attr name so that inherit can be used)
   mergeAttrBy = # { buildInputs = concatList; [...]; passthru = mergeAttr; [..]; }
     listToAttrs (map (n : nameValuePair n lib.concat) [ "nativeBuildInputs" "buildInputs" "propagatedBuildInputs" "configureFlags" "prePhases" "postAll" ])

pkgs/top-level/all-packages.nix

@@ -3582,13 +3582,14 @@ let
 
   db48 = callPackage ../development/libraries/db4/db4-4.8.nix { };
 
-  dbus = dbus_all;
+  dbus = let dbus_all = callPackage ../development/libraries/dbus { };
-  dbus_all = callPackage ../development/libraries/dbus { };
+    in dbus_all.libs // dbus_all; # previously dbus.libs also contained the daemon
-  inherit (dbus_all) dbus_libs dbus_daemon dbus_tools dbus_tests dbus_docs;
   dbus_cplusplus  = callPackage ../development/libraries/dbus-cplusplus { };
   dbus_glib       = callPackage ../development/libraries/dbus-glib { };
   dbus_java       = callPackage ../development/libraries/java/dbus-java { };
   dbus_python     = callPackage ../development/python-modules/dbus { };
+  # Should we deprecate these? Currently there are many references.
+  dbus_tools = dbus.tools; dbus_libs = dbus.libs;
 
   dclib = callPackage ../development/libraries/dclib { };
 

pkgs/top-level/release-small.nix

@@ -200,6 +200,7 @@ with (import ./release-lib.nix);
 
   dbus_all = {
     libs = linux;
+    daemon = linux;
     tools = linux;
   };