Re: [PATCH] agetty: map NL to CR-NL on output in initial termios settings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[CC: to Werner Fink]

On Wed, Nov 16, 2011 at 10:54:50AM +0100, Dennis Jensen wrote:
> This fixes situations where messages "scroll horizontally", making

Do you mean /etc/issue and prompt strings? The terminal end-of-line
should be detected in get_logname() and then the proper flags should
be set in termio_final().

> them unreadable, because agetty has opened and initialized the
> terminal.
> 
> From: Jacoby Hickerson <hickersonjl@xxxxxxxxx>
> Signed-off-by: Dennis Jensen <dennis.h.jensen@xxxxxxxxxxx>
> ---
>  term-utils/agetty.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/term-utils/agetty.c b/term-utils/agetty.c
> index 041e1f7..27edac5 100644
> --- a/term-utils/agetty.c
> +++ b/term-utils/agetty.c
> @@ -1091,7 +1091,8 @@ static void termio_init(struct options *op, struct termios *tp)
>  #else
>  	tp->c_iflag = 0;
>  #endif
> -	tp->c_lflag = tp->c_oflag = 0;
> +	tp->c_lflag = 0;
> +	tp->c_oflag = OPOST | ONLCR;

Is it correct to always set ONLCR? Would be better to follow the
current (kernel) setting? I mean:

  tp->c_oflag = (tp->c_oflag & POST) | (tp->c_oflag & ONLCR);

Can you try it?

We use ONLCR for virtual consoles, but for serial lines it depends on
user's input (usersname string).

    Karel

-- 
 Karel Zak  <kzak@xxxxxxxxxx>
 http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux