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

vimdoc-ja/doc/tags-ja もdeployする #268

Closed
h-east opened this issue Jun 22, 2018 · 15 comments
Closed

vimdoc-ja/doc/tags-ja もdeployする #268

h-east opened this issue Jun 22, 2018 · 15 comments

Comments

@h-east
Copy link
Member

h-east commented Jun 22, 2018

これ、やってない理由なんかあるんでしたっけ?
ローカルでgit pull後に毎回 :helptagsするの忘れがち。

@mattn
Copy link
Member

mattn commented Jun 22, 2018

賛否両論ありますね。プラグインマネージャによるインストールだと、ドキュメントとしてプラグインが配布されてる場合に作者がタグ生成し忘れてたりすると git diff が出ちゃうので、プラグイン界隈では doc/tags* は同梱しないというのが一般的なんですよね。

@h-east
Copy link
Member Author

h-east commented Jun 22, 2018

CIで自動生成するので「タグ生成し忘れ」は発生しないし同梱してもいいかなと思っています。

@koron
Copy link
Member

koron commented Jun 22, 2018

いまざっくりみてたら vimdoc-ja のほうで .gitignore に指定しているんですね。

vim-jp/vimdoc-ja@a2f1f07

@crazymaster doc/tags-ja を含める(= vimdoc-ja の .gitignoreから外す) ことについてなにか意見はありますか?

@koron
Copy link
Member

koron commented Jun 22, 2018

もともとはこれか。

vim-jp/vimdoc-ja#70

@mattn
Copy link
Member

mattn commented Jun 22, 2018

あるとすれば

  1. コントリビュータがドキュメントを修正
  2. git commit -a しちゃう
  3. tags-ja が含まれた PR がくる

とかでしょうか。

@h-east
Copy link
Member Author

h-east commented Jun 22, 2018

@mattn vimdoc-ja-working/docの方は今まで通りtags-jaは含めないのでそういうことにはならないんじゃないでしょうか?

@mattn
Copy link
Member

mattn commented Jun 22, 2018

そうですね。なお僕のスタンスは「どちらでもいい」ですが、いくぶんプラグインマネージャに関する知見が多い訳ではないので slack でも意見を募集しています。

@koron
Copy link
Member

koron commented Jun 22, 2018

vim-jp/vimdoc-ja#21 でも同じ議論をやってたんですね。
自分はもうすっかり忘れてる。

ざっくり見ると当時は
「どっちでも良い」vs「あったらプラグインマネージャはちょっと苦労するね」
みたいな感じで、「あるほうが良い」という立場は明確ではなかった。
だったらなしにしとこう、というくらいの判断だったんだと思います。

現時点においてどうするべきかは、僕は今も「どっちでも良い」

@h-east
Copy link
Member Author

h-east commented Jun 22, 2018

その議論してた時ってvimdoc-ja-workingあったんでしたっけ?

ちなみにローカルでgit pull後に再度:helptagsしても内容変わらないし、タイムスタンプは変わるけど、gitは変更ファイルとみなさないです。

@mattn
Copy link
Member

mattn commented Jun 22, 2018

vim のバージョンにより出力されるタグが違うって事が起きた時にどうするかですね。

@crazymaster
Copy link
Member

vim標準のパッケージ機能ではタグファイルを含めることを推奨してますね
:h package-create

@h-east
Copy link
Member Author

h-east commented Jun 22, 2018

そもそも、プラグインマネージャが 無条件 でtags-jaを更新するのどうなの?って思いました。

とりあえず、.vimrcにこんなのを書いて運用してみます。

packadd! vimdoc-ja
let docdir = filter(split(&rtp, ','), 'v:val =~# "vimdoc-ja$"')[0] . '/doc'
let doctag = docdir . '/tags-ja'
if !filereadable(doctag) || getftime(doctag) < getftime(docdir)
  exec "helptags" docdir
endif

(追記)
いや、:helptagsgit pullした後に1度だけおこなえばいいので、これは.vimrcに書くべきものではないですね。
package dir下の各リポジトリを更新して、必要な処理があればおこなう関数(とコマンド)を作れよって話か。

@h-east
Copy link
Member Author

h-east commented Jun 23, 2018

vim標準のパッケージ機能ではタグファイルを含めることを推奨してますね

では含めることにしましょう。
(mattnさん、koronさんも「どっちでもいい」という意見なのも踏まえて)

h-east added a commit to vim-jp/vimdoc-ja that referenced this issue Jun 25, 2018
h-east added a commit that referenced this issue Jun 25, 2018
Create vimdoc-ja/doc/tags-ja. (Fixed #268)
@machakann
Copy link
Contributor

今更ですけど、 helphelp.txt にもタグファイルは同梱するのが望ましい旨が書いてありました。

- Make a package with all the files and the tags file available for download.
Users can drop it in one of the "doc" directories and start use it.
Report this to Bram, so that he can add a link on www.vim.org.

@h-east
Copy link
Member Author

h-east commented Jun 30, 2018

そもそも、プラグインマネージャが 無条件 でtags-jaを更新するのどうなの?って思いました。

この問題を改善するために、まずvim-plugにPRを投げました。
junegunn/vim-plug#773
以下のいずれかの条件に一致したときに:helptagsを実行するようにしました。

  • doc/{tags,tags-??x} なファイルが存在しない
  • doc/{tags,tags-??x} なファイルがgit管理されていない

他のVimプラグインマネージャも改善されて欲しい👍

k-takata added a commit to vim-jp/vimdoc-ja that referenced this issue Oct 7, 2021
Always use the unix file format for doc/tags-ja even on Windows (with
`core.autocrlf=true`).

Even with the Windows version of Vim, the `:helptags` command creates
tags files with the unix file format.
To avoid unnecessary changes, doc/tags-ja should be checked out with the
unix file format.

Related: vim-jp/vimdoc-ja-working#268, #21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants