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

Grep error if using zsh #224

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@linyows
Contributor

linyows commented Aug 7, 2013

I added quote for regex in grep with zsh

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Aug 7, 2013

Owner

エスケープされてるのでクォートしなくて大丈夫なはずなんですけど、具体的に問題が出ているコマンドがあれば教えてもらえますか?

修正範囲が大きくて影響大きいのと、クォートを増やすとその分複雑になるので、クォートする以外にも対方法があるかどうか検討したいので、具体的にどういう問題が出てるのか教えてください。

Owner

mizzy commented Aug 7, 2013

エスケープされてるのでクォートしなくて大丈夫なはずなんですけど、具体的に問題が出ているコマンドがあれば教えてもらえますか?

修正範囲が大きくて影響大きいのと、クォートを増やすとその分複雑になるので、クォートする以外にも対方法があるかどうか検討したいので、具体的にどういう問題が出てるのか教えてください。

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Aug 7, 2013

Owner

修正範囲大きい、というほどでもないか。とは言え、自分はzshで特に問題起きてないので、直さなくていいものは直さないようにしたいので、具体的にどういう問題が起きてるのか教えてもらえると助かりますー。

Owner

mizzy commented Aug 7, 2013

修正範囲大きい、というほどでもないか。とは言え、自分はzshで特に問題起きてないので、直さなくていいものは直さないようにしたいので、具体的にどういう問題が起きてるのか教えてもらえると助かりますー。

@linyows

This comment has been minimized.

Show comment
Hide comment
@linyows

linyows Aug 7, 2013

Contributor

説明不足ですいません。えと、serverのshellがzshの場合にbe_installed.by('gem')だと^が原因でマッチしなくなります。

Contributor

linyows commented Aug 7, 2013

説明不足ですいません。えと、serverのshellがzshの場合にbe_installed.by('gem')だと^が原因でマッチしなくなります。

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Aug 7, 2013

Owner

おお、なるほど。見てみますねー。

Owner

mizzy commented Aug 7, 2013

おお、なるほど。見てみますねー。

@linyows

This comment has been minimized.

Show comment
Hide comment
@linyows

linyows Aug 7, 2013

Contributor

手元で gem list | grep ^pry とかやるとはやいすね

Contributor

linyows commented Aug 7, 2013

手元で gem list | grep ^pry とかやるとはやいすね

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Aug 7, 2013

Owner

あー、コマンドの組み立て方が他のと違って、^ がエスケープから漏れてるからですね。その上にある、check_grouped みたいな形に書き換えてもらう方が、全体として一貫性があってよさそうですね。

Owner

mizzy commented Aug 7, 2013

あー、コマンドの組み立て方が他のと違って、^ がエスケープから漏れてるからですね。その上にある、check_grouped みたいな形に書き換えてもらう方が、全体として一貫性があってよさそうですね。

@linyows

This comment has been minimized.

Show comment
Hide comment
@linyows

linyows Aug 7, 2013

Contributor

お、気づかなかった!ありがとうございます。変更します。

Contributor

linyows commented Aug 7, 2013

お、気づかなかった!ありがとうございます。変更します。

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Aug 7, 2013

Owner

gem 以外も該当するやつ直してくれたんですね。ありがとうございます!

Owner

mizzy commented Aug 7, 2013

gem 以外も該当するやつ直してくれたんですね。ありがとうございます!

@linyows

This comment has been minimized.

Show comment
Hide comment
@linyows

linyows Aug 7, 2013

Contributor

いえいえ、アドバイスありがとうございます 🍜

Contributor

linyows commented Aug 7, 2013

いえいえ、アドバイスありがとうございます 🍜

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Aug 7, 2013

Owner

785dfa1 だけ cherry-pick してマージしましたー。0.7.5 でリリースしてます。
ありがとうございました!

Owner

mizzy commented Aug 7, 2013

785dfa1 だけ cherry-pick してマージしましたー。0.7.5 でリリースしてます。
ありがとうございました!

@mizzy mizzy closed this Aug 7, 2013

@linyows linyows deleted the linyows:zsh_fixes branch Aug 7, 2013

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