Date: Wed, 13 Jan 2016 08:35:14 -0800 From: Matthew Ahrens <matt@mahrens.org> To: Warner Losh <imp@bsdimp.com> Cc: Allan Jude <allanjude@freebsd.org>, Slawa Olhovchenkov <slw@zxy.spb.ru>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org Subject: Re: svn commit: r293776 - stable/10/usr.sbin/fstyp Message-ID: <CAKUb7itMyoWwjqgfgRjwB_tdo%2Bpp-uTrNw2UNpGGEm=wS2-FmA@mail.gmail.com> In-Reply-To: <8694B9BA-0F48-4D1E-9974-2C095FF519A8@bsdimp.com> References: <201601121638.u0CGc9cJ076217@repo.freebsd.org> <20160113131858.GN70867@zxy.spb.ru> <56967AA1.3050201@freebsd.org> <8694B9BA-0F48-4D1E-9974-2C095FF519A8@bsdimp.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jan 13, 2016 at 8:32 AM, Warner Losh <imp@bsdimp.com> wrote: > > > On Jan 13, 2016, at 9:26 AM, Allan Jude <allanjude@FreeBSD.org> wrote: > > > > On 2016-01-13 08:18, Slawa Olhovchenkov wrote: > >> On Tue, Jan 12, 2016 at 04:38:09PM +0000, Allan Jude wrote: > >> > >>> Author: allanjude > >>> Date: Tue Jan 12 16:38:09 2016 > >>> New Revision: 293776 > >>> URL: https://svnweb.freebsd.org/changeset/base/293776 > >>> > >>> Log: > >>> MFC: r284589 > >>> Add the ability to detect ZFS and GELI encrypted file systems to > fstyp(8) > >>> > >>> MFC: r284644 > >>> Fix GCC Warnings > >>> > >>> MFC: r284728 > >>> Only build ZFS support in absense of WITHOUT_ZFS > >> > >> What purpose for switch off ZFS support in case WITHOUT_ZFS? > >> > > > > If the user builds FreeBSD WITHOUT_CDDL or WITHOUT_ZFS, then they are > > requesting that we not build the ZFS code, so building it anyway would > > be against their wishes. The most likely reason they would request this > > is for license reasons. > > For x86, that may be true. For arm and mips platforms, ZFS is too much of= a > memory pig-dog to run on much of the kit we run on in that space. Turning > it > off is part of removing the type 2 muda from the system. > > In any event, the double negative is true. Don=E2=80=99t build ZFS when i= ts > absence has > been requested with WITHOUT_ZFS is correct. > Some of the confusion may be because this is not building ZFS - just support for detecting ZFS pools in fstyp. Which would be reasonable to do even if you are not building support for ZFS. However, the fstyp ZFS code relies on libzfs and libnvpair which are CDDL licensed. So if you did not build ZFS (or CDDL), the fstyp code could not work. --matt
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAKUb7itMyoWwjqgfgRjwB_tdo%2Bpp-uTrNw2UNpGGEm=wS2-FmA>