Backport #10401: Ruby/Tk patch for Tcl/Tk8.6

• Status: Closed
• Assignee: -

[nagai (Hidetoshi Nagai)]

永井＠知能．九工大です．

Ruby/Tk を Tcl/Tk8.6 と組み合わせて使えるようにするためのパッチです．
2.1.3 p242 用のパッチとなっています．
適用いただけますと幸いです．

[usa (Usaku NAKAMURA)]

• Related to Bug #8000: "require 'tk'" segfaults on 64-bit linux with Tk 8.6 added

[nagachika (Tomoyuki Chikanaga)]

• Status changed from Open to Rejected

安定版ブランチでは基本的に不具合修正のみ行う方針にしています。Tcl/Tk の新しいバージョンのサポートというのは新規機能の一種と考えるのが自然かと思いますので、メンテナンスポリシーに照らして Reject とします。何か適用すべき事情があるようでしたら再openしてください。

[nagai (Hidetoshi Nagai)]

• File tk-diff-ruby-2.1.3-p242.for-SEGV-only tk-diff-ruby-2.1.3-p242.for-SEGV-only added
• Status changed from Rejected to Open

では，Tcl/Tk 8.6 と組み合わせた場合の SEGV 回避だけのパッチを添付します．
このパッチでは Ruby/Tk 側での機能追加は一切ありません．
外部ライブラリに依存した拡張において，外部ライブラリのバージョンアップに対して Ruby 側の機能追加なしに新バージョンに対応するのは，
(グレーゾーンかもしれませんが) メンテナンスポリシーに反することではないと思います．

[vo.x (Vit Ondruch)]

Can this be applied, please? We need this patch for Fedora [1].

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1181160

[usa (Usaku NAKAMURA)]

Thank you for reminding, Vit.

Do you need only tk-diff-ruby-2.1.3-p242.for-SEGV-only ?
If so, I can merge it.

[vo.x (Vit Ondruch)]

Usaku NAKAMURA wrote:

Do you need only tk-diff-ruby-2.1.3-p242.for-SEGV-only ?

Yep, that should be enough.

Actually since I noticed this backport ticket with the patch too late, I already applied patch [1] to our package. But these are the same, I just stripped a few additional hunks with DUMPs which seems unimportant.

[1] http://pkgs.fedoraproject.org/cgit/ruby.git/tree/ruby-2.2.0-Support-tcl-tk-8.6.patch?h=f21

[usa (Usaku NAKAMURA)]

• Status changed from Open to Closed

Applied in changeset r50474.

---

• ext/tk/extconf.rb: support Tcl/Tk8.6.

• ext/tk/tcltklib.c, ext/tk/lib/tk.rb: get rid of SEGV with Tcl/Tk8.6.
  [Backport #10401]

[usa (Usaku NAKAMURA)]

merged tk-diff-ruby-2.1.3-p242.for-SEGV-only at r50474.
Thank you, Vit! (and, of course, Nagai-sensei!)

[vo.x (Vit Ondruch)]

Thank you!