Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(28)

Issue 3181025: Always re-configure to use ssh even on incremental checkouts (Closed)

Created:
10 years, 4 months ago by sosa
Modified:
9 years, 6 months ago
Reviewers:
Chris Masone
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Base URL:
http://src.chromium.org/git/crosutils.git
Visibility:
Public.

Description

Always re-configure to use ssh even on incremental checkouts Change-Id: I0a729b14e34c61a8174edd63fb20066e0811128e

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M bin/cbuildbot.py View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
sosa
10 years, 4 months ago (2010-08-19 01:12:30 UTC) #1
Chris Masone
10 years, 4 months ago (2010-08-19 01:15:35 UTC) #2
LGTM

On Wed, Aug 18, 2010 at 6:12 PM, <sosa@chromium.org> wrote:

> Reviewers: Chris Masone,
>
> Description:
> Always re-configure to use ssh even on incremental checkouts
>
> Change-Id: I0a729b14e34c61a8174edd63fb20066e0811128e
>
> Please review this at http://codereview.chromium.org/3181025/show
>
> SVN Base: http://src.chromium.org/git/crosutils.git
>
> Affected files:
>  M bin/cbuildbot.py
>
>
> Index: bin/cbuildbot.py
> diff --git a/bin/cbuildbot.py b/bin/cbuildbot.py
> index
>
5bcb04788006c4fa7d907b151666c4a66ceb11fd..77b74bdd3a329debdd5de95ddb936913c6b3a880
> 100755
> --- a/bin/cbuildbot.py
> +++ b/bin/cbuildbot.py
> @@ -64,6 +64,11 @@ def _FullCheckout(buildroot):
>
>  def _IncrementalCheckout(buildroot):
>   RunCommand(['repo', 'sync'], cwd=buildroot)
> +  # Always re-run in case of new git repos or repo sync
> +  # failed in a previous run because of a forced Stop Build.
> +  RunCommand(['repo', 'forall', '-c', 'git', 'config',
> +              'url.ssh://git@gitrw.chromium.org:9222.pushinsteadof',
> +              'http://src.chromium.org/git'], cwd=buildroot)
>
>  def _MakeChroot(buildroot):
>   cwd = os.path.join(buildroot, 'src', 'scripts')
>
>
>

Powered by Google App Engine
This is Rietveld 408576698