From 8f997b38cbc762cbf6482bce3c96291e0b67bc1e Mon Sep 17 00:00:00 2001 From: Sebastian Schuberth Date: Mon, 20 Jan 2020 11:42:48 +0100 Subject: [PATCH] repo: Do not even try to set up GPG with opt.no_repo_verify In order to be able to use "--no-repo-verify" to work around an issue with gpg-agent and long socket paths (see e.g. [1]), this change avoids GPG being set up at all if that option is passed. [1] https://github.com/elastic/elasticsearch/issues/17053 Change-Id: I1e5cbd8be2dc0084f12afe0ca33c789fdbc6fef9 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/251108 Reviewed-by: David Pursehouse Reviewed-by: Mike Frysinger Tested-by: Mike Frysinger --- repo | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/repo b/repo index 90cdb60..e723986 100755 --- a/repo +++ b/repo @@ -366,15 +366,18 @@ def _Init(args, gitc_init=False): _CheckGitVersion() try: - if NeedSetupGnuPG(): - can_verify = SetupGnuPG(opt.quiet) + if opt.no_repo_verify: + do_verify = False else: - can_verify = True + if NeedSetupGnuPG(): + do_verify = SetupGnuPG(opt.quiet) + else: + do_verify = True dst = os.path.abspath(os.path.join(repodir, S_repo)) _Clone(url, dst, opt.quiet, not opt.no_clone_bundle) - if can_verify and not opt.no_repo_verify: + if do_verify: rev = _Verify(dst, branch, opt.quiet) else: rev = 'refs/remotes/origin/%s^0' % branch