[Mew-dist 14716] Re: incm-0.3

[SAKAI Kiyotaka]

>> In article <20001024.004048.74676404.momo at example.com>, Yasunari Momoi <momo at example.com> writes:

> incm-0.3 をリリースします．
> 例によって，incm -h でヘルプが出るので，使い方はそれを参考に
> してください．mew での設定例は incm.c の頭に書いてあります．

>   http://bug.org/~momo/incm-0.3.tar.gz

> 今の懸念事項に関しては 00todo.jis に書いてあります．コメント，
> 助言など，お待ちしています．よろしくお願いします．．．

1. autoconf.info によると、time.h, sys/time.h の include は、以下のパッ
  チのようにするのが標準的な作法のようです。

2. SunOS 4.1.4 だと LOCK_EX は sys/file.h に定義されているため、コンパ
  イルエラーが発生しました。mail.local だと以下のパッチのように対処し
  ていましたが、無条件に sys/file.h を include してしまってもいいかも
  しれません。

3. SunOS 4.1.4, HP-UX 10.20 では、snprintf() がないためにリンク時にエ
  ラーが発生しました。おそらく、snprintf() を使わないで頑張るしかない
  と思います。

4. gcc -Wall でコンパイルすると、

    incm.c:434: warning: `bytes' might be used uninitialized in this function

  という警告が発生しました。この部分のコードを読む限り、Content-Length 
  が付いていないメールに対して -c オプションを付けて実行すると問題が発
  生する可能性があると思いますので、何らかの対応をした方がいいと思いま
  す。

5. Makefile に、configure 時に作成されたファイルをすべて消す target、
  distclean が欲しいです。
-- 
酒井 清隆 (E-mail: ksakai at example.com)


===================================================================
RCS file: RCS/incm.c,v
retrieving revision 1.1
diff -u -r1.1 incm.c
--- incm.c	2000/10/24 00:15:37	1.1
+++ incm.c	2000/10/24 00:27:41
@@ -27,7 +27,18 @@
 #include <string.h>
 #include <sys/stat.h>
 #include <errno.h>
-#include <sys/time.h>
+
+#if TIME_WITH_SYS_TIME
+# include <sys/time.h>
+# include <time.h>
+#else
+# if HAVE_SYS_TIME_H
+#  include <sys/time.h>
+# else
+#  include <time.h>
+# endif
+#endif
+
 #if HAVE_DIRENT_H
 # include <dirent.h>
 #endif
@@ -85,6 +96,11 @@
 #  define USE_MAILLOCK	1
 # endif
 #endif
+
+# ifndef LOCK_EX
+#  include <sys/file.h>
+# endif
+
 /*
 #if HAVE_MAILLOCK_H && !USE_MAILLOCK
 # define USE_MAILLOCK	1