Created attachment 225331 [details]
Remove the dependency on ffmpeg
Actually ffmpeg is used for video delegation when it is found, but this is not a real dependency.
Before ImageMagick7 doesn't have dependency from ffmpeg.
Why in this update added?
Could you please remove this dependency?
(I use exactly for convert images, not for audio/video decoding)
Thanks in advance.
Thanks for the patch! I'm glad it was a simple change, as I had visions of having to tweak a configuration script. The patch works for me!
graphics/ImageMagick7-nox11 usually used on servers for image conversion and now it require a lot of libs for processing video/audio. It's not a good idea.
Please remove ffmpeg dependency or make it optional.
Created attachment 225520 [details]
Add FFMPEG option
Attach a patch that adds the FFMPEG option.
I did a build test on poudriere (amd64).
Created attachment 225740 [details]
graphics/ImageMagick6 version of attachment 225520 [details]
Tested with # portmaster -fr and yielded a successful build without ffmpeg.
Starting with Toshimichi Masubuchi's patch for graphics/ImageMagick7, modified for graphics/ImageMagick6.
Seconded. Please do not depend on video conversion tools by default (having an option FFMPEG which is off by default is fine)
I wanted to do package ugprades on a server today, and that's what I get in new dependencies
$ SU pkg update ImageMagick7
New packages to be INSTALLED:
Installed packages to be UPGRADED:
ImageMagick7: 22.214.171.124_3 -> 126.96.36.199
Committed after maintainer’s time-out, thanks for the patches!
Note: patches modified to not enable this option by default, like many people requested.
A commit in branch main references this bug:
Author: Thierry Thomas <thierry@FreeBSD.org>
AuthorDate: 2021-06-18 15:29:26 +0000
Commit: Thierry Thomas <thierry@FreeBSD.org>
CommitDate: 2021-06-18 15:39:42 +0000
graphics/ImageMagick[6-7]: optionize the dependency on ffmpeg
- Make an option for FFMPEG, and do not enable it by default.
- Do not bump PORTREVISION: pkg will take care of it, and ports users
may rebuild it if needed.
- Note: there is no simple way to deactivate this option when ffmpeg is
Reported by: Toshimichi Masubuchi (ImageMagick7) and gessel (6)
Approved by: maintainer’s time-out
graphics/ImageMagick6/Makefile | 12 +++++++++---
graphics/ImageMagick7/Makefile | 11 ++++++++---
2 files changed, 17 insertions(+), 6 deletions(-)
(In reply to Thierry Thomas from comment #9)
Thank you for your support!