qt59.qtbase: carefully avoid applying Linux flags to Darwin build

pkgs/development/libraries/qt-5/modules/qtbase.nix

@@ -31,6 +31,7 @@ assert withGtk3 -> gtk3 != null;
 
 let
   system-x86_64 = lib.elem stdenv.system lib.platforms.x86_64;
+  compareVersion = v: builtins.compareVersions version v;
 in
 
 stdenv.mkDerivation {
@@ -47,13 +48,18 @@ stdenv.mkDerivation {
 
       # Image formats
       libjpeg libpng libtiff
+      (if compareVersion "5.9.0" >= 0 then pcre2 else pcre16)
     ]
-    ++ (if builtins.compareVersions version "5.9.0" >= 0
+    ++ (
-           then [ pcre2 ] else [ pcre16 ])
+      if stdenv.isDarwin
-
+      then with darwin.apple_sdk.frameworks;
-    ++ lib.optional (mesaSupported && !stdenv.isDarwin) mesa
+        [
-
+          AGL AppKit ApplicationServices Carbon Cocoa CoreAudio CoreBluetooth
-    ++ lib.optionals (!stdenv.isDarwin) [
+          CoreLocation CoreServices DiskArbitration Foundation OpenGL
+          darwin.libobjc libiconv
+        ]
+      else
+        [
           dbus glib udev
 
           # Text rendering
@@ -63,17 +69,15 @@ stdenv.mkDerivation {
           libX11 libXcomposite libXext libXi libXrender libxcb libxkbcommon xcbutil
           xcbutilimage xcbutilkeysyms xcbutilrenderutil xcbutilwm
         ]
+        ++ lib.optional mesaSupported mesa
+    );
 
-    ++ lib.optionals stdenv.isDarwin (with darwin.apple_sdk.frameworks; [
+  buildInputs =
-      AGL AppKit ApplicationServices Carbon Cocoa
+    lib.optionals (!stdenv.isDarwin)
-      CoreAudio CoreBluetooth CoreLocation CoreServices
+    (
-      DiskArbitration Foundation OpenGL
+      [ libinput ]
-      darwin.libobjc libiconv
+      ++ lib.optional withGtk3 gtk3
-    ]);
+    )
-
-  buildInputs = [ ]
-    ++ lib.optional (!stdenv.isDarwin && withGtk3) gtk3
-    ++ lib.optional (!stdenv.isDarwin) libinput
     ++ lib.optional developerBuild gdb
     ++ lib.optional (cups != null) cups
     ++ lib.optional (mysql != null) mysql.lib
@@ -118,13 +122,10 @@ stdenv.mkDerivation {
       sed -i '/PATHS.*NO_DEFAULT_PATH/ d' mkspecs/features/data/cmake/Qt5BasicConfig.cmake.in
     ''
 
-    + lib.optionalString (mesaSupported && !stdenv.isDarwin) ''
+    + (
-      sed -i mkspecs/common/linux.conf \
+      if stdenv.isDarwin
-          -e "/^QMAKE_INCDIR_OPENGL/ s|$|${mesa.dev or mesa}/include|" \
+      then
-          -e "/^QMAKE_LIBDIR_OPENGL/ s|$|${mesa.out}/lib|"
         ''
-
-    + lib.optionalString stdenv.isDarwin ''
           sed -i \
               -e 's|! /usr/bin/xcode-select --print-path >/dev/null 2>&1;|false;|' \
               -e 's|! /usr/bin/xcrun -find xcodebuild >/dev/null 2>&1;|false;|' \
@@ -139,9 +140,17 @@ stdenv.mkDerivation {
                   --replace "/System/Library/Frameworks/OpenGL.framework/" "${darwin.apple_sdk.frameworks.OpenGL}/Library/Frameworks/OpenGL.framework/"
               substituteInPlace ./mkspecs/common/mac.conf \
                   --replace "/System/Library/Frameworks/AGL.framework/" "${darwin.apple_sdk.frameworks.AGL}/Library/Frameworks/AGL.framework/"
-     '';
+        ''
         # Note on the above: \x27 is a way if including a single-quote
         # character in the sed string arguments.
+      else
+        lib.optionalString mesaSupported
+          ''
+            sed -i mkspecs/common/linux.conf \
+                -e "/^QMAKE_INCDIR_OPENGL/ s|$|${mesa.dev or mesa}/include|" \
+                -e "/^QMAKE_LIBDIR_OPENGL/ s|$|${mesa.out}/lib|"
+          ''
+    );
 
   qtPluginPrefix = "lib/qt-${qtCompatVersion}/plugins";
   qtQmlPrefix = "lib/qt-${qtCompatVersion}/qml";
@@ -150,7 +159,7 @@ stdenv.mkDerivation {
   setOutputFlags = false;
   preConfigure = ''
     export LD_LIBRARY_PATH="$PWD/lib:$PWD/plugins/platforms:$LD_LIBRARY_PATH"
-    ${lib.optionalString (builtins.compareVersions version "5.9.0" < 0) ''
+    ${lib.optionalString (compareVersion "5.9.0" < 0) ''
     # We need to set LD to CXX or otherwise we get nasty compile errors
     export LD=$CXX
     ''}
@@ -189,16 +198,9 @@ stdenv.mkDerivation {
       ''-DNIXPKGS_LIBXCURSOR="${libXcursor.out}/lib/libXcursor"''
     ]
 
-    ++ lib.optional (mesaSupported && !stdenv.isDarwin)
+    ++ (
-       ''-DNIXPKGS_MESA_GL="${mesa.out}/lib/libGL"''
+      if stdenv.isDarwin
-
+      then
-    ++ lib.optionals (!stdenv.isDarwin && withGtk3)
-    [
-      ''-DNIXPKGS_QGTK3_XDG_DATA_DIRS="${gtk3}/share/gsettings-schemas/${gtk3.name}"''
-      ''-DNIXPKGS_QGTK3_GIO_EXTRA_MODULES="${dconf.lib}/lib/gio/modules"''
-    ]
-
-    ++ lib.optionals stdenv.isDarwin
         [
           "-Wno-missing-sysroot"
           "-D__MAC_OS_X_VERSION_MAX_ALLOWED=1090"
@@ -207,6 +209,14 @@ stdenv.mkDerivation {
           # 10.9 which necessitates the above macro definition that mentions
           # 10.10
         ]
+      else
+        lib.optional mesaSupported ''-DNIXPKGS_MESA_GL="${mesa.out}/lib/libGL"''
+        ++ lib.optionals withGtk3
+          [
+            ''-DNIXPKGS_QGTK3_XDG_DATA_DIRS="${gtk3}/share/gsettings-schemas/${gtk3.name}"''
+            ''-DNIXPKGS_QGTK3_GIO_EXTRA_MODULES="${dconf.lib}/lib/gio/modules"''
+          ]
+    )
 
     ++ lib.optional decryptSslTraffic "-DQT_DECRYPT_SSL_TRAFFIC";
 
@@ -232,8 +242,15 @@ stdenv.mkDerivation {
       "-strip"
       "-system-proxies"
       "-pkg-config"
+
+      "-gui"
+      "-widgets"
+      "-opengl desktop"
+      "-qml-debug"
+      "-icu"
+      "-pch"
     ]
-    ++ lib.optionals (builtins.compareVersions version "5.9.0" < 0)
+    ++ lib.optionals (compareVersion "5.9.0" < 0)
     [
       "-c++11"
       "-no-reduce-relocations"
@@ -242,18 +259,11 @@ stdenv.mkDerivation {
       "-developer-build"
       "-no-warnings-are-errors"
     ]
-    ++ [
+    ++ (
-      "-gui"
+      if (!system-x86_64)
-      "-widgets"
+      then [ "-no-sse2" ]
-      "-opengl desktop"
+      else lib.optional (compareVersion "5.9.0" >= 0) [ "-sse2" ]
-      "-qml-debug"
+    )
-      "-icu"
-      "-pch"
-    ]
-
-    ++ (if builtins.compareVersions version "5.9.0" >= 0
-           then [ (if system-x86_64 then "-sse2" else "-no-sse2") ]
-           else lib.optional (!system-x86_64) "-no-sse2")
     ++ [
       "-no-sse3"
       "-no-ssse3"
@@ -277,13 +287,24 @@ stdenv.mkDerivation {
 
       "-make libs"
       "-make tools"
-      ''-${lib.optionalString (buildExamples == false) "no"}make examples''
+      ''-${lib.optionalString (!buildExamples) "no"}make examples''
-      ''-${lib.optionalString (buildTests == false) "no"}make tests''
+      ''-${lib.optionalString (!buildTests) "no"}make tests''
       "-v"
     ]
 
-    ++ lib.optionals (!stdenv.isDarwin) [
+    ++ (
-      "-${lib.optionalString (builtins.compareVersions version "5.9.0" < 0) "no-"}rpath"
+      if stdenv.isDarwin
+      then
+        [
+          "-platform macx-clang"
+          "-no-use-gold-linker"
+          "-no-fontconfig"
+          "-qt-freetype"
+          "-qt-libpng"
+        ]
+      else
+        [
+          "-${lib.optionalString (compareVersion "5.9.0" < 0) "no-"}rpath"
 
           "-system-xcb"
           "-xcb"
@@ -308,15 +329,8 @@ stdenv.mkDerivation {
           "-no-use-gold-linker"
         ]
         ++ lib.optional withGtk3 "-gtk"
-    ++ lib.optional (builtins.compareVersions version "5.9.0" >= 0) "-inotify"
+        ++ lib.optional (compareVersion "5.9.0" >= 0) "-inotify"
-
+    );
-    ++ lib.optionals stdenv.isDarwin [
-      "-platform macx-clang"
-      "-no-use-gold-linker"
-      "-no-fontconfig"
-      "-qt-freetype"
-      "-qt-libpng"
-    ];
 
   enableParallelBuilding = true;
 
@@ -360,13 +374,10 @@ stdenv.mkDerivation {
       moveToOutput bin "$dev"
     ''
 
-    # fixup .pc file (where to find 'moc' etc.)
+    + (
-    + lib.optionalString (!stdenv.isDarwin) ''
+      if stdenv.isDarwin
-      sed -i "$dev/lib/pkgconfig/Qt5Core.pc" \
+      then
-          -e "/^host_bins=/ c host_bins=$dev/bin"
         ''
-
-    + lib.optionalString stdenv.isDarwin ''
           fixDarwinDylibNames_rpath() {
             local flags=()
 
@@ -380,7 +391,14 @@ stdenv.mkDerivation {
             done
           }
           fixDarwinDylibNames_rpath "QtConcurrent" "QtPrintSupport" "QtCore" "QtSql" "QtDBus" "QtTest" "QtGui" "QtWidgets" "QtNetwork" "QtXml" "QtOpenGL"
-    '';
+        ''
+      else
+        # fixup .pc file (where to find 'moc' etc.)
+        ''
+          sed -i "$dev/lib/pkgconfig/Qt5Core.pc" \
+              -e "/^host_bins=/ c host_bins=$dev/bin"
+        ''
+    );
 
   setupHook = ../hooks/qtbase-setup-hook.sh;