Project

General

Profile

Actions

Bug #3307

closed

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

Added by tarui (Masaya Tarui) almost 14 years ago. Updated almost 13 years ago.

Status:
Closed
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 <winsock2.h>
6: #include <ws2tcpip.h>
7: #include <windows.h>
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 <winsock2.h>
6: #include <ws2tcpip.h>
7: #include <windows.h>
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

Actions #1

Updated by mame (Yusuke Endoh) almost 14 years ago

=begin
遠藤です。

2010年5月18日4:41 Masaya Tarui :

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

snip

パッチです。

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

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

--
Yusuke Endoh

=end

Actions #2

Updated by kosaki (Motohiro KOSAKI) almost 14 years ago

=begin

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

+1 :)

=end

Actions #3

Updated by usa (Usaku NAKAMURA) almost 14 years ago

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

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

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

+1 :)

+1, too :P

それでは。

U.Nakamura

=end

Actions #4

Updated by nagai (Hidetoshi Nagai) almost 14 years ago

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

From: Masaya Tarui
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 をありがとうございました.

永井 秀利 ()
九州工業大学 大学院情報工学研究院 知能情報工学研究系 知能情報メディア部門
=end

Actions #5

Updated by nagai (Hidetoshi Nagai) almost 14 years ago

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

From: Masaya Tarui
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 をありがとうございました.

永井 秀利 ()
九州工業大学 大学院情報工学研究院 知能情報工学研究系 知能情報メディア部門

=end

Actions #6

Updated by nagai (Hidetoshi Nagai) almost 14 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

Actions #7

Updated by matz (Yukihiro Matsumoto) almost 14 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 writes:

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

了解です。

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

=end

Actions #8

Updated by tarui (Masaya Tarui) almost 14 years ago

=begin
樽家です。

ありがとうございます。

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

2010年5月24日15:41 Yukihiro Matsumoto :

まつもと ゆきひろです

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

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

了解です。

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

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

=end

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0Like0