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

load the gemspec only if the tDiary was installed by gem #364

Merged
merged 3 commits into from Aug 26, 2013
Merged

Conversation

machu
Copy link
Member

@machu machu commented Aug 22, 2013

#356 および #363 への対処です。

これまで、tDiaryのライブラリはインストール方法にかかわらずgemspec経由でロードしていましたが、gemspecの読み込みにgitが必要 (git が無い環境では動かない) なので、gemでインストールした時のみロードするように修正しました。

元々はhikidocなどの依存ライブラリをtdiary.gemspecに記述していたので、必ずgemspecを読み込んでいました。今は依存ライブラリをGemfileに記述しているので、tDiaryをgemでインストールした時だけgemspecを読みこめばよいです。

まとめるとこうなります。

  • tDiary を gem でインストール → Gemfile.local 経由で tdiary.gemspec をロードする
  • tDiary をパッケージ (tar.gz) もしくは git clone でインストール → gemspec はロードしない
    • $LOAD_PATHにカレントディレクトリが含まれているので tDiary ライブラリをロード可能

@hsbt
Copy link
Member

hsbt commented Aug 23, 2013

gem モードでのテストが死んでおりますね、gem モードでテストする時は Gemfile.local に gemspec とだけ書いたものを作ってからテスト実行って感じでしょうか。

@hsbt
Copy link
Member

hsbt commented Aug 26, 2013

テスト通るようにしたので、マージしてから 8/29 にでも 4.0.1 として出しましょうか

hsbt added a commit that referenced this pull request Aug 26, 2013
load the gemspec only if the tDiary was installed by gem
@hsbt hsbt merged commit cb8fe21 into master Aug 26, 2013
@hsbt hsbt deleted the fix-363 branch August 26, 2013 11:08
@machu
Copy link
Member Author

machu commented Aug 26, 2013

テストありがとうございます。29日に向けて準備しますね。

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.

None yet

2 participants