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: fix runtime qt environment #43426
Conversation
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)
|
Yay! Works now for me. Though I get annoying extra output in console:
|
@@ -1,16 +1,18 @@ | |||
{ stdenv, fetchurl, system, makeWrapper, makeDesktopItem, autoPatchelfHook | |||
{ stdenv, fetchurl, system, makeWrapper, makeDesktopItem, autoPatchelfHook, env |
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.
Where does this env
comes from? Name is so generic that it is impossible to grep out what it is.
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.
The PR mentions libsForQt5.env
, I guess that's what the param refers to.
Motivation for this change
Fixes #43350
Add a few missing runtime dependencies (discovered with
strace
andQT_DEBUG_PLUGINS=1
).Rather than manually setting runtime paths in environment variables, re-use
libsForQt5.env
to build a usable runtime Qt environment and symlink itsqt.conf
. This seems to fix the issue in #43350 by setting the correct QML import paths.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)