all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Glenn Morris <rgm@gnu.org>
To: Toby Cubitt <toby-dated-1351439087.748fe4@dr-qubit.org>
Cc: 12647@debbugs.gnu.org
Subject: bug#12647: 24.2.50; emacs --daemon broken in tty
Date: Tue, 16 Oct 2012 03:00:21 -0400	[thread overview]
Message-ID: <v5a9vm7vga.fsf@fencepost.gnu.org> (raw)
In-Reply-To: <20121014154458.GA14676@c3po> (Toby Cubitt's message of "Sun, 14 Oct 2012 17:44:58 +0200")

Toby Cubitt wrote:

> "emacsclient -c" throws an error if "emacs --daemon" is started outside
> of X windows, . This is on a recent bzr checkout (from today).
>
> Steps to reproduce:
>
> 1. switch to a linux virtual tty
>
> 2. emacs -Q --daemon
>
> 3. switch to X windows
>
> 4. emacsclient -c aborts with the following error:
>
>    Waiting for Emacs...
>    *ERROR*: Wrong type argument: stringp, nil

This is due to the various changes related to display handling in
r110444 (cygw32). The specific error is from x-initialize-window-system,
which cannot find a display (don't ask me why getenv DISPLAY doesn't
work here). In the past, frame.el set x-display-name, but it does not do
that any more. Presumably it was doing it for a reason.

The following example frame.el change makes it work again, but someone
should check this area.

--- lisp/frame.el   2012-09-17 12:07:36 +0000
+++ lisp/frame.el   2012-10-16 06:54:40 +0000
@@ -655,6 +655,7 @@
       (error "Don't know how to create a frame on window system %s" w))
 
     (unless (get w 'window-system-initialized)
+      (or x-display-name (setq x-display-name display))
       (funcall (cdr (assq w window-system-initialization-alist)))
       (put w 'window-system-initialized t))
 





  reply	other threads:[~2012-10-16  7:00 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-14 15:44 bug#12647: 24.2.50; emacs --daemon broken in tty Toby Cubitt
2012-10-16  7:00 ` Glenn Morris [this message]
2012-10-16 13:20   ` Stefan Monnier
2012-10-16 15:47     ` Glenn Morris
2012-10-16 21:34       ` Stefan Monnier
2012-10-16 21:42         ` Toby Cubitt
2012-10-18 23:46         ` Glenn Morris
2012-10-18 23:56 ` Ulrich Mueller
2012-10-19  0:23   ` Daniel Colascione
2012-10-19  1:40     ` Stefan Monnier
2012-10-20  3:45     ` Daniel Colascione
2012-10-20 12:30       ` Ulrich Mueller

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=v5a9vm7vga.fsf@fencepost.gnu.org \
    --to=rgm@gnu.org \
    --cc=12647@debbugs.gnu.org \
    --cc=toby-dated-1351439087.748fe4@dr-qubit.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.