Throw errors in ServiceManager by eliseomartelli · Pull Request #188 · apple/container
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 677
Merged
adityaramani merged 2 commits intoapple:mainfrom
Jun 14, 2025Merged
Throw errors in ServiceManager#188
adityaramani merged 2 commits intoapple:mainfrom
Throw errors in ServiceManager#188
adityaramani merged 2 commits intoapple:mainfrom
Conversation
makhov reviewed Jun 13, 2025
Sources/ContainerPlugin/ServiceManager.swift
Outdated
Show resolved
Hide resolved
Sources/ContainerPlugin/ServiceManager.swift Outdated Show resolved Hide resolved
- Redirect launchctl's standard error to a pipe instead of null device. - Include stderr output in thrown ContainerizationError messages when launchctl fails or output cannot be decoded. - Provides informative error messages for debugging launchctl invocation issues.
adityaramani approved these changes Jun 13, 2025
Copy link
Contributor
adityaramani
commented
Jun 13, 2025
adityaramani commented
Jun 13, 2025Thanks for the fix!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants