Opened 19 months ago

Last modified 10 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)

main.log (57.3 KB) - added by 0xdd-co 19 months ago.
main.log

Download all attachments as: .zip

Change History (22)

Changed 19 months ago by 0xdd-co

Attachment: main.log added

main.log

comment:1 Changed 19 months ago by jmroot (Joshua Root)

Owner: set to raimue
Port: vim added
Status: newassigned

comment:2 Changed 19 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 19 months ago by mascguy (Christopher Nielsen)

Cc: mascguy added

comment:4 Changed 19 months ago by mascguy (Christopher Nielsen)

Summary: vim @9.0.1276 build failure on Mac OS X 10.8.5 - os_macosx.mvim @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.

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

Last edited 19 months ago by mascguy (Christopher Nielsen) (previous) (diff)

comment:5 Changed 19 months ago by mascguy (Christopher Nielsen)

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

comment:6 Changed 19 months ago by Cebtenzzre

Cc: Cebtenzzre added

comment:7 Changed 18 months ago by Christopher Nielsen <mascguy@…>

In 42d8abd52fbd0d13e478aea1835dc9652c5d946f/macports-ports (master):

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

comment:8 Changed 18 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 in reply to:  8 Changed 18 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 Changed 14 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.

Last edited 14 months ago by sevan (Sevan Janiyan) (previous) (diff)

comment:11 Changed 14 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 in reply to:  11 Changed 14 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 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

comment:13 in reply to:  10 Changed 14 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 Changed 13 months ago by raimue (Rainer Müller)

Resolution: fixed
Status: assignedclosed

comment:15 in reply to:  14 Changed 13 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 13 months ago by sevan (Sevan Janiyan)

Resolution: fixed
Status: closedreopened

comment:17 Changed 13 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 13 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. :)

comment:21 Changed 10 months ago by 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)
Note: See TracTickets for help on using tickets.