Bug #1919: sparc-solaris-2.10 で 1.9.2-preview1 がビルドできない

なかだです。

At Mon, 10 Aug 2009 23:40:36 +0900,
Naohisa Goto wrote in [ruby-dev:39060]:
> ruby 1.9.2-preview1 が SPARC 上の Solaris10 で make に失敗します。
> 
> 手動で二分検索をやってみたところ、r23240 は make install-nodoc できましたが
> r23241 は make に失敗しました。

http://docs.sun.com/app/docs/doc/819-2243/readdir-r-3c?a=view を
みると、_POSIX_PTHREAD_SEMANTICSがないとreaddir_r()がPOSIX準拠に
ならないようですね。

これでどうでしょうか。


Index: configure.in
===================================================================
--- configure.in	(revision 24493)
+++ configure.in	(working copy)
@@ -743,4 +743,8 @@ esac

 case "$target_os" in
+when(solaris*)
+		RUBY_APPEND_OPTIONS(CPPFLAGS, -D_POSIX_PTHREAD_SEMANTICS)
+		LIBS="-lm $LIBS"
+		;;
 when(nextstep*)	;;
 when(openstep*)	;;


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦

• File config.h added

Solaris の readdir_r について調べてみたところ、
"readdir_r considered harmful"
http://womble.decadentplace.org.uk/readdir_r-advisory.html
という文章を見つけました。これによると、

> On Linux (with glibc) and most versions of Unix, struct dirent
> is large enough to hold maximum-length names from most filesystems,
> so this is safe (though > wasteful). This is not true of Solaris
> and BeOS, where the d_name member is an array of length 1. 

とのことでした。どうやら Solaris では、struct dirent entry のメモリを
確保する際、ファイル名の長さを考慮に入れた十分な長さを自前で確保する
必要があるようです。

> これでどうでしょうか。

config.h を示すのを忘れていました。
1.9.2-preview1の場合のconfig.hをredmine上にて添付します。
autoconf (バージョン 2.61 を使用)が自動的に
  #define _POSIX_PTHREAD_SEMANTICS 1
を追加してくれていたようです。

パッチも念のため試しましたが、やはりダメでした。
この場合の config.h の実質的な変更点は、添付のものから
#define _ALL_SOURCE 1
の行が抜けただけでした。

後藤といいます。自己レスです。

On Tue, 11 Aug 2009 12:49:55 +0900
Naohisa Goto wrote:

> Solaris の readdir_r について調べてみたところ、
> "readdir_r considered harmful"
> http://womble.decadentplace.org.uk/readdir_r-advisory.html
> という文章を見つけました。これによると、
> 
> > On Linux (with glibc) and most versions of Unix, struct dirent
> > is large enough to hold maximum-length names from most filesystems,
> > so this is safe (though > wasteful). This is not true of Solaris
> > and BeOS, where the d_name member is an array of length 1. 
> 
> とのことでした。どうやら Solaris では、struct dirent entry のメモリを
> 確保する際、ファイル名の長さを考慮に入れた十分な長さを自前で確保する
> 必要があるようです。

とりあえずは、共用体にして無理矢理メモリを確保すると、make は通り、
make install できました。以下にパッチを添付します。

より正しくは、上記文章のように毎回 fpathconf を呼び出して
ファイル名の最大長を取得し、それを元にメモリを確保する必要が
あるのだとは思いますが、とりあえずのパッチです。

相変わらずdlのテストは失敗しますが、dlについては、別チケットを
作りたいと思います。


Index: dir.c
===================================================================
--- dir.c	(revision 24578)
+++ dir.c	(working copy)
@@ -491,6 +491,39 @@
 # define IF_HAVE_READDIR_R(something) /* nothing */
 #endif

+#if defined SIZEOF_STRUCT_DIRENT_TOO_SMALL
+# include <limits.h>
+# define NAME_MAX_FOR_STRUCT_DIRENT 255
+# if defined NAME_MAX
+#  if NAME_MAX_FOR_STRUCT_DIRENT < NAME_MAX
+#   undef  NAME_MAX_FOR_STRUCT_DIRENT
+#   define NAME_MAX_FOR_STRUCT_DIRENT NAME_MAX
+#  endif
+# endif
+# if defined _POSIX_NAME_MAX
+#  if NAME_MAX_FOR_STRUCT_DIRENT < _POSIX_NAME_MAX
+#   undef  NAME_MAX_FOR_STRUCT_DIRENT
+#   define NAME_MAX_FOR_STRUCT_DIRENT _POSIX_NAME_MAX
+#  endif
+# endif
+# if defined _XOPEN_NAME_MAX
+#  if NAME_MAX_FOR_STRUCT_DIRENT < _XOPEN_NAME_MAX
+#   undef  NAME_MAX_FOR_STRUCT_DIRENT
+#   define NAME_MAX_FOR_STRUCT_DIRENT _XOPEN_NAME_MAX
+#  endif
+# endif
+# define DEFINE_STRUCT_DIRENT \
+  union { \
+    struct dirent dirent; \
+    char dummy[offsetof(struct dirent, d_name) + \
+	       NAME_MAX_FOR_STRUCT_DIRENT + 1]; \
+  }
+# define STRUCT_DIRENT(entry) ((entry).dirent)
+#else
+# define DEFINE_STRUCT_DIRENT struct dirent
+# define STRUCT_DIRENT(entry) (entry)
+#endif
+
 /*
  *  call-seq:
  *     dir.read => string or nil
@@ -508,11 +541,11 @@
 {
     struct dir_data *dirp;
     struct dirent *dp;
-    IF_HAVE_READDIR_R(struct dirent entry);
+    IF_HAVE_READDIR_R(DEFINE_STRUCT_DIRENT entry);

     GetDIR(dir, dirp);
     errno = 0;
-    if (READDIR(dirp->dir, dirp->enc, &entry, dp)) {
+    if (READDIR(dirp->dir, dirp->enc, &STRUCT_DIRENT(entry), dp)) {
 	return rb_external_str_new_with_enc(dp->d_name, NAMLEN(dp), dirp->enc);
     }
     else if (errno == 0) {	/* end of stream */
@@ -546,12 +579,12 @@
 {
     struct dir_data *dirp;
     struct dirent *dp;
-    IF_HAVE_READDIR_R(struct dirent entry);
+    IF_HAVE_READDIR_R(DEFINE_STRUCT_DIRENT entry);

     RETURN_ENUMERATOR(dir, 0, 0);
     GetDIR(dir, dirp);
     rewinddir(dirp->dir);
-    while (READDIR(dirp->dir, dirp->enc, &entry, dp)) {
+    while (READDIR(dirp->dir, dirp->enc, &STRUCT_DIRENT(entry), dp)) {
 	rb_yield(rb_external_str_new_with_enc(dp->d_name, NAMLEN(dp), dirp->enc));
 	if (dirp->dir == NULL) dir_closed();
     }
@@ -1270,11 +1303,11 @@
     if (magical || recursive) {
 	struct dirent *dp;
 	DIR *dirp;
-	IF_HAVE_READDIR_R(struct dirent entry);
+	IF_HAVE_READDIR_R(DEFINE_STRUCT_DIRENT entry);
 	dirp = do_opendir(*path ? path : ".", flags);
 	if (dirp == NULL) return 0;

-	while (READDIR(dirp, enc, &entry, dp)) {
+	while (READDIR(dirp, enc, &STRUCT_DIRENT(entry), dp)) {
 	    char *buf = join_path(path, dirsep, dp->d_name);
 	    enum answer new_isdir = UNKNOWN;

Index: configure.in
===================================================================
--- configure.in	(revision 24578)
+++ configure.in	(working copy)
@@ -750,6 +750,10 @@
 esac

 case "$target_os" in
+when(solaris*)
+		AC_DEFINE(SIZEOF_STRUCT_DIRENT_TOO_SMALL, 1)
+		LIBS="-lm $LIBS"
+		;;
 when(nextstep*)	;;
 when(openstep*)	;;
 when(rhapsody*)	;;


-- 
後藤 直久  [e-mail]

まつもと ゆきひろです

In message "Re: [ruby-dev:39132] Re: [Bug #1919] sparc-solaris-2.10  で 1.9.2-preview1  がビルドできない"
    on Wed, 19 Aug 2009 22:42:58 +0900, Naohisa GOTO <[e-mail]> writes:

|> Solaris の readdir_r について調べてみたところ、
|> "readdir_r considered harmful"
|> http://womble.decadentplace.org.uk/readdir_r-advisory.html
|> という文章を見つけました。これによると、
|> 
|> > On Linux (with glibc) and most versions of Unix, struct dirent
|> > is large enough to hold maximum-length names from most filesystems,
|> > so this is safe (though > wasteful). This is not true of Solaris
|> > and BeOS, where the d_name member is an array of length 1. 
|> 
|> とのことでした。どうやら Solaris では、struct dirent entry のメモリを
|> 確保する際、ファイル名の長さを考慮に入れた十分な長さを自前で確保する
|> 必要があるようです。
|
|とりあえずは、共用体にして無理矢理メモリを確保すると、make は通り、
|make install できました。以下にパッチを添付します。

ありがとうございます。取り込みます。fpathconfを使ったパッチ
が提供されればそれも考慮します。

• Category set to core
• Status changed from Open to Assigned
• Assignee set to matz (Yukihiro Matsumoto)
• Priority changed from Normal to High
• Target version set to 1.9.2

• Status changed from Assigned to Feedback
• Priority changed from High to Normal

• Status changed from Feedback to Closed

遠藤です。

[ruby-dev:39132] で提示されたとりあえずのパッチは r24585 で
取り込まれたようなので close します。ありがとうございました。

よりよいパッチができたら、reopen するか何かしてください。

-- 
Yusuke Endoh <[e-mail]>