edbrowse-dev - development list for edbrowse
 help / color / mirror / Atom feed
From: Adam Thompson <arthompson1990@gmail.com>
To: Karl Dahlke <eklhad@comcast.net>
Cc: Edbrowse-dev@lists.the-brannons.com
Subject: Re: [Edbrowse-dev] Error Legs
Date: Sun, 9 Feb 2014 15:13:35 +0000	[thread overview]
Message-ID: <20140209151335.GD11542@toaster.adamthompson.me.uk> (raw)
In-Reply-To: <20140109091846.eklhad@comcast.net>

On Sun, Feb 09, 2014 at 09:18:46AM -0500, Karl Dahlke wrote:
> > I'd still prefer it if it just killed the offending context as well
> 
> I agree, and I might be the one to work on this -
> but that would be one of those major changes that I think we said we would
> wait on until our work was stable.
> I keep coming back round to the fact that if we can't get this to work
> with moz js 24 libraries as distributed, then edbrowse is dead anyways,
> except perhaps on our three machines.
> If it can't be distributed then I don't know how much time I should spend on it.
> I'm not trying to be Debbie Downer, just practical.

Hmmm, it breaks with *one* distributed package of mozjs 24,
and not even a stable one. The fact is that it works with mozjs as distributed
by mozilla, which tells me that it works well enough for distribution.

> We should be doing what we can to get it to function in the debian world.
> If that is both in process and/or on hold,
> or if one of you is taking that on, then I could work on error legs in parallel,
> and would be willing to do that, as it is a systemic change throughout.

You're right that we *need* to work out how to get this to function with the
debian package, and I think both Chris and I are working on this.
I think it'd be helped greatly by improved error handling though,
as it *may* be that the bug with debian is a reflection of something wrong in
edbrowse's use of the lib which is showing up in other strange problems when
run against hand-compiled versions.
At the end of the day, if we know the browser is properly stable with one
version of the lib it'll make it easier to find where the instability is with
the debian package. At the moment, I 'm fairly sure that the bug is something
to do with not defining something or something missing in our initialisation or
the Debian package. However, when the thing segfaults for seemingly no reason
sometimes, there's always the suspician that something else is going on.

> I just keep looking up and seeing the sword of damocles overhead.

I think this is being a bit over-dramatic.
The cleanup for the new release and the stabilisation process is painful,
but certainly not fatal for the project.
I'm not surprised about this, there are things which need to be fixed and the
changes in the mozilla api are signifficant.
Debian are still packaging edbrowse so it's not gone from their repo and is
still in stable. As long as we can show them there's development this will
hopefully be ok. I think we need to make the release relatively soon,
but I think waiting for error handling,
whilst perhaps pushing the concept of code cleanup a bit, makes sense.
After all, what'll kill the project is a bug-filled release, not a delayed one.
If people view edbrowse as a segfault wating to happen then they'll stop using it.
If they have to wait a little longer for a major new release then they'll
probably be fine with that as long as they know why.

Cheers,
Adam.

  reply	other threads:[~2014-02-09 15:14 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-09 14:18 Karl Dahlke
2014-02-09 15:13 ` Adam Thompson [this message]
  -- strict thread matches above, loose matches on Subject: below --
2014-02-09 14:48 Karl Dahlke
2014-02-09 15:21 ` Adam Thompson
2014-02-07 20:06 Karl Dahlke
2014-02-07 19:37 Karl Dahlke
2014-02-07 19:55 ` Chris Brannon
2014-02-07 17:57 Karl Dahlke
2014-02-07 18:23 ` Chris Brannon
2014-02-09 10:45   ` Adam Thompson
2014-02-09 11:10     ` Adam Thompson
2014-02-07 14:01 Karl Dahlke
2014-02-07 15:05 ` Chris Brannon

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=20140209151335.GD11542@toaster.adamthompson.me.uk \
    --to=arthompson1990@gmail.com \
    --cc=Edbrowse-dev@lists.the-brannons.com \
    --cc=eklhad@comcast.net \
    /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.
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 NNTP newsgroup(s).