threads / patch / 12537

v5autoconf: Test FREAD_READS_DIRECTORIES

Subject: [PATCH v5] autoconf: Test FREAD_READS_DIRECTORIES

## tl;dr

2 messages between Mar 6, 2008 and Mar 6, 2008. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Michal Rokos· Mar 6, 2008, 10:41 UTC · lore

Add test for FREAD_READS_DIRECTORIES to detect when fread() reads fopen'ed directory.

Signed-off-by: Michal Rokos <michal.rokos@nextsoft.cz>
Tested-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Tested-by: Mike Ralphson <mike@abacus.co.uk>
Show changes to 2 files +21 −0

config.mak.in, configure.ac

diff --git a/config.mak.in b/config.mak.in
index ee6c33d..516c468 100644
--- a/config.mak.in
+++ b/config.mak.in
@@ -46,3 +46,4 @@ NO_MKDTEMP=@NO_MKDTEMP@
 NO_ICONV=@NO_ICONV@
 OLD_ICONV=@OLD_ICONV@
 NO_DEFLATE_BOUND=@NO_DEFLATE_BOUND@
+FREAD_READS_DIRECTORIES=@FREAD_READS_DIRECTORIES@
diff --git a/configure.ac b/configure.ac
index 85d7ef5..2deeb44 100644
--- a/configure.ac
+++ b/configure.ac
@@ -326,6 +326,26 @@ else
 	NO_C99_FORMAT=
 fi
 AC_SUBST(NO_C99_FORMAT)
+#
+# Define FREAD_READS_DIRECTORIES if your are on a system which succeeds
+# when attempting to read from an fopen'ed directory.
+AC_CACHE_CHECK([whether system succeeds to read fopen'ed directory],
+ [ac_cv_fread_reads_directories],
+[
+AC_RUN_IFELSE(
+	[AC_LANG_PROGRAM([AC_INCLUDES_DEFAULT],
+		[[char c;
+		FILE *f = fopen(".", "r");
+		return f && fread(&c, 1, 1, f)]])],
+	[ac_cv_fread_reads_directories=no],
+	[ac_cv_fread_reads_directories=yes])
+])
+if test $ac_cv_fread_reads_directories = yes; then
+	FREAD_READS_DIRECTORIES=UnfortunatelyYes
+else
+	FREAD_READS_DIRECTORIES=
+fi
+AC_SUBST(FREAD_READS_DIRECTORIES)
 
 
 ## Checks for library functions.
Johannes Schindelin· Mar 6, 2008, 11:13 UTC · re: Michal Rokos · lore

Re: [PATCH v5] autoconf: Test FREAD_READS_DIRECTORIES

Hi,
On Thu, 6 Mar 2008, Michal Rokos wrote:
Show 5 quoted lines
> Add test for FREAD_READS_DIRECTORIES to detect when fread() reads fopen'ed 
> directory.
> 
> Signed-off-by: Michal Rokos <michal.rokos@nextsoft.cz>
> Tested-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>

Umm... technically, you are correct, but because of that libsocket bug, it did not work...

Ciao, Dscho

← back to recent threads