Project

General

Profile

提案 #1375

twitter-text gem を twitter-text-simpleidn に差し替える

Added by Shibafu Midorino 3 months ago. Updated 3 months ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
twitter
ブランチ:
topic/1375-twitter-text-simpleidn

Description

twitter-text gem はネイティブの libidn v1 系に依存するため、しばしば厄介な依存解決を強いられます。
そこで、libidn に依存しないようカスタマイズされたフォークである twitter-text-simpleidn に差し替えてみませんか、という提案です。

メリット

  • ネイティブ依存が減る
    今後は Twitter プラグインが本体から外れることも計画されているため、依存関係が簡単になると導入もしやすいと思います。
    (~/.mikutter/plugin に入れるユーザープラグインの依存解決は、本体よりも面倒に思われることが度々あります)

デメリット

  • twitter-textの更新への追従が遅れる、または困難になる
    このフォークは私が作成していますが、上流の変更内容や私の都合によって追従が困難になる可能性を否定できません。

Files

0001-twitter-text-simpleidn.patch (844 Bytes) 0001-twitter-text-simpleidn.patch Shibafu Midorino, 2019-08-18 22:13

Associated revisions

Revision ab0ca21b (diff)
Added by Shibafu Midorino 3 months ago

twitter-text の代わりに twitter-text-simpleidn を使用する refs #1375

History

#1

Updated by Shibafu Midorino 3 months ago

  • Description updated (diff)
#2

Updated by toshi_a 初音 3 months ago

  • Status changed from 分類待ち to パッチ適用待ち
  • Assignee set to toshi_a 初音

オッ

#3

Updated by toshi_a 初音 3 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Shibafu Midorino
  • ブランチ set to topic/1375-twitter-text-simpleidn

字数カウントもちゃんと出来てるので良いと思います。
これから大幅に字数カウントの方法が替わったらどうなるか分かりませんが、基本的にはgTLDが追加されるくらいしか変更無さそうなので、メリットのほうが大きいと思います。

(環境によっては本当にインストール大変になったからなあ)

一応チェックお願いします。

#4

Updated by Shibafu Midorino 3 months ago

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

コミット確認しました、問題なく動作しています。

#5

Updated by toshi_a 初音 3 months ago

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

merged.

Also available in: Atom PDF