New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
renderdoc: 0.91 -> 1.0 #43159
renderdoc: 0.91 -> 1.0 #43159
Conversation
There's probably a way to make this work with less copying, but I went for the minimum amount of changes on top of @jansol's. |
"-DRENDERDOC_SWIG_PACKAGE=${custom_swig}" | ||
# TODO: add once pyside2 is in nixpkgs | ||
#"-DPYSIDE2_PACKAGE_DIR=${python36Packages.pyside2}" | ||
"-DVULKAN_LAYER_FOLDER=${placeholder "out"}/share/vulkan/implicit_layer.d/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
placeholder
still can't be used, #37875
nativeBuildInputs = [ autoreconfHook pcre ]; | ||
|
||
autoreconfPhase = '' | ||
phases = ["unpackPhase" "patchAndMove"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting phases directly considered harmful, you could use patchPhase
for the shebangs and installPhase
for the tar.
Thanks for the review! Addressed the issues with some simplifications. I'm not particularly happy with the chmod hack; is there a reasonable stdenvy way to copy a bunch of files while making them writable but not otherwise modifying their permissions? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe a normal cp & chmod +w would be better, which would indicate that it actually needs write permissions, instead of "abusing" the --no-preserve of cp. Ultimately it doesn't matter much though ;)
Where is |
Whoops, that wasn't supposed to go up. |
|
||
src = fetchFromGitHub { | ||
owner = "baldurk"; | ||
repo = "renderdoc"; | ||
rev = "2d8b2cf818746b6a2add54e2fef449398816a40c"; | ||
sha256 = "07yc3fk7j2nqmrhc4dm3v2pgbc37scd7d28nlzk6v0hw99zck8k0"; | ||
rev = "v1.0"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v${version}
Tweaked as requested. |
@GrahamcOfBorg build renderdoc |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: renderdoc Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: renderdoc Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: renderdoc Partial log (click to expand)
|
Motivation for this change
Features, bugfixes. Debugged and squashed version of #36761.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)