bug#19909: closed (Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is specified on cygwin)

[help-debbugs@gnu.org (GNU bug Tracking System)]

[-- Attachment #1: Type: text/plain, Size: 437 bytes --]

Your bug report

#19909: [PATCH] Error out if with-file-notification=w32 is specified on cygwin

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 19909@debbugs.gnu.org.

-- 
19909: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19909
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

[-- Attachment #2: Type: message/rfc822, Size: 3500 bytes --]

From: Ken Brown <kbrown@cornell.edu>
To: Robert Pluim <rpluim@gmail.com>
Cc: Eli Zaretskii <eliz@gnu.org>, 19909-done@debbugs.gnu.org
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is specified on cygwin
Date: Mon, 02 Mar 2015 09:05:40 -0500
Message-ID: <54F46E34.1090502@cornell.edu>

On 3/2/2015 4:35 AM, Robert Pluim wrote:
> I'd prefer "Robert Pluim" but it's not a big deal.

OK, I've pushed the change to the master branch and am closing the bug.

> It does show that
> there's a variable somewhere that I haven't set correctly though, since
> that ChangeLog entry should never have said "rpluim"

I think user-full-name is what you're looking for.

Ken



[-- Attachment #3: Type: message/rfc822, Size: 5130 bytes --]

From: Robert Pluim <rpluim@gmail.com>
To: bug-gnu-emacs@gnu.org
Subject: [PATCH] Error out if with-file-notification=w32 is specified on cygwin
Date: Fri, 20 Feb 2015 10:47:36 +0100
Message-ID: <821tlkhq2v.fsf@gmail.com>

It's a combination that won't compile, so it's probably best to error
out during configure.

diff --git a/ChangeLog b/ChangeLog
index 4365668..dec9ce0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2015-02-20  rpluim  <rpluim@gmail.com>
+
+	* configure.ac: Error out if with-file-notification=w32 is
+	specified on cygwin.
+
 2015-02-13  Jan Djärv  <jan.h.d@swipnet.se>
 
 	* configure.ac: Set locallisppath to empty for NS self contained,
diff --git a/configure.ac b/configure.ac
index 0bcc55c..048168a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2659,6 +2659,10 @@ fi
 
 dnl MS Windows native file monitor is available for mingw32 only.
 case $with_file_notification,$opsys in
+  w32,cygwin)
+    AC_MSG_ERROR([`--with-file-notification=w32' was specified, but
+    this is only supported on MS-Windows native and MinGW32 builds])
+    ;;
   w32,* | yes,mingw32)
     AC_CHECK_HEADER(windows.h)
     if test "$ac_cv_header_windows_h" = yes ; then