mattn / jvgrep

grep for japanese vimmer
https://mattn.kaoriya.net/
140 stars 19 forks source link

Search current directory if stdin is a tty and target is not specified #57

Closed k-takata closed 5 years ago

k-takata commented 7 years ago

現状、検索対象の指定がない場合、stdin が検索対象となりますが、stdin が端末の場合にカレントディレクトリを検索対象とするように変更します。 stdin がリダイレクトされている場合は、今までと同様に stdin を検索対象とします。 grepとの互換性は低くなってしまいますが、検索対象の指定がなく、かつ stdin が端末の場合に stdin を検索するという現状の動作に有用な使い道があるとは思えません。 いかがでしょうか。

mattn commented 7 years ago

jvgrep foo で ag ぽく動作させるって事ですね。なるほど。ちょっと考える時間ください。

koron commented 7 years ago

ttyなstdinからの検索を別途オプションでできるようにしておけば デフォルトの動作を変えてもいざというときに「こう回避してね」と言えるので それで良いのではないでしょうか。

このPRそのままではまったくできなくなってしまうので、 現時点では無用だと思いますが、 将来有用な使い方に遭遇した際に為す術がありません。

k-takata commented 7 years ago

type con | jvgrep ほげほげ とか cat | jvgrep ほげほげ という手段は残されています。 これがダメだとなると、環境変数による設定で動作を変えられるようにするとかですかね。

koron commented 7 years ago

たとえば jvgrep -tty foobar で以前の jvgrep foobar と同じ動作になるよ、とかで良いんじゃないですか?

k-takata commented 7 years ago

本当に使われることがあるのか分かりませんが、とりあえず実装しました。

k-takata commented 7 years ago

コンフリクトを修正。

k-takata commented 5 years ago

だいぶ古くなってきたのでrebaseしました。

mattn commented 5 years ago

Thanks