Skip to content

CLI enhancements #1

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

Merged
merged 4 commits into from
Feb 24, 2024
Merged

CLI enhancements #1

merged 4 commits into from
Feb 24, 2024

Conversation

fotonick
Copy link
Contributor

Please consider merging these commits, which are minor fixes and enhancements.

Note that the --quiet is for scripting with less line noise. For example, I have:

CAMERA_PASSWORD=$(op read "op://Shared/IP Camera/password")  # 1password
STREAM_URL=$(CAMERA_PASSWORD=$CAMERA_PASSWORD poetry run mipc_camera_client --host 192.168.99.221 --user 1jfiegbqwpjre --quiet stream)
/Applications/VLC.app/Contents/MacOS/VLC --no-drop-late-frames $STREAM_URL &

Manually tested all three commands with and without -q.

Thanks for this very helpful library and tool. I know the pain of interpreting and cleanly reimplementing extracted/minified code.

cli.py uses a match statement, which was introduced in Python 3.10.
In particular, exit before initiating a connection that is doomed to fail.
@pan-maruda
Copy link
Owner

thank you, appreciate it, I'll release it soon!

@pan-maruda pan-maruda merged commit d5466f7 into pan-maruda:main Feb 24, 2024
@pan-maruda
Copy link
Owner

@fotonick released as 0.1.3 📦 🚢

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