Skip to content

Commit 042dc72

Browse files
evanrittenhouseradoering
authored andcommitted
Implement reviewer comments
1 parent 9b59e88 commit 042dc72

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

Diff for: src/poetry/vcs/git/backend.py

+4-8
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,7 @@ def _clone_submodules(cls, repo: Repo) -> None:
333333
modules_config = repo_root.joinpath(".gitmodules")
334334

335335
# A relative URL by definition starts with ../ or ./
336-
relative_submodule_regex = re.compile("^(\\.\\./|\\./),?")
336+
relative_submodule_regex = re.compile(r"^(\.{1,2})/?")
337337

338338
if modules_config.exists():
339339
config = ConfigFile.from_path(str(modules_config))
@@ -347,12 +347,8 @@ def _clone_submodules(cls, repo: Repo) -> None:
347347
path_absolute = repo_root.joinpath(path_relative)
348348

349349
url_string = url.decode("utf-8")
350-
final_url = url_string
351-
submodule_is_relative = bool(
352-
relative_submodule_regex.search(url_string)
353-
)
354-
if submodule_is_relative:
355-
final_url = urljoin(f"{Git.get_remote_url(repo)}/", url_string)
350+
if relative_submodule_regex.search(url_string):
351+
url_string = urljoin(f"{Git.get_remote_url(repo)}/", url_string)
356352

357353
source_root = path_absolute.parent
358354
source_root.mkdir(parents=True, exist_ok=True)
@@ -370,7 +366,7 @@ def _clone_submodules(cls, repo: Repo) -> None:
370366
continue
371367

372368
cls.clone(
373-
url=final_url,
369+
url=url_string,
374370
source_root=source_root,
375371
name=path_relative.name,
376372
revision=revision,

0 commit comments

Comments
 (0)