Project

General

Profile

Bug #3307

ext/tk がheader fileの有無を誤判定する

Added by tarui (Masaya Tarui) about 10 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
ruby -v:
ruby 1.9.2dev (2010-05-17 revision 27861) [i386-mswin32_90]
Backport:

Description

=begin
樽家です。
tcl.hが無いのにコンパイルをしようとして失敗します。
mkmf.logを見ると次のようになっていました。

"cl -nologo -E -I../../.ext/include/i386-mswin32_90 -I../../../include -I../../../ext/tk -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\" -DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DHAVE_NO_LPFN_DECLS=1 -DHAVE_NO_FINDEX_ENUMS=1 -DTCL_CFGVAL_ENCODING=\"cp1252\" -DTCL_CFG_OPTIMIZED=1 -DTCL_CFG_DEBUG=1 -DHAVE_UXTHEME_H=1 -MD -Zi -W2 -wd4996 -O2sy- -Zm600 -I/mingw/include conftest.c -P"
conftest.c

checked program was:
/* begin /
1: #include "ruby.h"
2:
3: #define WIN32_LEAN_AND_MEAN
4: #define WIN32
5: #include
6: #include
7: #include
8: tcl.h
/
end */

"cl -nologo -E -I../../.ext/include/i386-mswin32_90 -I../../../include -I../../../ext/tk -I/mingw/include -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\" -DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DHAVE_NO_LPFN_DECLS=1 -DHAVE_NO_FINDEX_ENUMS=1 -DTCL_CFGVAL_ENCODING=\"cp1252\" -DTCL_CFG_OPTIMIZED=1 -DTCL_CFG_DEBUG=1 -DHAVE_UXTHEME_H=1 -MD -Zi -W2 -wd4996 -O2sy- -Zm600 conftest.c -P"
conftest.c

checked program was:
/* begin /
1: #include "ruby.h"
2:
3: #define WIN32_LEAN_AND_MEAN
4: #define WIN32
5: #include
6: #include
7: #include
8: tk.h
/
end */

パッチです。

Index: ext/tk/extconf.rb
===================================================================
--- ext/tk/extconf.rb (リビジョン 27864)
+++ ext/tk/extconf.rb (作業コピー)
@@ -989,7 +989,7 @@
end

if TclConfig_Info['TCL_INCLUDE_SPEC'] &&
  • have_tcl_h = try_cpp('tcl.h', TclConfig_Info['TCL_INCLUDE_SPEC'])
  •  have_tcl_h = try_cpp('#include<tcl.h>', TclConfig_Info['TCL_INCLUDE_SPEC'
    

    ])
    $INCFLAGS << " " << TclConfig_Info['TCL_INCLUDE_SPEC']
    elsif have_tcl_h = have_header('tcl.h')
    # find
    @@ -1009,7 +1009,7 @@
    end

    if TkConfig_Info['TK_INCLUDE_SPEC'] &&

  •  have_tk_h = try_cpp('tk.h', TkConfig_Info['TK_INCLUDE_SPEC'])
    
  •  have_tk_h = try_cpp('#include<tk.h>', TkConfig_Info['TK_INCLUDE_SPEC'])
    

    $INCFLAGS << " " << TkConfig_Info['TK_INCLUDE_SPEC']
    elsif have_tk_h = have_header('tk.h')
    # find
    =end

#1

Updated by mame (Yusuke Endoh) about 10 years ago

=begin
遠藤です。

2010年5月18日4:41 Masaya Tarui redmine@ruby-lang.org:

樽家です。
tcl.hが無いのにコンパイルをしようとして失敗します。

snip

パッチです。

再現し、このパッチで直ることを確認しました。

ところで、樽家さんはよく貢献してくれているので、コミット権を
差し上げるのはどうでしょうか。
ChangeLog には樽家さんの名前が 7 つほど載ってます。

--
Yusuke Endoh mame@tsg.ne.jp

=end

#2

Updated by kosaki (Motohiro KOSAKI) about 10 years ago

=begin

ところで、樽家さんはよく貢献してくれているので、コミット権を
差し上げるのはどうでしょうか。
ChangeLog には樽家さんの名前が 7 つほど載ってます。

+1 :)

=end

#3

Updated by usa (Usaku NAKAMURA) about 10 years ago

=begin
こんにちは、なかむら(う)です。

In message "[ruby-dev:41338] Re: [Bug #3307] ext/tk がheader fileの有無を誤判定する"
on May.18,2010 22:06:30, kosaki.motohiro@gmail.com wrote:

ところで、樽家さんはよく貢献してくれているので、コミット権を
差し上げるのはどうでしょうか。
ChangeLog には樽家さんの名前が 7 つほど載ってます。

+1 :)

+1, too :P

それでは。
--
U.Nakamura usa@garbagecollect.jp

=end

#4

Updated by nagai (Hidetoshi Nagai) about 10 years ago

=begin
永井@知能.九工大です.

From: Masaya Tarui redmine@ruby-lang.org
Subject: [ruby-dev:41334] [Bug #3307] ext/tk がheader fileの有無を誤判定する
Date: Tue, 18 May 2010 04:41:25 +0900
Message-ID:

Bug #3307: ext/tk がheader fileの有無を誤判定する

大変遅くなってしまい,本当にごめんなさい.
たった今,commit しました.
報告と patch をありがとうございました.
--
永井 秀利 (nagai@ai.kyutech.ac.jp)
九州工業大学 大学院情報工学研究院 知能情報工学研究系 知能情報メディア部門
=end

#5

Updated by nagai (Hidetoshi Nagai) about 10 years ago

=begin
永井@知能.九工大です.

From: Masaya Tarui redmine@ruby-lang.org
Subject: [ruby-dev:41334] [Bug #3307] ext/tk がheader fileの有無を誤判定する
Date: Tue, 18 May 2010 04:41:25 +0900
Message-ID:

Bug #3307: ext/tk がheader fileの有無を誤判定する

大変遅くなってしまい,本当にごめんなさい.
たった今,commit しました.
報告と patch をありがとうございました.
--
永井 秀利 (nagai@ai.kyutech.ac.jp)
九州工業大学 大学院情報工学研究院 知能情報工学研究系 知能情報メディア部門

=end

#6

Updated by nagai (Hidetoshi Nagai) about 10 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

=begin
This issue was solved with changeset r27904.
Masaya, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

=end

#7

Updated by matz (Yukihiro Matsumoto) about 10 years ago

=begin
まつもと ゆきひろです

In message "Re: [ruby-dev:41337] Re: [Bug #3307] ext/tk がheader fileの有無を誤判定する"
on Tue, 18 May 2010 21:26:29 +0900, Yusuke ENDOH mame@tsg.ne.jp writes:

|ところで、樽家さんはよく貢献してくれているので、コミット権を
|差し上げるのはどうでしょうか。
|ChangeLog には樽家さんの名前が 7 つほど載ってます。

了解です。

樽家さんは、cvs-admin@ruby-lang.org にpgpサインしたssh2の公
開鍵と、希望するアカウント名を送ってくださいませんか?

=end

#8

Updated by tarui (Masaya Tarui) about 10 years ago

=begin
樽家です。

ありがとうございます。

登録していただき、先ほどコミットできる事を確認しました。
また、遠藤さん始め推薦してくださった方々にも感謝を。

2010年5月24日15:41 Yukihiro Matsumoto matz@ruby-lang.org:

まつもと ゆきひろです

In message "Re: [ruby-dev:41337] Re: [Bug #3307] ext/tk がheader fileの有無を誤判定する"
on Tue, 18 May 2010 21:26:29 +0900, Yusuke ENDOH mame@tsg.ne.jp writes:

|ところで、樽家さんはよく貢献してくれているので、コミット権を
|差し上げるのはどうでしょうか。
|ChangeLog には樽家さんの名前が 7 つほど載ってます。

了解です。

樽家さんは、cvs-admin@ruby-lang.org にpgpサインしたssh2の公
開鍵と、希望するアカウント名を送ってくださいませんか?

--
樽家昌也(Masaya TARUI)
No Tool,No Life.

=end

Also available in: Atom PDF