#49051 closed update (fixed)
Update: VLC 2.2.1 with fix for the lua 5.3 incompatibility
Reported by: | RJVB (René Bertin) | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch maintainer | Cc: | ramundsen85 |
Port: | VLC |
Description
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.
Attachments (8)
Change History (29)
comment:1 Changed 9 years ago by RJVB (René Bertin)
Changed 9 years ago by RJVB (René Bertin)
diff against port:VLC version 2.1.5_6
Changed 9 years ago by RJVB (René Bertin)
Attachment: | VLC.tar.2.bz2 added |
---|
comment:2 Changed 9 years ago by RJVB (René Bertin)
Grrr, was sloppy creating the diff and tarball, and now updating the tarball attachment. :/
comment:3 Changed 9 years ago by mf2k (Frank Schima)
Keywords: | maintainer added |
---|
comment:4 follow-up: 6 Changed 9 years ago by 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.
Changed 9 years ago by ramundsen85
Attachment: | patch-opencv_example_include.diff added |
---|
insert missing include header in modules/video_filter/opencv_example.cpp
Changed 9 years ago by ramundsen85
Attachment: | patch-linker_framework_option.diff added |
---|
fix linker flags in modules/video_filter/
Changed 9 years ago by ramundsen85
Attachment: | Portfile.diff added |
---|
second patch for Portfile to include another two patches
comment:6 follow-up: 7 Changed 9 years ago by 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.
comment:7 Changed 9 years ago by 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.
comment:8 follow-up: 9 Changed 9 years ago by 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
comment:9 Changed 9 years ago by ramundsen85
Replying to rjvbertin@…:
If you're interested, you can do search for l-framework after
port extract VLC
,port patch VLC
, andport 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
comment:10 Changed 9 years ago by 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 ?
comment:11 Changed 9 years ago by 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.
comment:12 Changed 9 years ago by 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.
Changed 9 years ago by ramundsen85
Attachment: | Portfile2.diff added |
---|
Portfile patch for 2.2.2 version
Changed 9 years ago by ramundsen85
Attachment: | patch-opencv_includes.diff added |
---|
add missing include files for opencv
comment:13 Changed 9 years ago by 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.
comment:14 follow-up: 15 Changed 9 years ago by 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.
comment:15 Changed 9 years ago by 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 2.2.2 port works without any modifications for me os OS X 10.11, also default variant (+qtkit+quartz):
VLC @2.2.2_0+qtkit+quartz (active)
comment:16 Changed 9 years ago by 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.
comment:17 follow-up: 21 Changed 9 years ago by mojca (Mojca Miklavec)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Committed in r146098.
Please open a separate ticket for locale_select
.
comment:18 Changed 9 years ago by 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
comment:19 Changed 9 years ago by 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.
comment:20 Changed 9 years ago by mojca (Mojca Miklavec)
Ah, sorry, that's from VLC-devel
. I'll open a new ticket hijack #49022.
comment:21 Changed 9 years ago by RJVB (René Bertin)
Replying to mojca@…:
Please open a separate ticket for
locale_select
.
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.