[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 6/6] gnu: Add grub-efi.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 6/6] gnu: Add grub-efi. |
Date: |
Sun, 06 Nov 2016 23:00:16 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Marius Bakke <address@hidden> skribis:
> Leo Famulari <address@hidden> writes:
>
>> On Sat, Nov 05, 2016 at 12:55:11PM +0000, Marius Bakke wrote:
>>> * gnu/packages/grub.scm (grub-efi): New variable.
>>> ---
>>> gnu/packages/grub.scm | 21 +++++++++++++++++++++
>>> 1 file changed, 21 insertions(+)
>>>
>>> diff --git a/gnu/packages/grub.scm b/gnu/packages/grub.scm
>>> index ffce1bf..e06216f 100644
>>> --- a/gnu/packages/grub.scm
>>> +++ b/gnu/packages/grub.scm
>>> @@ -157,3 +157,24 @@ on the same computer; upon booting the computer, the
>>> user is presented with a
>>> menu to select one of the installed operating systems.")
>>> (license gpl3+)
>>> (properties '((cpe-name . "grub2")))))
>>> +
>>> +(define-public grub-efi
>>> + (package
>>> + (inherit grub)
>>> + (name "grub-efi")
>>> + (synopsis (string-append (package-synopsis grub) " (UEFI version)"))
>>> + (inputs
>>> + `(("efibootmgr" ,efibootmgr)
>>> + ,@(package-inputs grub)))
>>> + (arguments
>>> + #:tests? #f ; FIXME: 40 failures, 24 skipped, 17 passed.
>>
>> Does this package work for you?
>
> Oops, not sure what went wrong when fixing up this package for
> publishing. Updated patch attached.
I think Leo was asking whether you could get a bootable system with it.
:-)
> From 940c03c7dcddec019e27f6eb1470aeab4db57799 Mon Sep 17 00:00:00 2001
> From: Marius Bakke <address@hidden>
> Date: Thu, 20 Oct 2016 17:26:52 +0100
> Subject: [PATCH] gnu: Add grub-efi.
>
> * gnu/packages/grub.scm (grub-efi): New variable.
[...]
> + (name "grub-efi")
> + (synopsis (string-append (package-synopsis grub) " (UEFI version)"))
Please use a literal string for ‘synopsis’; use of ‘string-append’ like
this prevents i18n.
> + `(#:tests? #f ; FIXME: 40 failures, 24 skipped, 17 passed.
It would be good to investigate, especially if the tests pass in the
‘grub’ package.
Also, what’s the rationale for making ‘grub-efi’ separate instead of
incorporating the changes in ‘grub’ proper? Are there issues around the
portability of ‘efibootmgr’, or an increased closure size?
Thanks for working on it!
Ludo’.
- Re: [PATCH 3/6] file-systems: Suppress fsck status completion bar., (continued)
[PATCH 4/6] gnu: Add efivar., Marius Bakke, 2016/11/05
[PATCH 5/6] gnu: Add efibootmgr., Marius Bakke, 2016/11/05
[PATCH 6/6] gnu: Add grub-efi., Marius Bakke, 2016/11/05