treewide: Fix more lib.optional misuses

6 files changed, 19 insertions, 31 deletions

diff --git a/pkgs/applications/misc/webthree-umbrella/default.nix b/pkgs/applications/misc/webthree-umbrella/default.nix
index 2354c1377f9..8876b64ed0c 100644
--- a/pkgs/applications/misc/webthree-umbrella/default.nix
+++ b/pkgs/applications/misc/webthree-umbrella/default.nix
@@ -84,22 +84,10 @@ stdenv.mkDerivation rec {
     curl
     libmicrohttpd
     mesa
-    (optional withOpenCL [
-      opencl-headers
-      ocl-icd
-    ])
-    (optional withGUI [
-      qtwebengine
-      qtbase
-      qtdeclarative
-    ])
-    (optional withProfiling gperftools)
-    (optional withEVMJIT [
-      llvm
-      zlib
-      ncurses
-    ])
-  ];
+  ] ++ optionals withOpenCL [ opencl-headers ocl-icd ]
+    ++ optionals withGUI [ qtwebengine qtbase qtdeclarative ]
+    ++ optional withProfiling gperftools
+    ++ optionals withEVMJIT [ llvm zlib ncurses ];
 
   runPath = with stdenv.lib; (makeLibraryPath (flatten [ stdenv.cc.cc buildInputs ]));
 
diff --git a/pkgs/development/libraries/blitz/default.nix b/pkgs/development/libraries/blitz/default.nix
index ab34f64dfbf..96b53e1ad63 100644
--- a/pkgs/development/libraries/blitz/default.nix
+++ b/pkgs/development/libraries/blitz/default.nix
@@ -32,7 +32,7 @@ stdenv.mkDerivation rec {
   patches = [ ./blitz-gcc47.patch ./blitz-testsuite-stencil-et.patch ];
 
   buildInputs = [ pkgconfig gfortran texinfo ]
-    ++ optional (boost != null) [ boost.out ];
+    ++ optional (boost != null) boost;
 
   configureFlags =
     [ "--enable-shared"
diff --git a/pkgs/os-specific/linux/guvcview/default.nix b/pkgs/os-specific/linux/guvcview/default.nix
index 40af8c8553a..5385651030b 100644
--- a/pkgs/os-specific/linux/guvcview/default.nix
+++ b/pkgs/os-specific/linux/guvcview/default.nix
@@ -25,7 +25,7 @@ stdenv.mkDerivation rec {
       portaudio
       udev
       gsl
-    ] ++ stdenv.lib.optional pulseaudioSupport [ libpulseaudio ];
+    ] ++ stdenv.lib.optional pulseaudioSupport libpulseaudio;
 
   meta = {
     description = "A simple interface for devices supported by the linux UVC driver";
diff --git a/pkgs/servers/freeradius/default.nix b/pkgs/servers/freeradius/default.nix
index 50d3f0d53ad..cbafe16623e 100644
--- a/pkgs/servers/freeradius/default.nix
+++ b/pkgs/servers/freeradius/default.nix
@@ -15,6 +15,7 @@
 , withRedis ? false
 , libmysql
 , withMysql ? false
+, json_c
 , withJson ? false
 , libyubikey
 , withYubikey ? false
@@ -42,16 +43,16 @@ stdenv.mkDerivation rec {
   version = "3.0.11";
 
   buildInputs = [ autoreconfHook openssl talloc finger_bsd perl ]
-    ++ optional withLdap [ openldap ]
-    ++ optional withSqlite [ sqlite ]
-    ++ optional withPcap [ libpcap ]
-    ++ optional withCap [ libcap ]
-    ++ optional withMemcached [ libmemcached ]
-    ++ optional withRedis [ hiredis ]
-    ++ optional withMysql [ libmysql ]
-    ++ optional withJson [ pkgs."json-c" ]
-    ++ optional withYubikey [ libyubikey ]
-    ++ optional withCollectd [ collectd ];
+    ++ optional withLdap openldap
+    ++ optional withSqlite sqlite
+    ++ optional withPcap libpcap
+    ++ optional withCap libcap
+    ++ optional withMemcached libmemcached
+    ++ optional withRedis hiredis
+    ++ optional withMysql libmysql
+    ++ optional withJson json_c
+    ++ optional withYubikey libyubikey
+    ++ optional withCollectd collectd;
 
   # NOTE: are the --with-{lib}-lib-dir and --with-{lib}-include-dir necessary with buildInputs ?
 
diff --git a/pkgs/tools/archivers/dar/default.nix b/pkgs/tools/archivers/dar/default.nix
index b64b6e4ca0a..83a73eb388f 100644
--- a/pkgs/tools/archivers/dar/default.nix
+++ b/pkgs/tools/archivers/dar/default.nix
@@ -9,7 +9,7 @@ stdenv.mkDerivation rec {
   };
 
   buildInputs = [ zlib bzip2 openssl lzo libgcrypt gpgme xz ]
-    ++ stdenv.lib.optional stdenv.isLinux [ attr e2fsprogs ];
+    ++ stdenv.lib.optionals stdenv.isLinux [ attr e2fsprogs ];
 
   configureFlags = [ "--disable-dar-static" ];
 
diff --git a/pkgs/tools/misc/alarm-clock-applet/default.nix b/pkgs/tools/misc/alarm-clock-applet/default.nix
index 88992c8e4e3..b089240bcc6 100644
--- a/pkgs/tools/misc/alarm-clock-applet/default.nix
+++ b/pkgs/tools/misc/alarm-clock-applet/default.nix
@@ -29,7 +29,6 @@ stdenv.mkDerivation rec {
     glib
     gtk2
     gst_all_1.gstreamer
-    gst_plugins
     gnome2.GConf
     gnome2.gnome_icon_theme
     libnotify
@@ -37,7 +36,7 @@ stdenv.mkDerivation rec {
     libunique
     intltool
     wrapGAppsHook
-  ];
+  ] ++ gst_plugins;
 
   propagatedUserEnvPkgs = [ gnome2.GConf.out ];