pyrox.dev
lol
Merge pull request #129143 from fgaz/warzone2100/dataroot
warzone2100: better fix for absolute bindir
authored by
Luke Granger-Brown
and committed by
GitHub
e77ce56f
9817238e
+9
-4
+9
-4
pkgs/games/warzone2100/default.nix
+9
-4
pkgs/games/warzone2100/default.nix
···
84
84
cmakeFlags = [
85
85
"-DWZ_DISTRIBUTOR=NixOS"
86
86
# The cmake builder automatically sets CMAKE_INSTALL_BINDIR to an absolute
87
-
# path, but this results in an error.
88
-
# By resetting it, we let the CMakeLists set it to an accepted value
89
-
# based on prefix.
90
-
"-DCMAKE_INSTALL_BINDIR="
87
+
# path, but this results in an error:
88
+
#
89
+
# > An absolute CMAKE_INSTALL_BINDIR path cannot be used if the following
90
+
# > are not also absolute paths: WZ_DATADIR
91
+
#
92
+
# WZ_DATADIR is based on CMAKE_INSTALL_DATAROOTDIR, so we set that.
93
+
#
94
+
# Alternatively, we could have set CMAKE_INSTALL_BINDIR to "bin".
95
+
"-DCMAKE_INSTALL_DATAROOTDIR=${placeholder "out"}/share"
91
96
];
92
97
93
98
postInstall = lib.optionalString withVideos ''