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

Issue 8936004: Avoid using an invalid working directory in mjsunit/d8-os. (Closed)

Created:
9 years ago by kalmard
Modified:
9 years ago
CC:
v8-dev
Visibility:
Public.

Description

Avoid using an invalid working directory in mjsunit/d8-os. This test deleted its working directory and then tried to run several shell commands which caused a failure on nfs. Changes: -TEST_DIR is only removed at the very end of the test -the working directory is changed to /tmp at the beginning so that every iteration (when running with --stress-opt) has a valid working directory BUG= TEST= Committed: http://code.google.com/p/v8/source/detail?r=10240

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -35 lines) Patch
M test/mjsunit/d8-os.js View 2 chunks +38 lines, -35 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
kalmard
9 years ago (2011-12-13 09:43:32 UTC) #1
Yang
9 years ago (2011-12-13 12:57:02 UTC) #2
On 2011/12/13 09:43:32, kalmard wrote:

LGTM and landing.

Powered by Google App Engine
This is Rietveld 408576698