Skip to content

Commit

Permalink
qt5: fix to work on LLVM 4
Browse files Browse the repository at this point in the history
Including apple_sdk.sdk is generally a recipe for a bad time on LLVM 3.8
and above, since you end up with bad headers in the wrong place that hurt
the new libc++ in 3.8 and above. In this case, qt only wanted the super-
generic SDK for CUPS headers, which we can just depend on directly now.
  • Loading branch information
copumpkin committed Apr 7, 2017
1 parent aa31d4b commit 6b1957d
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions pkgs/development/libraries/qt-5/5.6/qtbase/default.nix
Expand Up @@ -80,7 +80,7 @@ stdenv.mkDerivation {
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;|' \
-e 's|sysroot=$(/usr/bin/xcodebuild -sdk $sdk -version Path 2>/dev/null)|sysroot="${darwin.apple_sdk.sdk}"|' \
-e 's|sysroot=$(/usr/bin/xcodebuild -sdk $sdk -version Path 2>/dev/null)|sysroot=/nonsense|' \
-e 's|QMAKE_CONF_COMPILER=`getXQMakeConf QMAKE_CXX`|QMAKE_CXX="clang++"\nQMAKE_CONF_COMPILER="clang++"|' \
-e 's|XCRUN=`/usr/bin/xcrun -sdk macosx clang -v 2>&1`|XCRUN="clang -v 2>&1"|' \
-e 's#sdk_val=$(/usr/bin/xcrun -sdk $sdk -find $(echo $val | cut -d \x27 \x27 -f 1))##' \
Expand Down Expand Up @@ -208,7 +208,7 @@ stdenv.mkDerivation {
xcbutil xcbutilimage xcbutilkeysyms xcbutilwm libxkbcommon
] ++ lib.optionals stdenv.isDarwin (with darwin.apple_sdk.frameworks; [
ApplicationServices CoreServices AppKit Carbon OpenGL AGL Cocoa
DiskArbitration darwin.cf-private libiconv darwin.apple_sdk.sdk
DiskArbitration darwin.cf-private libiconv
]);

buildInputs =
Expand Down
4 changes: 2 additions & 2 deletions pkgs/development/libraries/qt-5/5.8/qtbase/default.nix
Expand Up @@ -82,7 +82,7 @@ stdenv.mkDerivation {
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;|' \
-e 's|sysroot=$(/usr/bin/xcodebuild -sdk $sdk -version Path 2>/dev/null)|sysroot="${darwin.apple_sdk.sdk}"|' \
-e 's|sysroot=$(/usr/bin/xcodebuild -sdk $sdk -version Path 2>/dev/null)|sysroot=/nonsense|' \

This comment has been minimized.

Copy link
@periklis

periklis Apr 7, 2017

Contributor

This produces clang-3.7: warning: no such sysroot directory: '/nonsense'

This comment has been minimized.

Copy link
@copumpkin

copumpkin Apr 11, 2017

Author Member

It was intentional for it not to exist, but I guess there's probably a more elegant solution 😄

-e 's|QMAKE_CONF_COMPILER=`getXQMakeConf QMAKE_CXX`|QMAKE_CXX="clang++"\nQMAKE_CONF_COMPILER="clang++"|' \
-e 's|XCRUN=`/usr/bin/xcrun -sdk macosx clang -v 2>&1`|XCRUN="clang -v 2>&1"|' \
-e 's#sdk_val=$(/usr/bin/xcrun -sdk $sdk -find $(echo $val | cut -d \x27 \x27 -f 1))##' \
Expand Down Expand Up @@ -212,7 +212,7 @@ stdenv.mkDerivation {
xcbutilimage xcbutilkeysyms xcbutilrenderutil xcbutilwm
] ++ lib.optionals stdenv.isDarwin (with darwin.apple_sdk.frameworks; [
ApplicationServices Foundation CoreServices AppKit Carbon OpenGL AGL Cocoa
DiskArbitration darwin.cf-private libiconv darwin.apple_sdk.sdk
DiskArbitration darwin.cf-private libiconv
]);

buildInputs = [ ]
Expand Down
4 changes: 2 additions & 2 deletions pkgs/top-level/all-packages.nix
Expand Up @@ -9446,7 +9446,7 @@ with pkgs;
inherit newScope;
inherit stdenv fetchurl makeSetupHook makeWrapper;
bison = bison2; # error: too few arguments to function 'int yylex(...
cups = if stdenv.isLinux then cups else null;
inherit cups;
harfbuzz = harfbuzz-icu;
mesa = mesa_noglu;
inherit perl;
Expand All @@ -9459,7 +9459,7 @@ with pkgs;
inherit newScope;
inherit stdenv fetchurl makeSetupHook makeWrapper;
bison = bison2; # error: too few arguments to function 'int yylex(...
cups = if stdenv.isLinux then cups else null;
inherit cups;
harfbuzz = harfbuzz-icu;
mesa = mesa_noglu;
inherit perl;
Expand Down

0 comments on commit 6b1957d

Please sign in to comment.