Skip to content

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

Merged
merged 2 commits into from
Jul 22, 2017

Conversation

antonxy
Copy link
Contributor

@antonxy antonxy commented Jul 19, 2017

…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
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Sorry, something went wrong.

…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
@joachifm
Copy link
Contributor

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
Copy link
Member

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.

Copy link
Contributor

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?)

@canndrew
Copy link
Contributor

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?

@joachifm
Copy link
Contributor

Are they serving 671 at the 633 url?

@canndrew
Copy link
Contributor

canndrew commented Jul 19, 2017

There isn't a URL in the package (it uses requireFile). You have to go to their website, accept their EULA and then it starts downloading. I guess @antonxy could have found a link to download an old version through their website, but I'm guessing they just didn't change the version number.

@antonxy
Copy link
Contributor Author

antonxy commented Jul 19, 2017

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.

@canndrew
Copy link
Contributor

canndrew commented Jul 20, 2017

I did download the 633 version from the production builds section

Oh, sorry. Weird that the hash changed then :/

@Mic92 Mic92 merged commit 9f4750d into NixOS:master Jul 22, 2017
@Mic92
Copy link
Member

Mic92 commented Jul 22, 2017

Merged without testing since the download requires registration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants