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
zoom-us: 2.2.123200.0405 -> 2.2.128100.0627 #42742
Conversation
cc maintainers: @danbst |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: zoom-us Partial log (click to expand)
|
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: zoom-us Partial log (click to expand)
|
This is much cleaner - nice! |
When I build this locally and try to start it, I get this error:
Might be some kind of version clash between the bundled libraries and the nixpkgs ones, or an impurity in the build. |
@xeji It would have to be the latter, as this change removes all bundled libraries from the package. |
Turns out it doesn't launch from |
@tadfisher Does your webcam work in zoom after this change? Mine doesn't, but reverting 81c2f27 fixes it. With this PR I only see a black screen, even though my webcam is on (there's an LED indicator on my laptop) |
@tadfisher Seems unlikely since that error existed for some people before this PR. I guess I'll open a new issue to discuss what was broken in this one specifically. Thanks anyway. :) |
@tadfisher sorry, was on vacation, so missed this. Thanks for your help and investigations! Though there are several drawbacks for this approach:
Previously:
|
While that is technically true, Qt and its dependencies are very likely to already exist on a machine on which somebody wants to use zoom-us so the effective change in closure size is actually a net decrease in closure size. As ttuegel pointed out in the referenced commit, we definitely want to use our version of Qt rather than a vendored version. |
Motivation for this change
Rewrite
zoom-us
usingautoPatchelfHook
and use the system qt5 libraries.Add an update script so this can be kept up-to-date more easily.
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)