v.io/jiri/profiles/profilescmdline: avoid whitespace around installers.
Change-Id: I3087c6f0f90db33cb9a192607f62ca8114e6b784
diff --git a/profiles/profilescmdline/manager_cmdline.go b/profiles/profilescmdline/manager_cmdline.go
index 1fa9aa7..c329dee 100644
--- a/profiles/profilescmdline/manager_cmdline.go
+++ b/profiles/profilescmdline/manager_cmdline.go
@@ -297,8 +297,10 @@
fmt.Fprintf(jirix.Stderr(), "failed to run %s %s: %v", sc, strings.Join(args, " "), err)
return nil, err
}
- mgrs := strings.TrimSpace(out.String())
- names = append(names, strings.Split(mgrs, ",")...)
+ mgrs := out.String()
+ for _, m := range strings.Split(mgrs, ",") {
+ names = append(names, strings.TrimSpace(m))
+ }
}
return names, nil
}
diff --git a/profiles/profilescmdline/profile_manager.go b/profiles/profilescmdline/profile_manager.go
index e2e54dd..f2f7190 100644
--- a/profiles/profilescmdline/profile_manager.go
+++ b/profiles/profilescmdline/profile_manager.go
@@ -6,6 +6,7 @@
import (
"fmt"
+ "strings"
"v.io/jiri/jiri"
"v.io/jiri/profiles"
@@ -25,6 +26,7 @@
func newProfileManager(name string, db *profiles.DB) profileManager {
installer, profile := profiles.SplitProfileName(name)
+ installer = strings.TrimSpace(installer)
if len(installer) == 0 || installer == profileInstaller {
return &inproc{installer, profile, name, db}
}
