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

Previous Next

Full log

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)]

Previous Next

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