プロジェクト

全般

プロフィール

バグ #1474

完了

pluggaloid の要求バージョン

Izumi Tsutsui さんが約4年前に追加. 約4年前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
クラッシュする:
いいえ

説明

バグかどうかわかりませんが、こんなツイートがあります

https://twitter.com/slimymars/status/1307374543904755713

mikutterさんはアップデートしようとすると大抵起動しなくなるので元気なときにだけgit pullしているのだが、今回はGemfileの記載が古いというか、 gem 'pluggaloid','>= 1.2.0', '< 2.0'が多分'1.3.0'あたりじゃないと駄目なんじゃないだろうか。まあぼくはめんどくなったのでbundle updateしたけど。

確かに develop の 2f38d32c までは

  gem 'pluggaloid', '>= 1.5.0', '< 1.6'

だったものが 9a0fcdf8 の master からのマージで
  gem 'pluggaloid','>= 1.2.0', '< 2.0'

に変わっていて、ほかにも最低バージョンが下がっているものがあります。

diff --git a/Gemfile b/Gemfile
index 1e9b55ad..2cf13ed8 100644
--- a/Gemfile
+++ b/Gemfile
@@ -11,12 +11,12 @@ ruby '>= 2.5.0'

 group :default do
   gem 'addressable','>= 2.7.0', '< 2.8'
-  gem 'delayer', '>= 1.1.2', '< 1.2'
-  gem 'delayer-deferred', '>= 2.2.0', '< 2.3'
-  gem 'diva','>= 1.0.2', '< 1.1'
+  gem 'delayer','>= 1.0.2', '< 2.0'
+  gem 'delayer-deferred','>= 2.1.3', '< 3.0'
+  gem 'diva','>= 1.0.2', '< 2.0'
   gem 'memoist','>= 0.16.2', '< 0.17'
   gem 'oauth','>= 0.5.4'
-  gem 'pluggaloid', '>= 1.5.0', '< 1.6'
+  gem 'pluggaloid','>= 1.2.0', '< 2.0'
   gem 'typed-array','>= 0.1.2', '< 0.2'
 end

一方、 master の f8d5c6b5 では上限だけの変更でした。
実装上の意図というか、マージの意図としてはどんな感じなのでしょう。

toshi_a 初音 さんが約4年前に更新

  • ステータス分類待ち から 実装待ち に変更
  • 担当者toshi_a 初音 にセット

これは単純なマージミスですね。修正します。

-  gem 'pluggaloid', '>= 1.5.0', '< 1.6'
+  gem 'pluggaloid','>= 1.2.0', '< 2.0'

'>= 1.5.0', '< 1.6' は、4.1でライブラリの要求バージョンが変わったからです。

'>= 1.2.0', '< 2.0' は、マイナーバージョンアップで後方互換性が保証されるなら、'< 1.6'のような指定はおかしいのでは?とつついさんにMastodonで指摘されて行った変更です。

Pluggaloidなら

gem 'pluggaloid', '>= 1.5.0', '< 2.0'

が正しいです。他のgemも同様。

toshi_a 初音 さんが約4年前に更新

  • ステータス実装待ち から レビュー待ち に変更
  • 担当者toshi_a 初音 から Izumi Tsutsui に変更
  • ブランチtopic/1474-wrong-dependency にセット

修正しました。
確認…といっても何を確認するのかという感じですが、一応お願いします。

Izumi Tsutsui さんが約4年前に更新

  • ステータスレビュー待ち から マージ待ち に変更

とりあえず目視で master との差分
https://dev.mikutter.hachune.net/projects/mikutter/repository/main/diff/Gemfile?rev=a1c8334&rev_to=08a3f50
とスポンサー対応マージ直前との差分
https://dev.mikutter.hachune.net/projects/mikutter/repository/main/diff/Gemfile?rev=a1c8334&rev_to=2f38d32
を見てOKと思います。 (後者の微妙なスペース差分は現状が正?)

Izumi Tsutsui さんが約4年前に更新

  • ステータスマージ待ち から 終了 に変更

マージされてた a008f6b8

他の形式にエクスポート: Atom PDF