From bb52045c299b1e476b19e2516cc98c4218090a0f Mon Sep 17 00:00:00 2001
From: taoufik07 <abacidtaoufik@gmail.com>
Date: Mon, 2 Jan 2023 21:14:50 +0100
Subject: [PATCH] Remove `GOPATH` special build

---
 pre_commit/git.py              |  5 ----
 pre_commit/languages/golang.py | 47 ++++++++--------------------------
 tests/languages/golang_test.py | 22 ----------------
 3 files changed, 10 insertions(+), 64 deletions(-)
 delete mode 100644 tests/languages/golang_test.py

diff --git a/pre_commit/git.py b/pre_commit/git.py
index a76118f0b..333dc7ba3 100644
--- a/pre_commit/git.py
+++ b/pre_commit/git.py
@@ -93,11 +93,6 @@ def get_git_common_dir(git_root: str = '.') -> str:
         return get_git_dir(git_root)
 
 
-def get_remote_url(git_root: str) -> str:
-    _, out, _ = cmd_output('git', 'config', 'remote.origin.url', cwd=git_root)
-    return out.strip()
-
-
 def is_in_merge_conflict() -> bool:
     git_dir = get_git_dir('.')
     return (
diff --git a/pre_commit/languages/golang.py b/pre_commit/languages/golang.py
index 70f0e65d4..95921be75 100644
--- a/pre_commit/languages/golang.py
+++ b/pre_commit/languages/golang.py
@@ -7,7 +7,6 @@
 from typing import Sequence
 
 import pre_commit.constants as C
-from pre_commit import git
 from pre_commit.envcontext import envcontext
 from pre_commit.envcontext import PatchesT
 from pre_commit.envcontext import Var
@@ -15,7 +14,6 @@
 from pre_commit.languages import helpers
 from pre_commit.prefix import Prefix
 from pre_commit.util import cmd_output
-from pre_commit.util import cmd_output_b
 from pre_commit.util import rmtree
 
 ENVIRONMENT_DIR = 'golangenv'
@@ -36,53 +34,28 @@ def in_env(prefix: Prefix) -> Generator[None, None, None]:
         yield
 
 
-def guess_go_dir(remote_url: str) -> str:
-    if remote_url.endswith('.git'):
-        remote_url = remote_url[:-1 * len('.git')]
-    looks_like_url = (
-        not remote_url.startswith('file://') and
-        ('//' in remote_url or '@' in remote_url)
-    )
-    remote_url = remote_url.replace(':', '/')
-    if looks_like_url:
-        _, _, remote_url = remote_url.rpartition('//')
-        _, _, remote_url = remote_url.rpartition('@')
-        return remote_url
-    else:
-        return 'unknown_src_dir'
-
-
 def install_environment(
         prefix: Prefix,
         version: str,
         additional_dependencies: Sequence[str],
 ) -> None:
     helpers.assert_version_default('golang', version)
-    directory = helpers.environment_dir(prefix, ENVIRONMENT_DIR, C.DEFAULT)
-
-    remote = git.get_remote_url(prefix.prefix_dir)
-    repo_src_dir = os.path.join(directory, 'src', guess_go_dir(remote))
-
-    # Clone into the goenv we'll create
-    cmd = ('git', 'clone', '--recursive', '.', repo_src_dir)
-    helpers.run_setup_cmd(prefix, cmd)
+    env_dir = helpers.environment_dir(prefix, ENVIRONMENT_DIR, version)
 
     if sys.platform == 'cygwin':  # pragma: no cover
-        _, gopath, _ = cmd_output('cygpath', '-w', directory)
-        gopath = gopath.strip()
+        gopath = cmd_output('cygpath', '-w', env_dir)[1].strip()
     else:
-        gopath = directory
+        gopath = env_dir
     env = dict(os.environ, GOPATH=gopath)
     env.pop('GOBIN', None)
-    cmd_output_b('go', 'install', './...', cwd=repo_src_dir, env=env)
+
+    helpers.run_setup_cmd(prefix, ('go', 'install', './...'), env=env)
     for dependency in additional_dependencies:
-        cmd_output_b(
-            'go', 'install', dependency, cwd=repo_src_dir, env=env,
-        )
-    # Same some disk space, we don't need these after installation
-    rmtree(prefix.path(directory, 'src'))
-    pkgdir = prefix.path(directory, 'pkg')
-    if os.path.exists(pkgdir):  # pragma: no cover (go<1.10)
+        helpers.run_setup_cmd(prefix, ('go', 'install', dependency), env=env)
+
+    # Same some disk space, we don't need this after installation
+    pkgdir = os.path.join(env_dir, 'pkg')
+    if os.path.exists(pkgdir):  # pragma: no branch (always true on windows?)
         rmtree(pkgdir)
 
 
diff --git a/tests/languages/golang_test.py b/tests/languages/golang_test.py
deleted file mode 100644
index 9e393cb39..000000000
--- a/tests/languages/golang_test.py
+++ /dev/null
@@ -1,22 +0,0 @@
-from __future__ import annotations
-
-import pytest
-
-from pre_commit.languages.golang import guess_go_dir
-
-
-@pytest.mark.parametrize(
-    ('url', 'expected'),
-    (
-        ('/im/a/path/on/disk', 'unknown_src_dir'),
-        ('file:///im/a/path/on/disk', 'unknown_src_dir'),
-        ('git@github.com:golang/lint', 'github.com/golang/lint'),
-        ('git://github.com/golang/lint', 'github.com/golang/lint'),
-        ('http://github.com/golang/lint', 'github.com/golang/lint'),
-        ('https://github.com/golang/lint', 'github.com/golang/lint'),
-        ('ssh://git@github.com/golang/lint', 'github.com/golang/lint'),
-        ('git@github.com:golang/lint.git', 'github.com/golang/lint'),
-    ),
-)
-def test_guess_go_dir(url, expected):
-    assert guess_go_dir(url) == expected