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: localhorst: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

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
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

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

No dependencies set.

Reference: localhorst/media_management_scripts#3
No description provided.
Delete Branch "hannesbraun/media_management_scripts:allow-whitespace"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?