Opened 20 months ago
Last modified 11 months ago
#67209 reopened defect
vim @9.0.1276: builds fail for macOS releases 10.10 and earlier: os_macosx.m
Reported by: | 0xdd-co | Owned by: | raimue (Rainer Müller) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | mascguy (Christopher Nielsen), Cebtenzzre | |
Port: | vim, MacVim |
Description
Unable to build vim on Mountain Lion. See attached main.log. Also had warnings during config:
---> Fetching distfiles for vim ---> Verifying checksums for vim ---> Extracting vim ---> Configuring vim Warning: Configuration logfiles contain indications of -Wimplicit-function-declaration; check that features were not accidentally disabled: statacl: found in vim-9.0.1276/src/auto/config.log timer_create: found in vim-9.0.1276/src/auto/config.log ---> Building vim Error: Failed to build vim: command execution failed Error: See /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_editors_vim/vim/main.log for details. Error: Follow https://guide.macports.org/#project.tickets if you believe there is a bug. Error: Processing of port vim failed
Attachments (1)
Change History (22)
Changed 20 months ago by 0xdd-co
comment:1 Changed 20 months ago by jmroot (Joshua Root)
Owner: | set to raimue |
---|---|
Port: | vim added |
Status: | new → assigned |
comment:2 Changed 20 months ago by evanmiller (Evan Miller)
It looks like the Objective-C compiler or standard library does not support generic dictionaries.
:info:build os_macosx.m:389:36: error: expected '>' 755 :info:build static NSMutableDictionary<NSNumber*, NSSound*> *sounds_list = nil; 756 :info:build ^ 757 :info:build os_macosx.m:389:37: error: expected identifier or '(' 758 :info:build static NSMutableDictionary<NSNumber*, NSSound*> *sounds_list = nil; 759 :info:build ^
comment:3 Changed 20 months ago by mascguy (Christopher Nielsen)
Cc: | mascguy added |
---|
comment:4 Changed 20 months ago by mascguy (Christopher Nielsen)
Summary: | vim @9.0.1276 build failure on Mac OS X 10.8.5 - os_macosx.m → vim @9.0.1276: builds fail for macOS releases 10.11 and earlier: os_macosx.m |
---|
This is affecting a range of macOS releases, 10.10 and earlier; updated the summary to reflect that.
comment:5 Changed 20 months ago by mascguy (Christopher Nielsen)
Summary: | vim @9.0.1276: builds fail for macOS releases 10.11 and earlier: os_macosx.m → vim @9.0.1276: builds fail for macOS releases 10.10 and earlier: os_macosx.m |
---|
comment:6 Changed 20 months ago by Cebtenzzre
Cc: | Cebtenzzre added |
---|
comment:7 Changed 19 months ago by Christopher Nielsen <mascguy@…>
comment:8 follow-up: 9 Changed 19 months ago by raimue (Rainer Müller)
Port: | MacVim added |
---|
The reason must be this upstream change: https://github.com/vim/vim/commit/4314e4f7da4db5d85f63cdf43b73be3689502c93 (9.0.0694)
MacVim must be affected the same way as this change is also part of @9.0.release176.
comment:9 Changed 19 months ago by mascguy (Christopher Nielsen)
Replying to raimue:
The reason must be this upstream change: https://github.com/vim/vim/commit/4314e4f7da4db5d85f63cdf43b73be3689502c93 (9.0.0694)
MacVim must be affected the same way as this change is also part of @9.0.release176.
It looks like MacVim has been broken for 10.11 since 2022-08. For 10.10 and earlier, it's been broken since at least 2019. (And perhaps longer, but that's as far back as the build history goes.)
Version-wise, the first update to 9.x broke 10.11. As for 10.10 and earlier, it's been broken since at least version 8.x.
Since all of the build logs have aged out, I requeued builds for it. We can take a look at the details once the buildbots catch up with their current backlog.
comment:10 follow-up: 13 Changed 15 months ago by sevan (Sevan Janiyan)
v9.0.1910 guards off enabling sound support to El Capitan & newer. I've managed to build (not via macport) the huge feature set fine on 10.4 ppc, 10.6 x86, and 10.11 without issue.
comment:11 follow-up: 12 Changed 15 months ago by fhgwright (Fred Wright)
Although not directly related to this ticket, I see that vim
fails to declare a library dependency on libsodium
, causing the existing 9.0.0472 install to break with the recent libsodium
update. I see many references to libsodium
in the sources, so this is a missing direct dependency.
comment:12 Changed 15 months ago by mascguy (Christopher Nielsen)
Replying to fhgwright:
Although not directly related to this ticket, I see that
vim
fails to declare a library dependency onlibsodium
, causing the existing 9.0.0472 install to break with the recentlibsodium
update. I see many references tolibsodium
in the sources, so this is a missing direct dependency.
That will be taken care of with this PR:
PR 20541 - vim,MacVim: avoid opportunistic linking with libsodium
comment:13 Changed 15 months ago by mascguy (Christopher Nielsen)
Replying to sevan:
v9.0.1910 guards off enabling sound support to El Capitan & newer. I've managed to build (not via macport) the huge feature set fine on 10.4 ppc, 10.6 x86, and 10.11 without issue.
Awesome, looking forward to being able to use the latest version of vim
on older macOS releases. Thank you so much!
comment:14 follow-up: 15 Changed 14 months ago by raimue (Rainer Müller)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
This was fixed by now with https://github.com/macports/macports-ports/pull/20541.
comment:15 Changed 14 months ago by sevan (Sevan Janiyan)
Replying to raimue:
This was fixed by now with https://github.com/macports/macports-ports/pull/20541.
That pull request is for fixing the libsodium issue. The ticket is regarding enabling sound support on macos versions before el capitan. The fix for that is to drop the block which holds versions back to patch level 0472, as the issue has been fixed upstream so that sound support is only enabled when building with a recent enough toolchain. https://github.com/macports/macports-ports/blob/b89316123a4a60af3ba1e645842b91bd5e069929/editors/vim/Portfile#L11
comment:16 Changed 14 months ago by sevan (Sevan Janiyan)
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:17 Changed 14 months ago by raimue (Rainer Müller)
Thanks for reopening the issue. Therefore the issue is not fixed until we reach vim @9.0.1910.
comment:18 Changed 14 months ago by sevan (Sevan Janiyan)
Correct, but there's no longer a need to have a distinction between macos versions and packaging different versions of vim, you should be able to build and install the latest version of vim from Tiger & beyond.
comment:19 Changed 13 months ago by raimue (Rainer Müller)
The vim port follows the vim version used in the MacVim port. Both will be updated at the same time. That is because both use the same configuration files and a mismatch in those can cause problems for users to get it working with both at the same time. Therefore the vim port is currently still at 9.0.1897 even though upstream has already released a newer patch.
comment:20 Changed 13 months ago by sevan (Sevan Janiyan)
Ah, understood. I was thinking of the block which holds things back to 9.0.0472 for the older macOS versions.
if {${os.platform} eq "darwin" && ${os.major} <= 14} { # Fallback to previous port version, for 10.10 and earlier # https://trac.macports.org/ticket/67209 set vim_patchlevel 0472 set port_revision 1 checksums rmd160 ee809eb300deaf735c86b6e5b7f4bc4a8fe79a41 \ sha256 7185577d6cd3708b62b2079b69a29215bb6a6048070d34e960d2d565c18b8a9b \ size 16851486 } else {
Hadn't noticed that you'd not reached 9.0.1910 yet. :)
main.log