all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Peter Oliver <p.d.oliver@mavit.org.uk>
Cc: 49195@debbugs.gnu.org
Subject: bug#49195: Starting Emacs if required from emacsclient.desktop
Date: Thu, 24 Jun 2021 09:07:01 +0300	[thread overview]
Message-ID: <83im23u77e.fsf@gnu.org> (raw)
In-Reply-To: <c7e5066-2818-cf15-8e3b-e4c6af9354f4@froglet.home.mavit.org.uk> (message from Peter Oliver on Wed, 23 Jun 2021 20:59:31 +0100 (BST))

> Date: Wed, 23 Jun 2021 20:59:31 +0100 (BST)
> From: Peter Oliver <p.d.oliver@mavit.org.uk>
> 
> Attached is a patch which enhances emacsclient.desktop so that it will start a new instance of Emacs if none is found to be running.  Users who like to reuse an existing Emacs can then always use emacsclient.desktop.

FWIW, I would find this change surprising, and therefore would suggest
to leave this off by default.  But that's me.

> There was some discussion of how far to go with this on emacs-devel recently, but, if I understand correctly, the parts in this patch were not controversial.

How do you understand that?  My recollection was that there was no
consensus on this, but my memory is not what it used to be.

> +  If you launch Emacs from a desktop environment, you may use the

"Desktop environment" is too general.  You mean GNU/Linux desktop
environment, and possibly just the ones compatible with
freedesktop.org?  The text should make that clear.

More generally, I'm not sure we should even say this in the Emacs
manual, as these features belong to the desktop, not to Emacs.  What
is the advantage of having this text in our manual?

> +*** New desktop integration for connecting to the server.
> +You may use the "Emacs (Client)" menu entry to open files in an
> +existing Emacs instance rather than starting a new one.

This isn't detailed enough, especially if we decide not to say
anything about that in the manual.  And even if we leave that text in
the manual, it is buried in a place where finding it isn't easy, so
NEWS should be more detailed.

Thanks.





  reply	other threads:[~2021-06-24  6:07 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-23 19:59 bug#49195: Starting Emacs if required from emacsclient.desktop Peter Oliver
2021-06-24  6:07 ` Eli Zaretskii [this message]
2021-06-24 13:28   ` Peter Oliver
2021-06-24 14:22     ` Eli Zaretskii
2021-06-24 15:50       ` Peter Oliver
2021-06-30 13:12         ` Lars Ingebrigtsen

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

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

  git send-email \
    --in-reply-to=83im23u77e.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=49195@debbugs.gnu.org \
    --cc=p.d.oliver@mavit.org.uk \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.