seatd-service-type

seatd-service-type

[Josua Stingelin]

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

Hi Guix,

I'm running a wayland based system and use the sway window manager.
Sway supports the seat daemon (seatd) as an alternative to elogind.

Are you interested in a patch to provide the seatd as a service?

If yes please give me feedback about what needs to change in the following
definitions to be accepted upstream. I suppose a git-patch per mail would be
the way to contribute?


  (define seatd-shepherd-service
    (match-lambda
      ('()
       (list (shepherd-service
               (provision '(seatd))
               (documentation "Run the seat deamon (seatd)")
               (requirement '(user-processes))
               (start #~(make-forkexec-constructor
                          (list #$(file-append seatd "/bin/seatd")
                                "-g" "users")))
               (stop #~(make-kill-destructor)))))))
  (define seatd-service-type
    (service-type
      (name 'seatd)
      (extensions
        (list (service-extension shepherd-root-service-type
                                 seatd-shepherd-service)))
      (default-value '())
      (description "Run the seat daemon (seatd))")))


Kind Regards,
Josua a.k.a Joshua/Josh

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: seatd-service-type

[Josselin Poiret]

Hello Josua,
Josua Stingelin <josuast@hotmail.com> writes:

> Hi Guix,
>
> I'm running a wayland based system and use the sway window manager.
> Sway supports the seat daemon (seatd) as an alternative to elogind.
>
> Are you interested in a patch to provide the seatd as a service?

There already exists a patch-set to add seatd and greetd to Guix [1],
maybe you could try testing the patch-set and see if it works for you?

[1] https://issues.guix.gnu.org/49969

Best,
-- 
Josselin Poiret

Re: seatd-service-type

[Declan]

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


I am running patch-set from that link on my machine and it's working
without any modifications. I'd like to see it got merged.

FYI, I just switch to Guix recently. Also I am not very familar this
email workflow yet.

> There already exists a patch-set to add seatd and greetd to Guix [1],
> maybe you could try testing the patch-set and see if it works for you?
>
> [1] https://issues.guix.gnu.org/49969
>
> Best,
--

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 515 bytes --]

Re: seatd-service-type

[Josua Stingelin]

On Mon, Feb 14, 2022 at 05:08:14PM +0800, Declan wrote:
> 
> I am running patch-set from that link on my machine and it's working
> without any modifications. I'd like to see it got merged.
> 
> FYI, I just switch to Guix recently. Also I am not very familar this
> email workflow yet.
> 
> > There already exists a patch-set to add seatd and greetd to Guix [1],
> > maybe you could try testing the patch-set and see if it works for you?
> >
> > [1] https://issues.guix.gnu.org/49969
> >
> > Best,

Somehow I missed the mail above. Must have deleted it accidentally. I'm willing
to test the patches.

Is there a way to download the entire patch set at once or will I have to
download them one-by-one?

Thanks,

Re: seatd-service-type

[Declan Qian]

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

Josua Stingelin <josuast@hotmail.com> writes:

> On Mon, Feb 14, 2022 at 05:08:14PM +0800, Declan wrote:
>>
>> I am running patch-set from that link on my machine and it's working
>> without any modifications. I'd like to see it got merged.
>>
>> FYI, I just switch to Guix recently. Also I am not very familar this
>> email workflow yet.
>>
>> > There already exists a patch-set to add seatd and greetd to Guix [1],
>> > maybe you could try testing the patch-set and see if it works for you?
>> >
>> > [1] https://issues.guix.gnu.org/49969
>> >
>> > Best,
>
> Somehow I missed the mail above. Must have deleted it accidentally. I'm willing
> to test the patches.
>
> Is there a way to download the entire patch set at once or will I have to
> download them one-by-one?
>
> Thanks,
>
>

Hmm... I am reading guix.patches from news.gmane.io using Gnus.
Patch-set can be easily downloaded by selecting them and then invoked
gnus keybinding O m. Gnus can save them all in one file if you choose
the same filename while saving. For https://issues.guix.gnu.org/49969. I
downloaded the latest v8 [1/7 -- 7/7] (https://i.imgur.com/TSRZOtx.png)

Currently I pushed the those applied changes to this url
https://gitlab.com/declanqian/guix/-/commits/test_greetd/
so it can be easily rebased on master later.

Here is my unpolished Guix system config running seatd/greetd
https://gist.github.com/declanqian/79873926d3db23230b11cd9997a8c6f8

However, I didn't see the error from here
https://issues.guix.gnu.org/49969#89. Quite the contrary, `./bootstrap
./configure ./make` worked just fine on my machine. No idea whether I
did something wrong.

I am using `guix shell -D guix --pure` from guix info manul.
Also build greetd using `./pre-inst-env guix build greetd` to be sure.
Here's the output:
 /gnu/store/fdjk9mmif1038g9qjxachfmdmhs8p0rr-greetd-0.8.0

Let me know if you want further information.

-- 
~/path2signature

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 515 bytes --]

Re: seatd-service-type

[Josua Stingelin]

> Here is my unpolished Guix system config running seatd/greetd
> https://gist.github.com/declanqian/79873926d3db23230b11cd9997a8c6f8

Thanks for the inspiration!

> However, I didn't see the error from here
> https://issues.guix.gnu.org/49969#89. Quite the contrary, `./bootstrap
> ./configure ./make` worked just fine on my machine. No idea whether I
> did something wrong.

I'm also running into trouble. Specifically with patch 5 of 7. The not so
helpful log says:

  (repl-version 0 1 1)
  Generating package cache for '/gnu/store/f51aqcng42a9v9cd50i0a4c8ypyyrbz0-profile'...
  (exception unbound-variable (value #f) (value "Unbound variable: ~S") (value (seatd)) (value #f))


The only patch I'm actually interested in is [PATCH v8 6/7], witch does work on
my system.

Any chance we can cherry pick? :)


If anyone is willing to work with me on this I could provide the patches so
that they can be applied to master without conflicts and test if they can be
built.

That would allow us to merge the patches that can stand on themselves and
later we can update seatd in a separate step.

What do you think?