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


View this message in rfc822 format

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

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.