only set term if rsh is running /bin/sh

also set it without using env so it will work in containers
that don't have /bin/env

Author: bparees

pkg/cmd/cli/cmd/rsh.go

@@ -16,7 +16,10 @@ import (
 	"github.com/openshift/origin/pkg/cmd/util/clientcmd"
 )
 
-const RshRecommendedName = "rsh"
+const (
+	RshRecommendedName = "rsh"
+	DefaultShell       = "/bin/sh"
+)
 
 var (
 	rshLong = templates.LongDesc(`
@@ -94,7 +97,7 @@ func NewCmdRsh(name string, parent string, f *clientcmd.Factory, in io.Reader, o
 	}
 	cmd.Flags().BoolVarP(&options.ForceTTY, "tty", "t", false, "Force a pseudo-terminal to be allocated")
 	cmd.Flags().BoolVarP(&options.DisableTTY, "no-tty", "T", false, "Disable pseudo-terminal allocation")
-	cmd.Flags().StringVar(&options.Executable, "shell", "/bin/sh", "Path to the shell command")
+	cmd.Flags().StringVar(&options.Executable, "shell", DefaultShell, "Path to the shell command")
 	cmd.Flags().IntVar(&options.Timeout, "timeout", 10, "Request timeout for obtaining a pod from the server; defaults to 10 seconds")
 	cmd.Flags().StringVarP(&options.ContainerName, "container", "c", "", "Container name; defaults to first container")
 	cmd.Flags().SetInterspersed(false)
@@ -155,8 +158,9 @@ func (o *RshOptions) Validate() error {
 // Run starts a remote shell session on the server
 func (o *RshOptions) Run() error {
 	// Insert the TERM into the command to be run
-	term := fmt.Sprintf("TERM=%s", util.Env("TERM", "xterm"))
-	o.Command = append([]string{"env", term}, o.Command...)
-
+	if len(o.Command) == 1 && o.Command[0] == DefaultShell {
+		termsh := fmt.Sprintf("TERM=%q %s", util.Env("TERM", "xterm"), DefaultShell)
+		o.Command = append(o.Command, "-c", termsh)
+	}
 	return o.ExecOptions.Run()
 }