mpv @0.31.0 fails to build on mavericks: macosx_menubar.m:718:67: error: property 'path' not found on object of type 'id'

[tehcog (tehcog)]

please see attached main.log

[tehcog (tehcog)]

main.log

[kencu (Ken)]

728	:info:build ../osdep/macosx_menubar.m:718:67: error: property 'path' not found on object of type 'id'
729	:info:build                                                   [panel URLs][0].path];
730	:info:build                                                                   ^
731	:info:build ../osdep/macosx_menubar.m:732:12: warning: 'NSTextField' may not respond to 'setPlaceholderString:'
732	:info:build     [input setPlaceholderString:@"URL"];
733	:info:build      ~~~~~ ^
734	:info:build 6 warnings and 1 error generated.

[kencu (Ken)]

This works:

--- multimedia/mpv/Portfile	2020-05-12 22:55:03.000000000 -0700
+++ multimedia/mpv/Portfile	2020-05-12 22:56:03.000000000 -0700
@@ -168,11 +168,13 @@
         compiler.blacklist-append   *gcc* cc macports-*
     }
 
-    if {${os.major} <= 10 || ${configure.build_arch} ne "x86_64"} {
+    if {${os.major} <= 15 || ${configure.build_arch} ne "x86_64"} {
         # Disable Cocoa output.
         # Users on old systems will need to embrace X11.
         configure.args-replace      --enable-cocoa \
                                     --disable-cocoa
+        configure.args-replace      --enable-swift \
+                                    --disable-swift
 
         default_variants-append     +x11
     }
@@ -477,7 +479,7 @@
 }
 
 variant opengl description {Enable OpenGL output support. Both the CoreVideo and X11 (GLX) outputs are supported} {
-    if {${os.platform} eq "darwin" && ${os.major} > 10} {
+    if {${os.platform} eq "darwin" && ${os.major} > 15} {
         configure.args-replace  --disable-gl-cocoa \
                                 --enable-gl-cocoa
     }

[kencu (Ken)]

So, caveats. This changes the port to use +x11 on older systems. I don't have the inclination to try to fix the objective-c and it's probably a waste of time as it changes frequently. And I just picked "darwin 15" out of the air -- that might or might not be the proper cutoff -- have to check the buildbots more carefully.

But with this patch, mpv installs on 10.7 without any trouble.

[ryandesign (Ryan Carsten Schmidt)]

Replying to kencu:

And I just picked "darwin 15" out of the air -- that might or might not be the proper cutoff -- have to check the buildbots more carefully.

Well according to the ​mpv port status page version 0.31.0 build successfully on OS X 10.11 (Darwin 15) and later and failed to build on OS X 10.10 (Darwin 14) and earlier.

[kencu (Ken)]

Replying to ryandesign:

Well according to the ​mpv port status page version 0.31.0 build successfully on OS X 10.11 (Darwin 15) and later and failed to build on OS X 10.10 (Darwin 14) and earlier.

It's likely the same error -- but "no logs" so wasn't sure.

[kencu (Ken)]

patch needs a bit more work. On 10.10, I had to also disable video toolbox if using x11.

There are so many variants in this port it makes my head spin.

[kencu (Ken)]

[79baae91e0c39d4f3bb8018c208a3afc89513e4a/macports-ports]