trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: July 2012

Re: [trinity-devel] [Patch] Re: [trinity-devel] tdm login screen - gentdmconf, need to check existence of /etc/X11/Xsession before sourcing

From: Francois Andriot <francois.andriot@...>
Date: Fri, 20 Jul 2012 18:33:54 +0200
Le 20/07/2012 18:06, David C. Rankin a écrit :
> On 07/19/2012 12:45 PM, David C. Rankin wrote:
>> Tim, Darrell,
>>
>>    One problem with gentdmconf is that arch does not have an 
>> /etc/X11/Xsession
>> file. So when gentdeconf creates the Xsession file with an include for
>> /etc/X11/Xsession, the login fails. What needs to be changed is a 
>> simple check
>> to make sure the file exists before sourcing it. i.e.:
>>
>> [[ -r /etc/X11/Xsession ]] && . /etc/X11/Xsession
>>
>>
>>
>
> Patch attached.

Hello, under RHEL/Fedora, the Xsession file exists but is located elsewhere.
Can we please add the same kind of check, e.g:

[[ -r /etc/X11/xinit/Xsession ]] && . /etc/X11/xinit/Xsession

So that I wouldn't need to patch again when packaging.

Thanks
Francois