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.

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

Check specified path #11

Merged
merged 2 commits into from
Dec 3, 2014
Merged

Check specified path #11

merged 2 commits into from
Dec 3, 2014

Conversation

rhysd
Copy link
Contributor

@rhysd rhysd commented Dec 2, 2014

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

@rhysd
Copy link
Contributor Author

rhysd commented Dec 2, 2014

テストを追加しました.

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

@cohama
Copy link
Owner

cohama commented Dec 2, 2014

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

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

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

@rhysd
Copy link
Contributor Author

rhysd commented Dec 3, 2014

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

  • 僕のコードがまずい
  • エラー時も偶然うまく動いていた機能がエラーチェックによってエラー時に動かなくなった

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

@cohama
Copy link
Owner

cohama commented Dec 3, 2014

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

@rhysd
Copy link
Contributor Author

rhysd commented Dec 3, 2014

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

@cohama
Copy link
Owner

cohama commented Dec 3, 2014

再現しないですね。

@cohama
Copy link
Owner

cohama commented Dec 3, 2014

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

@rhysd
Copy link
Contributor Author

rhysd commented Dec 3, 2014

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

@rhysd
Copy link
Contributor Author

rhysd commented Dec 3, 2014

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

@cohama
Copy link
Owner

cohama commented Dec 3, 2014

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

@cohama
Copy link
Owner

cohama commented Dec 3, 2014

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

cohama added a commit that referenced this pull request Dec 3, 2014
cohama added a commit that referenced this pull request Dec 3, 2014
@cohama cohama merged commit 4391116 into cohama:master Dec 3, 2014
@cohama
Copy link
Owner

cohama commented Dec 3, 2014

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

@cohama cohama mentioned this pull request Dec 3, 2014
@rhysd
Copy link
Contributor Author

rhysd commented Dec 4, 2014

おお,解決&マージありがとうございました. 👽 🎉 🐶

@rhysd rhysd deleted the improve/check_path branch December 4, 2014 02:08
@lesguillemets
Copy link

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

@cohama
Copy link
Owner

cohama commented Dec 7, 2014

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

@lesguillemets
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants