Skip to content
This repository has been archived by the owner on Dec 3, 2023. It is now read-only.

Add support for pipes #62

Open
trqt opened this issue Aug 1, 2021 · 4 comments
Open

Add support for pipes #62

trqt opened this issue Aug 1, 2021 · 4 comments
Labels
enhancement New feature or request

Comments

@trqt
Copy link
Contributor

trqt commented Aug 1, 2021

$ echo string | ./ted
2021-08-01T15:29:51,151068207-03:00

@arthurbacci
Copy link
Owner

What's wrong? I never said that it works with pipes

@arthurbacci arthurbacci changed the title Freezes and 100% CPU usage on pipe Add support for pipes Aug 1, 2021
@arthurbacci arthurbacci added the enhancement New feature or request label Aug 1, 2021
@arthurbacci
Copy link
Owner

arthurbacci commented Aug 1, 2021

What should be done in this case? echo string | ted should be the same as echo string > ~/.config/ted/buffer and echo string | ted file.txt the same as echo string > file.txt? But and echo string | ted file1.txt file2.txt? The same as echo a | tee file1.txt > file2.txt?
Or should it just print an error message and die?
Or should it try to remove all files in /?

@arthurbacci
Copy link
Owner

sussy
Fixed?

@trqt
Copy link
Contributor Author

trqt commented Aug 9, 2021

What should be done in this case? echo string | ted should be the same as echo string > ~/.config/ted/buffer and echo string | ted file.txt the same as echo string > file.txt? But and echo string | ted file1.txt file2.txt? The same as echo a | tee file1.txt > file2.txt?
Or should it just print an error message and die?
Or should it try to remove all files in /?

It should paste the stdin in the file and start editing it, it already does that, but it hangs in the process

@arthurbacci arthurbacci reopened this Aug 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants