On Fri, Nov 14, 2025 at 07:13:14AM -0600, Chris Adams wrote:
> Once upon a time, Jocelyn Falempe <[email protected]> said:
> > But on my test laptop, it's vt102 too.
> > It turns out, it's because kmscon starts "agetty" as the login
> > process, and that agetty overrides the TERM environment variable.
> > https://github.com/Aetf/kmscon/commit/21cf668c58867a100272adea6f323cd56fc73e78
> > 
> > The main benefit of agetty is to process /etc/issue. So maybe we can
> > implement that in kmscon instead.
> 
> Rather than reimplement that, you can just call agetty with the term
> type to set.

Yeah, fixing the call to agetty seems like a better approach.
There has been a bunch of work in util-linux to support drop-ins
for /etc/issue, so nowadays it's much more complicated that
reading one file.

agetty has the 'term' arg, and since kmscon knows that what the
value should be, it can just set it.

(The full paths in the unit are not necessary, so
ExecStart=@bindir@/kmscon can be just ExecStart=kmscon
and hopefully /sbin/agetty can just become 'agetty' and then
the whole unit becomes static and doesn't have to be templated.)

Zbyszek
-- 
_______________________________________________
devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to