-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
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
firefox: fix gnome printing dialog crash #25518
Conversation
wrapGappsHook was insufficient to fix a firefox (and firefox-bin) crash when opening a print dialog without gsettings_desktop_schemas See NixOS#24880 for more discussion.
@bdimcheff, thanks for your PR! By analyzing the history of the files in this pull request, we identified @taku0, @garbas and @abbradar to be potential reviewers. |
hmmm i guess a file got moved in master since I branched off from unstable, one sec |
ok I need to wait for the hydra to catch up because I don't want to recompile gcc and everything, but I'll keep an eye out when most of the upstream binaries are compiled |
It's not good I didn't catch this (we have some sort of nondeterminism, maybe I have desktop schemas installed to environment by something -- I'll investigate later). Can you show the error visible when Firefox crashes? Also, what desktop environment do you use? |
I just tried
I wonder if firefox is detecting some gnome3 stuff and so it's trying to use gnome3 everything, but with Anyhow, my guess is if everything works with fwiw chrome[ium] still crashes for me on the native print dialog without the wrapper. I haven't tried |
Unsetting It's really strange that |
@vcunat yeah I actually don't know if my change is necessary on master because I was building off of the most recent nixpkgs unstable that passed in hydra, since I didn't want to rebuild everything locally. We can definitely hold off on this PR until your changes are live and I can test it in various ways. |
Is this still a problem? |
This also works for me (just printed this page) |
yeah this works for me too... for some reason under kde i still get a crash, but with |
Motivation for this change
wrapGappsHook was insufficient to fix a firefox (and firefox-bin) crash
when opening a print dialog without passing gsettings_desktop_schemas
See #24880 for more discussion.
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/
)