Update: VLC 2.2.1 with fix for the lua 5.3 incompatibility

[RJVB (René Bertin)]

Attached is a port:VLC version that bumps VLC to v2.2.1, and provides a solution for the incompatibility issue introduced by Lua 5.3 .

VLC version 2.2.2 is supposed to be compatible with Lua 5.3; earlier 2.2 versions actually search first for Lua 5.2 . I added a patch to make this work with port:lua52; this will hopefully be a temporary fix.

[RJVB (René Bertin)]

NB: this port also provides libVLC, which IMHO has much more reason to be in MacPorts than the VLC application which can just as well be installed with the official installer.

[RJVB (René Bertin)]

diff against port:VLC version 2.1.5_6

[RJVB (René Bertin)]

Grrr, was sloppy creating the diff and tarball, and now updating the tarball attachment. :/

[ramundsen85]

Thank you for providing your port files. In another ticket you mentioned, that you use OS X 10.9. So I tried with OS X 10.11.

With default variant (+qtkit+quartz) I could not build the port due to failures in the modules/video_filter/ part. First there is a missing include header (patch-opencv_example_include.diff) and then problems with linker flags, for example:

-l-framework Cocoa

This can be solved (patch-linker_framework_option.diff) by changing the flag to

-Wl,-framework,Cocoa

The problem is, the Makefile to patch will be created during configuration. So, I wrote my own patch command into post-configure{}. I'm wondering if there is a proper solution for this. The attached Portfile.diff has to be used on the Portfile provided in VLC.tar.bz2.

[ramundsen85]

insert missing include header in modules/video_filter/opencv_example.cpp

[ramundsen85]

fix linker flags in modules/video_filter/

[ramundsen85]

second patch for Portfile to include another two patches

[ramundsen85]

Cc Me!

[RJVB (René Bertin)]

Replying to framic5@…:

problems with linker flags, for example:

-l-framework Cocoa

That's weird, that should fail on any OS X version...

The problem is, the Makefile to patch will be created during configuration. So, I wrote my own patch command into post-configure{}. I'm wondering if there is a proper solution for this. The attached Portfile.diff has to be used on the Portfile provided in VLC.tar.bz2.

The proper way would be to figure out where the linker option error comes from and then either fix the autoconf/automake system if that's the source, or the Makefile.am file (or whatever). I'm double-checking if it's not an error I overlooked; I don't think I've every built only with the default variants.

[ramundsen85]

Replying to rjvbertin@…:

The proper way would be to figure out where the linker option error comes from and then either fix the autoconf/automake system if that's the source, or the Makefile.am file (or whatever).

Yes, I tried to do this. But I can not find the wrong flag. Not in Makefile.am not even with using find to see if the flag appears somewhere else in the sources. So my guess was, that this happens during creation of Makefile.

[RJVB (René Bertin)]

If I look at the raw files after port extract VLC, a grep 'framework.*Cocoa' -R port work VLC/vlc* reveals nothing that suggests that your issue might occur.

If you're interested, you can do search for l-framework after port extract VLC, port patch VLC, and port configure VLC. It might even be a good idea to do it after the autoreconf step (by interrupting or suspending the port command as soon as the actual configure command is started).

What does your log show for these 2 lines?

checking build system type... x86_64-apple-darwin13.4.0
checking host system type... x86_64-apple-darwin13.4.0

[ramundsen85]

Replying to rjvbertin@…:

If you're interested, you can do search for l-framework after port extract VLC, port patch VLC, and port configure VLC. It might even be a good idea to do it after the autoreconf step (by interrupting or suspending the port command as soon as the actual configure command is started).

No matches after port extract, port patch and autoreconf (interrupted by setting configure.cmd to an invalid command). After port configure I get 94 matches (87 in several Makefiles, rest in config.log and config.status).

I'm wondering why I only run into problems with the video_filter Makefile and not with the others. I looked at the failing command again and saw, that the linker flag is truncated:

