From: Paul Eggert <eggert@cs.ucla.edu>
To: Ken Brown <kbrown@cornell.edu>
Cc: 36405-done@debbugs.gnu.org
Subject: bug#36405: 26.2.90; O_PATH problem on some versions of Cygwin
Date: Thu, 27 Jun 2019 13:47:27 -0700 [thread overview]
Message-ID: <d8ae5b11-73cf-74c0-6f06-a772eccfa2aa@cs.ucla.edu> (raw)
In-Reply-To: <9308fcb1-bb7e-229b-aacf-8c55db00fca6@cornell.edu>
[-- Attachment #1: Type: text/plain, Size: 276 bytes --]
Thanks, but that patch intrudes on the mainline code a bit. It's simpler
to just disable O_PATH on Cygwin until the old Cygwin versions die out.
Also, there's a similar issue in fileio.c. I installed the attached,
which should work around the problem in both source files.
[-- Attachment #2: 0001-Work-around-Cygwin-bug-with-O_PATH.patch --]
[-- Type: text/x-patch, Size: 1105 bytes --]
From e7200b3cf9b4a6470197915a31c45739f078683b Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Thu, 27 Jun 2019 13:05:05 -0700
Subject: [PATCH] Work around Cygwin bug with O_PATH
Problem reported by Ken Brown (Bug#36405).
* src/dired.c, src/fileio.c (O_PATH) [__CYGWIN__]: Undef.
---
src/dired.c | 4 ++++
src/fileio.c | 4 ++++
2 files changed, 8 insertions(+)
diff --git a/src/dired.c b/src/dired.c
index 493758292b..b8197d36a0 100644
--- a/src/dired.c
+++ b/src/dired.c
@@ -41,6 +41,10 @@
#include "buffer.h"
#include "coding.h"
+#ifdef __CYGWIN__
+# undef O_PATH /* Buggy in Cygwin 3.0.0 through 3.0.7. */
+#endif
+
#ifdef MSDOS
#include "msdos.h" /* for fstatat */
#endif
diff --git a/src/fileio.c b/src/fileio.c
index ed1d2aedf3..e36118652c 100644
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -61,6 +61,10 @@ Copyright (C) 1985-1988, 1993-2019 Free Software Foundation, Inc.
# include <linux/fs.h>
#endif
+#ifdef __CYGWIN__
+# undef O_PATH /* Buggy in Cygwin 3.0.0 through 3.0.7. */
+#endif
+
#ifdef WINDOWSNT
#define NOMINMAX 1
#include <windows.h>
--
2.21.0
next prev parent reply other threads:[~2019-06-27 20:47 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-27 19:28 bug#36405: 26.2.90; O_PATH problem on some versions of Cygwin Ken Brown
2019-06-27 20:47 ` Paul Eggert [this message]
2019-06-28 6:25 ` Eli Zaretskii
2019-06-29 6:26 ` Paul Eggert
2019-06-29 7:06 ` Eli Zaretskii
2019-06-29 18:03 ` Ken Brown
2019-06-29 18:19 ` Eli Zaretskii
2019-06-28 17:57 ` Ken Brown
2019-06-29 6:20 ` Paul Eggert
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d8ae5b11-73cf-74c0-6f06-a772eccfa2aa@cs.ucla.edu \
--to=eggert@cs.ucla.edu \
--cc=36405-done@debbugs.gnu.org \
--cc=kbrown@cornell.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.