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

Added --verbose and --output options #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ethouris
Copy link

@ethouris ethouris commented Sep 4, 2020

Note: --verbose works only with a version of srt-test-live that supports -v ./filename specification.

@mbakholdina
Copy link
Owner

@ethouris, could you please share an example of how you launch the script for both receiver and sender? So that I understand more what do you pass as new options values and can test it on my side.

Regarding srt-test-live with -v ./filename support, is it in latest master of srt repo?

@mbakholdina mbakholdina self-assigned this Sep 4, 2020
@mbakholdina mbakholdina modified the milestones: v1.5.0 - Sprint 22, v1.5.0 - Sprint 23 Sep 4, 2020
@ethouris
Copy link
Author

ethouris commented Sep 7, 2020

The -v ./filename is already added to the latest master, likely today. The "./" prefix is obligatory because normally the argument is "1" or 2" (to mean stdout and stderr respectively).

The new options added to the script:

--verbose <filename> - add the -v ./filename option to srt-test-live in order to save the verbose info into the file. (<filename> should be in the local directory, ./ prefix is added by the script, so this should be only the name itself).

--output <spec> - changes the default file://con output into a different specification.

@mbakholdina mbakholdina modified the milestones: v1.5.0 - Sprint 23, v1.5.0 - Sprint 24 Sep 21, 2020
@mbakholdina
Copy link
Owner

@ethouris I still do not get the idea how have you changed the application so that it works in this way. What's output? In code it's filepath, here it's spec. Could you please provide more explanation?

@mbakholdina mbakholdina removed this from the v1.5.0 - Sprint 24 milestone Oct 14, 2020
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

Successfully merging this pull request may close these issues.

2 participants