bug#72827: [PATCH] Adds NSSpeechRecognitionUsageDescription to Info.plist.in

[Alvaro Ramirez <alvaro@xenodium.com>]

[-- Attachment #1: Type: text/plain, Size: 1873 bytes --]

Thanks Stefan

Stefan Kangas <stefankangas@gmail.com> writes:

> Alvaro Ramirez <alvaro@xenodium.com> writes:
>
>> ps. My copyright assignment paperwork has been is
>> submitted. Awaiting copyright-clerk reply.
>
> Thanks for your contributions to Emacs.
>
>
> The Apple documentation[1] says that:
>
>     A message that tells the user why the app is requesting to 
>     send user
>     data to Apple’s speech recognition servers.
>
> So I think the description should make it clear that this has 
> privacy
> implications, for example by saying something like "Emacs 
> requires
> permission to to send user data to Apple’s speech recognition 
> servers".

Indeed. That notice is already part of the system dialog 
itself. The full text looks as follows:

---
"Emacs.app" Would Like to
Access Speech Recognition.

Speech data from this app will be sent to
Apple to process your requests. This will
also help Apple improve its speech
recognition technology.

Emacs requires permission to handle any
speech recognition.

[Don't Allow] [Allow]
---

Also attaching a screenshot.

> The text in NEWS should also be changed accordingly.

Sounds good. Will update NEWS.

>
> It's fine that the user is asked, instead of the process just 
> getting
> silently terminated, but we do have to give the user enough 
> information
> to make an informed choice here.

Makes sense. The system dialog already includes these details, but 
happy to modify text in our control further if we still reckon 
it's needed.
>
> Since this has potential privacy (and security?) implications, I 
> think
> this should go to master.
>
>>  </dict>
>>  </plist>
>> --
>> 2.39.3 (Apple Git-146)
>
> [1] 
> https://developer.apple.com/documentation/bundleresources/information_property_list/nsspeechrecognitionusagedescription

[-- Attachment #2: speechrec-permission-dialog.png --]
[-- Type: image/png, Size: 262751 bytes --]