Skip to content

fix: separate path and error in the ffprobe log message - #2288

Open
tankxbvr wants to merge 1 commit into
xbapps:masterfrom
tankxbvr:fix/ffprobe-log-message
Open

tankxbvr wants to merge 1 commit into
xbapps:masterfrom
tankxbvr:fix/ffprobe-log-message

Conversation

@tankxbvr

Copy link
Copy Markdown
Contributor

Path and error ran together in the log, so it read ffprobe/videos/foo.mp4exit status 1. Uses Errorf now, matching the line above it.

fmt.Sprint adds no space between a string and an error, so the message
printed as "ffprobe/videos/foo.mp4exit status 1".
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.

1 participant