We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
There was an error while loading. Please reload this page.
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
Currently in PR #1184, the neovide wrapper for GL support is hardcoded to use nixGLIntel:
nixGLIntel
neovide = if cfg.needsGL then pkgs.writeShellScriptBin nvim.packages.x86_64-linux.neovide.meta.mainProgram '' ${lib.getExe nix-gl.packages.x86_64-linux.nixGLIntel} ${lib.getExe nvim.packages.x86_64-linux.neovide} '' else nvim.packages.x86_64-linux.neovide;
This implementation only works for Intel graphics cards and might cause issues on systems with NVIDIA or AMD graphics.
Make the GL implementation configurable, either by:
config.hardware
profiles.base.glPackage
Example implementation:
neovide = if cfg.needsGL then let glPackage = if config.hardware ? nvidia && config.hardware.nvidia.enable then nix-gl.packages.x86_64-linux.nixGLNvidia else nix-gl.packages.x86_64-linux.nixGLIntel; in pkgs.writeShellScriptBin nvim.packages.x86_64-linux.neovide.meta.mainProgram '' ${lib.getExe glPackage} ${lib.getExe nvim.packages.x86_64-linux.neovide} '' else nvim.packages.x86_64-linux.neovide;
This issue was requested by @NobbZ in PR #1184 comment.
The text was updated successfully, but these errors were encountered:
NobbZ
No branches or pull requests
Background
Currently in PR #1184, the neovide wrapper for GL support is hardcoded to use
nixGLIntel
:Issue
This implementation only works for Intel graphics cards and might cause issues on systems with NVIDIA or AMD graphics.
Suggested Solution
Make the GL implementation configurable, either by:
config.hardware
attributesprofiles.base.glPackage
to allow explicit configurationExample implementation:
This issue was requested by @NobbZ in PR #1184 comment.
The text was updated successfully, but these errors were encountered: