Testcase: the port dns/amass Ports framework recommends ${INSTALL_PROGRAM} to be used to install elf programs. INSTALL_PROGRAM resolves to 'install -s -m 555'. install(1) prints warnings: > ===> Generating temporary packing list > install -s -m 555 /usr/ports/dns/amass/work/amass-2.8.3-4-g935e6d5_GH0/bin/amass /usr/ports/dns/amass/work/stage/usr/local/bin > strip: moving loadable section .noptrbss, is this intentional? > install -s -m 555 /usr/ports/dns/amass/work/amass-2.8.3-4-g935e6d5_GH0/bin/amass.db /usr/ports/dns/amass/work/stage/usr/local/bin > strip: moving loadable section .noptrbss, is this intentional? > install -s -m 555 /usr/ports/dns/amass/work/amass-2.8.3-4-g935e6d5_GH0/bin/amass.netdomains /usr/ports/dns/amass/work/stage/usr/local/bin > strip: moving loadable section .noptrbss, is this intentional? > install -s -m 555 /usr/ports/dns/amass/work/amass-2.8.3-4-g935e6d5_GH0/bin/amass.viz /usr/ports/dns/amass/work/stage/usr/local/bin > strip: moving loadable section .noptrbss, is this intentional?
Can you attach (the unstripped) bin/amass to this PR or store it somewhere and link it here?
https://people.freebsd.org/~yuri/amass
One possible workaround would be to avoid stripping Go-built executables during install and omit symbol table and debug information during link instead (with -ldflags="-s -w" go build flag) [1] For ports that use USES=go this could be done on ports framework level (unless WITH_DEBUG specified) [1] https://golang.org/cmd/link/
(In reply to Dmitri Goutnik from comment #3) I am experiencing the same behavior when building textproc/markdownfmt.
(In reply to Dmitri Goutnik from comment #3) Do you think that we could included something in the framework to address those warnings?
https://reviews.freebsd.org/D21730
A commit references this bug: Author: dmgk Date: Sat Nov 16 12:39:25 UTC 2019 New revision: 517743 URL: https://svnweb.freebsd.org/changeset/ports/517743 Log: Mk/Uses/go.mk: Omit symbol table and debug information by default PR: 233335 Reported by: yuri Reviewed by: 0mp, tobik Approved by: araujo (mentor) Differential Revision: https://reviews.freebsd.org/D21730 Changes: head/Mk/Uses/go.mk