-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
houdini: change sha256, add missing libs, fix license dir patch, fix … #27480
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
Conversation
…keyboard input - Changed sha256 of src file, as this was apparantly changed on the website - Added missing libs: some X libs, alsa, dbus - Changed patch to $out/houdini/Licensing.opt localValidatorDir = ... was commented out in original file so sed had no effect - add export QT_XKB_CONFIG_ROOT Without it only modifier keys worked, no text input
cc @canndrew. I think the checksum change warrants some investigation. |
@@ -50,11 +58,13 @@ stdenv.mkDerivation rec { | |||
--no-root-check \ | |||
--accept-EULA \ | |||
$out | |||
sed -i "s|/usr/lib/sesi|${license_dir}|g" $out/houdini/Licensing.opt | |||
echo -e "localValidatorDir = ${license_dir}\nlicensingMode = localValidator" > $out/houdini/Licensing.opt |
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 changing licensing scheme the reason they broke checksums? I am not sure how insightful a request towards support@sidefx.com would be to ask, if they plan to continue this practice in future.
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.
I don't think it's related. Making the license checker happy was the main chore in writing this package in the first place. I stopped fiddling with things as soon as it started working on my system, @antonxy is just removing a redundant hack that I left in and changing a config option that should have been changed. (I wonder why it worked without that change on my box though?)
I just went to download the installer from their website and it looks like the minor version number has changed (633 => 671). Which would explain the checksum failure. @antonxy could you change that in the PR aswell? |
Are they serving 671 at the 633 url? |
There isn't a URL in the package (it uses |
I did download the 633 version from the production builds section. I could of course get the current version, check if it works and update it. |
Oh, sorry. Weird that the hash changed then :/ |
Merged without testing since the download requires registration. |
…keyboard input
Changed sha256 of src file, as this was apparantly changed on the
website
Added missing libs: some X libs, alsa, dbus
Changed patch to $out/houdini/Licensing.opt
localValidatorDir = ... was commented out in original file
so sed had no effect
add export QT_XKB_CONFIG_ROOT
Without it only modifier keys worked, no text input
Motivation for this change
I wanted to try houdini today but it did not work out of the box. With these changes I got it working.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)