Re: [FFmpeg-devel] [PATCH] lavc/audiotoolbox: fix build failure on osx 10.5 (v2)

From: Marvin Scholz <epirat07@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH] lavc/audiotoolbox: fix build failure on osx 10.5 (v2)
Date: Sat, 15 Jul 2023 18:11:36 +0200
Message-ID: <CE46BF81-D30C-4BEB-A70A-F964CE34ACAA@gmail.com> (raw)
In-Reply-To: <20230715154055.21035-1-pkoshevoy@gmail.com>

On 15 Jul 2023, at 17:40, Pavel Koshevoy wrote:

> ---
>  libavcodec/audiotoolboxdec.c | 2 ++
>  libavcodec/audiotoolboxenc.c | 4 ++++
>  2 files changed, 6 insertions(+)
>

Hi, thanks for the patch, small remark below:

> diff --git a/libavcodec/audiotoolboxdec.c b/libavcodec/audiotoolboxdec.c
> index 82babe3d31..e336aade8b 100644
> --- a/libavcodec/audiotoolboxdec.c
> +++ b/libavcodec/audiotoolboxdec.c
> @@ -71,10 +71,12 @@ static UInt32 ffat_get_format_id(enum AVCodecID codec, int profile)
>          return kAudioFormatAMR;
>      case AV_CODEC_ID_EAC3:
>          return kAudioFormatEnhancedAC3;
> +#if __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ >= 1060
>      case AV_CODEC_ID_GSM_MS:
>          return kAudioFormatMicrosoftGSM;
>      case AV_CODEC_ID_ILBC:
>          return kAudioFormatiLBC;
> +#endif
>      case AV_CODEC_ID_MP1:
>          return kAudioFormatMPEGLayer1;
>      case AV_CODEC_ID_MP2:
> diff --git a/libavcodec/audiotoolboxenc.c b/libavcodec/audiotoolboxenc.c
> index 1ccfda4d20..e5217d952d 100644
> --- a/libavcodec/audiotoolboxenc.c
> +++ b/libavcodec/audiotoolboxenc.c
> @@ -69,15 +69,19 @@ static UInt32 ffat_get_format_id(enum AVCodecID codec, int profile)
>              return kAudioFormatMPEG4AAC_HE_V2;
>          case FF_PROFILE_AAC_LD:
>              return kAudioFormatMPEG4AAC_LD;
> +#if __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ >= 1060

I think you should use __MAC_OS_X_VERSION_MIN_REQUIRED, as the __ENVIRONMENT_* is
an implementation detail and not documented IIRC. (Same below)

>          case FF_PROFILE_AAC_ELD:
>              return kAudioFormatMPEG4AAC_ELD;
> +#endif
>          }
>      case AV_CODEC_ID_ADPCM_IMA_QT:
>          return kAudioFormatAppleIMA4;
>      case AV_CODEC_ID_ALAC:
>          return kAudioFormatAppleLossless;
> +#if __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ >= 1060
>      case AV_CODEC_ID_ILBC:
>          return kAudioFormatiLBC;
> +#endif
>      case AV_CODEC_ID_PCM_ALAW:
>          return kAudioFormatALaw;
>      case AV_CODEC_ID_PCM_MULAW:
> -- 
> 2.35.3
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".