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
houdini: 17.5.327 -> 18.0.391 #81113
Conversation
Thanks for doing this, I was trying to install v18 but kept running into issues with qt5. I just tested this commit with We should maybe replace the message with something that tells the user to use |
@emptyflask that hash is good for the updated version, so I've bumped it again. This PR removes the custom message for
Does that seem suitable? It suggests your option and also an alternative using |
Looks good to me |
There's been no response from the maintainer @canndrew so if either of you want to add yourself as maintainer, then I'd be okay merging it. |
I don't mind being the maintainer -- but if someone else who actually uses Houdini on a daily basis would rather do it, that would make more sense. BTW, I've attempted to upgrade this to v18.5 after modifying the license date, but it looks like SideFX changed the license server in their latest release somehow, so I haven't been able to start it up yet. I'm not sure if I'll have the time to dedicate to that amount of work needed to get that running, but I'll give it a shot. |
I've found this on the Houdini website: ftp://ftp.sidefx.com/public Maybe it can be used to automatically download the software instead of requiring |
The previous version is no longer available for download. Also, remove the custom message - while cute, a similar message caused some confusion on the quartus package between nix-prefetch-url and nix-store --add-fixed.
@ryantm sorry, I missed that you wanted one of us as maintainer before merging. I've added myself now, and rebased onto latest master. I also added the ability to override the unwrapped package, as I did in #124906. This will make it easier to do a minor version bump in a personal overlay. I'm currently working on v18.5, and will add that patch to this PR if it has not been merged by the time I figure it out. Otherwise, I will create a new PR. @arximboldi - that looks great. I can see the mention of it here, so it seems like SideFX are OK with using FTP to get packages. |
@arximboldi - I'm actually finding using the FTP server with |
Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
I have a working derivation for 18.5 now, but I'd like to merge this PR separately as it's a significant rework, and there is a slight regression - |
Motivation for this change
Upstream package is no longer available for download, also new version is released.
Due to the first part (
17.5.327
no longer available for download) at least the first commit should be backported to stable I think?I'm happy to add myself as a maintainer if @canndrew has no objections?
@emptyflask and @PlumpMath were working on the v18.0 update, so they might also be interested in this PR.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)