[Slackbuilds-users] ZoneMinder build fails if ffmpeg is not installed

[Daniel Prosser]

Willy,

Here are three patches. The first one you should be able to apply to
ZoneMinder in the SBo git. The second two are patches executed by the
SlackBuild. fix_video_file_format.patch is the one I mentioned
previously that was needed on my system for ZoneMinder to encode mp4
videos for events. fix_zone_area_calc.patch applies upstream commits
4937a686 and 4da95369. I also added ffmpeg, libmp4v2, and x264 as
dependencies and noted php-apcu as an optional dependency. Hopefully
the maintainer will also weigh in on these changes.

Dan

On Fri, Jan 25, 2019 at 08:33:46PM +0700, Willy Sudiarto Raharjo wrote:
> > One final (I think) note: if you want to use the ZoneMinder API, you also
> > need to have  php-apcu installed. It would be good to document these things
> > so that users don't have to dig to find it.
> 
> >> Thanks Willy. A couple more notes/issues/fixes for this new version:
> >>
> >> 1. By default, ZoneMinder now saves events as mp4 videos instead of
> >> frames. For this to work, x264 and libmp4v2 must be installed prior to
> >> building. It's still possible to just use image frames as before, but I
> >> think x264 and libmp4v2 need to be at least optional dependencies.
> >> Personally, I think they should be required, as otherwise the user will
> >> have to switch to saving image frames only.
> >>
> >> 2. Even with x264 and libmp4v2 installed, ZoneMinder was still unable to
> >> save videos for me. I had to apply the attached patch for it to work.
> >> I'm not really sure why that's the case, since I didn't find any other
> >> reports of it not working as is. GenoBob, perhaps you and/or any other
> >> Slackware ZoneMinder users can report whether it works for you (and add
> >> this patch to the SlackBuild if appropriate)?
> >>
> >> 3. This version stores events and other data in ZM_CONTENTDIR, which
> >> defaults to /var/lib/zoneminder. However, the SlackBuild script doesn't
> >> create this directory, so there are initially errors when running until
> >> the user manually creates it. (It needs to be owned by apache:apache). I
> >> recommend having the SlackBuild script create this directory as part of the
> >> package, and make it a build option so it can be changed at build time.
> >>
> >> Don't take these comments as a criticism. I am really grateful for the
> >> script and I'm glad that it's basically working for me now. I was actually
> >> procrastinating upgrading to 1.32.x because of all the changes. I just
> >> want to help make the installation process as smooth as possible for
> >> others.
> 
> Hi Dan
> 
> Thanks for this investigations
> 
> would you mind sending me the patch for this improvements?
> 
> 
> 
> -- 
> Willy Sudiarto Raharjo
> 



-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-Add-ZoneMinder-patches-CONTENTDIR-and-update-require.patch
URL: <http://lists.slackbuilds.org/pipermail/slackbuilds-users/attachments/20190129/b97b6565/attachment.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: fix_video_file_format.patch
URL: <http://lists.slackbuilds.org/pipermail/slackbuilds-users/attachments/20190129/b97b6565/attachment-0001.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: fix_zone_area_calc.patch
URL: <http://lists.slackbuilds.org/pipermail/slackbuilds-users/attachments/20190129/b97b6565/attachment-0002.ksh>