Created attachment 239518 [details] Update devel/binutils from 2.39 to 2.40 Can you please schedule an exp-run for this new release of binutils?
Thanks for looking into this, we might also want to consider enabling zstd support?
Created attachment 239535 [details] Two commits: Update to 2.40 and secondly, enable zstd suppport. This patch, includes two commits. The first patch uploaded and a commit to enable zstd support.
Thanks, shouldn't we depend on archives/zstd too?
(In reply to Daniel Engberg from comment #3) The patch uploaded Jan 17 includes zstd support. You should be able to do an exp-run anytime.
(In reply to Daniel Engberg from comment #3) I don't know if we should depend on the zstd port. zstd should be in all versions of -STABLE now. Or am I mistaken? I run 14-CURRENT here.
Since it's a port I think we should always depend on the port (just like libarchive etc) abd simply because base can become stale for ports users.
(In reply to Daniel Engberg from comment #6) Good enough. I'll update the patch.
Created attachment 239949 [details] Two commits: Update to 2.40, add zstd support, depend on zstd port These two commits will: 1. Update to 2.40. 2. Add zstd support and depend on the zstd port.
binutils fails to configure: checking for libzstd >= 1.4.0... no configure: error: --with-zstd was given, but pkgconfig/libzstd.pc is not found
Created attachment 240151 [details] Patch for binutils This includes previous patch by cy and following changes - Use DISTVERSION instead of PORTVERSION - Drop (reset) PORTREVISION - Define zstd as LIB_DEPENDS rather than DEPENDS - Add USES= localbase pkgconfig to fix (lib)zstd detection - Set zstd as default compression algorithm for compressed debug sections - Enable verbose output during build - Drop STATIC option, it's unsupported upstream and fails to build [1] Compile and runtime tested on FreeBSD 13.2-BETA1 (amd64) (make, make check-plist) Poudriere testport (default options) OK 12.3-RELEASE (amd64) Poudriere testport (default options) OK 13.1-RELEASE (i386) 1: /bin/sh ./libtool --tag=CC --mode=link cc -W -Wall -Wstrict-prototypes -Wmissing-prototypes -Wshadow -I/usr/local/include -O2 -pipe -march=tigerlake -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing -all-static -fstack-protector-strong -o windres windres.o resrc.o rescoff.o resbin.o rcparse.o rclex.o winduni.o resres.o bucomm.o version.o filemode.o ../bfd/libbfd.la ../libiberty/libiberty.a -lfl /usr/local/lib/libiconv.so -Wl,-rpath -Wl,/usr/local/lib -L/usr/local/lib ld: error: attempted static link of dynamic object /usr/local/lib/libiconv.so cc: error: linker command failed with exit code 1 (use -v to see invocation) *** Notes *** --enable-threads=yes configure switch seems to be retired since years ago? Unit tests seems to be broken, disable(?) error: invalid value 'zlib-gabi' in '--compress-debug-sections=zlib-gabi'
(In reply to Daniel Engberg from comment #10) This is certainly an improvement of the original two patches. Proposed commit log message: devel/binutils: Update from 2.39 to 2.40 Update our binutils port from 2.39 to 2.40. Enable zstd support. Depend on the port in case the port is newer than base. Additional contributions by dizzy@: - Use DISTVERSION instead of PORTVERSION - Drop (reset) PORTREVISION - Define zstd as LIB_DEPENDS rather than DEPENDS - Add USES= localbase pkgconfig to fix (lib)zstd detection - Set zstd as default compression algorithm for compressed debug sections - Enable verbose output during build - Drop STATIC option, it's unsupported upstream and fails to build Exp-run PR: 268994 Exp-run by: dizzy
Hi, We still need an exp-run by antoine, I've only tested the port itself in Poudriere. If "make test" is broken we should disable it for now at least. Best regards, Daniel
Created attachment 240331 [details] This version disables tests. make test will fail because --compress-debug-sections is always set to zlib-gabi regardless of the compression setting by ./configure.
Exp-run looks fine (with patch from 15 feb)
This can be closed.