unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: "Ludovic Courtès" <ludo@gnu.org>
To: dan <i@dan.games>, Mathieu Othacehe <othacehe@gnu.org>
Cc: Tobias Geerinckx-Rice <me@tobias.gr>, 59132@debbugs.gnu.org
Subject: [bug#59132] [PATCH] system: images: wsl2: Create $XDG_RUNTIME_DIR on first login.
Date: Fri, 18 Nov 2022 15:00:49 +0100	[thread overview]
Message-ID: <87k03sbbvy.fsf_-_@gnu.org> (raw)
In-Reply-To: <96da2673-8ad4-cd7f-cf3a-ce0cef3ca313@dan.games> (dan's message of "Wed, 9 Nov 2022 21:48:50 +0800")

Hi Mathieu,

Could you comment on this patch dan submitted?

  https://issues.guix.gnu.org/59132

Thanks in advance.  :-)

Ludo’.

dan <i@dan.games> skribis:

> Hi Tobias,
>
> On 11/9/2022 8:26 PM, Tobias Geerinckx-Rice wrote:
>> Thanks!  Why is this needed?
>> On 2022-11-08 21:05, dan wrote:
>>> * gnu/system/images/wsl2.scm (wsl-boot-program): Create $XDG_RUNTIME_DIR
>> ^^^^^^^^^^^^^^^^
>> There's no reference to this in the actual patch.
> In my understanding, user level shepherd services need XDG_RUNTIME_DIR
> to run, and shepherd create a socket under
> XDG_RUNTIME_DIR/shepherd/. I'm not sure how WSL handle user login, but
> what we usually do is just type `wsl' in powershell, and it
> automatically logged into the default account, without prompting for
> password input.  Thus, relying ot elogind or greetd doesn't really
> help, it's better for us to manually create the directory.
>
> There is also a warning on login, and I think it's from shepherd:
>> warning: XDG_RUNTIME_DIR doesn't exists, on-first-login script won't
>> execute anything.  You can check if xdg runtime directory exists,
>> XDG_RUNTIME_DIR variable is set to appropriate value and manually
>> execute the script by running '$HOME/.guix-home/on-first-login
> Anyway... I wasn't fully sure about if it's the right thing to do,
> since I'm not familiar enough with both Linux and WSL. But at least
> this makes user level services usable.  If this is the right thing to
> do, I'll update the patch with explanation included in the commit
> message.
>
> Would like to hear feedback from someone authoritative!




  reply	other threads:[~2022-11-18 14:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-08 20:05 [bug#59132] [PATCH] system: images: wsl2: Create $XDG_RUNTIME_DIR on first login dan
2022-11-08 20:11 ` ( via Guix-patches via
2022-11-08 20:23   ` dan
2022-11-08 20:27     ` ( via Guix-patches via
2022-11-09  4:26       ` dan
2022-11-09  7:24       ` dan
2022-11-08 20:27     ` ( via Guix-patches via
2022-11-09 12:26 ` Tobias Geerinckx-Rice via Guix-patches via
2022-11-09 13:48   ` dan
2022-11-18 14:00     ` Ludovic Courtès [this message]
2022-12-26  8:36 ` bug#59132: " Mathieu Othacehe

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87k03sbbvy.fsf_-_@gnu.org \
    --to=ludo@gnu.org \
    --cc=59132@debbugs.gnu.org \
    --cc=i@dan.games \
    --cc=me@tobias.gr \
    --cc=othacehe@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).