GNU bug report logs - #6828
24.0.50; [PATCH] rcirc process buffer log filenames invalid on Windows

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 6828 in the body.
You can then email your comments to 6828 AT debbugs.gnu.org in the normal way.

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

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

From: Deniz Dogan <deniz.a.m.dogan <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org, Ryan Yeske <rcyeske <at> gmail.com>
Subject: 24.0.50; [PATCH] rcirc process buffer log filenames invalid on 
	Windows
Date: Mon, 9 Aug 2010 02:28:26 +0200

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

The rcirc logging facility doesn't play nicely when it comes to
writing process buffers (or "server buffers") to disk on Windows
systems. The generated log filenames contain asterisks which are not
allowed on Windows systems. This makes rcirc and Emacs as a whole
unusable when rcirc-log-flag is non-nil, i.e. when logging to disk is
turned on. Emacs just messages errors in the echo area when this
happens and you can't easily stop it.

Attached is a patch which introduces a new function and a new
customizable variable.

rcirc-generate-log-filename is the function which takes care of
generating filenames for rcirc buffer logs. It is now the default
instead of the old rcirc-generate-new-buffer-name.

rcirc-log-process-buffers is the new variable which decides whether or
not to log process buffers to disk at all. I get the feeling that not
that many people are interested in logging this as it rarely contains
any useful information that you would want to store. Nevertheless, the
variable is non-nil by default which turns on process buffer logging.

The setup I'm using by the way:

In GNU Emacs 24.0.50.1 (i386-mingw-nt6.1.7600)
 of 2010-08-02 on 3249CTO
Windowing system distributor `Microsoft Corp.', version 6.1.7600
configured using `configure --with-gcc (4.4) --no-opt --cflags -Ic:/xpm/include'

-- 
Deniz Dogan

[rcirc-log-patch-dd.diff (application/octet-stream, attachment)]

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

From: Chong Yidong <cyd <at> stupidchicken.com>
To: Ryan Yeske <rcyeske <at> gmail.com>
Cc: 6828 <at> debbugs.gnu.org
Subject: Re: 24.0.50;
	[PATCH] rcirc process buffer log filenames invalid on Windows
Date: Sat, 14 Aug 2010 19:07:13 -0400

Hi Ryan,

Could you review the patch at

  http://debbugs.gnu.org/cgi/bugreport.cgi?bug=6828

?  Thanks.

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

From: Ryan Yeske <rcyeske <at> gmail.com>
To: 6828 <at> debbugs.gnu.org, cyd <at> stupidchicken.com
Date: Sun, 15 Aug 2010 20:26:52 -0700

From: Ryan Yeske <rcyeske <at> gmail.com>
To: Chong Yidong <cyd <at> stupidchicken.com>
Cc: 6828 <at> debbugs.gnu.org
Subject: Re: 24.0.50; [PATCH] rcirc process buffer log filenames invalid on Windows
References: <87d3tk7pzy.fsf <at> stupidchicken.com>
Date: Sun, 15 Aug 2010 20:26:52 -0700
In-Reply-To: <87d3tk7pzy.fsf <at> stupidchicken.com> (Chong Yidong's message of
	"Sat, 14 Aug 2010 19:07:13 -0400")
Message-ID: <878w47s0eb.fsf <at> gmail.com>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii

These are good changes.  

I agree with Deniz that logging the process buffer is hardly ever
desirable, so I suggest `rcirc-log-process-buffers' should be nil by
default.

Ryan

Chong Yidong <cyd <at> stupidchicken.com> writes:

> Hi Ryan,
>
> Could you review the patch at
>
>   http://debbugs.gnu.org/cgi/bugreport.cgi?bug=6828
>
> ?  Thanks.

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

From: Chong Yidong <cyd <at> stupidchicken.com>
To: Ryan Yeske <rcyeske <at> gmail.com>
Cc: 6828 <at> debbugs.gnu.org, Deniz Dogan <deniz.a.m.dogan <at> gmail.com>
Subject: Re: 24.0.50;
	[PATCH] rcirc process buffer log filenames invalid on Windows
Date: Sun, 15 Aug 2010 23:35:27 -0400

Ryan Yeske <rcyeske <at> gmail.com> writes:

> These are good changes.
>
> I agree with Deniz that logging the process buffer is hardly ever
> desirable, so I suggest `rcirc-log-process-buffers' should be nil by
> default.

OK.  I've checked the patch into the trunk, with
rcirc-log-process-buffers defaulting to nil.  Thanks.

Previous Next

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