Project

General

Profile

バグ #1474

pluggaloid の要求バージョン

Added by Izumi Tsutsui about 1 month ago. Updated 18 days ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
ブランチ:
topic/1474-wrong-dependency
クラッシュする:
No

Description

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

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 では上限だけの変更でした。
実装上の意図というか、マージの意図としてはどんな感じなのでしょう。

#1

Updated by toshi_a 初音 30 days ago

  • Status changed from 分類待ち to 実装待ち
  • Assignee set to 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も同様。

#2

Updated by toshi_a 初音 29 days ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Izumi Tsutsui
  • ブランチ set to topic/1474-wrong-dependency

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

#3

Updated by Izumi Tsutsui 29 days ago

  • Status changed from レビュー待ち to マージ待ち

とりあえず目視で 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と思います。 (後者の微妙なスペース差分は現状が正?)

#4

Updated by Izumi Tsutsui 18 days ago

  • Status changed from マージ待ち to 終了

マージされてた a008f6b8

Also available in: Atom PDF