Skip to content

Declares camera usage in Info.plist - #400

Merged
gr5 merged 1 commit into
githubdoe:masterfrom
Lucassifoni:MacOS_fix_camera_permissions
Nov 3, 2026
Merged

gr5 merged 1 commit into
githubdoe:masterfrom
Lucassifoni:MacOS_fix_camera_permissions

Conversation

@Lucassifoni

Copy link
Copy Markdown
Contributor

Adds a necessary capability declaration in the generated Info.plist so MacOS doesn't crash on selecting a video source

@gr5

gr5 commented Oct 27, 2026

Copy link
Copy Markdown
Collaborator

Oh! Should we create an 8.5.2 already?

Have you tried connecting a web cam or using a laptop built in camera? You can use all the features just by pointing the camera at... anything. You can create wavefronts (even though they are meaningless you can still confirm all the features work).

@Lucassifoni

Copy link
Copy Markdown
Contributor Author

Oh! Should we create an 8.5.2 already?

Have you tried connecting a web cam or using a laptop built in camera? You can use all the features just by pointing the camera at... anything. You can create wavefronts (even though they are meaningless you can still confirm all the features work).

Not yet, it was the first time I was trying the video feature. It seems that any kind of live video access on Mac OS crashes the program if it doesn't declare the video acquisition capability... so it seems that a patch release would be needed once my PR is merged and the feature tested on mac.

@Lucassifoni

Copy link
Copy Markdown
Contributor Author

Hi, the windows build failing is due to permissions like on all the PRs open from my fork.

@gr5

gr5 commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

@Lucassifoni - sorry I forgot about this PR. Go ahead and merge this if you think it's ready. Let me know if I should create a new release (8.5.2).

@Lucassifoni

Copy link
Copy Markdown
Contributor Author

Hi @gr5, I don't have any right on the repo, it would be up to you to merge. Then I'll retry the macos build with the added capabilities.

@gr5
gr5 merged commit 24b5a52 into githubdoe:master Nov 3, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants