From d39ac6a4ec08bcf4b53acf53c603c715cfd595b8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nils=20Petter=20Ska=CC=8Alerud?=
 <nils.petter.skalerud@qt.io>
Date: Tue, 25 Mar 2025 13:06:37 +0100
Subject: [PATCH] iOS, FFmpeg: Add variable for controlling minimum iOS version

Currently this script relies on hard-coded values specify minimum
iOS version to link against.

This patch unifies the hard-coded values into a single variable.

No functional changes.

Pick-to: 6.9 6.8
Change-Id: I12bddeb53f0fe3388ea55c841a9864722a1d5f82
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
---
 coin/provisioning/common/unix/install-ffmpeg-ios.sh | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/coin/provisioning/common/unix/install-ffmpeg-ios.sh b/coin/provisioning/common/unix/install-ffmpeg-ios.sh
index 153e4194..abdfbc1a 100755
--- a/coin/provisioning/common/unix/install-ffmpeg-ios.sh
+++ b/coin/provisioning/common/unix/install-ffmpeg-ios.sh
@@ -5,6 +5,10 @@
 # This script will build and install FFmpeg static libs
 set -ex
 
+# Must match or be lower than the minimum iOS version supported by the version of Qt that is
+# is currently being built.
+readonly MINIMUM_IOS_VERSION="16.0"
+
 source "${BASH_SOURCE%/*}/../unix/ffmpeg-installation-utils.sh"
 
 ffmpeg_source_dir=$(download_ffmpeg)
@@ -18,15 +22,15 @@ build_ffmpeg_ios() {
     if [ "$target_platform" == "arm64-simulator" ]; then
         target_sdk="iphonesimulator"
         target_cpu_arch="arm64"
-        minos="-mios-simulator-version-min=16.0"
+        minos="-mios-simulator-version-min=$MINIMUM_IOS_VERSION"
     elif [ "$target_platform" == "x86_64-simulator" ]; then
         target_sdk="iphonesimulator"
         target_cpu_arch="x86_64"
-        minos="-mios-simulator-version-min=16.0"
+        minos="-mios-simulator-version-min=$MINIMUM_IOS_VERSION"
     elif [ "$target_platform" == "arm64-iphoneos" ]; then
         target_sdk="iphoneos"
         target_cpu_arch="arm64"
-        minos="-miphoneos-version-min=16.0"
+        minos="-miphoneos-version-min=$MINIMUM_IOS_VERSION"
     else
         echo "Error when building FFmpeg for iOS. Unknown parameter given for target_platform: '${target_platform}'"
         exit 1
@@ -89,7 +93,6 @@ build_info_plist() {
     local framework_id="$3"
 
     local minimum_version_key="MinimumOSVersion"
-    local minimum_os_version="16.0"
     local supported_platforms="iPhoneOS"
 
     # TODO: This should be filled out with the actual version of FFmpeg that we are
@@ -117,7 +120,7 @@ build_info_plist() {
     <key>CFBundleSignature</key>
     <string>????</string>
     <key>${minimum_version_key}</key>
-    <string>${minimum_os_version}</string>
+    <string>${MINIMUM_IOS_VERSION}</string>
     <key>CFBundleSupportedPlatforms</key>
     <array>
         <string>${supported_platforms}</string>