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

Add support for command line parameters and reading commands from config file. #5

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

Conversation

gfto
Copy link
Contributor

@gfto gfto commented Nov 21, 2014

Configuring obe using 'screen' hacks is not the best way for setting up the encoder, so I've added support for reading the commands from config file. This allows me to run obe using process supervision which is a nice thing (not that obe crashes but better safe than sorry).

Another small feature is --full-help parameter that lists what commands obe supports.

Georgi Chorbadzhiyski added 3 commits November 21, 2014 14:19
This patch adds support for running OBE like any other program
that needs configuration. Save the commands that you usually
type into OBE shell in regular text file and then run
'obecli --config-file=my.conf' and you are good to go.

NOTE: There is possible change in behaviour! After this patch
the interactive shell is run only if there are no command line
parameters after 'obecli' or '-s'/'--shell' parameter is used.
@midicase
Copy link

midicase commented Aug 8, 2019

I agree, config file is a nice start. Though I might go one step further and put a proper API on the system. The packages are a bit dated though. Project looks abandoned

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