|
1 |
| -{ stdenv, lib, makeQtWrapper, fetchFromGitHub, fetchpatch |
| 1 | +{ stdenv, makeQtWrapper, fetchFromGitHub, fetchpatch |
2 | 2 | , cmake, extra-cmake-modules, pkgconfig, libxcb, libpthreadstubs, lndir
|
3 | 3 | , libXdmcp, libXau, qtbase, qtdeclarative, qttools, pam, systemd
|
4 | 4 | , themes
|
|
28 | 28 | nativeBuildInputs = [ cmake extra-cmake-modules pkgconfig qttools ];
|
29 | 29 |
|
30 | 30 | buildInputs = [
|
31 |
| - libxcb libpthreadstubs libXdmcp libXau pam systemd |
32 |
| - qtbase qtdeclarative |
| 31 | + libxcb libpthreadstubs libXdmcp libXau qtbase pam systemd |
33 | 32 | ];
|
34 | 33 |
|
35 |
| - propagatedUserEnvPkgs = builtins.map lib.getBin [ |
36 |
| - qtbase qtdeclarative |
37 |
| - ]; |
| 34 | + propagatedBuildInputs = [ qtdeclarative ]; |
38 | 35 |
|
39 | 36 | cmakeFlags = [
|
40 | 37 | "-DCONFIG_FILE=/etc/sddm.conf"
|
@@ -83,21 +80,6 @@ stdenv.mkDerivation {
|
83 | 80 | installPhase = ''
|
84 | 81 | runHook preInstall
|
85 | 82 |
|
86 |
| - propagated= |
87 |
| - for i in $unwrapped $themes; do |
88 |
| - findInputs $i propagated propagated-user-env-packages |
89 |
| - if [ -z "$crossConfig" ]; then |
90 |
| - findInputs $i propagated propagated-native-build-inputs |
91 |
| - else |
92 |
| - findInputs $i propagated propagated-build-inputs |
93 |
| - fi |
94 |
| - done |
95 |
| -
|
96 |
| - for pkg in $propagated; do |
97 |
| - addToSearchPath RUNTIME_XDG_DATA_DIRS "$pkg/share" |
98 |
| - addToSearchPath RUNTIME_XDG_CONFIG_DIRS "$pkg/etc/xdg" |
99 |
| - done |
100 |
| -
|
101 | 83 | makeQtWrapper "$unwrapped/bin/sddm" "$out/bin/sddm"
|
102 | 84 |
|
103 | 85 | mkdir -p "$out/share/sddm"
|
|
3 commit comments
abbradar commentedon Mar 24, 2017
This seems to generally follow my failure from #24133 (comment) -- if Qt parts are installed into system closure it breaks things.
ttuegel commentedon Mar 24, 2017
That didn't install anything into the system environment. In fact, reverting it partially unwrapped SDDM, which means now it requires dependencies from the system environment. I believe this also breaks the Plasma theme, but I can't test that automatically.
abbradar commentedon Mar 24, 2017
Oh, I was confused by
propagatedUserEnvPkgs
but I see now that it's not its conventional usage.