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

Previous Next

Package: emacs;

Reported by: bug-gnu-emacs <at> gnu.org

Date: Fri, 20 Feb 2015 09:49:02 UTC

Severity: minor

Tags: patch

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

Bug is archived. No further changes may be made.

Full log


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




This bug report was last modified 10 years and 79 days ago.

Previous Next


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