[PATCH v2 0/6] clone: fix repo name when cloning a server's root

As proposed the previous patch has been changed such that we now utilize parse_connect_url(), which splits a connect URL by host and path components. If the path component is empty we now try to fall back to the host component, stripping it of its port and authentication information.

The first two patches fix the t1509-root-worktree tests. They currently were not working due to broken &&-chains and are required to test cloning from /.

The next two patches expose parse_connect_url(), which is then used in the fifth patch. The last patch provides tests for the behavior. To keep it simple we just use file:// URIs, which have the option of including a host in there, e.g. file://127.0.0.1/foobar.

Patrick Steinhardt (6):
  tests: fix broken && chains in t1509-root-worktree
  tests: fix cleanup after tests in t1509-root-worktree
  connect: expose parse_connect_url()
  connect: move error check to caller of parse_connect_url
  clone: fix hostname parsing when guessing dir
  clone: add tests for cloning / without workdir name

 builtin/clone.c          | 73 +++++++++++++++++++++++++++++++++++-------------
 connect.c                | 19 +++----------
 connect.h                | 13 +++++++++
 t/t1509-root-worktree.sh | 43 +++++++++++++++++++++++++---
 4 files changed, 109 insertions(+), 39 deletions(-)