[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#41541: [PATCH 8/8] system: Add `hurd-activation'.
From: |
Jan Nieuwenhuizen |
Subject: |
bug#41541: [PATCH 8/8] system: Add `hurd-activation'. |
Date: |
Sat, 06 Jun 2020 10:54:38 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Mathieu Othacehe writes:
Hello Mathieu,
>> (define (activation-script gexps)
>> "Return the system's activation script, which evaluates GEXPS."
>> + (program-file "activate.scm" (if (hurd-target?)
>> + (hurd-activation-script gexps)
>> + (gnu/linux-activation-script gexps))))
>
> "hurd-target?" is using %current-target-system. It would be safer to use
> let-system here I guess.
Okay, changed to
(let-system (system target)
(program-file "activate.scm" (if (hurd-triplet? (or target system))
(hurd-activation-script gexps)
(gnu/linux-activation-script gexps)))))
> I think Ludo will have some more comments, but let me say again that you
> are doing a great job here, many thanks!
Thank you! -- luckily I get some help ;)
Janneke
--
Jan Nieuwenhuizen <janneke@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | AvatarĀ® http://AvatarAcademy.com
- bug#41541: [PATCH 5/8] system: Use 'hurd' package in label., (continued)
- bug#41541: [PATCH 5/8] system: Use 'hurd' package in label., Jan (janneke) Nieuwenhuizen, 2020/06/04
- bug#41541: [PATCH 2/8] bootloader: Extend `<menu-entry>' for multiboot., Jan (janneke) Nieuwenhuizen, 2020/06/04
- bug#41541: [PATCH 7/8] services: hurd: Add `hurd-etc-service'., Jan (janneke) Nieuwenhuizen, 2020/06/04
- bug#41541: [PATCH 4/8] bootloader: grub: Add support for multiboot., Jan (janneke) Nieuwenhuizen, 2020/06/04
- bug#41541: [PATCH 6/8] system: examples: Add bare-hurd.tmpl., Jan (janneke) Nieuwenhuizen, 2020/06/04
- bug#41541: [PATCH 8/8] system: Add `hurd-activation'., Jan (janneke) Nieuwenhuizen, 2020/06/04