GNU bug report logs - #10931
23.4; inconsistency between open-tls-stream and open-gnutls-stream

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 10931 in the body.
You can then email your comments to 10931 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: Leo <sdl.web <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 23.4; inconsistency between open-tls-stream and open-gnutls-stream
Date: Sat, 03 Mar 2012 17:24:35 +0800

I started using encrypted connection in rcirc and noticed the following
difference between Emacs built with gnutls and one without gnutls. The
one without gnutls kills my irc process implicitly when I exit emacs.

This is due to the following form in open-tls-stream:

(funcall (if (fboundp 'set-process-query-on-exit-flag)
		       'set-process-query-on-exit-flag
		     'process-kill-without-query)
		   process nil)

This inconsistency is annoying and has caused me to lose conversations
in IRC. I wonder if the behaviour in these two functions can be unified.

Thanks,
Leo

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

From: Lars Magne Ingebrigtsen <larsi <at> gnus.org>
To: Leo <sdl.web <at> gmail.com>
Cc: Stefan Monnier <monnier <at> iro.umontreal.ca>, 10931 <at> debbugs.gnu.org
Subject: Re: bug#10931: 23.4;
	inconsistency between open-tls-stream and open-gnutls-stream
Date: Sat, 03 Mar 2012 14:58:50 +0100

Leo <sdl.web <at> gmail.com> writes:

> This is due to the following form in open-tls-stream:
>
> (funcall (if (fboundp 'set-process-query-on-exit-flag)
> 		       'set-process-query-on-exit-flag
> 		     'process-kill-without-query)
> 		   process nil)

Yes, setting this here in this low-level part of `open-tls-stream' is
just not a good idea.

The changelog just says

+2009-02-07  Dave Love  <fx <at> gnu.org>
+
+	* tls.el (open-tls-stream): Don't query killing process.
+

which isn't very helpful, but looking at the code, I think I can
understand why it's doing this.  `open-tls-stream' needs (in some error
cases) to kill off the process/buffer it's created, and create a new
one.  In these error cases, having the user prompted is a bug.  I
think.  Possibly.  Although I can't quite see why, really.

But that's what should be fixed, and the
`set-process-query-on-exit-flag' shouldn't be done.  I'll see whether I
can dig up the original bug report to see what the real problem was...

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/

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

From: Lars Magne Ingebrigtsen <larsi <at> gnus.org>
To: Leo <sdl.web <at> gmail.com>
Cc: Stefan Monnier <monnier <at> iro.umontreal.ca>, 10931 <at> debbugs.gnu.org
Subject: Re: bug#10931: 23.4;
	inconsistency between open-tls-stream and open-gnutls-stream
Date: Sat, 03 Mar 2012 15:05:49 +0100

Lars Magne Ingebrigtsen <larsi <at> gnus.org> writes:

> But that's what should be fixed, and the
> `set-process-query-on-exit-flag' shouldn't be done.  I'll see whether I
> can dig up the original bug report to see what the real problem was...

The original report didn't really say anything about what problem it was
supposed to fix, except "consistency", so I've now reverted it.

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/

Previous Next

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