Bug 684785

+++ This bug was initially created as a clone of Bug #671003 +++

Description of problem:

  Here's the content of the csh profile /etc/profile.d/cvs.csh:

    # change default from rsh to ssh for cvs command
    setenv CVS_RSH "${CVS_RSH-ssh}"

  This is not valid tcsh syntax.  This kind of variable substitution
  is only valid in POSIX shells.

  Right now, this bug does not result in an error when starting tcsh, but
  only because current tcsh versions ignore that line for the simple reason
  that it's missing a newline.  However, the upcoming tcsh 6.18.00 will NOT
  ignore that last line anymore and a syntax error will occur.

  The correct syntax for csh and tcsh is

    if ( "$?CVS_RSH" == 0 ) setenv CVS_RSH

  I hope this can be fixed soon.  And, please add a newline since otherwise
  the line will never be executed with the current tcsh 6.17.00.


Thanks,
Corinna

--- Additional comment from vinschen on 2011-01-19 21:36:08 GMT ---

Oops, I missed to set CVS_RSH to ssh.  So, here the *corrected*
correct syntax for csh and tcsh:

    if ( "$?CVS_RSH" == 0 ) setenv CVS_RSH ssh

---

cvs-1.11.23-11.el6_0.1.x86_64 in RHEL-6.1 is affected too. The CVS_RSH variable is not set on log-in into csh because missing new line causes silent ignorance of the command.

As we are adding CSH support into RHEL-5 (bug #571559), we should make sure RHEL-6 provides the same experience.