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

Feature request: command line options #2

Closed
jakimfett opened this issue Apr 27, 2016 · 14 comments
Closed

Feature request: command line options #2

jakimfett opened this issue Apr 27, 2016 · 14 comments

Comments

@jakimfett
Copy link

I'd like to request the ability to skip the pauses (as mentioned here when executing the program via a command line flag, eg cat somefile.txt | bin/nms --no-pause.

@bartobri
Copy link
Owner

Hi and thanks for the interest. So I have two pauses. The first one can possibly be skipped (maybe making it sleep for a second or two for effect), but the second pause is necessary for the user to actually read what the text says, otherwise it would immediately disappear after displaying. And in the case of displaying a menu, the pause is necessary to get user input.

@jakimfett
Copy link
Author

Ah, good point. Would it be possible to add a flag for just the first pause, so that text is displayed garbled, then immediately run through the "decrypting" effect?

A bit about what I'm trying to do...I have an old style typewriter hooked up to a raspberry pi, running a shell script that activates one of the GPIO pins when the correct key phrase is typed in. When incorrect input is typed in, the script will just display some variation of "incorrect input, please try again", but when the correct key phrase is typed in, some story type text gets printed on the screen after "decrypting" via the no-more-secrets effect.

@bartobri
Copy link
Owner

Skipping the first pause via command line option should be pretty easy to do. I'll add that option when I have some time.

@jakimfett
Copy link
Author

Fantastic! Do you have a Patreon or Paypal donation option? I'd like to send you a bit of a financial "thank you", this is a really fun little utility!

@bartobri
Copy link
Owner

Not necessary but thanks for the offer :)

@kryptonianson
Copy link

I would like the ability to set the speed of the "decryption" process on the command line. It is a bit too slow for me.

@shaneog
Copy link

shaneog commented Apr 28, 2016

Rather than --no-pause, how about --auto-decrypt for clarity?

@jakimfett
Copy link
Author

Works for me. I'm less concerned about what I type in, as long as it allows me to eliminate the pause.

@kryptonianson maybe open a new ticket for your additional, completely different, feature request?

@apjanke
Copy link
Contributor

apjanke commented May 1, 2016

Could we also get a -v or --version to output the version number instead of doing the effect display? Would be useful for Homebrew or other packagers, where we like to do at least a trivial execution test to confirm the executable was built and linked correctly.

@bartobri
Copy link
Owner

bartobri commented May 1, 2016

@kryptonianson Look in nms.c and play around with the speed settings and see if you can get what you want. Just change the number for the setting and run 'make' again.

#define TYPE_EFFECT_SPEED 4
#define JUMBLE_SECONDS 2
#define JUMBLE_LOOP_SPEED 35
#define REVEAL_LOOP_SPEED 100

@bartobri
Copy link
Owner

bartobri commented May 1, 2016

@jakimfett Just committed changes to support the -a command line option which skips the first pause and auto-starts the decryption effect. Give it a try.

ls -l | nms -a

@bartobri
Copy link
Owner

bartobri commented May 1, 2016

@apjanke Just committed changes to support the -v command line option which prints the version number.

@apjanke
Copy link
Contributor

apjanke commented May 2, 2016

Works as expected for -v.

$ ./bin/nms -v
nms version 0.1.0

Thanks!

@bartobri
Copy link
Owner

bartobri commented May 2, 2016

All issues addressed. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants