-
Notifications
You must be signed in to change notification settings - Fork 62
Git LFS confuses Atom's git change detection #68
Comments
Thanks for the follow up, @ruffsl. I'm not sure who is the best person on the team to look into this, so I'm adding it to the internal queue. |
How is the follow up on this? I just installed atom and I have this problem on my lfs repository. |
Any progress on this? I have same issue with my Git-LFS repos |
I've just hit this bug with a Git LFS repo as well :-( |
@rsese would you mind seeing if you can reproduce this? See libgit2/libgit2#3373 for vague repro steps and let me know if you have questions. |
hmm, scratch my comment... I've no idea what has changed in my environment but I'm no longer seeing this bug a few minutes later. I wonder whether I hadn't pulled down the |
Thanks for the update @grahamwhiteuk, glad things are working for you now (as a side note, I see you're on 1.30.0 - just wanted to mention that the latest stable release version of Atom is 1.37.0 so you may want to update). Can anyone else who was seeing this issue reproduce? If so, can you share specific steps to reproduce? I haven't played with LFS in a while so repro steps would be helpful for us when testing things out. |
So this morning I've git pulled and brought down some changes including some files from Git LFS and the problem has manifest itself again. So from my observations at the moment there doesn't seem to be a problem if I'm the one committing the change but if I pull changes from other people then I see the issue. Sorry, that probably doesn't help a huge amount but hoping to narrow it down a bit. I'm on 1.35.0, typo above. |
Finally had a chance to setup LFS again and test this out and I believe I can reproduce. I installed and configured LFS on 2 computers (1 Windows and 1 Linux, both on 1.37.0), then setup and pushed a test repository as described in this LFS tutorial on computer 1. Then I:
At this point, |
Follow-up: I've had same issue just now, but having a commit (removed an image file) solved the issue. I downloaded the git via Github desktop, initialized git lfs but had my yolo.h5 file marked orange. All github/lfs/atom using latest stable version. |
edit by @rsese to mention repro steps from #68 (comment)
Finally had a chance to setup LFS again and test this out and I believe I can reproduce.
I installed and configured LFS on 2 computers (1 Windows and 1 Linux, both on 1.37.0), then setup and pushed a test repository as described in this LFS tutorial on computer 1.
Then I:
head -c 1M /dev/urandom > cat.bin
)git pull
from computer 1At this point,
cat.bin
in the tree view is orange andgit status
reports that everything is up-to-date. I reloaded Atom as well andcat.bin
remained in orange until I changed it locally on computer 1 and added/committed that change.Just following up on this issue atom/atom#8291 that is thought to be rooted in this repo. From @envygeeks 's findings , looks like a lfs filter should be used. I'm unfamiliar with this codebase however.
The text was updated successfully, but these errors were encountered: