ccextractor icon indicating copy to clipboard operation
ccextractor copied to clipboard

[Bug] (version .94) parameter -inp=ts not understood

Open egossett opened this issue 3 years ago • 9 comments

Please prefix your issue with one of the following: [BUG], [PROPOSAL], [QUESTION].

I have a .ts file that contains programs 3, 4, and 6. I need subtitles for program 4. I set the program number to 4 in "Obscure settings" and changed from "auto/default" to "ts" in "Basic settings"/input file format.

I get the error: parameter -inp=ts not understood

If I use "auto/default" for the input file format, the program number is ignored.

egossett avatar Jun 03 '22 17:06 egossett

The parameter should be -in=ts

egossett avatar Jun 03 '22 18:06 egossett

That's a GUI issue (I believe you're using the GUI?), so @Techno-Disaster...

canihavesomecoffee avatar Jun 03 '22 18:06 canihavesomecoffee

Yes, it is a GUI issue. Too bad the gui doesn't let you edit the command line parameters in the basic settings tab. Should be a 2-minute fix.

egossett avatar Jun 03 '22 18:06 egossett

strange -in is -in according to https://github.com/CCExtractor/ccextractorfluttergui/blob/master/lib/models/settings_model.dart#L227

are you on the latest gui version? Also what do you see in the current commands box? Does that say -inp=ts as well?

techno-disaster avatar Jun 04 '22 06:06 techno-disaster

I have the latest version. The command box shows "inp=ts". If I run CCExtractor from the command line with "inp" replaced by "in", it works. That means the GUI has a small error in creating the correct command line.

egossett avatar Jun 04 '22 11:06 egossett

https://github.com/CCExtractor/ccextractorfluttergui/commit/155596b2633ef3be9050a70e8c4ba223eb5d2867 should fix it :D

techno-disaster avatar Jun 05 '22 20:06 techno-disaster

Thank you very much for the fix. Since I am running this on Windows 10 and not a Unix system, I will need to wait for the next non-source update to use this fix.

egossett avatar Jun 05 '22 21:06 egossett

Once the workflows finish you can normally download the artifacts for Windows, and replace the GUI exe in your installation folder. If you don't want to do that, yes, you'll have to wait until a new release is made.

canihavesomecoffee avatar Jun 05 '22 23:06 canihavesomecoffee

Ok. I will try that.

On Sun, Jun 5, 2022, 6:26 PM Willem @.***> wrote:

Once the workflows finish you can normally download the artifacts for Windows, and replace the GUI exe in your installation folder. If you don't want to do that, yes, you'll have to wait until a new release is made.

— Reply to this email directly, view it on GitHub https://github.com/CCExtractor/ccextractor/issues/1437#issuecomment-1146905876, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJLTSKEFRJJBUFWPC5AQRI3VNUZR3ANCNFSM5XZUMA6Q . You are receiving this because you authored the thread.Message ID: @.***>

egossett avatar Jun 05 '22 23:06 egossett

Closing since this was fixed - @egossett free feel to reopen if you still have issues with the last GUI version.

cfsmp3 avatar Mar 22 '23 04:03 cfsmp3