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.

Sign up for GitHub

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

Jump to bottom

Add a TEE modifier. #117

Merged
merged 1 commit into from
Apr 9, 2014
Merged

Add a TEE modifier. #117

merged 1 commit into from
Apr 9, 2014

Conversation

eevee
Copy link
Contributor

@eevee eevee commented Apr 9, 2014

I happened to need this recently, when wrapping a program that writes things I need to know to stdout, but occasionally prompts for user credentials on stdout as well. It's pretty dumb, but seems to do the job.

tomerfiliba added a commit that referenced this pull request Apr 9, 2014
@tomerfiliba
Merge pull request #117 from eevee/tee
5da8c70 
Add a TEE modifier.
@tomerfiliba tomerfiliba merged commit 5da8c70 into tomerfiliba:master Apr 9, 2014
@tomerfiliba
Copy link
Owner

thanks!

@eevee eevee deleted the tee branch April 9, 2014 18:33
@eevee eevee mentioned this pull request Apr 9, 2014
Closed
@gaschreiber
Copy link

It would be great to make the TEE available where FG and BG are available too.
plumbum/commands/init.py:2 reads:
from plumbum.commands.modifiers import ExecutionModifier, Future, FG, BG
The TEE should be added to this list?

@DanielJoyce
Copy link

Yes it should be added to that list. Its goofy right now

@DanielJoyce DanielJoyce mentioned this pull request Jan 18, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@eevee @tomerfiliba @gaschreiber @DanielJoyce