Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 26 Feb 2021 19:23:48 +0000 (UTC)
From:      Baptiste Daroussin <bapt@FreeBSD.org>
To:        rgrimes@freebsd.org
Cc:        Cy Schubert <Cy.Schubert@cschubert.com>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: 2a50a9de8340 - main - terminfo: add terminfo database
Message-ID:  <d0c4f575-9aba-4b04-a0f4-17e1c0fbedfb@FreeBSD.org>
In-Reply-To: <20210226173629.2sc6wlaasxjep3le@aniel.nours.eu>
References:  <202102261420.11QEKJoX006569@slippy.cwsent.com> <202102261730.11QHUh8n031156@gndrsh.dnsmgr.net> <20210226173629.2sc6wlaasxjep3le@aniel.nours.eu>

next in thread | previous in thread | raw e-mail | index | archive | help

26 f=C3=A9vr. 2021 18:36:40 Baptiste Daroussin <bapt@FreeBSD.org>:

> On Fri, Feb 26, 2021 at 09:30:43AM -0800, Rodney W. Grimes wrote:
>>> In message <202102251327.11PDR4eC083842@gitrepo.freebsd.org>, Baptiste
>>> Daroussi
>>> n writes:
>>>> The branch main has been updated by bapt:
>>>>
>>>> URL: https://cgit.FreeBSD.org/src/commit/?id=3D2a50a9de8340f08bd876e9e=
5993332ae
>>>> 14376f80
>>>>
>>>> commit 2a50a9de8340f08bd876e9e5993332ae14376f80
>>>> Author:=C2=A0=C2=A0=C2=A0=C2=A0 Baptiste Daroussin <bapt@FreeBSD.org>
>>>> AuthorDate: 2021-02-23 16:17:32 +0000
>>>> Commit:=C2=A0=C2=A0=C2=A0=C2=A0 Baptiste Daroussin <bapt@FreeBSD.org>
>>>> CommitDate: 2021-02-25 13:25:32 +0000
>>>>
>>>> =C2=A0=C2=A0=C2=A0 terminfo: add terminfo database
>>>> =C2=A0=C2=A0=C2=A0
>>>> =C2=A0=C2=A0=C2=A0 Tested by:=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 manu, jbei=
ch
>>>> ---
>>>> share/Makefile=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 |=
=C2=A0 1 +
>>>> share/terminfo/Makefile | 34 ++++++++++++++++++++++++++++++++++
>>>> 2 files changed, 35 insertions(+)
>>>>
>>>> diff --git a/share/Makefile b/share/Makefile
>>>> index c4e12b05f7db..d6854b230ae5 100644
>>>> --- a/share/Makefile
>>>> +++ b/share/Makefile
>>>> @@ -26,6 +26,7 @@ SUBDIR=3D ${_colldef} \
>>>> =C2=A0 ${_syscons} \
>>>> =C2=A0 tabset \
>>>> =C2=A0 termcap \
>>>> + terminfo \
>>>> =C2=A0 ${_timedef} \
>>>> =C2=A0 ${_vt} \
>>>> =C2=A0 ${_zoneinfo}
>>>> diff --git a/share/terminfo/Makefile b/share/terminfo/Makefile
>>>> new file mode 100644
>>>> index 000000000000..7bb11f3fdf24
>>>> --- /dev/null
>>>> +++ b/share/terminfo/Makefile
>>>> @@ -0,0 +1,34 @@
>>>> +PACKAGE=3D runtime
>>>> +
>>>> +.PATH: ${SRCTOP}/contrib/ncurses/misc
>>>> +TINFOBUILDDIR=3D ${.OBJDIR}/builddir
>>>> +CLEANDIRS+=3D=C2=A0 builddir
>>>> +
>>>> +.include <src.tools.mk>
>>>> +
>>>> +.if !defined(_SKIP_BUILD)
>>>> +all: terminfo
>>>> +.endif
>>>> +META_TARGETS+=3D terminfo install-terminfo
>>>> +
>>>> +terminfo: terminfo.src
>>>> + mkdir -p ${TINFOBUILDDIR}
>>>> + ${TIC_CMD} -x -o ${TINFOBUILDDIR} ${.ALLSRC}
>>>> +
>>>> +.if make(*install*)
>>>> +TINFOS!=3D=C2=A0 cd ${TINFOBUILDDIR} && find * -type f | LC_ALL=3DC s=
ort
>>>> +TINFOSDIRS=3D=C2=A0 ${TINFOS:C/(.).*/\1/g:O:u}
>>>> +.endif
>>>> +
>>>> +beforeinstall: install-terminfo
>>>> +install-terminfo:
>>>> + mkdir -p ${DESTDIR}/usr/share/terminfo
>>>> + cd ${DESTDIR}/usr/share/terminfo; mkdir -p ${TINFOSDIRS}
>>>> +.for f in ${TINFOS}
>>>> + ${INSTALL} ${TAG_ARGS} \
>>>> +=C2=A0=C2=A0=C2=A0=C2=A0 -o ${BINOWN} -g ${BINGRP} -m ${NOBINMODE} \
>>>> +=C2=A0=C2=A0=C2=A0=C2=A0 ${TINFOBUILDDIR}/${f} ${DESTDIR}/usr/share/t=
erminfo/${f}
>>>> +.endfor
>>>> +
>>>> +.include <bsd.prog.mk>
>>>> +
>>>>
>>>
>>> I think this had some unintended consequences, a POLA violation that sh=
ould
>>> be documented. Our termcap didn't support alternate screen buffers whil=
e
>>> terminfo does. Termcap did through t_te and t_ti but we didn't use them=
.
>>> Terminfo as delivered from the ncurses factory does supply rmcup and sm=
cup;
>>> our fullscreen apps such as nvi and top see them and use the alternate
>>> screen buffer.
>>>
>>> The use of the alternate screen buffer is one of the things I hate abou=
t
>>> Linux. After running vi or top I expect the output to remain on the scr=
een
>>> so I can use some bit of information in my next command. We now have th=
is
>>> behaviour too.
>>
>> Oh, I so hate that mis-feature of Linux, please please tell me there is
>> a simple knob to turn this off?=C2=A0 I believe LESS also does this kind
>> of stuff, or atleast "man" on linux does, and that pees me off to no
>> end when I quit the man page and what I wanted is no longer on the scree=
n.
>>
>>
> No this is not and I clear have 0 intention to have a complex patch on to=
p of
> the linux terminfo which would prefer easy sync in the futur.
>
> We can document how to use a definition which does not have the feature y=
ou do
> not like.
>
> Best,
> Bapt

Wow I think that is one of the most terrible mail I have ever written!

Let me rephrase it. I won t patch ncurses terminfo as that would make the m=
aintenance a huge pain. Ncurses is already not an easy beast to maintain ( =
should be a bit easier now).

The solution is to document how to workaround for the people who dislike th=
e new behaviour

Bapt



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?d0c4f575-9aba-4b04-a0f4-17e1c0fbedfb>