unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: joakim@verona.se
Cc: Emacs developers <emacs-devel@gnu.org>
Subject: Re: merging the xwidget_mvp branch
Date: Wed, 04 Nov 2015 08:52:25 -0500	[thread overview]
Message-ID: <jwvmvutlw1b.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <m2y4ee92fo.fsf@newartisans.com> (John Wiegley's message of "Tue,  03 Nov 2015 16:51:39 -0500")

>> The patch is large I suppose. It mostly affects the redisplay system.
> It's unfortunate that that is currently one of our most complex subsystems,
> where we've had the greatest difficulty finding other contributors.

That's an argument in favor of accepting this patch ;-)

>> I have maintained it in its own branch for a couple of years.  I can
>> continue doing that in master, or in its own branch indefinitely.
> I'm personally inclined to not want this feature, unless there is
> a great deal of desire for it from others; so I'll wait for them
> chime in.

I think it could open up very interesting new developments.

The embedded web-browser example is also a sexy proof-of-concept which
is likely to attract new blood (maybe not new users directly, but rather
new developers).

> I hate to say no given how much effort you've put into it, but there
> needs to be enough value to offset the additional technical debt we'd
> be accruing.

We've already been through several rounds of reviewing, so I'm sure he'd
be very disappointed if it doesn't get the code merged after all
those efforts.

And IIRC the technical debt seemed quite OK: the interaction between
xwidget's own code (the part that deals with the xwidget API) and
Emacs's own code is very similar to the interaction between Emacs and
the various image libraries.


        Stefan



  parent reply	other threads:[~2015-11-04 13:52 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-03 21:18 merging the xwidget_mvp branch joakim
2015-11-03 21:23 ` John Wiegley
2015-11-03 21:37   ` joakim
2015-11-03 21:51     ` John Wiegley
2015-11-04 11:01       ` Nicolas Petton
2015-11-04 16:37         ` raman
2015-11-04 17:22           ` Michael Abrahams
2015-11-04 13:52       ` Stefan Monnier [this message]
2015-11-06 21:21         ` John Wiegley
2015-11-08 16:34           ` joakim
2015-11-08 17:13             ` Eli Zaretskii
2015-11-09 19:53               ` Join us on emacs-tangents! (was: merging the xwidget_mvp branch) John Wiegley
2015-11-09 20:08                 ` Join us on emacs-tangents! Rasmus
2015-11-09 21:31                   ` John Wiegley
2015-11-09 20:17                 ` Dmitry Gutov
2015-11-09 21:30                   ` John Wiegley
2015-11-09 23:58                 ` Join us on emacs-tangents! (was: merging the xwidget_mvp branch) Aaron Ecay
2015-11-10 10:47                   ` Join us on emacs-tangents! Jose E. Marchesi
2015-11-11 10:33           ` merging the xwidget_mvp branch joakim
2015-11-11 16:32             ` John Wiegley
2015-11-04 15:29       ` Eli Zaretskii
2015-11-05 12:13 ` Nicolas Petton
2015-11-05 12:19   ` joakim
2015-11-08 16:32   ` joakim
2015-11-09  9:17     ` Nicolas Petton

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=jwvmvutlw1b.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=emacs-devel@gnu.org \
    --cc=joakim@verona.se \
    /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).