Skip to content

Conversation

@ann0see
Copy link
Member

@ann0see ann0see commented Dec 21, 2025

QTMultimedia seems to require camera access permissions even though we do not request this. As it's in the code, AppStore connect requests a description. It is safe to NOT allow access.

Some time ago, I tracked it down to QTMultimedia: https://doc.qt.io/qt-6.5/qtmultimedia-apple.html

Short description of changes

CHANGELOG: SKIP

Context: Fixes an issue?

Related to: #3406 (loosely)
Does this change need documentation? What needs to be documented and how?

Status of this Pull Request

Ready for review
What is missing until this pull request can be merged?
Link source

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

@ann0see ann0see requested review from pljones and softins December 22, 2025 14:12
@ann0see ann0see added the iOS iOS runtime issue label Dec 22, 2025
@ann0see ann0see added this to Tracking Dec 22, 2025
@github-project-automation github-project-automation bot moved this to Triage in Tracking Dec 22, 2025
@ann0see ann0see moved this from Triage to Waiting on Team in Tracking Dec 22, 2025
@ann0see ann0see added this to the Release 3.12.0 milestone Dec 22, 2025
QTMultimedia seems to require camera access permissions even though we do not request this. As it's in the code, AppStore connect requests a description. It is safe to NOT allow access.
@ann0see ann0see force-pushed the iOS/addAccessibilityCamDex branch from f76c1b0 to 0d3c577 Compare December 22, 2025 14:20
Jamulus does not use encryption
@ann0see
Copy link
Member Author

ann0see commented Dec 22, 2025

Added encryption exemption flag too. This should make submitting to App Store Connect easier.

@ann0see ann0see changed the title Add camera usage description for iOS Add camera usage description and Encryption Exception for iOS Dec 22, 2025
@ann0see ann0see merged commit 630dfe9 into jamulussoftware:main Dec 22, 2025
9 of 10 checks passed
@github-project-automation github-project-automation bot moved this from Waiting on Team to Done in Tracking Dec 22, 2025
@ann0see ann0see deleted the iOS/addAccessibilityCamDex branch December 22, 2025 14:24
@pljones
Copy link
Collaborator

pljones commented Dec 22, 2025

Can we document somewhere what you wrote above? It's not always easy to find explanations in pull requests. I hesitate to say it should be in the code... Maybe Jamulus.pro for iOS could mention it? Somewhere the dependency gets pulled in, really.

@ann0see
Copy link
Member Author

ann0see commented Dec 22, 2025

deploy_ios.sh ?

QT multimedia gives a huge headache for iOS from my gut feeling.

  1. Switching to FFMPEG breaks the app Add workaround from QTBUG-129651 for iOS FFMPEG linking #3581
  2. It requests camera access for no reason
  3. It introduces a dependency which can change at any time
    I'd be inclined to ifdef it out on iOS (but this breaks the broken accessibility on iOS even more)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

iOS iOS runtime issue

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants