Message ID | 1413749285-1108-25-git-send-email-thomas.petazzoni@free-electrons.com |
---|---|
State | Accepted |
Commit | ae2850b78c15c3c848bb01f723da439df8dfe68e |
Headers | show |
Thomas, All, On 2014-10-19 22:07 +0200, Thomas Petazzoni spake thusly: > ffmpeg currently uses BR2_GCC_TARGET_TUNE as the --cpu option > value. However, there are multiple architectures for which > BR2_GCC_TARGET_TUNE is not used. After inspecting the ffmpeg configure > script, we concluded that using the BR2_GCC_TARGET_CPU value if > available, or the BR2_GCC_TARGET_ARCH value as a fallback was the > appropriate behavior. > > This allows to remove the reference to BR2_GCC_TARGET_TUNE, which is > one step towards the removal of this option. > > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Regards, Yann E. MORIN. > --- > package/ffmpeg/ffmpeg.mk | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/package/ffmpeg/ffmpeg.mk b/package/ffmpeg/ffmpeg.mk > index a3865f3..6c8b899 100644 > --- a/package/ffmpeg/ffmpeg.mk > +++ b/package/ffmpeg/ffmpeg.mk > @@ -334,6 +334,12 @@ endif > > FFMPEG_CONF_OPTS += $(call qstrip,$(BR2_PACKAGE_FFMPEG_EXTRACONF)) > > +ifneq ($(call qstrip,$(BR2_GCC_TARGET_CPU)),) > +FFMPEG_CONF_OPTS += --cpu=$(BR2_GCC_TARGET_CPU) > +else ifneq ($(call qstrip,$(BR2_GCC_TARGET_ARCH),) > +FFMPEG_CONF_OPTS += --cpu=$(BR2_GCC_TARGET_ARCH) > +endif > + > # Override FFMPEG_CONFIGURE_CMDS: FFmpeg does not support --target and others > define FFMPEG_CONFIGURE_CMDS > (cd $(FFMPEG_SRCDIR) && rm -rf config.cache && \ > @@ -349,7 +355,6 @@ define FFMPEG_CONFIGURE_CMDS > --target-os="linux" \ > --disable-stripping \ > --pkg-config="$(PKG_CONFIG_HOST_BINARY)" \ > - $(if $(BR2_GCC_TARGET_TUNE),--cpu=$(BR2_GCC_TARGET_TUNE)) \ > $(SHARED_STATIC_LIBS_OPTS) \ > $(FFMPEG_CONF_OPTS) \ > ) > -- > 2.0.0 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
diff --git a/package/ffmpeg/ffmpeg.mk b/package/ffmpeg/ffmpeg.mk index a3865f3..6c8b899 100644 --- a/package/ffmpeg/ffmpeg.mk +++ b/package/ffmpeg/ffmpeg.mk @@ -334,6 +334,12 @@ endif FFMPEG_CONF_OPTS += $(call qstrip,$(BR2_PACKAGE_FFMPEG_EXTRACONF)) +ifneq ($(call qstrip,$(BR2_GCC_TARGET_CPU)),) +FFMPEG_CONF_OPTS += --cpu=$(BR2_GCC_TARGET_CPU) +else ifneq ($(call qstrip,$(BR2_GCC_TARGET_ARCH),) +FFMPEG_CONF_OPTS += --cpu=$(BR2_GCC_TARGET_ARCH) +endif + # Override FFMPEG_CONFIGURE_CMDS: FFmpeg does not support --target and others define FFMPEG_CONFIGURE_CMDS (cd $(FFMPEG_SRCDIR) && rm -rf config.cache && \ @@ -349,7 +355,6 @@ define FFMPEG_CONFIGURE_CMDS --target-os="linux" \ --disable-stripping \ --pkg-config="$(PKG_CONFIG_HOST_BINARY)" \ - $(if $(BR2_GCC_TARGET_TUNE),--cpu=$(BR2_GCC_TARGET_TUNE)) \ $(SHARED_STATIC_LIBS_OPTS) \ $(FFMPEG_CONF_OPTS) \ )
ffmpeg currently uses BR2_GCC_TARGET_TUNE as the --cpu option value. However, there are multiple architectures for which BR2_GCC_TARGET_TUNE is not used. After inspecting the ffmpeg configure script, we concluded that using the BR2_GCC_TARGET_CPU value if available, or the BR2_GCC_TARGET_ARCH value as a fallback was the appropriate behavior. This allows to remove the reference to BR2_GCC_TARGET_TUNE, which is one step towards the removal of this option. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> --- package/ffmpeg/ffmpeg.mk | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)