mirror of
https://gerrit.googlesource.com/git-repo
synced 2024-12-21 07:16:21 +00:00
Revert "Save cookies back to jar when fetching clone.bundle"
This reverts commit4abf8e6ef8
. The curl process for updating the cookie file is not atomic. When fetching many bundles in parallel, we can sometimes corrupt the file causing it to be cleared. Since users should manage gitcookies on their own, leave it read-only. Bug: https://crbug.com/gerrit/12300 Change-Id: Id472c99b197bc4cf8533c649f8881509f38643c1 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/254092 Reviewed-by: David Pursehouse <dpursehouse@collab.net> Tested-by: Mike Frysinger <vapier@google.com> (cherry picked from commitdc1d0e0c7f
)
This commit is contained in:
parent
aeb2eee9d3
commit
d92076d930
@ -2398,7 +2398,7 @@ class Project(object):
|
|||||||
platform_utils.remove(tmpPath)
|
platform_utils.remove(tmpPath)
|
||||||
with GetUrlCookieFile(srcUrl, quiet) as (cookiefile, proxy):
|
with GetUrlCookieFile(srcUrl, quiet) as (cookiefile, proxy):
|
||||||
if cookiefile:
|
if cookiefile:
|
||||||
cmd += ['--cookie', cookiefile, '--cookie-jar', cookiefile]
|
cmd += ['--cookie', cookiefile]
|
||||||
if proxy:
|
if proxy:
|
||||||
cmd += ['--proxy', proxy]
|
cmd += ['--proxy', proxy]
|
||||||
elif 'http_proxy' in os.environ and 'darwin' == sys.platform:
|
elif 'http_proxy' in os.environ and 'darwin' == sys.platform:
|
||||||
|
Loading…
Reference in New Issue
Block a user