On Sat, Nov 25, 2023 at 10:28:13AM +0700, Max Nikulin wrote: > SDDM does read /etc/profile and ~/.profile when starting a user session: > https://sources.debian.org/src/sddm/0.20.0-1/data/scripts/Xsession/
Interesting. I wondered whether that might be a Debian patch, since I couldn't see mention of it in the upstream SDDM changelog. But it's in the upstream repository as well, and it seems it's been there for a rather long time: https://github.com/sddm/sddm/blob/develop/data/scripts/Xsession The cute little unquoted $0 is practically mandatory -- you can't release a shell script without massively stupid quoting bugs, right? Ah well, I'm sure no operating system vendor would **EVER** make a directory with a space in its name, so it should be **just fine**, right?? :-( > (wayland-session is a sibling file). I do not like that a part of code is > similar, but not identical to /etc/X11/Xsession from x11-common. Ideally > there should be no code duplication. Well, if the ultimate goal is to have everything work consistently, then there's gonna be code duplication. Of course, we're nowhere near that goal. > I expect that GDM contains similar code reading /etc/profile and ~/.profile. > LightDM Ubuntu package (but not Debian one) has it. Ubuntu makes a change of that nature? That surprises me. It seems uncharacteristically nerdy for their mission goal of dumbing everything down to a common denominator. Could it be an upstream change instead, with Ubuntu simply using a newer upstream version than Debian?
