Bug 74592 - Port to mozjs24
Summary: Port to mozjs24
Status: RESOLVED FIXED
Alias: None
Product: PolicyKit
Classification: Unclassified
Component: daemon (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: David Zeuthen (not reading bugmail)
QA Contact: David Zeuthen (not reading bugmail)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-05 23:38 UTC by Adam Williamson
Modified: 2017-02-09 19:23 UTC (History)
14 users (show)

See Also:
i915 platform:
i915 features:

Attachments

Description Adam Williamson 2014-02-05 23:38:54 UTC 
Everything else has gone to mozjs24, it looks like, but polkit still doesn't seem to be compatible with it. Is the port planned?
Comment 1 Jürg Billeter 2014-03-26 07:06:16 UTC 
mozjs24 no longer has a C API, only C++ is supported.
Comment 2 Laurent Bigonville 2014-04-28 14:07:22 UTC 
According to http://lists.freedesktop.org/archives/polkit-devel/2014-April/000401.html this is in progress
Comment 3 Laurent Bigonville 2014-09-07 12:06:56 UTC 
Hello,

Any progress on this?
Comment 4 Bastien Nocera 2015-02-06 10:51:00 UTC 
Kalev, any news on this?
Comment 5 Frank Schaefer 2015-03-10 07:02:40 UTC 
Ok guys, where do we stand on this?  This has been in limbo for over a year.

On a side note, might I suggest we switch to some other JavaScript engine?  I'm trying to get this going on MIPS64, and so far no version of SpiderMonkey seems up to it (even the 31.2 RC).  mozjs17 doesn't build for 64-bit in any form without a lot of patching, with the patch effort increasing for every architecture.

A lot of this grief could probably be avoided just by switching to a more portable alternative.  I'm currently evaluating duktape (http://duktape.org), which so far builds and runs without complaint on MIPS64.
Comment 6 Hussam Al-Tayeb 2015-12-22 00:58:48 UTC 
Any news on this? MozJS17 is getting ancient and the build system needs a patch to use new Perl.
Comment 7 Jeremy Linton 2016-08-30 16:21:31 UTC 
A patch has been posted here:

https://lists.freedesktop.org/archives/polkit-devel/2016-August/000503.html
Comment 8 Hussam Al-Tayeb 2016-08-30 17:02:46 UTC 
(In reply to Jeremy Linton from comment #7)
> A patch has been posted here:
> 
> https://lists.freedesktop.org/archives/polkit-devel/2016-August/000503.html

Hi :) would it be possible to attach patches here for people not subscribed to the mailing list? Thank you :)
Comment 10 Miloslav Trmac 2017-02-09 19:23:45 UTC 
This has now been committed to master. Thanks, Jeremy!

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.