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

emacs: Use wrapGAppsHook always when on GTK #19731

Closed
wants to merge 1 commit into from

Conversation

DamienCassou
Copy link
Contributor

@DamienCassou DamienCassou commented Oct 20, 2016

Motivation for this change

The GTK theme is not recognized (e.g., the mouse pointer is different when inside an Emacs window).

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • OS X
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@mention-bot
Copy link

@DamienCassou, thanks for your PR! By analyzing the history of the files in this pull request, we identified @the-kenny, @jwiegley and @dudebout to be potential reviewers.

@DamienCassou
Copy link
Contributor Author

/cc @chaoflow @lovek323 @peti

@DamienCassou
Copy link
Contributor Author

I've been using that for some time now and it seems to work. Can I merge it?

@DamienCassou
Copy link
Contributor Author

DamienCassou commented Oct 29, 2016

I have the impression that this patch triggers a lot of warning messages in the terminal:

Warning: arch-dependent data dir '/nix/store/cr388ab8l3lxxdixxbiwg0cc4nixhw84-emacs-25.1/libexec/emacs/25.1/x86_64-unknown-linux-gnu/': Not a directory

this seem harmless but are still annoying.

@ttuegel
Copy link
Member

ttuegel commented Feb 14, 2017

I have the impression that this patch triggers a lot of warning messages in the terminal:

Could we alter the hook so it doesn't point GTK to non-existent directories?

@ttuegel
Copy link
Member

ttuegel commented Nov 15, 2017

This seems to be done in master now, albeit in a slightly different way.

@ttuegel ttuegel closed this Nov 15, 2017
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