Skip to content
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

ledger-live-desktop: fix missing gschema settings #106711

Closed
wants to merge 1 commit into from

Conversation

ryanswrt
Copy link

Similar issue to #89089

Motivation for this change

This package crashes on systems without gtk dependencies otherwise, with a (ledger-live-desktop:12516): GLib-GIO-ERROR **: 14:14:57.402: No GSettings schemas are installed on the system error otherwise

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • 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 nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@asymmetric
Copy link
Contributor

@TheAncientGoat this actually fixed it for me.

@ryanswrt
Copy link
Author

ryanswrt commented Jan 12, 2021

@TheAncientGoat this actually fixed it for me.

If I'm not mistaken, that's what the wrapGAppsHook essentially does according to the discussion here #89089 (comment) (and my fix worked - at least for me)

Not sure if the one approach is better than the other, but as there hasn't been any activity on the issue #106733 - not sure if I can push this one over the line

@stale
Copy link

stale bot commented Jul 11, 2021

I marked this as stale due to inactivity. → More info

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jul 11, 2021
@asymmetric
Copy link
Contributor

Still relevant

@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jul 11, 2021
@andresilva
Copy link
Member

@TheAncientGoat any reason why we can't take this PR out of draft so we can merge it? AFAIK the approach in this PR of using wrapGAppsHook solves the problem and is the correct solution (https://nixos.org/manual/nixpkgs/stable/#ssec-gnome-hooks). I would approve this PR.

@ryanswrt
Copy link
Author

@andresilva - no reason on my end besides not being able to find reviewers; will take it out of draft if you're happy with it, thanks

@ryanswrt ryanswrt marked this pull request as ready for review December 11, 2021 00:28
Copy link
Member

@andresilva andresilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I only actually looked at the patch now. I'm not sure how this is supposed to be working, you have just defined a variable extraPkgs which isn't used anywhere. I think wrapGAppsHook cannot be used by the app image builder since it needs to be defined as a native build input. So I suggest you update the PR to instead do the same as here as suggested by @asymmetric.

@ryanswrt
Copy link
Author

I'll have a look - not currently using the package because my ledger wasn't being detected anyhow; will give it another go

@andresilva
Copy link
Member

The ledger not being detected was fixed recently. Make sure you update nixpkgs before trying again.

@ryanswrt
Copy link
Author

Hmm, just tested it out while using Gnome - file manager seems to work as expected, don't know if it's a KDE only thing. As I'm using Gnome at the moment I don't have the time to change desktops and triage - if no one else is experiencing the issue I guess we can close?

@andresilva
Copy link
Member

I am using i3 and also could not replicate the issue. I used to have the XDG_DATA_DIRS fix on my zsh profile since this issue also affected other apps, but I don't anymore and didn't have problems opening the file picker on Ledger Live. @asymmetric if you can't replicate it either then we can probably close this.

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

Successfully merging this pull request may close these issues.

None yet

4 participants