git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] config.mak.uname: set FREAD_READS_DIRECTORIES for cygwin

From
Ramsay Jones <ramsay@ramsayjones.plus.com>
Date
Jul 21, 2017, 18:43 UTC
Message-ID
<506cb9f2-cf59-2df6-2d4c-30c82dfbf85b@ramsayjones.plus.com>
Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
---
Hi Junio,

My 'make test' run for the v2.14.0-rc0 on cygwin turned up a new test failure (over and above the v2.13.0 failures), namely t1308.23. Running the test in the debugger showed that cygwin was fopen-ing the 'a-directory' without problem.

On one occasion, while in the debugger, I did see fopen return NULL while trying to open a directory. So, I created a test program which showed reliably that fopen succeeds on a directory. (So, maybe I was just seeing things! :-D ).

t1308 was reliably failing before this patch, afterwards it reliably passes, so ...

[I have a full test run going at the moment, but it won't finish for about 3 hours.]

ATB, Ramsay Jones

 config.mak.uname | 1 +
 1 file changed, 1 insertion(+)
diff --git a/config.mak.uname b/config.mak.uname
index 551e465a7..6604b130f 100644
--- a/config.mak.uname
+++ b/config.mak.uname
@@ -185,6 +185,7 @@ ifeq ($(uname_O),Cygwin)
 	SPARSE_FLAGS = -isystem /usr/include/w32api -Wno-one-bit-signed-bitfield
 	OBJECT_CREATION_USES_RENAMES = UnfortunatelyNeedsTo
 	COMPAT_OBJS += compat/cygwin.o
+	FREAD_READS_DIRECTORIES = UnfortunatelyYes
 endif
 ifeq ($(uname_S),FreeBSD)
 	NEEDS_LIBICONV = YesPlease
-- 
2.13.0
Next: Junio C Hamano
Message 1 of 2 in “config.mak.uname: set FREAD_READS_DIRECTORIES for cygwin”
  1. config.mak.uname: set FREAD_READS_DIRECTORIES for cygwinRamsay Jones, Jul 21, 2017
  2. Junio C HamanoJul 21, 2017

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.