all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Dima Kogan <dima@secretsauce.net>
Cc: 21777@debbugs.gnu.org, schwab@linux-m68k.org
Subject: bug#21777: 25.0.50; gud-gdb uses a pager, which is harmful inside emacs
Date: Fri, 30 Oct 2015 11:32:47 +0200	[thread overview]
Message-ID: <838u6k1z28.fsf@gnu.org> (raw)
In-Reply-To: <87lhakhg81.fsf@secretsauce.net>

> From: Dima Kogan <dima@secretsauce.net>
> Cc: schwab@linux-m68k.org, 21777@debbugs.gnu.org
> Date: Fri, 30 Oct 2015 02:13:02 -0700
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > What if the user has "set height 100" in their ~/.gdbinit, or in the
> > system-wide gdbinit file? Will those settings be overridden? If so, we
> > cannot fix the problem this way, not by default anyway.
> 
> Yes. This overrides any user settings. I really can't imagine why
> somebody would want a pager inside emacs, but maybe I just need more
> imagination.

IME, users sometimes have some use cases that look really weird to me,
but are somehow very important to them.  So I'm trying to avoid
stepping on their toes as much as possible, even if I cannot imagine
those use cases in advance.

> So I'm in favor of overriding the user defaults here. Otherwise, how
> about this:
> 
> - we have a variable 'gud-gdb-set-height-unlimited',
>   which has 3 states: uninitialized, yes, no
> 
> - when gud starts up, if it's 'uninitialized', we ask the user if they
>   want to override, and whether to do so in the future; if they say yes,
>   we update their .emacs.d/init.el. narrow-to-region has this type of
>   user querying. We override only if it's 'yes'

Why not a simpler boolean, off by default?  This problem will go away
soon enough, so maybe solutions that are too complicated would be
over-engineering it?

Btw, does gud.el know the version of GDB it runs?  If so, this option
should be a no-op for GDB 7.11 and later.

> What do we do for vc-mode interaction with git? It has a similar
> situation where a user can configure git to use a pager. But in that
> case we completely override that setting, don't we? If so, that would be
> a precedent to handle gdb in the same way.

In vc-git.el, we don't present a CLI-like interface to Git, we just
consume all the Git output and then process it.  So the situation
there is slightly different, I think.  But I will let other chime in
and express their opinions.

Thanks.





  reply	other threads:[~2015-10-30  9:32 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-28 19:20 bug#21777: 25.0.50; gud-gdb uses a pager, which is harmful inside emacs Dima Kogan
2015-10-28 22:27 ` Andreas Schwab
2015-10-28 22:57   ` Dima Kogan
2015-10-29 16:16     ` Eli Zaretskii
2015-10-29 22:58       ` Dima Kogan
2015-10-30  3:43         ` Dima Kogan
2015-10-30  8:17           ` Eli Zaretskii
2015-10-30  9:13             ` Dima Kogan
2015-10-30  9:32               ` Eli Zaretskii [this message]
2015-10-30 14:01                 ` Eli Zaretskii
2015-10-30 19:05                   ` Dima Kogan
2015-10-30 20:34                     ` Eli Zaretskii
2020-09-20 21:18                 ` Lars Ingebrigtsen
2020-09-21 14:03                   ` Eli Zaretskii
2020-09-21 14:07                     ` Lars Ingebrigtsen
2020-09-22 15:46                       ` Eli Zaretskii
2020-09-22 15:51                         ` Lars Ingebrigtsen
2020-09-22 15:57                           ` Andreas Schwab
2020-09-22 16:09                           ` Eli Zaretskii
2020-09-22 16:19                             ` Andreas Schwab
2020-09-22 16:51                               ` Eli Zaretskii
2020-09-22 17:18                                 ` Andreas Schwab
2020-09-22 17:39                                   ` Eli Zaretskii
2020-09-22 17:46                                     ` Andreas Schwab
2020-09-23 12:53                             ` Lars Ingebrigtsen
2020-09-23 14:41                               ` Eli Zaretskii
2020-09-24 14:15                                 ` Lars Ingebrigtsen
2020-09-24 15:06                                   ` Eli Zaretskii
2020-09-25  9:55                                     ` Lars Ingebrigtsen
2020-09-24 14:16                               ` Eli Zaretskii
2020-09-24 14:29                                 ` Lars Ingebrigtsen
2020-09-24 15:08                                   ` Eli Zaretskii
2015-10-30  7:54         ` Eli Zaretskii
2019-09-29 21:49 ` Stefan Kangas

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=838u6k1z28.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=21777@debbugs.gnu.org \
    --cc=dima@secretsauce.net \
    --cc=schwab@linux-m68k.org \
    /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.