mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Ladislav Michl <ladis@linux-mips.org>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] js: return of spidermonkey
Date: Fri, 29 Jul 2016 11:16:03 +0200	[thread overview]
Message-ID: <20160729091603.GA25108@localhost.localdomain> (raw)
In-Reply-To: <20160729071156.GJ31919@pengutronix.de>

On Fri, Jul 29, 2016 at 09:11:56AM +0200, Michael Olbrich wrote:
> Hi,
> 
> On Fri, Jul 29, 2016 at 02:22:00AM +0200, Ladislav Michl wrote:
> > Commit a33ab9d2dee7c0cc727279c83e6eaf50eaba5e9e removed js with explanation
> > "It was in staging for a year and nobody cared."
> > A patch to bring it back into life was sent in February:
> > https://www.mail-archive.com/ptxdist@pengutronix.de/msg10388.html
> 
> Indeed. Looks like it missed that one. I was rather busy at the time.
> 
> > (gmane is currently down) Meanwhile SpiderMonkey 45 was released.
> 
> I'd be happy to look at a new patch. I've taken a quick look at the old
> patch and the only thing that really stands out is a missing patch header
> for the second patch.

Indeed, fortunately those patches was digged from git and are already
present in version 45.

> > Do we want it in PTXdist? The only package depending on js is polkit,
> > but that was already solved by upgrading to 0.105 - the last version
> > not depending on js.
> 
> If someone needs js for something and provides a good patch, then I'll
> apply it. I don't think updating polkit to a version that needs it, is a
> good idea. Maybe as a different package if someone needs it.

I just give a try to version 45 and despite huge amount of memory needed
to compile that beast it all looks fine. However providing package
which is not quite usefull alone does not seem like a good idea. Originaly
I did it for udisks2, but as it (at version 2.1.7) requires polkit 0.102
or newer, that dependency is already satisfied. So I'll leave it in my
tree with polkit 0.113 for a while and resubmit just in case someone
really needs it.

	ladis (unhappy where linux userspace is heading...)

PS. I'll probably forget all this adventure and try this instead:
https://github.com/LemonBoy/ldm

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

  parent reply	other threads:[~2016-07-29  9:16 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-29  0:22 Ladislav Michl
2016-07-29  6:14 ` Robert Schwebel
2016-07-29  7:11 ` Michael Olbrich
2016-07-29  8:44   ` Bruno Thomsen
2016-07-29  9:16   ` Ladislav Michl [this message]
2016-07-31 20:15     ` [ptxdist] [PATCH] js: mozjs-45.0.2 Ladislav Michl

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=20160729091603.GA25108@localhost.localdomain \
    --to=ladis@linux-mips.org \
    --cc=ptxdist@pengutronix.de \
    /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