Created attachment 219971 [details] Patch to update devel/binutils to upstream version 2.35.1 Good morning, This is my first submission as the official maintainer of this port; any and all feedback is both welcome and appreciated. Changes: * The 0b398d6, gas-configure, and libctf_swap patches were no longer required, as their changes have already been incorporated into the upstream branch. Notes: The patches for the s390* and aarch64 platforms required manual edits, but I made sure to copy over the changes exactly as they were in the previous version. However, I am not familiar with those platforms, nor do I have access to them to be able to test the changes, so please reach out to me with any problems. You can email me at jflopezfernandez@gmail.com.
^Triage: [tags] in issue Titles are deprecated. ^Triage: Simplifying title ^Triage: If there is a changelog or release notes URL available for this version, please add it to the URL field. ^Triage: Maintainer-feedback flag (+) not required unless requested (?) first. There are some warnings from linters. Some of the are false positives, others are not. Q/A: Makefile: [141]: possible direct use of command "rm" found. use ${RM} instead. Makefile: new ports should not set PORTREVISION. Makefile: new ports should not set PORTEPOCH. Makefile: unless this is a master port, MAINTAINER has to be set by "=", not by "?=". Makefile: unless this is a master port, COMMENT has to be set by "=", not by "?=". Makefile: "PKGNAMEPREFIX" has to appear earlier. Makefile: "BROKEN_mips" has to appear earlier. Makefile: "BUILD_DEPENDS" has to appear earlier. Makefile: "USES" has to appear earlier. ^Triage: Please confirm this change passes QA (portlint, poudriere at least). -- https://www.freebsd.org/doc/en/books/porters-handbook/testing.html Q/A: PORTREVISION should be removed Thanks!
(In reply to Fernando Apesteguía from comment #1) Good afternoon, thank you so much for your reply. I apologize for the errors; I will submit a corrected patch once the necessary modifications have been made. Thank you again for your help, Jose
(In reply to Jose Fernando Lopez Fernandez from comment #2) No need to apologise :-) Just FYI, portlint, portclippy and portfmt are tools in ports that can help you with maintaining ports.
Created attachment 219991 [details] Revised patch updating devel/binutils port to 2.35.1 (In reply to Fernando Apesteguía from comment #3) This revised patch contains nearly all of the requested modifications, with the sole exception being the portlint changes. I was able to fix all but three of the warnings ("BROKEN", "BUILD_DEPENDS", and "USES"), as they rely on the particular flavor of the package the user is building. I believe page 209 of the Porter's Handbook makes a specific allowance for this case, but please do let me know if this is not correct. All other portlint warnings have been fixed, and the package passes all QA tests and builds successfully with poudriere. Thank you again for all of your help, and please do let me know if I have missed anything. Jose Change log for GNU binutils version 2.35.1: https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=blob_plain;f=binutils/NEWS;hb=refs/tags/binutils-2_35_1
(In reply to Jose Fernando Lopez Fernandez from comment #4) Hi Jose, Unfortunately, the port does not build. It fails at the patch phase: ===> Applying FreeBSD patches for binutils-2.35.1,1 from /usr/ports/devel/binutils/files Ignoring previously applied (or reversed) patch. 1 out of 1 hunks ignored--saving rejects to binutils/coffdump.c.rej Ignoring previously applied (or reversed) patch. 1 out of 1 hunks ignored--saving rejects to binutils/srconv.c.rej Ignoring previously applied (or reversed) patch. 1 out of 1 hunks ignored--saving rejects to binutils/sysdump.c.rej ===> FAILED Applying FreeBSD patch-0b398d6.diff ===> FAILED to apply cleanly FreeBSD patch(es) patch-0b398d6.diff *** Error code 1 Stop. make: stopped in /usr/ports/devel/binutils Would you have a look at it?
Hi, Fernando, I will figure out what I did wrong first thing tomorrow morning. I promise future updates will not be this tedious; I am determined to get this right, but I am very embarrassed to have wasted so much of your time. Thank you again, and I look forward to having an answer for you tomorrow morning. Jose
(In reply to Jose Fernando Lopez Fernandez from comment #6) There is no rush :-) Maintaining ports can be challenging, specially the complex ones.
Created attachment 224892 [details] First attempt to update devel/binutils to 2.36.1 My first attempt to update devel/binutils. Current status is: 1. Build succeeds with following conditions Ports tree: ports caced2ad007d of main Host OS: 13.0-RELEASE amd64 Options settings: everything is default Flavors: all but mips64 2. No run-time test at all yet My working repository: https://github.com/yasuhirokimura/freebsd-ports/tree/binutils_work
Created attachment 224894 [details] Upddate devel/binutils to 2.36.1 (2nd version)
(In reply to Yasuhiro Kimura from comment #9) Updated patch fixes build error of mips64 flavor. So now build succeeds with all flavors.
Created attachment 224961 [details] Update devel/binutils to 2.36.1 (3rd version) Fix `make check-plist` when STATIC option is ON.
Comment on attachment 224961 [details] Update devel/binutils to 2.36.1 (3rd version) > +lib/bfd-plugins/libdep.a This file is problematic since it causes conflicts across all the binutils flavors but you need to be able to install them at the same time.
(In reply to Tobias Kortkamp from comment #12) Has any of you been able to make test? It fails in an i386 jail with: pr20308_gd.o:function test_gd: error: missing expected TLS relocation pr20308_ld.o:function test_ld: error: missing expected TLS relocation cc: error: linker command failed with exit code 1 (use -v to see invocation) and in amd64 with: c++: error: unsupported argument '-madd-bnd-prefix' to option 'Wa,'
Created attachment 226164 [details] Update devel/binutils to 2.36.1 (4th version) * Add upstream patches * Fix build error with `make test`
(In reply to Yasuhiro Kimura from comment #14) Testing... Thanks for the update. Jose Fernando, did you have time to test some of these? Cheers!
Hi Yasuhiro, Sorry for the late response. This update fails to build: (pod2man --center="GNU Development Tools" --release="binutils-2.36.1" --section=1 as.pod | \ sed -e '/^.if n .na/d' > as.1.T$$ && \ mv -f as.1.T$$ as.1) || \ (rm -f as.1.T$$ && exit 1) /bin/sh: pod2man: not found rm -f as.pod /wrkdirs/usr/ports/devel/binutils/work-native/binutils-2.36.1/missing: makeinfo: not found WARNING: 'makeinfo' is missing on your system. You should only need it if you modified a '.texi' file, or any other file indirectly affecting the aspect of the manual. You might want to install the Texinfo package: <http://www.gnu.org/software/texinfo/> The spurious makeinfo call might also be the consequence of using a buggy 'make' (AIX, DU, IRIX), in which case you might want to install GNU make: <http://www.gnu.org/software/make/> gmake[5]: *** [Makefile:508: as.info] Error 127 gmake[5]: Leaving directory '/wrkdirs/usr/ports/devel/binutils/work-native/binutils-2.36.1/gas/doc' gmake[4]: *** [Makefile:1275: all-recursive] Error 1 gmake[4]: Leaving directory '/wrkdirs/usr/ports/devel/binutils/work-native/binutils-2.36.1/gas' gmake[3]: *** [Makefile:817: all] Error 2 gmake[3]: Leaving directory '/wrkdirs/usr/ports/devel/binutils/work-native/binutils-2.36.1/gas' gmake[2]: *** [Makefile:4939: all-gas] Error 2 gmake[2]: Leaving directory '/wrkdirs/usr/ports/devel/binutils/work-native/binutils-2.36.1' gmake[1]: *** [Makefile:853: all] Error 2 gmake[1]: Leaving directory '/wrkdirs/usr/ports/devel/binutils/work-native/binutils-2.36.1' ===> Compilation failed unexpectedly. Try to set MAKE_JOBS_UNSAFE=yes and rebuild before reporting the failure to the maintainer. *** Error code 1
Created attachment 226384 [details] Update devel/binutils to 2.36.1 (5th version) * Fix build with poudriere * Add more upstream patch
(In reply to Fernando Apesteguía from comment #16) Fixed with latest patch. Sorry for breakage.
There is a ton of added patches. Were are they coming from?
Created attachment 226557 [details] Update devel/binutils to 2.37 Update to 2.37. ChangeLog: https://sourceware.org/git/?p=binutils-gdb.git;a=blob;f=ChangeLog;h=641d82c5358c863fbaba07a61e14a135c5785959;hb=116a737f438d03a1bd6aa706b6ea0b4022f3b7e2 Note that patch-bfd_elfxx-sparc.c fails to be applied to 2.37 because the contents of bfd/elfxx-sparc.c has changed significantly from that of 2.36.1. I tried to adjust the patch but failed and decided to removed it. If there is someone who understands this patch, please update it correctly.
(In reply to Yasuhiro Kimura from comment #20) At this point, speaking as one of the former sparc users, sparc is simply OBE, and thus is indeed not worth spending time getting to work.
(In reply to Mark Linimon from comment #21) Thanks for comment. I leave the patch removed.
(In reply to Yasuhiro Kimura from comment #20) If sparc patch removal means the port fails to build/run, the appropriate BROKEN/IGNORE should be set, and probably referencing the patch that needs rebasing in order to remove BROKEN/IGNORE
Created attachment 226621 [details] Patch with small adjustments I did some adjustments to the Makefile and re-included the SPARC patch with small changes to make so it applies cleanly. With this, devel/binutils builds in {11.4,12.2}{amd64,i386} and 13-current amd64. I run a poor man's exp-run and checked that some ports which have devel/binutils as a BUILD dependency build fine: audio/ardour6 comms/libfec devel/sdl20
Hi there, Has anyone given it a go? Do you think we should ask for an exp-run? Cheers!
(In reply to Fernando Apesteguía from comment #25) > Do you think we should ask for an exp-run? Yes, I think so.
Requesting exp-run. There are 94 ports referencing devel/binutils in their Makefiles in different ways (BUILD_DEPENDS, RUN_DEPENDS...). ./audio/ardour6 ./audio/musicpd ./base/binutils ./comms/libfec ./comms/syncterm ./databases/galera ./databases/galera26 ./databases/mongodb40 ./databases/mongodb42 ./databases/mongodb44 ./databases/mongodb49 ./databases/mongodb50 ./devel/aarch64-none-elf-gcc ./devel/abi-compliance-checker ./devel/arduino-tools ./devel/arm-none-eabi-gcc492 ./devel/arm-none-eabi-newlib ./devel/avr-gcc ./devel/avr-libc ./devel/electron12 ./devel/elf-dissector ./devel/fpc-bfd ./devel/freebsd-gcc6 ./devel/freebsd-gcc9 ./devel/gcc-arm-embedded ./devel/gcc-msp430-ti-toolchain ./devel/gnu-efi ./devel/kcov ./devel/llvm-devel ./devel/llvm10 ./devel/llvm11 ./devel/llvm12 ./devel/llvm70 ./devel/llvm80 ./devel/llvm90 ./devel/mingw32-gcc ./devel/plan9port ./devel/plasma ./devel/psptoolchain-gcc-stage1 ./devel/riscv64-gcc ./devel/sdl20 ./editors/lazarus ./emulators/visualboyadvance-m ./emulators/vmw ./games/dxx-rebirth ./java/openjdk8 ./lang/ccl ./lang/fpc ./lang/gcc10 ./lang/gcc10-devel ./lang/gcc11 ./lang/gcc11-devel ./lang/gcc12-devel ./lang/gcc48 ./lang/gcc6-aux ./lang/gcc7 ./lang/gcc8 ./lang/gcc9 ./lang/gcc9-devel ./lang/go ./lang/gprolog ./lang/hla ./lang/mecrisp-stellaris ./lang/mit-scheme ./lang/myrddin ./lang/ocaml ./lang/rust ./lang/rust-bootstrap ./lang/v8 ./math/ldouble ./math/opensolaris-libm ./multimedia/ffmpeg ./multimedia/libde265 ./multimedia/libvpx ./multimedia/libxine ./multimedia/x265 ./net/ceph14 ./net/dpdk ./net/zerotier ./science/libghemical ./security/afl ./security/afl++ ./security/akmos ./security/nflib ./security/nss ./sysutils/opensbi ./www/chromium ./www/deno ./www/node ./www/node10 ./www/node14 ./www/webkit2-gtk3 ./x11-toolkits/qt5-gui ./x11/pixman