Date: Fri, 15 Sep 2017 12:33:39 +0200 From: =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= <des@des.no> To: Steve Wills <swills@FreeBSD.org> Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r449798 - head/net-mgmt/collectd5 Message-ID: <8660cki73w.fsf@desk.des.no> In-Reply-To: <305b87d4-297e-e569-7011-9c322984206d@FreeBSD.org> (Steve Wills's message of "Thu, 14 Sep 2017 18:12:15 -0400") References: <201709131455.v8DEtjVq069144@repo.freebsd.org> <305b87d4-297e-e569-7011-9c322984206d@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Steve Wills <swills@FreeBSD.org> writes: > I'm happy to see more options enabled by default in this port, thanks > for that. We do have a policy that packaged binaries should be > stripped. Enabling the DEBUG option by default would seem to be at > odds with that policy to me. Could you please turn that one off by > default? # file /usr/local/sbin/collectd /usr/local/sbin/collectd: ELF 64-bit LSB executable, x86-64, version 1 (Fre= eBSD), dynamically linked, interpreter /libexec/ld-elf.so.1, for FreeBSD 11= .0 (1100122), FreeBSD-style, stripped The DEBUG option enables log messages which are indispensable when troubleshooting a collectd installation (or rather, makes an additional log level available). I didn't realize that it also disables stripping. If that is the case, it needs to be split in two, or maybe the relevant configure argument should simply be hardcoded. DES --=20 Dag-Erling Sm=C3=B8rgrav - des@des.no
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?8660cki73w.fsf>