unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Ulrich Mueller <ulm@kph.uni-mainz.de>
To: "Juanma Barranquero" <lekktu@gmail.com>
Cc: Eli Zaretskii <eliz@gnu.org>, emacs-devel@gnu.org
Subject: Re: error in server-running-p on M$
Date: Sat, 22 Nov 2008 17:52:15 +0100	[thread overview]
Message-ID: <18728.14527.648410.433140@a1ihome1.kph.uni-mainz.de> (raw)
In-Reply-To: <f7ccd24b0811220804h1daa5cafpcb3ed1387a6f921a@mail.gmail.com>

>>>>> On Sat, 22 Nov 2008, Juanma Barranquero wrote:

>> With the proposed test, server-running-p will return nil in this case,
>> since it will find an auth file but no associated Emacs process.

> It only will do that if the current, local server is trying to write
> an authentication file in the NFS server.

If you assume that the user's home directory is NFS mounted, then this
is the normal case.

> Still, the point is, either we try to do a complicate,
> potentially-network-delaying check, or we opt for a simple,
> not-totally-reliable check.

Yeah, I see the problem that one wouldn't like to have network
timeouts here ...

> I don't mind one way or the other (though I won't be writing the
> complex one), but we should bear in mind that even if the check
> fails, consequences are hardly catastrophic. Currently we don't even
> check, just clobber the auth file. So any partial fix is a net
> benefit, as long as we rename the function and/or write a clear
> docstring stating that it cannot be assumed to be a reliable
> indicator of the presence of another Emacs instance.

Agreed, of course a partial fix is better than not checking at all.
And I would vote against renaming the function, since I think its
intention is to find out if a server is running. Even if the
implementation (at the time being?) uses some heuristic that doesn't
cover 100% of all cases.

Another question, is the plan now to call server-running-p from
server-start (and refuse to start if it is returning non-nil)?

Ulrich




  reply	other threads:[~2008-11-22 16:52 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-22 11:44 error in server-running-p on M$ dhruva
2008-11-22 11:58 ` Juanma Barranquero
2008-11-22 12:51   ` Eli Zaretskii
2008-11-22 13:00     ` Juanma Barranquero
2008-11-22 13:20       ` Eli Zaretskii
2008-11-22 13:37         ` Juanma Barranquero
2008-11-22 13:53           ` Eli Zaretskii
2008-11-22 14:13             ` Ulrich Mueller
2008-11-22 14:17               ` Juanma Barranquero
2008-11-22 14:29                 ` Ulrich Mueller
2008-11-22 14:36                   ` Juanma Barranquero
2008-11-22 15:35                     ` Ulrich Mueller
2008-11-22 16:04                       ` Juanma Barranquero
2008-11-22 16:52                         ` Ulrich Mueller [this message]
2008-11-22 21:47                           ` Juanma Barranquero
2008-11-23  3:56                             ` dhruva
2008-11-23  3:59                               ` Juanma Barranquero
2008-11-23  4:01                                 ` dhruva
2008-11-23  4:03                                   ` Juanma Barranquero
2008-11-23  4:10                                     ` dhruva
2008-11-23  4:15                                       ` Juanma Barranquero
2008-11-23  4:17                                         ` dhruva
2008-11-23  4:19                                           ` Juanma Barranquero
2008-11-23  8:14                                             ` Server protocol (was: Re: error in server-running-p on M$) Ulrich Mueller
2008-11-23 11:33                                               ` Juanma Barranquero
2008-11-23 16:33                                                 ` dhruva
2008-11-23 17:48                                                   ` Juanma Barranquero
2008-11-23  4:02                 ` error in server-running-p on M$ Stefan Monnier
2008-11-23  4:10                   ` Juanma Barranquero
2008-11-23  5:21                     ` Stefan Monnier
2008-11-23  5:32                       ` Juanma Barranquero
2008-11-23  5:57                         ` Stefan Monnier
2008-11-23 11:26                           ` Juanma Barranquero
2008-11-23 12:46                             ` Juanma Barranquero
2008-11-23 13:13                               ` Ulrich Mueller
2008-11-24  2:55                               ` Stefan Monnier
2008-11-24 14:46                                 ` Juanma Barranquero
2008-11-24 15:10                                   ` Ulrich Mueller
2008-11-24 15:16                                     ` Juanma Barranquero
2008-11-24 15:34                                       ` Ulrich Mueller
2008-11-24 15:45                                         ` Juanma Barranquero
2008-12-09 20:46                                   ` Stefan Monnier
2008-12-10  7:05                                     ` Chetan Pandya
2008-12-10 12:59                                     ` Juanma Barranquero
2008-12-10 18:30                                       ` Stefan Monnier
2008-12-10 18:47                                         ` Juanma Barranquero
2008-12-11  2:07                                           ` Stefan Monnier
2008-12-11 16:30                                             ` Juanma Barranquero
2008-12-11 18:47                                               ` Stefan Monnier
2008-12-12  0:40                                                 ` Juanma Barranquero
2008-12-12  4:51                                                   ` Stefan Monnier
2008-12-12  8:19                                                     ` Juanma Barranquero
2008-12-12 18:49                                                       ` Stefan Monnier
2008-12-12 20:20                                                         ` Juanma Barranquero
2008-11-23  3:58               ` Stefan Monnier
2008-11-23  3:59           ` Stefan Monnier
2008-11-23  4:12             ` Juanma Barranquero
2008-11-23  5:19               ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=18728.14527.648410.433140@a1ihome1.kph.uni-mainz.de \
    --to=ulm@kph.uni-mainz.de \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=lekktu@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).