localhorst/media_management_scripts
0
0
Fork 1
Code Issues Pull Requests 4 Projects Releases Wiki Activity

codecVis: allow whitespace in file path #3

Open
hannesbraun wants to merge 1 commits from hannesbraun/media_management_scripts:allow-whitespace into master
pull from: hannesbraun/media_management_scripts:allow-whitespace
merge into: :master
Conversation 0 Commits 1 Files Changed 1
+3 -3
hannesbraun commented 2023-01-01 19:22:23 +01:00
First-time contributor
Copy Link
Copy Source

If the path of an input file contains any whitespace characters, the codec visualizer script wouldn't work until now because that path contained in the ffmpeg command for extracting a frame wouldn't be escaped.

By not using shlex at all, we can circumvent this.

If the path of an input file contains any whitespace characters, the codec visualizer script wouldn't work until now because that path contained in the ffmpeg command for extracting a frame wouldn't be escaped. By not using shlex at all, we can circumvent this.
hannesbraun added 1 commit 2023-01-01 19:22:24 +01:00
codecVis: allow whitespace in file path 6949d4cecf
You are not authorized to merge this pull request.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u https://git.mosad.xyz/hannesbraun/media_management_scripts allow-whitespace:hannesbraun-allow-whitespace
git checkout hannesbraun-allow-whitespace
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Seil0 (Jannik) localhorst (Hendrik Schutter)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: localhorst/media_management_scripts#3