[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v3 0/2] btrfs: Add zstd support to grub btrfs

From: Paul Menzel
Subject: Re: [PATCH v3 0/2] btrfs: Add zstd support to grub btrfs
Date: Thu, 11 Oct 2018 09:56:56 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1

Dear Nick,

Am 10.10.2018 um 22:28 schrieb Nick Terrell:

On Oct 10, 2018, at 12:34 AM, Paul Menzel wrote:

Sorry for being ignorant, but you explain, why the library needs to
be imported and it is not enough to use that library as an external

Importing the library means, it has to be maintained in the GRUB
repository, which will result in some maintenance burden.

I've imported zstd because thats the way the rest of the
decompressors are imported.

We could potentially use libzstd as an external dependency, since its
only dependency is libc and GRUB provides the definitions of the libc
functions that zstd needs to decompress (memcpy, and memmove). Theres
some other stuff in the library that requires libc functionality that
GRUB doesn't provide, but that isn't used during decompression. We
strip those files out in the import.

Thank you for the explanation.

Let me know if you want me to switch to an external dependency.

I cannot make that decision. The main developers and the distribution
folks should comment on that.

Kind regards,


reply via email to

[Prev in Thread] Current Thread [Next in Thread]