From: David Kastrup <dak@gnu.org>
To: "Károly Lőrentey" <karoly@lorentey.hu>
Cc: emacs-devel@gnu.org
Subject: Re: multi-tty branch created
Date: Wed, 16 May 2007 17:34:13 +0200 [thread overview]
Message-ID: <86sl9wpytm.fsf@lola.quinscape.zz> (raw)
In-Reply-To: <f2f6hu$598$1@sea.gmane.org> ("Károly Lőrentey"'s message of "Wed\, 16 May 2007 17\:04\:25 +0200")
Károly Lőrentey <karoly@lorentey.hu> writes:
> Juanma Barranquero wrote:
>
>> Then it should continue to be done in the multi-tty branch until
>> merging it to the trunk does not represent a regression.
>
> I don't believe it represents a regression in its current form. As
> we know, Windows support is broken, but thankfully people are
> working on that now.
Uh what? In its current form, Windows support is broken but it does
not represent a regression? We must have different definitions of
"regression" then. Actually, at the moment emacsclient does not
compile which _is_ a regression. And such regressions will
_certainly_ occur until we get this to compile on all platforms.
make[1]: Entering directory `/rep/emacs-build/lib-src'
gcc -D_BSD_SOURCE -DHAVE_CONFIG_H -I. -I../src -I/rep/emacs/lib-src -I/rep/emacs/lib-src/../src -Wl,-znocombreloc -D_BSD_SOURCE -g -O2 -fno-crossjumping /rep/emacs/lib-src/emacsclient.c -DVERSION="\"23.0.51\"" -lc -o emacsclient
/rep/emacs/lib-src/emacsclient.c: In function ‘handle_sigcont’:
/rep/emacs/lib-src/emacsclient.c:960: error: ‘s’ undeclared (first use in this function)
/rep/emacs/lib-src/emacsclient.c:960: error: (Each undeclared identifier is reported only once
/rep/emacs/lib-src/emacsclient.c:960: error: for each function it appears in.)
/rep/emacs/lib-src/emacsclient.c: In function ‘handle_sigtstp’:
/rep/emacs/lib-src/emacsclient.c:984: error: ‘s’ undeclared (first use in this function)
make[1]: *** [emacsclient] Error 1
make[1]: Leaving directory `/rep/emacs-build/lib-src'
make: *** [lib-src] Error 2
> If there is a good chance that Emacs 23 will be released without
> multi-tty, then of course things are different. I think people
> should look at the code and report if it is basically sound or not.
> If not, then it's better to have that decided early than to waste
> more time developing it.
My first impression is the following: there are serious design and use
problems that need to be addressed before finalizing the merge. It
would appear, however, that multi-tty went through several stages in
the course of its development that were rather close to what would
constitute a robust design. So scrapping the branch and
reimplementing from scratch would appear quite nonsensical, and the
design discussion will also be helped by you having actual experience
with several approaches.
> You can also decide to keep a subset of multi-tty changes (C-level
> changes, Lisp interface, emacsclient improvements, environment
> implementation, Lisp package adaptations, etc.) and
> discard/reimplement the rest. For example, David has already
> expressed his dissatisfaction with the environment implementation.
I am still putting together some more detailed critique. Note that
getting me satisfied is not the same as getting everybody else
satisfied, but of course addressing points raised by me on the list
will help your case also with other people choosing to mostly listen
at the moment.
It is quite too early to predict how things will progress. My
impression is that the multi-tty branch is on a good track (and it
quite helps that you are not trying to block contributions and
criticism). That's not enough to put forward a plan, but it's a good
sign.
--
David Kastrup
next prev parent reply other threads:[~2007-05-16 15:34 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-13 13:31 multi-tty branch created Miles Bader
2007-05-13 14:50 ` David Kastrup
2007-05-13 16:12 ` Miles Bader
2007-05-13 16:14 ` David Kastrup
2007-05-13 16:13 ` David Kastrup
2007-05-13 19:28 ` Robert J. Chassell
2007-05-16 13:24 ` Károly Lo"rentey
2007-05-16 13:46 ` Juanma Barranquero
2007-05-16 15:04 ` Károly Lőrentey
2007-05-16 15:34 ` David Kastrup [this message]
2007-05-16 16:11 ` Károly Lőrentey
2007-05-16 21:17 ` Jason Rumney
2007-05-16 16:20 ` Dan Nicolaescu
2007-05-16 17:39 ` David Kastrup
2007-05-16 20:48 ` Károly Lőrentey
2007-05-17 14:02 ` Stefan Monnier
2007-05-17 16:04 ` Károly Lo"rentey
2007-05-17 23:24 ` Stefan Monnier
2007-05-18 18:02 ` Károly Lo"rentey
2007-05-19 14:07 ` Stefan Monnier
2007-05-13 20:07 ` Jason Rumney
2007-05-14 10:21 ` Karoly Lorentey
2007-05-14 11:55 ` Jason Rumney
2007-05-15 13:03 ` Karoly Lorentey
2007-05-15 22:05 ` Ken Raeburn
2007-05-14 6:00 ` Manoj Srivastava
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=86sl9wpytm.fsf@lola.quinscape.zz \
--to=dak@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=karoly@lorentey.hu \
/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.