bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Ken Brown
Subject: bug#19909: [PATCH] Error out if with-file-notification=w32 is specified on cygwin
Date: Sat, 28 Feb 2015 13:44:38 -0500
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0

On 2/28/2015 12:57 PM, Eli Zaretskii wrote:
Date: Sat, 28 Feb 2015 17:56:28 +0100
From: Robert Pluim <rpluim@gmail.com>
Cc: Ken Brown <kbrown@cornell.edu>, bug-gnu-emacs@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@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





reply via email to

[Prev in Thread] Current Thread [Next in Thread]