needed for https://veyron-review.googlesource.com/#/c/6158/
You can use veryon.namespace.roots to set the namespace roots for
a veyron process, but I took away the option since there would then
be 4 ways to configure the same info!
- the environment variables
- the command line
- the SetRoots call on the namespace
- and, if it remained, the option to the factory!
Change-Id: I4106a721a5512507e880b91cd5e60baf5b56f3b3
diff --git a/services/wsprd/app/app.go b/services/wsprd/app/app.go
index 3417322..3d13afb 100644
--- a/services/wsprd/app/app.go
+++ b/services/wsprd/app/app.go
@@ -145,7 +145,7 @@
// NewController creates a new Controller. writerCreator will be used to create a new flow for rpcs to
// javascript server. veyronProxyEP is an endpoint for the veyron proxy to serve through. It can't be empty.
-// opts are any options that should be passed to the rt.New(), such as the mounttable root.
+// opts are any options that should be passed to the rt.New().
func NewController(writerCreator func(id int64) lib.ClientWriter,
listenSpec *ipc.ListenSpec, opts ...veyron2.ROpt) (*Controller, error) {
r, err := rt.New(opts...)
diff --git a/services/wsprd/app/app_test.go b/services/wsprd/app/app_test.go
index c99a45e..920fb51 100644
--- a/services/wsprd/app/app_test.go
+++ b/services/wsprd/app/app_test.go
@@ -355,11 +355,12 @@
}
spec := profiles.LocalListenSpec
spec.Proxy = "/" + proxyEndpoint
- controller, err := NewController(writerCreator, &spec, options.NamespaceRoots{"/" + endpoint.String()}, options.RuntimePrincipal{testPrincipal})
+ controller, err := NewController(writerCreator, &spec, options.RuntimePrincipal{testPrincipal})
if err != nil {
return nil, err
}
+ controller.rt.Namespace().SetRoots("/" + endpoint.String())
controller.serve(serveRequest{
Name: "adder",
