[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#26341: [PATCH 1/5] build: syscalls: Add reboot.
From: |
Ludovic Courtès |
Subject: |
bug#26341: [PATCH 1/5] build: syscalls: Add reboot. |
Date: |
Mon, 10 Apr 2017 15:41:51 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Hi,
Mathieu Othacehe <address@hidden> skribis:
>> The problem is that we cannot #:replace unconditionally (when not using
>> the patched Guile, there’s nothing to replace¹).
>
> Yes I was relying on #:replace ability to just export the symbol if
> there's nothing to replace (as explained in your note¹).
>
>>
>> WDYT?
>
> Anyway, your macro seems a nicer way than dealing with export/replace.
> I gave it a try and everything still seems warning free.
>
> My only concern is that it won't be obvious for people who want to use
> reboot or mount or any other syscall that they have to use (guix build
> syscalls) because the #:export list won't contain those syscalls.
>
> Maybe a big explanation on top of the file would be enough ?
Yes, I think it would be enough.
Also, the initrd is the only context where one can omit (use-modules
(guix build syscalls)) so I think people will choose to use it by
default.
Thanks,
Ludo’.
bug#26341: [PATCH 1/2] build: syscalls: Allow mount and umount use from static Guile., Mathieu Othacehe, 2017/04/06
bug#26341: [PATCH 0/5] Fix warnings related to syscalls in static Guile., Mathieu Othacehe, 2017/04/08
bug#26341: [PATCH 3/5] build: syscalls: Add mount and umount to #:replace list., Mathieu Othacehe, 2017/04/08
bug#26341: [PATCH 4/5] build: syscalls: Add load-linux-module., Mathieu Othacehe, 2017/04/08
bug#26341: [PATCH 5/5] build: Fix compilation warnings., Mathieu Othacehe, 2017/04/08
bug#26341: [PATCH 2/5] build: syscalls: Allow use to network-interface syscalls independently of calling context., Mathieu Othacehe, 2017/04/08
bug#26341: [PATCH 1/5] build: syscalls: Add reboot., Mathieu Othacehe, 2017/04/10