all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Noam Postavsky <npostavs@gmail.com>
To: Aaron Jensen <aaronjensen@gmail.com>
Cc: 35726@debbugs.gnu.org, changqi chen <ccq777@gmail.com>
Subject: bug#35726: 27.0.50; global-linum-mode doesn't work in emacsclient in several modes
Date: Wed, 15 May 2019 18:23:56 -0400	[thread overview]
Message-ID: <87ef4zfjb7.fsf_-_@gmail.com> (raw)
In-Reply-To: <CAHyO48zg15vHEY+tYDmes+hNhz3NTQNtu5rLDap3Lhgo14KXXQ@mail.gmail.com> (Aaron Jensen's message of "Wed, 15 May 2019 12:21:42 -0700")

Aaron Jensen <aaronjensen@gmail.com> writes:

>>  3. Remove from linum.el and display-line-numbers.el the condition
>>     that causes this:
>>
>>        (and (daemonp) (null (frame-parameter nil 'client)))
>>
>>     I don't think that bug#27210 will affect display-line-numbers, and
>>     linum is semi-deprecated.
>
> For what it's worth, I'd vote for #3 since linum is semi-deprecated
> and I rather like the change, so I'd prefer not to revert it.

I think removing the check from display-line-numbers.el would make
sense, but IMO we should leave it in linum.el because having Emacs
freeze on startup is worse than having to toggle global-linum-mode when
using emacsclient.

Or if we really want to remove it from linum.el, there is my patch at
https://debbugs.gnu.org/27210#28 which can fix that bug without the
above check.






  reply	other threads:[~2019-05-15 22:23 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-14  2:07 bug#35726: 27.0.50; global-linum-mode doesn't work in emacsclient in several modes ccQ's Gmail
2019-05-14  4:21 ` bug#35726: Further information of line number bug changqi chen
2019-05-15 15:32   ` Eli Zaretskii
2019-05-15 17:35     ` npostavs
2019-05-15 18:05       ` Eli Zaretskii
2019-05-15 19:21     ` Aaron Jensen
2019-05-15 22:23       ` Noam Postavsky [this message]
2019-06-07 14:33         ` bug#35726: 27.0.50; global-linum-mode doesn't work in emacsclient in several modes Eli Zaretskii

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=87ef4zfjb7.fsf_-_@gmail.com \
    --to=npostavs@gmail.com \
    --cc=35726@debbugs.gnu.org \
    --cc=aaronjensen@gmail.com \
    --cc=ccq777@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 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.