GitPython icon indicating copy to clipboard operation
GitPython copied to clipboard

Support additional worktrees

Open hilkoc opened this issue 7 years ago • 9 comments

As described in this issue: #344 . Since it's now almost 3 years later, it would be great if support for additional worktrees can be added.

hilkoc avatar Jan 29 '18 10:01 hilkoc

There isn't a wrapper for worktrees yet, but I found using Repo.git.worktree("add", folder, branch) will load the worktree for you.

From there you can navigate into the worktree and use it like you world any other worktree.

To unload the worktree, simply remove the folder and then call Repo.git.worktree("prune").

This works with the latest version.

Full example:

import os
import git
import shutil

os.mkdir(<desired-folder>)

repo = git.Repo()
print (repo) # You'll see the original branch path that you're on here

repo.git.worktree("add", <folder>, <branch>)

test_repo = git.Repo(<folder>)
print (test_repo) # Now you'll see that with this new git.Repo object that it's referencing your worktree
print (repo) # Just to confirm that this is pointing to the original location

# Run a simple command to make sure everythings working - might be better to use the wrapper library to confirm though
print (repo.git.status())
print (test_repo.git.status())

# Remove the worktree like this:
shutil.rmtree(<folder>)
repo.git.worktree("prune")

KellyDeveloped avatar Jan 30 '18 16:01 KellyDeveloped

Thanks for this. I managed to get it working. The main problem was this: When you're in an additional worktree,

remote = repo.remote(name='origin')
remote.push()

does not work.

So instead, all the remote commands need to be called with repo.git.push() or repo.git.pull() etc.

hilkoc avatar Feb 01 '18 11:02 hilkoc

@KellyDeveloped @hilkoc

I am curious how you got this working. The example where you used Repo.git.worktree("add", folder, branch) and then initialized with git.Repo() doesn't seem to work. git.Repo() always throws an InvalidGitRepositoryError when the folder path is a worktree.

My main objective is to get all the file names from the previous commit, which I can do using "repo.git.diff('HEAD~1', name_only=True)". This works with git clones, but with worktrees, git.Repo() throws an InvalidGitRepositoryError.

Do you know how I could get this working? or is there another way to get file names from git commits?

skharat8 avatar Sep 22 '18 01:09 skharat8

@KellyDeveloped @hilkoc

I am curious how you got this working. The example where you used Repo.git.worktree("add", folder, branch) and then initialized with git.Repo() doesn't seem to work. git.Repo() always throws an InvalidGitRepositoryError when the folder path is a worktree.

My main objective is to get all the file names from the previous commit, which I can do using "repo.git.diff('HEAD~1', name_only=True)". This works with git clones, but with worktrees, git.Repo() throws an InvalidGitRepositoryError.

Do you know how I could get this working? or is there another way to get file names from git commits?

It sounds like you're navigating into the wrong folder. This error throws when you aren't inside any type of git repo. I'd suggest using REPL to go through it line by line and see what folder you're in and checking to see if there's a .git folder in there.

The .git folder simply points back to the real git repo.

KellyDeveloped avatar Sep 22 '18 10:09 KellyDeveloped

@KellyDeveloped

I think the problem is that there is no .git folder for worktrees. There is a .git file instead.

I tried printing out the path that is_git_dir() is checking in fun.py in GitPython (called in Repo class __init__ function). It looks like it goes in a loop like this and then fails with InvalidGitRepositoryError:

C:/test_worktree C:/test_worktree/.git C:/test_clone/.git/worktrees/test_worktree C:/test_worktree/.git C:/test_clone/.git/worktrees/test_worktree

Just tried changing the is_git_dir() function to add a check for a file as well, and that actually worked! Since worktrees are not really supported, I am not sure if this would cause other issues, but it seems to initialize the git repo properly with git.Repo() for my use case.

def is_git_dir(d):
    """ This is taken from the git setup.c:is_git_directory
    function.
    @throws WorkTreeRepositoryUnsupported if it sees a worktree directory. It's quite hacky to do that here,
            but at least clearly indicates that we don't support it.
            There is the unlikely danger to throw if we see directories which just look like a worktree dir,
            but are none."""
    if osp.isdir(d):
        if osp.isdir(osp.join(d, 'objects')) and osp.isdir(osp.join(d, 'refs')):
            headref = osp.join(d, 'HEAD')
            return osp.isfile(headref) or \
                (osp.islink(headref) and
                 os.readlink(headref).startswith('refs'))
        elif (osp.isfile(osp.join(d, 'gitdir')) and
              osp.isfile(osp.join(d, 'commondir')) and
              osp.isfile(osp.join(d, 'gitfile'))):
            raise WorkTreeRepositoryUnsupported(d)
   # Added this for worktree support
    elif osp.isfile(d):
        return True
    return False

skharat8 avatar Sep 22 '18 21:09 skharat8

Thanks for sharing all your insights! It's much appreciated and certainly helpful to others.

Byron avatar Oct 14 '18 10:10 Byron

Keeps biting us in DataLad, so if some brave soul gets to resolve it -- would be much appreciated!

yarikoptic avatar May 03 '19 00:05 yarikoptic

This was closed via #894 and released with v3.0.0

Byron avatar Aug 14 '19 09:08 Byron

Reopened as the fix was reverted to deal with the performance regression reported in #906 . We hope to let the fix return in a version that caches the output it receives from git-revparse or obtains the information in another way.

Byron avatar Aug 14 '19 09:08 Byron