You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
submodule.move seems to get me half the way there, but it misses a couple things. In new versions of git the .git is just a textfile that references a path in the base directory. This file isn't updated if you move directory depths because it is relative.
It also doesn't update the submodule name in the .git/config which would be helpful. I'll probably be writing the code to do this, but documentation on submodule.move would help too.
Thanks,
Mike
The text was updated successfully, but these errors were encountered:
Indeed submodule.move needs to be updated to newer git versions. Currently it should do a move correctly only for git versions prior to the change in the way submodules are handled.
Actually I have a patch for this in my mail box, kindly donated by http://www.alliedtelesis.co.nz/ , that I will upload here for completeness.
In the latest revision of master and the upcoming release v0.3.6, this issue will be fixed so that .git files are fully supported. During move, these files will be rewritten with the respective relative path.
The second part of this issue, automatically renaming submodules during move, is something I will implement when doing #238. It will only be automatic if the submodule name equals the previous submodule checkout path.
I will close this issue once the linked issue is closed.
Hi,
I am trying to make submodule moving easy.
submodule.move seems to get me half the way there, but it misses a couple things. In new versions of git the .git is just a textfile that references a path in the base directory. This file isn't updated if you move directory depths because it is relative.
It also doesn't update the submodule name in the .git/config which would be helpful. I'll probably be writing the code to do this, but documentation on submodule.move would help too.
Thanks,
Mike
The text was updated successfully, but these errors were encountered: