vim @9.0.1276: builds fail for macOS releases 10.10 and earlier: os_macosx.m

[0xdd-co]

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

[0xdd-co]

main.log

[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                                     ^

[mascguy (Christopher Nielsen)]

This is affecting a range of macOS releases, 10.10 and earlier; updated the summary to reflect that.

​https://ports.macports.org/port/vim/details/

[Christopher Nielsen <mascguy@…>]

In 42d8abd52fbd0d13e478aea1835dc9652c5d946f/macports-ports (master):

vim: fallback to previous version, for 10.10 and earlier
See: #67209

[raimue (Rainer Müller)]

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.

[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.

[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.

[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.

[mascguy (Christopher Nielsen)]

Replying to fhgwright:

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.

That will be taken care of with this PR:

​PR 20541 - vim,MacVim: avoid opportunistic linking with libsodium

[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!

[raimue (Rainer Müller)]

This was fixed by now with ​https://github.com/macports/macports-ports/pull/20541.

[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

[raimue (Rainer Müller)]

Thanks for reopening the issue. Therefore the issue is not fixed until we reach vim @9.0.1910.

[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.

[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.

[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. :)

[Thomas Nyman <thomas.nyman@…>]

In 7eaf80b7b19b1b4d5b590f0854329b54c4ec6bd9/macports-ports (master):

vim, MacVim: update to vim version 9.1.0000

• update Vim to v9.1.0000
• update MacVim to release-179
• remove fallback version for macOS 10.10 and earlier (#67209)