Created attachment 227692 [details] v1 (use "git am") mail/neomutt needs to respect CC per https://docs.freebsd.org/en/books/porters-handbook/porting-dads/#dads-cc Reproduce by deleting 'cc' from jail and overriding CC to another value. Package contents did not change: 11.4/amd64: https://codeberg.org/ei/misc/commit/eb085f3ba60 11.4/i386: https://codeberg.org/ei/misc/commit/4ad3ffef396 12.2/amd64: https://codeberg.org/ei/misc/commit/b16c45d9963 12.2/i386: https://codeberg.org/ei/misc/commit/e1f88aff135 13.0/amd64: https://codeberg.org/ei/misc/commit/6629ac89ec6 Log diff with the patch applied: 11.4/amd64: https://codeberg.org/ei/misc/commit/3b4159a7192 11.4/i386: https://codeberg.org/ei/misc/commit/c8392642678 12.2/amd64: https://codeberg.org/ei/misc/commit/726ed7004ab 12.2/i386: https://codeberg.org/ei/misc/commit/9c311e55600 13.0/amd64: https://codeberg.org/ei/misc/commit/7a161f0037f Testport logs with the patch applied: 11.4/amd64: https://codeberg.org/ei/misc/commit/8a30ca1a975 11.4/i386: https://codeberg.org/ei/misc/commit/dd7e0c22d3e 12.2/amd64: https://codeberg.org/ei/misc/commit/3284b358fb9 12.2/i386: https://codeberg.org/ei/misc/commit/82572f8dd21 13.0/amd64: https://codeberg.org/ei/misc/commit/6724a3cf900 13.0/aarch64 QA via qemu-user-static is omitted because the dependency 'mail/notmuch' can't configure here, maybe a bug?
Are environment/user CFLAGS and other flags (LDFLAGS, etc) honoured? Honoured meaning 'appended', not just included
(In reply to Kubilay Kocak from comment #1) Yes. Checked via "Summary of build options" and build commands from build log: ---Begin make.conf--- [...] #### /usr/local/etc/poudriere.d/130-amd64-make.conf #### CFLAGS+=-O0 LDFLAGS+=-Wl,--as-needed [...] -CFlags: -O2 -pipe -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing -std=c99 -D_ALL_SOURCE=1 -D_GNU_SOURCE=1 -D__EXTENSIONS__ -I/usr/local/include -DNCURSES_WIDECHAR -I/usr/include -I/usr/include -CFlags: -O2 -pipe -O0 -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing -std=c99 -D_ALL_SOURCE=1 -D_GNU_SOURCE=1 -D__EXTENSIONS__ -I/usr/local/include -DNCURSES_WIDECHAR -I/usr/include -I/usr/include -LDFlags: -fstack-protector-strong -L/usr/local/lib -L/usr/lib -L/usr/lib -lgssapi -lgssapi_krb5 -lheimntlm -lkrb5 -lhx509 -lcom_err -lcrypto -lasn1 -lwind -lheimbase -lroken -lcrypt -pthread +LDFlags: -Wl,--as-needed -fstack-protector-strong -L/usr/local/lib -L/usr/lib -L/usr/lib -lgssapi -lgssapi_krb5 -lheimntlm -lkrb5 -lhx509 -lcom_err -lcrypto -lasn1 -lwind -lheimbase -lroken -lcrypt -pthread > 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. "CC_FOR_BUILD" is also used by many ports, e.g. ports-mgmt/pkg.
Maintainer timeout.
Ping.
ping
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=735d210819c11c4db5fc544106bc24c0ebaa2094 commit 735d210819c11c4db5fc544106bc24c0ebaa2094 Author: Evgeniy Khramtsov <evgeniy@khramtsov.org> AuthorDate: 2021-09-03 12:13:08 +0000 Commit: Baptiste Daroussin <bapt@FreeBSD.org> CommitDate: 2022-01-17 15:49:43 +0000 mail/neomutt: respect CC PR: 258292 mail/neomutt/Makefile | 1 + 1 file changed, 1 insertion(+)
sorry it took so long