Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 06 Sep 2021 11:31:45 +0000
From:      bugzilla-noreply@freebsd.org
To:        ports-bugs@FreeBSD.org
Subject:   [Bug 258294] security/suricata: Respect CC
Message-ID:  <bug-258294-7788-VjKRinXtbQ@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-258294-7788@https.bugs.freebsd.org/bugzilla/>
References:  <bug-258294-7788@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D258294

--- Comment #2 from Evgeniy Khramtsov <evgeniy@khramtsov.org> ---
(In reply to Kubilay Kocak from comment #1)

> Are environment/user CFLAGS and other flags (LDFLAGS, etc) honoured?

Yes. Checked via build log:

[...]
 #### /usr/local/etc/poudriere.d/130-amd64-make.conf ####
-CFLAGS+=3D-O0
-LDFLAGS+=3D-Wl,--as-needed
+CFLAGS+=3D-O3
+LDFLAGS+=3D-Wl,--no-as-needed
[...]
-  CFLAGS                                   -O2 -pipe -O0=20
-fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing=20
-DOS_FREEBSD -std=3Dc11 -I${srcdir}/../rust/gen -I${srcdir}/../rust/dist
+  CFLAGS                                   -O2 -pipe -O3=20
-fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing=20
-DOS_FREEBSD -std=3Dc11 -I${srcdir}/../rust/gen -I${srcdir}/../rust/dist
[...]
-/bin/sh ../../libtool  --tag=3DCC   --mode=3Dlink cc -I../.. -D_GNU_SOURCE=
 -g
-Wall -Wextra -std=3Dgnu99 -pedantic -Wextra -Wno-missing-field-initializers
-Wshadow -Wpointer-arith -Wstrict-prototypes -Wmissing-prototypes
-Wno-unused-parameter -O2 -pipe -O0  -fstack-protector-strong -isystem
/usr/local/include -fno-strict-aliasing  -O2 -Wstrict-overflow=3D1
-fstack-protector -D_FORTIFY_SOURCE=3D2 -Wformat -Wformat-security -fPIC=20
-Wl,--as-needed -fstack-protector-strong  -L/usr/local/lib -o liblzma-c.la=
=20=20=20
LzFind.lo LzmaDec.lo  -lz -L/usr/local/lib
+/bin/sh ../../libtool  --tag=3DCC   --mode=3Dlink cc -I../.. -D_GNU_SOURCE=
 -g
-Wall -Wextra -std=3Dgnu99 -pedantic -Wextra -Wno-missing-field-initializers
-Wshadow -Wpointer-arith -Wstrict-prototypes -Wmissing-prototypes
-Wno-unused-parameter -O2 -pipe -O3  -fstack-protector-strong -isystem
/usr/local/include -fno-strict-aliasing  -O2 -Wstrict-overflow=3D1
-fstack-protector -D_FORTIFY_SOURCE=3D2 -Wformat -Wformat-security -fPIC=20
-Wl,--no-as-needed -fstack-protector-strong  -L/usr/local/lib -o liblzma-c.=
la=20=20
 LzFind.lo LzmaDec.lo  -lz -L/usr/local/lib
[...]

I didn't expose rust related build log via cargo.mk verbosity due to
unfamiliarity with cargo, though the difference can be observed via build s=
peed
(-O0 vs -O3):

[...]
    Compiling suricata v6.0.3
(/wrkdirs/usr/ports/security/suricata/work/suricata-6.0.3/rust)
-    Finished release [optimized + debuginfo] target(s) in 39.10s
+    Finished release [optimized + debuginfo] target(s) in 36.99s
[...]

> needs-qa

Can you clarify what QA does this change need? This port does not have
consumers
and I already did QA on Tier 1 platforms and made sure that the package
contents did not change.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-258294-7788-VjKRinXtbQ>