evmar / n2

n2 ("into"), a ninja compatible build system

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Difficulties with git-lfs

Sarcasm opened this issue · comments

Hello,

Seeing the recent improvements, I wanted to update my clone of n2, to run cargo install --path . on it.

I have issues updating my clone due to the git-lfs dependencies for the tests:

$ git pull --rebase
remote: Enumerating objects: 133, done.
remote: Counting objects: 100% (102/102), done.
remote: Compressing objects: 100% (46/46), done.
remote: Total 133 (delta 65), reused 82 (delta 52), pack-reused 31
Receiving objects: 100% (133/133), 54.01 KiB | 6.00 MiB/s, done.
Resolving deltas: 100% (74/74), completed with 19 local objects.
From https://github.com/evmar/n2
   4bae87a..cc45ae4  main       -> origin/main
 * [new branch]      bufwrite   -> origin/bufwrite
Updating 4bae87a..cc45ae4
git-lfs filter-process: git-lfs: command not found
fatal: the remote end hung up unexpectedly

It's not easy for me to add git-lfs to my git install (which is custom built) and I don't plan on developing n2 for now (so I doubt I would make use of the snapshot.zip).

I will look for a way to get the update without git complaining but it's not immediately obvious to me.
I wanted to empty the tests/snapshot/.gitattributes but I cannot do it before the rebase, and doing the rebase aborts in the middle because of the file.

I created this issue to let you know of this potential issue, but I understand it may be working as intended.
If you think it's worth supporting and have ideas on how to make the git-lfs dependency optional, I'm interested.

FYI, after reading gitattributes documentation I came up with this:

$ cat .git/info/attributes
tests/snapshot/snapshot.zip -filter -diff -merge

Not sure it's optimal but it allowed me to git pull and cargo install.

Oh, that is super annoying argh. I don't really need to use git-lfs here so I will probably try to undo it.

I just wanted a place to stash a zip file that some developers occasionally might use, and GitHub doesn't seem to provide one...

(Let me know if this still is busted, I don't have a lot of experience with git lfs)

Looks good.
I could remove the .git/info/attributes after your change and now git pull --rebase does not complain.

Thank you!