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
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.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b hannesbraun-allow-whitespace master
git pull allow-whitespace

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff hannesbraun-allow-whitespace
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

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?