unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* seatd-service-type
@ 2022-02-11 11:59 Josua Stingelin
  2022-02-11 13:10 ` seatd-service-type Josselin Poiret
  0 siblings, 1 reply; 6+ messages in thread
From: Josua Stingelin @ 2022-02-11 11:59 UTC (permalink / raw)
  To: guix-devel

[-- 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 --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: seatd-service-type
  2022-02-11 11:59 seatd-service-type Josua Stingelin
@ 2022-02-11 13:10 ` Josselin Poiret
  2022-02-14  9:08   ` seatd-service-type Declan
  0 siblings, 1 reply; 6+ messages in thread
From: Josselin Poiret @ 2022-02-11 13:10 UTC (permalink / raw)
  To: Josua Stingelin, guix-devel

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: seatd-service-type
  2022-02-11 13:10 ` seatd-service-type Josselin Poiret
@ 2022-02-14  9:08   ` Declan
  2022-02-16 15:38     ` seatd-service-type Josua Stingelin
  0 siblings, 1 reply; 6+ messages in thread
From: Declan @ 2022-02-14  9:08 UTC (permalink / raw)
  To: guix-devel

[-- 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 --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: seatd-service-type
  2022-02-14  9:08   ` seatd-service-type Declan
@ 2022-02-16 15:38     ` Josua Stingelin
  2022-02-17  5:39       ` seatd-service-type Declan Qian
  0 siblings, 1 reply; 6+ messages in thread
From: Josua Stingelin @ 2022-02-16 15:38 UTC (permalink / raw)
  To: guix-devel

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,


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: seatd-service-type
  2022-02-16 15:38     ` seatd-service-type Josua Stingelin
@ 2022-02-17  5:39       ` Declan Qian
  2022-02-19 17:34         ` seatd-service-type Josua Stingelin
  0 siblings, 1 reply; 6+ messages in thread
From: Declan Qian @ 2022-02-17  5:39 UTC (permalink / raw)
  To: guix-devel

[-- 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 --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: seatd-service-type
  2022-02-17  5:39       ` seatd-service-type Declan Qian
@ 2022-02-19 17:34         ` Josua Stingelin
  0 siblings, 0 replies; 6+ messages in thread
From: Josua Stingelin @ 2022-02-19 17:34 UTC (permalink / raw)
  To: guix-devel

> 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?


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-02-19 17:56 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-02-11 11:59 seatd-service-type Josua Stingelin
2022-02-11 13:10 ` seatd-service-type Josselin Poiret
2022-02-14  9:08   ` seatd-service-type Declan
2022-02-16 15:38     ` seatd-service-type Josua Stingelin
2022-02-17  5:39       ` seatd-service-type Declan Qian
2022-02-19 17:34         ` seatd-service-type Josua Stingelin

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).