all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Tomáš Čech" <sleep_walker@gnu.org>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] linux-initrd: Introduce way to add more packages to initrd.
Date: Tue, 2 Aug 2016 20:03:28 +0200	[thread overview]
Message-ID: <20160802180327.GJ1062@venom> (raw)
In-Reply-To: <871t27wcux.fsf@gnu.org>

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

On Tue, Aug 02, 2016 at 02:53:58PM +0200, Ludovic Courtès wrote:
>Hello,
>
>Tomáš Čech <sleep_walker@gnu.org> skribis:
>>>All the packages used by the initrd are automatically part of the
>>>initrd.  The proposed patch would allow people to add unused packages to
>>>the initrd.
>>
>> It is for the packages which you may want to use interactivelly in
>> case of failure or for some extra initrd hacking you may not want/be
>> able to write in Guile.
>>
>> Features like
>> - extra authentication
>> - full disk encryption
>> - root on NFS
>> - LVM :)
>> - ...
>
>OK but if you need these packages, for instance because you have a LUKS
>boot device, they’ll already be in the initrd.  No need to manually list
>them in #:extra-packages.

That is correct only for cases which are already handle. Last time I
booted from LVM it didn't work. IOW your answer is valid for user but
not for (potential) contributor.

This can help me (and possibly others) to write it.

Otherwise it is hard to write for Guix in Guile because it is hard to
write for Guix in Guile. My solution is not perfect but helps.

>>>Could you explain how/when this would be used?  Maybe as commands for
>>>use by Bournish when it’s used as a rescue shell?
>>
>> I agree that it is more for debugging and to balance my inability to
>> express it in Guile but it lowers the barrier a bit.
>>
>> Bournish is too young to rely on it. I miss pipes, accessing files in
>> different directories or `ls' with wildcards. I can put in minimal
>> static busybox which is more than sufficient for rescue, problem
>> analysis or even data recovery.
>>
>> I like the idea of Bournish but I'd rather have an alternative
>> until it is more capable.
>
>I agree, but hopefully, you don’t run into Bournish too often?

Of course I did until I used exactly this patch and was able to solve
my problem.

>I guess my main concern (again, as a lazy maintainer) is the cost of
>turning ‘base-initrd’ into a kitchen sink, as discussed in the other
>thread about #:extra-modules.
>
>I would prefer to provide simple tools that people can build upon, like
>‘expression->initrd’ or the ‘raw-initrd’ procedure I proposed, than
>trying to come up with a one-size-fits-all procedure with many
>parameters.

I know that your approach is cleaner and nicer for any Guile
programmer. On the other hand it can be harder to use for the rest of
the world.

Kitchen sink issue is matter of configuration and we're all adults
here, we can make our decisions.

I'm not ignoring your proposal but

- I haven't seen a way how it could help me in previous thread
- I'm afraid I can't deliver you such solution until I improve my
  Guile skills.

I'm fine with keeping my patches out of tree (they can be found in
devel mailing list if anyone is interested in it), I just don't think
that I'm the only one facing this sort of problems.

I'm sorry I didn't provide you more constructive answer.

S_W

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

  reply	other threads:[~2016-08-02 18:03 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-25  7:46 extending initrd Tomáš Čech
2016-07-25 20:10 ` [PATCH] linux-initrd: Introduce way to add more packages to initrd Tomáš Čech
2016-08-01 10:39   ` Tomáš Čech
2016-08-01 15:44     ` Andy Wingo
2016-08-02  7:43       ` extra-packages - second wave Tomáš Čech
2016-08-02  7:43         ` [PATCH 1/2] linux-initrd: Introduce way to add more packages to initrd Tomáš Čech
2016-08-02  7:43         ` [PATCH 2/2] doc: Document extra-packages argument of base-initrd Tomáš Čech
2016-08-02  8:19           ` Vincent Legoll
2016-08-01 16:09     ` [PATCH] linux-initrd: Introduce way to add more packages to initrd Ludovic Courtès
2016-08-01 18:26       ` Tomáš Čech
2016-08-02 12:53         ` Ludovic Courtès
2016-08-02 18:03           ` Tomáš Čech [this message]
2016-07-25 21:48 ` extending initrd Ludovic Courtès

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

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

  git send-email \
    --in-reply-to=20160802180327.GJ1062@venom \
    --to=sleep_walker@gnu.org \
    --cc=guix-devel@gnu.org \
    --cc=ludo@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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.