libtool: link: /usr/bin/clang  -o .libs/libopencv_wrapper_plugin.dylib -bundle  .libs/libopencv_wrapper_plugin_la-opencv_wrapper.o   ../../compat/.libs/libcompat.a -L/opt/local/lib -L/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_multimedia_VLC/VLC/work/vlc-2.2.1/contrib/lib ../../src/.libs/libvlccore.dylib -lintl -lidn -liconv -lm -lopencv_shape -lopencv_stitching -lopencv_objdetect -lopencv_superres -lopencv_videostab -lopencv_calib3d -lopencv_features2d -lopencv_highgui -lopencv_videoio -lopencv_imgcodecs -lopencv_video -lopencv_photo -lopencv_ml -lopencv_imgproc -lopencv_flann -lopencv_core -lopencv_hal -lpng -ltiff -ljasper -ljpeg -lImath -lIlmImf -lIex -lHalf -lIlmThread -lavcodec -lavformat -lavutil -lswscale -lavresample -lz -lbz2 -l-framework  -Os -arch x86_64 -O3 -Wl,-headerpad_max_install_names -arch x86_64 -Wl,-headerpad_max_install_names   -Wl,-exported_symbols_list,.libs/libopencv_wrapper_plugin-symbols.expsym
ld: library not found for -l-framework
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[5]: *** [libopencv_wrapper_plugin.la] Error 1

-l-framework is everything what's left from 5 flags. Compared to the command after I apply my patch on Makefile:

libtool: link: /usr/bin/clang  -o .libs/libopencv_wrapper_plugin.dylib -bundle  .libs/libopencv_wrapper_plugin_la-opencv_wrapper.o   ../../compat/.libs/libcompat.a -L/opt/local/lib -L/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_multimedia_VLC/VLC/work/vlc-2.2.1/contrib/lib ../../src/.libs/libvlccore.dylib -lintl -lidn -liconv -lm -lopencv_shape -lopencv_stitching -lopencv_objdetect -lopencv_superres -lopencv_videostab -lopencv_calib3d -lopencv_features2d -lopencv_highgui -lopencv_videoio -lopencv_imgcodecs -lopencv_video -lopencv_photo -lopencv_ml -lopencv_imgproc -lopencv_flann -lopencv_core -lopencv_hal -lpng -ltiff -ljasper -ljpeg -lImath -lIlmImf -lIex -lHalf -lIlmThread -lavcodec -lavformat -lavutil -lswscale -lavresample -lz -lbz2  -Os -arch x86_64 -O3 -Wl,-headerpad_max_install_names -arch x86_64 -Wl,-headerpad_max_install_names -Wl,-framework -Wl,Cocoa -Wl,-framework -Wl,VideoDecodeAcceleration -Wl,-framework -Wl,QTKit -Wl,-framework -Wl,QuartzCore -Wl,-framework -Wl,AppKit   -Wl,-exported_symbols_list,.libs/libopencv_wrapper_plugin-symbols.expsym

What does your log show for these 2 lines?

:info:configure checking build system type... x86_64-apple-darwin15.0.0
:info:configure checking host system type... x86_64-apple-darwin15.0.0

[RJVB (René Bertin)]

framic5 : my bad, I completely forgot about your patches. Do you mind checking and where necessary updating them after this port update to VLC 2.2.2 ?

[ramundsen85]

I tested the port provided, thank you for that. But I'm facing new problems:

In file included from opencv_wrapper.c:42:
In file included from /opt/local/include/opencv2/core/core_c.h:48:
/opt/local/include/opencv2/core/types_c.h:929:13: error: implicit declaration of function 'cvRound' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
    ipt.x = cvRound(point.x);
            ^

The function in question is declared in opencv2/core/fast_math.hpp. I'll try to find out why it is not included.

It seems, in comment:6 we have some open issues. Do we know that we have a general problem here with OS X 10.11 or if this is problem from my side? I'll by happy to work on this, but I want to make sure this is of general interest.

[ramundsen85]

