Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

good job! but there is a bug #4

Open
gabin22 opened this issue Apr 23, 2021 · 5 comments
Open

good job! but there is a bug #4

gabin22 opened this issue Apr 23, 2021 · 5 comments

Comments

@gabin22
Copy link

gabin22 commented Apr 23, 2021

the bug is about \n,when I open time_pairs.txt with vim, the \n was show in text,that's wrong.with this bug,you can only cut one segment, no two or more segments,because parsing \n is wrong.I don't know about lua and shell,so I change \n to a special char,like:

cutter.lua : line 82 , change "\n" to ";"
c_concat.sh: line 39 ,change "\n" to ";"

now I can cut two segments.
以上。

@rushmj
Copy link
Owner

rushmj commented Apr 24, 2021

which os do you use? I use mac os,it has no problem,\n was not found in vim.

@gabin22
Copy link
Author

gabin22 commented Apr 25, 2021

manjaro linux.

cut rmvb file fail.would you support rmvb?

@rushmj
Copy link
Owner

rushmj commented Apr 26, 2021

manjaro linux.

cut rmvb file fail.would you support rmvb?

This script convert video with FFmpeg. maybe original FFmpeg has problem when support rmvb.

@fastrizwaan
Copy link

fastrizwaan commented Nov 11, 2021

in Linux, it is not working with \n;

cutter.lua: line 90
change
if (k%2==1) then str=str..v..',' else str=str..v..'\\n' to
if (k%2==1) then str=str..v..',' else str=str..v..';' end

c_concat.sh: line 39
change "\n" to ";"

IFS=$'\n' a=(`eval cat $time_pairs_txt`)

to

IFS=$';' a=(`eval cat $time_pairs_txt`)

Thank you @gabin22 for the fix!

the error output for more than 1 or multiple cut segments is:

Invalid duration specification for t: -c
cp: cannot stat '.cut_video/clip0.mp4': No such file or directory

@fastrizwaan
Copy link

Thank you guys, I've forked it for flatpak https://github.com/fastrizwaan/mpv-video-cutter-linux

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

No branches or pull requests

3 participants