mirror of
https://github.com/NixOS/nixpkgs.git
synced 2024-12-29 00:53:57 +00:00
e0464e4788
In preparation for the deprecation of `stdenv.isX`. These shorthands are not conducive to cross-compilation because they hide the platforms. Darwin might get cross-compilation for which the continued usage of `stdenv.isDarwin` will get in the way One example of why this is bad and especially affects compiler packages https://www.github.com/NixOS/nixpkgs/pull/343059 There are too many files to go through manually but a treewide should get users thinking when they see a `hostPlatform.isX` in a place where it doesn't make sense. ``` fd --type f "\.nix" | xargs sd --fixed-strings "stdenv.is" "stdenv.hostPlatform.is" fd --type f "\.nix" | xargs sd --fixed-strings "stdenv'.is" "stdenv'.hostPlatform.is" fd --type f "\.nix" | xargs sd --fixed-strings "clangStdenv.is" "clangStdenv.hostPlatform.is" fd --type f "\.nix" | xargs sd --fixed-strings "gccStdenv.is" "gccStdenv.hostPlatform.is" fd --type f "\.nix" | xargs sd --fixed-strings "stdenvNoCC.is" "stdenvNoCC.hostPlatform.is" fd --type f "\.nix" | xargs sd --fixed-strings "inherit (stdenv) is" "inherit (stdenv.hostPlatform) is" fd --type f "\.nix" | xargs sd --fixed-strings "buildStdenv.is" "buildStdenv.hostPlatform.is" fd --type f "\.nix" | xargs sd --fixed-strings "effectiveStdenv.is" "effectiveStdenv.hostPlatform.is" fd --type f "\.nix" | xargs sd --fixed-strings "originalStdenv.is" "originalStdenv.hostPlatform.is" ```
71 lines
1.9 KiB
Nix
71 lines
1.9 KiB
Nix
{ AppKit
|
|
, cmake
|
|
, fetchFromGitHub
|
|
, fetchpatch2
|
|
, Foundation
|
|
, jsoncpp
|
|
, lib
|
|
, libGL
|
|
, stdenv
|
|
, nix-update-script
|
|
}:
|
|
|
|
stdenv.mkDerivation (finalAttrs: {
|
|
pname = "openvr";
|
|
version = "2.5.1";
|
|
|
|
src = fetchFromGitHub {
|
|
owner = "ValveSoftware";
|
|
repo = "openvr";
|
|
rev = "v${finalAttrs.version}";
|
|
hash = "sha256-bIKjZ7DvJVmDK386WgXaAFQrS0E1TNEUMhfQp7FNnvk=";
|
|
};
|
|
|
|
patches = [
|
|
# https://github.com/ValveSoftware/openvr/pull/594
|
|
(fetchpatch2 {
|
|
name = "use-correct-CPP11-definition-for-vsprintf_s.patch";
|
|
url = "https://github.com/ValveSoftware/openvr/commit/0fa21ba17748efcca1816536e27bdca70141b074.patch";
|
|
hash = "sha256-0sPNDx5qKqCzN35FfArbgJ0cTztQp+SMLsXICxneLx4=";
|
|
})
|
|
# https://github.com/ValveSoftware/openvr/pull/1716
|
|
(fetchpatch2 {
|
|
name = "add-ability-to-build-with-system-installed-jsoncpp.patch";
|
|
url = "https://github.com/ValveSoftware/openvr/commit/54a58e479f4d63e62e9118637cd92a2013a4fb95.patch";
|
|
hash = "sha256-aMojjbNjLvsGev0JaBx5sWuMv01sy2tG/S++I1NUi7U=";
|
|
})
|
|
];
|
|
|
|
postUnpack = ''
|
|
# Move in-tree jsoncpp out to complement the patch above
|
|
# fetchpatch2 is not able to handle these renames
|
|
mkdir source/thirdparty
|
|
mv source/src/json source/thirdparty/jsoncpp
|
|
'';
|
|
|
|
nativeBuildInputs = [
|
|
cmake
|
|
];
|
|
|
|
buildInputs = [
|
|
jsoncpp
|
|
libGL
|
|
] ++ lib.optionals stdenv.hostPlatform.isDarwin [
|
|
AppKit
|
|
Foundation
|
|
];
|
|
|
|
cmakeFlags = [ "-DUSE_SYSTEM_JSONCPP=ON" "-DBUILD_SHARED=1" ];
|
|
|
|
passthru.updateScript = nix-update-script { };
|
|
|
|
meta = {
|
|
broken = stdenv.hostPlatform.isDarwin;
|
|
description = "API and runtime that allows access to VR hardware from multiple vendors without requiring that applications have specific knowledge of the hardware they are targeting";
|
|
homepage = "https://github.com/ValveSoftware/openvr";
|
|
license = lib.licenses.bsd3;
|
|
maintainers = with lib.maintainers; [ Scrumplex ];
|
|
platforms = lib.platforms.unix;
|
|
};
|
|
})
|