-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Rebase failed with error: "unable to unlink old <filename> Invalid argument" (works Ok with older version) #1262
Comments
Thanks for the report @AlexeyKryshen ! In Git 2.11 we changed the default protocol between Git and Git LFS (see blog post, Git Merge 2017 talk, change in Git). During the implementation I hit the same issue that you are seeing now and I fixed it for my use cases. Looks like there are other (totally valid!) use cases that my fix did not cover. That needs to be fixed! Quick SolutionEdit your Proper SolutionI'll try to reproduce your problem locally. If I succeed then I should be able to fix it. If not then we need to find another way to debug this. However, all this might take a while (weeks) as I have lots of other things on my todo list 🙂 /CC @ttaylorr |
@larsxschneider, thank you for your reply and especially for the provided Quick Fix. I made and store a full copy of the project in problematic state so if you will be in need, I could do some remote diagnosis/debug for you by your requests/instructions. Many thanks to you and whole Git for Windows team for the great product! (especially for Portable version) :) |
We are currently facing the same issue in combination with Linux shares that are hosted on Isilon storage. However, in our case the workaround by removing the "process = git-lfs filter-process" does not temporary fix the issue. The most recent version that works is Git for Windows 2.12.2(2). |
@naatje80 Can you explain your Linux share setup further? I assume the following:
Can you run In general Git repos on samba shares are not recommended and error prone AFAIK. |
Dear larsxschneider, I'm very sorry for my delayed response. I got the following answers from our System Management department. To answer your questions:
The output of the command with verbose options is:
Some further information: I also setup a samba share on a Centos 7.4 workstation. The disk there is using ext4. On this machine it still seems to be working with git/2.14.1.windows.1. |
This is very confusing, as the original report is about a |
Ok, I thought the issues were related. But I will create a new ticket for this. Thank you for the reply. |
@AlexeyKryshen @larsxschneider what is the state of this ticket? |
Setup
Portable version
defaults?
Default Portable version installation
to the issue you're seeing?
LFS installed and used on repository caused the issue
Details
Bash
Minimal, Complete, and Verifiable example
this will help us understand the issue.
master is just 1 commit ahead of branching point and branch I want to rebase contains 2 commits (branch based on master). Branch commits contains only new files under LFS (no any single modified from master). Branch commits age huge (~1 Gb/ ~3000 files each). First branch commit contains only newly added files and second one delete ~50 of them and modify each remaining ( ~50 deleted & ~2950 modified). Again, all files in branch commits are under LFS.
branch successfully rebased on newest master commit
Rebased failed with the following message:
URL to that repository to help us with testing?
Unfortunately it is happened with local repository.
The scenario (rebasing branches with huge LFS commits) is common for the projects I am currently working with and for about an year it is the first time the issue occur (we start using LFS about an year ago). The issue occur when I was trying newest version of Portable Git for windows (2.14.0.2). Version 2.10.2 (the version we working with currently) works as expected and doesn't raise the issue with this rebase. Version 2.13.3 also cause the same rebase issue. Other versions are not tested by me.
The text was updated successfully, but these errors were encountered: