FS#14967 - [tcsh] csh.cshrc, mentions environment variables no longer used

Attached to Project: Arch Linux
Opened by Stephen (wingedsubmariner) - Friday, 05 June 2009, 22:12 GMT
Last edited by Eric Belanger (Snowman) - Wednesday, 24 June 2009, 02:39 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Eric Belanger (Snowman)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
/etc/csh.cshrc forces the creation of a .tcsh.config in the user's home for setting certain environment variables, e.g. TCSH_SHELL_CTRLD, that are not actually used by tcsh. tcsh does not use environment variables for configuration, rather it has its own "shell variables" to avoid polluting the environment, and tcsh's manpage does not mention these environment variables. As far as I can tell, they are not read by any of the configuration scripts either, except for one mention of TCSH_SHELL_COMPLETION in csh.cshrc. This code is all unneccessary and I think is left over from when the scripts used to work differently (maybe only in Gentoo, where they came from).

This is not an upstream problem, the tarball from tcsh.org doesn't mention these environment variables (grep -r) and doesn't include csh.cshrc (except for cygwin).

How to fix:
apply my patch to csh.cshrc (attached).
remove /etc/skel/.tcsh.config

Additional info:
extra/tcsh 6.16.00-3

Steps to reproduce:
less /etc/csh.cshrc
or launch tcsh as a new user and see the message about the creation of .tcsh.config.
   csh.cshrc.patch (1.1 KiB)
This task depends upon

Closed by  Eric Belanger (Snowman)
Wednesday, 24 June 2009, 02:39 GMT
Reason for closing:  Fixed
Additional comments about closing:  fixed in tcsh-6.16.00-4
Comment by Gerardo Exequiel Pozzi (djgera) - Monday, 15 June 2009, 04:08 GMT
@Stephen, please provide the patch in unified format (diff -u) or against SVN trunk (svn diff). This make an easy way to review this issue. Thanks.
Comment by Stephen (wingedsubmariner) - Monday, 15 June 2009, 14:33 GMT
Here it is in unified format (diff -u).
   csh.cshrc.patch (1.4 KiB)

Loading...