[PATCH 0/4] Fix relative path issues in recursive submodules.
- From
Stefan Beller <sbeller@google.com>
- Date
- Mar 31, 2016, 00:17 UTC
- Message-ID
- <1459383457-6848-1-git-send-email-sbeller@google.com>
It took me longer than expected to fix this bug.
It comes with a test and minor refactoring and applies on top of origin/sb/submodule-helper, such that it can be merged into 2.7, 2.8 as well as master.
Patch 1 is a test which fails; it has a similar layout as the real failing repository Norio Nomura pointed out, but simplified to the bare essentials for reproduction of the bug.
Patch 2&3 are not strictly necessary for fixing the isseu, but it removes stupid code I wrote, so the resulting code looks a little better.
Patch 4 fixes the issue by giving more information to relative_path, such that a relative path can be found in all cases.
Thanks, Stefan
Stefan Beller (4): recursive submodules: test for relative paths submodule--helper clone: simplify path check submodule--helper clone: remove double path checking submodule--helper: use relative path if possible
builtin/submodule--helper.c | 16 ++++++++-------- t/t7400-submodule-basic.sh | 41 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 49 insertions(+), 8 deletions(-)
-- 2.5.0.264.g4004fdc.dirty