JuliaIO / VideoIO.jl

Reading and writing of video files in Julia via ffmpeg
https://juliaio.github.io/VideoIO.jl/stable
Other
128 stars 53 forks source link

ERROR: av_log_set_level not defined #49

Closed lucasb-eyer closed 9 years ago

lucasb-eyer commented 10 years ago

I don't understand this error:

julia> using VideoIO
julia> VideoIO.versioninfo()
Using ffmpeg
AVCodecs version 56.1.100
AVFormat version 56.4.101
AVUtil version 54.7.100
SWScale version 3.0.100
AVDevice version 56.0.100
AVFilters version 5.1.100

julia> v = VideoIO.openvideo("/home/beyer/data/bbenfold/TownCentreXVID.avi")
ERROR: av_log_set_level not defined
 in AVInput at /home/beyer/.julia/v0.4/VideoIO/src/avio.jl:222
 in AVInput at /home/beyer/.julia/v0.4/VideoIO/src/avio.jl:221
 in VideoReader at /home/beyer/.julia/v0.4/VideoIO/src/avio.jl:364
 in openvideo at /home/beyer/.julia/v0.4/VideoIO/src/avio.jl:482

But when I grep for it, I do have the file AVUtil/v54/log.jl (as well as v51, v52 and v53) which defines this function:

src/ffmpeg/AVUtil/v54/log.jl
8:    av_log_set_level,
26:function av_log_set_level(level::Integer)
27:    ccall((:av_log_set_level,libavutil),Void,(Cint,),level)

I'm on Julia master and just did Pkg.update(), don't know how recent that error is, though, as it's been a little while I haven't used VideoIO.

maxruby commented 10 years ago

Although the function is in src/ffmpeg/AVUtil/v54/log.jl, it is likely not being called by LIBAVUTIL.jl. (this is addressed in the the pending PR).

Go to src/ffmpeg/AVUtil/v54/LIBAVUTIL.jl, and make sure to uncomment #include("log.jl").

lucasb-eyer commented 10 years ago

Thanks, that works, though in case #47 is still far from being merged, I'd vote for doing that change upfront.

maxruby commented 10 years ago

Adding a fix for #49 alone should be no problem. I will wait for Kevin´s feedback regarding the PR and further changes.

kmsquire commented 9 years ago

Should be fixed by 26b301cb. Please reopen if not (and sorry for the slow response).