fix: download mechanism works with GIT_DIR and GIT_WORK_TREE set

[Bahex]

Explicitly provide --git-dir and --work-tree in git commands.

Issue #274

[Saghen]

Thanks!

[debugloop]

This seems to break on my machine somehow, I have not had the time to look into it yet. I use the flake to install, and I get the following problem on main. Everything works nicely with 1628800e1747ecc767368cab45916177c723da82 (that is the commit before this PR's merge). It seems repo_dir is nil?

Failed to run `config` for blink.cmp

vim/fs.lua:0: invalid value (nil) at index 1 in table for 'concat'

# stacktrace:
  - vim/fs.lua:0 _in_ **joinpath**
  - .local/share/nvim/nixpkgs/blink-cmp/lua/blink/cmp/fuzzy/download.lua:75 _in_ **get_git_tag**
  - .local/share/nvim/nixpkgs/blink-cmp/lua/blink/cmp/fuzzy/download.lua:23 _in_ **ensure_downloaded**
  - .local/share/nvim/nixpkgs/blink-cmp/lua/blink/cmp/init.lua:8 _in_ **setup**
  - .config/nvim/init.lua:8

[Bahex]

I think I understand what's happening. The flake doesn't keep a ".git" directory around

local repo_dir = vim.fs.root(root_dir, '.git')

so repo_dir is nil.

That line only concerns users who download the compiled binary from github, it should be easy to fix by moving that line into download.get_git_tag().

Thanks for the feedback!

[Bahex]

After a familiarizing myself more with the code, what should happen is repo_dir == nil should trigger the same code path as out.code == 128. Sending another PR shortly.

[Bahex]

@debugloop can you check #281 to see if it works?

[debugloop]

Works for me, thanks for the quick response and fix 🚀 .git being stripped from a flake at eval time makes sense, yeah 👍🏻