rhysd / committia.vim

A Vim plugin for more pleasant editing on commit messages
MIT License
731 stars 30 forks source link

Fix #27 work_tree for submodule and worktree #31

Closed anekos closed 7 years ago

anekos commented 7 years ago

git worktree 対応版です。 thinca さんの再現手順では修正できていました。 ただ、worktree は自分では使った記憶ないので、若干自信なしです。

anekos commented 7 years ago

お好きな方をお選びください。

rhysd commented 7 years ago

こちらって #30 に比べてマイナスなポイントありますか?

rhysd commented 7 years ago

問題は #30 に書いてあるやり方で再現できました.ありがとうございます :+1:

anekos commented 7 years ago

マイナスポイントは、コマンド使うのと較べるとマニュアルに書いてもいなさそうな仕様に依存してること、ということでした。 といっても、コミットメッセージのファイルパスに頼ってるし、いまさらですね。 こういうのがそうそう変わるとも思えないし…。

anekos commented 7 years ago

変更しますた。

rhysd commented 7 years ago

修正ありがとうございます :cat:

マイナスポイントは、コマンド使うのと較べるとマニュアルに書いてもいなさそうな仕様に依存してること、ということでした。

その箇所は下記という認識で合ってますか?

let work_tree = fnamemodify(readfile(git_dir . '/gitdir')[0], ':h')

確かに僕自身も work tree を使っていないのでうまく動かなくなる可能性はありますが,壊れた時に考えれば良いかなと思います…!

anekos commented 7 years ago

そこです! :cat:

rhysd commented 7 years ago

了解です!