Ok, all problems I reported with 2.2.1 are gone with your new port version. To get the new port installed, I only had to include the fast_math.hpp header.

I'll attach the 2.2.2 Portfile patch file and the header include patch file which did the trick for my system.

[ramundsen85]

Portfile patch for 2.2.2 version

[ramundsen85]

add missing include files for opencv

[RJVB (René Bertin)]

Sorry, I saw your earlier message about the C99 message but am not on my Mac so couldn't check if I even build that component (I'd hope so though :)). Curiously enough I got a very similar error when I tried to build against lua 5.3 which is supposed to be possible. Maybe we're simply looking at the effect of imposing the use of C99 where it shouldn't be imposed.

[RJVB (René Bertin)]

I understand now why I didn't have issues with opencv: I was still using an older version of that port.

However, after updating I ran into 2 issues:

• when opencv is installed with its Qt5 module (variants +qt5 or +qt4 because of a current bug in the port), VLC fails to build because the opencv pkg-config file contains errors. Do you have either of those variants active in your opencv build?
• the opencv_example doesn't compile at all for me on 10.9 . Looks like a missing definition in a header, but more complicated than I currently am willing to investigate ... for an example.

I've addressed the 2nd issue by disabling building the opencv_example. The 1st issue can only be addressed by registering a build conflict when opencv is installed +qt4 or +qt5 , until the pkg-config file is corrected. After building VLC's opencv wrapper module loads fine when opencv is installed +qt4 or +qt5.

[RJVB (René Bertin)]

upgraded to 2.2.2

[ramundsen85]

Replying to rjvbertin@…:

• when opencv is installed with its Qt5 module (variants +qt5 or +qt4 because of a current bug in the port), VLC fails to build because the opencv pkg-config file contains errors. Do you have either of those variants active in your opencv build?

I have the default opencv variant installed (no +qt4 or +qt5):

opencv @3.1.0_1 (active)

Your new VLC 2.2.2 port works without any modifications for me on OS X 10.11, also default variant (+qtkit+quartz):

VLC @2.2.2_0+qtkit+quartz (active)

[RJVB (René Bertin)]

Good, that's as I would have hoped, thanks for confirming.

If you're feeling adventurous you could try to build VLC with +full. (That doesn't really increase VLC's footprint, but you may end up installing a bunch of additional dependencies.

[mojca (Mojca Miklavec)]

Committed in r146098.

Please open a separate ticket for locale_select.

[mojca (Mojca Miklavec)]

The initial build on the buildbots failed due to a missing dependency on libvpx which I fixed in r146100.

I see that "Jeremy's" VLC-devel also depends on:

port:FreeRDP
port:pulseaudio
port:XviD

plus a number of other "X11" dependencies. Maybe worth investigating.

One probably also has to fix

path:lib/libssl.dylib:openssl

[mojca (Mojca Miklavec)]

Hmmm. I don't understand that. The build work on my machine, but on the ​buildbot we get a lua-related error:

lua/demux.c:55:13: error: implicit declaration of function 'luaL_checkint' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
    int n = luaL_checkint( L, 1 );
            ^
lua/demux.c:55:13: note: did you mean 'luaL_checkany'?
/opt/local/include/lauxlib.h:52:18: note: 'luaL_checkany' declared here
LUALIB_API void (luaL_checkany) (lua_State *L, int arg);
                 ^
lua/demux.c:76:13: warning: unused variable 'i_seek' [-Wunused-variable]
        int i_seek = stream_Read( p_demux->s, NULL, i_read );
            ^
1 warning and 1 error generated.
clang: warning: -O4 is equivalent to -O3
make[4]: *** [lua/liblua_plugin_la-demux.lo] Error 1
make[4]: *** Waiting for unfinished jobs....

Any help appreciated.

[mojca (Mojca Miklavec)]

Ah, sorry, that's from VLC-devel. I'll open a new ticket hijack #49022.

[RJVB (René Bertin)]

Replying to mojca@…:

Please open a separate ticket for locale_select.

​https://trac.macports.org/ticket/50776