playground: Address https://v.io/i/1220
- reverts https://v.io/c/20483
- changes v23test to not share a bin dir across test main's
(bin dir is still shared across tests within a given test
main, but that's safe since "go test" will not run such
tests concurrently by default)
- changes v23test.StartRootMountTable and
v23test.StartSyncbase to use gosh.Shell.FuncCmd
(refactoring syncbased and mounttabled as needed to make
this possible)
A side effect of this change is that tests should run much
faster, because for the common case of tests that previously
used StartRootMountTable and/or StartSyncbase, but didn't
directly call BuildGoPkg, we'll no longer build Go packages
during test execution.
MultiPart: 4/5
Change-Id: I05fd1597c728c9850917b37ce1b4e136b0c354fc
diff --git a/go/src/v.io/x/playground/builder/builder_v23_test.go b/go/src/v.io/x/playground/builder/builder_v23_test.go
index 4837f7a..ff99b03 100644
--- a/go/src/v.io/x/playground/builder/builder_v23_test.go
+++ b/go/src/v.io/x/playground/builder/builder_v23_test.go
@@ -50,7 +50,7 @@
sh.Pushd(tmp)
defer sh.Popd()
- PATH := os.Getenv("V23_BIN_DIR")
+ PATH := filepath.Dir(builderPath)
if path := os.Getenv("PATH"); len(path) > 0 {
PATH += ":" + path
}
