[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#53912] [PATCH 3/5] system: image: Add wsl-tarball support.
From: |
Maxime Devos |
Subject: |
[bug#53912] [PATCH 3/5] system: image: Add wsl-tarball support. |
Date: |
Tue, 15 Feb 2022 17:31:46 +0100 |
User-agent: |
Evolution 3.38.3-1 |
Alex Griffin via Guix-patches via schreef op do 10-02-2022 om 06:27
[+0000]:
> (with-directory-excursion image-root
> + (when #$wsl?
> + ;; WSL requires /bin/sh. Will be overwritten by
> system
> + ;; activation.
> + (symlink #$root-shell "./bin/sh")
> +
> + ;; WSL requires /bin/mount to access the host
> fs.
> + (symlink #+(file-append util-linux "/bin/mount")
> + "./bin/mount"))
I would change this to
(with-directory-excursion image-root
#$@(if wsl?
;; Comment ...
#~((symlink ...)
(symlink ...)
#~())
other things ...)
such that the tarball could be built without building util-linux.
Also, (symlink #+(file-append util-linux ...) "./bin/mount") seems
wrong here when cross-compiling, wouldn't a 'mount' compiled for --
target be required instead of a 'mount' compiled for --system when
cross-compiling? Do you mean #$(file-append ...) instead?
Greetings,
Maxime.
signature.asc
Description: This is a digitally signed message part
- [bug#53912] [PATCH 0/5] WIP Add WSL support., Alex Griffin, 2022/02/10
- [bug#53912] [PATCH 1/5] scripts: Fix typo., Alex Griffin, 2022/02/10
- [bug#53912] [PATCH 2/5] system: image: Add tarball support., Alex Griffin, 2022/02/10
- [bug#53912] [PATCH 3/5] system: image: Add wsl-tarball support., Alex Griffin, 2022/02/10
- [bug#53912] [PATCH 4/5] system: Add wsl module., Alex Griffin, 2022/02/10
- [bug#53912] [PATCH 5/5] image: Add wsl image., Alex Griffin, 2022/02/10