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: Fixes build with missing SWIG_PACKAGE #32230
Conversation
nativeBuildInputs = [ cmake makeWrapper pkgconfig python3 bison autoconf automake pcre ]; | ||
|
||
SWIG_PACKAGE = fetchurl { | ||
url = "https://github.com/baldurk/swig/archive/renderdoc-modified-1.zip"; | ||
sha256 = "0xxvkkaa5rp2xnnms6i1i4kbvq9fj3is0kiawwdix4hfys16m4mz"; |
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.
Is this url stable?
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.
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.
These tarballs are generated by github. On regeneration they might change also the content does not change. Therefor it is required fetchzip
or even better fetchFromGitHub
, which is a wrapper around fetchzip
.
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.
Okay, I already tried fetchFromGitHub
, but gave up after some tries. I will try again, I think I have an idea was is wrong.
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.
@bkchr What errors did you encounter?
dacc8ef
to
29d0f99
Compare
Reminder for @Mic92 |
Motivation for this change
Required by nox-review for #32100
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)