Re: SLiM session types

[tcech@suse.cz]

[-- Attachment #1: Type: text/plain, Size: 1954 bytes --]

On Sat, Dec 13, 2014 at 04:28:50PM +0100, Ludovic Courtès wrote:
>宋文武 <iyzsong@gmail.com> skribis:
>
>> And currently ratposion and windowmaker are hardcoded,
>> how about make sessions configurable?
>> Like:
>>     (define %default-sessions
>>       `(("windowmaker" . #~(execl #$windowmaker "/bin/wmaker"))
>>         ("ratposion" . #~(execl #$ratposion "/bin/ratposion"))))
>>
>>     (define* (slim-service #:key (sessions %default-sessions)
>>     ...
>
>I agree that something along these lines is needed.
>
>However, these session types are currently unused.  SLiM is supposed to
>support session choice from the log-in screen, but for that, a
>‘session_dir’ key needs to be added to slim.cfg (in xorg.scm).
>
>The ‘session_dir’ value should be the name of a directory containing
>entries describing the various session types available, in a format that
>SLiM understands.

I'm not able to understand how the proposed solution should work and
it's a bit unclear to me what is needed to do to add new session -
enlightenment in my case.

Is there reason, why xsession files should not be stored within
profile? I can hardly imagine login manager with session selection to
be anything else than system wide service and in that case path like
/run/current-system/profile/share/xsessions sounds like proper place.

In any case, it would be helpful if packager could just ignore this
architectural decisions and could use some function which returns
desired target directory for xsession files (maybe defining as public
xsessions-directory from gnu/services/xorg.scm?) or even take xsession
file name as argument and do whatever is needed.

And one additional question - if xsession file can be part of multiple
profiles, I can't install or create symlink to target directory during
the build, but it needs to be done on adding to profile - is there any
hook for that?

TIA for answers,

S_W

[-- Attachment #2: Type: application/pgp-signature, Size: 181 bytes --]