Re: Update of wip/sddm to version 0.21

["David H. Gutteridge" <david%gutteridge.ca@localhost>]

On Fri, 2 Jan 2026 at 02:51:30 +0100, Stepan Ipatov wrote:

> On 12/30/2025 7:07 PM, David H. Gutteridge wrote:
> > Have you considered sending the patches upstream? (I ask this being
> a
> > guilty party who sometimes doesn't do this; the scale can get
> > unmanageable quickly.)
> 
> 
> Thank you for the kind words.
> 
> Yes, I am planning to send the patches upstream to the sddm developers
> in the near future.
> 
> 
> In the meantime, could you please update the package from wip once
> again?
> 
> I've found a rather annoying issue: when sddm is started during system
> boot, the keyboard does not work. It happens due to the way sddm
> selects the virtual TTY for starting the X server. It calls ioctl(fd,
> VT_OPENQRY, &vt) and gets the first free TTY. The problem is that at
> the stage of system boot when the rc scripts are executed, the ioctl()
> call returns /dev/ttyE1. Apparently, sddm and getty do not coexist
> well on the same TTY. The only workaround I've found so far is to
> delay starting sddm until init has already spawned getty. Once that
> happens, ioctl() returns the expected /dev/ttyE4.
> 
> 
> I've committed an updated version to wip/sddm. Only two files were
> changed:
> 
>  * The sddm.sh rc script
> 
> * patch-src_common_VirtualTerminal.cpp — fixed the TTY device naming
> scheme, since wscons does not support more than 8 devices

Thanks for your diligence. I hadn't tested before "cold", only after my
system was already up.

You'll see I've made a few extra or different minor changes. (I'd
noticed one of them before, worked around it, and then forgot to remove
the "s" in the variable name when I committed.) I didn't apply your
LOCALBASE change, as the norm in pkgsrc is to use PREFIX pretty much
everywhere. (I couldn't find anywhere that consisely explained that, but
there are mailing list discussions floating around that I half
remember.) I added more standard and package-defined substitutions into
the rc.d script instead of some of the other changes you'd made.

Thanks,

Dave