-
Notifications
You must be signed in to change notification settings - Fork 161
Open
Labels
Type: Feature RequestA request for a new feature that can be included with the next minor version release.A request for a new feature that can be included with the next minor version release.
Description
Original issue opened by:
Describe the problem
The WindowsTextToSpeechSubsystem is inside the com.microsoft.mrtk.windowsspeech package, and WindowsTextToSpeechSubsystem requires com.microsoft.mrtk.tts.windows. Given this strong dependency, and app is required to include com.microsoft.mrtk.tts.windows even if it doesn't require text-to-speech (TTS).
Describe the solution you'd like
Either update com.microsoft.mrtk.windowsspeech so that it disables TTS code paths if the com.microsoft.mrtk.tts.windows package is not included. Or move WindowsTextToSpeechSubsystem into a different package.
Describe alternatives you've considered
- Leave everything as is, and always require
com.microsoft.mrtk.tts.windows. - Move
com.microsoft.mrtk.tts.windowsintocom.microsoft.mrtk.windowsspeech
Additional context
It's unclear as to why WindowsTextToSpeechSubsystem wasn't in com.microsoft.mrtk.tts.windows in the first place.
ISSUE MIGRATION
Issue migrated from: microsoft/MixedRealityToolkit-Unity#11453
Metadata
Metadata
Assignees
Labels
Type: Feature RequestA request for a new feature that can be included with the next minor version release.A request for a new feature that can be included with the next minor version release.