svn commit: r346081 - head/cddl/usr.sbin/zfsd

[Enji Cooper]

> On Apr 10, 2019, at 08:55, Konstantin Belousov <kostikbel at gmail.com> wrote:
> 
>> On Wed, Apr 10, 2019 at 01:42:37PM +0000, Edward Tomasz Napierala wrote:
>> Author: trasz
>> Date: Wed Apr 10 13:42:37 2019
>> New Revision: 346081
>> URL: https://svnweb.freebsd.org/changeset/base/346081
>> 
>> Log:
>>  Make zfsd(8) build obey CFLAGS.
>> 
>>  Reviewed by:    imp
>>  Obtained from:    CheriBSD
>>  MFC after:    2 weeks
>>  Sponsored by:    DARPA, AFRL
>>  Differential Revision:    https://reviews.freebsd.org/D19865
>> 
>> Modified:
>>  head/cddl/usr.sbin/zfsd/Makefile.common
>> 
>> Modified: head/cddl/usr.sbin/zfsd/Makefile.common
>> ==============================================================================
>> --- head/cddl/usr.sbin/zfsd/Makefile.common    Wed Apr 10 13:41:34 2019    (r346080)
>> +++ head/cddl/usr.sbin/zfsd/Makefile.common    Wed Apr 10 13:42:37 2019    (r346081)
>> @@ -30,7 +30,7 @@ INCFLAGS+= -I${SRCTOP}/sys/cddl/contrib/opensolaris/ut
>> INCFLAGS+= -I${SRCTOP}/sys/cddl/contrib/opensolaris/uts/common/fs/zfs
>> INCFLAGS+= -I${SRCTOP}/cddl/usr.sbin
>> 
>> -CFLAGS= -g -DNEED_SOLARIS_BOOLEAN ${INCFLAGS}
>> +CFLAGS+= -g -DNEED_SOLARIS_BOOLEAN ${INCFLAGS}
> Why the flags unconditionally set -g ?

It shouldn’t be explicitly set. That’s what DEBUG_FLAGS is for.

-Enji