Bug 272853 - devel/binutils: Update to 2.41
Summary: devel/binutils: Update to 2.41
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Port Management Team
URL: https://sourceware.org/git/?p=binutil...
Keywords:
Depends on: 272988 273023 273024 274846 274847
Blocks:
  Show dependency treegraph
 
Reported: 2023-07-31 20:20 UTC by Yasuhiro Kimura
Modified: 2024-06-09 14:11 UTC (History)
3 users (show)

See Also:
cy: maintainer-feedback+
yasu: exp-run?


Attachments
Patch file (5.44 KB, patch)
2023-07-31 20:20 UTC, Yasuhiro Kimura
no flags Details | Diff
Updated patch file (5.42 KB, patch)
2023-08-01 14:45 UTC, Yasuhiro Kimura
no flags Details | Diff
Updated patch file (8.65 KB, patch)
2023-10-26 01:38 UTC, Yasuhiro Kimura
no flags Details | Diff
patch for INFO option (960 bytes, patch)
2024-01-17 06:47 UTC, firk
no flags Details | Diff
Updated patch file (9.40 KB, patch)
2024-02-08 08:52 UTC, Yasuhiro Kimura
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yasuhiro Kimura freebsd_committer freebsd_triage 2023-07-31 20:20:20 UTC
Created attachment 243754 [details]
Patch file

Update to 2.41

ChangeLog:      https://sourceware.org/git/?p=binutils-gdb.git;a=blob;f=binutils/ChangeLog;h=866cf56d16af7994e23414c2d87cf08d119188cb;hb=2c73aeb8d2e02de7b69cbcb13361cfbca9d76a4e

Build of all flavors succeeds with poudriere and 13.2-RELEASE amd64 jail.
Comment 1 Cy Schubert freebsd_committer freebsd_triage 2023-07-31 23:39:26 UTC
The patch doesn't apply.

slippy$ git co -b binutils
Switched to a new branch 'binutils'
slippy$ git am /tmp/devel_binutils.patch 
Applying: devel/binutils: Update to 2.41
error: patch failed: devel/binutils/Makefile:1
error: devel/binutils/Makefile: patch does not apply
Patch failed at 0001 devel/binutils: Update to 2.41
hint: Use 'git am --show-current-patch=diff' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
slippy$ git am --abort
slippy$ pwd                                 
/home/cy/freebsd/git/ports
slippy$ git apply /tmp/devel_binutils.patch 
error: patch failed: devel/binutils/Makefile:1
error: devel/binutils/Makefile: patch does not apply
slippy$ 

I'll try to rework it to make git happy.
Comment 2 Cy Schubert freebsd_committer freebsd_triage 2023-07-31 23:42:31 UTC
I managed to use patch(1) to apply it.

I will push this commit upon the successful completion of an exp-run. Assigning to portmgr for this task.

@portmgr, assign this ticket back to me when the exp-run has completed.
Comment 3 Cy Schubert freebsd_committer freebsd_triage 2023-08-01 01:08:07 UTC
This needs to be build tested on 12.4-RELEASE and 14-CURRENT.
Comment 4 Antoine Brodin freebsd_committer freebsd_triage 2023-08-01 13:36:48 UTC
Please provide a pach that can be applied either using git apply or git am
Comment 5 Yasuhiro Kimura freebsd_committer freebsd_triage 2023-08-01 14:45:55 UTC
Created attachment 243770 [details]
Updated patch file

Regenerate patch.

It seems ports 182015786a35 make oritinal patch unapplicable.
Comment 6 Cy Schubert freebsd_committer freebsd_triage 2023-08-01 14:56:01 UTC
Same problem as before.

