unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Drew Adams <drew.adams@oracle.com>
Cc: 23604@debbugs.gnu.org, eggert@cs.ucla.edu
Subject: bug#23604: desktop-restore-in-current-display should default to t
Date: Mon, 23 May 2016 21:19:49 +0300	[thread overview]
Message-ID: <831t4s63i2.fsf@gnu.org> (raw)
In-Reply-To: <76836464-8187-469c-9ddd-df27b6da1acc@default> (message from Drew Adams on Mon, 23 May 2016 10:47:36 -0700 (PDT))

> Date: Mon, 23 May 2016 10:47:36 -0700 (PDT)
> From: Drew Adams <drew.adams@oracle.com>
> Cc: eggert@cs.ucla.edu, 23604@debbugs.gnu.org
> 
> > Leaving it at nil makes no sense to me.  It means that you invoke
> > Emacs, which will then display on a different terminal -- could be a
> > terminal half a world away.  That could be something that is desired
> > in some specialized use cases, but makes no sense at all as the
> > default.
> > 
> > This problem will exist on any platform that supports X displays,
> > i.e. on any Posix system.
> > 
> > > Anyway, the right course of action now (if Emacs 25.1 is released
> > > with the default value changed to t) is (after the 25.1 release)
> > > to fix bug #20247 and revert the default value to nil.
> > 
> > No, the right course is to set it to t and leave it at that.
> 
> You elided this, just after that paragraph:
> 
>   At that point, if there is disagreement about the default value,
>   ^^^^^^^^^^^^^
>   i.e., if someone really thinks it should be t by design (and not
>   just as a workaround), then we can discuss the pros & cons for
>   the default behavior.

And you elided the part where you claim that no arguments were
presented in favor of the non-nil value.

> The proper default value to use, once bug #20274 is fixed, should
> be discussed in that context, not in the context of this temporary
> workaround (#23604).
> 
> Let's have that discussion there and then.

I don't see any point in delaying the discussion.  The correct default
value is clear already.

> It should be possible to try that and fall back if it is not
> possible for some reason (e.g. the display does not exist).

Feel free to suggest such a test.

> But we should discuss this later and not here.  This bug is only
> about the temporary workaround of setting the value to `t' _to
> avoid bug #20274_ rearing its head by default.

I don't see that as a temporary workaround, I think the default should
be t regardless.





  reply	other threads:[~2016-05-23 18:19 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <<278b113b-21aa-5c54-6550-79bf1e481530@cs.ucla.edu>
     [not found] ` <<f47f4679-8838-4410-968b-c3e9335fa806@default>
     [not found]   ` <<83a8jg65vd.fsf@gnu.org>
2016-05-23 17:47     ` bug#23604: desktop-restore-in-current-display should default to t Drew Adams
2016-05-23 18:19       ` Eli Zaretskii [this message]
     [not found] <<<<278b113b-21aa-5c54-6550-79bf1e481530@cs.ucla.edu>
     [not found] ` <<<<f47f4679-8838-4410-968b-c3e9335fa806@default>
     [not found]   ` <<<<83a8jg65vd.fsf@gnu.org>
     [not found]     ` <<<76836464-8187-469c-9ddd-df27b6da1acc@default>
     [not found]       ` <<<831t4s63i2.fsf@gnu.org>
     [not found]         ` <<48df821e-5abc-4e05-8b1e-82365ff6f15c@default>
     [not found]           ` <<83wpmk426m.fsf@gnu.org>
2016-05-24  4:43             ` Drew Adams
     [not found] <<<278b113b-21aa-5c54-6550-79bf1e481530@cs.ucla.edu>
     [not found] ` <<<f47f4679-8838-4410-968b-c3e9335fa806@default>
     [not found]   ` <<<83a8jg65vd.fsf@gnu.org>
     [not found]     ` <<76836464-8187-469c-9ddd-df27b6da1acc@default>
     [not found]       ` <<831t4s63i2.fsf@gnu.org>
2016-05-23 20:33         ` Drew Adams
2016-05-24  2:31           ` Eli Zaretskii
2016-05-23 16:37 Paul Eggert
2016-05-23 17:13 ` Drew Adams
2016-05-23 17:28   ` Eli Zaretskii
2016-05-23 18:07   ` Paul Eggert
2016-05-23 20:33     ` Drew Adams
2016-05-23 21:01       ` Paul Eggert
2016-05-23 21:11         ` Drew Adams
2016-05-24  0:23           ` Paul Eggert
2016-05-24  0:37             ` Juanma Barranquero
2016-05-24  0:54               ` Drew Adams
2016-05-30  5:50 ` Paul Eggert

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=831t4s63i2.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=23604@debbugs.gnu.org \
    --cc=drew.adams@oracle.com \
    --cc=eggert@cs.ucla.edu \
    /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).