バグ #1474
closedpluggaloid の要求バージョン
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 では上限だけの変更でした。
実装上の意図というか、マージの意図としてはどんな感じなのでしょう。
Updated by toshi_a 初音 about 4 years 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も同様。
Updated by toshi_a 初音 about 4 years ago
- Status changed from 実装待ち to レビュー待ち
- Assignee changed from toshi_a 初音 to Izumi Tsutsui
- ブランチ set to topic/1474-wrong-dependency
修正しました。
確認…といっても何を確認するのかという感じですが、一応お願いします。
Updated by Izumi Tsutsui about 4 years 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と思います。 (後者の微妙なスペース差分は現状が正?)