Re: tcsh backtick hang info

On 2007-07-12 14:56, Christos Zoulas <christos_at_zoulas.com> wrote:
> On Jul 12, 11:48am, dwhite_at_gumbysoft.com (Doug White) wrote:
> -- Subject: Re: tcsh backtick hang info
>
> | Thanks for the good words, Christos! If you can confirm this will be the
> | official patch then we can apply it to our vendor sources. If there are
> | any other fixes for nasty bugs in 6.15.00 that you're hanging on to it'd
> | be good to get those out and imported as well.
>
> You are welcome!
> This will be the official patch, and yes I have another 9 bug fixes since
> 6.15.00. In general, I don't make 6.15.XX generally available, but the
> severity of some of those problems fixed is high enough to possibly justify
> a 6.16.00 sooner than later. Here's the list of fixes:
>
>  10. kill `foo` got stuck because sigchld was disabled too soon (Mark Peek)
>   9. Avoid null pointer dereference in proc cwd (Kurt Miller)
>   8. eval "foreach a b c" exits (Anthony Menasse)
>   7. Quoting was broken in substitutions (Joe Wells)
>   6. QNX patches via pkgsrc
>   5. cd - twice from a directory that contained a glob pattern,
>      expands the glob twice (Mark Santcroos)
>   4. MidnightBsd support (Lucas Holt)
>   3. Fix history substitution core-dump with no history entries
>   2. Merge two character tables that are the same (Martin Kraemer)
>   1. On ancient 7 bit locales, punctuation characters are used to
>      denote special characters such as umlaut, adiaresis, etc.
>      These characters return true for isalpha/isalnum. Ignore them
>      because they break parsing (Martin Kraemer)

Hi Christos,

Can we persuade you to include the following local patch I keep for
autologout detection with /dev/pts/XXX ptys in FreeBSD?

%%%
Fix pty detection logic of tcsh autologout initialization.

Noticed by:     kris

diff --git a/contrib/tcsh/sh.c b/contrib/tcsh/sh.c
--- a/contrib/tcsh/sh.c
+++ b/contrib/tcsh/sh.c
_at__at_ -457,7 +457,7 _at__at_ main(int argc, char **argv)
        if (*cp) {
            /* only for login shells or root and we must have a tty */
            if ((cp2 = Strrchr(cp, (Char) '/')) != NULL) {
-               cp = cp2 + 1;
+               cp2 = cp2 + 1;
            }
            else
                cp2 = cp;
%%%