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

nixpkgs plugin.dependencies breaking lazy-loaded plugins #535

Open
1 task done
horriblename opened this issue Jan 7, 2025 · 0 comments
Open
1 task done

nixpkgs plugin.dependencies breaking lazy-loaded plugins #535

horriblename opened this issue Jan 7, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@horriblename
Copy link
Collaborator

⚠️ Please verify that this bug has NOT been reported before.

  • I checked all existing issues and didn't find a similar issue

Description

currently startPlugins and optPlugins .dependencies are installed automatically (by mnw).

This causes duplicate plugins installed, and may break a lazy-loaded plugin.

Example: Telescope

with our telescope option enabled, and adding a telescope extension like telescope-undo from nixpkgs, we end up with nixpkgs' telescope in start/ and our telescope in opt/

The one in start/ will load telescope on startup

When a keybind is triggered, lz.n will remove the existing :Telescope command and run telescope.setup

at this point the setup doesn't re-add :Telescope as telescope just assumes it's already loaded

so we end up with a missing :Telescope command

👟 Reproduction steps

add a telescope extension from nixpkgs

👀 Expected behavior

doesn't mess with lazy-loaded plugin

😓 Actual Behavior

does

💻 Metadata

  • system: "x86_64-linux" - host os: Linux 6.6.64, NixOS, 25.05 (Warbler), 25.05.20241211.5d67ea6 - multi-user?: yes - sandbox: yes - version: `nix-env (Lix, like Nix) 2.91.0

📝 Relevant log output

-
@horriblename horriblename added the bug Something isn't working label Jan 7, 2025
horriblename added a commit to horriblename/nvf that referenced this issue Jan 7, 2025
force a reload when telescope is loaded to workaround NotAShelf#535
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant