GNU bug report logs - #19909
[PATCH] Error out if with-file-notification=w32 is specified on cygwin

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 19909 in the body.
You can then email your comments to 19909 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #3 received at submit <at> debbugs.gnu.org (full text, mbox):

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

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 <at> gmail.com>
+
+	* configure.ac: Error out if with-file-notification=w32 is
+	specified on cygwin.
+
 2015-02-13  Jan Djärv  <jan.h.d <at> 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

Message #6 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32
 is	specified on cygwin
Date: Fri, 20 Feb 2015 12:06:58 +0200

> From: Robert Pluim <rpluim <at> gmail.com>
> Date: Fri, 20 Feb 2015 10:47:36 +0100
> 
> It's a combination that won't compile, so it's probably best to error
> out during configure.

Why doesn't it compile?  Perhaps it's better to allow it to compile,
than to error out.

Message #9 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32
 is	specified on cygwin
Date: Fri, 20 Feb 2015 11:19:16 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Robert Pluim <rpluim <at> gmail.com>
>> Date: Fri, 20 Feb 2015 10:47:36 +0100
>> 
>> It's a combination that won't compile, so it's probably best to error
>> out during configure.
>
> Why doesn't it compile?  Perhaps it's better to allow it to compile,
> than to error out.

I guess whoever put the following in w32.h had his reasons:

#ifdef CYGWIN
#error "w32.h is not compatible with Cygwin"
#endif


In file included from w32notify.c:95:0:
w32.h:23:2: error: #error "w32.h is not compatible with Cygwin"
 #error "w32.h is not compatible with Cygwin"
  ^
In file included from ../lib/sys/types.h:28:0,
                 from ../lib/signal.h:49,
                 from /usr/include/machine/setjmp.h:357,
                 from /usr/include/setjmp.h:10,
                 from lisp.h:24,
                 from w32notify.c:92:

Message #12 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out if
 with-file-notification=w32	is	specified on cygwin
Date: Fri, 20 Feb 2015 12:46:36 +0200

> From: Robert Pluim <rpluim <at> gmail.com>
> Date: Fri, 20 Feb 2015 11:19:16 +0100
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> >> From: Robert Pluim <rpluim <at> gmail.com>
> >> Date: Fri, 20 Feb 2015 10:47:36 +0100
> >> 
> >> It's a combination that won't compile, so it's probably best to error
> >> out during configure.
> >
> > Why doesn't it compile?  Perhaps it's better to allow it to compile,
> > than to error out.
> 
> I guess whoever put the following in w32.h had his reasons:
> 
> #ifdef CYGWIN
> #error "w32.h is not compatible with Cygwin"
> #endif

According to the comment in w32notify.c, w32.h is included only for
w32_strerror.  Could you try commenting out the w32.h inclusion, and
instead copy the w32_strerror prototype into w32notify.c?  Does Emacs
compile then as part of the Cygwin-w32 build, and do file
notifications work?  (I don't have Cygwin installed to try that,
sorry.)

If getting the prototype of w32_strerror is the only problem, then
it's easy to solve in a way that will be compatible with the
Cygwin-w32 build.

Thanks.

Message #15 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out if
 with-file-notification=w32	is	specified on cygwin
Date: Fri, 20 Feb 2015 15:16:49 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Robert Pluim <rpluim <at> gmail.com>
>> Date: Fri, 20 Feb 2015 11:19:16 +0100
>> 
>> Eli Zaretskii <eliz <at> gnu.org> writes:
>> 
>> >> From: Robert Pluim <rpluim <at> gmail.com>
>> >> Date: Fri, 20 Feb 2015 10:47:36 +0100
>> >> 
>> >> It's a combination that won't compile, so it's probably best to error
>> >> out during configure.
>> >
>> > Why doesn't it compile?  Perhaps it's better to allow it to compile,
>> > than to error out.
>> 
>> I guess whoever put the following in w32.h had his reasons:
>> 
>> #ifdef CYGWIN
>> #error "w32.h is not compatible with Cygwin"
>> #endif
>
> According to the comment in w32notify.c, w32.h is included only for
> w32_strerror.  Could you try commenting out the w32.h inclusion, and
> instead copy the w32_strerror prototype into w32notify.c?  Does Emacs
> compile then as part of the Cygwin-w32 build, and do file
> notifications work?  (I don't have Cygwin installed to try that,
> sorry.)
>

No, that doesn't compile. w32notify.c needs (at least)
w32_unicode_filenames, filename_to_utf16(), filename_to_ansi() and
w32_valid_pointer_p() from w32.c.

I don't think adding w32.c into the build on cygwin is going to work.

Regards

Robert

Message #18 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out
 if	with-file-notification=w32	is	specified on cygwin
Date: Fri, 20 Feb 2015 17:10:45 +0200

> From: Robert Pluim <rpluim <at> gmail.com>
> Date: Fri, 20 Feb 2015 15:16:49 +0100
> 
> > According to the comment in w32notify.c, w32.h is included only for
> > w32_strerror.  Could you try commenting out the w32.h inclusion, and
> > instead copy the w32_strerror prototype into w32notify.c?  Does Emacs
> > compile then as part of the Cygwin-w32 build, and do file
> > notifications work?  (I don't have Cygwin installed to try that,
> > sorry.)
> >
> 
> No, that doesn't compile. w32notify.c needs (at least)
> w32_unicode_filenames, filename_to_utf16(), filename_to_ansi() and
> w32_valid_pointer_p() from w32.c.

Right.  But those problems are quite simple to solve.

Ken, would you like to try to solve them?  I could suggest a patch,
but it would be 100% untested.

> I don't think adding w32.c into the build on cygwin is going to work.

Because of the above, or because of something else?

Message #21 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Cc: 19909 <at> debbugs.gnu.org, Ken Brown <kbrown <at> cornell.edu>
Subject: Re: bug#19909: [PATCH] Error out
 if	with-file-notification=w32	is	specified on cygwin
Date: Fri, 20 Feb 2015 16:46:48 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Robert Pluim <rpluim <at> gmail.com>
>> Date: Fri, 20 Feb 2015 15:16:49 +0100
>> 
>> > According to the comment in w32notify.c, w32.h is included only for
>> > w32_strerror.  Could you try commenting out the w32.h inclusion, and
>> > instead copy the w32_strerror prototype into w32notify.c?  Does Emacs
>> > compile then as part of the Cygwin-w32 build, and do file
>> > notifications work?  (I don't have Cygwin installed to try that,
>> > sorry.)
>> >
>> 
>> No, that doesn't compile. w32notify.c needs (at least)
>> w32_unicode_filenames, filename_to_utf16(), filename_to_ansi() and
>> w32_valid_pointer_p() from w32.c.
>
> Right.  But those problems are quite simple to solve.
>
> Ken, would you like to try to solve them?  I could suggest a patch,
> but it would be 100% untested.
>
>> I don't think adding w32.c into the build on cygwin is going to work.
>
> Because of the above, or because of something else?

Because (perhaps mistakenly) I think that calling emacs code that
assumes it's running under Windows native will not interact well with
emacs code that's running under cygwin.

Having said that, I'm willing to test any patches over the next week or
so (after that I'm going to be without a cygwin environment for a couple
of weeks).

Robert

Message #27 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ken Brown <kbrown <at> cornell.edu>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: bug-gnu-emacs <at> gnu.org, Robert Pluim <rpluim <at> gmail.com>
Subject: Re: bug#19909: [PATCH] Error out if	with-file-notification=w32	is
 specified on cygwin
Date: Fri, 20 Feb 2015 11:15:18 -0500

On 2/20/2015 10:10 AM, Eli Zaretskii wrote:
>> From: Robert Pluim <rpluim <at> gmail.com>
>> Date: Fri, 20 Feb 2015 15:16:49 +0100
>>
>>> According to the comment in w32notify.c, w32.h is included only for
>>> w32_strerror.  Could you try commenting out the w32.h inclusion, and
>>> instead copy the w32_strerror prototype into w32notify.c?  Does Emacs
>>> compile then as part of the Cygwin-w32 build, and do file
>>> notifications work?  (I don't have Cygwin installed to try that,
>>> sorry.)
>>>
>>
>> No, that doesn't compile. w32notify.c needs (at least)
>> w32_unicode_filenames, filename_to_utf16(), filename_to_ansi() and
>> w32_valid_pointer_p() from w32.c.
>
> Right.  But those problems are quite simple to solve.
>
> Ken, would you like to try to solve them?  I could suggest a patch,
> but it would be 100% untested.

I'm busy with other things at the moment and can't get to it right away. 
 If it's easy for you to suggest a patch, why don't you go ahead, and 
Robert can test it.  Or you can wait a couple weeks until I can look at it.

BTW, file-notification=gfile works fine in the Cygwin-w32 build, so I'm 
not sure there's any need to put a lot of effort into this.

Ken

Message #30 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error
 out	if	with-file-notification=w32	is	specified on cygwin
Date: Fri, 20 Feb 2015 18:17:36 +0200

> From: Robert Pluim <rpluim <at> gmail.com>
> Date: Fri, 20 Feb 2015 16:46:48 +0100
> 
> >> I don't think adding w32.c into the build on cygwin is going to work.
> >
> > Because of the above, or because of something else?
> 
> Because (perhaps mistakenly) I think that calling emacs code that
> assumes it's running under Windows native will not interact well with
> emacs code that's running under cygwin.

Sorry, maybe I'm confused, but isn't your Cygwin build configured
"--with-w32"?  If so, it already uses some of the Windows native code.

> Having said that, I'm willing to test any patches over the next week or
> so

Thanks.

Message #33 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error
 out	if	with-file-notification=w32	is	specified on cygwin
Date: Fri, 20 Feb 2015 17:44:30 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Robert Pluim <rpluim <at> gmail.com>
>> Date: Fri, 20 Feb 2015 16:46:48 +0100
>> 
>> >> I don't think adding w32.c into the build on cygwin is going to work.
>> >
>> > Because of the above, or because of something else?
>> 
>> Because (perhaps mistakenly) I think that calling emacs code that
>> assumes it's running under Windows native will not interact well with
>> emacs code that's running under cygwin.
>
> Sorry, maybe I'm confused, but isn't your Cygwin build configured
> "--with-w32"?  If so, it already uses some of the Windows native code.
>

Yes, but only the bits relevant to the GUI. I don't know what the rest
of it does :)

Message #36 received at 19909 <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: 19909 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#19909: [PATCH] Error out if	with-file-notification=w32	is
 specified on cygwin
Date: Fri, 20 Feb 2015 18:06:21 +0100

Ken Brown <kbrown <at> cornell.edu> writes:

> On 2/20/2015 10:10 AM, Eli Zaretskii wrote:
>>> From: Robert Pluim <rpluim <at> gmail.com>
>>> Date: Fri, 20 Feb 2015 15:16:49 +0100
>>>
>>>> According to the comment in w32notify.c, w32.h is included only for
>>>> w32_strerror.  Could you try commenting out the w32.h inclusion, and
>>>> instead copy the w32_strerror prototype into w32notify.c?  Does Emacs
>>>> compile then as part of the Cygwin-w32 build, and do file
>>>> notifications work?  (I don't have Cygwin installed to try that,
>>>> sorry.)
>>>>
>>>
>>> No, that doesn't compile. w32notify.c needs (at least)
>>> w32_unicode_filenames, filename_to_utf16(), filename_to_ansi() and
>>> w32_valid_pointer_p() from w32.c.
>>
>> Right.  But those problems are quite simple to solve.
>>
>> Ken, would you like to try to solve them?  I could suggest a patch,
>> but it would be 100% untested.
>
> I'm busy with other things at the moment and can't get to it right
> away. If it's easy for you to suggest a patch, why don't you go ahead,
> and Robert can test it.  Or you can wait a couple weeks until I can
> look at it.
>
> BTW, file-notification=gfile works fine in the Cygwin-w32 build, so
> I'm not sure there's any need to put a lot of effort into this.

I didn't know that.  I'll give it a go.  Perhaps the following slight
modification to my original patch?


diff --git a/ChangeLog b/ChangeLog
index 4365668..dec9ce0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2015-02-20  rpluim  <rpluim <at> gmail.com>
+
+	* configure.ac: Error out if with-file-notification=w32 is
+	specified on cygwin.
+
 2015-02-13  Jan Djärv  <jan.h.d <at> swipnet.se>
 
 	* configure.ac: Set locallisppath to empty for NS self contained,
diff --git a/configure.ac b/configure.ac
index 0bcc55c..19d9c20 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2659,6 +2659,11 @@ 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.
+    Consider using gfile instead.])
+    ;;
   w32,* | yes,mingw32)
     AC_CHECK_HEADER(windows.h)
     if test "$ac_cv_header_windows_h" = yes ; then

Message #39 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH]
 Error	out	if	with-file-notification=w32	is	specified on cygwin
Date: Fri, 20 Feb 2015 19:16:37 +0200

> From: Robert Pluim <rpluim <at> gmail.com>
> Date: Fri, 20 Feb 2015 17:44:30 +0100
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> >> From: Robert Pluim <rpluim <at> gmail.com>
> >> Date: Fri, 20 Feb 2015 16:46:48 +0100
> >> 
> >> >> I don't think adding w32.c into the build on cygwin is going to work.
> >> >
> >> > Because of the above, or because of something else?
> >> 
> >> Because (perhaps mistakenly) I think that calling emacs code that
> >> assumes it's running under Windows native will not interact well with
> >> emacs code that's running under cygwin.
> >
> > Sorry, maybe I'm confused, but isn't your Cygwin build configured
> > "--with-w32"?  If so, it already uses some of the Windows native code.
> 
> Yes, but only the bits relevant to the GUI. I don't know what the rest
> of it does :)

Most of native Windows code in Emacs is related to GUI.  That includes
drawing GUI frames and accepting input from the Windows message queue;
both are used by the Cygwin build --with-w32.

The rest is emulation of Posix functions, which Cygwin doesn't need.

Message #42 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
Subject: Re: bug#19909: [PATCH] Error out if	with-file-notification=w32	is
 specified on cygwin
Date: Fri, 20 Feb 2015 19:18:16 +0200

> Date: Fri, 20 Feb 2015 11:15:18 -0500
> From: Ken Brown <kbrown <at> cornell.edu>
> CC: bug-gnu-emacs <at> gnu.org, Robert Pluim <rpluim <at> gmail.com>
> 
> I'm busy with other things at the moment and can't get to it right away. 
>   If it's easy for you to suggest a patch, why don't you go ahead, and 
> Robert can test it.  Or you can wait a couple weeks until I can look at it.

OK, I will try to find time.

> BTW, file-notification=gfile works fine in the Cygwin-w32 build, so I'm 
> not sure there's any need to put a lot of effort into this.

If it turns out to need a lot of effort, I'll agree with you.

Message #45 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ken Brown <kbrown <at> cornell.edu>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
Subject: Re: bug#19909: [PATCH] Error out if	with-file-notification=w32	is
 specified on cygwin
Date: Fri, 27 Feb 2015 17:25:54 -0500

[Message part 1 (text/plain, inline)]

On 2/20/2015 12:18 PM, Eli Zaretskii wrote:
>> Date: Fri, 20 Feb 2015 11:15:18 -0500
>> From: Ken Brown <kbrown <at> cornell.edu>
>> CC: bug-gnu-emacs <at> gnu.org, Robert Pluim <rpluim <at> gmail.com>
>>
>> I'm busy with other things at the moment and can't get to it right away.
>>    If it's easy for you to suggest a patch, why don't you go ahead, and
>> Robert can test it.  Or you can wait a couple weeks until I can look at it.
>
> OK, I will try to find time.
>
>> BTW, file-notification=gfile works fine in the Cygwin-w32 build, so I'm
>> not sure there's any need to put a lot of effort into this.
>
> If it turns out to need a lot of effort, I'll agree with you.

It looks like it might not take much effort.  What I did for now, just 
for testing purposes, is to copy into w32notify.c whatever was necessary 
to make the Cygwin-w32 build compile with w32 file notification.  I 
tested it briefly and it seems to work.  Obviously it needs much more 
testing to make sure this doesn't break anything else, and then we can 
worry about how to do this right.

Robert, please test when you get a chance (patch attached).

Ken

[cygw32_notify.patch (text/plain, attachment)]

Message #48 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
Subject: Re: bug#19909: [PATCH] Error out if	with-file-notification=w32	is
 specified on cygwin
Date: Sat, 28 Feb 2015 09:59:45 +0200

> Date: Fri, 27 Feb 2015 17:25:54 -0500
> From: Ken Brown <kbrown <at> cornell.edu>
> CC: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
> 
> It looks like it might not take much effort.  What I did for now, just 
> for testing purposes, is to copy into w32notify.c whatever was necessary 
> to make the Cygwin-w32 build compile with w32 file notification.  I 
> tested it briefly and it seems to work.  Obviously it needs much more 
> testing to make sure this doesn't break anything else, and then we can 
> worry about how to do this right.

Thanks.  A few comments:

  . Cygwin doesn't need to call MultiByteToWideChar and
    WideCharToMultiByte via function pointers, it can call them
    directly.

  . I think there shouldn't be a need to call filename_to_utf16 and
    its ilk, but instead use the to_unicode etc., like the rest of
    Cygwin-w32 code does.  E.g., just use GUI_ENCODE_FILE defined on
    w32term.h.

Message #51 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ken Brown <kbrown <at> cornell.edu>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
Subject: Re: bug#19909: [PATCH] Error out if	with-file-notification=w32	is
 specified on cygwin
Date: Sat, 28 Feb 2015 11:32:29 -0500

On 2/28/2015 2:59 AM, Eli Zaretskii wrote:
>> Date: Fri, 27 Feb 2015 17:25:54 -0500
>> From: Ken Brown <kbrown <at> cornell.edu>
>> CC: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
>>
>> It looks like it might not take much effort.  What I did for now, just
>> for testing purposes, is to copy into w32notify.c whatever was necessary
>> to make the Cygwin-w32 build compile with w32 file notification.  I
>> tested it briefly and it seems to work.  Obviously it needs much more
>> testing to make sure this doesn't break anything else, and then we can
>> worry about how to do this right.
>
> Thanks.  A few comments:
>
>    . Cygwin doesn't need to call MultiByteToWideChar and
>      WideCharToMultiByte via function pointers, it can call them
>      directly.
>
>    . I think there shouldn't be a need to call filename_to_utf16 and
>      its ilk, but instead use the to_unicode etc., like the rest of
>      Cygwin-w32 code does.  E.g., just use GUI_ENCODE_FILE defined on
>      w32term.h.

It turns out that filename_to_utf16 wasn't even being called in the 
Cygwin-w32 build, because w32_unicode_filenames == 0.

It also turns out that when I "tested" the build, which I did via M-x 
auto-revert-tail-mode, I wasn't actually testing w32notify, because I 
forgot to do this:

diff --git a/src/emacs.c b/src/emacs.c
index ca5633d..342d0fe 100644
--- a/src/emacs.c
+++ b/src/emacs.c
@@ -1334,11 +1334,12 @@ Using an Emacs configured with 
--with-x-toolkit=lucid does not have this problem
   globals_of_gfilenotify ();
 #endif

-#ifdef WINDOWSNT
-  globals_of_w32 ();
 #ifdef HAVE_W32NOTIFY
   globals_of_w32notify ();
 #endif
+
+#ifdef WINDOWSNT
+  globals_of_w32 ();
   /* Initialize environment from registry settings.  */
   init_environment (argv);
   init_ntproc (dumping); /* must precede init_editfns.  */
@@ -1512,11 +1513,12 @@ Using an Emacs configured with 
--with-x-toolkit=lucid does not have this problem
       syms_of_dbusbind ();
 #endif /* HAVE_DBUS */

-#ifdef WINDOWSNT
-      syms_of_ntterm ();
 #ifdef HAVE_W32NOTIFY
       syms_of_w32notify ();
-#endif /* HAVE_W32NOTIFY */
+#endif
+
+#ifdef WINDOWSNT
+      syms_of_ntterm ();
 #endif /* WINDOWSNT */

       syms_of_profiler ();


When I add that patch, auto-revert-tail-mode no longer works; there's no 
error message, but the buffer doesn't get auto-reverted when something 
is appended.

I think I have again lost motivation to pursue this further, since gfile 
works fine.  And glib is required for the default Cygwin-w32 build 
anyway, because ImageMagick brings it in, so why not just use it?

Ken

Message #54 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
Subject: Re: bug#19909: [PATCH] Error out if	with-file-notification=w32	is
 specified on cygwin
Date: Sat, 28 Feb 2015 18:41:10 +0200

> Date: Sat, 28 Feb 2015 11:32:29 -0500
> From: Ken Brown <kbrown <at> cornell.edu>
> CC: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
> 
> I think I have again lost motivation to pursue this further, since gfile 
> works fine.  And glib is required for the default Cygwin-w32 build 
> anyway, because ImageMagick brings it in, so why not just use it?

Perhaps the OP could explain why he wanted that.

Message #57 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: bug-gnu-emacs <at> gnu.org, Ken Brown <kbrown <at> cornell.edu>
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is
 specified on cygwin
Date: Sat, 28 Feb 2015 17:53:38 +0100

[Message part 1 (text/plain, inline)]

On Sat, Feb 28, 2015 at 5:41 PM, Eli Zaretskii <eliz <at> gnu.org> wrote:

> > Date: Sat, 28 Feb 2015 11:32:29 -0500
> > From: Ken Brown <kbrown <at> cornell.edu>
> > CC: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
> >
> > I think I have again lost motivation to pursue this further, since gfile
> > works fine.  And glib is required for the default Cygwin-w32 build
> > anyway, because ImageMagick brings it in, so why not just use it?
>
> Perhaps the OP could explain why he wanted that.
>

[Message part 2 (text/html, inline)]

Message #60 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: bug-gnu-emacs <at> gnu.org, Ken Brown <kbrown <at> cornell.edu>
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is
 specified on cygwin
Date: Sat, 28 Feb 2015 17:56:28 +0100

[Message part 1 (text/plain, inline)]

@#$ web interface, sorry about the duplicate

On Sat, Feb 28, 2015 at 5:41 PM, Eli Zaretskii <eliz <at> gnu.org> wrote:

> > Date: Sat, 28 Feb 2015 11:32:29 -0500
> > From: Ken Brown <kbrown <at> cornell.edu>
> > CC: bug-gnu-emacs <at> gnu.org, rpluim <at> gmail.com
> >
> > I think I have again lost motivation to pursue this further, since gfile
> > works fine.  And glib is required for the default Cygwin-w32 build
> > anyway, because ImageMagick brings it in, so why not just use it?
>
> Perhaps the OP could explain why he wanted that.
>

If you're referring to me, my motivation was to prevent a compilation error
for an incorrect configure combination. I believe you started us all down
the path of trying to get w32 notifying to work. I have no problem with
using glib.

Robert

[Message part 2 (text/html, inline)]

Message #63 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Robert Pluim <rpluim <at> gmail.com>
Cc: bug-gnu-emacs <at> gnu.org, kbrown <at> cornell.edu
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is
 specified on cygwin
Date: Sat, 28 Feb 2015 19:57:01 +0200

> Date: Sat, 28 Feb 2015 17:56:28 +0100
> From: Robert Pluim <rpluim <at> gmail.com>
> Cc: Ken Brown <kbrown <at> cornell.edu>, bug-gnu-emacs <at> gnu.org
> 
>     Perhaps the OP could explain why he wanted that.
> 
> If you're referring to me, my motivation was to prevent a compilation error for
> an incorrect configure combination. I believe you started us all down the path
> of trying to get w32 notifying to work. I have no problem with using glib.

No, I meant your reasons for specifying --with-file-notification=w32.
You could only bump into this issue if you specified it, AFAIU.

Message #66 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ken Brown <kbrown <at> cornell.edu>
To: Eli Zaretskii <eliz <at> gnu.org>, Robert Pluim <rpluim <at> gmail.com>
Cc: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is
 specified on cygwin
Date: Sat, 28 Feb 2015 13:44:38 -0500

On 2/28/2015 12:57 PM, Eli Zaretskii wrote:
>> Date: Sat, 28 Feb 2015 17:56:28 +0100
>> From: Robert Pluim <rpluim <at> gmail.com>
>> Cc: Ken Brown <kbrown <at> cornell.edu>, bug-gnu-emacs <at> gnu.org
>>
>>      Perhaps the OP could explain why he wanted that.
>>
>> If you're referring to me, my motivation was to prevent a compilation error for
>> an incorrect configure combination. I believe you started us all down the path
>> of trying to get w32 notifying to work. I have no problem with using glib.
>
> No, I meant your reasons for specifying --with-file-notification=w32.
> You could only bump into this issue if you specified it, AFAIU.

Based on http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19909#36, I think 
he specified --with-file-notification=w32 because he wanted file 
notification and didn't know he could use gfile.  Robert, please correct 
me if I'm misinterpreting what you wrote.

I'm inclined to go ahead with the revised patch he submitted in the 
above URL.  I'll commit it (with Robert as author and with the change 
marked as a "tiny change") if that's OK with you, Eli.

Two questions:

1. Shouldn't the name in the ChangeLog be "Robert Pluim" instead of 
"rpluim" to match all the other entries?

2. I tested making a git commit with

  --author="Robert Pluim <rpluim <at> gmail.com>  (tiny change)"

but git stripped away the "(tiny change)" part.  What should I do to get 
"tiny change" into the git history?  Maybe it's not so important now, 
because I can put it into the ChangeLog, but I'm thinking about the 
future, when ChangeLogs are automatically generated.

Ken

Message #69 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: rpluim <at> gmail.com, bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is
 specified on cygwin
Date: Sat, 28 Feb 2015 21:14:06 +0200

> Date: Sat, 28 Feb 2015 13:44:38 -0500
> From: Ken Brown <kbrown <at> cornell.edu>
> CC: bug-gnu-emacs <at> gnu.org
> 
> > No, I meant your reasons for specifying --with-file-notification=w32.
> > You could only bump into this issue if you specified it, AFAIU.
> 
> Based on http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19909#36, I think 
> he specified --with-file-notification=w32 because he wanted file 
> notification and didn't know he could use gfile.

Isn't gfile the default for Cygwin?

> I'm inclined to go ahead with the revised patch he submitted in the 
> above URL.  I'll commit it (with Robert as author and with the change 
> marked as a "tiny change") if that's OK with you, Eli.

If that's what Cygwin users want, fine with me.

> 1. Shouldn't the name in the ChangeLog be "Robert Pluim" instead of 
> "rpluim" to match all the other entries?

I don't think it matters when there's no assignment on file.

> 2. I tested making a git commit with
> 
>    --author="Robert Pluim <rpluim <at> gmail.com>  (tiny change)"
> 
> but git stripped away the "(tiny change)" part.  What should I do to get 
> "tiny change" into the git history?  Maybe it's not so important now, 
> because I can put it into the ChangeLog, but I'm thinking about the 
> future, when ChangeLogs are automatically generated.

The projects I know of use

  Copyright-paperwork-exempt: yes

on a separate line in the log message.  You will see that I already
used that in Emacs, at least once.

Message #72 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ken Brown <kbrown <at> cornell.edu>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: rpluim <at> gmail.com, bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is
 specified on cygwin
Date: Sat, 28 Feb 2015 15:14:48 -0500

On 2/28/2015 2:14 PM, Eli Zaretskii wrote:
>> Date: Sat, 28 Feb 2015 13:44:38 -0500
>> From: Ken Brown <kbrown <at> cornell.edu>
>> CC: bug-gnu-emacs <at> gnu.org
>>
>>> No, I meant your reasons for specifying --with-file-notification=w32.
>>> You could only bump into this issue if you specified it, AFAIU.
>>
>> Based on http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19909#36, I think
>> he specified --with-file-notification=w32 because he wanted file
>> notification and didn't know he could use gfile.
>
> Isn't gfile the default for Cygwin?

Yes.  (Of course, the builder has to have the appropriate glib 
development package installed, or else file notification won't actually 
be used.)

>> I'm inclined to go ahead with the revised patch he submitted in the
>> above URL.  I'll commit it (with Robert as author and with the change
>> marked as a "tiny change") if that's OK with you, Eli.
>
> If that's what Cygwin users want, fine with me.

I personally don't care, but Robert requested it, and I don't see any 
harm in it.  So I'll go ahead as soon as I hear back from him as to how 
he wants his name shown.

>> 1. Shouldn't the name in the ChangeLog be "Robert Pluim" instead of
>> "rpluim" to match all the other entries?
>
> I don't think it matters when there's no assignment on file.
>
>> 2. I tested making a git commit with
>>
>>     --author="Robert Pluim <rpluim <at> gmail.com>  (tiny change)"
>>
>> but git stripped away the "(tiny change)" part.  What should I do to get
>> "tiny change" into the git history?  Maybe it's not so important now,
>> because I can put it into the ChangeLog, but I'm thinking about the
>> future, when ChangeLogs are automatically generated.
>
> The projects I know of use
>
>    Copyright-paperwork-exempt: yes
>
> on a separate line in the log message.  You will see that I already
> used that in Emacs, at least once.

Thanks.

Ken

Message #75 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: Eli Zaretskii <eliz <at> gnu.org>, bug-gnu-emacs <at> gnu.org
Subject: Re: bug#19909: [PATCH] Error out if with-file-notification=w32 is
 specified on cygwin
Date: Mon, 2 Mar 2015 10:35:50 +0100

[Message part 1 (text/plain, inline)]

On Sat, Feb 28, 2015 at 9:14 PM, Ken Brown <kbrown <at> cornell.edu> wrote:

> On 2/28/2015 2:14 PM, Eli Zaretskii wrote:
>
>> Date: Sat, 28 Feb 2015 13:44:38 -0500
>>> From: Ken Brown <kbrown <at> cornell.edu>
>>> CC: bug-gnu-emacs <at> gnu.org
>>>
>>>  No, I meant your reasons for specifying --with-file-notification=w32.
>>>> You could only bump into this issue if you specified it, AFAIU.
>>>>
>>>
>>> Based on http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19909#36, I think
>>> he specified --with-file-notification=w32 because he wanted file
>>> notification and didn't know he could use gfile.
>>>
>>
>>
Exactly


> Isn't gfile the default for Cygwin?
>>
>
> Yes.  (Of course, the builder has to have the appropriate glib development
> package installed, or else file notification won't actually be used.)
>
>  I'm inclined to go ahead with the revised patch he submitted in the
>>> above URL.  I'll commit it (with Robert as author and with the change
>>> marked as a "tiny change") if that's OK with you, Eli.
>>>
>>
>> If that's what Cygwin users want, fine with me.
>>
>
> I personally don't care, but Robert requested it, and I don't see any harm
> in it.  So I'll go ahead as soon as I hear back from him as to how he wants
> his name shown.
>
>  1. Shouldn't the name in the ChangeLog be "Robert Pluim" instead of
>>> "rpluim" to match all the other entries?
>>>
>>
>> I don't think it matters when there's no assignment on file.
>>
>
I'd prefer "Robert Pluim" but it's not a big deal. 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"


>
>>  2. I tested making a git commit with
>>>
>>>     --author="Robert Pluim <rpluim <at> gmail.com>  (tiny change)"
>>>
>>> but git stripped away the "(tiny change)" part.  What should I do to get
>>> "tiny change" into the git history?  Maybe it's not so important now,
>>> because I can put it into the ChangeLog, but I'm thinking about the
>>> future, when ChangeLogs are automatically generated.
>>>
>>
>> The projects I know of use
>>
>>    Copyright-paperwork-exempt: yes
>>
>> on a separate line in the log message.  You will see that I already
>> used that in Emacs, at least once.
>>
>
I'll test the waters with my future employers as to copyright assignment,
and this discussion will hopefully not have to happen again :)

Regards

Robert

[Message part 2 (text/html, inline)]

Message #80 received at 19909-done <at> debbugs.gnu.org (full text, mbox):

From: Ken Brown <kbrown <at> cornell.edu>
To: Robert Pluim <rpluim <at> gmail.com>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 19909-done <at> 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

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

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.