slippy$ git am /tmp/devel_binutils.patch 
Applying: devel/binutils: Update to 2.41
error: patch failed: devel/binutils/Makefile:1
error: devel/binutils/Makefile: patch does not apply
Patch failed at 0001 devel/binutils: Update to 2.41
hint: Use 'git am --show-current-patch=diff' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
slippy$ lh /tmp/devel_binutils.patch 
-rw-r--r--  1 cy wheel  5.4K Aug  1 07:49 /tmp/devel_binutils.patch
slippy$ 
slippy$ git apply /tmp/devel_binutils.patch 
error: patch failed: devel/binutils/Makefile:1
error: devel/binutils/Makefile: patch does not apply
slippy$
Comment 7 Cy Schubert freebsd_committer freebsd_triage 2023-08-01 15:04:13 UTC
Never mind. It applies to my development tree but not to my day-to-day tree.
Comment 8 Yasuhiro Kimura freebsd_committer freebsd_triage 2023-08-01 15:11:08 UTC
(In reply to Cy Schubert from comment #6)

With ports 6226965770f0 updated patch works fine with both 'git apply' and 'git am'.

yasu@rolling-vm-freebsd2[1156]% pwd                                                                                    ~/tmp
/home/yasu/tmp
yasu@rolling-vm-freebsd2[1157]% wget -O binutils.patch 'https://bz-attachments.freebsd.org/attachment.cgi?id=243770'   ~/tmp
--2023-08-02 00:07:35--  https://bz-attachments.freebsd.org/attachment.cgi?id=243770
Resolving bz-attachments.freebsd.org (bz-attachments.freebsd.org)... 2610:1c1:1:606c::50:3, 96.47.72.106
Connecting to bz-attachments.freebsd.org (bz-attachments.freebsd.org)|2610:1c1:1:606c::50:3|:443... connected.
HTTP request sent, awaiting response... 200 OK
Cookie coming from bz-attachments.freebsd.org attempted to set domain to bugs.freebsd.org
Length: unspecified [text/plain]
Saving to: ‘binutils.patch’

binutils.patch                      [ <=>                                                 ]   5.42K  --.-KB/s    in 0s      

2023-08-02 00:07:36 (57.9 MB/s) - ‘binutils.patch’ saved [5545]

yasu@rolling-vm-freebsd2[1158]% cd /usr/ports                                                                          ~/tmp
yasu@rolling-vm-freebsd2[1159]% git show | grep '^commit'                                                         /usr/ports
commit 6226965770f0
yasu@rolling-vm-freebsd2[1160]% git apply ~/tmp/binutils.patch                                                    /usr/ports
yasu@rolling-vm-freebsd2[1161]% git restore .                                                                     /usr/ports
yasu@rolling-vm-freebsd2[1162]% git am ~/tmp/binutils.patch                                                       /usr/ports
Applying: devel/binutils: Update to 2.41
yasu@rolling-vm-freebsd2[1163]
Comment 9 Yasuhiro Kimura freebsd_committer freebsd_triage 2023-08-01 15:18:06 UTC
(In reply to Cy Schubert from comment #7)

OK, I overlooked comment #7.
Comment 10 Cy Schubert freebsd_committer freebsd_triage 2023-08-01 16:15:06 UTC
Approving the commit if the exp-run passes without error or errors are mitigated.
Comment 13 Yasuhiro Kimura freebsd_committer freebsd_triage 2023-08-09 02:42:15 UTC
(In reply to Daniel Engberg from comment #12)

Thanks! I'm working to fix build error of multimedia/avidemux.
Comment 14 Yasuhiro Kimura freebsd_committer freebsd_triage 2023-10-26 01:38:42 UTC
Created attachment 245872 [details]
Updated patch file

Add upstream patch to fix build error of sysutils/uefi-edk2-bhyve-csm.
Comment 15 Yasuhiro Kimura freebsd_committer freebsd_triage 2023-11-18 04:19:34 UTC
(In reply to Antoine Brodin from comment #11)

With ports 88d031b44927 and attachment 245872 [details] all build errors listed in comment #11 are fixed. So would you please try another exp-run?

Regards.
Comment 17 firk 2024-01-17 06:47:40 UTC
Created attachment 247721 [details]
patch for INFO option

patch for INFO option that should be merged into update-2.41 patch
Comment 18 Yasuhiro Kimura freebsd_committer freebsd_triage 2024-02-08 08:52:37 UTC
Created attachment 248253 [details]
Updated patch file

Fix build of devel/elf-dissector.
Comment 19 Yasuhiro Kimura freebsd_committer freebsd_triage 2024-02-08 08:57:32 UTC
(In reply to Antoine Brodin from comment #16)

Build of devel/elf-dissector is fixed with latest patch, and
comms/ncid is removed from ports tree with ports d897f7a4c43c. So
would you please try another exp-run?

Regards.
Comment 20 Yasuhiro Kimura freebsd_committer freebsd_triage 2024-02-08 08:59:41 UTC
(In reply to firk from comment #17)

Please submit your patch as separate bug report as there is nothing to do with updating to 2.41.
Comment 21 firk 2024-02-14 21:23:34 UTC
(In reply to Yasuhiro Kimura from comment #20)

No, it is. It updates INFO option support from 2.40 to 2.41. 2.41 won't build with unchecked INFO otherwise.
Comment 22 Älven 2024-06-09 14:11:51 UTC
GNU Binutils 2.42 Released
https://lists.gnu.org/archive/html/info-gnu/2024-01/msg00016.html