[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#57590] [PATCH] Adding libldm: Manager for Windows dynamic disks inc
From: |
Ludovic Courtès |
Subject: |
[bug#57590] [PATCH] Adding libldm: Manager for Windows dynamic disks including software RAID. It creates device mapper entries for dynamic disks allowing them to be mounted. |
Date: |
Tue, 18 Oct 2022 17:20:15 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux) |
Hi,
Lukasz Olszewski <dev@lukaszolszewski.info> skribis:
>> > +++ b/gnu/packages/libldm.scm
>> > @@ -0,0 +1,70 @@
>> > +(define-module (gnu packages libldm)
>>
>> We’ll need the license/copyright header as you noted.
>>
>
> I've posted a later patch that included those, but it was posted as a patch
> on top of a patch so perhaps it wasn't well visible. I'll integrate it in
> the next version.
I did see it (thanks!). It would be great though if you could send a
single “v2” patch that includes everything.
>> > + #:parallel-build? #t
>>
>> This is unnecessary.
>>
>
> Are parallel builds enabled by default?
Yes, that’s why.
>> > + (replace 'bootstrap
>> > + (lambda _
>> > + (invoke "autoreconf" "-fiv"))))))
>>
>> Is it necessary? The default ‘bootstrap’ phase does something similar.
>>
>
> I've copied this phase from another package. If I remember correctly the
> configure phase failed without. I'll have to test again to check.
Yes please.
Thanks,
Ludo’.