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

Issue 147313006: Make sure cygwin mounts are set up before launching cygwin lighttpd (Closed)

Created:
6 years, 10 months ago by scottmg
Modified:
6 years, 10 months ago
Reviewers:
Nico, M-A Ruel
CC:
chromium-reviews, cmp-cc_chromium.org, ilevy-cc_chromium.org
Visibility:
Public.

Description

Make sure cygwin mounts are set up before launching cygwin lighttpd R=thakis@chromium.org BUG=123026

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M slave/run_webserver.bat View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
scottmg
6 years, 10 months ago (2014-02-01 00:20:51 UTC) #1
Nico
maruel, do you know this script?
6 years, 10 months ago (2014-02-01 00:27:10 UTC) #2
scottmg
On 2014/02/01 00:27:10, Nico wrote: > maruel, do you know this script? And, in particular ...
6 years, 10 months ago (2014-02-01 00:32:02 UTC) #3
M-A Ruel
On 2014/02/01 00:32:02, scottmg wrote: > On 2014/02/01 00:27:10, Nico wrote: > > maruel, do ...
6 years, 10 months ago (2014-02-01 21:11:32 UTC) #4
Nico
lgtm stamp then I guess Please work with someone in infra to check that this ...
6 years, 10 months ago (2014-02-03 20:20:33 UTC) #5
scottmg
On 2014/02/03 20:20:33, Nico wrote: > lgtm stamp then I guess > > Please work ...
6 years, 10 months ago (2014-02-03 20:50:53 UTC) #6
scottmg
6 years, 10 months ago (2014-02-04 00:45:27 UTC) #7
On 2014/02/03 20:50:53, scottmg wrote:
> On 2014/02/03 20:20:33, Nico wrote:
> > lgtm stamp then I guess
> > 
> > Please work with someone in infra to check that this script still works with
> > your stuff in.
> 
> Thanks, I try to figure out if anyone knows how/if mounts are set up, and
> hopefully we don't need this.

On further reflection, the buildbot webserver would need to come up before DEPS
runhooks happened, so I conclude this is unnecessary.

Powered by Google App Engine
This is Rietveld 408576698