These are chat archives for zplug/zplug

18th
Feb 2016
Masaki ISHIYAMA
@b4b4r07
Feb 18 2016 01:43

OK かと思います。

あと、外部ソースの場所ですが、 $ZPLUG_ROOT/src/ext/hoge.zsh で良さそうですか?

前回質問を見逃していたことを思い出しました。今のところはそれで問題なさそうです。ただ、src 自体何か他のいいディレクトリ名に変えたい気もします。また、そのタイミングで ext も変わるかもそれませんが、いまはそれで良さそうです。

Naoki Mizuno
@NigoroJr
Feb 18 2016 03:23

ありがとうございます。自分で言っておいてなんですが、少し不安になったのでもう少しいろいろなoh-my-zsh内のもので試して大丈夫なことを確認してから送らせてもらいます :bow:

今のところはそれで問題なさそうです。

了解です。

Naoki Mizuno
@NigoroJr
Feb 18 2016 04:08
zplug 'themes/sunaku', \
    from:oh-my-zsh

とすると、

https://github.com/b4b4r07/zplug2/blob/9cb52cc44de16356d0fd94afa0beed6122970e13/autoload/commands/__load__#L103

この行でno such file or directoryとエラーが出るのですが、これってもともとでした?
確認のためmasterブランチのコミット1be1e6f (Change commit/at tag and add depth tag (shallow clone) )で上記を使用しましたところ、

__load__:cd:103: no such file or directory: /home/naoki/src/forks/zplug2/zvanilla_T13C/repos/robbyrussell/oh-my-zsh/themes/sunaku

となります。それ以降のコミットを使用するとcloneで失敗するため、確認できてません。

Masaki ISHIYAMA
@b4b4r07
Feb 18 2016 04:27

大丈夫なことを確認してから

これについて OK です。

zplug 'themes/sunaku', from:oh-my-zsh についてですが、そのコミットを使用しましたが再現できませんでした。
該当箇所を細かくトレースしながら確認しましたが問題はなさそうでした(別件で installed が正しく行われたかどうかのチェックが失敗していることを発見しました。これは themes/sunaku が plugins/foo などとちがいディレクトリではなくファイルであるためです。https://github.com/b4b4r07/zplug2/blob/master/autoload/commands/__install__#L154-L157 これは -d-e にすれば解決できることですが、 zplugs に登録される dir タグを ~/.zplug/repos/robbyrussell/oh-my-zsh/themes/sunaku ではなく ~/.zplug/repos/robbyrussell/oh-my-zsh にしないと…)
Masaki ISHIYAMA
@b4b4r07
Feb 18 2016 04:36

ひとまずもうすこし調べてみます。

これってもともとでした?

この質問について、以前は問題なかった気がします。こういったバグが発生するのも oh-my-zsh のコードがばらついていることにも原因がありそうです。はやく external sources を利用して切り分けを行いたいですね。

Naoki Mizuno
@NigoroJr
Feb 18 2016 05:50

そのコミットを使用しましたが再現できませんでした

ご確認ありがとうございます。私のほうでも明日、別マシンで試してみたりします。だいぶ前から走らせてるtmuxプロセス内だったりするので、環境変数まわりで何かおかしいことがあるのかも…

installed が正しく行われたかどうかのチェックが失敗している

あ、本当ですね。

zplugs に登録される dir タグを〜

賛成です。外部ソースによりそのような細かい調整は各外部ソースの関数に任せられるため、dirタグは統一して $ZPLUG_HOME/repos/user/repo_name にするというのはどうでしょう?また、外部ソースを追加する際にも、レポジトリのトップレベルディレクトリを指していると分かっていると便利そうです。
ただ、今回のno such file or directoryの問題がこちらのポカミスである可能性を考慮して、この提案は一旦保留にしてください。

Masaki ISHIYAMA
@b4b4r07
Feb 18 2016 06:13
ありがとうございます :coffee: いつもコントリビュート感謝しています。ひとまず omz 用のソースを追加するコミットがマージされたらこちらでも
Masaki ISHIYAMA
@b4b4r07
Feb 18 2016 06:24
external sources について設計をよりエレガントにできるか見たりしたいです。
Naoki Mizuno
@NigoroJr
Feb 18 2016 19:10

いつもコントリビュート感謝しています。

いえいえ、こちらこそ色々と勉強させてもらっています。

external sources について設計をよりエレガントに

はい、よろしくお願いします。

ところでno such file or directoryの件なのですが、1be1e6fで commit タグのdeprecationによりエラー箇所のコードが導入されたみたいです。 zspec[dir] が存在しなければcdする先を親ディレクトリに変更すればひとまずエラーは回避できますが、packageのトップレベルディレクトリより上に行ってしまうことがあるかもしれないことを懸念しています。一応 #15 送りました。

Naoki Mizuno
@NigoroJr
Feb 18 2016 19:59
とりあえずoh-my-zshのexternal sourceでplugins, lib, themesの3つとも動作確認しましたので、あとでupstreamを取り入れてからsquash & rebaseしてPR送らせてもらいます。