cohama / agit.vim

A powerful Git log viewer
234 stars 25 forks source link

Check specified path #11

Closed rhysd closed 9 years ago

rhysd commented 9 years ago

指定したファイルのパスが存在しない場合に内部で E684 が出てエラーになってしまっていたので,ファイルが実際に存在するかどうかをチェックするようにしました.

rhysd commented 9 years ago

テストを追加しました.

これは単なる感想ですが,agit#launch() で例外握りつぶすと異常系のテストが面倒なような…?(今回は :redir で回避しました)

cohama commented 9 years ago

ありがとうございます! が何故かテスト落ちてますね...。

これは単なる感想ですが,agit#launch() で例外握りつぶすと異常系のテストが面倒なような…?(今回は :redir で回避しました)

確かに感ありますね。(というかなんでこんな書き方したのか覚えていない)

rhysd commented 9 years ago

ウッ テスト落ちてますね…

どっちなんでしょう.少なくとも手元だとテスト通っていたので,こちらで調べるのは難しそうです…

cohama commented 9 years ago

はい。こちらで調べて何とかします。

rhysd commented 9 years ago

すみません.とりあえず,cohama さんのほうで再現するかどうかだけ教えていただけると助かります. あと,テストの書き方は --dir のテストを真似ただけなので,そこに問題があるかもしれません.

cohama commented 9 years ago

再現しないですね。

cohama commented 9 years ago

あれ、よく見たら追加されたのとは別のテストがコケてる。。。

rhysd commented 9 years ago

ぬ… とりあえず,Travis と同じ Ubuntu 12.04 の環境が会社のほうにあるので,時間あったらそっちの Vim と git で試してみます.

rhysd commented 9 years ago

そもそも 14d3178 の時点(テスト追加前)でコケてますね…

cohama commented 9 years ago

手元の Ubuntu12.04 で再現しました。

cohama commented 9 years ago

しかし master ではこけていないので完全に謎

cohama commented 9 years ago

解決したので取り込みました。 原因は Agit の終了を :q としていたことでした。古い Vim だと QuitPre がないのでこれだけだと Agit をちゃんと終了できていなかったみたいです。(そもそもなぜ最初から通っていたのかは謎) ありがとうございました。

rhysd commented 9 years ago

おお,解決&マージありがとうございました. :alien: :tada: :dog:

lesguillemets commented 9 years ago

すみません,この commit 以降,git.path が空やディレクトリの時に File not found が投げられるようになり,例えばファイルを開かない状態での挙動が変わった(以前: git 管理下のディレクトリで起動してファイルを開いていない vim から :Agit するとそのディレクトリについて agit が起動 / 現在:File not found.)ようなのですが,これは意図されたことでしょうか?

cohama commented 9 years ago

すみません。これは意図した動作ではないのでバグです。 この現象については #13 で対応するつもりです。

lesguillemets commented 9 years ago

その issue を見ていませんでした,申し訳ありません.また,対応ありがとうございます.File not found については起こらなくなったことを確認しました.